1use std::fmt::Write;
2use std::path::Path;
3
4use fallow_core::duplicates::DuplicationReport;
5use fallow_core::results::{AnalysisResults, UnusedExport, UnusedMember};
6
7use super::{normalize_uri, relative_path};
8
9fn escape_backticks(s: &str) -> String {
11 s.replace('`', "\\`")
12}
13
14pub(super) fn print_markdown(results: &AnalysisResults, root: &Path) {
15 println!("{}", build_markdown(results, root));
16}
17
18pub fn build_markdown(results: &AnalysisResults, root: &Path) -> String {
20 let rel = |p: &Path| {
21 escape_backticks(&normalize_uri(
22 &relative_path(p, root).display().to_string(),
23 ))
24 };
25
26 let total = results.total_issues();
27 let mut out = String::new();
28
29 if total == 0 {
30 out.push_str("## Fallow: no issues found\n");
31 return out;
32 }
33
34 let _ = write!(
35 out,
36 "## Fallow: {total} issue{} found\n\n",
37 if total == 1 { "" } else { "s" }
38 );
39
40 markdown_section(&mut out, &results.unused_files, "Unused files", |file| {
42 vec![format!("- `{}`", rel(&file.path))]
43 });
44
45 markdown_grouped_section(
47 &mut out,
48 &results.unused_exports,
49 "Unused exports",
50 root,
51 |e| e.path.as_path(),
52 format_export,
53 );
54
55 markdown_grouped_section(
57 &mut out,
58 &results.unused_types,
59 "Unused type exports",
60 root,
61 |e| e.path.as_path(),
62 format_export,
63 );
64
65 markdown_section(
67 &mut out,
68 &results.unused_dependencies,
69 "Unused dependencies",
70 |dep| format_dependency(&dep.package_name, &dep.path, root),
71 );
72
73 markdown_section(
75 &mut out,
76 &results.unused_dev_dependencies,
77 "Unused devDependencies",
78 |dep| format_dependency(&dep.package_name, &dep.path, root),
79 );
80
81 markdown_section(
83 &mut out,
84 &results.unused_optional_dependencies,
85 "Unused optionalDependencies",
86 |dep| format_dependency(&dep.package_name, &dep.path, root),
87 );
88
89 markdown_grouped_section(
91 &mut out,
92 &results.unused_enum_members,
93 "Unused enum members",
94 root,
95 |m| m.path.as_path(),
96 format_member,
97 );
98
99 markdown_grouped_section(
101 &mut out,
102 &results.unused_class_members,
103 "Unused class members",
104 root,
105 |m| m.path.as_path(),
106 format_member,
107 );
108
109 markdown_grouped_section(
111 &mut out,
112 &results.unresolved_imports,
113 "Unresolved imports",
114 root,
115 |i| i.path.as_path(),
116 |i| format!(":{} `{}`", i.line, escape_backticks(&i.specifier)),
117 );
118
119 markdown_section(
121 &mut out,
122 &results.unlisted_dependencies,
123 "Unlisted dependencies",
124 |dep| vec![format!("- `{}`", escape_backticks(&dep.package_name))],
125 );
126
127 markdown_section(
129 &mut out,
130 &results.duplicate_exports,
131 "Duplicate exports",
132 |dup| {
133 let locations: Vec<String> = dup
134 .locations
135 .iter()
136 .map(|p| format!("`{}`", rel(p)))
137 .collect();
138 vec![format!(
139 "- `{}` in {}",
140 escape_backticks(&dup.export_name),
141 locations.join(", ")
142 )]
143 },
144 );
145
146 markdown_section(
148 &mut out,
149 &results.type_only_dependencies,
150 "Type-only dependencies (consider moving to devDependencies)",
151 |dep| format_dependency(&dep.package_name, &dep.path, root),
152 );
153
154 markdown_section(
156 &mut out,
157 &results.circular_dependencies,
158 "Circular dependencies",
159 |cycle| {
160 let chain: Vec<String> = cycle.files.iter().map(|p| rel(p)).collect();
161 let mut display_chain = chain.clone();
162 if let Some(first) = chain.first() {
163 display_chain.push(first.clone());
164 }
165 vec![format!(
166 "- {}",
167 display_chain
168 .iter()
169 .map(|s| format!("`{s}`"))
170 .collect::<Vec<_>>()
171 .join(" \u{2192} ")
172 )]
173 },
174 );
175
176 out
177}
178
179fn format_export(e: &UnusedExport) -> String {
180 let re = if e.is_re_export { " (re-export)" } else { "" };
181 format!(":{} `{}`{re}", e.line, escape_backticks(&e.export_name))
182}
183
184fn format_member(m: &UnusedMember) -> String {
185 format!(
186 ":{} `{}.{}`",
187 m.line,
188 escape_backticks(&m.parent_name),
189 escape_backticks(&m.member_name)
190 )
191}
192
193fn format_dependency(dep_name: &str, pkg_path: &Path, root: &Path) -> Vec<String> {
194 let name = escape_backticks(dep_name);
195 let pkg_label = relative_path(pkg_path, root).display().to_string();
196 if pkg_label == "package.json" {
197 vec![format!("- `{name}`")]
198 } else {
199 let label = escape_backticks(&pkg_label);
200 vec![format!("- `{name}` ({label})")]
201 }
202}
203
204fn markdown_section<T>(
206 out: &mut String,
207 items: &[T],
208 title: &str,
209 format_lines: impl Fn(&T) -> Vec<String>,
210) {
211 if items.is_empty() {
212 return;
213 }
214 let _ = write!(out, "### {title} ({})\n\n", items.len());
215 for item in items {
216 for line in format_lines(item) {
217 out.push_str(&line);
218 out.push('\n');
219 }
220 }
221 out.push('\n');
222}
223
224fn markdown_grouped_section<'a, T>(
226 out: &mut String,
227 items: &'a [T],
228 title: &str,
229 root: &Path,
230 get_path: impl Fn(&'a T) -> &'a Path,
231 format_detail: impl Fn(&T) -> String,
232) {
233 if items.is_empty() {
234 return;
235 }
236 let _ = write!(out, "### {title} ({})\n\n", items.len());
237
238 let mut indices: Vec<usize> = (0..items.len()).collect();
239 indices.sort_by(|&a, &b| get_path(&items[a]).cmp(get_path(&items[b])));
240
241 let rel = |p: &Path| normalize_uri(&relative_path(p, root).display().to_string());
242 let mut last_file = String::new();
243 for &i in &indices {
244 let item = &items[i];
245 let file_str = rel(get_path(item));
246 if file_str != last_file {
247 let _ = writeln!(out, "- `{file_str}`");
248 last_file = file_str;
249 }
250 let _ = writeln!(out, " - {}", format_detail(item));
251 }
252 out.push('\n');
253}
254
255pub(super) fn print_duplication_markdown(report: &DuplicationReport, root: &Path) {
258 println!("{}", build_duplication_markdown(report, root));
259}
260
261pub fn build_duplication_markdown(report: &DuplicationReport, root: &Path) -> String {
263 let rel = |p: &Path| normalize_uri(&relative_path(p, root).display().to_string());
264
265 let mut out = String::new();
266
267 if report.clone_groups.is_empty() {
268 out.push_str("## Fallow: no code duplication found\n");
269 return out;
270 }
271
272 let stats = &report.stats;
273 let _ = write!(
274 out,
275 "## Fallow: {} clone group{} found ({:.1}% duplication)\n\n",
276 stats.clone_groups,
277 if stats.clone_groups == 1 { "" } else { "s" },
278 stats.duplication_percentage,
279 );
280
281 out.push_str("### Duplicates\n\n");
282 for (i, group) in report.clone_groups.iter().enumerate() {
283 let instance_count = group.instances.len();
284 let _ = write!(
285 out,
286 "**Clone group {}** ({} lines, {instance_count} instance{})\n\n",
287 i + 1,
288 group.line_count,
289 if instance_count == 1 { "" } else { "s" }
290 );
291 for instance in &group.instances {
292 let relative = rel(&instance.file);
293 let _ = writeln!(
294 out,
295 "- `{relative}:{}-{}`",
296 instance.start_line, instance.end_line
297 );
298 }
299 out.push('\n');
300 }
301
302 if !report.clone_families.is_empty() {
304 out.push_str("### Clone Families\n\n");
305 for (i, family) in report.clone_families.iter().enumerate() {
306 let file_names: Vec<_> = family.files.iter().map(|f| rel(f)).collect();
307 let _ = write!(
308 out,
309 "**Family {}** ({} group{}, {} lines across {})\n\n",
310 i + 1,
311 family.groups.len(),
312 if family.groups.len() == 1 { "" } else { "s" },
313 family.total_duplicated_lines,
314 file_names
315 .iter()
316 .map(|s| format!("`{s}`"))
317 .collect::<Vec<_>>()
318 .join(", "),
319 );
320 for suggestion in &family.suggestions {
321 let savings = if suggestion.estimated_savings > 0 {
322 format!(" (~{} lines saved)", suggestion.estimated_savings)
323 } else {
324 String::new()
325 };
326 let _ = writeln!(out, "- {}{savings}", suggestion.description);
327 }
328 out.push('\n');
329 }
330 }
331
332 let _ = writeln!(
334 out,
335 "**Summary:** {} duplicated lines ({:.1}%) across {} file{}",
336 stats.duplicated_lines,
337 stats.duplication_percentage,
338 stats.files_with_clones,
339 if stats.files_with_clones == 1 {
340 ""
341 } else {
342 "s"
343 },
344 );
345
346 out
347}
348
349#[cfg(test)]
350mod tests {
351 use super::*;
352 use fallow_core::duplicates::{
353 CloneFamily, CloneGroup, CloneInstance, DuplicationReport, DuplicationStats,
354 RefactoringKind, RefactoringSuggestion,
355 };
356 use fallow_core::extract::MemberKind;
357 use fallow_core::results::*;
358 use std::path::PathBuf;
359
360 fn sample_results(root: &Path) -> AnalysisResults {
362 let mut r = AnalysisResults::default();
363
364 r.unused_files.push(UnusedFile {
365 path: root.join("src/dead.ts"),
366 });
367 r.unused_exports.push(UnusedExport {
368 path: root.join("src/utils.ts"),
369 export_name: "helperFn".to_string(),
370 is_type_only: false,
371 line: 10,
372 col: 4,
373 span_start: 120,
374 is_re_export: false,
375 });
376 r.unused_types.push(UnusedExport {
377 path: root.join("src/types.ts"),
378 export_name: "OldType".to_string(),
379 is_type_only: true,
380 line: 5,
381 col: 0,
382 span_start: 60,
383 is_re_export: false,
384 });
385 r.unused_dependencies.push(UnusedDependency {
386 package_name: "lodash".to_string(),
387 location: DependencyLocation::Dependencies,
388 path: root.join("package.json"),
389 });
390 r.unused_dev_dependencies.push(UnusedDependency {
391 package_name: "jest".to_string(),
392 location: DependencyLocation::DevDependencies,
393 path: root.join("package.json"),
394 });
395 r.unused_enum_members.push(UnusedMember {
396 path: root.join("src/enums.ts"),
397 parent_name: "Status".to_string(),
398 member_name: "Deprecated".to_string(),
399 kind: MemberKind::EnumMember,
400 line: 8,
401 col: 2,
402 });
403 r.unused_class_members.push(UnusedMember {
404 path: root.join("src/service.ts"),
405 parent_name: "UserService".to_string(),
406 member_name: "legacyMethod".to_string(),
407 kind: MemberKind::ClassMethod,
408 line: 42,
409 col: 4,
410 });
411 r.unresolved_imports.push(UnresolvedImport {
412 path: root.join("src/app.ts"),
413 specifier: "./missing-module".to_string(),
414 line: 3,
415 col: 0,
416 });
417 r.unlisted_dependencies.push(UnlistedDependency {
418 package_name: "chalk".to_string(),
419 imported_from: vec![root.join("src/cli.ts")],
420 });
421 r.duplicate_exports.push(DuplicateExport {
422 export_name: "Config".to_string(),
423 locations: vec![root.join("src/config.ts"), root.join("src/types.ts")],
424 });
425 r.type_only_dependencies.push(TypeOnlyDependency {
426 package_name: "zod".to_string(),
427 path: root.join("package.json"),
428 });
429 r.circular_dependencies.push(CircularDependency {
430 files: vec![root.join("src/a.ts"), root.join("src/b.ts")],
431 length: 2,
432 });
433
434 r
435 }
436
437 #[test]
438 fn markdown_empty_results_no_issues() {
439 let root = PathBuf::from("/project");
440 let results = AnalysisResults::default();
441 let md = build_markdown(&results, &root);
442 assert_eq!(md, "## Fallow: no issues found\n");
443 }
444
445 #[test]
446 fn markdown_contains_header_with_count() {
447 let root = PathBuf::from("/project");
448 let results = sample_results(&root);
449 let md = build_markdown(&results, &root);
450 assert!(md.starts_with(&format!(
451 "## Fallow: {} issues found\n",
452 results.total_issues()
453 )));
454 }
455
456 #[test]
457 fn markdown_contains_all_sections() {
458 let root = PathBuf::from("/project");
459 let results = sample_results(&root);
460 let md = build_markdown(&results, &root);
461
462 assert!(md.contains("### Unused files (1)"));
463 assert!(md.contains("### Unused exports (1)"));
464 assert!(md.contains("### Unused type exports (1)"));
465 assert!(md.contains("### Unused dependencies (1)"));
466 assert!(md.contains("### Unused devDependencies (1)"));
467 assert!(md.contains("### Unused enum members (1)"));
468 assert!(md.contains("### Unused class members (1)"));
469 assert!(md.contains("### Unresolved imports (1)"));
470 assert!(md.contains("### Unlisted dependencies (1)"));
471 assert!(md.contains("### Duplicate exports (1)"));
472 assert!(md.contains("### Type-only dependencies"));
473 assert!(md.contains("### Circular dependencies (1)"));
474 }
475
476 #[test]
477 fn markdown_unused_file_format() {
478 let root = PathBuf::from("/project");
479 let mut results = AnalysisResults::default();
480 results.unused_files.push(UnusedFile {
481 path: root.join("src/dead.ts"),
482 });
483 let md = build_markdown(&results, &root);
484 assert!(md.contains("- `src/dead.ts`"));
485 }
486
487 #[test]
488 fn markdown_unused_export_grouped_by_file() {
489 let root = PathBuf::from("/project");
490 let mut results = AnalysisResults::default();
491 results.unused_exports.push(UnusedExport {
492 path: root.join("src/utils.ts"),
493 export_name: "helperFn".to_string(),
494 is_type_only: false,
495 line: 10,
496 col: 4,
497 span_start: 120,
498 is_re_export: false,
499 });
500 let md = build_markdown(&results, &root);
501 assert!(md.contains("- `src/utils.ts`"));
502 assert!(md.contains(":10 `helperFn`"));
503 }
504
505 #[test]
506 fn markdown_re_export_tagged() {
507 let root = PathBuf::from("/project");
508 let mut results = AnalysisResults::default();
509 results.unused_exports.push(UnusedExport {
510 path: root.join("src/index.ts"),
511 export_name: "reExported".to_string(),
512 is_type_only: false,
513 line: 1,
514 col: 0,
515 span_start: 0,
516 is_re_export: true,
517 });
518 let md = build_markdown(&results, &root);
519 assert!(md.contains("(re-export)"));
520 }
521
522 #[test]
523 fn markdown_unused_dep_format() {
524 let root = PathBuf::from("/project");
525 let mut results = AnalysisResults::default();
526 results.unused_dependencies.push(UnusedDependency {
527 package_name: "lodash".to_string(),
528 location: DependencyLocation::Dependencies,
529 path: root.join("package.json"),
530 });
531 let md = build_markdown(&results, &root);
532 assert!(md.contains("- `lodash`"));
533 }
534
535 #[test]
536 fn markdown_circular_dep_format() {
537 let root = PathBuf::from("/project");
538 let mut results = AnalysisResults::default();
539 results.circular_dependencies.push(CircularDependency {
540 files: vec![root.join("src/a.ts"), root.join("src/b.ts")],
541 length: 2,
542 });
543 let md = build_markdown(&results, &root);
544 assert!(md.contains("`src/a.ts`"));
545 assert!(md.contains("`src/b.ts`"));
546 assert!(md.contains("\u{2192}"));
547 }
548
549 #[test]
550 fn markdown_strips_root_prefix() {
551 let root = PathBuf::from("/project");
552 let mut results = AnalysisResults::default();
553 results.unused_files.push(UnusedFile {
554 path: PathBuf::from("/project/src/deep/nested/file.ts"),
555 });
556 let md = build_markdown(&results, &root);
557 assert!(md.contains("`src/deep/nested/file.ts`"));
558 assert!(!md.contains("/project/"));
559 }
560
561 #[test]
562 fn markdown_single_issue_no_plural() {
563 let root = PathBuf::from("/project");
564 let mut results = AnalysisResults::default();
565 results.unused_files.push(UnusedFile {
566 path: root.join("src/dead.ts"),
567 });
568 let md = build_markdown(&results, &root);
569 assert!(md.starts_with("## Fallow: 1 issue found\n"));
570 }
571
572 #[test]
573 fn markdown_type_only_dep_format() {
574 let root = PathBuf::from("/project");
575 let mut results = AnalysisResults::default();
576 results.type_only_dependencies.push(TypeOnlyDependency {
577 package_name: "zod".to_string(),
578 path: root.join("package.json"),
579 });
580 let md = build_markdown(&results, &root);
581 assert!(md.contains("### Type-only dependencies"));
582 assert!(md.contains("- `zod`"));
583 }
584
585 #[test]
586 fn markdown_escapes_backticks_in_export_names() {
587 let root = PathBuf::from("/project");
588 let mut results = AnalysisResults::default();
589 results.unused_exports.push(UnusedExport {
590 path: root.join("src/utils.ts"),
591 export_name: "foo`bar".to_string(),
592 is_type_only: false,
593 line: 1,
594 col: 0,
595 span_start: 0,
596 is_re_export: false,
597 });
598 let md = build_markdown(&results, &root);
599 assert!(md.contains("foo\\`bar"));
600 assert!(!md.contains("foo`bar`"));
601 }
602
603 #[test]
604 fn markdown_escapes_backticks_in_package_names() {
605 let root = PathBuf::from("/project");
606 let mut results = AnalysisResults::default();
607 results.unused_dependencies.push(UnusedDependency {
608 package_name: "pkg`name".to_string(),
609 location: DependencyLocation::Dependencies,
610 path: root.join("package.json"),
611 });
612 let md = build_markdown(&results, &root);
613 assert!(md.contains("pkg\\`name"));
614 }
615
616 #[test]
619 fn duplication_markdown_empty() {
620 let report = DuplicationReport::default();
621 let root = PathBuf::from("/project");
622 let md = build_duplication_markdown(&report, &root);
623 assert_eq!(md, "## Fallow: no code duplication found\n");
624 }
625
626 #[test]
627 fn duplication_markdown_contains_groups() {
628 let root = PathBuf::from("/project");
629 let report = DuplicationReport {
630 clone_groups: vec![CloneGroup {
631 instances: vec![
632 CloneInstance {
633 file: root.join("src/a.ts"),
634 start_line: 1,
635 end_line: 10,
636 start_col: 0,
637 end_col: 0,
638 fragment: String::new(),
639 },
640 CloneInstance {
641 file: root.join("src/b.ts"),
642 start_line: 5,
643 end_line: 14,
644 start_col: 0,
645 end_col: 0,
646 fragment: String::new(),
647 },
648 ],
649 token_count: 50,
650 line_count: 10,
651 }],
652 clone_families: vec![],
653 stats: DuplicationStats {
654 total_files: 10,
655 files_with_clones: 2,
656 total_lines: 500,
657 duplicated_lines: 20,
658 total_tokens: 2500,
659 duplicated_tokens: 100,
660 clone_groups: 1,
661 clone_instances: 2,
662 duplication_percentage: 4.0,
663 },
664 };
665 let md = build_duplication_markdown(&report, &root);
666 assert!(md.contains("**Clone group 1**"));
667 assert!(md.contains("`src/a.ts:1-10`"));
668 assert!(md.contains("`src/b.ts:5-14`"));
669 assert!(md.contains("4.0% duplication"));
670 }
671
672 #[test]
673 fn duplication_markdown_contains_families() {
674 let root = PathBuf::from("/project");
675 let report = DuplicationReport {
676 clone_groups: vec![CloneGroup {
677 instances: vec![CloneInstance {
678 file: root.join("src/a.ts"),
679 start_line: 1,
680 end_line: 5,
681 start_col: 0,
682 end_col: 0,
683 fragment: String::new(),
684 }],
685 token_count: 30,
686 line_count: 5,
687 }],
688 clone_families: vec![CloneFamily {
689 files: vec![root.join("src/a.ts"), root.join("src/b.ts")],
690 groups: vec![],
691 total_duplicated_lines: 20,
692 total_duplicated_tokens: 100,
693 suggestions: vec![RefactoringSuggestion {
694 kind: RefactoringKind::ExtractFunction,
695 description: "Extract shared utility function".to_string(),
696 estimated_savings: 15,
697 }],
698 }],
699 stats: DuplicationStats {
700 clone_groups: 1,
701 clone_instances: 1,
702 duplication_percentage: 2.0,
703 ..Default::default()
704 },
705 };
706 let md = build_duplication_markdown(&report, &root);
707 assert!(md.contains("### Clone Families"));
708 assert!(md.contains("**Family 1**"));
709 assert!(md.contains("Extract shared utility function"));
710 assert!(md.contains("~15 lines saved"));
711 }
712}