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(|loc| format!("`{}`", rel(&loc.path)))
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![
424 DuplicateLocation {
425 path: root.join("src/config.ts"),
426 line: 15,
427 col: 0,
428 },
429 DuplicateLocation {
430 path: root.join("src/types.ts"),
431 line: 30,
432 col: 0,
433 },
434 ],
435 });
436 r.type_only_dependencies.push(TypeOnlyDependency {
437 package_name: "zod".to_string(),
438 path: root.join("package.json"),
439 });
440 r.circular_dependencies.push(CircularDependency {
441 files: vec![root.join("src/a.ts"), root.join("src/b.ts")],
442 length: 2,
443 });
444
445 r
446 }
447
448 #[test]
449 fn markdown_empty_results_no_issues() {
450 let root = PathBuf::from("/project");
451 let results = AnalysisResults::default();
452 let md = build_markdown(&results, &root);
453 assert_eq!(md, "## Fallow: no issues found\n");
454 }
455
456 #[test]
457 fn markdown_contains_header_with_count() {
458 let root = PathBuf::from("/project");
459 let results = sample_results(&root);
460 let md = build_markdown(&results, &root);
461 assert!(md.starts_with(&format!(
462 "## Fallow: {} issues found\n",
463 results.total_issues()
464 )));
465 }
466
467 #[test]
468 fn markdown_contains_all_sections() {
469 let root = PathBuf::from("/project");
470 let results = sample_results(&root);
471 let md = build_markdown(&results, &root);
472
473 assert!(md.contains("### Unused files (1)"));
474 assert!(md.contains("### Unused exports (1)"));
475 assert!(md.contains("### Unused type exports (1)"));
476 assert!(md.contains("### Unused dependencies (1)"));
477 assert!(md.contains("### Unused devDependencies (1)"));
478 assert!(md.contains("### Unused enum members (1)"));
479 assert!(md.contains("### Unused class members (1)"));
480 assert!(md.contains("### Unresolved imports (1)"));
481 assert!(md.contains("### Unlisted dependencies (1)"));
482 assert!(md.contains("### Duplicate exports (1)"));
483 assert!(md.contains("### Type-only dependencies"));
484 assert!(md.contains("### Circular dependencies (1)"));
485 }
486
487 #[test]
488 fn markdown_unused_file_format() {
489 let root = PathBuf::from("/project");
490 let mut results = AnalysisResults::default();
491 results.unused_files.push(UnusedFile {
492 path: root.join("src/dead.ts"),
493 });
494 let md = build_markdown(&results, &root);
495 assert!(md.contains("- `src/dead.ts`"));
496 }
497
498 #[test]
499 fn markdown_unused_export_grouped_by_file() {
500 let root = PathBuf::from("/project");
501 let mut results = AnalysisResults::default();
502 results.unused_exports.push(UnusedExport {
503 path: root.join("src/utils.ts"),
504 export_name: "helperFn".to_string(),
505 is_type_only: false,
506 line: 10,
507 col: 4,
508 span_start: 120,
509 is_re_export: false,
510 });
511 let md = build_markdown(&results, &root);
512 assert!(md.contains("- `src/utils.ts`"));
513 assert!(md.contains(":10 `helperFn`"));
514 }
515
516 #[test]
517 fn markdown_re_export_tagged() {
518 let root = PathBuf::from("/project");
519 let mut results = AnalysisResults::default();
520 results.unused_exports.push(UnusedExport {
521 path: root.join("src/index.ts"),
522 export_name: "reExported".to_string(),
523 is_type_only: false,
524 line: 1,
525 col: 0,
526 span_start: 0,
527 is_re_export: true,
528 });
529 let md = build_markdown(&results, &root);
530 assert!(md.contains("(re-export)"));
531 }
532
533 #[test]
534 fn markdown_unused_dep_format() {
535 let root = PathBuf::from("/project");
536 let mut results = AnalysisResults::default();
537 results.unused_dependencies.push(UnusedDependency {
538 package_name: "lodash".to_string(),
539 location: DependencyLocation::Dependencies,
540 path: root.join("package.json"),
541 });
542 let md = build_markdown(&results, &root);
543 assert!(md.contains("- `lodash`"));
544 }
545
546 #[test]
547 fn markdown_circular_dep_format() {
548 let root = PathBuf::from("/project");
549 let mut results = AnalysisResults::default();
550 results.circular_dependencies.push(CircularDependency {
551 files: vec![root.join("src/a.ts"), root.join("src/b.ts")],
552 length: 2,
553 });
554 let md = build_markdown(&results, &root);
555 assert!(md.contains("`src/a.ts`"));
556 assert!(md.contains("`src/b.ts`"));
557 assert!(md.contains("\u{2192}"));
558 }
559
560 #[test]
561 fn markdown_strips_root_prefix() {
562 let root = PathBuf::from("/project");
563 let mut results = AnalysisResults::default();
564 results.unused_files.push(UnusedFile {
565 path: PathBuf::from("/project/src/deep/nested/file.ts"),
566 });
567 let md = build_markdown(&results, &root);
568 assert!(md.contains("`src/deep/nested/file.ts`"));
569 assert!(!md.contains("/project/"));
570 }
571
572 #[test]
573 fn markdown_single_issue_no_plural() {
574 let root = PathBuf::from("/project");
575 let mut results = AnalysisResults::default();
576 results.unused_files.push(UnusedFile {
577 path: root.join("src/dead.ts"),
578 });
579 let md = build_markdown(&results, &root);
580 assert!(md.starts_with("## Fallow: 1 issue found\n"));
581 }
582
583 #[test]
584 fn markdown_type_only_dep_format() {
585 let root = PathBuf::from("/project");
586 let mut results = AnalysisResults::default();
587 results.type_only_dependencies.push(TypeOnlyDependency {
588 package_name: "zod".to_string(),
589 path: root.join("package.json"),
590 });
591 let md = build_markdown(&results, &root);
592 assert!(md.contains("### Type-only dependencies"));
593 assert!(md.contains("- `zod`"));
594 }
595
596 #[test]
597 fn markdown_escapes_backticks_in_export_names() {
598 let root = PathBuf::from("/project");
599 let mut results = AnalysisResults::default();
600 results.unused_exports.push(UnusedExport {
601 path: root.join("src/utils.ts"),
602 export_name: "foo`bar".to_string(),
603 is_type_only: false,
604 line: 1,
605 col: 0,
606 span_start: 0,
607 is_re_export: false,
608 });
609 let md = build_markdown(&results, &root);
610 assert!(md.contains("foo\\`bar"));
611 assert!(!md.contains("foo`bar`"));
612 }
613
614 #[test]
615 fn markdown_escapes_backticks_in_package_names() {
616 let root = PathBuf::from("/project");
617 let mut results = AnalysisResults::default();
618 results.unused_dependencies.push(UnusedDependency {
619 package_name: "pkg`name".to_string(),
620 location: DependencyLocation::Dependencies,
621 path: root.join("package.json"),
622 });
623 let md = build_markdown(&results, &root);
624 assert!(md.contains("pkg\\`name"));
625 }
626
627 #[test]
630 fn duplication_markdown_empty() {
631 let report = DuplicationReport::default();
632 let root = PathBuf::from("/project");
633 let md = build_duplication_markdown(&report, &root);
634 assert_eq!(md, "## Fallow: no code duplication found\n");
635 }
636
637 #[test]
638 fn duplication_markdown_contains_groups() {
639 let root = PathBuf::from("/project");
640 let report = DuplicationReport {
641 clone_groups: vec![CloneGroup {
642 instances: vec![
643 CloneInstance {
644 file: root.join("src/a.ts"),
645 start_line: 1,
646 end_line: 10,
647 start_col: 0,
648 end_col: 0,
649 fragment: String::new(),
650 },
651 CloneInstance {
652 file: root.join("src/b.ts"),
653 start_line: 5,
654 end_line: 14,
655 start_col: 0,
656 end_col: 0,
657 fragment: String::new(),
658 },
659 ],
660 token_count: 50,
661 line_count: 10,
662 }],
663 clone_families: vec![],
664 stats: DuplicationStats {
665 total_files: 10,
666 files_with_clones: 2,
667 total_lines: 500,
668 duplicated_lines: 20,
669 total_tokens: 2500,
670 duplicated_tokens: 100,
671 clone_groups: 1,
672 clone_instances: 2,
673 duplication_percentage: 4.0,
674 },
675 };
676 let md = build_duplication_markdown(&report, &root);
677 assert!(md.contains("**Clone group 1**"));
678 assert!(md.contains("`src/a.ts:1-10`"));
679 assert!(md.contains("`src/b.ts:5-14`"));
680 assert!(md.contains("4.0% duplication"));
681 }
682
683 #[test]
684 fn duplication_markdown_contains_families() {
685 let root = PathBuf::from("/project");
686 let report = DuplicationReport {
687 clone_groups: vec![CloneGroup {
688 instances: vec![CloneInstance {
689 file: root.join("src/a.ts"),
690 start_line: 1,
691 end_line: 5,
692 start_col: 0,
693 end_col: 0,
694 fragment: String::new(),
695 }],
696 token_count: 30,
697 line_count: 5,
698 }],
699 clone_families: vec![CloneFamily {
700 files: vec![root.join("src/a.ts"), root.join("src/b.ts")],
701 groups: vec![],
702 total_duplicated_lines: 20,
703 total_duplicated_tokens: 100,
704 suggestions: vec![RefactoringSuggestion {
705 kind: RefactoringKind::ExtractFunction,
706 description: "Extract shared utility function".to_string(),
707 estimated_savings: 15,
708 }],
709 }],
710 stats: DuplicationStats {
711 clone_groups: 1,
712 clone_instances: 1,
713 duplication_percentage: 2.0,
714 ..Default::default()
715 },
716 };
717 let md = build_duplication_markdown(&report, &root);
718 assert!(md.contains("### Clone Families"));
719 assert!(md.contains("**Family 1**"));
720 assert!(md.contains("Extract shared utility function"));
721 assert!(md.contains("~15 lines saved"));
722 }
723}