feat(models): support Astra API and Latest browser selection - #465
Conversation
|
🦞👀 Pull request received. I will update this pull request when review starts. ClawSweeper review completeClawSweeper finished reviewing this revision. The review result is being finalized. |
|
Codex review: needs maintainer review before merge. Reviewed September 7, 2026, 10:03 PM ET / September 8, 2026, 02:03 UTC. ClawSweeper reviewWhat this changesAdds Astra API support and verified ChatGPT Latest/Pro browser selection, with model-specific reasoning validation, documentation, and CLI/MCP regression coverage. Merge readiness✅ Ready for maintainer review This PR remains useful: current main lacks the combined Astra support. The new maintainer evidence resolves the previous API proof blocker, and no blocking patch defects remain. Likely related people: steipete (high-confidence routing); enki (low-confidence routing). Priority: P2 Review scores
Verification
How this fits togetherOracle sends prompts and files through either a provider API or a signed-in browser. Model resolution translates CLI/MCP choices into API request settings or ChatGPT model and effort selections before consultation. flowchart TD
A[CLI or MCP request] --> B[Model and engine resolution]
C[Saved configuration] --> B
B --> D[API reasoning validation]
B --> E[Browser model and effort selection]
D --> F[OpenAI Responses API]
E --> G[Signed-in ChatGPT]
F --> H[Consultation result]
G --> H
Before mergeNone. Agent review detailsSecurityNone. Review metrics
Root-cause clusterRelationship: Members:
Proposal only: this assessment does not dispatch repair, suppress jobs, mutate sibling items, close, or merge anything. Technical reviewBest possible solution: Keep Astra support within Oracle's existing API and browser pipelines, preserving saved preferences, explicit Pro intent, and verified selection evidence. Do we have a high-confidence way to reproduce the issue? Not applicable as a feature request; source confirms the missing main-branch support, and the maintainer provides concrete after-change API and browser observations. Is this the best way to solve the issue? Yes. The patch extends existing model resolution, request validation, and browser selectors without creating a competing implementation or replacing existing defaults. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning medium; reviewed against fe2c8d990562. LabelsLabel changes:
Label justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (6 earlier review cycles)
|
|
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. |
|
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. |
|
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. |
|
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. |
Preserve browser-only Pro aliases through CLI engine discovery, retain model-specific API capabilities, and verify localized Latest and effort controls. Combine the compatible work from steipete#448, steipete#449, and steipete#465; release notes are deferred to the final notes PR.
|
Maintainer proof for
No default model changed. Contributor credit and the deferred release note are preserved in the PR body/final notes change. |
Support GPT-6 Astra through the first-party API and ChatGPT's Latest picker. The browser-only
gpt-6-proalias now reaches Latest plus verified Pro effort instead of failing during CLI route preflight; API requests usegpt-6-astrawith explicit reasoning mode.API validation uses the model's supported effort set, rejects
none(including saved defaults), and permits an explicit supported effort to override an invalid saved default. GPT-5.6 behavior and the default model are preserved. Browser selection recognizes exact Latest/最新 radios and the observed version-prefixed effort controls while keeping Pro selection fail-closed.This consolidates the compatible API work from #449 and the Latest/localization work from #448. Credit belongs to @FNDEVVE, @oraclexing, @malvarezcastillo, and @kiyo-e. Their overlapping PRs can close after this one lands.
Validation at
ecc104ec0435a1221053c3716947468a832630fd:noneeffort failed before inference; an explicit low effort completed through that saved configuration.status=switchedand completed. The normal High and Pro runs used the unmodified built CLI.Browser evidence describes observed UI selection, not independent backend model attestation. The current API contract was checked against https://developers.openai.com/api/docs/models/gpt-6-astra.
The complete changelog entry and contributor thanks are deferred to the final notes PR; this implementation has no CHANGELOG.md delta against main.