Skip to content

fix: collect jscpd findings above project threshold - #742

Closed
liorp wants to merge 1 commit into
peteromallet:mainfrom
liorp:fix/jscpd-report-threshold
Closed

liorp wants to merge 1 commit into
peteromallet:mainfrom
liorp:fix/jscpd-report-threshold

Conversation

@liorp

@liorp liorp commented Sep 12, 2026

Copy link
Copy Markdown

When a repository sets jscpd's duplication threshold to zero, jscpd writes a valid JSON report and then exits nonzero. The adapter discards that report, so duplication findings disappear from the scan.

Pass --threshold 100 for report collection. This does not alter the repository's CI configuration or suppress any reported clones.

Validation: the new regression fails without the fix; all 66 external-adapter tests pass. A real repository scan recovered 1,054 duplicate clusters. The full suite had 5,661 passed, 152 skipped, and three unrelated Bash unused-import failures that also reproduce on the unchanged baseline.

@liorp liorp closed this Sep 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant