Skip to content

feat: add MiniMax as alternative LLM provider#57

Open
octo-patch wants to merge 1 commit intoInteguru-AI:mainfrom
octo-patch:feature/add-minimax-provider
Open

feat: add MiniMax as alternative LLM provider#57
octo-patch wants to merge 1 commit intoInteguru-AI:mainfrom
octo-patch:feature/add-minimax-provider

Conversation

@octo-patch
Copy link

Summary

  • Add MiniMax as a first-class LLM provider alongside OpenAI
  • Support MiniMax-M2.7 and MiniMax-M2.7-highspeed models via OpenAI-compatible API
  • Auto-detect provider from available API keys (MINIMAX_API_KEY / OPENAI_API_KEY)
  • Add --llm-provider CLI option for explicit provider selection
  • Add MiniMaxChatOpenAI subclass that adapts deprecated function-calling kwargs to tools/tool_choice format and strips <think> tags from responses

Changes

File Description
integuru/util/LLM.py Provider presets, MiniMaxChatOpenAI subclass, _detect_provider(), updated LLMSingleton
integuru/__main__.py Add --llm-provider CLI option
integuru/main.py Pass provider to LLMSingleton.set_provider()
README.md Provider table, MiniMax setup instructions
tests/test_llm_provider.py 18 unit tests (presets, detection, provider switching, backward compat)
tests/test_minimax_integration.py 3 integration tests (basic invoke, function calling, alternate model)

Usage

Test plan

  • All 18 unit tests pass
  • All 3 MiniMax integration tests pass (basic invoke, function calling, alternate model)
  • Backward compatibility: existing OpenAI behavior unchanged when no provider specified
  • Pre-existing test failures (missing test.har fixture) are unrelated to this PR

Add MiniMax (MiniMax-M2.7, MiniMax-M2.7-highspeed) as a first-class LLM
provider alongside OpenAI via the OpenAI-compatible API.

- Add provider presets with model defaults, API key env vars, and base URLs
- Add MiniMaxChatOpenAI subclass for function-calling compat and think-tag strip
- Auto-detect provider from API keys (MINIMAX_API_KEY / OPENAI_API_KEY)
- Add --llm-provider CLI option (openai|minimax)
- 18 unit tests + 3 integration tests
- Update README with provider table and setup instructions
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants