From a483e43f80595755ba7876f0095318a38a7d3c46 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Sat, 2 May 2026 08:34:11 +0100
Subject: [PATCH] refactor: hide command helper internals

---
 src/auto-reply/reply/commands-acp/shared.ts          | 6 +++---
 src/cli/daemon-cli/response.ts                       | 4 ++--
 src/cli/plugins-command-helpers.ts                   | 2 +-
 ui/src/ui/chat/slash-commands.browser-import.test.ts | 7 +++++--
 4 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/src/auto-reply/reply/commands-acp/shared.ts b/src/auto-reply/reply/commands-acp/shared.ts
index c4c633c06a1..4a0b3cc4e8e 100644
--- a/src/auto-reply/reply/commands-acp/shared.ts
+++ b/src/auto-reply/reply/commands-acp/shared.ts
@@ -13,13 +13,13 @@ import type { CommandHandlerResult, HandleCommandsParams } from "../commands-typ
 import { resolveAcpCommandChannel, resolveAcpCommandThreadId } from "./context.js";
 
 export const COMMAND = "/acp";
-export const ACP_SPAWN_USAGE =
+const ACP_SPAWN_USAGE =
   "Usage: /acp spawn [harness-id] [--mode persistent|oneshot] [--thread auto|here|off] [--bind here|off] [--cwd <path>] [--label <label>].";
-export const ACP_STEER_USAGE =
+const ACP_STEER_USAGE =
   "Usage: /acp steer [--session <session-key|session-id|session-label>] <instruction>";
 export const ACP_SET_MODE_USAGE =
   "Usage: /acp set-mode <mode> [session-key|session-id|session-label]";
-export const ACP_SET_USAGE = "Usage: /acp set <key> <value> [session-key|session-id|session-label]";
+const ACP_SET_USAGE = "Usage: /acp set <key> <value> [session-key|session-id|session-label]";
 export const ACP_CWD_USAGE = "Usage: /acp cwd <path> [session-key|session-id|session-label]";
 export const ACP_PERMISSIONS_USAGE =
   "Usage: /acp permissions <profile> [session-key|session-id|session-label]";
diff --git a/src/cli/daemon-cli/response.ts b/src/cli/daemon-cli/response.ts
index e2ea3566974..9e9382dedd8 100644
--- a/src/cli/daemon-cli/response.ts
+++ b/src/cli/daemon-cli/response.ts
@@ -41,7 +41,7 @@ export type DaemonActionResponse = {
   };
 };
 
-export function emitDaemonActionJson(payload: DaemonActionResponse) {
+function emitDaemonActionJson(payload: DaemonActionResponse) {
   defaultRuntime.writeJson(payload);
 }
 
@@ -90,7 +90,7 @@ export function buildDaemonServiceSnapshot(service: GatewayService, loaded: bool
   };
 }
 
-export function createNullWriter(): Writable {
+function createNullWriter(): Writable {
   return new Writable({
     write(_chunk, _encoding, callback) {
       callback();
diff --git a/src/cli/plugins-command-helpers.ts b/src/cli/plugins-command-helpers.ts
index 59c61a0f9fd..3e0f0935e88 100644
--- a/src/cli/plugins-command-helpers.ts
+++ b/src/cli/plugins-command-helpers.ts
@@ -220,7 +220,7 @@ export function parseNpmPrefixSpec(raw: string): string | null {
   return trimmed.slice("npm:".length).trim();
 }
 
-export const PREFERRED_CLAWHUB_FALLBACK_DECISION = {
+const PREFERRED_CLAWHUB_FALLBACK_DECISION = {
   FALLBACK_TO_NPM: "fallback_to_npm",
   STOP: "stop",
 } as const;
diff --git a/ui/src/ui/chat/slash-commands.browser-import.test.ts b/ui/src/ui/chat/slash-commands.browser-import.test.ts
index 830cae83d90..a32a27ae048 100644
--- a/ui/src/ui/chat/slash-commands.browser-import.test.ts
+++ b/ui/src/ui/chat/slash-commands.browser-import.test.ts
@@ -2,9 +2,12 @@
 import { readFile } from "node:fs/promises";
 import { describe, expect, it } from "vitest";
 
+type SlashCommandsModule = typeof import("./slash-commands.js");
+const browserImportPath = "./slash-commands.ts?browser-import";
+
 describe("slash command browser import", () => {
   it("builds fallback commands from the browser-safe shared registry", async () => {
-    const mod = await import("./slash-commands.ts?browser-import");
+    const mod = (await import(browserImportPath)) as SlashCommandsModule;
 
     expect(mod.SLASH_COMMANDS.find((command) => command.name === "think")).toMatchObject({
       name: "think",
@@ -22,7 +25,7 @@ describe("slash command browser import", () => {
       new URL("../../../../src/auto-reply/commands-registry.data.ts", import.meta.url),
       "utf8",
     );
-    const mod = await import("./slash-commands.ts?browser-import");
+    const mod = (await import(browserImportPath)) as SlashCommandsModule;
 
     expect(mod.SLASH_COMMANDS.some((command) => command.name === "think")).toBe(true);
     expect(slashCommands).toContain("commands-registry.shared.js");