feat: add MiniMax as LLM provider for task decomposer

[octo-patch]

Summary

Add MiniMax as an alternative LLM provider for the task decomposition module, alongside Anthropic.

• Introduce provider field in DecomposerConfig ("anthropic" | "minimax")
• Create an internal LLMClient abstraction that routes to either Anthropic SDK or MiniMax OpenAI-compatible API
• Handle MiniMax-specific behaviors: temperature clamping to [0, 1], <think> tag stripping for M2.5/M2.7 models
• Support MINIMAX_API_KEY env var and optional MINIMAX_BASE_URL override
• Update Zod config schema, example config, and README with MiniMax documentation

Files changed (8 files, ~982 additions)

File	Change
packages/core/src/decomposer.ts	LLM client abstraction + MiniMax provider
packages/core/src/config.ts	Add provider enum to decomposer schema
packages/core/src/types.ts	Add provider field to decomposer config
packages/core/src/index.ts	Export new types
packages/core/src/__tests__/decomposer.test.ts	36 unit tests
packages/core/src/__tests__/decomposer-integration.test.ts	6 integration tests
agent-orchestrator.yaml.example	Decomposer config with MiniMax option
README.md	MiniMax provider documentation

Test plan

• 36 unit tests pass
• 6 integration tests pass
• All 42 new tests pass alongside existing 525 tests
• pnpm build succeeds with no type errors

[cursor]

Cursor Bugbot has reviewed your changes and found 2 potential issues.

^{Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

[cursor]

Switch default silently masks unknown provider values

Medium Severity

The default case in createLLMClient falls through to createAnthropicClient(), meaning any future DecomposerProvider union member added without a corresponding switch case would silently use Anthropic instead of failing. Because default is combined with case "anthropic", TypeScript's exhaustiveness checking is also defeated — a never check in the default branch won't trigger a compile error when the union grows.

 

[cursor]

Internal abstractions unnecessarily exported as public API

Low Severity

stripThinkingTags and createLLMClient are exported from index.ts but are not imported by any consumer package — only by internal tests (which import directly from decomposer.js). createLLMClient is explicitly documented as an "Internal abstraction" and its return type LLMClient is not exported, making it an awkward public API addition that increases the package's surface area without need.

 

^{Triggered by project rule: BugBot Configuration}