Skip to content

Clarify exec yield timing on Windows - #33150

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/32f90b3a56604fcf1734cc81ba7ab2f0bec41643
Jul 14, 2026
Merged

Clarify exec yield timing on Windows#33150
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/32f90b3a56604fcf1734cc81ba7ab2f0bec41643

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 14, 2026

Copy link
Copy Markdown

Clarify exec yield timing on Windows

Why

The generic yield_time_ms description does not explain the Windows-specific
initial wait behavior or when callers should choose a shorter value.

What changed

  • Describe yield_time_ms on Windows as the maximum wait before a running
    command returns a session ID, while commands that finish sooner still return
    immediately.
  • Recommend the 10-second default for ordinary commands and shorter waits for
    intentionally long-lived or interactive processes.
  • Keep the existing parameter description on other platforms.

Testing

  • Update the expected exec_command tool schema for both platform variants.

## Why

The generic `yield_time_ms` description does not explain the Windows-specific
initial wait behavior or when callers should choose a shorter value.

## What changed

- Describe `yield_time_ms` on Windows as the maximum wait before a running
  command returns a session ID, while commands that finish sooner still return
  immediately.
- Recommend the 10-second default for ordinary commands and shorter waits for
  intentionally long-lived or interactive processes.
- Keep the existing parameter description on other platforms.

## Testing

- Update the expected `exec_command` tool schema for both platform variants.

GitOrigin-RevId: 32f90b3a56604fcf1734cc81ba7ab2f0bec41643
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/32f90b3a56604fcf1734cc81ba7ab2f0bec41643 branch from 2bbe70d to 4da39db Compare July 14, 2026 20:44
@copyberry
copyberry Bot merged commit 4da39db into main Jul 14, 2026
14 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/32f90b3a56604fcf1734cc81ba7ab2f0bec41643 branch July 14, 2026 20:45
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants