Skip to content

[codex] Handle sync timeout in CLI#1

Merged
chengzeyi merged 2 commits into
mainfrom
codex/sync-timeout-errors
Jun 16, 2026
Merged

[codex] Handle sync timeout in CLI#1
chengzeyi merged 2 commits into
mainfrom
codex/sync-timeout-errors

Conversation

@chengzeyi

Copy link
Copy Markdown
Member

What changed

  • Detect sync-mode timeout errors from the SDK/API in wavespeed run --sync.
  • Show a clear message that the task is still processing asynchronously.
  • Include the prediction ID, result URL, and wavespeed show <id> query command in human output when available.
  • Include sync_timeout, prediction_id, result_url, and query_command in --json error output.
  • Update README command help and align the lockfile package version with package.json.

Why

The API no longer converts sync timeout into an async success response. A sync timeout means the task was submitted and is still queryable, so CLI users need an actionable error instead of a generic failure.

User impact

Successful run() output remains unchanged. On sync timeout, the command exits non-zero but gives users the prediction ID and query path.

Validation

  • npm run lint
  • npm run build
  • npm run test:smoke
  • git diff --check

@chengzeyi chengzeyi marked this pull request as ready for review June 16, 2026 17:35
@chengzeyi chengzeyi merged commit 6978080 into main Jun 16, 2026
1 check passed
@chengzeyi chengzeyi deleted the codex/sync-timeout-errors branch June 16, 2026 17:35
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.

1 participant