Repository navigation
Conversation
|
✅ Deterministic PR hygiene checks passed. |
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
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 |
|
Current-head hosted validation at The original attempt's test 4/4 failed in the unchanged optional Desktop runtime fixture while binding its remembered loopback CONNECT port ( Keep Draft for independent replay-boundary review. No live provider/Codex UI conversation, Windows runtime, deployment, merge or release is certified by these fixtures. |
Summary
Closes #7369.
A restored Responses conversation can carry native opaque reasoning without the process-local route-retention marker. Its HTTP-200 SSE ciphertext refusal bypasses zero-output recovery; an explicitly identified reasoning-only HTTP 502 also fails the current status gate.
Reuse actual outbound opaque-state detection for bounded stream preflight, retain the same explicit identity through flat/nested SSE failures, and admit an identified 502 through the existing one-shot outbound rebuild. The flat projector preserves the caller-validated 400 code/message instead of replacing it with a generic upstream failure. Keep the existing function-output-specific evidence requirement, ambiguity/send limits, non-replayable and text/tool commitment guards.
Native success retains its ciphertext. Rebuilds keep store:false and tool-call/result pairs; the original input is preserved. No conversation file is read/rewritten, no native ciphertext is decrypted, and no new plaintext reconstruction policy is introduced. Update the owning contract/ADR and both test maps.
Verification
79fa97be6e6ae974ccd31d9aeedc2da96a0baa73.bun test --isolate tests/responses/responses-cold-opaque-recovery.test.ts tests/responses/responses-opaque-blob-recovery.test.ts tests/responses/opaque-blob-wrapped-rejection.test.ts tests/responses/responses-azure-opaque-blob-recovery.test.ts tests/responses/reasoning-retention-routing.test.ts tests/routing/combo-stream-preflight.test.ts— 165 pass, 0 fail.Checklist
No merge, release or installed-service change is requested by publication.