Skip to content

Conversation

@Supgb
Copy link

@Supgb Supgb commented Feb 1, 2026

Description

Adds Kimi CLI as a new supported AI tool in OpenSpec.

Changes

  • Added Kimi CLI configuration to AI_TOOLS array in src/core/config.ts
  • Updated docs/supported-tools.md with Kimi CLI documentation

Skills Directory

  • Skills are installed to .kimi/skills/
  • Commands are installed to .kimi/commands/

Summary by CodeRabbit

  • New Features

    • Added Kimi CLI as a supported AI tool, expanding available options.
  • Documentation

    • Updated tool directory reference and setup documentation to include Kimi CLI with configuration details and command locations.

✏️ Tip: You can customize this high-level summary in your review settings.

Copilot AI review requested due to automatic review settings February 1, 2026 05:58
@Supgb Supgb requested a review from TabishB as a code owner February 1, 2026 05:58
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 1, 2026

📝 Walkthrough

Walkthrough

This 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

Cohort / File(s) Summary
Configuration & Setup
src/core/config.ts
Added Kimi CLI entry to AI_TOOLS array with value "kimi", availability enabled, and skills directory path ".kimi".
Documentation
docs/supported-tools.md
Added Kimi CLI to the Tool Directory Reference table and appended kimi to the available tool IDs list in non-interactive setup examples.
Release Management
.changeset/plenty-bats-smile.md
Created changeset entry declaring a patch for @fission-ai/openspec to document Kimi CLI support addition.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Boing! A new tool hops into view,
Kimi CLI joins the crew!
From config to docs, we've made it clear,
This AI companion is now here!
Adding support with just a few lines,
Our project shines with brighter signs! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the main change: adding Kimi CLI as a supported AI tool, which aligns with all file modifications in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

❤️ Share

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

@vibe-kanban-cloud
Copy link

Review Complete

Your review story is ready!

View Story

Comment !reviewfast on this PR to re-generate the story.

Copy link
Contributor

Copilot AI left a 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 CLI to the AI_TOOLS configuration with value: 'kimi' and skillsDir: '.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-apps
Copy link

greptile-apps bot commented Feb 1, 2026

Greptile Overview

Greptile Summary

Added Kimi CLI as a new supported AI tool, but the implementation is incomplete.

Changes made:

  • Added Kimi CLI configuration to AI_TOOLS array with .kimi as the skills directory
  • Updated documentation to list Kimi CLI in the tool directory reference and available tool IDs

Critical issue:

  • Missing command adapter implementation - When users run openspec init --tools kimi, only skills will be installed (in .kimi/skills/) but commands won't be generated because there's no corresponding adapter in the CommandAdapterRegistry. Users will see "Commands skipped for: kimi (no adapter)".

Required to complete this feature:

  • Create src/core/command-generation/adapters/kimi.ts following the pattern of similar CLI tools like Gemini or Qwen
  • Export the adapter in src/core/command-generation/adapters/index.ts
  • Register the adapter in src/core/command-generation/registry.ts

Confidence Score: 2/5

  • This PR has incomplete implementation that will result in degraded functionality for users
  • The score reflects a critical missing component - the command adapter. While the code won't break existing functionality, Kimi CLI users will only get partial support (skills but no commands). This is a significant gap that makes the feature incomplete
  • Pay close attention to src/core/config.ts - the adapter implementation is required before this PR should be merged

Important Files Changed

Filename Overview
.changeset/plenty-bats-smile.md Added changeset documenting the new Kimi CLI support
docs/supported-tools.md Added Kimi CLI to the tool directory reference table and available tool IDs list
src/core/config.ts Added Kimi CLI configuration to AI_TOOLS array, but missing corresponding command adapter implementation

Sequence Diagram

sequenceDiagram
    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)
Loading

Copy link

@greptile-apps greptile-apps bot left a 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

Edit Code Review Agent Settings | Greptile

@TabishB
Copy link
Contributor

TabishB commented Feb 2, 2026

@Supgb let me know if skills are invocable directly for kimi cli: #640 (comment)

@Supgb
Copy link
Author

Supgb commented Feb 2, 2026

Yes, it works as expected. @TabishB

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.

2 participants