Skip to content

Add New Models minimax-m2.7#488

Merged
bao-byterover merged 3 commits intocampfirein:mainfrom
zeki893:main
Apr 21, 2026
Merged

Add New Models minimax-m2.7#488
bao-byterover merged 3 commits intocampfirein:mainfrom
zeki893:main

Conversation

@zeki893
Copy link
Copy Markdown
Contributor

@zeki893 zeki893 commented Apr 21, 2026

Summary

  • Problem: Codebase only supports minimax-m2, missing newer MiniMax model variants
  • Why it matters: Users can't access the newer minimax-m2.7 model through the platform without this change
  • What changed: Added minimax-m2.7 to the list of supported MiniMax model identifiers in the LLM provider configuration/constants
  • What did NOT change (scope boundary): No changes to the MiniMax API client logic, authentication, request/response handling, or any other provider's configuration

Type of change

  • Bug fix
  • New feature
  • Refactor (no behavior change)
  • Documentation
  • Test
  • Chore (build, dependencies, CI)

Scope (select all touched areas)

  • TUI / REPL
  • Agent / Tools
  • LLM Providers
  • Server / Daemon
  • Shared (constants, types, transport events)
  • CLI Commands (oclif)
  • Hub / Connectors
  • Cloud Sync
  • CI/CD / Infra

Linked issues

  • Closes #
  • Related #

Root cause (bug fixes only, otherwise write N/A)

  • Root cause:
  • Why this was not caught earlier:

Test plan

  • Coverage added:
    • Unit test
    • Integration test
    • Manual verification only
  • Test file(s): N/A
  • Key scenario(s) covered: Minimax-m2.7 appears in the model list/picker. Selecting minimax-m2.7 successfully routes requests to MiniMax API without errors. Minimax-m2 still works as before (no regression)

User-visible changes

Minimax-m2.7 now available as a selectable model in the LLM provider list

Evidence

none

Checklist

  • Tests added or updated and passing (npm test)
  • Lint passes (npm run lint)
  • Type check passes (npm run typecheck)
  • Build succeeds (npm run build)
  • Commits follow Conventional Commits format
  • Documentation updated (if applicable)
  • No breaking changes (or clearly documented above)
  • Branch is up to date with main

Risks and mitigations

List real risks for this PR. If none, write None.

  • Risk: Missing changes in code for model to work. Possibly more files need to be edited.
    • Mitigation: Roll back files/version

Copy link
Copy Markdown
Collaborator

@cuongdo-byterover cuongdo-byterover left a comment

Choose a reason for hiding this comment

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

lgtm!

Copy link
Copy Markdown
Member

@leehpham leehpham left a comment

Choose a reason for hiding this comment

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

lgtm.
@bao-byterover we can arrange this to be merged to main for our coming release.

Copy link
Copy Markdown
Collaborator

@bao-byterover bao-byterover left a comment

Choose a reason for hiding this comment

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

Nice one, thanks @zeki893, merging this for next release.

@bao-byterover bao-byterover merged commit 233ec12 into campfirein:main Apr 21, 2026
6 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.

4 participants