test(server): match V2 tool output redaction in retention coverage - #10865
Open
saphid wants to merge 383 commits into
Open
test(server): match V2 tool output redaction in retention coverage#10865saphid wants to merge 383 commits into
saphid wants to merge 383 commits into
Conversation
Keep positive sub-minute schedules readable without accepting corrupt zero or negative persisted intervals. Co-authored-by: codex <codex@users.noreply.github.com>
Cancel pending prompt completions without restarting ACP, and keep text-only monitor acknowledgements tracked as background work. Co-authored-by: codex <codex@users.noreply.github.com>
Allow merge-back from provider-finished runs awaiting checkpoint capture while keeping waiting runs visible but non-interruptible in mobile. Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Adopt TypeScript 7 and Effect rc.112 across orchestration v2, including the TaggedError API migration and updated Effect-aware tests. Restore main's composer-aware scroll-to-end clearance while retaining selected-model settings sync, preview recording transfer, image galleries, desktop context menus, and layout hit targets. Regenerate the lockfile on the upgraded dependency baseline.
Advertise bounded socket snapshots and authoritative dispatch validation, omit raw command output and inline file bodies at the wire boundary, and preserve compact status metadata across web and mobile. Add transport-budget coverage for snapshots, resume, commands, legacy import, and projection maintenance.
Contributor
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — This is a self-contained, one-line test expectation update in a test-only file. It aligns replay/live transport assertions with existing output redaction while continuing to verify that full output remains retained in persistence and projections. Notes:
You can add or adjust custom eligibility rules. Learn more. |
This was referenced Sep 9, 2026
juliusmarminge
force-pushed
the
t3code/codex-turn-mapping
branch
from
September 9, 2026 06:39
8187b17 to
ada72ce
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The V2 tool-output redaction change leaves the foundation retention test expecting the old
{ truncated: true }output preview. Test Server 1 fails on current V2 integrations for #10024 and #10025 because replay and live payloads now intentionally omit that output.Update the assertion to require omission. The test still checks the transport size limit and verifies that both the stored event and full detail projection retain the original 9 MB output. Production behavior is unchanged.
Validation on V2 base
7dd9873ed: 56 focused foundation/wire-projection tests pass, server typecheck passes, and targeted lint/format/diff checks pass. The stale assertion is reproduced in both linked PRs' Test Server 1 failures. Independent Claude Opus 5 high review was unavailable during this audit: the direct launcher exited 1 with expired OAuth before inference.Prepared with GPT-6 in the Codex/T3 harness.