Add OpenRouter summary provider#20
Conversation
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughAdds OpenRouter as a summary provider across the app: types, defaults, UI options, provider helpers, centralized request headers (including OpenRouter metadata), API-key gating and JSON-schema response-format selection, Rust backend checks and requests, tests, and docs. Test-only exports for two helpers added. ChangesOpenRouter Provider Support
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
There was a problem hiding this comment.
Code Review
This pull request introduces OpenRouter as a summary provider across the desktop application and the Rust-based native host. The changes include updated configuration defaults, new UI icons, centralized header management, and expanded documentation. Feedback focuses on aligning with OpenRouter's documentation by using the X-Title header instead of X-OpenRouter-Title and improving the maintainability of provider display name mappings by explicitly handling all supported providers in both the JavaScript and Rust implementations.
Summary
Adds OpenRouter as an AI summary provider alongside OpenAI, Ollama, and LM Studio.
Why
Users should be able to use OpenRouter's OpenAI-compatible chat completions API for summary and source-context generation.
Changes
Validation:
Summary by CodeRabbit
New Features
Bug Fixes / Improvements
Documentation