feat(models): add routeway models and metadata - #712
Conversation
Added multiple free models under the RouteWay provider, including metadata such as pricing, context size, and streaming capabilities. Removed llama-3.3-70b-instruct-free model entry from Meta models.
WalkthroughAdds a new Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 💡 Knowledge Base configuration:
You can enable these sources in your CodeRabbit configuration. 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
✨ Finishing Touches
🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
steebchen
changed the title
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (3)
packages/models/src/models/routeway.ts (3)
1-1: Avoid potential self-import cycles; use a relative type import.Importing
ModelDefinitionfrom the package entry can create circular type deps withmodels.ts. Prefer a relative import.Apply:
-import type { ModelDefinition } from "@llmgateway/models"; +import type { ModelDefinition } from "../models";
9-10: Drop explicitundefinedfields for noise reduction.
deprecatedAt/deactivatedAtdefault to undefined; omitting them keeps objects terse.Example (apply to all blocks):
- deprecatedAt: undefined, - deactivatedAt: undefined,Also applies to: 33-34, 57-58, 81-82, 105-106, 129-130
3-148: Consider marking RouteWay free models as non-stable initially.If these are newly integrated/free-tier routes, set
stability: "beta"(or appropriate level) to avoid over-preference by selection heuristics.Example (apply to each model object):
{ id: "deepseek-r1t2-chimera-free", name: "DeepSeek R1T2 Chimera (Free)", family: "deepseek", free: true, + stability: "beta",
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (3)
packages/models/src/models.ts(2 hunks)packages/models/src/models/meta.ts(0 hunks)packages/models/src/models/routeway.ts(1 hunks)
💤 Files with no reviewable changes (1)
- packages/models/src/models/meta.ts
🧰 Additional context used
📓 Path-based instructions (3)
**/*.{js,jsx,ts,tsx}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Use localStorage instead of cookies for client-side data persistence
Files:
packages/models/src/models.tspackages/models/src/models/routeway.ts
**/*.{js,ts}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{js,ts}: Use drizzle with the latest object syntax for database operations
For read queries, always usedb().query.<table>.findMany()ordb().query.<table>.findFirst()
Files:
packages/models/src/models.tspackages/models/src/models/routeway.ts
**/*.{ts,tsx}
📄 CodeRabbit inference engine (.cursor/rules/general.mdc)
Never use
as anyor: anyin TypeScript files.
Files:
packages/models/src/models.tspackages/models/src/models/routeway.ts
🧬 Code graph analysis (2)
packages/models/src/models.ts (1)
packages/models/src/models/routeway.ts (1)
routewayModels(3-148)
packages/models/src/models/routeway.ts (1)
packages/models/src/models.ts (1)
ModelDefinition(104-157)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: build / run
🔇 Additional comments (5)
packages/models/src/models.ts (1)
13-13: LGTM: routeway models import is wired correctly.No issues with the new import.
packages/models/src/models/routeway.ts (4)
26-26: Validate and adjustjsonOutputflags
Ensure each model listed inrouteway.tstruly supports strict JSON output under RouteWay; only leavejsonOutput: truewhere you’ve verified compliance, and disable it for models that don’t.
19-19: Verify each model’s contextSize against RouteWay documentation
All six RouteWay-backed models in packages/models/src/models/routeway.ts (lines 19, 43, 67, 91, 115, 139) are set to 128000. Confirm these values match each provider’s supported context window and adjust per-model if needed.
3-148: Provider entry and routing verified. No action required.
7-7: No action needed — allfamilylabels match the supported set. Verified that"deepseek"(and the families at lines 31, 55, 79, 103, and 127) appear in the repo-wide inventory offamilyvalues and align with existing selection logic.
1 participant
Added multiple free models under the RouteWay provider, including metadata such as pricing, context size, and streaming capabilities. Removed llama-3.3-70b-instruct-free model entry from Meta models.
Summary by CodeRabbit
New Features
Changes