Skip to content

Conversation

tomasgvivo
Copy link

Changes

  • Added the optional final_output_json_schema field to the app-server protocol’s user-turn payload so callers can supply
    a JSON schema with each turn (app-server-protocol/src/protocol.rs:543).
  • Plumbed the new field through the app server so the conversation submission passes the schema down to core when
    present (app-server/src/codex_message_processor.rs:1008).
  • Expanded the send-user-turn flow tests to populate the new field explicitly, keeping existing behavioural
    assertions intact (app-server/tests/suite/codex_message_processor_flow.rs:337, app-server/tests/suite/
    codex_message_processor_flow.rs:470, app-server/tests/suite/codex_message_processor_flow.rs:506).
  • Added focused protocol serialization tests that confirm the schema is emitted only when provided (app-server-protocol/
    src/protocol.rs:935, app-server-protocol/src/protocol.rs:965).

Copy link

github-actions bot commented Oct 15, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@tomasgvivo
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Oct 15, 2025
@tomasgvivo
Copy link
Author

recheck

@tomasgvivo tomasgvivo marked this pull request as draft October 15, 2025 21:15
@tomasgvivo tomasgvivo marked this pull request as ready for review October 15, 2025 22:01
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