Skip to content

feat(skills): add MiniMax-AI/cli as default skill tap#8869

Open
octo-patch wants to merge 1 commit intoKilo-Org:mainfrom
octo-patch:feat/tap-minimax-cli
Open

feat(skills): add MiniMax-AI/cli as default skill tap#8869
octo-patch wants to merge 1 commit intoKilo-Org:mainfrom
octo-patch:feat/tap-minimax-cli

Conversation

@octo-patch
Copy link
Copy Markdown

Summary

  • Add mmx-cli skill to BUILTIN_SKILLS in packages/opencode/src/kilocode/skills/builtin.ts so it is available out of the box to all Kilo users
  • Skill content is sourced from MiniMax-AI/cli and follows the agentskills.io standard
  • Skill updates are decoupled: MiniMax maintains the upstream SKILL.md; this PR snapshots the current version as a built-in

2 files changed (1 new file, 1 modified).

What is mmx-cli?

mmx-cli is a CLI tool for the MiniMax AI platform, providing:

  • Text generation (MiniMax-M2.7 model)
  • Image generation (image-01)
  • Video generation (MiniMax-Hailuo-2.3)
  • Speech synthesis (speech-2.8-hd, 300+ voices)
  • Music generation (music-2.6, with lyrics, cover, and instrumental)
  • Web search

The skill follows the agentskills.io specification and includes agent-specific flags (--non-interactive, --quiet, --output json).

Test plan

  • Ask agent "generate an image of a sunset using mmx" — agent invokes mmx image generate via terminal
  • Ask "use mmx to convert text to speech" — agent uses mmx speech synthesize
  • Verify mmx-cli appears in /skills listing within Kilo Code

content: KILO_CONFIG,
},
{
name: "mmx-cli",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

WARNING: This entry is never registered as an available skill

BUILTIN_SKILLS is imported in packages/opencode/src/skill/index.ts, but nothing ever copies those definitions into state.skills. As written, mmx-cli is dead data: it will not appear in /skills, and Skill.get("mmx-cli") will still return undefined. Please wire built-ins into loadSkills() or state initialization before adding new entries here.

Download a completed video by task ID.

```bash
mmx video download --file-id <id> [--out <path>]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

WARNING: Conflicting video download flags

This synopsis uses --file-id, but the async workflow later in the same skill uses --task-id. Because the agent loads this markdown verbatim, inconsistent flags here can cause it to emit a command the CLI rejects. Please align both sections with the actual mmx contract.

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot bot commented Apr 13, 2026

Code Review Summary

Status: 2 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 2
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
packages/opencode/src/kilocode/skills/builtin.ts 23 mmx-cli is added to BUILTIN_SKILLS, but built-in entries are never loaded into state.skills, so the skill will not appear in /skills or be loadable.
packages/opencode/src/kilocode/skills/mmx-cli.md 154 The video download section documents --file-id, while the later async workflow uses --task-id, so the skill contains conflicting command syntax.
Other Observations (not in diff)

Issues found in unchanged code that cannot receive inline comments:

File Line Issue
None N/A None.
Files Reviewed (2 files)
  • packages/opencode/src/kilocode/skills/builtin.ts - 1 issue
  • packages/opencode/src/kilocode/skills/mmx-cli.md - 1 issue

Reviewed by gpt-5.4-20260305 · 450,781 tokens

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.

1 participant