-
Notifications
You must be signed in to change notification settings - Fork 1.5k
feat: add Kimi CLI as a supported AI tool #640
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
📝 WalkthroughWalkthroughThis pull request adds support for Kimi CLI as a new AI tool. Changes include registering Kimi CLI in the configuration, documenting it in the supported tools reference, and creating a changeset entry for release tracking. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
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 |
Review CompleteYour review story is ready! Comment !reviewfast on this PR to re-generate the story. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Adds Kimi CLI as a supported AI tool in OpenSpec, keeping the configuration and documentation in sync.
Changes:
- Added
Kimi CLIto theAI_TOOLSconfiguration withvalue: 'kimi'andskillsDir: '.kimi'. - Documented Kimi CLI’s skills and commands directories in the supported tools table.
- Added a changeset to release this as a patch update.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
src/core/config.ts |
Registers Kimi CLI in the AI_TOOLS array with consistent naming and skills directory. |
docs/supported-tools.md |
Updates the tools table and the list of available tool IDs to include Kimi CLI and its directory layout. |
.changeset/plenty-bats-smile.md |
Declares a patch-level changeset describing the addition of Kimi CLI support. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Greptile OverviewGreptile SummaryAdded Kimi CLI as a new supported AI tool, but the implementation is incomplete. Changes made:
Critical issue:
Required to complete this feature:
Confidence Score: 2/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant User
participant CLI as OpenSpec CLI
participant Init as InitCommand
participant Config as AI_TOOLS Config
participant Registry as CommandAdapterRegistry
participant FS as File System
User->>CLI: openspec init --tools kimi
CLI->>Init: execute()
Init->>Config: Get AI_TOOLS array
Config-->>Init: Returns Kimi CLI config
Note over Config: { name: 'Kimi CLI', value: 'kimi',<br/>skillsDir: '.kimi' }
Init->>FS: Create .kimi/skills/ directory
Init->>FS: Write 10 skill files
FS-->>Init: Skills written successfully
Init->>Registry: CommandAdapterRegistry.get('kimi')
Registry-->>Init: Returns undefined (no adapter)
Note over Init,Registry: Commands are skipped<br/>due to missing adapter
Init->>User: Display: "Commands skipped for: kimi (no adapter)"
Init->>User: Setup complete (skills only)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3 files reviewed, 1 comment
|
@Supgb let me know if skills are invocable directly for kimi cli: #640 (comment) |
|
Yes, it works as expected. @TabishB |
Description
Adds Kimi CLI as a new supported AI tool in OpenSpec.
Changes
AI_TOOLSarray insrc/core/config.tsdocs/supported-tools.mdwith Kimi CLI documentationSkills Directory
.kimi/skills/.kimi/commands/Summary by CodeRabbit
New Features
Documentation
✏️ Tip: You can customize this high-level summary in your review settings.