Context
packages/loopover-engine/src/duplicate-winner.ts:46 (JSDoc at lines 43-44) marks isDuplicateClusterWinner @deprecated ... retained only for legacy compatibility callers that do not have claim timestamps. A repo-wide grep across packages/loopover-engine, packages/loopover-miner, and src/** finds zero real call sites — only re-exports (index.ts, src/signals/duplicate-winner.ts) and {@link} mentions in unrelated comments (src/queue/duplicate-detection.ts, src/services/maintainer-activation.ts). The legacy callers the deprecation comment describes don't actually exist anywhere in the current codebase.
Requirements
- Confirm via a fresh repo-wide search (including the re-export chain through
index.ts/src/signals/duplicate-winner.ts) that isDuplicateClusterWinner truly has zero real callers, not just zero direct callers.
- Remove
isDuplicateClusterWinner and its re-exports.
- Update the
{@link} mentions in src/queue/duplicate-detection.ts and src/services/maintainer-activation.ts to stop referencing a function that no longer exists.
Deliverables
Expected Outcome
No deprecated dead function remains whose own justifying comment describes callers that don't exist in the current codebase.
Links & Resources
packages/loopover-engine/src/duplicate-winner.ts:43-46src/queue/duplicate-detection.ts, src/services/maintainer-activation.ts (stale {@link} mentions)
Context
packages/loopover-engine/src/duplicate-winner.ts:46(JSDoc at lines 43-44) marksisDuplicateClusterWinner@deprecated ... retained only for legacy compatibility callers that do not have claim timestamps. A repo-wide grep acrosspackages/loopover-engine,packages/loopover-miner, andsrc/**finds zero real call sites — only re-exports (index.ts,src/signals/duplicate-winner.ts) and{@link}mentions in unrelated comments (src/queue/duplicate-detection.ts,src/services/maintainer-activation.ts). The legacy callers the deprecation comment describes don't actually exist anywhere in the current codebase.Requirements
index.ts/src/signals/duplicate-winner.ts) thatisDuplicateClusterWinnertruly has zero real callers, not just zero direct callers.isDuplicateClusterWinnerand its re-exports.{@link}mentions insrc/queue/duplicate-detection.tsandsrc/services/maintainer-activation.tsto stop referencing a function that no longer exists.Deliverables
isDuplicateClusterWinnerand its re-exports removed.{@link}references updated.duplicate-winner.ts's remaining exports continue passing.Expected Outcome
No deprecated dead function remains whose own justifying comment describes callers that don't exist in the current codebase.
Links & Resources
packages/loopover-engine/src/duplicate-winner.ts:43-46src/queue/duplicate-detection.ts,src/services/maintainer-activation.ts(stale{@link}mentions)