fix: remove kapi runtime alias (#225)

Squash merge PR #225 after kapi-agent approval and successful required checks.

Author: devkade

bin/kapi.mjs

@@ -14,8 +14,8 @@ git grep -n -i -E 'kapi|ilchul' -- ':!package-lock.json' ':!node_modules'
 
 | Surface | Current state | Classification | Follow-up direction |
 | --- | --- | --- | --- |
-| Package metadata | `package.json` is still named `ka-pi`; package bins include public `ilchul` and `kapi-review`. | package brand / external review integration | Do not retain a `kapi` runtime CLI alias. Prefer `ilchul` for public runtime docs and examples; reserve `runctl` for internal runtime implementation naming. Keep `kapi-review` while the GitHub review integration uses that command. |
-| CLI runtime entrypoint | `src/cli/runctl-cli.ts` and related tests use semantic runtime names. This slice keeps the public `ilchul` command wired through the internal `bin/runctl.mjs` wrapper and semantic `parseRuntimeArgs` / `runRuntimeCli` exports without retaining a `kapi` runtime CLI alias. | public brand command with internal semantic implementation | Internal runtime callers should import from `src/cli/runctl-cli.ts` and use semantic runtime exports; product-named command aliases stay at the package/bin edge. |
+| Package metadata | `package.json` is still named `ka-pi`; package bins include public `ilchul` and external `kapi-review`, but no `kapi` runtime bin. | package brand / external review integration | Do not retain a `kapi` runtime CLI alias. Prefer `ilchul` for public runtime docs and examples; reserve `runctl` for internal runtime implementation naming. Keep `kapi-review` while the GitHub review integration uses that command. |
+| CLI runtime entrypoint | `src/cli/runctl-cli.ts` and related tests use semantic runtime names. This slice keeps the public `ilchul` command wired through the internal `bin/runctl.mjs` wrapper and semantic `parseRuntimeArgs` / `runRuntimeCli` exports, removes the old `bin/kapi.mjs` shim, and verifies package-local/prefix installs do not expose `kapi`. | public brand command with internal semantic implementation | Internal runtime callers should import from `src/cli/runctl-cli.ts` and use semantic runtime exports; product-named runtime command aliases must not be reintroduced at the package/bin edge. |
 | CLI setup helper | `setup:cli` still invokes `scripts/setup-runctl-cli.mjs`, but its completion text now points to `ilchul --help`. | internal semantic filename plus brand-facing user guidance | Keep output brand-facing where users see it and implementation filename semantic internally. |
 | Slash commands and workflow IDs | `/kapi-*`, `kapi-ralph`, `kapi-autoresearch`, and related state fields remain serialized workflow contracts. | persisted compatibility identifiers | Do not rename without an explicit compatibility/deprecation migration. |
 | Storage/config | `.ilchul`, `~/.ilchul`, `.kapi` historical references, and `ILCHUL_*` environment variables appear in docs/tests/runtime setup. | storage/config namespace | `.ilchul` is active storage; `.kapi` is legacy evidence only and must not be deleted by this issue. |
@@ -40,18 +40,28 @@ git grep -n -i -E 'kapi|ilchul' -- ':!package-lock.json' ':!node_modules'
 - Presentation registration/UI action helpers now use semantic workflow names (`registerWorkflowExtension`, `registerWorkflowCommandsAndTools`, `registerWorkflowTools`, `WorkflowUiActionContext`, `installWorkflowAutocomplete`, `toggleWorkflowWidgetExpanded`) while leaving public `kapi_*` tool names and `/kapi-*` contracts intact.
 - Adapter/domain private helper names now use semantic state/workflow naming for symlink guards, managed autoresearch symlinks, worker branch validation, and prompt rule rendering while preserving user-facing Kapi contract text.
 - Review gate helper exports now use semantic review-gate names (`ReviewGateInput`, `ReviewGateResult`, `buildReviewGate`, `runReviewGateCli`, `parseReviewGateArgs`, `REVIEW_GATE_MAX_CHANGED_LINES`) while preserving the `kapi-review` CLI and `kapi-agent` review contract text.
+- Runtime package metadata now exposes only the `ilchul` runtime bin and the external `kapi-review` review harness; `kapi` is absent from `package.json` / `package-lock.json`, `bin/kapi.mjs` is removed, and `test/cli-bin.test.ts` asserts package-local and isolated-prefix installs do not expose a `kapi` runtime command.
 
-## Residual scan after service filename rename
+## Residual scan after runtime alias removal
 
 ```text
 find src -iname '*kapi*' -print
 src/.kapi
 src/.kapi/lanes/kapi-autoresearch
 src/.kapi/lanes/kapi-autoresearch/kapi-ux-개선-계획
 src/cli/kapi-review-cli.ts
+
+node -e 'const p=require("./package.json"); console.log(JSON.stringify(p.bin,null,2))'
+{
+  "ilchul": "./bin/runctl.mjs",
+  "kapi-review": "./src/cli/kapi-review-cli.ts"
+}
+
+rg -n '"kapi"\\s*:|bin/kapi|npm exec -- kapi|kapi compatibility alias|kapi --help' package.json package-lock.json README.md docs test bin src scripts
+ test/cli-bin.test.ts:41:test("package metadata exposes ilchul runtime bin without kapi compatibility alias", async () => {
 ```
 
-Remaining filename hits are legacy fixture state under `src/.kapi` and the literal `kapi-review` external integration CLI. The former application service/factory filename hits moved to semantic `workflow-service` paths in this slice.
+Remaining `src/` filename hits are legacy fixture state under `src/.kapi` and the literal `kapi-review` external integration CLI. Runtime package metadata no longer exposes a `kapi` bin, and the old direct `bin/kapi.mjs` shim is removed.
 
 ## Remaining justified product-name identifiers
 

docs/product-name-audit.md

@@ -6,7 +6,6 @@
   "type": "module",
   "bin": {
     "ilchul": "./bin/runctl.mjs",
-    "kapi": "./bin/kapi.mjs",
     "kapi-review": "./src/cli/kapi-review-cli.ts"
   },
   "keywords": [

package-lock.json

@@ -10,7 +10,7 @@ const execFileAsync = promisify(execFile);
 const repoRoot = process.cwd();
 const setupScript = path.join(repoRoot, "scripts", "setup-runctl-cli.mjs");
 
-test("package-local ilchul bin runs help while kapi stays a compatibility alias", async () => {
+test("package-local ilchul bin runs help without exposing kapi runtime alias", async () => {
   const binPath = path.join(repoRoot, "bin", "runctl.mjs");
   const mode = (await stat(binPath)).mode;
   assert.notEqual(mode & 0o111, 0, "bin/runctl.mjs must be executable for npm bin shims");
@@ -27,16 +27,21 @@ test("package-local ilchul bin runs help while kapi stays a compatibility alias"
   assert.doesNotMatch(ilchul.stdout, /kapi remains available|compatibility alias/i);
   assert.doesNotMatch(ilchul.stderr, /Permission denied/i);
 
-  const kapi = await execFileAsync("npm", ["exec", "--", "kapi", "--help"], { cwd: repoRoot, timeout: 20_000 });
-  assert.match(kapi.stdout, /kapi — workflow runtime/);
-  assert.match(kapi.stdout, /kapi start ralph/);
-  assert.doesNotMatch(kapi.stderr, /Permission denied/i);
+  await assert.rejects(
+    execFileAsync("npm", ["exec", "--", "kapi", "--help"], { cwd: repoRoot, timeout: 20_000 }),
+    (error: unknown) => {
+      const failure = error as { code?: number; stdout?: string; stderr?: string };
+      assert.notEqual(failure.code, 0);
+      assert.doesNotMatch(`${failure.stdout ?? ""}\n${failure.stderr ?? ""}`, /kapi — workflow runtime|kapi start ralph/);
+      return true;
+    },
+  );
 });
 
-test("package metadata exposes ilchul runtime bin and kapi compatibility alias", async () => {
+test("package metadata exposes ilchul runtime bin without kapi compatibility alias", async () => {
   const packageJson = JSON.parse(await readFile(path.join(repoRoot, "package.json"), "utf8")) as { bin: Record<string, string> };
   assert.equal(packageJson.bin.ilchul, "./bin/runctl.mjs");
-  assert.equal(packageJson.bin.kapi, "./bin/kapi.mjs");
+  assert.equal(packageJson.bin.kapi, undefined);
 });
 
 test("setup helper uses Windows-safe npm execution and prefix paths", async () => {
@@ -74,7 +79,7 @@ test("setup helper uses Windows-safe npm execution and prefix paths", async () =
   ]);
 });
 
-test("setup helper installs ilchul into an isolated prefix", async () => {
+test("setup helper installs only ilchul into an isolated prefix", async () => {
   const prefix = await mkdtemp(path.join(os.tmpdir(), "kapi-cli-setup-"));
   const result = await execFileAsync("npm", ["run", "setup:cli", "--", "--prefix", prefix], { cwd: repoRoot, timeout: 120_000 });
   assert.match(result.stdout, /Ilchul CLI setup complete/);
@@ -89,10 +94,14 @@ test("setup helper installs ilchul into an isolated prefix", async () => {
   assert.doesNotMatch(ilchul.stderr, /Permission denied/i);
 
   const kapiCommand = path.join(binDir, process.platform === "win32" ? "kapi.cmd" : "kapi");
-  const kapi = await execFileAsync(kapiCommand, ["--help"], { cwd: repoRoot, timeout: 20_000 });
-  assert.match(kapi.stdout, /kapi — workflow runtime/);
-  assert.match(kapi.stdout, /kapi start ralph/);
-  assert.doesNotMatch(kapi.stderr, /Permission denied/i);
+  await assert.rejects(
+    execFileAsync(kapiCommand, ["--help"], { cwd: repoRoot, timeout: 20_000 }),
+    (error: unknown) => {
+      const failure = error as NodeJS.ErrnoException & { stdout?: string; stderr?: string };
+      assert.equal(failure.code, "ENOENT");
+      return true;
+    },
+  );
 });
 
 test("package install works when dev dependencies are omitted", async () => {