Skip to content

Lock Kimi ACP startup to subcommand syntax - #585

Draft
phodal with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-kimi-provider-acp-parameter
Draft

Lock Kimi ACP startup to subcommand syntax#585
phodal with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-kimi-provider-acp-parameter

Conversation

Copilot AI commented Jul 26, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

  • What changed:
    • Regression coverage: Assert the built-in Kimi preset launches: 
      kimi acp
      in both TypeScript and Rust ACP preset definitions.
    • Fitness evidence: Record this ACP provider behavior in unit-test evidence.
  • Why: New Kimi Code CLI releases require acp as a subcommand; the removed --acp flag causes startup and ACP handshake failures.

Validation

  • npm run lint
  • npm run test:run
  • UI screenshots or recording attached when applicable

Notes

  • Related issue: Kimi ACP launch compatibility.
  • Risks or follow-up work: Full fitness gates remain blocked by unrelated environment and existing-suite failures.

Copilot AI linked an issue Jul 26, 2026 that may be closed by this pull request
[Bug] 内置 Kimi provider 使用了已被移除的 "--acp" 参数 #583
Open
Copilot AI changed the title [WIP] Fix Kimi provider to use 'kimi acp' subcommand Lock Kimi ACP startup to subcommand syntax Jul 26, 2026
Copilot AI requested a review from phodal July 26, 2026 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@phodal phodal Awaiting requested review from phodal phodal is a code owner

Assignees

@phodal phodal

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug] 内置 Kimi provider 使用了已被移除的 "--acp" 参数

2 participants

@phodal