magus-cli 7.10.3 → 7.10.5

package/src/services/toolchain.ts

@@ -193,7 +193,6 @@ const pinned = (version: string | undefined) => (version ? { version } : {});
 export function managedTool(bin: SelectedBin): ManagedTool | null {
 	const option = bin.chosen;
 	if (!option) return null;
-	const check = bin.check.join(" ");
 	switch (option.via) {
 		case "bun":
 		case "npm":
@@ -202,7 +201,6 @@ export function managedTool(bin: SelectedBin): ManagedTool | null {
 				name: bin.name,
 				via: option.via,
 				package: option.package,
-				check,
 				...pinned(option.version),
 			};
 		case "go":
@@ -210,7 +208,6 @@ export function managedTool(bin: SelectedBin): ManagedTool | null {
 				name: bin.name,
 				via: "go",
 				module: option.package,
-				check,
 				...pinned(option.version),
 			};
 		case "brew":
@@ -218,7 +215,7 @@ export function managedTool(bin: SelectedBin): ManagedTool | null {
 			// version-managed view does not speak: its declaration manages it.
 			return option.cask !== undefined
 				? null
-				: { name: bin.name, via: "brew", formula: option.formula, check };
+				: { name: bin.name, via: "brew", formula: option.formula };
 		case "apt":
 		case "release":
 		case "bootstrap":
@@ -281,6 +278,15 @@ export function managedInstallCommand(t: ManagedTool): Command {
 	return { cmd, args };
 }
 
+/**
+ * The command that releases a brew pin, for a tool brew reports behind but will
+ * not move (`held-back`). It is the one fix the user can run for that row, so
+ * `magus update` shows it rather than running it: unpinning is the user's call.
+ */
+export function brewUnpinCommand(t: ManagedTool): Command {
+	return { cmd: "brew", args: ["unpin", t.formula ?? t.name] };
+}
+
 /**
  * The command that advances an ALREADY-INSTALLED binary to the newest version.
  *

package/src/services/update-engine.ts

@@ -1177,7 +1177,7 @@ function binNoMove(item: BinUpdateItem, after: ToolFacts): string | null {
 		return `the installer exited 0 but ${item.name} is still not on PATH`;
 	}
 	if (after.managed === false) {
-		return `installed, but PATH still runs ${after.resolvedPath ?? "another copy"}`;
+		return "installed, but another copy still runs first";
 	}
 	if (
 		item.action === "upgrade" &&
@@ -1190,19 +1190,6 @@ function binNoMove(item: BinUpdateItem, after: ToolFacts): string | null {
 	return null;
 }
 
-/**
- * The read-back probe, and the ONLY caller that lets the probe run a tool's
- * `check` command.
- *
- * Planning never does: `check` comes from a marketplace `plugin.json`, and
- * `--dry-run` / `--check` must execute nothing a catalogue supplied. Here the
- * installer for this very tool has just run with the user's consent, and the
- * probe's own narrowing means the only argv that can run is the tool asking
- * itself its version (`tool-probe.ts`, `versionFromCheck`). Without it a tool
- * whose installer keeps no readable record reads back as "version unknown",
- * which R6 counts as changed — the safe answer, but never "no change — still
- * 3.7c", the one the user needs.
- */
 /** How long one `magus update` installer may run before it is stopped. */
 const INSTALLER_TIMEOUT_MS = 15 * 60_000;
 
@@ -1210,8 +1197,15 @@ const INSTALLER_TIMEOUT_MS = 15 * 60_000;
 const runInstaller = async (command: Command): Promise<boolean> =>
 	succeeded(await runArgv(command, { timeoutMs: INSTALLER_TIMEOUT_MS }));
 
+/**
+ * The read-back probe: the same probe planning runs, so the version after an
+ * install is read exactly as the plan read it — the installer's own record,
+ * else magus's own version command for the tool (`data/tool-versions.ts`).
+ * Never a plugin's `check` string. A tool with neither reads back as "version
+ * unknown", which R6 counts as changed: the safe answer.
+ */
 const REAL_READ_BACK = (spec: ManagedBin): Promise<ToolFacts> =>
-	probeTool(spec, undefined, { allowCheckCommand: true });
+	probeTool(spec);
 
 export async function applyBins(
 	plan: UpdatePlan,

package/src/services/update-plan.ts

@@ -1034,8 +1034,13 @@ export interface BinUpdateItem {
 	 * the same value the TUI's row runs on Enter.
 	 */
 	upgradeCommand?: Command;
-	/** {@link ToolStatus.shadowed}: a newer copy later on PATH, unused. */
-	shadowed?: string;
+	/**
+	 * {@link ToolFacts.otherCopies}: how many other copies of the tool sit
+	 * later on PATH, which nothing runs. Counted in the CLI header, never
+	 * described: which copy to keep is the user's call, and magus has no
+	 * command to offer for it.
+	 */
+	otherCopies?: number;
 	/** The exact pin, when the profile declares one. */
 	version?: string;
 	/** The version PATH runs, or null when absent / unreadable. */
@@ -1091,7 +1096,7 @@ export function planBinUpdates(
 		};
 		if (status.note) item.note = status.note;
 		if (status.adopted) item.adopted = status.adopted;
-		if (status.shadowed) item.shadowed = status.shadowed;
+		if (f?.otherCopies) item.otherCopies = f.otherCopies;
 		if (f?.upgradeCommand) item.upgradeCommand = f.upgradeCommand;
 		if (bin.version) item.version = bin.version;
 		return item;
@@ -1193,14 +1198,18 @@ export function planHasWork(plan: UpdatePlan): boolean {
  * NEITHER an `unknown` plugin NOR an `unknown` tool counts. "We could not check
  * this" is not evidence of drift, and most unknowns are structural and
  * permanent rather than transient: a plugin whose marketplace publishes no
- * version (278 of the official ones), a tool on PATH but installed by a
- * different npm, `go` absent, a private registry. A gate that exits 1 on those
- * can never pass, so it protects nothing and is switched off. The cost is that
- * an item whose check merely FAILED this run — a rate-limited catalog fetch, a
- * timed-out probe — also passes, so the exit code alone must never be read as
- * "everything verified": the caller prints how many items went unchecked
- * (`summarizePlan`'s `unknownPlugins`/`unknownTools`, rendered by
- * `cli/update-view.ts`), and every unknown row carries its own reason above.
+ * version (278 of the official ones), a tool whose installer timed out, `go`
+ * absent, a private registry. A gate that exits 1 on those can never pass, so
+ * it protects nothing and is switched off. The cost is that an item whose check
+ * merely FAILED this run — a rate-limited catalog fetch, a timed-out probe —
+ * also passes, so the exit code alone must never be read as "everything
+ * verified": the caller prints how many items went unchecked (`summarizePlan`'s
+ * `unknownPlugins`/`unknownTools`, rendered by `cli/update-view.ts`).
+ *
+ * A tool on PATH that another installer put there is not unknown at all: it is
+ * `external`, found and correctly left alone, and is not drift either — nothing
+ * magus could run would change it. A held-back tool ({@link binHeldBack}) is
+ * behind, but pinned by the user, so it does not fail the gate either.
  *
  * A `remove` DOES count. It used to be an `unknown` ("not offered"), and the
  * argument above kept it out while nothing could act on it. `magus update` now
@@ -1226,6 +1235,16 @@ export function planIsBehind(plan: UpdatePlan): boolean {
 	);
 }
 
+/**
+ * A tool brew reports behind but holds back with a pin. It is `unknown` to the
+ * classifier — nothing magus runs can move it — but it is not unchecked, so
+ * every count and label calls it "held back", and its row shows the one fix the
+ * user can run.
+ */
+export function binHeldBack(item: BinUpdateItem): boolean {
+	return item.action === "unknown" && item.reason === "held-back";
+}
+
 /** Counts for the closing summary line, in a stable order. */
 export function summarizePlan(plan: UpdatePlan): {
 	installed: number;
@@ -1236,25 +1255,32 @@ export function summarizePlan(plan: UpdatePlan): {
 	unknown: number;
 	/** Plugins that could not be checked — never a {@link isKeptDeprecated} one. */
 	unknownPlugins: number;
+	/** Tools that could not be checked — never a {@link binHeldBack} one. */
 	unknownTools: number;
 	/**
-	 * Left as they are, on purpose. Plugins no longer published by their
-	 * marketplace and kept (moved, or still required), and tools another
-	 * installer put on PATH (`external`). Determined facts, so not part of
-	 * `unknown`.
+	 * No longer published by their marketplace, and kept on purpose: moved, or
+	 * still required. A determined fact, so not part of `unknown`.
 	 */
 	kept: number;
+	/**
+	 * Tools on PATH that the declared installer does not have: installed another
+	 * way, found, and left alone. A determined fact, so not part of `unknown`.
+	 */
+	external: number;
+	/**
+	 * Tools brew reports behind but will not move, because they are pinned. brew
+	 * DID check them, so not part of `unknown`; the user's fix is `brew unpin`.
+	 */
+	heldBack: number;
 } {
 	const count = (a: PluginUpdateAction) =>
 		plan.plugins.filter((p) => p.action === a).length;
 	const bins = (a: BinUpdateAction) =>
 		plan.bins.filter((b) => b.action === a).length;
-	const keptPlugins = plan.plugins.filter(isKeptDeprecated).length;
-	const unknownPlugins = count("unknown") - keptPlugins;
-	// A tool another installer put on PATH is left as it is: settled, like a
-	// kept plugin, and never "could not be checked".
-	const kept = keptPlugins + bins("external");
-	const unknownTools = bins("unknown");
+	const kept = plan.plugins.filter(isKeptDeprecated).length;
+	const unknownPlugins = count("unknown") - kept;
+	const heldBack = plan.bins.filter(binHeldBack).length;
+	const unknownTools = bins("unknown") - heldBack;
 	return {
 		installed: count("install") + bins("install"),
 		updated: count("update") + bins("upgrade"),
@@ -1264,5 +1290,7 @@ export function summarizePlan(plan: UpdatePlan): {
 		unknownPlugins,
 		unknownTools,
 		kept,
+		external: bins("external"),
+		heldBack,
 	};
 }

package/src/types/index.ts

@@ -468,8 +468,6 @@ export interface ManagedTool {
 	formula?: string;
 	/** Module path for `go install` (defaults to `name`). */
 	module?: string;
-	/** The check, as one display line, e.g. "tmux -V" (defaults to `<name> --version`). */
-	check?: string;
 	/** Optional version pin (real for bun/npm/pip; advisory for brew/go). */
 	version?: string;
 }

package/src/ui/adapters/cliToolsAdapter.ts

@@ -29,7 +29,6 @@ import {
 	type ToolReason,
 	type ToolStatus,
 	classifyTool,
-	installerWord,
 } from "../../services/tool-probe.js";
 import { binCommandForState } from "../../services/toolchain.js";
 import type { Presence } from "../../shared/plugin/hooks/lib/requires-probe.js";
@@ -214,7 +213,8 @@ export interface CliToolEnterAction {
  * pane beside it said "Update to v0.95.0". All three read this now.
  *
  * An `unknown` row still offers the detected method's command: an explicit
- * user action, not a plan, and the detail pane shows why beside it.
+ * user action, not a plan, and the detail pane shows why beside it. An
+ * `external` row offers nothing: it is installed another way.
  */
 export function cliToolEnterAction(row: CliToolRow): CliToolEnterAction | null {
 	const { status, updateCommand, installCommand } = row;
@@ -236,6 +236,14 @@ export function cliToolEnterAction(row: CliToolRow): CliToolEnterAction | null {
 	// Both commands come from the row, which built them from the SAME rule its
 	// state came from — a declared tool's pin included.
 	const command = updateCommand ?? installCommand;
+	// Installed another way: nothing magus could run is the user's fix, and
+	// falling through to `reinstall` would install a second copy through the
+	// installer that does not own it.
+	if (status.state === "external") return null;
+	// Held back by the installer (a brew pin): nothing magus runs can move it,
+	// so `update anyway` would be a no-op. The fix is the user's — the row's
+	// note, like the CLI's `brew unpin <formula>`, says what it is.
+	if (status.reason === "held-back") return null;
 	if (status.state === "upgrade") {
 		return {
 			kind: "update",
@@ -247,9 +255,7 @@ export function cliToolEnterAction(row: CliToolRow): CliToolEnterAction | null {
 			command,
 		};
 	}
-	// Another installer's file, or one magus could not place: still offered,
-	// as an explicit choice, with the note beside it (see the renderer).
-	if (status.state === "unknown" || status.state === "external") {
+	if (status.state === "unknown") {
 		return {
 			kind: "update-anyway",
 			label: "update anyway",
@@ -276,21 +282,19 @@ export interface CliToolStatusText {
 /**
  * The row's short word for WHY an `unknown` could not be checked.
  *
- * A label, never the note. The note is a sentence with a path in it ("on PATH
- * at /Users/…, which npm … does not own — …"); a list row has room for a
- * version, a method tag and a few words, so printing the note there cut it off
- * mid-path on every machine. The full note stays in the detail pane's "Why"
- * line, the same string `magus update` prints.
+ * A label, never the note. The note is a sentence; a list row has room for a
+ * version, a method tag and a few words, so printing the note there cut it
+ * off on every machine. The full note stays in the detail pane's "Why" line.
  *
  * Every label is at most 19 cells, so the row suffix (" ? " + label) stays
- * within 24. Keyed by reason alone.
+ * within 24. Keyed by reason alone. A held-back tool never reaches here: it was
+ * checked, so its row says "held back" without the `?` (see
+ * {@link cliToolStatusText}).
  */
 function unknownLabel(reason: ToolReason): string {
 	switch (reason) {
-		// Only the undetermined case: a file known to be another installer's
-		// is `external`, not `unknown`.
-		case "not-managed":
-			return "owner not known";
+		case "owner-unknown":
+			return "owner unknown";
 		case "version-unreadable":
 			return "version unreadable";
 		case "latest-unavailable":
@@ -305,12 +309,14 @@ function unknownLabel(reason: ToolReason): string {
 			return "pin not installable";
 		case "not-probed":
 			return "not checked";
-		// Never `unknown`: these reasons plan work or report `current`.
+		// Never `unknown`: these reasons plan work, report `current`, or are
+		// `external`.
 		case "absent":
 		case "up-to-date":
 		case "behind":
 		case "at-pin":
 		case "behind-pin":
+		case "installed-elsewhere":
 			return "not checked";
 	}
 }
@@ -324,15 +330,14 @@ export function cliToolStatusText(row: CliToolRow): CliToolStatusText {
 	if (row.checking || !status) return { suffix: " ...", tone: "muted" };
 	switch (status.state) {
 		case "unknown":
+			// Behind and pinned: brew checked it, so no `?` — the same two words
+			// `magus update` uses.
+			if (status.reason === "held-back")
+				return { suffix: " held back", tone: "warning" };
 			return {
 				suffix: ` ? ${unknownLabel(status.reason)}`,
 				tone: "warning",
 			};
-		case "external":
-			return {
-				suffix: ` · outside ${installerWord(row.spec.via)}`,
-				tone: "muted",
-			};
 		case "upgrade":
 			return {
 				suffix: status.target
@@ -340,6 +345,8 @@ export function cliToolStatusText(row: CliToolRow): CliToolStatusText {
 					: " update available",
 				tone: "warning",
 			};
+		case "external":
+			return { suffix: " installed elsewhere", tone: "muted" };
 		case "install":
 		case "current":
 			return { suffix: "", tone: "muted" };
@@ -362,7 +369,9 @@ export function countToolRows(rows: readonly CliToolsRow[]): {
 			continue;
 		}
 		if (row.status?.state === "upgrade") updates++;
-		if (row.status?.state === "unknown") unverified++;
+		// A held-back tool was checked: brew says it is behind and pinned.
+		if (row.status?.state === "unknown" && row.status.reason !== "held-back")
+			unverified++;
 	}
 	return { installed, updates, unverified };
 }

package/src/ui/renderers/cliToolRenderers.tsx

@@ -247,11 +247,16 @@ export function renderCliToolRow(
 	} else if (status?.state === "upgrade") {
 		icon = "*";
 		iconColor = theme.colors.warning;
+	} else if (status?.state === "unknown" && status.reason === "held-back") {
+		// Checked, behind, and pinned: no `?`, the same mark `magus update` uses.
+		icon = "‖";
+		iconColor = theme.colors.warning;
 	} else if (status?.state === "unknown") {
 		icon = "?";
 		iconColor = theme.colors.warning;
 	} else if (status?.state === "external") {
-		// Installed, and left alone on purpose: present, not a finding.
+		// Installed another way: present and fine, but not magus's to move — a
+		// muted dot, never the green of a tool magus keeps up to date.
 		icon = "●";
 		iconColor = theme.colors.muted;
 	} else {
@@ -358,7 +363,12 @@ export function renderCliToolDetail(
 					) : state === "upgrade" ? (
 						<text fg={theme.colors.warning}>{"● Update available"}</text>
 					) : state === "external" ? (
-						<text fg={theme.colors.muted}>{"● Installed, kept as is"}</text>
+						// Never "Up to date": magus did not check it against anything.
+						<text fg={theme.colors.muted} flexShrink={0}>
+							{"● Installed elsewhere"}
+						</text>
+					) : state === "unknown" && status.reason === "held-back" ? (
+						<text fg={theme.colors.warning}>{"‖ Held back"}</text>
 					) : state === "unknown" ? (
 						// A version that was read but cannot be checked for updates is
 						// still a version: say "installed", and let Why say the rest.
@@ -371,17 +381,12 @@ export function renderCliToolDetail(
 						<text fg={theme.colors.success}>{"● Up to date"}</text>
 					)}
 				</box>
-				{(state === "unknown" || state === "external") && status?.note ? (
+				{state === "unknown" && status?.note ? (
 					<box flexDirection="row">
 						<text fg={theme.colors.muted} flexShrink={0}>
 							{"Why      "}
 						</text>
-						<text
-							fg={
-								state === "external" ? theme.colors.muted : theme.colors.warning
-							}
-							minWidth={0}
-						>
+						<text fg={theme.colors.warning} minWidth={0}>
 							{wrapValue(status.note, width)}
 						</text>
 					</box>
@@ -397,17 +402,6 @@ export function renderCliToolDetail(
 						</text>
 					</box>
 				) : null}
-				{/* A newer copy later on PATH, which nothing runs. */}
-				{status?.shadowed ? (
-					<box flexDirection="row">
-						<text fg={theme.colors.muted} flexShrink={0}>
-							{"Warning  "}
-						</text>
-						<text fg={theme.colors.warning} minWidth={0}>
-							{wrapValue(`⚠ ${status.shadowed}`, width)}
-						</text>
-					</box>
-				) : null}
 				{status?.installed ? (
 					<box flexDirection="row">
 						<text fg={theme.colors.muted} flexShrink={0}>
@@ -567,7 +561,7 @@ export function renderCliToolDetail(
 						    running binary; the menu still offers it, as an explicit
 						    user action, and says so beside it. */}
 						{enter.kind === "update-anyway" &&
-						status?.reason === "not-managed" &&
+						status?.reason === "owner-unknown" &&
 						status.note ? (
 							<text fg={theme.colors.warning}>
 								{wrapText(`  ${status.note}`, width).join("\n")}

package/src/ui/state/types.ts

@@ -258,7 +258,6 @@ export interface CliTool {
 	displayName: string;
 	description: string;
 	installCommand: string;
-	checkCommand: string;
 	homebrewFormula?: string;
 	npmPackage?: string;
 	installedVersion?: string;

package/src/__tests__/blind/c9-cli-tool-version.test.ts

@@ -1,239 +0,0 @@
-/**
- * Blind tests — contract C9 (requirement R9.3).
- *
- * Written from requirements.md and architecture.md only. Each test builds the
- * on-disk shape a package manager leaves behind (a bin symlink into a global
- * node_modules package) inside its own temp dir.
- */
-import { afterEach, describe, expect, test } from "bun:test";
-import {
-	chmodSync,
-	mkdirSync,
-	mkdtempSync,
-	realpathSync,
-	rmSync,
-	symlinkSync,
-	writeFileSync,
-} from "node:fs";
-import os from "node:os";
-import path from "node:path";
-
-const load = () => import("../../services/cli-tool-version.js");
-
-const dirs: string[] = [];
-afterEach(() => {
-	for (const dir of dirs.splice(0))
-		rmSync(dir, { recursive: true, force: true });
-});
-
-function temp(): string {
-	// realpath so relative symlinks are computed against the real location.
-	const dir = realpathSync(
-		mkdtempSync(path.join(os.tmpdir(), "magus-blind-c9-")),
-	);
-	dirs.push(dir);
-	return dir;
-}
-
-function writeJson(file: string, value: unknown): void {
-	mkdirSync(path.dirname(file), { recursive: true });
-	writeFileSync(file, JSON.stringify(value, null, 2));
-}
-
-function writeBin(file: string): void {
-	mkdirSync(path.dirname(file), { recursive: true });
-	writeFileSync(file, "#!/usr/bin/env node\nconsole.log('hi');\n");
-	chmodSync(file, 0o755);
-}
-
-function link(target: string, at: string, relative = false): void {
-	mkdirSync(path.dirname(at), { recursive: true });
-	symlinkSync(relative ? path.relative(path.dirname(at), target) : target, at);
-}
-
-describe("C9 — readPackageVersionForBin", () => {
-	test("R9.3 npm global layout: bin/magus -> ../lib/node_modules/magus-cli/bin/magus.js reads magus-cli's version", async () => {
-		const { readPackageVersionForBin } = await load();
-		const prefix = temp();
-		const pkg = path.join(prefix, "lib", "node_modules", "magus-cli");
-		writeJson(path.join(pkg, "package.json"), {
-			name: "magus-cli",
-			version: "7.3.0",
-		});
-		writeBin(path.join(pkg, "bin", "magus.js"));
-		link(
-			path.join(pkg, "bin", "magus.js"),
-			path.join(prefix, "bin", "magus"),
-			true,
-		);
-		expect(readPackageVersionForBin(path.join(prefix, "bin", "magus"))).toBe(
-			"7.3.0",
-		);
-	});
-
-	test("R9.3 bun global layout with a scoped package (Claude Code) reads its version", async () => {
-		const { readPackageVersionForBin } = await load();
-		const root = temp();
-		const pkg = path.join(
-			root,
-			".bun",
-			"install",
-			"global",
-			"node_modules",
-			"@anthropic-ai",
-			"claude-code",
-		);
-		writeJson(path.join(pkg, "package.json"), {
-			name: "@anthropic-ai/claude-code",
-			version: "2.1.300",
-		});
-		writeBin(path.join(pkg, "cli.js"));
-		link(path.join(pkg, "cli.js"), path.join(root, ".bun", "bin", "claude"));
-		expect(
-			readPackageVersionForBin(path.join(root, ".bun", "bin", "claude")),
-		).toBe("2.1.300");
-	});
-
-	test("R9.3 resolves a two-hop symlink chain before walking up", async () => {
-		const { readPackageVersionForBin } = await load();
-		const root = temp();
-		const pkg = path.join(root, "store", "node_modules", "@charmland", "crush");
-		writeJson(path.join(pkg, "package.json"), {
-			name: "@charmland/crush",
-			version: "0.9.1",
-		});
-		writeBin(path.join(pkg, "bin", "crush"));
-		link(path.join(pkg, "bin", "crush"), path.join(root, "hop", "crush"));
-		link(
-			path.join(root, "hop", "crush"),
-			path.join(root, "bin", "crush"),
-			true,
-		);
-		expect(readPackageVersionForBin(path.join(root, "bin", "crush"))).toBe(
-			"0.9.1",
-		);
-	});
-
-	test("R9.3 walks up from the binary's REAL location, not from where the symlink sits", async () => {
-		const { readPackageVersionForBin } = await load();
-		const root = temp();
-		// The symlink lives inside an unrelated package; that package must not answer.
-		writeJson(path.join(root, "host", "package.json"), {
-			name: "host",
-			version: "0.0.1",
-		});
-		const pkg = path.join(
-			root,
-			"global",
-			"node_modules",
-			"@charmland",
-			"crush",
-		);
-		writeJson(path.join(pkg, "package.json"), {
-			name: "@charmland/crush",
-			version: "0.9.1",
-		});
-		writeBin(path.join(pkg, "bin", "crush"));
-		link(
-			path.join(pkg, "bin", "crush"),
-			path.join(root, "host", "bin", "crush"),
-		);
-		expect(
-			readPackageVersionForBin(path.join(root, "host", "bin", "crush")),
-		).toBe("0.9.1");
-	});
-
-	test("R9.3 a binary given by its real path (no symlink) reads its package's version", async () => {
-		const { readPackageVersionForBin } = await load();
-		const root = temp();
-		const pkg = path.join(root, "node_modules", "tool");
-		writeJson(path.join(pkg, "package.json"), {
-			name: "tool",
-			version: "1.2.3",
-		});
-		writeBin(path.join(pkg, "dist", "bin", "tool.js"));
-		expect(
-			readPackageVersionForBin(path.join(pkg, "dist", "bin", "tool.js")),
-		).toBe("1.2.3");
-	});
-
-	test("R9.3 skips package.json files missing name or version and keeps walking up", async () => {
-		const { readPackageVersionForBin } = await load();
-		const root = temp();
-		const pkg = path.join(root, "node_modules", "tool");
-		writeJson(path.join(pkg, "package.json"), {
-			name: "tool",
-			version: "4.5.6",
-		});
-		// Nearer package.json files that do not qualify:
-		writeJson(path.join(pkg, "dist", "package.json"), { type: "module" });
-		writeJson(path.join(pkg, "dist", "esm", "package.json"), {
-			name: "tool-esm",
-		});
-		writeJson(path.join(pkg, "dist", "esm", "bin", "package.json"), {
-			version: "0.0.0",
-		});
-		writeBin(path.join(pkg, "dist", "esm", "bin", "tool.js"));
-		link(
-			path.join(pkg, "dist", "esm", "bin", "tool.js"),
-			path.join(root, "bin", "tool"),
-		);
-		expect(readPackageVersionForBin(path.join(root, "bin", "tool"))).toBe(
-			"4.5.6",
-		);
-	});
-
-	test("R9.3 the nearest qualifying package.json wins over an enclosing project's", async () => {
-		const { readPackageVersionForBin } = await load();
-		const root = temp();
-		writeJson(path.join(root, "package.json"), {
-			name: "workspace",
-			version: "9.9.9",
-		});
-		const pkg = path.join(root, "node_modules", "tool");
-		writeJson(path.join(pkg, "package.json"), {
-			name: "tool",
-			version: "1.0.0",
-		});
-		writeBin(path.join(pkg, "bin", "tool.js"));
-		expect(readPackageVersionForBin(path.join(pkg, "bin", "tool.js"))).toBe(
-			"1.0.0",
-		);
-	});
-
-	test("R9.3 returns null when no package.json with name and version is found", async () => {
-		const { readPackageVersionForBin } = await load();
-		const root = temp();
-		writeBin(path.join(root, "bin", "lonely"));
-		expect(
-			readPackageVersionForBin(path.join(root, "bin", "lonely")),
-		).toBeNull();
-	});
-
-	test("R9.3 returns null (does not throw) for a path that does not exist", async () => {
-		const { readPackageVersionForBin } = await load();
-		const root = temp();
-		expect(
-			readPackageVersionForBin(path.join(root, "bin", "missing")),
-		).toBeNull();
-	});
-
-	test("R9.3 returns null (does not throw) for a dangling symlink", async () => {
-		const { readPackageVersionForBin } = await load();
-		const root = temp();
-		link(path.join(root, "gone", "tool.js"), path.join(root, "bin", "tool"));
-		expect(readPackageVersionForBin(path.join(root, "bin", "tool"))).toBeNull();
-	});
-
-	test("R9.3 returns null (does not throw) when the only package.json is unreadable JSON", async () => {
-		const { readPackageVersionForBin } = await load();
-		const root = temp();
-		const pkg = path.join(root, "node_modules", "broken");
-		mkdirSync(pkg, { recursive: true });
-		writeFileSync(path.join(pkg, "package.json"), "{ not json");
-		writeBin(path.join(pkg, "bin", "broken.js"));
-		expect(
-			readPackageVersionForBin(path.join(pkg, "bin", "broken.js")),
-		).toBeNull();
-	});
-});