Repository navigation
feat(prompt): route MiniMax models to the open-weight agentic prompt - #166
Merged
Merged
Conversation
Prompt selection matches on the model id and had no MiniMax case, so every MiniMax model — a primary target for this fork's gateway — fell through to PROMPT_DEFAULT, which is written for Claude. PROMPT_KIMI carries nothing Kimi-specific: it is upstream's prompt for open-weight agentic models, spelling out that tools must be used rather than described, encouraging parallel tool calls, giving <system-reminder> authority, and matching the user's language. Sharing it beats duplicating ~95 lines that upstream will keep improving. Model ids vary in case across catalogs (MiniMax-M1, minimax/minimax-m2.5, MiniMax/MiniMax-M3), so the match is case-insensitive like the trinity and kimi branches next to it.
quickbeard
force-pushed
the
minimax-prompt
branch
from
August 7, 2026 09:27
45642b6 to
32280fc
Compare
3 tasks done
quickbeard
added a commit
that referenced
this pull request
Aug 7, 2026
* docs: link the MiniMax prompt entry to its fork and upstream PRs The MiniMax changelog entry landed before its PR number existed. Add the merged fork PR (#166) and the upstream submission (anomalyco/opencode#41032), matching the retire-on-upstream-merge note the dialog-paste entry already uses. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> * minor changes --------- Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue for this PR
Closes #
Type of change
What does this PR do?
SystemPrompt.provider()selects a prompt by substring-matching the model id, and there wasno MiniMax case — so every MiniMax model fell through to
PROMPT_DEFAULT. MiniMax is aprimary model for this fork's gateway, and
default.txtis upstream's Claude-flavouredprompt, so those sessions were running on the least suitable prompt available.
MiniMax now shares
PROMPT_KIMI, which despite the filename contains nothing Kimi- orMoonshot-specific (checked: zero mentions). It is upstream's prompt for open-weight agentic
models — it spells out that tools must be used rather than described, encourages parallel
tool calls, gives
<system-reminder>authority, and matches the user's language. That is theright profile for M3.
Sharing rather than adding a
minimax.txtcopy is deliberate: a fork-only duplicate of ~95lines that upstream keeps improving would drift, with no upstream counterpart to sync against.
If upstream later adds its own MiniMax branch, ours is dropped in favour of theirs — recorded
in the
system.tsrow ofDIVERGENCES.md.Model ids vary in case across catalogs (
MiniMax-M1,minimax/minimax-m2.5,MiniMax/MiniMax-M3), so the match is case-insensitive like thetrinityandkimibranchesnext to it.
How did you verify your code works?
test/session/system.test.ts: all three id spellings resolve to the sameprompt as
kimi-k2-thinking.test/session/brand.test.ts— the "routes support at the fork" case used MiniMax asits example of a model falling through to
default.txt, which is exactly what this PRchanges; it now uses an id that matches no branch, and MiniMax stays in the list that must
introduce the fork by name.
bun run typecheckand both session test files pass (9 tests).Screenshots / recordings
Not a UI change.
Checklist
🤖 Generated with Claude Code