Repository navigation
docs(examples): list MiniMax M3.1 Flash and M3 as harness_duo model options - #1112
Conversation
Performance
✓ No regressions detected |
AbirAbbas
left a comment
There was a problem hiding this comment.
One concern before this comes out of draft: openrouter/minimax/minimax-m2.7 still has six live providers on OpenRouter (GMICloud, Novita, AtlasCloud, Minimax, Groq), while minimax-m3.1-flash:free has exactly one, the Minimax free endpoint, and the bare Flash slug has none. Switching the example's default to a single free route makes harness_duo less reliable for people who copy it: free routes are rate-limited, can disappear without notice, and the dated "as of 2026-10-19" notes will go stale.
I'd keep m2.7 as the default (or move to minimax-m3, which has 14 providers) and mention the Flash :free route as an option you can set through HARNESS_MODEL. The new TestPiFamilyPreservesOpenRouterFreeSuffix is worth keeping either way. The red coverage (sdk-python) check was the FastAPI 0.148 test break that #1120 fixed on main; merging main will clear it.
# Conflicts: # examples/go_agent_nodes/cmd/harness_duo/README.md
M2.7 still has several live OpenRouter providers, while the minimax-m3.1-flash:free route currently has no published endpoint. List M3.1 Flash and M3 as HARNESS_MODEL options instead of making a single free route the default, and drop the dated availability notes. The :free suffix pass-through test stays. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Took this over since it had been sitting for a while. I re-checked OpenRouter today and the picture flipped: |
📊 Coverage gateThresholds from
✅ Gate passedNo surface regressed past the allowed threshold and the aggregate stayed above the floor. |
📐 Patch coverage gateThreshold: 80% on lines this PR touches vs
✅ Patch gate passedEvery surface whose lines were touched by this PR has patch coverage at or above the threshold. |
AbirAbbas
left a comment
There was a problem hiding this comment.
Net change against main is now the two doc updates plus the :free suffix test. The example default is unchanged. Harness tests pass and CI is green.
Maintainer takeover (2026-10-30): re-checked OpenRouter:
minimax-m3.1-flash:freecurrently has no published endpoint, whileminimax-m3.1-flashhas 12 providers andminimax-m2.7still has 5 live ones with tool support. The example keepsminimax-m2.7as its default; the README and provider guide listminimax-m3.1-flashandminimax-m3asHARNESS_MODELoptions and drop the dated availability notes.TestPiFamilyPreservesOpenRouterFreeSuffixstays. main merged in to clear the README conflict. The original description follows.Reason: the active Pi/OMP Go workflow still defaults to MiniMax M2.7. MiniMax M3.1 Flash is now available through OpenRouter's
:freeroute, while the bare Flash slug has no published endpoint.Summary
Use
openrouter/minimax/minimax-m3.1-flash:freeas theharness_duodefault and update its README and provider guide. Document the verifiedopenrouter/minimax/minimax-m3alternative, preserveHARNESS_MODELand per-call overrides, and cover the:freesuffix through both Go adapters.The current Flash endpoint supports tools with automatic tool choice; the adapters do not force tool choice and structured output uses files. No direct MiniMax route, model-specific runtime capabilities, or price estimates are added.
Availability was checked on 2026-10-19:
Type of change
Test plan
cd examples/go_agent_nodes && go test ./cmd/harness_duo && go vet ./cmd/harness_duo.cd sdk/go && go test ./harness -count=1, includingTestPiFamilyPreservesOpenRouterFreeSuffixfor Pi and OMP.cd sdk/go && go test ./...andgo mod tidy -diffpassed.git diff --checkand diff secret scan../scripts/test-all.shreported an unrelated control-plane failure inTestStopHTTPServerShutdownTimeout(graceful_shutdown_test.go:170: expected an error, got nil). Stopped the remaining run after recording that failure; Python/TypeScript/UI stages were not reached. This PR remains a draft pending broader validation and coverage CI.No live inference or external agent CLI was run. The route remains subject to OpenRouter availability/free-route limits and the installed CLI's model catalog. The SDK's generic model passthrough and historical invalid-model cases are unchanged.
Test coverage
FastAPI.__init__() got an unexpected keyword argument 'enable_mcp'; the event-loop message is an additional warning. The coverage summary subsequently failed because the Python coverage artifact was missing. Go, control-plane/Go/TypeScript/UI coverage, local/PostgreSQL functional tests, and the main build passed. No Python files changed. Both affected test modules were independently rerun on the unmodified base2f5583e5d88f27c1175e58978a4464463553b38din an isolated Python 3.11 environment with the declared dev dependencies: the same six failures reproduced, with 20 tests passing.Checklist
docs/CONTRIBUTING.mdanddocs/DEVELOPMENT.md.AI-assisted implementation and validation: GPT-6 through MiniMax Code, under a human-authorized MiniMax release rollout.
Related issues / PRs
Follows the Pi/OMP integration in #913. Existing MiniMax media integrations are outside this change.