argsbarg 5.1.13 → 5.1.14

package/CHANGELOG.md

@@ -7,11 +7,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [5.1.14] - 2026-07-06
+
+
 ## [5.1.13] - 2026-07-06
 
 ### Fixed
 
 - **`configure --remove-all`** — skill uninstall actions no longer reinstall skills after printing "Removing …".
+- **Interactive `configure`** — choosing `n` on an installed skill now removes the skill directory (same bug as `--remove-all`, different code path).
 
 ### Changed
 
@@ -673,7 +677,8 @@ const cli = { ... } satisfies CliProgram;  // or : CliProgram
 - Migrate schemas: rename every `children` property to **`commands`**; move positional definitions to **`CliPositional`** objects on `positionals` and strip `positional` / `argMin` / `argMax` from flag definitions under `options` (flags only carry `name`, `description`, `kind`, and optional `shortName`).
 - Imports: use `CliPositional` where needed; replace `CliOptionDef` with `CliOption` or `CliPositional` as appropriate.
 
-[Unreleased]: https://github.com/bdombro/bun-argsbarg/compare/v5.1.13...HEAD
+[Unreleased]: https://github.com/bdombro/bun-argsbarg/compare/v5.1.14...HEAD
+[5.1.14]: https://github.com/bdombro/bun-argsbarg/releases/tag/v5.1.14
 [5.1.13]: https://github.com/bdombro/bun-argsbarg/releases/tag/v5.1.13
 [5.1.11]: https://github.com/bdombro/bun-argsbarg/releases/tag/v5.1.11
 [5.1.10]: https://github.com/bdombro/bun-argsbarg/releases/tag/v5.1.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "argsbarg",
-  "version": "5.1.13",
+  "version": "5.1.14",
   "type": "module",
   "engines": {
     "bun": ">=1.3"

package/src/configure/configure.test.ts

@@ -13,6 +13,8 @@ import { detectInstalledArtifacts } from "../install/detect-installed.ts";
 import { parseInstallOpts } from "../install/opts.ts";
 import { resolveClaudeDesktopMcpPath, resolveInstallPaths } from "../install/paths.ts";
 import { buildInstallPlan, buildUpdatePlan } from "../install/plan.ts";
+import { installTargetForKey } from "../install/target-registry.ts";
+import { buildDetectedSnapshot, buildTargetPlanContext } from "../install/target-scope.ts";
 import { buildUninstallPlan } from "../install/uninstall.ts";
 import type { CliProgram } from "../types.ts";
 
@@ -153,6 +155,29 @@ describe("remove plan", () => {
     expect(result.exitCode).toBe(0);
     expect(existsSync(paths.cursorSkillDir)).toBe(false);
   });
+
+  test("scoped skill uninstall action removes directory without global uninstall flag", () => {
+    const skillApp: CliProgram = {
+      ...fixture,
+      configure: { agentIntegration: "skill" },
+    };
+    const paths = resolveInstallPaths(skillApp);
+    mkdirSync(paths.cursorSkillDir, { recursive: true });
+    writeFileSync(join(paths.cursorSkillDir, "SKILL.md"), "# test\n", "utf8");
+
+    const detected = buildDetectedSnapshot(skillApp, paths);
+    const ctx = buildTargetPlanContext(skillApp, paths, {}, detected);
+    const target = installTargetForKey("cursorSkill");
+    expect(target).toBeDefined();
+    const actions = target!.planUninstall({
+      ...ctx,
+      mode: "uninstall-scoped",
+      include: (key) => key === "cursorSkill",
+    });
+    expect(actions).toHaveLength(1);
+    expect(actions[0]!.run().length).toBeGreaterThan(0);
+    expect(existsSync(paths.cursorSkillDir)).toBe(false);
+  });
 });
 
 /** Tests for app config wizard. */

package/src/configure/index.ts

@@ -121,7 +121,10 @@ function executePlan(
     if ("kind" in action && action.kind) {
       const skillTarget = skillTargetFromActionKind(action.kind);
       if (skillTarget) {
-        if (opts.uninstall) {
+        const runResult = action.run();
+        if (runResult.length > 0) {
+          changed.push(...runResult);
+        } else if (opts.uninstall) {
           const skillDir = skillDirFromUninstallSummary(action.summary, paths);
           if (skillDir) {
             changed.push(...uninstallSkillDir(skillDir, !!opts.dry));
@@ -228,7 +231,7 @@ async function runInteractiveConfigure(root: CliProgram, opts: ConfigureOpts): P
       runTargetPreflight(root, paths, mutationOpts, installActions);
     }
 
-    changed.push(...executePlan(root, actions, mutationOpts, true));
+    changed.push(...executePlan(root, actions, { ...mutationOpts, uninstall: choice === "uninstall" }, true));
   }
 
   return changed;

package/src/install/target-skill.ts

@@ -12,6 +12,7 @@ import type {
   TargetPlanContext,
   UninstallAction,
 } from "./target-types.ts";
+import { uninstallSkillDir } from "./uninstall.ts";
 
 export interface SkillHostSpec {
   key: CliInstallArtifactKey;
@@ -97,7 +98,7 @@ export class SkillInstallTarget extends InstallTarget {
         kind: this.actionKind,
         summary: `${this.uninstallPrefix}: ${displayInstallPath(dir)}/`,
         message: `Removing ${this.spec.label} skill ${displayInstallPath(dir)}/`,
-        run: () => [],
+        run: () => uninstallSkillDir(dir, ctx.dry),
       },
     ];
   }