Stream Codex preambles in channel progress drafts#81887
Conversation
|
Codex review: needs real behavior proof before merge. Summary Reproducibility: Do we have a high-confidence way to reproduce the issue? No live high-confidence reproduction is present; source inspection shows current main lacks the projector and keyed draft behavior, and the PR adds focused tests for that path. Real behavior proof Next step before merge Security Review detailsBest possible solution: Land only after maintainer approval and redacted real channel proof, preferably Telegram, showing one Codex preamble line updating in place and not becoming the final answer. Do we have a high-confidence way to reproduce the issue? Do we have a high-confidence way to reproduce the issue? No live high-confidence reproduction is present; source inspection shows current main lacks the projector and keyed draft behavior, and the PR adds focused tests for that path. Is this the best way to solve the issue? Is this the best way to solve the issue? Yes in direction: carrying Codex item phases and stable item IDs through the shared progress draft contract is the narrow maintainable shape, but visible channel proof and maintainer approval are still required. Acceptance criteria:
What I checked:
Likely related people:
Remaining risk / open question:
Codex review notes: model gpt-5.5, reasoning high; reviewed against 697103604367. |
Summary
Codex app-server already marks preamble and progress narration as commentary-phase agent messages. OpenClaw was preserving those messages internally, so channel progress drafts could show tools but not Codex's own "I am checking..." narration.
This teaches the Codex projector to remember agent-message phases from item lifecycle events, stream commentary deltas as keyed progress updates, and keep commentary out of final reply selection. The shared progress renderer now supports stable line ids so one preamble updates in place instead of stacking partial prefixes, and the edit-capable channel surfaces pass those ids through.
The streaming docs and changelog now describe Codex preambles as part of the tool-progress preview path.