@ree-auz/nook 0.1.0 → 0.1.1

package/src/commands/ls.mjs

@@ -1,107 +1,107 @@
-import { NookError } from "../errors.mjs";
-import { baseRef, remoteUrl } from "../git.mjs";
-import { githubPrs } from "../forge.mjs";
-import { aheadBehind, changeCount } from "../status.mjs";
-import { useColor } from "../ui.mjs";
-import { branchOf, repoFromCwd, repoWorkspaces } from "../context.mjs";
-import { isInside } from "../home.mjs";
-
-const plural = (n, word) => `${n} ${word}${n === 1 ? "" : "s"}`;
-
-function table(rows) {
-  const widths = rows[0].map((_, i) => Math.max(...rows.map((row) => row[i].length)));
-  return rows.map((row) => row.map((cell, i) => (i === row.length - 1 ? cell : cell.padEnd(widths[i]))).join("  "));
-}
-
-/** Every workspace with its local state and, when gh can tell, its PR. */
-export async function collect(ctx, { repo: repoFlag, prs: lookup = githubPrs } = {}) {
-  const only = repoFlag ?? repoFromCwd(ctx);
-  if (only && !ctx.config.repos[only]) throw new NookError(`Unknown repository "${only}"`);
-  const repos = Object.entries(ctx.config.repos).filter(([name]) => !only || name === only);
-  const fetched = await Promise.all(repos.map(([, repo]) => (remoteUrl(repo.path) ? lookup(repo.path, ctx.env) : { prs: null })));
-  const items = [];
-  const problems = new Set();
-  repos.forEach(([repoName, repo], i) => {
-    const { prs, error } = fetched[i];
-    if (error) problems.add(error);
-    const base = baseRef(repo.path, repo.default_branch);
-    for (const item of repoWorkspaces(repoName, repo)) {
-      const branch = branchOf(item.name, item.ws);
-      const pr = item.main || !branch ? null : (prs?.get(branch) ?? null);
-      const commits = item.main || !branch ? null : aheadBehind(repo.path, branch, base);
-      items.push({ ...item, branch, base, changes: changeCount(item.ws.path), commits, pr });
-    }
-  });
-  return { items, problems: [...problems] };
-}
-
-const portsText = (ws) => (Number.isInteger(ws.port) ? `${ws.port}-${ws.port + 9}` : "-");
-
-function changesText(n) {
-  if (n === null) return "missing";
-  return n ? `${n} changed` : "clean";
-}
-
-function commitsText(c) {
-  if (!c) return "";
-  return [c.ahead && `↑${c.ahead}`, c.behind && `↓${c.behind}`].filter(Boolean).join(" ");
-}
-
-function prText(pr) {
-  if (!pr) return "";
-  return `#${pr.number} ${pr.state}${pr.ci ? ` ${pr.ci}` : ""}`;
-}
-
-/** nk ls [--json] */
-export async function listWorkspaces(ctx, { repo: repoFlag, json = false, prs } = {}) {
-  const { items, problems } = await collect(ctx, { repo: repoFlag, prs });
-  if (json) {
-    const data = items.map(({ name, branch, repoName, ws, base, changes, commits, pr, main, managed }) => ({
-      name,
-      branch,
-      main,
-      managed,
-      locked: Boolean(ws.locked),
-      repo: repoName,
-      path: ws.path,
-      port: ws.port ?? null,
-      base,
-      changes,
-      ahead: commits?.ahead ?? null,
-      behind: commits?.behind ?? null,
-      pr,
-    }));
-    ctx.print(JSON.stringify(data, null, 2));
-    return 0;
-  }
-  if (!items.length) {
-    ctx.info("No repositories yet. Run nk init inside one");
-    return 0;
-  }
-  // Gutter as in worktrunk: ^ main checkout, @ where you are, + another worktree.
-  const mark = (item) => (isInside(ctx.cwd, item.ws.path) && !items.some((o) => o !== item && o.ws.path.length > item.ws.path.length && isInside(ctx.cwd, o.ws.path)) ? "@" : item.main ? "^" : "+");
-  const rows = [["", "NAME", "REPO", "PORTS", "CHANGES", "BASE", "PR", "PATH"]];
-  for (const item of items) {
-    const name = item.ws.locked ? `${item.name} (locked)` : item.name;
-    rows.push([mark(item), name, item.repoName, portsText(item.ws), changesText(item.changes), commitsText(item.commits), prText(item.pr), item.ws.path]);
-  }
-  const lines = table(rows);
-  // Merged rows are dimmed, as worktrunk does for branches that are safe to delete.
-  const dim = useColor(ctx.out, ctx.env);
-  items.forEach((item, i) => {
-    if (dim && item.pr?.state === "merged") lines[i + 1] = `\x1b[2m${lines[i + 1]}\x1b[0m`;
-  });
-  ctx.print(lines.join("\n"));
-
-  const workspaces = items.filter((item) => !item.main);
-  const changed = workspaces.filter((item) => item.changes).length;
-  const merged = workspaces.filter((item) => item.pr?.state === "merged").length;
-  const outside = workspaces.filter((item) => !item.managed).length;
-  const parts = [`Showing ${plural(workspaces.length, "workspace")} and the main checkout`];
-  if (outside) parts.push(`${outside} made by other tools`);
-  if (changed) parts.push(`${changed} with changes`);
-  if (merged) parts.push(`${merged} merged (nk clean removes them)`);
-  ctx.info(parts.join(", "));
-  for (const problem of problems) ctx.warn(`No PR status: ${problem}`);
-  return 0;
-}
+import { NookError } from "../errors.mjs";
+import { baseRef, remoteUrl } from "../git.mjs";
+import { githubPrs } from "../forge.mjs";
+import { aheadBehind, changeCount } from "../status.mjs";
+import { useColor } from "../ui.mjs";
+import { branchOf, repoFromCwd, repoWorkspaces } from "../context.mjs";
+import { isInside } from "../home.mjs";
+
+const plural = (n, word) => `${n} ${word}${n === 1 ? "" : "s"}`;
+
+function table(rows) {
+  const widths = rows[0].map((_, i) => Math.max(...rows.map((row) => row[i].length)));
+  return rows.map((row) => row.map((cell, i) => (i === row.length - 1 ? cell : cell.padEnd(widths[i]))).join("  "));
+}
+
+/** Every workspace with its local state and, when gh can tell, its PR. */
+export async function collect(ctx, { repo: repoFlag, prs: lookup = githubPrs } = {}) {
+  const only = repoFlag ?? repoFromCwd(ctx);
+  if (only && !ctx.config.repos[only]) throw new NookError(`Unknown repository "${only}"`);
+  const repos = Object.entries(ctx.config.repos).filter(([name]) => !only || name === only);
+  const fetched = await Promise.all(repos.map(([, repo]) => (remoteUrl(repo.path) ? lookup(repo.path, ctx.env) : { prs: null })));
+  const items = [];
+  const problems = new Set();
+  repos.forEach(([repoName, repo], i) => {
+    const { prs, error } = fetched[i];
+    if (error) problems.add(error);
+    const base = baseRef(repo.path, repo.default_branch);
+    for (const item of repoWorkspaces(repoName, repo)) {
+      const branch = branchOf(item.name, item.ws);
+      const pr = item.main || !branch ? null : (prs?.get(branch) ?? null);
+      const commits = item.main || !branch ? null : aheadBehind(repo.path, branch, base);
+      items.push({ ...item, branch, base, changes: changeCount(item.ws.path), commits, pr });
+    }
+  });
+  return { items, problems: [...problems] };
+}
+
+const portsText = (ws) => (Number.isInteger(ws.port) ? `${ws.port}-${ws.port + 9}` : "-");
+
+function changesText(n) {
+  if (n === null) return "missing";
+  return n ? `${n} changed` : "clean";
+}
+
+function commitsText(c) {
+  if (!c) return "";
+  return [c.ahead && `↑${c.ahead}`, c.behind && `↓${c.behind}`].filter(Boolean).join(" ");
+}
+
+function prText(pr) {
+  if (!pr) return "";
+  return `#${pr.number} ${pr.state}${pr.ci ? ` ${pr.ci}` : ""}`;
+}
+
+/** nk ls [--json] */
+export async function listWorkspaces(ctx, { repo: repoFlag, json = false, prs } = {}) {
+  const { items, problems } = await collect(ctx, { repo: repoFlag, prs });
+  if (json) {
+    const data = items.map(({ name, branch, repoName, ws, base, changes, commits, pr, main, managed }) => ({
+      name,
+      branch,
+      main,
+      managed,
+      locked: Boolean(ws.locked),
+      repo: repoName,
+      path: ws.path,
+      port: ws.port ?? null,
+      base,
+      changes,
+      ahead: commits?.ahead ?? null,
+      behind: commits?.behind ?? null,
+      pr,
+    }));
+    ctx.print(JSON.stringify(data, null, 2));
+    return 0;
+  }
+  if (!items.length) {
+    ctx.info("No repositories yet. Run nk init inside one");
+    return 0;
+  }
+  // Gutter as in worktrunk: ^ main checkout, @ where you are, + another worktree.
+  const mark = (item) => (isInside(ctx.cwd, item.ws.path) && !items.some((o) => o !== item && o.ws.path.length > item.ws.path.length && isInside(ctx.cwd, o.ws.path)) ? "@" : item.main ? "^" : "+");
+  const rows = [["", "NAME", "REPO", "PORTS", "CHANGES", "BASE", "PR", "PATH"]];
+  for (const item of items) {
+    const name = item.ws.locked ? `${item.name} (locked)` : item.name;
+    rows.push([mark(item), name, item.repoName, portsText(item.ws), changesText(item.changes), commitsText(item.commits), prText(item.pr), item.ws.path]);
+  }
+  const lines = table(rows);
+  // Merged rows are dimmed, as worktrunk does for branches that are safe to delete.
+  const dim = useColor(ctx.out, ctx.env);
+  items.forEach((item, i) => {
+    if (dim && item.pr?.state === "merged") lines[i + 1] = `\x1b[2m${lines[i + 1]}\x1b[0m`;
+  });
+  ctx.print(lines.join("\n"));
+
+  const workspaces = items.filter((item) => !item.main);
+  const changed = workspaces.filter((item) => item.changes).length;
+  const merged = workspaces.filter((item) => item.pr?.state === "merged").length;
+  const outside = workspaces.filter((item) => !item.managed).length;
+  const parts = [`Showing ${plural(workspaces.length, "workspace")} and the main checkout`];
+  if (outside) parts.push(`${outside} made by other tools`);
+  if (changed) parts.push(`${changed} with changes`);
+  if (merged) parts.push(`${merged} merged (nk clean removes them)`);
+  ctx.info(parts.join(", "));
+  for (const problem of problems) ctx.warn(`No PR status: ${problem}`);
+  return 0;
+}

package/src/commands/rm.mjs

@@ -1,190 +1,190 @@
-import { rmSync } from "node:fs";
-import { join } from "node:path";
-import { NookError } from "../errors.mjs";
-import { exists, isInside, samePath } from "../home.mjs";
-import { baseRef, gitTry, listWorktrees, remoteUrl } from "../git.mjs";
-import { githubPrs } from "../forge.mjs";
-import { findHook } from "../hooks.mjs";
-import { backupDir, createBackup, inspectWorkspace, needsBackup } from "../backup.mjs";
-import { describeOccupants, findOccupants, stopProcesses } from "../occupancy.mjs";
-import { createPrompter } from "../prompt.mjs";
-import { branchOf, resolveWorkspace } from "../context.mjs";
-import { toName } from "../names.mjs";
-import { approveHooks, hook } from "./workspace.mjs";
-import { listFiles } from "../shared.mjs";
-import { setupChanges } from "./check-hooks.mjs";
-import { describeProject, dockerCli, findComposeProjects, removeComposeProject } from "../compose.mjs";
-
-function occupants(ctx, path, scan) {
-  const result = scan(path);
-  if (result.unsupported) ctx.warn(`Could not check for running programs: ${result.unsupported}`);
-  return result.processes;
-}
-
-const plural = (n, word) => `${n} ${word}${n === 1 ? "" : "s"}`;
-
-/**
- * Backs up what would be lost, runs the remove hook, deletes the folder and
- * the branch, and forgets the workspace. Programs must already be stopped.
- */
-export async function deleteWorkspace(ctx, target, { state, base, backupRoot, prompter, scan = findOccupants, docker = dockerCli }) {
-  const { repo, ws, name } = target;
-  const branchName = branchOf(name, ws);
-  if (needsBackup(state)) {
-    const dir = createBackup({ repo: repo.path, path: ws.path, branch: branchName, base, state, dir: backupDir(backupRoot, name) });
-    ctx.ok(`Backed up changes to ${dir}`);
-  }
-  if (state.present) {
-    try {
-      await hook(ctx, target, "remove", { prompter });
-    } catch (error) {
-      if (error instanceof NookError) error.message += "\nStopped; nothing was deleted";
-      throw error;
-    }
-  }
-  // What the workspace started with Docker Compose goes too, so no remove hook is needed for it.
-  // Checked again after the hook, which may have removed some of it already.
-  for (const project of findComposeProjects(ws.path, docker).projects) {
-    try {
-      removeComposeProject(project, docker);
-    } catch (error) {
-      throw new NookError(`${error.message}\nStopped; the folder was not deleted`);
-    }
-    ctx.ok(`Removed ${describeProject(project)}`);
-  }
-  if (state.present) {
-    const after = occupants(ctx, ws.path, scan);
-    if (after.length) throw new NookError(`Programs started running in ${ws.path}:\n${describeOccupants(after)}\nNothing was deleted`);
-    // Changes are backed up by now, so the folder goes even when git cannot remove the worktree
-    // (a damaged one, say); prune below then drops git's record of it.
-    const removed = gitTry(["worktree", "remove", "--force", ws.path], { cwd: repo.path });
-    if (!removed.ok) ctx.warn(`git worktree remove failed (${removed.err.split("\n")[0]}); deleting the folder instead`);
-    // git can leave ignored folders behind on Windows (node_modules, .venv), even when it reports success.
-    deleteFolder(ws.path);
-  }
-  gitTry(["worktree", "prune"], { cwd: repo.path });
-  if (listWorktrees(repo.path).some((tree) => samePath(tree.path, ws.path)))
-    throw new NookError(`git still lists ${ws.path} as a worktree. Run git worktree prune in ${repo.path}, then nk rm ${name} again`);
-  if (branchName) {
-    const branch = gitTry(["branch", "-D", branchName], { cwd: repo.path });
-    if (!branch.ok && !/not found/i.test(branch.err)) ctx.warn(`Branch ${branchName} was kept: ${branch.err}`);
-  }
-  delete repo.workspaces[name];
-  ctx.save();
-}
-
-// Folders a project rebuilds from its own files, and files nobody keeps: losing them loses nothing.
-const REBUILT = new Set(["node_modules", ".venv", "venv", "vendor", "dist", "build", "out", "target", "coverage", "__pycache__", ".pytest_cache", ".mypy_cache", ".ruff_cache", ".next", ".nuxt", ".svelte-kit", ".turbo", ".cache", ".parcel-cache", ".gradle", ".tox", "_build", "deps", "tmp", "log", "logs", ".idea", ".vscode", ".DS_Store"]);
-
-/** Ignored files and folders in a workspace that hold more than a rebuild or a shared copy: a local database, say. */
-export function ignoredData(path, sharedDir) {
-  const listed = gitTry(["ls-files", "--others", "--ignored", "--exclude-standard", "--directory", "--no-empty-directory"], { cwd: path });
-  if (!listed.ok) return [];
-  const shared = new Set(listFiles(sharedDir));
-  return listed.out
-    .split(/\r?\n/)
-    .filter((entry) => entry && !shared.has(entry) && !/\.(log|pyc)$/.test(entry) && !entry.split("/").some((part) => REBUILT.has(part)));
-}
-
-/** Deletes a folder and everything in it, retrying while Windows still holds a file open. */
-export function deleteFolder(path) {
-  if (!exists(path)) return;
-  try {
-    rmSync(path, { recursive: true, force: true, maxRetries: 5, retryDelay: 200 });
-  } catch (error) {
-    // A container that mounts the workspace and runs as root leaves files this user cannot delete.
-    if (error.code === "EACCES" || (error.code === "EPERM" && process.platform !== "win32"))
-      throw new NookError(`Could not delete ${path}: ${error.message}\nA container probably wrote these files as another user (often root). Delete the folder with sudo rm -rf, or have the remove hook delete them from inside the container`);
-    throw new NookError(`Could not delete ${path}: ${error.message}\nClose what uses it and run the command again`);
-  }
-  if (exists(path)) throw new NookError(`Could not delete ${path}; close what uses it and run the command again`);
-}
-
-/** What removing one workspace involves, checked before anything changes. */
-async function planRemoval(ctx, input, { repoFlag, scan, prs, docker }) {
-  const name = toName(input);
-  const target = resolveWorkspace(ctx, name, repoFlag);
-  const { repo, ws } = target;
-  if (target.main) throw new NookError(`${ws.path} is the main checkout of ${target.repoName}; git cannot remove it`);
-  if (ws.locked) throw new NookError(`${ws.path} is locked. Unlock it first: git worktree unlock ${ws.path}`);
-  if (exists(ws.path) && isInside(ctx.cwd, ws.path))
-    throw new NookError(`You are inside ${ws.path}. Change to another folder first, e.g. cd ${repo.path}`);
-
-  const base = baseRef(repo.path, repo.default_branch);
-  const branchName = branchOf(target.name, ws);
-  const state = inspectWorkspace({ repo: repo.path, path: ws.path, branch: branchName ?? "", base });
-  const backupRoot = join(ctx.paths.backup, target.repoName);
-  const removeHook = state.present ? findHook(repo.path, "remove") : null;
-
-  ctx.info(`Removing ${target.name} @ ${ws.path}${state.present ? "" : " (folder already missing)"}${!branchName ? "" : branchName === target.name ? " and its branch" : ` and its branch ${branchName}`}`);
-  if (removeHook) ctx.info(`Runs .nook/${removeHook.name} first`);
-  const changed = setupChanges(repo).changes.filter((what) => what.startsWith(".nook/"));
-  if (state.present && changed.length) ctx.warn(`${changed.join(", ")} changed since nk hooks test last passed; remove may leave things behind`);
-  const compose = findComposeProjects(ws.path, docker);
-  for (const project of compose.projects) ctx.info(`Removes ${describeProject(project)}`);
-  for (const name of compose.shared) ctx.warn(`Compose project ${name} also has containers outside this workspace; it is kept`);
-  if (compose.unavailable && compose.unavailable !== "not installed") ctx.warn(`Could not check for Docker containers: ${compose.unavailable}`);
-  if (needsBackup(state)) ctx.info(`Backs up changes to ${backupRoot}`);
-  const lost = state.present ? ignoredData(ws.path, join(ctx.paths.shared, target.repoName)) : [];
-  if (lost.length) ctx.warn(`Not backed up, since git ignores them: ${lost.slice(0, 8).join(", ")}${lost.length > 8 ? ` and ${lost.length - 8} more` : ""}`);
-  if (state.changes.length) {
-    ctx.warn(`${plural(state.changes.length, "uncommitted change")}:`);
-    for (const change of state.changes.slice(0, 10)) ctx.item(change);
-    if (state.changes.length > 10) ctx.item(`... ${state.changes.length - 10} more`);
-  }
-  if (state.unpushed.length) ctx.warn(`${plural(state.unpushed.length, "commit")} not pushed`);
-  if (branchName && !state.merged) {
-    // A squash merge leaves the branch outside the base; the PR says whether it was merged.
-    const pr = remoteUrl(repo.path) ? (await prs(repo.path, ctx.env)).prs?.get(branchName) : null;
-    if (pr?.state === "merged") ctx.info(`PR #${pr.number} is merged`);
-    else ctx.warn(`Branch ${branchName} is not merged into ${base}`);
-  }
-  const running = state.present ? occupants(ctx, ws.path, scan) : [];
-  return { target, state, base, backupRoot, removeHook, running };
-}
-
-/** nk rm <name>... [-n] [-y]: checks every workspace first, asks once, then removes them in turn. */
-export async function removeWorkspace(ctx, { name, names = [name], repo: repoFlag, dryRun = false, yes = false, scan = findOccupants, stop = stopProcesses, prompter = createPrompter(), prs = githubPrs, docker = dockerCli }) {
-  const plans = [];
-  for (const input of [...new Set(names.map(toName))]) plans.push(await planRemoval(ctx, input, { repoFlag, scan, prs, docker }));
-  if (dryRun) {
-    ctx.info("Dry run; nothing was removed");
-    return 0;
-  }
-
-  const running = plans.flatMap((plan) => plan.running);
-  if (running.length) {
-    ctx.warn(`${plural(running.length, "program")} running in ${plans.length === 1 ? "the workspace" : "these workspaces"}:`);
-    for (const item of running) ctx.item(`${item.pid} ${item.name}`);
-  }
-  // Hook approval comes before anything is stopped or deleted.
-  const approved = new Set();
-  for (const { target, removeHook } of plans)
-    if (removeHook && !approved.has(target.repoName)) {
-      approved.add(target.repoName);
-      await approveHooks(ctx, target, prompter);
-    }
-  if (!yes) {
-    const what = plans.map((plan) => plan.target.name).join(", ");
-    const question = running.length ? `Stop ${running.length === 1 ? "it" : "them"} and remove ${what}?` : `Remove ${what}?`;
-    let ok;
-    try {
-      ok = await prompter.confirm(question, false);
-    } catch {
-      throw new NookError("Confirmation needs a terminal; pass -y to remove without asking");
-    }
-    if (!ok) throw new NookError("Nothing was removed");
-  }
-  if (running.length) {
-    await stop(running);
-    const left = plans.flatMap((plan) => (plan.state.present ? occupants(ctx, plan.target.ws.path, scan) : []));
-    if (left.length) throw new NookError(`Could not stop:\n${describeOccupants(left)}\nNothing was removed`);
-    ctx.ok(`Stopped ${plural(running.length, "program")}`);
-  }
-
-  for (const { target, state, base, backupRoot } of plans) {
-    await deleteWorkspace(ctx, target, { state, base, backupRoot, prompter, scan, docker });
-    ctx.ok(`Removed ${target.name}`);
-  }
-  return 0;
-}
+import { rmSync } from "node:fs";
+import { join } from "node:path";
+import { NookError } from "../errors.mjs";
+import { exists, isInside, samePath } from "../home.mjs";
+import { baseRef, gitTry, listWorktrees, remoteUrl } from "../git.mjs";
+import { githubPrs } from "../forge.mjs";
+import { findHook } from "../hooks.mjs";
+import { backupDir, createBackup, inspectWorkspace, needsBackup } from "../backup.mjs";
+import { describeOccupants, findOccupants, stopProcesses } from "../occupancy.mjs";
+import { createPrompter } from "../prompt.mjs";
+import { branchOf, resolveWorkspace } from "../context.mjs";
+import { toName } from "../names.mjs";
+import { approveHooks, hook } from "./workspace.mjs";
+import { listFiles } from "../shared.mjs";
+import { setupChanges } from "./check-hooks.mjs";
+import { describeProject, dockerCli, findComposeProjects, removeComposeProject } from "../compose.mjs";
+
+function occupants(ctx, path, scan) {
+  const result = scan(path);
+  if (result.unsupported) ctx.warn(`Could not check for running programs: ${result.unsupported}`);
+  return result.processes;
+}
+
+const plural = (n, word) => `${n} ${word}${n === 1 ? "" : "s"}`;
+
+/**
+ * Backs up what would be lost, runs the remove hook, deletes the folder and
+ * the branch, and forgets the workspace. Programs must already be stopped.
+ */
+export async function deleteWorkspace(ctx, target, { state, base, backupRoot, prompter, scan = findOccupants, docker = dockerCli }) {
+  const { repo, ws, name } = target;
+  const branchName = branchOf(name, ws);
+  if (needsBackup(state)) {
+    const dir = createBackup({ repo: repo.path, path: ws.path, branch: branchName, base, state, dir: backupDir(backupRoot, name) });
+    ctx.ok(`Backed up changes to ${dir}`);
+  }
+  if (state.present) {
+    try {
+      await hook(ctx, target, "remove", { prompter });
+    } catch (error) {
+      if (error instanceof NookError) error.message += "\nStopped; nothing was deleted";
+      throw error;
+    }
+  }
+  // What the workspace started with Docker Compose goes too, so no remove hook is needed for it.
+  // Checked again after the hook, which may have removed some of it already.
+  for (const project of findComposeProjects(ws.path, docker).projects) {
+    try {
+      removeComposeProject(project, docker);
+    } catch (error) {
+      throw new NookError(`${error.message}\nStopped; the folder was not deleted`);
+    }
+    ctx.ok(`Removed ${describeProject(project)}`);
+  }
+  if (state.present) {
+    const after = occupants(ctx, ws.path, scan);
+    if (after.length) throw new NookError(`Programs started running in ${ws.path}:\n${describeOccupants(after)}\nNothing was deleted`);
+    // Changes are backed up by now, so the folder goes even when git cannot remove the worktree
+    // (a damaged one, say); prune below then drops git's record of it.
+    const removed = gitTry(["worktree", "remove", "--force", ws.path], { cwd: repo.path });
+    if (!removed.ok) ctx.warn(`git worktree remove failed (${removed.err.split("\n")[0]}); deleting the folder instead`);
+    // git can leave ignored folders behind on Windows (node_modules, .venv), even when it reports success.
+    deleteFolder(ws.path);
+  }
+  gitTry(["worktree", "prune"], { cwd: repo.path });
+  if (listWorktrees(repo.path).some((tree) => samePath(tree.path, ws.path)))
+    throw new NookError(`git still lists ${ws.path} as a worktree. Run git worktree prune in ${repo.path}, then nk rm ${name} again`);
+  if (branchName) {
+    const branch = gitTry(["branch", "-D", branchName], { cwd: repo.path });
+    if (!branch.ok && !/not found/i.test(branch.err)) ctx.warn(`Branch ${branchName} was kept: ${branch.err}`);
+  }
+  delete repo.workspaces[name];
+  ctx.save();
+}
+
+// Folders a project rebuilds from its own files, and files nobody keeps: losing them loses nothing.
+const REBUILT = new Set(["node_modules", ".venv", "venv", "vendor", "dist", "build", "out", "target", "coverage", "__pycache__", ".pytest_cache", ".mypy_cache", ".ruff_cache", ".next", ".nuxt", ".svelte-kit", ".turbo", ".cache", ".parcel-cache", ".gradle", ".tox", "_build", "deps", "tmp", "log", "logs", ".idea", ".vscode", ".DS_Store"]);
+
+/** Ignored files and folders in a workspace that hold more than a rebuild or a shared copy: a local database, say. */
+export function ignoredData(path, sharedDir) {
+  const listed = gitTry(["ls-files", "--others", "--ignored", "--exclude-standard", "--directory", "--no-empty-directory"], { cwd: path });
+  if (!listed.ok) return [];
+  const shared = new Set(listFiles(sharedDir));
+  return listed.out
+    .split(/\r?\n/)
+    .filter((entry) => entry && !shared.has(entry) && !/\.(log|pyc)$/.test(entry) && !entry.split("/").some((part) => REBUILT.has(part)));
+}
+
+/** Deletes a folder and everything in it, retrying while Windows still holds a file open. */
+export function deleteFolder(path) {
+  if (!exists(path)) return;
+  try {
+    rmSync(path, { recursive: true, force: true, maxRetries: 5, retryDelay: 200 });
+  } catch (error) {
+    // A container that mounts the workspace and runs as root leaves files this user cannot delete.
+    if (error.code === "EACCES" || (error.code === "EPERM" && process.platform !== "win32"))
+      throw new NookError(`Could not delete ${path}: ${error.message}\nA container probably wrote these files as another user (often root). Delete the folder with sudo rm -rf, or have the remove hook delete them from inside the container`);
+    throw new NookError(`Could not delete ${path}: ${error.message}\nClose what uses it and run the command again`);
+  }
+  if (exists(path)) throw new NookError(`Could not delete ${path}; close what uses it and run the command again`);
+}
+
+/** What removing one workspace involves, checked before anything changes. */
+async function planRemoval(ctx, input, { repoFlag, scan, prs, docker }) {
+  const name = toName(input);
+  const target = resolveWorkspace(ctx, name, repoFlag);
+  const { repo, ws } = target;
+  if (target.main) throw new NookError(`${ws.path} is the main checkout of ${target.repoName}; git cannot remove it`);
+  if (ws.locked) throw new NookError(`${ws.path} is locked. Unlock it first: git worktree unlock ${ws.path}`);
+  if (exists(ws.path) && isInside(ctx.cwd, ws.path))
+    throw new NookError(`You are inside ${ws.path}. Change to another folder first, e.g. cd ${repo.path}`);
+
+  const base = baseRef(repo.path, repo.default_branch);
+  const branchName = branchOf(target.name, ws);
+  const state = inspectWorkspace({ repo: repo.path, path: ws.path, branch: branchName ?? "", base });
+  const backupRoot = join(ctx.paths.backup, target.repoName);
+  const removeHook = state.present ? findHook(repo.path, "remove") : null;
+
+  ctx.info(`Removing ${target.name} @ ${ws.path}${state.present ? "" : " (folder already missing)"}${!branchName ? "" : branchName === target.name ? " and its branch" : ` and its branch ${branchName}`}`);
+  if (removeHook) ctx.info(`Runs .nook/${removeHook.name} first`);
+  const changed = setupChanges(repo).changes.filter((what) => what.startsWith(".nook/"));
+  if (state.present && changed.length) ctx.warn(`${changed.join(", ")} changed since nk hooks test last passed; remove may leave things behind`);
+  const compose = findComposeProjects(ws.path, docker);
+  for (const project of compose.projects) ctx.info(`Removes ${describeProject(project)}`);
+  for (const name of compose.shared) ctx.warn(`Compose project ${name} also has containers outside this workspace; it is kept`);
+  if (compose.unavailable && compose.unavailable !== "not installed") ctx.warn(`Could not check for Docker containers: ${compose.unavailable}`);
+  if (needsBackup(state)) ctx.info(`Backs up changes to ${backupRoot}`);
+  const lost = state.present ? ignoredData(ws.path, join(ctx.paths.shared, target.repoName)) : [];
+  if (lost.length) ctx.warn(`Not backed up, since git ignores them: ${lost.slice(0, 8).join(", ")}${lost.length > 8 ? ` and ${lost.length - 8} more` : ""}`);
+  if (state.changes.length) {
+    ctx.warn(`${plural(state.changes.length, "uncommitted change")}:`);
+    for (const change of state.changes.slice(0, 10)) ctx.item(change);
+    if (state.changes.length > 10) ctx.item(`... ${state.changes.length - 10} more`);
+  }
+  if (state.unpushed.length) ctx.warn(`${plural(state.unpushed.length, "commit")} not pushed`);
+  if (branchName && !state.merged) {
+    // A squash merge leaves the branch outside the base; the PR says whether it was merged.
+    const pr = remoteUrl(repo.path) ? (await prs(repo.path, ctx.env)).prs?.get(branchName) : null;
+    if (pr?.state === "merged") ctx.info(`PR #${pr.number} is merged`);
+    else ctx.warn(`Branch ${branchName} is not merged into ${base}`);
+  }
+  const running = state.present ? occupants(ctx, ws.path, scan) : [];
+  return { target, state, base, backupRoot, removeHook, running };
+}
+
+/** nk rm <name>... [-n] [-y]: checks every workspace first, asks once, then removes them in turn. */
+export async function removeWorkspace(ctx, { name, names = [name], repo: repoFlag, dryRun = false, yes = false, scan = findOccupants, stop = stopProcesses, prompter = createPrompter(), prs = githubPrs, docker = dockerCli }) {
+  const plans = [];
+  for (const input of [...new Set(names.map(toName))]) plans.push(await planRemoval(ctx, input, { repoFlag, scan, prs, docker }));
+  if (dryRun) {
+    ctx.info("Dry run; nothing was removed");
+    return 0;
+  }
+
+  const running = plans.flatMap((plan) => plan.running);
+  if (running.length) {
+    ctx.warn(`${plural(running.length, "program")} running in ${plans.length === 1 ? "the workspace" : "these workspaces"}:`);
+    for (const item of running) ctx.item(`${item.pid} ${item.name}`);
+  }
+  // Hook approval comes before anything is stopped or deleted.
+  const approved = new Set();
+  for (const { target, removeHook } of plans)
+    if (removeHook && !approved.has(target.repoName)) {
+      approved.add(target.repoName);
+      await approveHooks(ctx, target, prompter);
+    }
+  if (!yes) {
+    const what = plans.map((plan) => plan.target.name).join(", ");
+    const question = running.length ? `Stop ${running.length === 1 ? "it" : "them"} and remove ${what}?` : `Remove ${what}?`;
+    let ok;
+    try {
+      ok = await prompter.confirm(question, false);
+    } catch {
+      throw new NookError("Confirmation needs a terminal; pass -y to remove without asking");
+    }
+    if (!ok) throw new NookError("Nothing was removed");
+  }
+  if (running.length) {
+    await stop(running);
+    const left = plans.flatMap((plan) => (plan.state.present ? occupants(ctx, plan.target.ws.path, scan) : []));
+    if (left.length) throw new NookError(`Could not stop:\n${describeOccupants(left)}\nNothing was removed`);
+    ctx.ok(`Stopped ${plural(running.length, "program")}`);
+  }
+
+  for (const { target, state, base, backupRoot } of plans) {
+    await deleteWorkspace(ctx, target, { state, base, backupRoot, prompter, scan, docker });
+    ctx.ok(`Removed ${target.name}`);
+  }
+  return 0;
+}