Skip to content

Cover managed model discovery in Codex - #702

Open
andy-xu-db wants to merge 1 commit into
codex/model-discovery-policyfrom
andy-xu-db/stack/codex/codex-scoped-model-discovery
Open

andy-xu-db wants to merge 1 commit into
codex/model-discovery-policyfrom
andy-xu-db/stack/codex/codex-scoped-model-discovery

Conversation

@andy-xu-db

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

Copy link
Copy Markdown
Collaborator

Stack

Summary

  • Verify Codex inherits managed discovery 0 and 1 on every exec path.
  • Preserve provider and parent-schema catalogs.
  • Let explicit --model-location override a saved local provider.

Test

  • Combined stack: 2,383 tests passed, 41 skipped.
  • Ruff and format checks passed.

Tested against a workspace-managed Codex MPS.

Managed Codex MPS
Screen.Recording.2026-09-17.at.10.43.20.AM.mov

@lilly-luo

Copy link
Copy Markdown
Collaborator

can u add video of full e2e? define in managed config and then load up in codex

@andy-xu-db
andy-xu-db force-pushed the codex/model-discovery-policy branch from e767aec to de464fe Compare September 17, 2026 16:13
@andy-xu-db
andy-xu-db force-pushed the andy-xu-db/stack/codex/codex-scoped-model-discovery branch from 30c7e92 to c3081cd Compare September 17, 2026 16:13
@andy-xu-db
andy-xu-db force-pushed the andy-xu-db/stack/codex/codex-scoped-model-discovery branch from c3081cd to 4c92659 Compare September 17, 2026 18:51
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