use tramli::{FlowDefinition, FlowState, TransitionType};
use crate::api::PluginReport;
pub type FlowPolicy<S> = Box<dyn Fn(&FlowDefinition<S>, &mut PluginReport) + Send + Sync>;
pub fn default_policies<S: FlowState>() -> Vec<FlowPolicy<S>> {
vec![
Box::new(warn_terminal_with_outgoing),
Box::new(warn_too_many_externals),
Box::new(warn_dead_produced_data),
Box::new(warn_overwide_processors),
]
}
fn warn_terminal_with_outgoing<S: FlowState>(def: &FlowDefinition<S>, report: &mut PluginReport) {
for state in def.terminal_states() {
if !def.transitions_from(*state).is_empty() {
report.warn("policy/terminal-outgoing", &format!("terminal state {:?} has outgoing transitions", state));
}
}
}
fn warn_too_many_externals<S: FlowState>(def: &FlowDefinition<S>, report: &mut PluginReport) {
for state in S::all_states() {
let externals: Vec<_> = def.transitions_from(*state)
.into_iter()
.filter(|t| t.transition_type == TransitionType::External)
.collect();
if externals.len() > 3 {
report.warn("policy/external-count", &format!("state {:?} has {} external transitions", state, externals.len()));
}
}
}
fn warn_dead_produced_data<S: FlowState>(def: &FlowDefinition<S>, report: &mut PluginReport) {
let dead = def.data_flow_graph().dead_data();
for type_id in dead {
let name = def.data_flow_graph().type_name(&type_id);
report.warn("policy/dead-data", &format!("produced but never consumed: {}", name));
}
}
fn warn_overwide_processors<S: FlowState>(def: &FlowDefinition<S>, report: &mut PluginReport) {
for t in &def.transitions {
if let Some(ref p) = t.processor {
if p.produces().len() > 3 {
report.warn(
"policy/overwide-processor",
&format!("{} produces {} types; consider splitting it", p.name(), p.produces().len()),
);
}
}
}
}
pub struct PolicyLintPlugin<S: FlowState> {
policies: Vec<FlowPolicy<S>>,
}
impl<S: FlowState> PolicyLintPlugin<S> {
pub fn new(policies: Vec<FlowPolicy<S>>) -> Self {
Self { policies }
}
pub fn defaults() -> Self {
Self::new(default_policies())
}
pub fn analyze(&self, definition: &FlowDefinition<S>, report: &mut PluginReport) {
for policy in &self.policies {
policy(definition, report);
}
}
}