Skip to content

feat: add MiniMax as alternative LLM provider#111

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

feat: add MiniMax as alternative LLM provider#111
octo-patch wants to merge 1 commit intodw-dengwei:mainfrom
octo-patch:feature/add-minimax-provider

Conversation

@octo-patch
Copy link
Copy Markdown

Summary

  • Add multi-provider support with auto-detection for MiniMax, DeepSeek, and OpenAI
  • MiniMax models (M2.7, M2.5, M2.5-highspeed) supported via OpenAI-compatible API
  • Automatic temperature clamping per provider, LLM_PROVIDER env var for explicit selection
  • README updated with MiniMax setup instructions (auto-detect + manual GitHub Actions config)

Changes

File Description
ai/providers.py Provider presets, auto-detection via API key env vars, config resolution
ai/enhance.py Use provider system for LLM initialization (base_url, api_key, temperature)
README.md Supported providers table, MiniMax setup guide (Option A/B)
tests/test_providers.py 24 unit tests (presets, detect, clamp, config)
tests/test_integration.py 3 config-flow + 2 live API integration tests

How it works

Users can switch to MiniMax by simply setting MINIMAX_API_KEY:

export MINIMAX_API_KEY="your-key"
python enhance.py --data papers.jsonl

The provider is auto-detected and the correct base URL, model, and temperature range are applied. Existing DeepSeek/OpenAI workflows remain fully backward-compatible.

Test plan

  • 24 unit tests pass (provider presets, auto-detection, temperature clamping, config resolution)
  • 3 integration tests pass (auto-detect flow, explicit provider, manual config)
  • Live API tests require MINIMAX_API_KEY (skipped in CI by default)

Add multi-provider support with auto-detection for MiniMax, DeepSeek,
and OpenAI. MiniMax models (M2.7, M2.5, M2.5-highspeed) are supported
via OpenAI-compatible API with automatic temperature clamping.

- ai/providers.py: provider presets, auto-detection, config resolution
- ai/enhance.py: use provider system for LLM initialization
- README.md: document MiniMax setup (auto-detect + manual)
- tests/: 24 unit + 3 integration tests
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.

1 participant