Issue Station
Sign In
Tools hereToolbox

openclaw - ✅(Solved) Fix Codex ACP rejects timeout config option during sessions_spawn [1 pull requests, 3 comments, 2 participants]

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…
GitHub stats
openclaw/openclaw#73052•Fetched 2026-04-28 06:28:10
View on GitHub
Comments
3
Participants
2
Timeline
5
Reactions
0
Author
richa65richa65
Participants
richa65richa65steipetesteipete
Timeline (top)
commented ×3closed ×1cross-referenced ×1

Fix Action

Fixed

PR fix notes

PR #73070: fix(acpx): ignore Codex ACP timeout config

Description (problem / solution / changelog)

Summary

  • Ignore unsupported timeout / timeout_seconds ACP config pushes for Codex ACP sessions
  • Keep existing Codex model/thinking normalization behavior intact
  • Add regression coverage for the Codex ACP timeout config path

Fixes #73052

Validation

  • node scripts/run-vitest.mjs run --config test/vitest/vitest.extension-acpx.config.ts extensions/acpx/src/runtime.test.ts
  • pnpm exec oxfmt --check --threads=1 extensions/acpx/src/runtime.ts extensions/acpx/src/runtime.test.ts

Changed files

  • extensions/acpx/src/runtime.test.ts (modified, +31/-0)
  • extensions/acpx/src/runtime.ts (modified, +30/-26)
RAW_BUFFERClick to expand / collapse

When OpenClaw ACP runtime spawns Codex ACP with runTimeoutSeconds / timeoutSeconds, OpenClaw forwards the runtime timeout as ACP session/set_config_option key timeout. The current @zed-industries/codex-acp adapter rejects that key with Invalid params (ACP -32602), so the child task fails before running.

I prepared a small source fix locally on branch fix/acpx-codex-timeout-config at commit 9fbd9cc90a:

  • extensions/acpx/src/runtime.ts: for Codex ACP commands, no-op unsupported timeout / timeout_seconds config-option pushes.
  • extensions/acpx/src/runtime.test.ts: adds coverage that Codex ACP timeout controls are ignored.

Validation run locally:

  • node scripts/run-vitest.mjs run --config test/vitest/vitest.extension-acpx.config.ts extensions/acpx/src/runtime.test.ts — 21 tests passed.
  • pnpm exec oxfmt --check --threads=1 extensions/acpx/src/runtime.ts extensions/acpx/src/runtime.test.ts — passed.
  • pnpm tsgo:extensions:test — passed.

I can open a PR once write/fork auth is available.

extent analysis

TL;DR

The issue can be resolved by modifying the @zed-industries/codex-acp adapter to ignore unsupported timeout config options.

Guidance

  • The current @zed-industries/codex-acp adapter rejects the timeout key with an error, causing the child task to fail.
  • A potential fix is to no-op unsupported timeout / timeout_seconds config-option pushes in the extensions/acpx/src/runtime.ts file.
  • To verify the fix, run the validation tests using the commands provided, such as node scripts/run-vitest.mjs run --config test/vitest/vitest.extension-acpx.config.ts extensions/acpx/src/runtime.test.ts.
  • Consider opening a PR with the proposed fix once write/fork auth is available.

Example

No code snippet is provided as the issue already includes a proposed fix.

Notes

The proposed fix is specific to the @zed-industries/codex-acp adapter and may not apply to other adapters or configurations.

Recommendation

Apply the workaround by modifying the extensions/acpx/src/runtime.ts file to ignore unsupported timeout config options, as this is a targeted fix that addresses the specific error message and has been validated locally.

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…
#api#ssr#installation#tensor shape#autograd error

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING

openclaw - ✅(Solved) Fix Codex ACP rejects timeout config option during sessions_spawn [1 pull requests, 3 comments, 2 participants]