Skip to content

fix(claude): add MiniMax models to the picker - #129

Open
octo-patch wants to merge 2 commits into
nexu-io:mainfrom
octo-patch:octo/20260726-provider-add-recvpdruIPiFgh
Open

fix(claude): add MiniMax models to the picker#129
octo-patch wants to merge 2 commits into
nexu-io:mainfrom
octo-patch:octo/20260726-provider-add-recvpdruIPiFgh

Conversation

@octo-patch

Copy link
Copy Markdown

Reason: Add the configured MiniMax Claude-compatible models to the picker.

What changed

  • add MiniMax/MiniMax-M3 and MiniMax/MiniMax-M2.7 to the Claude Code curated model list used by the Next.js app
  • keep the CLI-side agent registry in sync with the same curated model entries
  • add focused tests that assert both MiniMax model ids remain available in the Claude picker

Checks

  • pnpm --dir cli test -- src/__tests__/agents-detect.test.ts
  • pnpm --dir next test -- src/lib/agents/__tests__/detect.test.ts

@lefarcen
lefarcen requested a review from mrcfps July 26, 2026 20:34
@lefarcen lefarcen added size/S Small change: 20-99 changed lines risk/medium Medium risk change type/bugfix Bug fix labels Jul 26, 2026

@mrcfps mrcfps 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.

@octo-patch The MiniMax M3 and M2.7 model entries are kept in sync across the Next.js and CLI Claude registries, and the new coverage verifies both picker lists. I also verified the workspace guard, both package typechecks, all unit tests, and both production builds; everything passes. Thanks for the focused, well-tested fix! 🙌

🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.

@octo-patch

Copy link
Copy Markdown
Author

I corrected the Claude picker entries to pass the exact MiniMax model IDs (MiniMax-M3 and MiniMax-M2.7) in both the CLI and Next.js registries, and updated the matching tests. Both package test suites and typechecks passed.

@lefarcen

Copy link
Copy Markdown

Hey @octo-patch — thanks for the focused MiniMax picker fix.

I’ve caught up the PR triage on our side and re-requested @mrcfps on the current head so the latest MiniMax-M3 / MiniMax-M2.7 correction is back in the review queue. Appreciate the clear follow-up note and the matching test updates. ❤️

@lefarcen
lefarcen requested a review from mrcfps August 11, 2026 06:20

@mrcfps mrcfps 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.

@octo-patch Nice, focused fix — the Claude picker now exposes the exact MiniMax model IDs (MiniMax-M3 and MiniMax-M2.7) in both the Next.js and CLI agent registries, and the new tests lock that shape in place.

I reviewed the changed ranges across next/src/lib/agents/detect.ts, cli/src/agents-detect.ts, and both detect test files. The second commit correctly drops the provider-prefixed form so the IDs match what Claude Code expects for --model. I also ran the CLI and Next unit suites; both passed, including the new MiniMax coverage.

Thanks for keeping the two registries in sync and following up with the ID correction — clean and solid work! 🙌

🔁 Powered by Looper · runner=reviewer · agent=grok-build · An autonomous AI dev team for your GitHub repos.

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

Labels

risk/medium Medium risk change size/S Small change: 20-99 changed lines type/bugfix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants