[GH-ISSUE #1446] [Build] Combined main exceeds the duplicate-code gate with comparison fixtures #492

Closed
opened 2026-10-15 18:25:31 +00:00 by bsakaguchi · 0 comments
Owner

Originally created by @N0zoM1z0 on GitHub (Oct 10, 2026).
Original GitHub issue: https://github.com/morluto/rea/issues/1446

Frozen current main 33b9b31a fails the unchanged npm run jscpd gate: 8,476 duplicated lines out of 338,754 total lines (approximately 2.5021%), exceeding the configured 2.5% threshold. The same totals appear in the static check of #1443.

The diagnostic PR changes only an Android fixture that is excluded by the scanner. This is a combined-main regression after the independently green comparison work merged. The new javascriptApplicationChangeGraph.test.ts contributes three reported clones with existing version-comparison/relationship tests: imports, graph construction and paired projection input.

Keep the meaningful right-hand Evidence provenance regression and the existing threshold. Reuse the established application-graph fixture owner to remove duplicated node construction, then verify the complete duplicate-code scan and adjacent comparison workflows.

Independent reproduction uses pinned Node 24.18.0 with two CPUs and the normal process-family guard. No provider or runtime behavior is implicated. Searched open issues/PRs; the earlier broad duplication roadmap #172 is closed.

Refs #1434, #1443, #172, #1065.

Originally created by @N0zoM1z0 on GitHub (Oct 10, 2026). Original GitHub issue: https://github.com/morluto/rea/issues/1446 Frozen current main `33b9b31a` fails the unchanged `npm run jscpd` gate: 8,476 duplicated lines out of 338,754 total lines (approximately 2.5021%), exceeding the configured 2.5% threshold. The same totals appear in [the static check of #1443](https://github.com/morluto/rea/actions/runs/38027563892/job/114141573990). The diagnostic PR changes only an Android fixture that is excluded by the scanner. This is a combined-main regression after the independently green comparison work merged. The new `javascriptApplicationChangeGraph.test.ts` contributes three reported clones with existing version-comparison/relationship tests: imports, graph construction and paired projection input. Keep the meaningful right-hand Evidence provenance regression and the existing threshold. Reuse the established application-graph fixture owner to remove duplicated node construction, then verify the complete duplicate-code scan and adjacent comparison workflows. Independent reproduction uses pinned Node 24.18.0 with two CPUs and the normal process-family guard. No provider or runtime behavior is implicated. Searched open issues/PRs; the earlier broad duplication roadmap #172 is closed. Refs #1434, #1443, #172, #1065.
bsakaguchi 2026-10-15 18:25:31 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
morluto/rea#492
No description provided.