The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Jun 16, 2026 6:14pm

PR Summary

Low Risk
Display-only reordering of dropdown options with no inference, auth, or persistence changes; behavior is covered by new unit tests.

Overview
Model comboboxes that use getModelOptions (agent, evaluator, router, guardrails, translate, etc.) now list catalog base models with newest releaseDate first inside each provider, without changing which provider group appears first.

Adds orderModelIdsByReleaseDate in models.ts: a one-time MODEL_CATALOG_INDEX from static PROVIDER_DEFINITIONS, then groups IDs by provider in first-seen input order, sorts each group by release date (undated models keep declaration order after dated ones), and leaves unknown / dynamic IDs at the end unchanged. getModelOptions runs this on providers.base.models only; Ollama, OpenRouter, and other runtime lists are untouched.

Unit tests cover grouping, newest-first ordering, cross-provider order, unknown IDs, case insensitivity, and stable ID sets.

^{Reviewed by Cursor Bugbot for commit 7ee4d97. Configure here.}

Greptile Summary

This PR adds a orderModelIdsByReleaseDate helper to models.ts and applies it in getModelOptions so that all model dropdowns (agent, evaluator, router, guardrails, translate) show models newest-first within each provider group. The MODEL_CATALOG_INDEX map is built once at module load from the static PROVIDER_DEFINITIONS constant, avoiding any per-call iteration overhead.

• models.ts: Introduces MODEL_CATALOG_INDEX (module-scoped) and orderModelIdsByReleaseDate, which groups input IDs by provider (preserving first-seen provider order), sorts each group descending by releaseTime with a declIndex tiebreaker for undated/same-date models, and appends unknown/dynamic IDs at the end.
• blocks/utils.ts: Single-line change wrapping base.models with orderModelIdsByReleaseDate in getModelOptions.
• models.test.ts: New test file covering provider grouping, newest-first ordering, unknown IDs, case-insensitivity, empty input, and no-add/no-drop invariants.

Confidence Score: 5/5

Safe to merge — the change is a pure display-ordering helper with no effect on stored state, execution, or dynamic providers.

The sorting logic is correct: Number.NEGATIVE_INFINITY as the sentinel for undated models means the NEGATIVE_INFINITY !== NEGATIVE_INFINITY branch is never taken (falls through to the declIndex tiebreaker, avoiding any NaN in the comparator). MODEL_CATALOG_INDEX is built once at module load. The only affected path is the model dropdown render in getModelOptions, which is a read-only, side-effect-free operation. Tests cover all documented invariants.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["getModelOptions()"] --> B["providersState.providers.base.models\n(string[])"]
    B --> C["orderModelIdsByReleaseDate(modelIds)"]
    C --> D{"For each ID:\nlookup MODEL_CATALOG_INDEX"}
    D -- "Found" --> E["Group by providerId\n(Map, insertion-order preserved)"]
    D -- "Not found" --> F["unknown[] — dynamic provider IDs"]
    E --> G["Sort each bucket:\nnewer releaseTime first\ndeclIndex tiebreaker"]
    G --> H["ordered: string[]"]
    F --> I["Append unknown IDs at end"]
    H --> I
    I --> J["allModels = Set([...baseModels, ...ollamaModels, ...])"]
    J --> K["Return model options for dropdown"]

    M["MODULE LOAD\nMODEL_CATALOG_INDEX\nbuilt once from\nPROVIDER_DEFINITIONS"] -.-> D

%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A["getModelOptions()"] --> B["providersState.providers.base.models\n(string[])"]
    B --> C["orderModelIdsByReleaseDate(modelIds)"]
    C --> D{"For each ID:\nlookup MODEL_CATALOG_INDEX"}
    D -- "Found" --> E["Group by providerId\n(Map, insertion-order preserved)"]
    D -- "Not found" --> F["unknown[] — dynamic provider IDs"]
    E --> G["Sort each bucket:\nnewer releaseTime first\ndeclIndex tiebreaker"]
    G --> H["ordered: string[]"]
    F --> I["Append unknown IDs at end"]
    H --> I
    I --> J["allModels = Set([...baseModels, ...ollamaModels, ...])"]
    J --> K["Return model options for dropdown"]

    M["MODULE LOAD\nMODEL_CATALOG_INDEX\nbuilt once from\nPROVIDER_DEFINITIONS"] -.-> D

_{Reviews (2): Last reviewed commit: "fix(models): preserve input provider ord..." | Re-trigger Greptile}

@greptile

@cursor review