feat(context): add CPG-08 evaluation harness and rollout gates - #23
feat(context): add CPG-08 evaluation harness and rollout gates#23Wibias wants to merge 7 commits into
Conversation
|
✅ Deterministic PR hygiene checks passed. |
64447fa to
dcc5d8e
Compare
bb1e7dc to
3777bfc
Compare
dcc5d8e to
9373c37
Compare
3777bfc to
4459006
Compare
✅ READY
Hygiene✅ Deterministic PR hygiene checks passed. |
ac7ee9b to
771d8c2
Compare
4459006 to
23433de
Compare
771d8c2 to
6b856fd
Compare
23433de to
89b5e2e
Compare
4fe3405 to
22d5800
Compare
fe78b6b to
e550646
Compare
b06d215 to
5e1f0d8
Compare
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 🗂️ Base branches to auto review (2)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
969fab3 to
15e1424
Compare
5e1f0d8 to
6de79db
Compare
15e1424 to
a558136
Compare
6de79db to
7d2dba0
Compare
a558136 to
9d8f3ed
Compare
7d2dba0 to
8a44406
Compare
9d8f3ed to
4c21418
Compare
Summary
docs/context-projection-live-evaluation.mdwith the exact live trial protocol and per-attempt evidence fields.Scope decision after deep review
This PR is the evaluation framework + synthetic preflight, not the completed live rollout evaluation from the original roadmap. The checked-in evidence currently has zero live provider attempts, so it cannot justify default-on recovery or provider-stateful expansion.
Before the live gate can pass, the evaluation matrix must include provider-native continuation controls rather than comparing only context-reduction strategies. Required live controls are:
previous_response_idwhen the cell supports it;previous_response_idwhen the cell supports it;The live workload must also include adversarial evidence placement and long agent trajectories: required evidence in the middle of a large result, root cause far from an error-log tail, near-identical outputs with one critical differing line, omitted edit anchors, recovery followed by a real mutating action, evidence needed many turns later, provider reconnect/state loss, tool-catalog changes, and 20-50 tool-call coding tasks.
Per-cell evidence gate
liveProviderAttemptsand caller-suppliedcontrolCoverageCompleteare retained only for backward readability of old synthetic reports. They cannot approve rollout.The gate now derives evidence from declared provider/workload/cache cells. Every required arm in every cell must have at least 30 completed attempts. OpenAI Responses cells can require both native HTTP and WebSocket continuation controls; provider cells can independently require native compaction. Missing controls and under-sampled arms fail closed.
Metric separation
Live evaluation must not collapse different optimisations into one "context savings" number. It must report separately:
TDD / rollout gate
The original first commit is RED and pins the decision gate: zero protocol violations, 2pp task-success non-inferiority, at least 10% cost-per-success improvement, <=15% p90 total-latency regression, and live-provider evidence required.
The deep-review hardening adds tests that reject aggregate-only evidence, reject any required arm below 30 attempts, and reject missing OpenAI WebSocket or provider-compaction controls when a cell declares those capabilities.
If the evidence is missing or the gate fails, recovery remains opt-in and CPG-07 full sidecar composition plus CPG-09 provider-stateful recovery remain deferred.
Stack
agent/context-projection-cp07/ replacement PR feat(context): add CPG-07 explicit sidecar composition gates #25