fix: use kiro-cli-chat directly to prevent orphan child processes

CHANGELOG.md

@@ -22,6 +22,7 @@ Repo: https://github.com/openclaw/acpx
 
 ### Fixes
 
+- Agents/kiro: use `kiro-cli-chat acp` for the built-in Kiro adapter command to avoid orphan child processes. (#129) Thanks @vokako.
 - Agents/cursor: recognize Cursor's `Session \"...\" not found` `session/load` error format so reconnects fall back to `session/new` instead of failing. (#162) Thanks @log-li.
 - Output/thinking: preserve line breaks in text-mode `[thinking]` output instead of flattening multi-line thought chunks into one line. (#144) Thanks @Huarong.
 - Sessions/load: fall back to a fresh ACP session when adapters reject `session/load` with JSON-RPC `-32601` or `-32602`, so persistent session reconnects do not crash on partial load support. (#174) Thanks @Bortlesboat.

README.md

@@ -339,7 +339,7 @@ Built-ins:
 | `iflow`    | native (`iflow --experimental-acp`)                                         | [iFlow CLI](https://github.com/iflow-ai/iflow-cli)                                                              |
 | `kilocode` | `npx -y @kilocode/cli acp`                                                  | [Kilocode](https://kilocode.ai)                                                                                 |
 | `kimi`     | native (`kimi acp`)                                                         | [Kimi CLI](https://github.com/MoonshotAI/kimi-cli)                                                              |
-| `kiro`     | native (`kiro-cli acp`)                                                     | [Kiro CLI](https://kiro.dev)                                                                                    |
+| `kiro`     | native (`kiro-cli-chat acp`)                                                | [Kiro CLI](https://kiro.dev)                                                                                    |
 | `opencode` | `npx -y opencode-ai acp`                                                    | [OpenCode](https://opencode.ai)                                                                                 |
 | `qoder`    | native (`qodercli --acp`)                                                   | [Qoder CLI](https://docs.qoder.com/cli/acp)                                                                     |
 | `qwen`     | native (`qwen --acp`)                                                       | [Qwen Code](https://github.com/QwenLM/qwen-code)                                                                |

agents/Kiro.md

@@ -1,5 +1,5 @@
 # Kiro
 
 - Built-in name: `kiro`
-- Default command: `kiro-cli acp`
+- Default command: `kiro-cli-chat acp`
 - Upstream: https://kiro.dev

agents/README.md

@@ -13,7 +13,7 @@ Built-in agents:
 - `iflow -> iflow --experimental-acp`
 - `kilocode -> npx -y @kilocode/cli acp`
 - `kimi -> kimi acp`
-- `kiro -> kiro-cli acp`
+- `kiro -> kiro-cli-chat acp`
 - `opencode -> npx -y opencode-ai acp`
 - `qoder -> qodercli --acp`
 - `qwen -> qwen --acp`
@@ -29,7 +29,7 @@ Harness-specific docs in this directory:
 - [iFlow](Iflow.md): built-in `iflow -> iflow --experimental-acp`
 - [Kilocode](Kilocode.md): built-in `kilocode -> npx -y @kilocode/cli acp`
 - [Kimi](Kimi.md): built-in `kimi -> kimi acp`
-- [Kiro](Kiro.md): built-in `kiro -> kiro-cli acp`
+- [Kiro](Kiro.md): built-in `kiro -> kiro-cli-chat acp`
 - [OpenCode](OpenCode.md): built-in `opencode -> npx -y opencode-ai acp`
 - [Qoder](Qoder.md): built-in `qoder -> qodercli --acp`
 - [Qwen](Qwen.md): built-in `qwen -> qwen --acp`

skills/acpx/SKILL.md

@@ -82,7 +82,7 @@ Friendly agent names resolve to commands:
 - `iflow` -> `iflow --experimental-acp`
 - `kilocode` -> `npx -y @kilocode/cli acp`
 - `kimi` -> `kimi acp`
-- `kiro` -> `kiro-cli acp`
+- `kiro` -> `kiro-cli-chat acp`
 - `opencode` -> `npx -y opencode-ai acp`
 - `qoder` -> `qodercli --acp`
   Forwards Qoder-native `--allowed-tools` and `--max-turns` startup flags from `acpx` session options.

src/agent-registry.ts

@@ -16,7 +16,7 @@ export const AGENT_REGISTRY: Record<string, string> = {
   iflow: "iflow --experimental-acp",
   kilocode: "npx -y @kilocode/cli acp",
   kimi: "kimi acp",
-  kiro: "kiro-cli acp",
+  kiro: "kiro-cli-chat acp",
   opencode: "npx -y opencode-ai acp",
   qoder: "qodercli --acp",
   qwen: "qwen --acp",

test/agent-registry.test.ts

@@ -37,6 +37,11 @@ test("trae built-in uses the standard traecli executable", () => {
   assert.equal(resolveAgentCommand("trae"), "traecli acp serve");
 });
 
+test("kiro built-in uses kiro-cli-chat directly", () => {
+  assert.equal(AGENT_REGISTRY.kiro, "kiro-cli-chat acp");
+  assert.equal(resolveAgentCommand("kiro"), "kiro-cli-chat acp");
+});
+
 test("listBuiltInAgents preserves the required example prefix and alphabetical tail", () => {
   const agents = listBuiltInAgents();
   assert.deepEqual(agents, Object.keys(AGENT_REGISTRY));