Skip to content

fix: align app-server capabilities and thread params with the current codex protocol#389

Closed
armanobosyan wants to merge 1 commit into
openai:mainfrom
armanobosyan:fix/codex-0141-protocol
Closed

fix: align app-server capabilities and thread params with the current codex protocol#389
armanobosyan wants to merge 1 commit into
openai:mainfrom
armanobosyan:fix/codex-0141-protocol

Conversation

@armanobosyan

Copy link
Copy Markdown

Summary

  • npm run build (the tsc check against the generated app-server types) currently fails on main with the latest codex CLI (0.141.0): InitializeCapabilities now requires requestAttestation, and experimentalRawEvents has been removed from thread/start params.
  • Declares requestAttestation: false in the default capabilities and stops sending the removed experimentalRawEvents field, so the typecheck passes against the current protocol again.

Testing

  • npm run build
  • node --test tests/runtime.test.mjs

The generated app-server types now require the requestAttestation capability
and have dropped experimentalRawEvents from thread/start params, so
`npm run build` (tsc) fails against the current codex CLI (0.141.0). Declare
requestAttestation in the default capabilities and stop sending the removed
experimentalRawEvents field.
@armanobosyan

Copy link
Copy Markdown
Author

Closing as redundant: main (v1.0.5) already declares requestAttestation in the default capabilities and drops the removed experimentalRawEvents from thread/start params (via #374), so the typecheck passes. The stalled-turn fix that depended on this is in #390 (now rebased onto v1.0.5 and standalone).

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