Audit Sdlc/Dry/DuplicatedStatementSequence against source test files #376
Labels
No labels
P0
P1
P2
P3
bug
create
delete
enhancement
filed-by/agent
filed-by/user
frozen
lint-rule
needs-info
needs-triage
next
plugin/cc-architect
plugin/os
plugin/os-adr
plugin/os-aidd-lint
plugin/os-backlog
plugin/os-context
plugin/os-doc-hygiene
plugin/os-sdlc
plugin/os-vault
project/cc-os
ready-for-agent
ready-for-human
recurring
review
update
waiting
wayfinder:grilling
wayfinder:map
wayfinder:map
wayfinder:research
wayfinder:task
wayfinder:task
wontfix
worklist/deviations
worklist/lint-rule
worklist/new-implement-build
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
jared/cc-os#376
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Parent
#369
What to build
A verdict on the 152 test-file offenses from Sdlc/Dry/DuplicatedStatementSequence, a cop mined from the very test files it now flags: sampled findings classified as real test-quality defects versus cop miscalibration, with a recommended threshold/config change if miscalibrated, posted as a comment on #369.
Acceptance criteria
Blocked by
Started: auditing DuplicatedStatementSequence findings.
Resolution
Done: Audited Sdlc/Dry/DuplicatedStatementSequence: 164 current offenses, ~30 sampled; ~80% miscalibration (idiomatic minitest AAA pairs), ~15% real helper-worthy duplication, ~5% borderline
Evidence: Verdict comment posted on #369; cop's WINDOW_SIZE=2 over-generalizes — recommended raising to 3-4 and/or excluding assert_/refute_ statement windows
Follow-ups: Consolidated decision (incl. cop retune and the one real non-test hit in plugins/os-vault/eval/bin/check) is #378's scope; no separate tickets