argsbarg 6.2.2 → 6.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "argsbarg",
-  "version": "6.2.2",
+  "version": "6.3.1",
   "main": "./src/index.ts",
   "module": "./src/index.ts",
   "dependencies": {

package/src/builtins/builtins.test.ts

@@ -9,7 +9,7 @@ import { cliBuiltinDocsGroup } from "../docs/builtin.ts";
 import { resolveCapabilities } from "../runtime/capabilities.ts";
 import { completionBashScript, completionFishScript, completionZshScript } from ".";
 import { cliBuiltinConfigureCommand, configureBuiltinOptions } from "./configure.ts";
-import { configureCommandDescription, configureSyncOptionDescription } from "./configure-copy.ts";
+import { configureCommandDescription, configureRefreshOptionDescription } from "./configure-copy.ts";
 import { exportPresentationBuiltins } from "./export.ts";
 import { cliBuiltinMcpCommand } from "./mcp.ts";
 import { cliParseRoot, cliPresentationRoot } from "./presentation.ts";
@@ -43,7 +43,7 @@ describe("builtins help copy", () => {
     expect(configure.description).toContain("MCP config");
     expect(configure.notes).toContain("brew upgrade");
     const names = configureBuiltinOptions(fixture).map((o) => o.name);
-    expect(names).toContain("sync");
+    expect(names).toContain("refresh");
     expect(names).toContain("remove-all");
     expect(names).toContain("status");
     const yesOpt = configureBuiltinOptions(fixture).find((o) => o.name === "yes");
@@ -54,7 +54,7 @@ describe("builtins help copy", () => {
     const caps = resolveCapabilities(noMcp);
     expect(configureCommandDescription(noMcp, caps)).toBe("Set up agent skills for this app (binary via Homebrew).");
     expect(configureCommandDescription(noMcp, caps)).not.toContain("MCP");
-    expect(configureSyncOptionDescription(noMcp, caps)).not.toContain("MCP");
+    expect(configureRefreshOptionDescription(noMcp, caps)).not.toContain("MCP");
     const configure = cliBuiltinConfigureCommand(noMcp);
     expect(configure.description).not.toContain("MCP");
   });
@@ -63,7 +63,7 @@ describe("builtins help copy", () => {
   test("configure notes mention brew upgrade and interactive configure", () => {
     const configure = cliBuiltinConfigureCommand(fixture);
     expect(configure.notes).toContain("brew upgrade");
-    expect(configure.notes).toContain("configure --sync --yes");
+    expect(configure.notes).toContain("configure --refresh --yes");
     expect(configure.notes).toContain(`${fixture.key} configure`);
 
     const withConfig: CliProgram = {
@@ -77,11 +77,11 @@ describe("builtins help copy", () => {
 
   test("configure -y parses as --yes", () => {
     const root = cliParseRoot(fixture);
-    const pr = postParseValidate(root, parse(root, ["configure", "-y", "--sync"]));
+    const pr = postParseValidate(root, parse(root, ["configure", "-y", "--refresh"]));
     expect(pr.kind).toBe(ParseKind.Ok);
     if (pr.kind === ParseKind.Ok) {
       expect(pr.opts.yes).toBe("1");
-      expect(pr.opts.sync).toBe("1");
+      expect(pr.opts.refresh).toBe("1");
     }
   });
 
@@ -134,7 +134,7 @@ describe("completion emitters", () => {
     const schema = cliPresentationRoot(fixture);
     const bash = completionBashScript(schema);
     expect(bash).toContain("hello");
-    expect(bash).toContain("--sync");
+    expect(bash).toContain("--refresh");
   });
 
   test("zsh script registers compdef", () => {

package/src/builtins/config.test.ts

@@ -30,8 +30,8 @@ describe("builtins/config", () => {
   /** Tests that config get redacts sensitive values. */
   test("config get redacts sensitive values", async () => {
     const dir = mkdtempSync(join(tmpdir(), "cfg-builtin-"));
-    const prevHome = process.env.HOME;
-    process.env.HOME = dir;
+    const prevTestHome = process.env.TEST_USER_HOME;
+    process.env.TEST_USER_HOME = dir;
     const prev = process.env.API_TOKEN;
     delete process.env.API_TOKEN;
     try {
@@ -43,8 +43,8 @@ describe("builtins/config", () => {
       expect(result.exitCode).toBe(0);
       expect(result.stdout.trim()).toBe("REDACTED");
     } finally {
-      if (prevHome === undefined) delete process.env.HOME;
-      else process.env.HOME = prevHome;
+      if (prevTestHome === undefined) delete process.env.TEST_USER_HOME;
+      else process.env.TEST_USER_HOME = prevTestHome;
       if (prev !== undefined) process.env.API_TOKEN = prev;
       rmSync(dir, { recursive: true, force: true });
     }
@@ -53,8 +53,8 @@ describe("builtins/config", () => {
   /** Tests that config get --json redacts sensitive as { set: true }. */
   test("config get --json redacts sensitive as { set: true }", async () => {
     const dir = mkdtempSync(join(tmpdir(), "cfg-builtin-"));
-    const prevHome = process.env.HOME;
-    process.env.HOME = dir;
+    const prevTestHome = process.env.TEST_USER_HOME;
+    process.env.TEST_USER_HOME = dir;
     const prev = process.env.API_TOKEN;
     delete process.env.API_TOKEN;
     try {
@@ -66,8 +66,8 @@ describe("builtins/config", () => {
       expect(result.exitCode).toBe(0);
       expect(JSON.parse(result.stdout)).toEqual({ set: true });
     } finally {
-      if (prevHome === undefined) delete process.env.HOME;
-      else process.env.HOME = prevHome;
+      if (prevTestHome === undefined) delete process.env.TEST_USER_HOME;
+      else process.env.TEST_USER_HOME = prevTestHome;
       if (prev !== undefined) process.env.API_TOKEN = prev;
       rmSync(dir, { recursive: true, force: true });
     }
@@ -76,8 +76,8 @@ describe("builtins/config", () => {
   /** Tests that config set writes and resolves without required exit. */
   test("config set writes and resolves without required exit", async () => {
     const dir = mkdtempSync(join(tmpdir(), "cfg-builtin-"));
-    const prevHome = process.env.HOME;
-    process.env.HOME = dir;
+    const prevTestHome = process.env.TEST_USER_HOME;
+    process.env.TEST_USER_HOME = dir;
     const prev = process.env.API_TOKEN;
     delete process.env.API_TOKEN;
     try {
@@ -88,8 +88,8 @@ describe("builtins/config", () => {
       const get = await new Cli(program).invoke(["configure", "get", "port"]);
       expect(get.stdout.trim()).toBe("9090");
     } finally {
-      if (prevHome === undefined) delete process.env.HOME;
-      else process.env.HOME = prevHome;
+      if (prevTestHome === undefined) delete process.env.TEST_USER_HOME;
+      else process.env.TEST_USER_HOME = prevTestHome;
       if (prev !== undefined) process.env.API_TOKEN = prev;
       rmSync(dir, { recursive: true, force: true });
     }
@@ -97,8 +97,8 @@ describe("builtins/config", () => {
 
   test("config set --from-env stores binding without literal", async () => {
     const dir = mkdtempSync(join(tmpdir(), "cfg-from-env-"));
-    const prevHome = process.env.HOME;
-    process.env.HOME = dir;
+    const prevTestHome = process.env.TEST_USER_HOME;
+    process.env.TEST_USER_HOME = dir;
     const prev = process.env.API_TOKEN;
     process.env.API_TOKEN = "from-env";
     try {
@@ -114,8 +114,8 @@ describe("builtins/config", () => {
       expect(onDisk.apiToken).toBeUndefined();
       expect(readBindings(onDisk).apiToken).toBe("env");
     } finally {
-      if (prevHome === undefined) delete process.env.HOME;
-      else process.env.HOME = prevHome;
+      if (prevTestHome === undefined) delete process.env.TEST_USER_HOME;
+      else process.env.TEST_USER_HOME = prevTestHome;
       if (prev === undefined) delete process.env.API_TOKEN;
       else process.env.API_TOKEN = prev;
       rmSync(dir, { recursive: true, force: true });

package/src/builtins/configure-copy.ts

@@ -37,7 +37,7 @@ export function configureCommandDescription(program: CliProgram, caps: CliCapabi
   return `Set up ${prose(program, caps)} for this app (binary via Homebrew).`;
 }
 
-export function configureSyncOptionDescription(program: CliProgram, caps: CliCapabilities): string {
+export function configureRefreshOptionDescription(program: CliProgram, caps: CliCapabilities): string {
   return `Refresh installed ${short(program, caps)}. Used by Homebrew post_install.`;
 }
 
@@ -54,7 +54,7 @@ export function configureCommandNotes(program: CliProgram, _caps: CliCapabilitie
     "Set up agent artifacts after the binary is installed via Homebrew (see README for tap install).",
     "",
     "Homebrew post_install runs:",
-    `  ${app} configure --sync --yes`,
+    `  ${app} configure --refresh --yes`,
     "",
     "Interactive setup (per target):",
     `  ${app} configure`,

package/src/builtins/configure.ts

@@ -12,7 +12,7 @@ import { configureConfigSubcommands } from "./config.ts";
 import {
   configureCommandDescription,
   configureCommandNotes,
-  configureSyncOptionDescription,
+  configureRefreshOptionDescription,
 } from "./configure-copy.ts";
 
 /** Hidden fallback leaf for bare `configure` (interactive / flag modes). */
@@ -23,8 +23,8 @@ export function configureBuiltinOptions(root: CliProgram): CliOption[] {
   const caps = resolveCapabilities(root);
   const opts: CliOption[] = [
     {
-      name: "sync",
-      description: configureSyncOptionDescription(root, caps),
+      name: "refresh",
+      description: configureRefreshOptionDescription(root, caps),
       kind: CliOptionKind.Presence,
     },
     {
@@ -50,7 +50,7 @@ export function configureBuiltinOptions(root: CliProgram): CliOption[] {
     },
     {
       name: "yes",
-      description: "Skip confirmation (required for --sync, --remove-all, --remove-config).",
+      description: "Skip confirmation (required for --refresh, --remove-all, --remove-config).",
       kind: CliOptionKind.Presence,
       shortName: "y",
     },

package/src/config/context.test.ts

@@ -29,8 +29,8 @@ describe("config/context", () => {
   /** Tests that AppConfigSnapshot get, require, read, set. */
   test("AppConfigSnapshot get, require, read, set", () => {
     const dir = mkdtempSync(join(tmpdir(), "ctx-test-"));
-    const prevHome = process.env.HOME;
-    process.env.HOME = dir;
+    const prevTestHome = process.env.TEST_USER_HOME;
+    process.env.TEST_USER_HOME = dir;
     const prevToken = process.env.API_TOKEN;
     delete process.env.API_TOKEN;
     try {
@@ -48,8 +48,8 @@ describe("config/context", () => {
       expect(ctx.get("note")).toBe("updated");
       expect(ctx.read().note).toBe("updated");
     } finally {
-      if (prevHome === undefined) delete process.env.HOME;
-      else process.env.HOME = prevHome;
+      if (prevTestHome === undefined) delete process.env.TEST_USER_HOME;
+      else process.env.TEST_USER_HOME = prevTestHome;
       if (prevToken === undefined) delete process.env.API_TOKEN;
       else process.env.API_TOKEN = prevToken;
       rmSync(dir, { recursive: true, force: true });
@@ -82,8 +82,8 @@ describe("config/context", () => {
 
   test("AppConfigSnapshot unsafe read/write", () => {
     const dir = mkdtempSync(join(tmpdir(), "ctx-unsafe-"));
-    const prevHome = process.env.HOME;
-    process.env.HOME = dir;
+    const prevTestHome = process.env.TEST_USER_HOME;
+    process.env.TEST_USER_HOME = dir;
     const prevToken = process.env.API_TOKEN;
     delete process.env.API_TOKEN;
     try {
@@ -95,8 +95,8 @@ describe("config/context", () => {
       ctx.setUnsafe("note", "raw");
       expect(ctx.getUnsafe("note")).toBe("raw");
     } finally {
-      if (prevHome === undefined) delete process.env.HOME;
-      else process.env.HOME = prevHome;
+      if (prevTestHome === undefined) delete process.env.TEST_USER_HOME;
+      else process.env.TEST_USER_HOME = prevTestHome;
       if (prevToken === undefined) delete process.env.API_TOKEN;
       else process.env.API_TOKEN = prevToken;
       rmSync(dir, { recursive: true, force: true });
@@ -105,8 +105,8 @@ describe("config/context", () => {
 
   test("EmptyAppConfigSnapshot unsafe read/write", () => {
     const dir = mkdtempSync(join(tmpdir(), "ctx-empty-unsafe-"));
-    const prevHome = process.env.HOME;
-    process.env.HOME = dir;
+    const prevTestHome = process.env.TEST_USER_HOME;
+    process.env.TEST_USER_HOME = dir;
     try {
       const programWithoutConfig: CliProgram = {
         key: "rawapp",
@@ -119,8 +119,8 @@ describe("config/context", () => {
       expect(empty.getUnsafe("custom")).toBe(42);
       expect(empty.readUnsafe().custom).toBe(42);
     } finally {
-      if (prevHome === undefined) delete process.env.HOME;
-      else process.env.HOME = prevHome;
+      if (prevTestHome === undefined) delete process.env.TEST_USER_HOME;
+      else process.env.TEST_USER_HOME = prevTestHome;
       rmSync(dir, { recursive: true, force: true });
     }
   });

package/src/config/file.test.ts

@@ -37,13 +37,13 @@ const program: CliProgram = {
 
 function withHome<T>(fn: (home: string) => T): T {
   const home = mkdtempSync(join(tmpdir(), "cfg-test-"));
-  const prevHome = process.env.HOME;
-  process.env.HOME = home;
+  const prevTestHome = process.env.TEST_USER_HOME;
+  process.env.TEST_USER_HOME = home;
   try {
     return fn(home);
   } finally {
-    if (prevHome === undefined) delete process.env.HOME;
-    else process.env.HOME = prevHome;
+    if (prevTestHome === undefined) delete process.env.TEST_USER_HOME;
+    else process.env.TEST_USER_HOME = prevTestHome;
     rmSync(home, { recursive: true, force: true });
   }
 }

package/src/configure/artifacts/binary-placement.test.ts

@@ -84,14 +84,14 @@ describe("isAppInstalled", () => {
   /** Tests that false when not on PATH and no local copy. */
   test("false when not on PATH and no local copy", () => {
     const home = mkdtempSync(join(tmpdir(), "argsbarg-placement-home-"));
-    const prevHome = process.env.HOME;
-    process.env.HOME = home;
+    const prevTestHome = process.env.TEST_USER_HOME;
+    process.env.TEST_USER_HOME = home;
     process.env.PATH = tmp;
     try {
       expect(isAppInstalled(program)).toBe(false);
     } finally {
-      if (prevHome === undefined) delete process.env.HOME;
-      else process.env.HOME = prevHome;
+      if (prevTestHome === undefined) delete process.env.TEST_USER_HOME;
+      else process.env.TEST_USER_HOME = prevTestHome;
       rmSync(home, { recursive: true, force: true });
     }
   });

package/src/configure/artifacts/mcp-codex.test.ts

@@ -9,17 +9,17 @@ import { join } from "node:path";
 import { codexMcpHasServer, readCodexMcpEntry, resolveCodexConfigPath } from "./mcp-codex.ts";
 
 let home: string;
-let prevHome: string | undefined;
+let prevTestHome: string | undefined;
 
 beforeEach(() => {
   home = mkdtempSync(join(tmpdir(), "argsbarg-codex-"));
-  prevHome = process.env.HOME;
-  process.env.HOME = home;
+  prevTestHome = process.env.TEST_USER_HOME;
+  process.env.TEST_USER_HOME = home;
 });
 
 afterEach(() => {
-  if (prevHome === undefined) delete process.env.HOME;
-  else process.env.HOME = prevHome;
+  if (prevTestHome === undefined) delete process.env.TEST_USER_HOME;
+  else process.env.TEST_USER_HOME = prevTestHome;
   rmSync(home, { recursive: true, force: true });
 });
 

package/src/configure/artifacts/mcp-openclaw.test.ts

@@ -9,17 +9,17 @@ import { join } from "node:path";
 import { openclawMcpHasServer, resolveOpenclawConfigPath } from "./mcp-openclaw.ts";
 
 let home: string;
-let prevHome: string | undefined;
+let prevTestHome: string | undefined;
 
 beforeEach(() => {
   home = mkdtempSync(join(tmpdir(), "argsbarg-openclaw-"));
-  prevHome = process.env.HOME;
-  process.env.HOME = home;
+  prevTestHome = process.env.TEST_USER_HOME;
+  process.env.TEST_USER_HOME = home;
 });
 
 afterEach(() => {
-  if (prevHome === undefined) delete process.env.HOME;
-  else process.env.HOME = prevHome;
+  if (prevTestHome === undefined) delete process.env.TEST_USER_HOME;
+  else process.env.TEST_USER_HOME = prevTestHome;
   rmSync(home, { recursive: true, force: true });
 });
 

package/src/configure/artifacts/mcp-opencode.test.ts

@@ -26,20 +26,20 @@ const fixture: CliProgram = {
 };
 
 let home: string;
-let prevHome: string | undefined;
+let prevTestHome: string | undefined;
 let prevXdg: string | undefined;
 
 beforeEach(() => {
   home = mkdtempSync(join(tmpdir(), "argsbarg-opencode-"));
-  prevHome = process.env.HOME;
+  prevTestHome = process.env.TEST_USER_HOME;
   prevXdg = process.env.XDG_CONFIG_HOME;
-  process.env.HOME = home;
+  process.env.TEST_USER_HOME = home;
   delete process.env.XDG_CONFIG_HOME;
 });
 
 afterEach(() => {
-  if (prevHome === undefined) delete process.env.HOME;
-  else process.env.HOME = prevHome;
+  if (prevTestHome === undefined) delete process.env.TEST_USER_HOME;
+  else process.env.TEST_USER_HOME = prevTestHome;
   if (prevXdg === undefined) delete process.env.XDG_CONFIG_HOME;
   else process.env.XDG_CONFIG_HOME = prevXdg;
   rmSync(home, { recursive: true, force: true });

package/src/configure/artifacts/status.test.ts

@@ -12,17 +12,17 @@ import { printInstallStatus } from "./status.ts";
 import { resolveInstallTargetPreview } from "./target-scope.ts";
 
 let home: string;
-let prevHome: string | undefined;
+let prevTestHome: string | undefined;
 
 beforeEach(() => {
   home = mkdtempSync(join(tmpdir(), "argsbarg-status-"));
-  prevHome = process.env.HOME;
-  process.env.HOME = home;
+  prevTestHome = process.env.TEST_USER_HOME;
+  process.env.TEST_USER_HOME = home;
 });
 
 afterEach(() => {
-  if (prevHome === undefined) delete process.env.HOME;
-  else process.env.HOME = prevHome;
+  if (prevTestHome === undefined) delete process.env.TEST_USER_HOME;
+  else process.env.TEST_USER_HOME = prevTestHome;
   rmSync(home, { recursive: true, force: true });
 });
 

package/src/configure/configure.test.ts

@@ -33,20 +33,20 @@ const fixture: CliProgram = {
 };
 
 let home: string;
-let prevHome: string | undefined;
+let prevTestHome: string | undefined;
 let prevXdg: string | undefined;
 
 beforeEach(() => {
   home = mkdtempSync(join(tmpdir(), "argsbarg-configure-"));
-  prevHome = process.env.HOME;
+  prevTestHome = process.env.TEST_USER_HOME;
   prevXdg = process.env.XDG_CONFIG_HOME;
-  process.env.HOME = home;
+  process.env.TEST_USER_HOME = home;
   delete process.env.XDG_CONFIG_HOME;
 });
 
 afterEach(() => {
-  if (prevHome === undefined) delete process.env.HOME;
-  else process.env.HOME = prevHome;
+  if (prevTestHome === undefined) delete process.env.TEST_USER_HOME;
+  else process.env.TEST_USER_HOME = prevTestHome;
   if (prevXdg === undefined) delete process.env.XDG_CONFIG_HOME;
   else process.env.XDG_CONFIG_HOME = prevXdg;
   rmSync(home, { recursive: true, force: true });
@@ -55,12 +55,12 @@ afterEach(() => {
 /** Tests for configure opts. */
 describe("configure opts", () => {
   test("validate rejects multiple modes", () => {
-    const opts = parseConfigureOpts({ sync: "1", status: "1" });
+    const opts = parseConfigureOpts({ refresh: "1", status: "1" });
     expect(validateConfigureOpts(opts)).toContain("only one");
   });
 
-  test("sync requires --yes", () => {
-    const opts = parseConfigureOpts({ sync: "1" });
+  test("refresh requires --yes", () => {
+    const opts = parseConfigureOpts({ refresh: "1" });
     expect(validateConfigureOpts(opts)).toContain("--yes");
   });
 
@@ -92,12 +92,12 @@ describe("configure mutation summary", () => {
     expect(msg).toBe("Installed 1 artifact.");
   });
 
-  test("sync mode uses synced verb and artifact count", () => {
+  test("refresh mode uses refreshed verb and artifact count", () => {
     const msg = formatConfigureMutationSummary(
       { paths: ["a", "b", "c"], installed: 3, removed: 0, configured: 0 },
-      { sync: true },
+      { refresh: true },
     );
-    expect(msg).toBe("Synced 3 artifacts.");
+    expect(msg).toBe("Refreshed 3 artifacts.");
   });
 
   test("remove-all uses removed verb", () => {
@@ -147,8 +147,8 @@ describe("detect installed", () => {
   });
 });
 
-/** Tests for sync plan. */
-describe("sync plan", () => {
+/** Tests for refresh plan. */
+describe("refresh plan", () => {
   test("buildUpdatePlan greenfield includes skill when enabled", () => {
     const paths = resolveInstallPaths(fixture);
     const plan = buildUpdatePlan(fixture, paths, parseInstallOpts({ reinstall: "1", yes: "1" }));
@@ -229,19 +229,92 @@ describe("app config wizard", () => {
   });
 });
 
-describe("configure --sync bootstrap", () => {
+describe("configure --refresh bootstrap", () => {
   test("creates config.json for apps without appConfig", async () => {
     const program: CliProgram = {
-      key: "syncboot",
+      key: "refreshboot",
       version: "0.0.0",
-      description: "Sync bootstrap test.",
+      description: "Refresh bootstrap test.",
       skill: { enabled: true },
       handler: () => {},
       configure: { enabled: true },
     };
     expect(appConfigFileExists(program)).toBe(false);
-    const result = await new Cli(program).invoke(["configure", "--sync", "--yes"]);
+    const result = await new Cli(program).invoke(["configure", "--refresh", "--yes"]);
     expect(result.exitCode).toBe(0);
     expect(appConfigFileExists(program)).toBe(true);
   });
 });
+
+describe("configure lifecycle hooks", () => {
+  test("afterRefresh runs after --refresh plan", async () => {
+    const calls: string[] = [];
+    const program: CliProgram = {
+      ...fixture,
+      key: "hookrefresh",
+      configure: {
+        afterRefresh: async (ctx) => {
+          calls.push(`after:${ctx.paths.mcpName}:dry=${ctx.dry}`);
+        },
+      },
+    };
+    const result = await new Cli(program).invoke(["configure", "--refresh", "--yes"]);
+    expect(result.exitCode).toBe(0);
+    expect(calls).toEqual([`after:hookrefresh:dry=false`]);
+  });
+
+  test("beforeRemoveAll runs before --remove-all plan", async () => {
+    const paths = resolveInstallPaths({ ...fixture, key: "hookremove" });
+    mkdirSync(paths.agentsSkillDir, { recursive: true });
+    writeFileSync(join(paths.agentsSkillDir, "SKILL.md"), "# test\n", "utf8");
+
+    const calls: string[] = [];
+    const program: CliProgram = {
+      ...fixture,
+      key: "hookremove",
+      configure: {
+        beforeRemoveAll: async (ctx) => {
+          calls.push(`before:${ctx.paths.skillDirName}`);
+          expect(existsSync(ctx.paths.agentsSkillDir)).toBe(true);
+        },
+      },
+    };
+    const result = await new Cli(program).invoke(["configure", "--remove-all", "--yes"]);
+    expect(result.exitCode).toBe(0);
+    expect(calls).toEqual(["before:hookremove"]);
+    expect(existsSync(paths.agentsSkillDir)).toBe(false);
+  });
+
+  test("hooks receive dry from --dry", async () => {
+    const calls: string[] = [];
+    const program: CliProgram = {
+      ...fixture,
+      key: "hookdry",
+      configure: {
+        afterRefresh: (ctx) => {
+          calls.push(`dry=${ctx.dry}`);
+        },
+      },
+    };
+    const result = await new Cli(program).invoke(["configure", "--refresh", "--yes", "--dry"]);
+    expect(result.exitCode).toBe(0);
+    expect(calls).toEqual(["dry=true"]);
+  });
+
+  test("beforeRemoveAll is not called for --remove-config", async () => {
+    let called = false;
+    const program: CliProgram = {
+      ...fixture,
+      key: "hookcfg",
+      appConfig: { entries: { token: { description: "Token." } } },
+      configure: {
+        beforeRemoveAll: () => {
+          called = true;
+        },
+      },
+    };
+    const result = await new Cli(program).invoke(["configure", "--remove-config", "--yes"]);
+    expect(result.exitCode).toBe(0);
+    expect(called).toBe(false);
+  });
+});