[feat] Refresh model catalogs and preserve provider names - #6640
Open
mmabrouk wants to merge 2 commits into
Open
[feat] Refresh model catalogs and preserve provider names#6640mmabrouk wants to merge 2 commits into
mmabrouk wants to merge 2 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
|
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Team Run ID: ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (26)
Included review availability: Your plan provides up to 8 included reviews per hour; 5 remain after this review. 📝 SummarySummary by CodeRabbit
WalkthroughThe change refreshes model catalogs and Pi version pins. Provider discovery now returns display names. Web connection models preserve, search, render, and serialize those names. ChangesModel catalog refresh and runtime pins
Provider discovery names
Connection persistence and model UI
Estimated code review effort: 4 (Complex) | ~45 minutes Merge Risk: ⚪ Minimal · up to This updates model catalogs and preserves provider-supplied display names through settings and model selection. The changes are covered across the affected layers and are ready to merge. Sequence Diagram(s)sequenceDiagram
participant ProviderConnectionCard
participant probeProvider
participant ProviderAdapter
participant ConnectionPayload
ProviderConnectionCard->>probeProvider: request provider discovery
probeProvider->>ProviderAdapter: fetch model catalog
ProviderAdapter-->>probeProvider: return models and model_names
probeProvider-->>ProviderConnectionCard: provide available names
ProviderConnectionCard->>ConnectionPayload: save selected models with extras.name
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 8.70% which is insufficient. The required threshold is 60.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 23 functions across 21 files. (5 skipped: 5 unsupported.)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
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. Comment |
mmabrouk
marked this pull request as ready for review
September 7, 2026 19:24
Contributor
Railway Preview Environment
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Context
Astra and other current models were absent from the Pi, Codex, OpenAI, Anthropic, and OpenRouter choices. Provider refreshes also discarded human-readable model names, so newly discovered OpenRouter models appeared as raw slugs.
Changes
This updates the pinned Pi package to 0.85.1, regenerates its 512-model catalog, and adds GPT-6 Astra to the Pi subscription, Codex, and OpenAI selections. Anthropic defaults now use its current four-model lineup.
OpenRouter now exposes and preselects exactly the 10 most-used tool-capable text models returned by
GET /api/v1/models?sort=most-popular. The canonical list and recommended defaults use the same ordered top 10.Provider probes now return a backward-compatible
model_namesmap alongside model ids. OpenRoutername, Anthropicdisplay_name, and GeminidisplayNamevalues are saved with selected models and shown in provider settings and the model picker after reload.The
sync-model-catalogskill now documents the complete refresh workflow. Its audit script compares both checked-in OpenRouter lists with the live usage ranking and defaults to 10 for each.Sources:
Tests / notes
What to QA
Astraand be selectable.A browser recording is outstanding because this branch has not been deployed to a QA environment.