1use std::path::Path;
2use std::process::Command;
3
4use crate::analyses::EnrichedGraph;
5use crate::config::{Config, RuleConfig};
6use crate::diagnostic::Diagnostic;
7
8pub fn run_custom_rules(enriched: &EnrichedGraph, root: &Path, config: &Config) -> Vec<Diagnostic> {
20 let mut diagnostics = Vec::new();
21 let config_dir = config.config_dir.as_deref().unwrap_or(root);
22
23 for (rule_name, rule_config) in config.custom_rules() {
24 match run_one(rule_name, rule_config, enriched, root, config_dir) {
25 Ok(mut results) => diagnostics.append(&mut results),
26 Err(e) => {
27 eprintln!("warn: custom rule \"{rule_name}\" failed: {e}");
28 diagnostics.push(Diagnostic {
30 rule: rule_name.to_string(),
31 severity: rule_config.severity,
32 message: format!("custom rule failed: {e}"),
33 fix: Some(format!(
34 "custom rule \"{rule_name}\" failed to execute — check the command path and script"
35 )),
36 ..Default::default()
37 });
38 }
39 }
40 }
41
42 diagnostics
43}
44
45fn run_one(
46 rule_name: &str,
47 rule_config: &RuleConfig,
48 enriched: &EnrichedGraph,
49 root: &Path,
50 config_dir: &Path,
51) -> anyhow::Result<Vec<Diagnostic>> {
52 let command = rule_config
53 .command
54 .as_deref()
55 .ok_or_else(|| anyhow::anyhow!("rule \"{rule_name}\" has no command"))?;
56
57 let graph_json = build_enriched_json(enriched, rule_config.options.as_ref());
59
60 let parts: Vec<&str> = command.split_whitespace().collect();
62 if parts.is_empty() {
63 anyhow::bail!("empty command");
64 }
65
66 let cmd = if parts[0].starts_with("./") || parts[0].starts_with("../") {
68 config_dir.join(parts[0]).to_string_lossy().to_string()
69 } else {
70 parts[0].to_string()
71 };
72
73 let output = Command::new(&cmd)
74 .args(&parts[1..])
75 .current_dir(root)
76 .stdin(std::process::Stdio::piped())
77 .stdout(std::process::Stdio::piped())
78 .stderr(std::process::Stdio::piped())
79 .spawn()
80 .and_then(|mut child| {
81 use std::io::Write;
82 if let Some(ref mut stdin) = child.stdin {
83 let _ = stdin.write_all(graph_json.as_bytes());
84 }
85 child.wait_with_output()
86 })?;
87
88 if !output.status.success() {
89 let stderr = String::from_utf8_lossy(&output.stderr);
90 anyhow::bail!("exited with {}: {}", output.status, stderr.trim());
91 }
92
93 let stdout = String::from_utf8_lossy(&output.stdout);
94 let mut diagnostics = Vec::new();
95
96 for line in stdout.lines() {
97 let line = line.trim();
98 if line.is_empty() {
99 continue;
100 }
101
102 match serde_json::from_str::<CustomDiagnostic>(line) {
103 Ok(cd) => {
104 diagnostics.push(Diagnostic {
105 rule: rule_name.to_string(),
106 severity: rule_config.severity,
107 message: cd.message,
108 source: cd.source,
109 target: cd.target,
110 node: cd.node,
111 fix: cd.fix,
112 ..Default::default()
113 });
114 }
115 Err(e) => {
116 eprintln!("warn: custom rule \"{rule_name}\": failed to parse output line: {e}");
117 }
118 }
119 }
120
121 Ok(diagnostics)
122}
123
124#[derive(serde::Deserialize)]
125struct CustomDiagnostic {
126 message: String,
127 #[serde(default)]
128 source: Option<String>,
129 #[serde(default)]
130 target: Option<String>,
131 #[serde(default)]
132 node: Option<String>,
133 #[serde(default)]
134 fix: Option<String>,
135}
136
137fn build_enriched_json(enriched: &EnrichedGraph, options: Option<&toml::Value>) -> String {
142 let graph = &enriched.graph;
143
144 let mut nodes = serde_json::Map::new();
145 for (path, node) in &graph.nodes {
146 let mut meta = serde_json::Map::new();
147 meta.insert("type".into(), serde_json::json!(node.node_type));
148 if let Some(h) = &node.hash {
149 meta.insert("hash".into(), serde_json::json!(h));
150 }
151 nodes.insert(path.clone(), serde_json::json!({ "metadata": meta }));
152 }
153
154 let edges: Vec<serde_json::Value> = graph
155 .edges
156 .iter()
157 .filter(|e| graph.nodes.contains_key(&e.target))
158 .map(|e| {
159 let mut edge = serde_json::json!({
160 "source": e.source,
161 "target": e.target,
162 "parser": e.parser,
163 });
164 if let Some(ref r) = e.link {
165 edge["link"] = serde_json::json!(r);
166 }
167 edge
168 })
169 .collect();
170
171 let analyses = serde_json::json!({
172 "betweenness": enriched.betweenness,
173 "bridges": enriched.bridges,
174 "change_propagation": enriched.change_propagation,
175 "connected_components": enriched.connected_components,
176 "degree": enriched.degree,
177 "depth": enriched.depth,
178 "graph_boundaries": enriched.graph_boundaries,
179 "graph_stats": enriched.graph_stats,
180 "impact_radius": enriched.impact_radius,
181 "pagerank": enriched.pagerank,
182 "scc": enriched.scc,
183 "transitive_reduction": enriched.transitive_reduction,
184 });
185
186 let output = serde_json::json!({
187 "graph": {
188 "directed": true,
189 "nodes": nodes,
190 "edges": edges,
191 "analyses": analyses,
192 },
193 "options": options.unwrap_or(&toml::Value::Table(Default::default())),
194 });
195
196 serde_json::to_string(&output).unwrap()
197}
198
199#[cfg(test)]
200mod tests {
201 use super::*;
202 use crate::analyses::enrich_graph;
203 use crate::graph::{Edge, Graph, Node, NodeType};
204 use std::collections::HashMap;
205 use std::fs;
206 use tempfile::TempDir;
207
208 fn make_enriched(dir: &Path) -> EnrichedGraph {
209 let mut g = Graph::new();
210 g.add_node(Node {
211 path: "index.md".into(),
212 node_type: NodeType::File,
213 hash: Some("b3:aaa".into()),
214 graph: None,
215 is_graph: false,
216 metadata: HashMap::new(),
217 });
218 g.add_node(Node {
219 path: "setup.md".into(),
220 node_type: NodeType::File,
221 hash: Some("b3:bbb".into()),
222 graph: None,
223 is_graph: false,
224 metadata: HashMap::new(),
225 });
226 g.add_edge(Edge {
227 source: "index.md".into(),
228 target: "setup.md".into(),
229 link: None,
230 parser: "markdown".into(),
231 });
232 let config = crate::config::Config {
233 include: vec!["*.md".into()],
234 exclude: vec![],
235 interface: None,
236 parsers: std::collections::HashMap::new(),
237 rules: std::collections::HashMap::new(),
238 config_dir: None,
239 };
240 enrich_graph(g, dir, &config, None)
241 }
242
243 #[test]
244 fn runs_custom_script() {
245 let dir = TempDir::new().unwrap();
246
247 let script = dir.path().join("my-rule.sh");
249 fs::write(
250 &script,
251 "#!/bin/sh\necho '{\"message\": \"custom issue\", \"node\": \"index.md\", \"fix\": \"do something\"}'\n",
252 )
253 .unwrap();
254
255 #[cfg(unix)]
256 {
257 use std::os::unix::fs::PermissionsExt;
258 fs::set_permissions(&script, fs::Permissions::from_mode(0o755)).unwrap();
259 }
260
261 let config = RuleConfig {
262 command: Some(script.to_string_lossy().to_string()),
263 severity: crate::config::RuleSeverity::Warn,
264 files: Vec::new(),
265 ignore: Vec::new(),
266 options: None,
267 files_compiled: None,
268 ignore_compiled: None,
269 };
270
271 let enriched = make_enriched(dir.path());
272 let diagnostics = run_one("my-rule", &config, &enriched, dir.path(), dir.path()).unwrap();
273
274 assert_eq!(diagnostics.len(), 1);
275 assert_eq!(diagnostics[0].rule, "my-rule");
276 assert_eq!(diagnostics[0].message, "custom issue");
277 assert_eq!(diagnostics[0].node.as_deref(), Some("index.md"));
278 assert_eq!(diagnostics[0].fix.as_deref(), Some("do something"));
279 }
280
281 #[test]
282 fn handles_failing_script() {
283 let dir = TempDir::new().unwrap();
284 let script = dir.path().join("bad-rule.sh");
285 fs::write(&script, "#!/bin/sh\nexit 1\n").unwrap();
286
287 #[cfg(unix)]
288 {
289 use std::os::unix::fs::PermissionsExt;
290 fs::set_permissions(&script, fs::Permissions::from_mode(0o755)).unwrap();
291 }
292
293 let config = RuleConfig {
294 command: Some(script.to_string_lossy().to_string()),
295 severity: crate::config::RuleSeverity::Warn,
296 files: Vec::new(),
297 ignore: Vec::new(),
298 options: None,
299 files_compiled: None,
300 ignore_compiled: None,
301 };
302
303 let enriched = make_enriched(dir.path());
304 let result = run_one("bad-rule", &config, &enriched, dir.path(), dir.path());
305 assert!(result.is_err());
306 }
307
308 #[test]
309 fn resolves_command_relative_to_config_dir() {
310 let dir = TempDir::new().unwrap();
311
312 let config_dir = dir.path();
314 let root = dir.path().join("docs");
315 fs::create_dir_all(&root).unwrap();
316
317 let scripts_dir = config_dir.join("scripts");
319 fs::create_dir_all(&scripts_dir).unwrap();
320 let script = scripts_dir.join("check.sh");
321 fs::write(
322 &script,
323 "#!/bin/sh\necho '{\"message\": \"found issue\", \"node\": \"index.md\"}'\n",
324 )
325 .unwrap();
326
327 #[cfg(unix)]
328 {
329 use std::os::unix::fs::PermissionsExt;
330 fs::set_permissions(&script, fs::Permissions::from_mode(0o755)).unwrap();
331 }
332
333 let config = RuleConfig {
334 command: Some("./scripts/check.sh".to_string()),
335 severity: crate::config::RuleSeverity::Warn,
336 files: Vec::new(),
337 ignore: Vec::new(),
338 options: None,
339 files_compiled: None,
340 ignore_compiled: None,
341 };
342
343 let enriched = make_enriched(dir.path());
344 let diagnostics = run_one("my-rule", &config, &enriched, &root, config_dir).unwrap();
346
347 assert_eq!(diagnostics.len(), 1);
348 assert_eq!(diagnostics[0].message, "found issue");
349 }
350
351 #[test]
352 fn passes_options_to_script() {
353 let dir = TempDir::new().unwrap();
354
355 let script = dir.path().join("options-rule.sh");
357 fs::write(
358 &script,
359 r#"#!/bin/sh
360INPUT=$(cat)
361# Check if options.threshold exists in the JSON
362HAS_OPTIONS=$(echo "$INPUT" | grep -c '"threshold"')
363if [ "$HAS_OPTIONS" -gt 0 ]; then
364 echo '{"message": "got options"}'
365else
366 echo '{"message": "no options"}'
367fi
368"#,
369 )
370 .unwrap();
371
372 #[cfg(unix)]
373 {
374 use std::os::unix::fs::PermissionsExt;
375 fs::set_permissions(&script, fs::Permissions::from_mode(0o755)).unwrap();
376 }
377
378 let options: toml::Value = toml::from_str("threshold = 5").unwrap();
379 let config = RuleConfig {
380 command: Some(script.to_string_lossy().to_string()),
381 severity: crate::config::RuleSeverity::Warn,
382 files: Vec::new(),
383 ignore: Vec::new(),
384 options: Some(options),
385 files_compiled: None,
386 ignore_compiled: None,
387 };
388
389 let enriched = make_enriched(dir.path());
390 let diagnostics =
391 run_one("options-rule", &config, &enriched, dir.path(), dir.path()).unwrap();
392
393 assert_eq!(diagnostics.len(), 1);
394 assert_eq!(diagnostics[0].message, "got options");
395 }
396
397 #[test]
398 fn includes_analyses_in_graph_json() {
399 let dir = TempDir::new().unwrap();
400
401 let script = dir.path().join("analyses-rule.sh");
403 fs::write(
404 &script,
405 r#"#!/bin/sh
406INPUT=$(cat)
407HAS_ANALYSES=$(echo "$INPUT" | grep -c '"analyses"')
408if [ "$HAS_ANALYSES" -gt 0 ]; then
409 echo '{"message": "has analyses"}'
410else
411 echo '{"message": "no analyses"}'
412fi
413"#,
414 )
415 .unwrap();
416
417 #[cfg(unix)]
418 {
419 use std::os::unix::fs::PermissionsExt;
420 fs::set_permissions(&script, fs::Permissions::from_mode(0o755)).unwrap();
421 }
422
423 let config = RuleConfig {
424 command: Some(script.to_string_lossy().to_string()),
425 severity: crate::config::RuleSeverity::Warn,
426 files: Vec::new(),
427 ignore: Vec::new(),
428 options: None,
429 files_compiled: None,
430 ignore_compiled: None,
431 };
432
433 let enriched = make_enriched(dir.path());
434 let diagnostics =
435 run_one("analyses-rule", &config, &enriched, dir.path(), dir.path()).unwrap();
436
437 assert_eq!(diagnostics.len(), 1);
438 assert_eq!(diagnostics[0].message, "has analyses");
439 }
440}