Skip to content

feat(ai): add MiniMax as LLM summarization provider#1116

Open
ximiximi423 wants to merge 1 commit intokoala73:mainfrom
ximiximi423:feat/minimax-provider
Open

feat(ai): add MiniMax as LLM summarization provider#1116
ximiximi423 wants to merge 1 commit intokoala73:mainfrom
ximiximi423:feat/minimax-provider

Conversation

@ximiximi423
Copy link
Copy Markdown

Add MiniMax as a new LLM provider for article summarization, providing an additional fallback option alongside Groq and OpenRouter.

Changes:

  • Add MINIMAX_API_KEY support with optional URL/model overrides
  • Integrate MiniMax provider into summarization fallback chain
  • Add API key validation in sidecar server
  • Update settings UI to include MiniMax configuration
  • Add provider test suite

Made-with: Cursor

Add MiniMax (OpenAI-compatible API) as a new LLM provider for article
summarization, providing an additional fallback option alongside Groq
and OpenRouter.

Changes:
- Add MINIMAX_API_KEY support with optional URL/model overrides
- Integrate MiniMax provider into summarization fallback chain
- Add API key validation in sidecar server
- Update settings UI to include MiniMax configuration
- Add provider test suite

Made-with: Cursor
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 6, 2026

@ximiximi423 is attempting to deploy a commit to the Elie Team on Vercel.

A member of the Team first needs to authorize it.

@koala73 koala73 added High Value Meaningful contribution to the project Ready to Merge PR is mergeable, passes checks, and adds value labels Mar 26, 2026
@SebastienMelki
Copy link
Copy Markdown
Collaborator

Thanks for the MiniMax contribution @ximiximi423! We're closing this in favor of PR #1496 which implements the same feature with a tighter scope — it only touches server/_shared/llm.ts, .env.example, and tests, without modifying the proto definition, Tauri sidecar, or settings UI. The simpler approach reduces review surface and merge risk.

If you'd like to contribute the settings UI for MiniMax as a follow-up once #1496 is merged, that would be welcome.

@koala73 — recommending close in favor of #1496.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Value Meaningful contribution to the project Ready to Merge PR is mergeable, passes checks, and adds value

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants