feat(core): expand MODEL_PROVIDERS with OpenAI, Google, and OpenRouter#111
feat(core): expand MODEL_PROVIDERS with OpenAI, Google, and OpenRouter#111
Conversation
- Add openai, google, openrouter entries to MODEL_PROVIDERS - Add KEY_INSTRUCTIONS for each provider's API key - Add getProviderForModel() helper to extract provider from model strings - Add comprehensive unit tests for new constants and helper Closes AGE-148
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ✨ Finishing Touches🧪 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 |
|
🔍 QA Review — APPROVED All acceptance criteria for AGE-148 verified against the diff:
Ship it 🚢 |
|
🔍 QA — APPROVED by Scout Build: ✅ clean Ship it 🚢 |
stepandel
left a comment
There was a problem hiding this comment.
🔍 QA Review — APPROVED
All acceptance criteria verified against the diff. Build passes clean. All 45 tests pass (including new constants.test.ts).
Ship it 🚢
|
🔍 QA Review — Approved ✅ AGE-148: MODEL_PROVIDERS expanded, KEY_INSTRUCTIONS added, getProviderForModel() with tests. 45/45 pass. Tested by Scout (automated QA) — build, tests, and acceptance criteria verified. |
Summary
Adds OpenAI, Google Gemini, and OpenRouter to
MODEL_PROVIDERSandKEY_INSTRUCTIONSinpackages/core/src/constants.ts. Adds agetProviderForModel()helper to extract provider keys from model strings.Changes
packages/core/src/constants.ts— Added 3 new provider entries, 3 new KEY_INSTRUCTIONS, and getProviderForModel() helperpackages/core/src/index.ts— Re-exported getProviderForModelpackages/core/src/__tests__/constants.test.ts— 13 new testsTest Results
All 26 tests pass. The 2 pre-existing CLI test failures are unrelated package resolution issues on main.
Closes AGE-148