fix(codex): preserve streamed command output by clawsweeper[bot] · Pull Request #83222 · openclaw/openclaw

clawsweeper · 2026-05-17T18:17:04Z

Makes #83200 merge-ready for the ClawSweeper automerge loop.
The edit pass should inspect the live PR diff, review comments, and failing checks; rebase if needed; keep the contributor branch credited; and stop only when validation is green or an external blocker is proven.

ClawSweeper 🐠 replacement reef notes:

Cluster: automerge-openclaw-openclaw-83200
Source PRs: fix(codex): preserve streamed command output #83200
Credit: Source PR: fix(codex): preserve streamed command output #83200
Validation: pnpm check:changed
Replacement reason: ClawSweeper could not update the source PR branch directly, so it opened a writable replacement PR instead.
Automerge requested by: @Takhoffman

Repair fallback: GitHub rejected the repair branch push because it updates workflow files and the ClawSweeper app token does not have workflows permission

Co-author credit kept:

@rozmiarD: Co-authored-by: Krzysztof Probola 32790662+rozmiarD@users.noreply.github.com

fish notes: model gpt-5.5, reasoning high; reviewed against 07393a3.

clawsweeper · 2026-05-17T18:18:01Z

Codex review: passed.

Workflow note: Future ClawSweeper reviews update this same comment in place.

How this review workflow works

ClawSweeper keeps one durable marker-backed review comment per issue or PR.
Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
A fresh review can be triggered by maintainer comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
PR/issue authors can comment @clawsweeper re-review or @clawsweeper re-run on their own open PR or issue to request a fresh review only.
Maintainers can also comment @clawsweeper review to request a fresh review only.
Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

Summary
The PR buffers Codex command-output deltas per command item and uses them as a fallback for transcripts, trajectory output, final tool output, and after-tool-call errors when aggregatedOutput is empty.

Reproducibility: yes. A source-level reproduction is clear: send current-turn command-output delta notifications, then complete the command snapshot with aggregatedOutput: null; current main has no final transcript or trajectory fallback for the streamed text.

Real behavior proof
Sufficient (live_output): The linked source PR includes redacted after-fix live OpenClaw Codex output and a trajectory excerpt showing command stdout preserved in the final toolResult transcript.

Next step before merge
No separate repair job is needed; this PR is already the ClawSweeper automerge replacement and should proceed through normal exact-head CI and mergeability gates.

Security
Cleared: The diff changes only Codex projector state, a focused regression test, and changelog text; it adds no dependencies, workflows, permissions, auth handling, or new execution surface.

Review details

Best possible solution:

Land this focused Codex projector fallback after exact-head CI and automerge gates, preserving the newer current-main verbose-progress suppression behavior.

Do we have a high-confidence way to reproduce the issue?

Yes. A source-level reproduction is clear: send current-turn command-output delta notifications, then complete the command snapshot with aggregatedOutput: null; current main has no final transcript or trajectory fallback for the streamed text.

Is this the best way to solve the issue?

Yes. Buffering command deltas before verbose-output gating and using them only when final aggregate output is empty is the narrowest maintainable fix for the existing Codex app-server projector path.

What I checked:

Current-main source gap: Current main drops command output deltas before retaining them unless live tool-output streaming is enabled, and command final output is read only from aggregatedOutput, so a completed command snapshot with aggregatedOutput: null has no transcript or trajectory fallback. (extensions/codex/src/app-server/event-projector.ts:709, 3e765263dd6a)
Protocol permits missing aggregate output: The Codex app-server protocol type allows aggregatedOutput to be string | null, so the projector needs to tolerate completed command snapshots without an aggregate string. (extensions/codex/src/app-server/protocol.ts:235, 3e765263dd6a)
PR implementation: The PR appends output deltas before verbose-output gating and passes the per-item buffer into trajectory, transcript, final output, and error helpers while preserving aggregatedOutput precedence. (extensions/codex/src/app-server/event-projector.ts:707, 07393a304f26)
Current-main integration check: A merge-tree check against current main showed the PR fallback composes with the newer transcript-progress suppression changes without textual conflicts. (extensions/codex/src/app-server/event-projector.ts:707, 3e765263dd6a)
Regression coverage: The new test sends item/commandExecution/outputDelta notifications, completes a command with aggregatedOutput: null, then asserts the streamed text appears in the mirrored tool result and trajectory output. (extensions/codex/src/app-server/event-projector.test.ts:1341, 07393a304f26)
Real behavior proof: The linked source PR provides a redacted live OpenClaw Codex run and trajectory excerpt showing OPENCLAW_OUTPUT_DELTA_PROOF_83199=passed preserved in the final toolResult transcript entry and assistant-visible output, plus a targeted Vitest run reporting 1 file and 48 tests passed.

Likely related people:

@steipete: Blame and git log -S tie the current Codex app-server projector output, trajectory, and transcript helpers to Peter Steinberger's current-main work. (role: recent area contributor; confidence: high; commits: aaf85166deba, f066dd2f31c2; files: extensions/codex/src/app-server/event-projector.ts, extensions/codex/src/app-server/event-projector.test.ts)
@kurplunkin: The current main commit for nearby Codex app-server verbose tool-progress behavior credits Tyler Bea as a source contributor on the same projector and tests. (role: recent adjacent contributor; confidence: medium; commits: fb028cadc8c4; files: extensions/codex/src/app-server/event-projector.ts, extensions/codex/src/app-server/event-projector.test.ts, extensions/codex/src/app-server/run-attempt.ts)
@Takhoffman: The replacement PR records automerge as requested by this user, and the recent same-area current-main merge includes an approval signal from the same person. (role: reviewer and automerge requester; confidence: medium; commits: fb028cadc8c4; files: extensions/codex/src/app-server/event-projector.ts, extensions/codex/src/app-server/event-projector.test.ts)

Codex review notes: model gpt-5.5, reasoning high; reviewed against 3e765263dd6a.

clawsweeper · 2026-05-17T18:22:24Z

🦞✅
ClawSweeper merged this PR after the passing review.

Source: clawsweeper[bot]
Feedback: structured ClawSweeper verdict: pass (sha=07393a304f26c589d125516a7269120dc14ec0ca)
Merge status: merged by ClawSweeper automerge
Merged at: 2026-05-17T18:41:01Z
Merge commit: f36a1b0c8120

What merged:

The PR buffers Codex command-output deltas per command item and uses them as a fallback for transcripts, trajectory output, final tool output, and after-tool-call errors when aggregatedOutput is empty.
Reproducibility: yes. A source-level reproduction is clear: send current-turn command-output delta notificat ... aggregatedOutput: null`; current main has no final transcript or trajectory fallback for the streamed text.

Automerge notes:

PR branch already contained follow-up commit before automerge: fix(codex): preserve streamed command output

The automerge loop is complete.

Automerge progress:

2026-05-17 18:40:48 UTC review passed 07393a304f26 (structured ClawSweeper verdict: pass (sha=07393a304f26c589d125516a7269120dc14ec...)

2026-05-17 18:22:24 UTC repair queued 07393a304f26 (autonomous) Run: https://github.com/openclaw/clawsweeper/actions/runs/25998976275

2026-05-17 18:26:48 UTC repair started (running) in 1s Run: https://github.com/openclaw/clawsweeper/actions/runs/25998976275 automerge-openclaw-openclaw-83200

2026-05-17 18:26:50 UTC repair completed (no branch change) in 2s Run: https://github.com/openclaw/clawsweeper/actions/runs/25998976275 no planned fix actions

2026-05-17 18:35:23 UTC review queued 07393a304f26 (queued)

2026-05-17 18:41:03 UTC merged 07393a304f26 (merged by ClawSweeper automerge)

Takhoffman · 2026-05-17T18:40:00Z

@clawsweeper automerge

Summary: - The PR buffers Codex command-output deltas per command item and uses them as a fallback for transcripts, trajectory output, final tool output, and after-tool-call errors when `aggregatedOutput` is empty. - Reproducibility: yes. A source-level reproduction is clear: send current-turn command-output delta notificat ... aggregatedOutput: null`; current main has no final transcript or trajectory fallback for the streamed text. Automerge notes: - PR branch already contained follow-up commit before automerge: fix(codex): preserve streamed command output Validation: - ClawSweeper review passed for head 07393a304f26c589d125516a7269120dc14ec0ca. - Required merge gates passed before the squash merge. Prepared head SHA: 07393a304f26c589d125516a7269120dc14ec0ca Review: openclaw/openclaw#83222 (comment) Co-authored-by: 0x505badc0de <32790662+rozmiarD@users.noreply.github.com> Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com> Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com> Approved-by: takhoffman Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>

Summary: - The PR buffers Codex command-output deltas per command item and uses them as a fallback for transcripts, trajectory output, final tool output, and after-tool-call errors when `aggregatedOutput` is empty. - Reproducibility: yes. A source-level reproduction is clear: send current-turn command-output delta notificat ... aggregatedOutput: null`; current main has no final transcript or trajectory fallback for the streamed text. Automerge notes: - PR branch already contained follow-up commit before automerge: fix(codex): preserve streamed command output Validation: - ClawSweeper review passed for head 07393a3. - Required merge gates passed before the squash merge. Prepared head SHA: 07393a3 Review: openclaw#83222 (comment) Co-authored-by: 0x505badc0de <32790662+rozmiarD@users.noreply.github.com> Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com> Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com> Approved-by: takhoffman Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>

rozmiarD and others added 2 commits May 17, 2026 18:10

fix(codex): preserve streamed command output

6b06445

fix(codex): preserve streamed command output

07393a3

clawsweeper Bot mentioned this pull request May 17, 2026

fix(codex): preserve streamed command output #83200

Closed

openclaw-barnacle Bot removed proof: supplied External PR includes structured after-fix real behavior proof. proof: sufficient ClawSweeper judged the real behavior proof convincing. labels May 17, 2026

clawsweeper Bot added the proof: sufficient ClawSweeper judged the real behavior proof convincing. label May 17, 2026

clawsweeper Bot merged commit f36a1b0 into main May 17, 2026
177 of 189 checks passed

clawsweeper Bot deleted the clawsweeper/automerge-openclaw-openclaw-83200 branch May 17, 2026 18:41

github-actions Bot mentioned this pull request May 17, 2026

📡 Upstream Digest — 2026-05-17 18:56 UTC curtismercier/openclaw-mods#885

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix(codex): preserve streamed command output#83222

fix(codex): preserve streamed command output#83222
clawsweeper[bot] merged 2 commits into
mainfrom
clawsweeper/automerge-openclaw-openclaw-83200

clawsweeper Bot commented May 17, 2026

Uh oh!

clawsweeper Bot commented May 17, 2026 •

edited

Loading

Uh oh!

clawsweeper Bot commented May 17, 2026 •

edited

Loading

Uh oh!

Takhoffman commented May 17, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

clawsweeper Bot commented May 17, 2026

Uh oh!

clawsweeper Bot commented May 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

clawsweeper Bot commented May 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Takhoffman commented May 17, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

clawsweeper Bot commented May 17, 2026 •

edited

Loading

clawsweeper Bot commented May 17, 2026 •

edited

Loading