Skip to content

Keep Codex source and STATE probes on the pinned plaintext protocol - #274

Merged
HTExplicit merged 1 commit into
mainfrom
codex/borrow-probe-wire-contract
Oct 10, 2026
Merged

HTExplicit merged 1 commit into
mainfrom
codex/borrow-probe-wire-contract

Conversation

@HTExplicit

Copy link
Copy Markdown
Owner

With the production compression setting enabled, borrowing cleared stale Content-Encoding headers and then silently re-entered the business zstd encoder at dispatch. Source acquisition and both target STATE probes therefore differed from the pinned ranxi plaintext wire protocol. The existing header regression passed only because its test process had compression disabled.

Mark only source acquisition and the two-shot target protocol as plaintext observations. Keep normal business and Pelican generation on the existing compression setting; do not toggle that setting. Record source and per-shot request encoding in additive diagnostic fields. The existing authenticated probe comparison can also compare plaintext with the configured codec on one fixed candidate, with identical plaintext body/tier/identity/exit and the same eight-request bound. Probe-only results never qualify business traffic.

Validation: enabling the real compression condition reproduced the unexpected zstd headers before the fix. The corrected source/target dispatch contract, preserved native headers and actual models/tiers, normal-generation compression, unchanged global switch, isolated encoding comparison and ordinary native HTTP tool-continuation regressions pass. Live wire-encoding comparison and complete model acceptance remain pending deployment; prior body-only comparisons do not establish plaintext protocol parity.

@HTExplicit
HTExplicit enabled auto-merge (squash) October 10, 2026 05:07
@HTExplicit
HTExplicit merged commit c661d40 into main Oct 10, 2026
18 checks passed
@HTExplicit
HTExplicit deleted the codex/borrow-probe-wire-contract branch October 10, 2026 05:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant