test(codex): preserve mirrored transcript branch across turns#77051
test(codex): preserve mirrored transcript branch across turns#77051mjamiv wants to merge 2 commits intoopenclaw:mainfrom
Conversation
|
Codex review: needs maintainer review before merge. Summary Reproducibility: yes. for the PR coverage path: the added test performs two mirror writes with distinct scopes and inspects the resulting JSONL branch. I did not run local tests because this review is read-only, but the source path and PR-head checks support the scenario. Next step before merge Security Review detailsBest possible solution: Merge the focused regression test after exact-head CI finishes, and handle the WebChat overwrite behavior in the linked implementation issue or PR. Do we have a high-confidence way to reproduce the issue? Yes for the PR coverage path: the added test performs two mirror writes with distinct scopes and inspects the resulting JSONL branch. I did not run local tests because this review is read-only, but the source path and PR-head checks support the scenario. Is this the best way to solve the issue? Yes: for a test-only PR, the updated approach is boundary-clean and narrowly covers consecutive Codex mirror turns without adding a new public API or reaching into core internals. What I checked:
Likely related people:
Remaining risk / open question:
Codex review notes: model gpt-5.5, reasoning high; reviewed against 90d25d59c610. |
|
Updated #77051 at f58124c to address the plugin-test core import blocker. The regression now stays inside the Codex extension test boundary and asserts the mirrored JSONL roles, text content, and parent chain directly. Validation passed:
|
|
Follow-up on the remaining red CI after f58124c: the local focused test, format, and changed-check lane passed, and the failed GitHub jobs are downstream of The log shows visible gateway-server tests passing, then Vitest hitting the 300s no-output watchdog, retrying, and hitting the same no-output watchdog again. |
Summary
Test
Refs #77012