Skip to content

feat(models): put the GPT-5.6 Pro tiers in the picker - #460

Merged
VickyXAI merged 1 commit into
mainfrom
feat/top-models-gpt56-pro-chat-latest
Oct 13, 2026
Merged

VickyXAI merged 1 commit into
mainfrom
feat/top-models-gpt56-pro-chat-latest

Conversation

@VickyXAI

@VickyXAI VickyXAI commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Adds openai/gpt-5.6-terra-pro, openai/gpt-5.6-sol-pro and openai/gpt-5.6-luna-pro to src/top-models.json, right after their base tiers (same terra → sol → luna order).
  • All three have been in BLOCKRUN_MODELS since 2026-08-03 (so pricing and the maxCostPerRun estimate already work) and are advertised by https://blockrun.ai/api/v1/models (checked 2026-10-13). They just never showed in the picker, unlike gpt-5.5-pro / gpt-5.4-pro.
  • openai/chat-latest is left out on purpose: models.test.ts asserts it stays out of the picker because a rolling alias is a pin-on-purpose model.
  • dist/ rebuilt, because tsup inlines top-models.json.

Motivation: BlockRunAI/ClawRouter-Hermes#48 wanted these in the Hermes picker, which mirrors this file in both membership and order. Once this lands, Hermes can follow.

Test plan

  • npm test: 1205 passed, 1 skipped (top-models.test.ts now asserts the three Pro tiers)
  • npm run typecheck, npm run lint, prettier check
  • npm run build, and the dist smoke check passes

Summary by CodeRabbit

  • New Features
    • Added three GPT-5.6 Pro models to the top-model list.

openai/gpt-5.6-terra-pro, -sol-pro and -luna-pro have been in
BLOCKRUN_MODELS since 2026-08-03 and are advertised by the live gateway
catalog, but were never in top-models.json, so they never showed in the
picker. gpt-5.5-pro and gpt-5.4-pro are already listed, so the 5.6 Pro
tiers go right after their base tiers in the same terra/sol/luna order.

openai/chat-latest is still left out on purpose: models.test.ts keeps
rolling aliases as pin-on-purpose models, not curated picks.

dist rebuilt because tsup inlines top-models.json.
@coderabbitai

coderabbitai Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: BlockRunAI/ClawRouter/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 12156ec6-cac9-49de-94b7-e98b72f33da4
📥 Commits

Reviewing files that changed from the base of the PR and between 147d1b9 and 7de198f.

⛔ Files ignored due to path filters: 4
  • dist/cli.js is excluded by !**/dist/**
  • dist/cli.js.map is excluded by !**/dist/**, !**/*.map
  • dist/index.js is excluded by !**/dist/**
  • dist/index.js.map is excluded by !**/dist/**, !**/*.map
📒 Files selected for processing: 2
  • src/top-models.json
  • src/top-models.test.ts

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 7 remain after this review.

📝 Walkthrough

The top-model list adds three OpenAI GPT-5.6 Pro model identifiers. The TOP_MODELS test adds membership expectations for those identifiers.

Changes: GPT-5.6 Pro model catalog

Layer / File(s) Summary
Model entries and membership checks
src/top-models.json, src/top-models.test.ts
The top-model list adds the Terra, Sol, and Luna Pro model identifiers. The test checks that all three identifiers belong to TOP_MODELS and adds a comment about their catalog and picker status.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Feature

Merge Risk: ⚪ Minimal · up to 7de19

The picker now includes all three supported GPT-5.6 Pro tiers, and the packaged runtime bundle contains them. No merge-blocking risk is evident.

Pre-merge checks: Passed 5
Passed Passed checks: 5 passed
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes adding the GPT-5.6 Pro tiers to the model picker, which is the main change.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Warning

Some tools did not complete. Review the errors below.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install failed due to a network error.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@VickyXAI
VickyXAI merged commit 5d6cd13 into main Oct 13, 2026
5 checks passed
VickyXAI pushed a commit that referenced this pull request Oct 13, 2026
Ships #460: openai/gpt-5.6-terra-pro, -sol-pro and -luna-pro are now
picker-visible, right after their base tiers.

Also corrects the GPT-5.6 Pro comment in models.ts: it said Terra Pro is
half the Terra rate, but both gateways price every Pro tier the same as
its base tier (checked 2026-10-13). The registry prices were already right.
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