Skip to content

[AIGTWY-4564] Add Codex parent schema discovery - #541

Open
andy-xu-db wants to merge 10 commits into
mainfrom
aigtwy-4564-codex-parent-schema
Open

[AIGTWY-4564] Add Codex parent schema discovery#541
andy-xu-db wants to merge 10 commits into
mainfrom
aigtwy-4564-codex-parent-schema

Conversation

@andy-xu-db

@andy-xu-db andy-xu-db commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

🥞 Stacked PR

Use this link to review incremental changes.


Summary

  • add --parent <catalog>.<schema> to Codex launches
  • persist the model-service parent schema header in modern and legacy configs
  • replace stale provider and parent-schema headers on later launches

Testing

  • focused parent-schema suites (592 passed)
  • full non-external suite (2,298 passed, 38 skipped)
  • Ruff check and format check

e2e test with Liteswap

Screenshot 2026-09-09 at 7 04 24 PM

@andy-xu-db
andy-xu-db changed the base branch from aigtwy-4564-claude-parent-schema to main September 9, 2026 20:26
@andy-xu-db
andy-xu-db requested a review from lilly-luo September 9, 2026 23:07
@andy-xu-db
andy-xu-db force-pushed the aigtwy-4564-codex-parent-schema branch 2 times, most recently from 2348526 to 04e07ee Compare September 10, 2026 14:47
Comment thread src/ucode/agents/codex.py Outdated
Comment on lines +160 to +163
if provider:
http_headers["Databricks-Model-Provider-Service"] = provider
elif parent_schema:
http_headers[MODEL_SERVICE_PARENT_SCHEMA_HEADER] = parent_schema

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add comment. also would be nice to extract this into a util that can be shared by both codex + cc

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extracted into util file shared by codex + cc

@andy-xu-db
andy-xu-db force-pushed the aigtwy-4564-codex-parent-schema branch from b00a4b2 to ce55c11 Compare September 11, 2026 17:04
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.

2 participants