test(context): pin CPG-09 stateful recovery deferral - #24
Conversation
|
✅ Deterministic PR hygiene checks passed. |
5f51cfa to
c9f6d41
Compare
3777bfc to
4459006
Compare
b952613 to
092ad37
Compare
4459006 to
23433de
Compare
092ad37 to
da36646
Compare
23433de to
89b5e2e
Compare
da36646 to
a20d295
Compare
9de2c7a to
f62134a
Compare
39b2ce5 to
91f21bf
Compare
fe78b6b to
e550646
Compare
ba1bf9d to
86c2987
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
28337e7 to
4957f8f
Compare
15e1424 to
a558136
Compare
4957f8f to
4357e0c
Compare
a558136 to
9d8f3ed
Compare
4357e0c to
cb8ff73
Compare
9d8f3ed to
4c21418
Compare
cb8ff73 to
99e0ede
Compare
Summary
runTurn, while Kiro uses build/fetch/parse with provider-private continuation state.ProviderAdapter.contextProjectionProviderStatefulinstead of inferring recovery safety from adapter names or transport shape.stateful_adapterfail-open behaviour for both forms of provider state.__ocx_context_v1can be pruned from a full catalogue.Deep-review hardening
Kiro and Cursor now declare
contextProjectionProviderStateful: truein the adapter registry. The core recovery capability planner consumes that explicit property. Stateless adapters assert that the capability is absent. This separates provider-owned continuation semantics from the unrelatedadapterNeedsForcedContinuation()persistence helper and prevents a future stateful adapter from silently becoming recovery-eligible merely because it does not userunTurn.Focused provider-path/stateful-deferral/evaluation/review-hardening tests and the full TypeScript typecheck passed after the final rebase onto CPG-08.
Scope decision after deep review
This PR is the CPG-09 readiness/deferral gate, not the completed provider-stateful recovery milestone from the original roadmap. That is intentional, not an implementation shortcut.
Stateful recovery must remain disabled until CPG-08 has live evidence for the simpler stateless path, including comparison against OpenAI HTTP
previous_response_idcontinuation and Responses WebSocket continuation. Byte reduction or synthetic exact-recovery checks are not sufficient evidence.Preconditions for enabling Cursor/Kiro recovery later
All of the following must be true first:
If those conditions do not pass, this deferral is the intended final CPG-09 behaviour.
Why this is a deferred stage
The implementation plan permits provider-stateful recovery only after the stateless path is proven with live provider evidence. The checked-in CPG-08 evaluation has zero live provider attempts, so enabling recovery for Cursor or Kiro would violate that rollout gate.
Stack
agent/context-projection-cp08/ PR feat(context): add CPG-08 evaluation harness and rollout gates #23