Skip to content

Remove optional provider configuration discovery - #73

Merged
SaladDay merged 1 commit into
mainfrom
codex/remove-configuration-discovery
Sep 23, 2026
Merged

SaladDay merged 1 commit into
mainfrom
codex/remove-configuration-discovery

Conversation

@SaladDay

@SaladDay SaladDay commented Sep 23, 2026 •

Copy link
Copy Markdown
Collaborator

Remove optional provider-configuration discovery. Startup configuration returns its original four-field snapshot and rejects query parameters; the client opt-in, public schema and unused Runtime provider descriptors are removed. Internal provider validation, Agent defaults and frozen Session execution-configuration reads remain unchanged.

The startup route remains an explicit Core extension. Official documentation and read-only API probes confirmed that OpenAI does not expose this Core route; no replacement discovery API or unrelated protocol changes are introduced. Old Runtime heartbeats with the retired field remain accepted.

Validation: regenerated OpenAPI, identical sqlc output, affected and broad Go/PostgreSQL checks, standalone builds, all 394 client tests, 715 Web unit tests, TypeScript, Claude SDK packaging, MiniMax and Rust checks. Independent blind review found no actionable issues. Browser acceptance on isolated ports passed 123 of 127 cases; the four failures match clean-baseline Dashboard failures.

The full gate is not green: clean baseline 3fb5f87 independently reproduces the KVM-dependent installer failure and cascading fixture-directory error, plus four existing Dashboard browser failures. Initial local toolchain, test-environment and occupied-port issues were corrected without source changes. Production services are unchanged.

@SaladDay
SaladDay marked this pull request as ready for review September 23, 2026 17:34
@SaladDay
SaladDay merged commit 91ca260 into main Sep 23, 2026
2 of 3 checks passed
@SaladDay
SaladDay deleted the codex/remove-configuration-discovery branch October 7, 2026 06: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