Skip to main content

fallow_core/duplicates/
mod.rs

1//! Code duplication / clone detection module.
2//!
3//! This module implements suffix array + LCP based clone detection
4//! for TypeScript/JavaScript source files. It supports multiple detection
5//! modes from strict (exact matches only) to semantic (structure-aware
6//! matching that ignores identifier names and literal values).
7
8pub mod detect;
9pub mod families;
10pub mod normalize;
11pub mod token_types;
12mod token_visitor;
13pub mod tokenize;
14pub(crate) mod types;
15
16use rustc_hash::FxHashMap;
17use std::path::{Path, PathBuf};
18
19use globset::{Glob, GlobSet, GlobSetBuilder};
20use rayon::prelude::*;
21
22use detect::CloneDetector;
23use normalize::normalize_and_hash_resolved;
24use tokenize::{tokenize_file, tokenize_file_cross_language};
25pub use types::{
26    CloneFamily, CloneGroup, CloneInstance, DetectionMode, DuplicatesConfig, DuplicationReport,
27    DuplicationStats, RefactoringKind, RefactoringSuggestion,
28};
29
30use crate::discover::{self, DiscoveredFile};
31use crate::suppress::{self, IssueKind, Suppression};
32
33/// Run duplication detection on the given files.
34///
35/// This is the main entry point for the duplication analysis. It:
36/// 1. Reads and tokenizes all source files in parallel
37/// 2. Normalizes tokens according to the detection mode
38/// 3. Runs suffix array + LCP clone detection
39/// 4. Groups clone instances into families with refactoring suggestions
40/// 5. Applies inline suppression filters
41pub fn find_duplicates(
42    root: &Path,
43    files: &[DiscoveredFile],
44    config: &DuplicatesConfig,
45) -> DuplicationReport {
46    let _span = tracing::info_span!("find_duplicates").entered();
47
48    // Build extra ignore patterns for duplication analysis
49    let extra_ignores = build_ignore_set(&config.ignore);
50
51    // Resolve normalization: mode defaults + user overrides
52    let normalization =
53        fallow_config::ResolvedNormalization::resolve(config.mode, &config.normalization);
54
55    let strip_types = config.cross_language;
56
57    // Step 1 & 2: Tokenize and normalize all files in parallel, also parse suppressions
58    let file_data: Vec<(
59        PathBuf,
60        Vec<normalize::HashedToken>,
61        tokenize::FileTokens,
62        Vec<Suppression>,
63    )> = files
64        .par_iter()
65        .filter_map(|file| {
66            // Apply extra ignore patterns
67            let relative = file.path.strip_prefix(root).unwrap_or(&file.path);
68            if let Some(ref ignores) = extra_ignores
69                && ignores.is_match(relative)
70            {
71                return None;
72            }
73
74            // Read the file
75            let source = std::fs::read_to_string(&file.path).ok()?;
76
77            // Parse inline suppression comments
78            let suppressions = suppress::parse_suppressions_from_source(&source);
79
80            // Check for file-wide code-duplication suppression
81            if suppress::is_file_suppressed(&suppressions, IssueKind::CodeDuplication) {
82                return None;
83            }
84
85            // Tokenize (with optional type stripping for cross-language detection)
86            let file_tokens = if strip_types {
87                tokenize_file_cross_language(&file.path, &source, true)
88            } else {
89                tokenize_file(&file.path, &source)
90            };
91            if file_tokens.tokens.is_empty() {
92                return None;
93            }
94
95            // Normalize and hash using resolved normalization flags
96            let hashed = normalize_and_hash_resolved(&file_tokens.tokens, normalization);
97            if hashed.len() < config.min_tokens {
98                return None;
99            }
100
101            Some((file.path.clone(), hashed, file_tokens, suppressions))
102        })
103        .collect();
104
105    tracing::info!(
106        files = file_data.len(),
107        "tokenized files for duplication analysis"
108    );
109
110    // Collect per-file suppressions for line-level filtering
111    let suppressions_by_file: FxHashMap<PathBuf, Vec<Suppression>> = file_data
112        .iter()
113        .filter(|(_, _, _, supps)| !supps.is_empty())
114        .map(|(path, _, _, supps)| (path.clone(), supps.clone()))
115        .collect();
116
117    // Strip suppressions from the data passed to the detector
118    let detector_data: Vec<(PathBuf, Vec<normalize::HashedToken>, tokenize::FileTokens)> =
119        file_data
120            .into_iter()
121            .map(|(path, hashed, tokens, _)| (path, hashed, tokens))
122            .collect();
123
124    // Step 3 & 4: Detect clones
125    let detector = CloneDetector::new(config.min_tokens, config.min_lines, config.skip_local);
126    let mut report = detector.detect(detector_data);
127
128    // Step 5: Apply line-level suppressions
129    if !suppressions_by_file.is_empty() {
130        apply_line_suppressions(&mut report, &suppressions_by_file);
131    }
132
133    // Step 6: Group into families with refactoring suggestions
134    report.clone_families = families::group_into_families(&report.clone_groups, root);
135
136    // Sort all result arrays for deterministic output ordering.
137    // Parallel tokenization (par_iter) doesn't guarantee collection order.
138    report.sort();
139
140    report
141}
142
143/// Filter out clone instances that are suppressed by line-level comments.
144#[expect(
145    clippy::cast_possible_truncation,
146    reason = "line numbers are bounded by source size"
147)]
148fn apply_line_suppressions(
149    report: &mut DuplicationReport,
150    suppressions_by_file: &FxHashMap<PathBuf, Vec<Suppression>>,
151) {
152    report.clone_groups.retain_mut(|group| {
153        group.instances.retain(|instance| {
154            if let Some(supps) = suppressions_by_file.get(&instance.file) {
155                // Check if any line in the instance range is suppressed
156                for line in instance.start_line..=instance.end_line {
157                    if suppress::is_suppressed(supps, line as u32, IssueKind::CodeDuplication) {
158                        return false;
159                    }
160                }
161            }
162            true
163        });
164        // Keep group only if it still has 2+ instances
165        group.instances.len() >= 2
166    });
167}
168
169/// Run duplication detection on a project directory using auto-discovered files.
170///
171/// This is a convenience function that handles file discovery internally.
172#[must_use]
173pub fn find_duplicates_in_project(root: &Path, config: &DuplicatesConfig) -> DuplicationReport {
174    let resolved = crate::default_config(root);
175    let files = discover::discover_files(&resolved);
176    find_duplicates(root, &files, config)
177}
178
179/// Build a `GlobSet` from ignore patterns.
180fn build_ignore_set(patterns: &[String]) -> Option<GlobSet> {
181    if patterns.is_empty() {
182        return None;
183    }
184
185    let mut builder = GlobSetBuilder::new();
186    for pattern in patterns {
187        match Glob::new(pattern) {
188            Ok(glob) => {
189                builder.add(glob);
190            }
191            Err(e) => {
192                tracing::warn!("Invalid duplication ignore pattern '{pattern}': {e}");
193            }
194        }
195    }
196
197    builder.build().ok()
198}
199
200#[cfg(test)]
201mod tests {
202    use super::*;
203    use crate::discover::FileId;
204
205    #[test]
206    fn find_duplicates_empty_files() {
207        let config = DuplicatesConfig::default();
208        let report = find_duplicates(Path::new("/tmp"), &[], &config);
209        assert!(report.clone_groups.is_empty());
210        assert!(report.clone_families.is_empty());
211        assert_eq!(report.stats.total_files, 0);
212    }
213
214    #[test]
215    fn build_ignore_set_empty() {
216        assert!(build_ignore_set(&[]).is_none());
217    }
218
219    #[test]
220    fn build_ignore_set_valid_patterns() {
221        let set = build_ignore_set(&["**/*.test.ts".to_string(), "**/*.spec.ts".to_string()]);
222        assert!(set.is_some());
223        let set = set.unwrap();
224        assert!(set.is_match("src/foo.test.ts"));
225        assert!(set.is_match("src/bar.spec.ts"));
226        assert!(!set.is_match("src/baz.ts"));
227    }
228
229    #[test]
230    fn find_duplicates_with_real_files() {
231        // Create a temp directory with duplicate files
232        let dir = tempfile::tempdir().expect("create temp dir");
233        let src_dir = dir.path().join("src");
234        std::fs::create_dir_all(&src_dir).expect("create src dir");
235
236        let code = r#"
237export function processData(input: string): string {
238    const trimmed = input.trim();
239    if (trimmed.length === 0) {
240        return "";
241    }
242    const parts = trimmed.split(",");
243    const filtered = parts.filter(p => p.length > 0);
244    const mapped = filtered.map(p => p.toUpperCase());
245    return mapped.join(", ");
246}
247
248export function validateInput(data: string): boolean {
249    if (data === null || data === undefined) {
250        return false;
251    }
252    const cleaned = data.trim();
253    if (cleaned.length < 3) {
254        return false;
255    }
256    return true;
257}
258"#;
259
260        std::fs::write(src_dir.join("original.ts"), code).expect("write original");
261        std::fs::write(src_dir.join("copy.ts"), code).expect("write copy");
262        std::fs::write(dir.path().join("package.json"), r#"{"name": "test"}"#)
263            .expect("write package.json");
264
265        let files = vec![
266            DiscoveredFile {
267                id: FileId(0),
268                path: src_dir.join("original.ts"),
269                size_bytes: code.len() as u64,
270            },
271            DiscoveredFile {
272                id: FileId(1),
273                path: src_dir.join("copy.ts"),
274                size_bytes: code.len() as u64,
275            },
276        ];
277
278        let config = DuplicatesConfig {
279            min_tokens: 10,
280            min_lines: 2,
281            ..DuplicatesConfig::default()
282        };
283
284        let report = find_duplicates(dir.path(), &files, &config);
285        assert!(
286            !report.clone_groups.is_empty(),
287            "Should detect clones in identical files"
288        );
289        assert!(report.stats.files_with_clones >= 2);
290
291        // Should also have clone families
292        assert!(
293            !report.clone_families.is_empty(),
294            "Should group clones into families"
295        );
296    }
297
298    #[test]
299    fn file_wide_suppression_excludes_file() {
300        let dir = tempfile::tempdir().expect("create temp dir");
301        let src_dir = dir.path().join("src");
302        std::fs::create_dir_all(&src_dir).expect("create src dir");
303
304        let code = r#"
305export function processData(input: string): string {
306    const trimmed = input.trim();
307    if (trimmed.length === 0) {
308        return "";
309    }
310    const parts = trimmed.split(",");
311    const filtered = parts.filter(p => p.length > 0);
312    const mapped = filtered.map(p => p.toUpperCase());
313    return mapped.join(", ");
314}
315"#;
316        let suppressed_code = format!("// fallow-ignore-file code-duplication\n{code}");
317
318        std::fs::write(src_dir.join("original.ts"), code).expect("write original");
319        std::fs::write(src_dir.join("suppressed.ts"), &suppressed_code).expect("write suppressed");
320        std::fs::write(dir.path().join("package.json"), r#"{"name": "test"}"#)
321            .expect("write package.json");
322
323        let files = vec![
324            DiscoveredFile {
325                id: FileId(0),
326                path: src_dir.join("original.ts"),
327                size_bytes: code.len() as u64,
328            },
329            DiscoveredFile {
330                id: FileId(1),
331                path: src_dir.join("suppressed.ts"),
332                size_bytes: suppressed_code.len() as u64,
333            },
334        ];
335
336        let config = DuplicatesConfig {
337            min_tokens: 10,
338            min_lines: 2,
339            ..DuplicatesConfig::default()
340        };
341
342        let report = find_duplicates(dir.path(), &files, &config);
343        // With only 2 files and one suppressed, there should be no clones
344        assert!(
345            report.clone_groups.is_empty(),
346            "File-wide suppression should exclude file from duplication analysis"
347        );
348    }
349}