update Kimi CLI to Kimi Code#1208
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (8)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughThe PR renames the Kimi integration to “Kimi Code”, changes managed skills from ChangesKimi Code configuration and migration
Estimated code review effort: 3 (Moderate) | ~25 minutes Sequence Diagram(s)sequenceDiagram
participant InitUpdate
participant migrateLegacySkillDirs
participant FileSystem
InitUpdate->>migrateLegacySkillDirs: migrate project skill directories
migrateLegacySkillDirs->>FileSystem: inspect legacy .kimi/skills workflows
migrateLegacySkillDirs->>FileSystem: move managed workflows to .kimi-code/skills
migrateLegacySkillDirs->>FileSystem: preserve user files and remove empty directories
migrateLegacySkillDirs-->>InitUpdate: return migration records
Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint install timed out. The project may have too many dependencies for the sandbox. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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.
Inline comments:
In `@docs/commands.md`:
- Around line 675-677: Update the command reference table to remove the stale
Kimi CLI row, then consolidate the duplicate Trae entries into one authoritative
Trae row using the correct invocation syntax. Preserve the surrounding table
structure and other tool entries.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: fd9778c9-b4b0-4942-b5ae-d338837aff7d
📒 Files selected for processing (3)
docs/commands.mddocs/supported-tools.mdsrc/core/config.ts
🚧 Files skipped from review as they are similar to previous changes (2)
- src/core/config.ts
- docs/supported-tools.md
# Conflicts: # docs/commands.md
…o fix/kimi-code # Conflicts: # docs/commands.md
alfred-openspec
left a comment
There was a problem hiding this comment.
Changing skillsDir strands OpenSpec-managed .kimi/skills files because update and cleanup only inspect current AI_TOOLS paths. Please add legacy cleanup or migration for managed OpenSpec skills, preserving user files, with a focused update test before moving generation to the current .kimi-code/skills path.
Renaming the Kimi skillsDir stranded OpenSpec-managed skills under .kimi/skills: update and cleanup only inspect current AI_TOOLS paths, so old installs would never be detected or refreshed again. Add a legacy skillsDir migration (run by init and update before tool detection) that moves openspec-* skill directories to .kimi-code/skills, preserves user files, and removes the legacy directories only when empty. Keep .kimi as a detection path and cover the migration with focused init and update tests. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Pushed e8ad256 addressing @alfred-openspec's review:
Verified end-to-end with the built CLI: a project with legacy |
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
alfred-openspec
left a comment
There was a problem hiding this comment.
The migration now preserves legacy detection, moves known OpenSpec workflow directories before init and update, retains sibling user skills and config, and removes old roots only when empty. The current .kimi-code path matches official Kimi Code docs, focused lifecycle tests pass, and full CI is green.
Resolve conflicts with Fission-AI#1354 (displayMissingCoreWorkflowsNote rename) and Fission-AI#1208 (kimi-code migration tests): keep both sides' tests, call the renamed note method alongside the Hermes displaySetupNotes hook. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Union the tool-ID list (hermes from main + zcode) and keep both the new ZCode init test and the renamed Kimi Code test from Fission-AI#1208. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Union resolutions with the hermes (Fission-AI#1292), zcode (Fission-AI#1209), and kimi-code (Fission-AI#1208) merges: combined tool-ID list, both detection test groups, the CodeArts assertion folded into the shared skills-only registry test, and all adapterless init tests kept. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Kimi official code is now Kimi Code. The project detect folder has changed from .kimi to .kimi-code.
https://www.kimi.com/code
Summary by CodeRabbit
opsx-*command files..kimi-code(with legacy detection for.kimi)..kimito.kimi-codewhile preserving user-owned files.