Skip to content

fix(ci): align OpenAI auth contract with provider default#103760

Merged
steipete merged 1 commit into
mainfrom
codex/fix-openai-auth-contract
Jul 10, 2026
Merged

fix(ci): align OpenAI auth contract with provider default#103760
steipete merged 1 commit into
mainfrom
codex/fix-openai-auth-contract

Conversation

@steipete

Copy link
Copy Markdown
Contributor

Closes #103750

AI-assisted: Codex

What Problem This Solves

Resolves a release-validation regression where Plugin Prerelease failed six OpenAI ChatGPT/Codex OAuth contract assertions after fresh Codex setups moved from GPT-5.5 to GPT-5.6 Sol.

Why This Change Was Made

The shared provider-auth test contract now requires the owning extension to supply its expected Codex default model. The OpenAI contract passes OPENAI_CODEX_DEFAULT_MODEL, keeping the reusable identity/auth assertions aligned with the provider-owned default without duplicating a model literal in shared test infrastructure.

User Impact

No production behavior changes. Maintainers can run the OpenAI plugin prerelease and full release-validation lanes without a stale GPT-5.5 test expectation cancelling the remaining extension and Docker fanout.

Evidence

  • Before: Plugin Prerelease run 29101126635 failed checks-node-extensions-shard-7 with six identical openai/gpt-5.5 expected vs openai/gpt-5.6-sol received diffs: https://github.com/openclaw/openclaw/actions/runs/29101126635/job/86390028801
  • corepack pnpm test extensions/openai/provider-auth.contract.test.ts on Blacksmith Testbox lease tbx_01kx68cssa6v0czk2wge2aynss: 7/7 passed.
  • corepack pnpm test:extension openai on the same Testbox lease: 403/403 passed across 29 files.
  • corepack pnpm check:changed on the same Testbox lease: passed (core/extension typechecks, lint, and boundary guards).
  • .agents/skills/autoreview/scripts/autoreview --mode local: clean, no accepted/actionable findings.
  • Codex contract check at sibling Codex SHA 656a2d0905c9e0b9bdade1badab07ef6d42ca17c: codex-rs/models-manager/models.json places gpt-5.6-sol first; codex-rs/models-manager/src/manager.rs sorts by priority and marks the first picker-visible model as default through codex-rs/protocol/src/openai_models.rs.

@steipete
steipete force-pushed the codex/fix-openai-auth-contract branch from ee509fa to 2ef1647 Compare July 10, 2026 15:16
@steipete

Copy link
Copy Markdown
Contributor Author

Land-ready at exact head 2ef16479642af01142352ce39f491cd41fe7917e.

Before/fix proof:

  • Release failure: Plugin Prerelease run 29101126635, job 86390028801 failed six OpenAI auth contract cases only because shared expectations still named openai/gpt-5.5; production returned the provider-owned openai/gpt-5.6-sol default.
  • corepack pnpm test extensions/openai/provider-auth.contract.test.ts on Blacksmith Testbox lease tbx_01kx68cssa6v0czk2wge2aynss: 7/7 passed.
  • corepack pnpm test:extension openai on the same lease: 403/403 passed across 29 files.
  • corepack pnpm check:changed on the same lease: passed core/extension typechecks, lint, and boundary guards.
  • Fresh branch autoreview: clean, no accepted/actionable findings.
  • Exact PR CI: https://github.com/openclaw/openclaw/actions/runs/29103068075 completed successfully.
  • Repo-native OPENCLAW_TESTBOX=1 scripts/pr prepare-run 103760: passed hosted exact-head gates.

Contract check: sibling Codex SHA 656a2d0905c9e0b9bdade1badab07ef6d42ca17c has gpt-5.6-sol first in codex-rs/models-manager/models.json; codex-rs/models-manager/src/manager.rs sorts by priority and delegates default marking to codex-rs/protocol/src/openai_models.rs, where the first picker-visible model wins.

Known remaining proof: the canonical Plugin Prerelease lane will be rerun at the landed main SHA after merge.

@steipete
steipete merged commit 9b4c7c0 into main Jul 10, 2026
99 checks passed
@steipete
steipete deleted the codex/fix-openai-auth-contract branch July 10, 2026 15:28
@steipete

Copy link
Copy Markdown
Contributor Author

Merged via squash.

github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jul 11, 2026
steipete added a commit that referenced this pull request Jul 11, 2026
…faults for the stable cut (#104656)

* feat(openai): default new setups to GPT-5.6 (#103581)

* feat(openai): default fresh setup to GPT-5.6

* test(crestodian): expect GPT-5.6 Codex defaults

* test(crestodian): expect GPT-5.6 bootstrap default

(cherry picked from commit ab5d143)

* fix(infra): converge legacy state migrations on archive collisions

(cherry picked from commit 474d660)

* fix(codex): prevent startup hangs after binding migration (#103281)

* fix(codex): archive imported orphan binding sidecars

* fix(codex): harden binding sidecar migration

Co-authored-by: Benjamin Badejo <[email protected]>

* chore: keep release notes out of contributor PRs

---------

Co-authored-by: Benjamin Badejo <[email protected]>
Co-authored-by: Peter Steinberger <[email protected]>
(cherry picked from commit 1f3ea6f)

* fix(migrations): converge recoverable legacy state (#104529)

Signed-off-by: sallyom <[email protected]>
(cherry picked from commit cdb58ca)

* test(openai): align auth contract with provider default (#103760)

(cherry picked from commit 9b4c7c0)

* fix: clear remaining release validation blockers (#104555)

* test: align provider tool call lifecycles

* test: isolate plugin install repair migrations

* fix: accept catalog temperature compatibility

* fix: disable GPT-5.6 tool reasoning on completions

* style: avoid shadowed stream model

(cherry picked from commit 57af2bb)

* test(release): keep GPT-5.6 backport scoped

* test(release): restore doctor config type import

---------

Signed-off-by: sallyom <[email protected]>
Co-authored-by: Ayaan Zaidi <[email protected]>
Co-authored-by: Ben Badejo <[email protected]>
Co-authored-by: Benjamin Badejo <[email protected]>
Co-authored-by: Sally O'Malley <[email protected]>
steipete added a commit that referenced this pull request Jul 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Plugin prerelease auth contract still expects GPT-5.5 default

1 participant