Skip to content

[codex] Handle Codex tool search transcript shapes#24

Merged
steipete merged 2 commits into
mainfrom
codex/codex-tool-search-output-shape
Jun 11, 2026
Merged

[codex] Handle Codex tool search transcript shapes#24
steipete merged 2 commits into
mainfrom
codex/codex-tool-search-output-shape

Conversation

@scoootscooob

Copy link
Copy Markdown
Member

Summary

  • Advertise OpenClaw protocol v4 compatibility while continuing to support v3
  • Parse Codex/OpenAI tool-search and function-call output blocks into ClawBench canonical tool calls/results
  • Configure container lane runs to select the Codex agent runtime and set codexDynamicToolsLoading explicitly

Why

OpenClaw latest main can run GPT-5.5 through the Codex runtime with dynamic tool loading. Without parser hardening, bridge/control output shapes can fail to correlate tool results back to calls, which makes trajectory/verifier signals look like zeros even when tools ran. The lane config also needs to pass the Codex plugin setting from ClawBench until OpenClaw exposes a fully stable external contract.

Validation

  • bash -n scripts/container_lane_eval.sh
  • /Users/zhentongfan/Desktop/openclaw/clawbench/.venv/bin/python -m pytest -q tests/test_client.py -> 19 passed
  • /Users/zhentongfan/Desktop/openclaw/clawbench/.venv/bin/python -m pytest -q -> 299 passed, 5 skipped

@scoootscooob
scoootscooob marked this pull request as ready for review May 16, 2026 00:27
@scoootscooob
scoootscooob requested a review from a team as a code owner May 16, 2026 00:27

@steipete steipete left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved after maintainer fixup.

Proof: local ruff, runtime contract smoke, full pytest, built wheel install, live ClawBench/OpenClaw protocol v4 run, Docker build, and Docker parser smoke all passed. CI is green at exact head 0f9b95b.

@steipete
steipete merged commit 8834e7d into main Jun 11, 2026
8 checks passed
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.

2 participants