Skip to content

feat(core): add Kimi Code CLI platform support#90

Merged
benym merged 3 commits into
rpamis:masterfrom
Joechan11:feat/kimi-code-cli-support
Jun 11, 2026
Merged

feat(core): add Kimi Code CLI platform support#90
benym merged 3 commits into
rpamis:masterfrom
Joechan11:feat/kimi-code-cli-support

Conversation

@Joechan11

@Joechan11 Joechan11 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

✨ Summary

🎯 Scope

  • CLI commands (init, status, doctor, update)
  • Core installer / platform detection
  • Comet skills (assets/skills/, assets/skills-zh/)
  • Comet shell scripts (assets/skills/comet/scripts/)
  • Tests / CI
  • Documentation / changelog
  • Other:

🧪 Testing

  • pnpm build
  • pnpm lint
  • pnpm format:check
  • pnpm test
  • pnpm test -- test/ts/comet-scripts.test.ts
  • pnpm test:shell
  • Not run:

✅ Checklist

  • PR title follows Conventional Commits, for example fix: handle project-scope init
  • User-facing behavior is documented in README.md, README-zh.md, or CONTRIBUTING.md
  • CHANGELOG.md is updated when behavior changes
  • Skill changes were made in Chinese first when applicable, then synced to English
  • New scripts are included in assets/manifest.json and relevant tests
  • Shell scripts remain portable across macOS, Linux, and Windows Git Bash
  • No unrelated generated files or local artifacts are included

👀 Notes for Reviewers

Summary by CodeRabbit

  • New Features

    • Added Kimi Code as a supported AI coding platform; total supported platforms increased to 29.
  • Documentation

    • Updated README (English and Chinese) to list Kimi Code and reflect the new platform count and table ordering.
  • Tests / CI

    • CI checks and test suites updated to expect 29 platforms and include detection/installation coverage for Kimi Code.

@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7e77c2c7-e2d6-4242-af73-ef0bdb57b5e2

📥 Commits

Reviewing files that changed from the base of the PR and between 80132b5 and bdf1306.

📒 Files selected for processing (1)
  • test/ts/openspec.test.ts
🚧 Files skipped from review as they are similar to previous changes (1)
  • test/ts/openspec.test.ts

📝 Walkthrough

Walkthrough

This PR adds support for the Kimi Code AI coding platform by introducing its platform definition with skills directory configuration, mapping it to the kimi-code-cli agent, and extending detection, initialization, and OpenSpec tests to verify correct handling. Documentation and CI workflows are updated to reflect 29 supported platforms.

Changes

Kimi Code Platform Support

Layer / File(s) Summary
Platform definition and agent mapping
src/core/platforms.ts, src/core/superpowers.ts
Kimi Code platform entry is added with id kimicode, skills directories set to .kimi-code, and openspec tool id kimi. SKILLS_AGENT_MAP is extended to map kimicodekimi-code-cli.
Platform detection tests
test/ts/detect.test.ts
New test verifies kimicode declares both skillsDir and globalSkillsDir as .kimi-code with openspecToolId: 'kimi'. Multi-platform detection test updated to include .kimi-code.
CI workflow verification and CI tests
.github/workflows/ci.yml, test/ts/ci-workflows.test.ts
CI e2e verification steps updated to expect 29 platforms; multi-line platform directory lists and success messages adjusted. CI tests updated to assert "All 29 platforms" for project and global scopes.
End-to-end init tests
test/ts/init-e2e.test.ts
"Installs all platforms" E2E test updated to expect ≥29 results and include .kimi-code in the allowlist. New E2E test verifies global initCommand installs Kimi Code skills under ~/.kimi-code/skills.
Agent mapping and OpenSpec tests
test/ts/superpowers.test.ts, test/ts/openspec.test.ts
Superpowers tests expanded to include kimicode mapping and 29-platform coverage. New OpenSpec test ensures kimi tool id is passed to installOpenSpec.
User-facing documentation
README.md, README-zh.md
Supported platform count updated from 28 to 29 and platform tables adjusted to include Kimi Code and its .kimi-code skills directory mapping.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related issues

Poem

🐰 Kimi hops into the code-lined glade,
A twenty-ninth friend in the Comet parade.
With .kimi-code snug and neat,
Init finds home and takes a seat—
Hop, install, now the skills are made.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat(core): add Kimi Code CLI platform support' clearly and specifically summarizes the main change: adding support for a new platform (Kimi Code CLI). It is concise, follows conventional commit format, and directly relates to the primary objective of the PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

Tools execution failed with the following error:

Failed to run tools: 13 INTERNAL: Received RST_STREAM with code 2 (Internal server error)


Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
test/ts/detect.test.ts (1)

92-92: ⚡ Quick win

Use exact detected count in this deterministic test.

tmpDir is controlled and only three platform dirs are created in this case, so toBe(3) will catch accidental over-detection better than toBeGreaterThanOrEqual(3).

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@test/ts/detect.test.ts` at line 92, The test assertion is nondeterministic;
replace the loose check expect(detected.size).toBeGreaterThanOrEqual(3) with an
exact assertion expect(detected.size).toBe(3) so the deterministic setup (three
platform dirs in tmpDir) fails on accidental over-detection; update the
assertion in test/ts/detect.test.ts where the variable detected is checked.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@test/ts/detect.test.ts`:
- Line 92: The test assertion is nondeterministic; replace the loose check
expect(detected.size).toBeGreaterThanOrEqual(3) with an exact assertion
expect(detected.size).toBe(3) so the deterministic setup (three platform dirs in
tmpDir) fails on accidental over-detection; update the assertion in
test/ts/detect.test.ts where the variable detected is checked.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5596469a-6c68-4fa7-9bbb-effe57b94f0b

📥 Commits

Reviewing files that changed from the base of the PR and between e628c25 and f9fd8e3.

📒 Files selected for processing (10)
  • .github/workflows/ci.yml
  • README-zh.md
  • README.md
  • src/core/platforms.ts
  • src/core/superpowers.ts
  • test/ts/ci-workflows.test.ts
  • test/ts/detect.test.ts
  • test/ts/init-e2e.test.ts
  • test/ts/openspec.test.ts
  • test/ts/superpowers.test.ts

@benym benym linked an issue Jun 11, 2026 that may be closed by this pull request
@benym

benym commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

LGTM

@benym benym merged commit 76ab1b6 into rpamis:master Jun 11, 2026
15 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.

Feature request: Add support for Kimi Code CLI

2 participants