argsbarg 5.1.11 → 5.1.13

package/CHANGELOG.md

@@ -7,6 +7,20 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [5.1.13] - 2026-07-06
+
+### Fixed
+
+- **`configure --remove-all`** — skill uninstall actions no longer reinstall skills after printing "Removing …".
+
+### Changed
+
+- **Homebrew formula `uninstall`** — generated formulae run `{key} configure --remove-all --yes` (skills, MCP, and app config).
+
+### Changed
+
+- **`scripts/dev-formula.ts`** — `install` stages a `file://` dev formula; `reset` restores the release formula. `just install-local` orchestrates stage → `brew install` → reset.
+
 ## [5.1.11] - 2026-07-05
 
 
@@ -659,7 +673,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.11...HEAD
+[Unreleased]: https://github.com/bdombro/bun-argsbarg/compare/v5.1.13...HEAD
+[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
 [5.1.9]: https://github.com/bdombro/bun-argsbarg/releases/tag/v5.1.9

package/README.md

@@ -266,7 +266,7 @@ To refresh the Cursor rule in an existing consumer: `bun scripts/merge-cli-progr
 | Command layout | `src/commands/<name>/command.ts`; registration in `src/program.ts` |
 | MCP doc topics | `docs.topics` auto-exposed as `<key>://docs/<topic>` resources when docs + MCP enabled |
 | Package import | `from "argsbarg"` (not relative to argsbarg `src/`) |
-| Homebrew distribution | `scripts/formula-shared.ts`, `scripts/with-dev-formula.ts`, `Formula/`, `justfile` |
+| Homebrew distribution | `scripts/formula-shared.ts`, `scripts/dev-formula.ts`, `Formula/`, `justfile` |
 | Dev tooling | Biome (`just format` / `just lint`), TypeScript, colocated tests |
 | Cursor rules | `.cursor/rules/cli-program.mdc`, `.cursor/rules/code.mdc` |
 

package/docs/configure.md

@@ -16,14 +16,15 @@ brew install <tap>/<key>
 
 Upgrade with `brew upgrade <key>`. Shell completions are installed by Homebrew during `brew install`. Users must configure their shell per [Homebrew Shell Completion](https://docs.brew.sh/Shell-Completion).
 
-**Uninstall the binary:** remove agent artifacts first (while the CLI is still on PATH), then `brew uninstall`:
+**Uninstall the binary:**
 
 ```bash
-<key> configure --remove-all --yes
 brew uninstall <tap>/<key>
 ```
 
-`brew uninstall` runs the formula `uninstall` hook, which removes app config (`configure --remove-config --yes`). Skills and MCP entries are not removed by Homebrew — use `--remove-all` before uninstall when you want those gone too.
+`brew uninstall` runs the formula `uninstall` hook (`configure --remove-all --yes`), which removes detected skills, MCP entries, and app config while the binary is still on PATH.
+
+To remove app config only (keep skills/MCP), run `configure --remove-config --yes` before uninstall.
 
 ## Developer install
 
@@ -177,11 +178,11 @@ Release formulae should run:
 
 ```ruby
 def uninstall
-  system bin/"myapp", "configure", "--remove-config", "--yes"
+  system bin/"myapp", "configure", "--remove-all", "--yes"
 end
 ```
 
-Homebrew calls this before removing the keg, so the binary is still on PATH. Safe no-op when app config was never created or `program.appConfig` is unset.
+Removes detected skills, MCP entries, and app config while the binary is still on PATH. Safe no-op when nothing was installed.
 
 ## Bootstrapping a new CLI
 

package/docs/distribution-homebrew.md

@@ -10,7 +10,7 @@ Argsbarg apps distribute the **binary and shell completions** through Homebrew,
 | Skills + MCP | Formula `post_install` → `{key} configure --sync --yes` |
 | App config file | Bootstrapped as `{}` on `post_install` via `--sync` (`~/.local/lib/<key>/config.json`) |
 | App config values | User opt-in: `{key} configure` (interactive wizard when `program.appConfig` has entries) |
-| App config cleanup | Formula `uninstall` → `{key} configure --remove-config --yes` |
+| App config cleanup | Formula `uninstall` → `{key} configure --remove-all --yes` |
 
 **Only tap-from-repo** — in-repo `Formula/` or GitHub tap. Not Homebrew core.
 
@@ -48,19 +48,27 @@ just reinstall-local   # fast binary swap (`install -m 755` into Cellar; run ins
 just uninstall        # undo formula + agent artifacts (app config removed by formula uninstall)
 ```
 
-Dev and release use the **same formula file** (`Formula/{key}.rb`, class name, install/post_install/test). `just install-local` temporarily overwrites it with a `file://` dev formula (staged under `Formula/.staging/`), runs `brew install`, then **restores** the release formula from backup. Release bumps rewrite the GitHub URL via `scripts/release.ts`.
+Dev and release use the **same formula file** (`Formula/{key}.rb`). `just install-local` runs:
+
+```bash
+bun scripts/dev-formula.ts install    # back up release formula, write file:// dev formula
+brew install --formula <tap>/{key}  # install from the dev formula
+bun scripts/dev-formula.ts reset      # restore release formula
+```
+
+If `brew install` fails, run `bun scripts/dev-formula.ts reset` manually to restore the release formula.
 
 ### Developer uninstall
 
 | Recipe | Removes |
 | --- | --- |
-| `just uninstall` | Formula `{key}` + tap symlink + skills/MCP; app config via formula `uninstall` |
+| `just uninstall` | Formula `{key}` + tap symlink + skills/MCP + app config via formula `uninstall` |
 | `just uninstall-config` | App config file only (`configure --remove-config --yes`) |
-| `just uninstall-release` | Release formula from `{tap}` (keeps tap; app config via formula `uninstall`) |
-| `just uninstall-release-tap` | Release formula + `brew untap {tap}` (app config via formula `uninstall`) |
+| `just uninstall-release` | Release formula from `{tap}` (keeps tap; agent artifacts via formula `uninstall`) |
+| `just uninstall-release-tap` | Release formula + `brew untap {tap}` (agent artifacts via formula `uninstall`) |
 | `just test-release` | Install release formula and run formula test |
 
-End users: `<key> configure --remove-all --yes` then `brew uninstall <tap>/<key>`. `brew uninstall` also removes app config via the formula `uninstall` hook (skills/MCP are not removed automatically).
+End users: `brew uninstall <tap>/<key>`. The formula `uninstall` hook runs `configure --remove-all --yes` (skills, MCP, and app config).
 
 ## Formula pattern
 
@@ -75,7 +83,7 @@ def post_install
 end
 
 def uninstall
-  system bin/"{key}", "configure", "--remove-config", "--yes"
+  system bin/"{key}", "configure", "--remove-all", "--yes"
 end
 ```
 

package/docs/output-schema.md

@@ -43,7 +43,7 @@ Production CLIs with several JSON commands tend to use **codegen** so types, han
 
 ## Recommended pipeline (copy per repo)
 
-No shared npm package — each app copies the same **contract**. Reference implementations: **sqsp-qa-manager-poc**, **idp-trees**, **sqsp-i18n-tools-poc** (see each repo’s `docs/architecture.md` for which commands use which schema root).
+No shared npm package — each app copies the same **contract**. Reference implementations: **sqsp-qa-manager-poc**, **sqsp-workspaces**, **sqsp-i18n-tools-poc** (see each repo’s `docs/architecture.md` for which commands use which schema root).
 
 ```mermaid
 flowchart LR

package/examples/full-example/justfile

@@ -45,12 +45,14 @@ install: install-local
 sync-artifacts:
     just run configure --sync --yes
 
-# Dev install: build, dev formula (restored after brew), symlink tap, brew install
+# Dev install: build, stage dev formula, brew install, restore release formula
 install-local: build
     @brew untap {{tap}} 2>/dev/null || true
     mkdir -p {{tap_parent}}
     ln -sfn '{{justfile_directory()}}' {{tap_path}}
-    bun scripts/with-dev-formula.ts -- brew install --formula {{tap}}/{{cli_key}}
+    bun scripts/dev-formula.ts install
+    brew install --formula {{tap}}/{{cli_key}}
+    bun scripts/dev-formula.ts reset
     @echo ""
     @echo "Next: {{cli_key}} configure"
 

package/examples/full-example/scripts/dev-formula.ts

@@ -0,0 +1,97 @@
+#!/usr/bin/env bun
+/**
+ * Stage or restore the dev Homebrew formula for `just install-local`.
+ * `install` — back up release `Formula/{key}.rb` and write a `file://` dev formula.
+ * `reset` — restore the release formula from backup.
+ */
+
+import { createHash } from "node:crypto";
+import { chmodSync, copyFileSync, existsSync, mkdirSync, readFileSync, unlinkSync, writeFileSync } from "node:fs";
+import { join } from "node:path";
+import { createIdentity } from "./create-identity.ts";
+import { renderDevFormula } from "./formula-shared.ts";
+
+const { key } = createIdentity;
+const root = join(import.meta.dir, "..");
+const stagingDir = join(root, "Formula", ".staging");
+const stagingPath = join(stagingDir, key);
+const formulaPath = join(root, "Formula", `${key}.rb`);
+const backupPath = join(stagingDir, `${key}.rb.bak`);
+const programPath = join(root, "src/program.ts");
+
+type Command = "install" | "reset";
+
+function usage(): never {
+  process.stderr.write("Usage: bun scripts/dev-formula.ts <install|reset>\n");
+  process.exit(1);
+}
+
+function parseCommand(argv: string[]): Command {
+  const cmd = argv[2];
+  if (cmd === "install" || cmd === "reset") {
+    return cmd;
+  }
+  usage();
+}
+
+function backupReleaseFormula(): void {
+  if (!existsSync(formulaPath)) {
+    return;
+  }
+  mkdirSync(stagingDir, { recursive: true });
+  copyFileSync(formulaPath, backupPath);
+}
+
+function restoreReleaseFormula(): void {
+  if (!existsSync(backupPath)) {
+    return;
+  }
+  copyFileSync(backupPath, formulaPath);
+  unlinkSync(backupPath);
+}
+
+function readVersion(): string {
+  const content = readFileSync(programPath, "utf8");
+  const match = /version:\s*"([^"]+)"/.exec(content);
+  if (!match?.[1]) {
+    process.stderr.write(`Could not read version from ${programPath}\n`);
+    process.exit(1);
+  }
+  return match[1];
+}
+
+function stageDevFormula(): void {
+  const distPath = join(root, "dist", key);
+  if (!existsSync(distPath)) {
+    process.stderr.write(`Missing binary: ${distPath} (run just build first)\n`);
+    process.exit(1);
+  }
+  mkdirSync(stagingDir, { recursive: true });
+  copyFileSync(distPath, stagingPath);
+  chmodSync(stagingPath, 0o755);
+
+  const sha256 = createHash("sha256").update(readFileSync(stagingPath)).digest("hex");
+  writeFileSync(formulaPath, renderDevFormula(stagingPath, readVersion(), sha256), "utf8");
+}
+
+function install(): void {
+  backupReleaseFormula();
+  stageDevFormula();
+  console.log(`Wrote dev formula: ${formulaPath}`);
+}
+
+function reset(): void {
+  if (!existsSync(backupPath)) {
+    console.log("No dev formula backup to restore.");
+    return;
+  }
+  restoreReleaseFormula();
+  console.log(`Restored ${formulaPath}`);
+}
+
+const command = parseCommand(process.argv);
+if (command === "install") {
+  install();
+} else {
+  reset();
+}

package/examples/full-example/scripts/formula-shared.ts

@@ -24,7 +24,7 @@ export const formulaPostInstallRuby = `def post_install
   end`;
 
 export const formulaUninstallRuby = `def uninstall
-    system bin/"${key}", "configure", "--remove-config", "--yes"
+    system bin/"${key}", "configure", "--remove-all", "--yes"
   end`;
 
 export const formulaCaveatsRuby = `def caveats
@@ -80,7 +80,10 @@ export function selectStaleReleaseTags(releases: ReleaseTag[]): string[] {
 export const releaseRepoSlug = releaseRepo;
 
 /** Resolves private GitHub release assets via the API at download time. */
-export const githubPrivateReleaseDownloadStrategyRuby = `  class GitHubPrivateReleaseDownloadStrategy < CurlDownloadStrategy
+export const githubPrivateReleaseDownloadStrategyRuby = `  # Private/internal releases: default CurlDownloadStrategy cannot fetch non-public
+  # GitHub release assets. Resolve the asset via the API and authenticate with
+  # GitHub::API.credentials (set up via \`gh auth login\` or HOMEBREW_GITHUB_API_TOKEN).
+  class GitHubPrivateReleaseDownloadStrategy < CurlDownloadStrategy
     def initialize(url, name, version, **meta)
       super
       pattern = %r{https://github\\.com/([^/]+)/([^/]+)/releases/download/([^/]+)/(\\S+)}

package/package.json

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

package/src/builtins/configure-copy.ts

@@ -68,11 +68,10 @@ export function configureCommandNotes(program: CliProgram, _caps: CliCapabilitie
     "See what is installed:",
     `  ${app} configure --status`,
     "",
-    "Remove agent artifacts before brew uninstall:",
-    `  ${app} configure --remove-all --yes`,
+    "Uninstall:",
     `  brew uninstall <tap>/${app}`,
     "",
-    "App config is removed automatically by the formula uninstall hook during brew uninstall.",
+    "The formula uninstall hook runs `configure --remove-all --yes` (skills, MCP, and app config).",
     "",
   ];
   if (program.appConfig) {

package/src/configure/configure.test.ts

@@ -3,7 +3,7 @@ Tests for configure/configure module behavior.
 */
 
 import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
+import { existsSync, mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { appConfigFileExists } from "../config/file.ts";
@@ -137,6 +137,22 @@ describe("remove plan", () => {
     const plan = buildUninstallPlan(fixture, paths, parseInstallOpts({ uninstall: "1", all: "1" }));
     expect(plan.length).toBe(0);
   });
+
+  test("configure --remove-all removes installed cursor skill", async () => {
+    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");
+
+    expect(detectInstalledArtifacts(paths, skillApp).cursorSkill).toBe(true);
+
+    const result = await new Cli(skillApp).invoke(["configure", "--remove-all", "--yes"]);
+    expect(result.exitCode).toBe(0);
+    expect(existsSync(paths.cursorSkillDir)).toBe(false);
+  });
 });
 
 /** Tests for app config wizard. */

package/src/configure/index.ts

@@ -121,7 +121,14 @@ function executePlan(
     if ("kind" in action && action.kind) {
       const skillTarget = skillTargetFromActionKind(action.kind);
       if (skillTarget) {
-        changed.push(...runSkillAction(root, action.kind as InstallActionKind, opts));
+        if (opts.uninstall) {
+          const skillDir = skillDirFromUninstallSummary(action.summary, paths);
+          if (skillDir) {
+            changed.push(...uninstallSkillDir(skillDir, !!opts.dry));
+          }
+        } else {
+          changed.push(...runSkillAction(root, action.kind as InstallActionKind, opts));
+        }
         continue;
       }
     }

package/examples/full-example/scripts/with-dev-formula.ts

@@ -1,79 +0,0 @@
-#!/usr/bin/env bun
-
-/** Back up release formula, write dev formula, run a command, then restore. Usage: bun scripts/with-dev-formula.ts -- brew install … */
-
-import { spawnSync } from "node:child_process";
-import { createHash } from "node:crypto";
-import { chmodSync, copyFileSync, existsSync, mkdirSync, readFileSync, unlinkSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-import { createIdentity } from "./create-identity.ts";
-import { renderDevFormula } from "./formula-shared.ts";
-
-const { key } = createIdentity;
-const root = join(import.meta.dir, "..");
-const stagingDir = join(root, "Formula", ".staging");
-const stagingPath = join(stagingDir, key);
-const formulaPath = join(root, "Formula", `${key}.rb`);
-const backupPath = join(stagingDir, `${key}.rb.bak`);
-
-function backupReleaseFormula(): void {
-  if (!existsSync(formulaPath)) {
-    return;
-  }
-  mkdirSync(stagingDir, { recursive: true });
-  copyFileSync(formulaPath, backupPath);
-}
-
-function restoreReleaseFormula(): void {
-  if (!existsSync(backupPath)) {
-    return;
-  }
-  copyFileSync(backupPath, formulaPath);
-  unlinkSync(backupPath);
-}
-
-function writeDevFormula(): void {
-  const distPath = join(root, "dist", key);
-  mkdirSync(stagingDir, { recursive: true });
-  copyFileSync(distPath, stagingPath);
-  chmodSync(stagingPath, 0o755);
-
-  const sha256 = createHash("sha256").update(readFileSync(stagingPath)).digest("hex");
-  const version = JSON.parse(readFileSync(join(root, "package.json"), "utf8")).version as string;
-  writeFileSync(formulaPath, renderDevFormula(stagingPath, version, sha256), "utf8");
-}
-
-const sep = process.argv.indexOf("--");
-if (sep === -1 || sep === process.argv.length - 1) {
-  process.stderr.write("Usage: bun scripts/with-dev-formula.ts -- <command...>\n");
-  process.exit(1);
-}
-
-const cmd = process.argv.slice(sep + 1);
-const executable = cmd[0];
-if (!executable) {
-  process.stderr.write("Usage: bun scripts/with-dev-formula.ts -- <command...>\n");
-  process.exit(1);
-}
-let restored = false;
-
-function restore(): void {
-  if (restored) {
-    return;
-  }
-  restored = true;
-  restoreReleaseFormula();
-}
-
-try {
-  backupReleaseFormula();
-  writeDevFormula();
-  const result = spawnSync(executable, cmd.slice(1), { stdio: "inherit" });
-  process.exitCode = result.status === null ? 1 : result.status;
-} finally {
-  restore();
-}
-
-if (process.exitCode === 0) {
-  console.log(`Restored ${formulaPath}`);
-}