Skip to content

fix(kilocode): install commands under .kilo/commands#3672

Open
WOLIKIMCHENG wants to merge 2 commits into
github:mainfrom
WOLIKIMCHENG:fix/3635-kilocode-commands
Open

fix(kilocode): install commands under .kilo/commands#3672
WOLIKIMCHENG wants to merge 2 commits into
github:mainfrom
WOLIKIMCHENG:fix/3635-kilocode-commands

Conversation

@WOLIKIMCHENG

Copy link
Copy Markdown
Contributor

Description

Closes #3635

Updates the Kilo Code integration to install Spec Kit command files under .kilo/commands, matching Kilo Code's current project command location.

This keeps the existing kilocode integration key and requires_cli=False, while adding .kilocode/workflows as a legacy fallback for extension and preset command registration/removal in existing projects.

Also updates focused tests and docs examples that referenced the old command directory.

Testing

  • .venv/bin/python -m pytest tests/integrations/test_integration_kilocode.py -v
  • .venv/bin/python -m pytest tests/integrations/test_integration_forge.py::TestForgeCommandRegistrar::test_registrar_does_not_affect_other_agents -v
  • .venv/bin/python -m pytest tests/test_agent_config_consistency.py -v
  • .venv/bin/python -m pytest tests/integrations/test_registry.py -v
  • .venv/bin/python -m pytest tests/integrations/test_integration_subcommand.py::TestIntegrationUpgrade::test_upgrade_migrates_opencode_legacy_dir -v
  • git diff --check -- AGENTS.md docs/reference/integrations.md docs/upgrade.md src/specify_cli/integrations/kilocode/__init__.py tests/integrations/test_integration_forge.py tests/integrations/test_integration_kilocode.py

Copilot AI left a comment

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.

Pull request overview

Updates Kilo Code command installation to its current .kilo/commands location while retaining legacy registration compatibility.

Changes:

  • Moves generated Kilo commands to .kilo/commands.
  • Adds .kilocode/workflows fallback handling.
  • Updates focused tests and documentation.
Show a summary per file
File Description
src/specify_cli/integrations/kilocode/__init__.py Updates Kilo paths and legacy fallback.
tests/integrations/test_integration_kilocode.py Tests canonical and legacy paths.
tests/integrations/test_integration_forge.py Updates Kilo registrar expectation.
docs/reference/integrations.md Documents the new command location.
docs/upgrade.md Updates Kilo cleanup examples.
AGENTS.md Updates integration guidance examples.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comments suppressed due to low confidence (1)

docs/upgrade.md:252

  • This duplicate-command cleanup example also omits the legacy .kilocode/workflows directory, which is where stale Kilo commands from pre-migration installations may remain. Show both Kilo command locations so the documented remediation actually finds the duplicates.
# Find the agent folder (example: .kilo/commands/)
cd .kilo/commands/
  • Files reviewed: 6/6 changed files
  • Comments generated: 2
  • Review effort level: Medium

Comment thread src/specify_cli/integrations/kilocode/__init__.py
Comment thread docs/upgrade.md Outdated
@mnriem

mnriem commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

Please address Copilot feedback

@ItachiSan

Copy link
Copy Markdown

Hello, thanks for posting this 😄
Kilo now has a CLI command named kilo; shouldn't then the key be renamed to it?

@WOLIKIMCHENG

Copy link
Copy Markdown
Contributor Author

Addressed the Copilot feedback for Kilo command directory migration. Kilo upgrades now handle .kilocode/workflows to .kilo/commands by migrating core commands, reconciling enabled extension commands into the canonical directory, preserving disabled extension and user-owned legacy files, and refusing the migration before mutation when Kilo preset command overrides are installed. The duplicate-command upgrade docs now tell users to inspect both canonical and legacy Kilo command directories.

Copilot AI left a comment

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.

Review details

  • Files reviewed: 10/10 changed files
  • Comments generated: 1
  • Review effort level: Medium

| `junie` | `.junie/commands` |
| `kilocode` | `.kilo/commands` |
| `kiro-cli` | `.kiro/prompts` |
| `omp` | `.omp/commands` |

@mnriem mnriem left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please address Copilot feedback and resolve conflicts

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]: Update Kilo(Code) support

4 participants