Found while triaging #9553's unused-symbol list.
sweepRepoBacklogConvergence (src/queue/processors.ts) accepts requestedBy: "schedule" | "api" | "test" and never reads it. Every sibling sweep — fanOutRepoDocRefreshSweepJobs, the re-gate sweep, the evidence sweep — stamps it into recordAuditEvent's metadata.
Both agent.sweep.backlog_convergence audit events in this function omit it, so those records cannot be attributed to a scheduled run vs a manual API trigger. Observability gap only — no gating logic reads it.
Worth recording because the mechanical response to the compiler warning is to rename the parameter _requestedBy, which cements the gap permanently instead of closing it. The intended behaviour is plainly the sibling sweeps'.
Found while triaging #9553's unused-symbol list.
sweepRepoBacklogConvergence(src/queue/processors.ts) acceptsrequestedBy: "schedule" | "api" | "test"and never reads it. Every sibling sweep —fanOutRepoDocRefreshSweepJobs, the re-gate sweep, the evidence sweep — stamps it intorecordAuditEvent'smetadata.Both
agent.sweep.backlog_convergenceaudit events in this function omit it, so those records cannot be attributed to a scheduled run vs a manual API trigger. Observability gap only — no gating logic reads it.Worth recording because the mechanical response to the compiler warning is to rename the parameter
_requestedBy, which cements the gap permanently instead of closing it. The intended behaviour is plainly the sibling sweeps'.