You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Everything in #8082 — the calibration ground-truth wiring, the backtest primitives, the advisory CI check, the confirmed-signal design — is ORB-only, by explicit stated scope in every one of its sub-issues. AMS has its own class of deterministic/policy decisions (eligibility exclusions — the kind #7982's own AMS adapter, packages/loopover-miner/lib/signal-tracking-store.ts, already records fired events for via discover-cli.ts) that deserve the same backtest-before-ship discipline eventually. Building both sides blind, in parallel, risks duplicating design mistakes the ORB build will surface first — #8082 itself has already gone through one real supersession (#8101 → #8104) from research turning up a better design mid-flight.
Area
Mirrors #8082's own shape, applied to packages/loopover-miner/packages/loopover-engine's AMS-facing surfaces instead of src/rules/**/src/review/**.
Proposal
Once #8082 is fully complete (every sub-issue closed, including the confirmed-signal design and the Phase-2 merge-gate decision) and has run in production long enough to surface real lessons, decompose this epic into AMS-mirrored sub-issues. The likely shape (named here so the roadmap is legible now — NOT precisely scoped, see Deliverables/Boundaries):
Deliberately none precisely scoped yet — the numbered list above is a roadmap sketch, not build-ready issue bodies. Decomposing this into the exact-file/exact-function precision #8082's own sub-issues required, before the ORB side is proven, would risk scoping against code that will have moved by the time this is actually unblocked (the ORB epic itself needed one real correction, #8101 → #8104, from research turning up a better design mid-build — the AMS side deserves the same chance to be built with the ORB side's finished lessons in hand, not guessed at now).
packages/loopover-miner/lib/signal-tracking-store.ts (AMS's own already-built calibration adapter — the AMS-side counterpart to ORB's signal-tracking-wire.ts)
Boundaries
No sub-issues get filed under this epic, and no code gets written against it, until #8082 is fully closed. This is a deliberate, explicit hold — not an oversight if it sits untouched for a while.
maintainer-only — cross-cutting review-infrastructure work, blocked pending #8082, not a build task for contributors.
Problem
Everything in #8082 — the calibration ground-truth wiring, the backtest primitives, the advisory CI check, the confirmed-signal design — is ORB-only, by explicit stated scope in every one of its sub-issues. AMS has its own class of deterministic/policy decisions (eligibility exclusions — the kind #7982's own AMS adapter,
packages/loopover-miner/lib/signal-tracking-store.ts, already records fired events for viadiscover-cli.ts) that deserve the same backtest-before-ship discipline eventually. Building both sides blind, in parallel, risks duplicating design mistakes the ORB build will surface first — #8082 itself has already gone through one real supersession (#8101 → #8104) from research turning up a better design mid-flight.Area
Mirrors #8082's own shape, applied to
packages/loopover-miner/packages/loopover-engine's AMS-facing surfaces instead ofsrc/rules/**/src/review/**.Proposal
Once #8082 is fully complete (every sub-issue closed, including the confirmed-signal design and the Phase-2 merge-gate decision) and has run in production long enough to surface real lessons, decompose this epic into AMS-mirrored sub-issues. The likely shape (named here so the roadmap is legible now — NOT precisely scoped, see Deliverables/Boundaries):
packages/loopover-miner/lib/signal-tracking-store.tsadapter. AMS's "reversal" equivalent needs its own real-precedent audit at that time (AMS has no PR-reopen concept the way ORB does — what a miner or maintainer overriding an AMS exclusion actually looks like needs checking against AMS's real event-ledger vocabulary then, not guessed at now).Deliverables
Deliberately none precisely scoped yet — the numbered list above is a roadmap sketch, not build-ready issue bodies. Decomposing this into the exact-file/exact-function precision #8082's own sub-issues required, before the ORB side is proven, would risk scoping against code that will have moved by the time this is actually unblocked (the ORB epic itself needed one real correction, #8101 → #8104, from research turning up a better design mid-build — the AMS side deserves the same chance to be built with the ORB side's finished lessons in hand, not guessed at now).
Resources
packages/loopover-miner/lib/signal-tracking-store.ts(AMS's own already-built calibration adapter — the AMS-side counterpart to ORB'ssignal-tracking-wire.ts)Boundaries
No sub-issues get filed under this epic, and no code gets written against it, until #8082 is fully closed. This is a deliberate, explicit hold — not an oversight if it sits untouched for a while.
maintainer-only — cross-cutting review-infrastructure work, blocked pending #8082, not a build task for contributors.