Skip to content

[#999] Add GPT-5.6 Sol/Terra/Luna to codex model dropdown#1000

Merged
realproject7 merged 1 commit into
mainfrom
task/999-gpt56-models
Jul 13, 2026
Merged

[#999] Add GPT-5.6 Sol/Terra/Luna to codex model dropdown#1000
realproject7 merged 1 commit into
mainfrom
task/999-gpt56-models

Conversation

@realproject7

Copy link
Copy Markdown
Owner

Closes #999

EPIC Alignment

Self-Verification

Deviations

none

🤖 Generated with Claude Code

Append gpt-5.6-sol/terra/luna to MODEL_OPTIONS.codex after the existing
concrete rows so modelsForBackend("codex")[0] stays "gpt-5.4" (#931
default/heal target). Pure data addition; effectiveModel/sanitizeModel
already handle any concrete slug. Adds a test asserting optionsForBackend(
"codex") includes the 3 new slugs.

Closes #999

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@realproject7

Copy link
Copy Markdown
Owner Author

@re2 APPROVE at c29d5ade690e9a75454c8dbd021be676c15ede4c

Pure data addition per #999, behavior-preserving for existing agents. Standard QuadWork flow (standalone, not EPIC #967).

Checked (evidence)

No concerns. Both-approval + CI-green gate met on the review side; @Head merges per standard rule.

@project7-interns project7-interns left a comment

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.

Verdict: APPROVE

Epic Alignment: PASS

Standalone #999 contract is met: only MODEL_OPTIONS.codex gains the three GPT-5.6 slugs, labels match values, and modelsForBackend("codex")[0] remains gpt-5.4.

Checked (evidence)

  • Structural gate: PR body has filled ## EPIC Alignment and ## Self-Verification; UI design table not required for this data-only dropdown option change.
  • Issue context: gh issue view 999 → standalone, parent none; contract requires src/lib/agentModels.ts codex-only append, no reorder, no logic/spawn/UI refactors.
  • Scope: gh pr diff 1000 / REST files → exactly src/lib/agentModels.ts (+6) and server/agentModels.test.js (+1), no deletions.
  • Model rows: src/lib/agentModels.ts:19, src/lib/agentModels.ts:20, src/lib/agentModels.ts:21 add gpt-5.6-sol, gpt-5.6-terra, gpt-5.6-luna with label == value after existing concrete codex rows.
  • Ordering invariant: server/agentModels.test.js:39 still pins modelsForBackend("codex")[0] === "gpt-5.4".
  • New coverage: server/agentModels.test.js:34 asserts optionsForBackend("codex") includes all three new slugs.
  • Consumer path: src/components/AgentModelsWidget.tsx:25 and src/components/AgentModelsWidget.tsx:248 use optionsForBackend; src/components/SettingsPage.tsx:6 imports shared helpers from @/lib/agentModels.
  • Riskiest part of this diff: stale duplicate model catalogs would make the UI miss the new slugs; checked PR head consumers and they read the shared helper, so acceptable.
  • Kill-list: scanned all items — clean.
  • CI: gh pr checks 1000test pass 51s.

Findings

None.

Decision

Approve. The PR is the requested data-only addition with focused test coverage and no behavior-changing logic outside the model catalog.

@realproject7
realproject7 merged commit f280d57 into main Jul 13, 2026
1 check passed
This was referenced Jul 13, 2026
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.

Add GPT-5.6 Sol/Terra/Luna to the codex model dropdown

2 participants