Skip to content

Add MiniMax AI provider#142

Merged
mithun50 merged 1 commit into
mithun50:mainfrom
octo-patch:octo/20260708-add-target-provider-model-to-existing-provider-registry-recvoNmzEmFa5h
Jul 9, 2026
Merged

Add MiniMax AI provider#142
mithun50 merged 1 commit into
mithun50:mainfrom
octo-patch:octo/20260708-add-target-provider-model-to-existing-provider-registry-recvoNmzEmFa5h

Conversation

@octo-patch

@octo-patch octo-patch commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Reason: add target provider/model to existing provider registry

Summary

  • Add MiniMax to the AI provider registry.
  • Configure MiniMax-M3 as the default model with the OpenAI-compatible MiniMax endpoint.

Checks

  • git add -A -N && git diff HEAD | grep -E "$SECRET_RE" (no matches)
  • flutter analyze (not run: Flutter SDK not installed in environment)

Summary by cubic

Add MiniMax to the provider registry so users can select it in the app. Sets MiniMax-M3 as the default model and configures the https://api.minimax.io/v1 endpoint.

Written for commit 0a962fc. Summary will update on new commits.

Review in cubic

Summary by CodeRabbit

  • New Features
    • Added support for a new AI provider option, making it available in the provider list for selection.
    • Expanded the available provider choices so users can access an additional service alongside the existing options.

Reason: add target provider/model to existing provider registry

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f589ba19-9f4f-46d2-a2a0-f8cbed318790

📥 Commits

Reviewing files that changed from the base of the PR and between 09e8e7d and 0a962fc.

📒 Files selected for processing (1)
  • flutter_app/lib/models/ai_provider.dart

📝 Walkthrough

Walkthrough

A new static provider constant minimax was added to the AiProvider class, including its id, name, description, icon, color, base URL, default models, and API key hint. The AiProvider.all list was updated to include this new provider.

Changes

MiniMax Provider Addition

Layer / File(s) Summary
Add minimax provider constant and register in all list
flutter_app/lib/models/ai_provider.dart
Adds AiProvider.minimax static constant with id/name/description/icon/color/baseUrl/defaultModels/apiKeyHint and updates AiProvider.all to include it.

Estimated code review effort: 1 (Trivial) | ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely matches the main change: adding the MiniMax AI provider.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mithun50 mithun50 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for raising this pr.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 1 file

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="flutter_app/lib/models/ai_provider.dart">

<violation number="1" location="flutter_app/lib/models/ai_provider.dart:146">
P3: The `apiKeyHint` for MiniMax uses `sk-...` but MiniMax API keys have the documented prefix `sk-cp-...`. Updating the hint to `sk-cp-...` would make it consistent with the other providers (Anthropic uses `sk-ant-...`, OpenRouter uses `sk-or-...`, etc.) and help users match their key to the right field more quickly.</violation>
</file>

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

defaultModels: [
'MiniMax-M3',
],
apiKeyHint: 'sk-...',

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P3: The apiKeyHint for MiniMax uses sk-... but MiniMax API keys have the documented prefix sk-cp-.... Updating the hint to sk-cp-... would make it consistent with the other providers (Anthropic uses sk-ant-..., OpenRouter uses sk-or-..., etc.) and help users match their key to the right field more quickly.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At flutter_app/lib/models/ai_provider.dart, line 146:

<comment>The `apiKeyHint` for MiniMax uses `sk-...` but MiniMax API keys have the documented prefix `sk-cp-...`. Updating the hint to `sk-cp-...` would make it consistent with the other providers (Anthropic uses `sk-ant-...`, OpenRouter uses `sk-or-...`, etc.) and help users match their key to the right field more quickly.</comment>

<file context>
@@ -133,6 +133,28 @@ class AiProvider {
+    defaultModels: [
+      'MiniMax-M3',
+    ],
+    apiKeyHint: 'sk-...',
+  );
+
</file context>
Suggested change
apiKeyHint: 'sk-...',
apiKeyHint: 'sk-cp-...',

@mithun50 mithun50 merged commit 7e5a8f2 into mithun50:main Jul 9, 2026
4 of 5 checks passed
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