docs(openrouter-models): sync with openrouter-web changes#63
docs(openrouter-models): sync with openrouter-web changes#63devin-ai-integration[bot] wants to merge 2 commits into
Conversation
…ptions The GET /models API now supports two new server-side sort values: - intelligence-high-to-low (Artificial Analysis intelligence index) - design-arena-elo-high-to-low (best Design Arena ELO across arenas) Updates: - list-models.ts: switch from client-side to server-side sorting via the sort query parameter, add intelligence and design-arena-elo flags - SKILL.md: document new sort options in decision tree, sort section, and API query parameters table - README.md: mention new sort options in feature list Source: OpenRouterTeam/openrouter-web#25600 Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Original prompt from Automation
|
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
There was a problem hiding this comment.
Perry's Review
Syncs list-models.ts to serve sort options server-side and adds two new sort flags (intelligence, design-arena-elo) matching the openrouter-web#25600 sort parameter additions.
Verdict: ✅ LGTM
Details
Risk: 🟢 Low — CLI tool only; additive sort options, existing sort keys all preserved with same semantics
CI: no required CI checks
Findings (see inline comment for full context):
- 🟡 skills/openrouter-models/scripts/list-models.ts:39 — sort validated after API fetch, wastes a round-trip on unknown flags
Security: no concerns — no credentials, auth, or data mutations touched
Test coverage: no tests exist for this CLI script; the change is low-risk (read-only, additive)
Unresolved threads: none
Scope: first review (full)
Review: tier=small · model=claude-sonnet-latest · score=?
Move unknown-sort guard before fetchApi to avoid a wasted round-trip on invalid flags. Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Perry's Review
Moves sort validation before the API call (implementing Perry's prior suggestion), and adds --sort intelligence and --sort design-arena-elo flags to list-models.ts with matching SKILL.md docs.
Verdict: ✅ LGTM
Details
Risk: 🟢 Low
CI: all passing ✅
Findings: none
Research: skipped (small tier)
Security: no concerns
Test coverage: no new branches requiring coverage; sort mapping fully exercised by existing paths
Unresolved threads: 1 outdated Perry thread (prior suggestion to move sort guard — addressed by Devin in a079d86; resolved on APPROVE)
Scope: incremental — 1 net-new line (guard reordering) since prior review at 6fad4de
Review: tier=small · model=claude-sonnet-latest · score=0.6
Summary
Syncs the
openrouter-modelsskill with OpenRouterTeam/openrouter-web#25600, which added two new server-side sort values toGET /api/v1/models:intelligence-high-to-low— Artificial Analysis intelligence indexdesign-arena-elo-high-to-low— best Design Arena ELO across arenasModels without a score for the chosen benchmark are placed last.
Script change:
list-models.tsnow passes asortquery parameter to the API for server-side sorting (previously did all sorting client-side). New flags:--sort intelligence,--sort design-arena-elo. Also exposes--sort latencyand--sort popularwhich were already available server-side but not surfaced in the script.SKILL.md: Documents the new sort options in the decision tree, sort section, and adds the
sortquery parameter to the API reference table with all available enum values.Link to Devin session: https://openrouter.devinenterprise.com/sessions/21bf651a080045a0922eed326441bd18