Skip to content

fix(sdk): handle Codex MCP elicitation requests - #162

Merged
ross-rl merged 5 commits into
mainfrom
codex/mcp-elicitation-support
Aug 13, 2026
Merged

fix(sdk): handle Codex MCP elicitation requests#162
ross-rl merged 5 commits into
mainfrom
codex/mcp-elicitation-support

Conversation

@ross-rl

@ross-rl ross-rl commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • classify and route mcpServer/elicitation/request as a typed Codex server request
  • safely cancel unattended elicitations and round-trip structured handler responses
  • treat serverRequest/resolved as terminal during live handling and Axon replay
  • expose an abort signal so applications can clear parked UI state
  • document the recovery boundary: client reconnect is replay-safe while Codex lives, but Codex process restart requires durably reissuing the operation

Reproduction trace

Built against the deterministic broker functional trace in runloopai/runloop#10790. The trace directly invokes an MCP tool whose server sends elicitation/create, producing:

{"method":"mcpServer/elicitation/request","id":0,"params":{"mode":"form","message":"Which city should the test use?","requestedSchema":{"type":"object","properties":{"city":{"type":"string"}},"required":["city"]}}}

Tests

  • bun run check
  • bun run typecheck
  • bun run build
  • bun run test (25 files, 586 tests)

@ross-rl ross-rl changed the title fix(codex): handle MCP elicitation requests fix(sdk): handle Codex MCP elicitation requests Aug 11, 2026
@ross-rl
ross-rl merged commit 3d908ad into main Aug 13, 2026
5 checks passed
@ross-rl
ross-rl deleted the codex/mcp-elicitation-support branch August 13, 2026 18:38
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