-
Notifications
You must be signed in to change notification settings - Fork 15
Closed
Labels
auditCodebase audit findingCodebase audit findingdead-codeDead or unreachable codeDead or unreachable code
Description
Problem
Two utility modules exist with full test suites but zero production consumers. They appear to have been built for features that were never integrated.
1. src/utils/fileOwnership.ts + src/utils/fileOwnership.test.ts
Exports: shouldClaimFile, getClaimPriority, FileOwnershipInput, FileOwnershipResult
Grep confirms these are only imported by the test file — no component, hook, store, or plugin uses them.
2. src/utils/selectionPulse.ts + src/utils/selectionPulse.test.ts
Exports: triggerSelectionPulse
Same situation — only imported by its own test file. The visual feedback feature was never wired up.
Impact
- Dead code inflates the codebase and test suite maintenance burden
- Coverage thresholds count these tests, masking actual coverage of live code
- Future developers may assume these features are active
Suggested Fix
Delete both source and test files:
src/utils/fileOwnership.tssrc/utils/fileOwnership.test.tssrc/utils/selectionPulse.tssrc/utils/selectionPulse.test.ts
If these features are planned for future use, they can be re-added when actually needed.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
auditCodebase audit findingCodebase audit findingdead-codeDead or unreachable codeDead or unreachable code