docs: clarify router vs model slug in analytics-schema and generations skills#52
docs: clarify router vs model slug in analytics-schema and generations skills#52Justin-Summerville wants to merge 2 commits into
Conversation
…s skills Co-Authored-By: Justin Summerville <justin.summerville@openrouter.ai>
Original prompt from Justin
|
🤖 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
Clarifies that model in analytics and generations APIs returns the resolved downstream model (not the router slug), and explains when to use the router field for router traffic analysis.
Verdict: ✅ LGTM
Details
Risk: 🟢 Low — docs-only clarifications, no code changes
CI: 1 pending (perry/review — this review, not a blocker)
Findings (see inline comments for full context):
- nit: the
routerdimension is referenced in the updated model description but not listed in the Dimension Categories section
Codex: not run (trivial tier)
Research: not run (trivial tier)
Security: no concerns — no code touched, no credentials, no API routes
Test coverage: n/a — docs-only
Unresolved threads: none
Scope: first review (full)
Review: tier=trivial · model=claude-sonnet-latest · score=0.1
Co-Authored-By: Justin Summerville <justin.summerville@openrouter.ai>
Superseded by updated Perry review
There was a problem hiding this comment.
Perry's Review
Clarifies router vs model slug distinction in analytics-schema and generations skill docs — the prior suggestion about the router dimension redirect has been addressed correctly.
Verdict: ✅ LGTM
Details
Risk: 🟢 Low
CI: all passing ✅
Findings: none
Research: skipped — trivial tier
Security: no concerns
Test coverage: docs-only, no tests applicable
Unresolved threads: 1 outdated (Perry's prior router dimension suggestion — addressed by Devin; code updated to guide agents to generation_id + the openrouter-generations skill or ClickHouse instead)
Scope: incremental — 0 new lines since prior review
Review: tier=trivial · model=claude-sonnet-latest · score=?
Summary
Companion to the data-warehouse and Looker PRs — updates skills so agents using the OpenRouter Analytics and Generations APIs understand the router vs model slug distinction.
openrouter-analytics-schema:modeldimension description now states it's the resolved downstream model, not the router, and directs to the router field for router traffic.openrouter-generations:modelfield description clarifies it's the resolved model;routerfield description explains it identifies the router and should be used for router traffic. The "What you get back" summary also clarifies the relationship.Link to Devin session: https://openrouter.devinenterprise.com/sessions/65ba8131d98d4c32a2995b7ba0829d7c
Requested by: @Justin-Summerville