Conversation
Prepare openai:gpt-6-sol (unannounced; community API sightings only) as a default-hidden known model so it can be enabled quickly at release. No aliases, no warm, no speculative thinking policy or pricing. Adds a gpt6SolModelHidden seed-once migration flag so existing configs hide it without re-hiding models users re-enabled after the Daybreak seed.
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Automations to automatically generate PRs for you. |
|
@codex review |
This comment has been minimized.
This comment has been minimized.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 01ca0537be
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
🛡️ Codex Security Review · Automatically triggered
Here are some automated security review suggestions for this pull request.
Reviewed commit: 01ca0537be
ℹ️ About Codex security reviews in GitHub
This is an experimental Codex feature. Security reviews are triggered when:
- You comment "@codex security review"
- A regular code review gets triggered (for example, "@codex review" or when a PR is opened), and you’re opted in so security review runs alongside code review
Once complete, Codex will leave suggestions, or a comment if no findings are found.
…tions Codex round-1 finding: getHigherContextCompactionSuggestion scanned every KNOWN_MODELS entry, so restricted/provisional default-hidden models (Daybreak, GPT-6 Sol) could be offered as Compact & retry targets. Skip DEFAULT_HIDDEN_MODELS in the automatic scan; explicit user-configured compaction models are unaffected.
Codex security round-1 finding: explicit $0 stats made getTotalCost return a defined $0, bypassing the CLI --budget unknown-pricing rejection and the goal-budget modelHasPricingData guard on a billable API model. Remove the provisional models-extra entry (null stats = unknown pricing = fail-closed budgets) and exempt provisional entries from the curated-coverage bar.
|
@codex review |
🛡️ Codex Security Review · Automatically triggeredSecurity review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cb5ac4b075
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
…n hidden-model seeding Codex round-2 findings: (1) a later seed flipped hiddenModelsInitialized just because an earlier seed had created the array, letting the browser migration drop legacy local hides — flip only on the first-ever seed pass; (2) seeding hid models the config already references (defaultModel, agent AI defaults, fallback chains) — explicit references now count as opt-ins and are skipped.
|
@codex review |
🛡️ Codex Security Review · Automatically triggeredSecurity review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
|
Review-gate bookkeeping: the Codex Review Summary card (node |
…zero rates) Advisor accuracy pass: modelHasPricingData already treats all-zero rates as unpriced, so only the CLI --budget unknown-pricing rejection was bypassed by a defined $0 total. Comment-only change.
…PT-5.6 Sol baseline Maintainer-authorized change: replace the null-stats fail-closed approach with an explicitly labeled PROVISIONAL ESTIMATE copied 1:1 (by reference) from GPT_56_SOL_STATS, the repo's effective gpt-5.6-sol baseline. Budgets now enforce against estimated rates instead of rejecting the model; if real pricing lands higher, spend is under-counted until the entry is corrected (required before-undraft checklist item). Reverts the now-unneeded curated-coverage and Treat-as catalog exemptions so those checks return to full strictness, and pins estimate-baseline equality in the registry test.
1 participant
Important
DRAFT — do not merge.
gpt-6-solis NOT an officially announced OpenAI model. This PR is deliberate advance preparation. Its pricing is an explicitly PROVISIONAL ESTIMATE (maintainer-authorized), not official data; see the before-undraft checklist.Summary
Adds
openai:gpt-6-solto the curated known-model registry as a provisional, default-hidden entry so Xum can enable it quickly if OpenAI officially releases it. The model is hidden in the selector, has no aliases, is not warmed, gets the conservative default thinking policy, and is excluded from automatic compaction suggestions. Its stats (limits + pricing) are a maintainer-authorized provisional estimate copied 1:1 from the repo's effective GPT-5.6 Sol baseline.Background
Lead: an X post claiming Opus 5.1 and GPT-6 Sol are imminent (not official), plus community reports that the id
gpt-6-solappeared on the OpenAI API (Reddit r/singularity, HN 49665088). Verification, calibrated to the sources checked (OpenAI announcement pages, developer-docs search results, news coverage) as of 2026-09-15:gpt-6-astra) was announced Sept 3, 2026 and is already integrated (🤖 feat: add first-class support for OpenAI GPT-6 Astra #4064).gpt-6-sol.Implementation
Follows the default-hidden pattern from the Daybreak models (#4113) rather than the first-class pattern used for Astra (#4064), because Astra's PR encoded confirmed capabilities and none exist here.
knownModels.ts:GPT_6_SOLentry — no aliases (solstays on GPT-5.6 Sol), not warmed,openai/gpt-5tokenizer approximation, ordered after GPT-5.6 Sol/Astra; added toDEFAULT_HIDDEN_MODELS.config/index.ts+appConfigOnDisk.ts: new seed-once migration flaggpt6SolModelHidden; the Daybreak seed block is generalized into per-flag seeds (flags are frozen history — a flag that already ran never re-hides its models). The seed preserves the pending local-preference migration marker (hiddenModelsInitializedflips only on the first-ever seed pass) and treats explicit config references (default model, advisor model, agent AI defaults, fallback chains) as prior opt-ins that are never hidden.models-extra.ts): pricing and token limits copied by reference fromGPT_56_SOL_STATS— the repo's effective gpt-5.6-sol baseline ($5/M in, $30/M out, 1.05M context, 272K long-context tier at 2x in / 1.5x out, cache costs). The models-extra override wins over LiteLLM's stalemodels.jsonrow ($4/$20, 922K) at runtime, so the effective baseline is unambiguous; the discrepancy is upstream staleness, already documented in the baseline's comment. Capability flags and knowledge cutoff are deliberately omitted (nothing official is published). A registry test pins estimate ↔ baseline equality so divergence must be a conscious decision.--budgetenforces the cap against estimated rates. If real pricing lands higher (Astra bills 2x Sol), spend is under-counted until the entry is corrected — hence the required checklist item below. This supersedes the interim null-stats fail-closed approach from the review loop.compaction/suggestion.ts: automatic higher-context suggestions skip everyDEFAULT_HIDDEN_MODELSentry (also closes the same pre-existing exposure for the Daybreak tiers); explicit user-configured compaction models are unaffected.gpt-6-solfalls through to the default["off","low","medium","high"]policy; tests pin that it does not inherit Astra's or the GPT-5.6 family's native-max/no-off surfaces.Before-undraft checklist (REQUIRED)
Do not mark this PR ready for review/merge until every item is done against official OpenAI documentation:
models-extra.tswith the official model-page rates (incl. cached-input, cache-write, long-context tier and threshold) and update the baseline-equality pin inknownModels.test.tsaccordingly.gpt-6-sol(currently based on community API sightings only); rename/remove the entry if it differs.max? rejectsnone? compare Astra'sopenaiRejectsDisabledReasoning),reasoning.mode: "pro", tool calling on Chat Completions vs Responses-only, vision/PDF, knowledge cutoff, tokenizer (currently approximated withopenai/gpt-5).sol/gptstaying on GPT-5.6 Sol vs moving) — deliberate product decision, not part of this PR.DEFAULT_HIDDEN_MODELSfor new installs (existing users keep their visibility choices; the seed flag never re-runs).Validation
make static-checkgreen on every pushed head (lint, typecheck, fmt, docs sync, generated content).Test / UnitCI failure (listModelCatalogIdsrequired every curated id to resolve stats) is fixed by the stats entry itself — no checks were weakened: the interim coverage/catalog exemptions were reverted and both checks are back to full strictness.Risks
The model is default-hidden and alias-free, so no default flow changes. Cost tracking/budget enforcement for this specific model relies on an estimate that may understate real spend (documented above and in code; accepted trade-off per maintainer authorization). The hidden-model seed migration is covered by the pre-existing suite plus new frozen-history/marker/opt-in tests.
Review record (cap exhausted — no further cycles)
Instruction budget: six completed code/security reviews plus one independent advisor pass — now exhausted. No further review or advisor cycles will be requested on this draft.
01ca053--budget252bba9,cb5ac4b; threads replied + resolvedcb5ac4bhiddenModelsInitialized(legacy local hides droppable); P2: seed hides models the config already references64169a3; threads replied + resolved64169a3f04892f); scope decision stays open at releaseUnreviewed post-cap commits (maintainer-authorized):
f04892f(comment accuracy only) and782cca6(provisional pricing estimate replacing the null-stats approach).782cca6intentionally supersedes part of the round-1 security fix: the security review recommended treating unpublished pricing as unknown or rejecting the model under budgets; the maintainer explicitly authorized an estimate instead. This finding is therefore resolved by authorized trade-off, not by code, and remains listed below until official pricing lands. Codex verdicts are bound to64169a3.Unresolved assumptions / open items
gpt-6-solexistence, specs, pricing, capabilities, and availability are entirely unconfirmed (see checklist).Generated with
xum• Model:anthropic:claude-fable-5• Thinking:xhigh