feat: Add support for OpenRouter as an LLM provider.#35
feat: Add support for OpenRouter as an LLM provider.#35elskito wants to merge 1 commit intovas3k:mainfrom
Conversation
d0ed6f1 to
eb963b3
Compare
|
I've never heard of such a platform, to be honest. It looks like an aggregator that, in theory, gives access to many LLMs at once, which is useful for sure. But I'm doubting the need to include it in the standard deployment. Only models that work good with images (webp, to be precise) are suitable for TaxHacker, so I would rather add proven LLMs to avoid confusing users and leaving them with a non-functional application. For example, I would be happy if one of the DeepSeek and Ollama models were a separate option for self-hosted users |
Code Review Analysis 🔍Thanks for adding OpenRouter support! This is a clean implementation. Here are some suggestions: ✅ What looks good
💡 Suggestions for improvement
🔗 RelatedNote: I've opened PR #58 and #59 which improve error messages for invalid models and add dynamic model fetching from provider APIs. These changes would complement your OpenRouter addition well! Automated review by Aetheris |
Summary
OpenRouter API support added to app
PR contains: