Skip to content

Delegate readiness waits to tool runtimes - #36120

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/8718589fa06f14d053a35a288186613f649d25ff
Jul 30, 2026
Merged

Delegate readiness waits to tool runtimes#36120
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/8718589fa06f14d053a35a288186613f649d25ff

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 30, 2026

Copy link
Copy Markdown

Delegate readiness waits to tool runtimes

What changed

  • Add an optional wait_until_ready hook to CoreToolRuntime and invoke it
    before acquiring the tool execution gate.
  • Move MCP refresh and server-startup waiting into the MCP runtime's hook.
  • Forward readiness waits through tool exposure and namespace wrappers.
  • Test exact runtime selection for plain and namespaced tools, including
    exposure overrides and missing tools.

## What changed

- Add an optional `wait_until_ready` hook to `CoreToolRuntime` and invoke it
  before acquiring the tool execution gate.
- Move MCP refresh and server-startup waiting into the MCP runtime's hook.
- Forward readiness waits through tool exposure and namespace wrappers.
- Test exact runtime selection for plain and namespaced tools, including
  exposure overrides and missing tools.

GitOrigin-RevId: 8718589fa06f14d053a35a288186613f649d25ff
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/8718589fa06f14d053a35a288186613f649d25ff branch from 2785da5 to 9a46fd3 Compare July 30, 2026 09:26
@copyberry
copyberry Bot merged commit 9a46fd3 into main Jul 30, 2026
31 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/8718589fa06f14d053a35a288186613f649d25ff branch July 30, 2026 09:27
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 30, 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.

1 participant