@williamthorsen/nmr 0.38.0 → 0.39.0

package/.meta/changelog.json

@@ -1,4 +1,88 @@
 [
+  {
+    "version": "0.39.0",
+    "date": "2026-09-22",
+    "sections": [
+      {
+        "title": "🎉 Features",
+        "audience": "all",
+        "items": [
+          {
+            "description": "Report why a workspace selects no package, and no-op nmr's own -R steps (#841)",
+            "body": "- Drops a `-R` step that nmr composed into a script where the workspace holds no package, so `typecheck` and the five `test` scripts run there instead of failing the run.\n- Adds a no-op verdict for a command that the drop leaves with no step at all.\n- Replaces the single refusal for a workspace that selects no package with one message per cause.\n\nMigration: In a repo that declares a workspace package pattern and deliberately holds no package, delete the `.config/nmr.config.ts` entries added to work around the `-R` refusal, the `build: ':'` override included.",
+            "migration": "In a repo that declares a workspace package pattern and deliberately holds no package, delete the `.config/nmr.config.ts` entries added to work around the `-R` refusal, the `build: ':'` override included.",
+            "hash": "0b21f4b29a40aab5c56f10026789e8c0e72161ab"
+          },
+          {
+            "description": "Prefix the booleans and reject unrecognized Vitest option keys (#849)",
+            "body": "- Renames two boolean options of `defineVitestConfig` and `defineRootVitestConfig`, the factories that build a repo's Vitest config: `isolateGit` becomes `shouldIsolateGit`, and `resolveFromSource` becomes `shouldResolveFromSource`.\n- Makes both factories reject an option key outside the recognized set, naming the offending keys and the recognized ones, rather than ignoring a misspelled or retired key and letting its option fall back to the default with nothing in the run reporting it.\n- Adds a retired-key check that runs ahead of the unrecognized-key check, reporting the new name for a config that still sets `isolateGit` or `resolveFromSource`.\n\nMigration: In each options object passed to `defineVitestConfig` or `defineRootVitestConfig`, rename `isolateGit` to `shouldIsolateGit` and `resolveFromSource` to `shouldResolveFromSource`, and correct any other key that the factory now rejects. `defineRootVitestConfig` recognizes `monorepoRoot` on the last layer alone, so a config that sets it on an earlier layer now throws where the old code ignored it.",
+            "breaking": true,
+            "migration": "In each options object passed to `defineVitestConfig` or `defineRootVitestConfig`, rename `isolateGit` to `shouldIsolateGit` and `resolveFromSource` to `shouldResolveFromSource`, and correct any other key that the factory now rejects. `defineRootVitestConfig` recognizes `monorepoRoot` on the last layer alone, so a config that sets it on an earlier layer now throws where the old code ignored it.",
+            "hash": "ed2d1ba9a505c3a6d9fd5455c23bd110b9b8a7c5"
+          },
+          {
+            "description": "End nmr's source value names with the kind of the value (#850)",
+            "body": "- Renames two fields of `TestFileConventionsReport`, `misplaced` and `untiered`, to `misplacedFiles` and `untieredFiles`, and renames `TestFileConventionsOptions`'s `exclude` to `excludedBasenames`, in the `@williamthorsen/nmr/tests` entry point.\n- Adds a guard to `checkTestFileConventions` and `reportTestFileConventions` that throws a `TypeError` naming `excludedBasenames` when a call still sets `exclude`.\n\nMigration: Read `report.misplacedFiles` and `report.untieredFiles` in place of `report.misplaced` and `report.untiered`, and pass `excludedBasenames` in place of `exclude`. A call that still sets `exclude` throws rather than running with no exclusions, so the failure surfaces where Vitest loads the call site.",
+            "breaking": true,
+            "migration": "Read `report.misplacedFiles` and `report.untieredFiles` in place of `report.misplaced` and `report.untiered`, and pass `excludedBasenames` in place of `exclude`. A call that still sets `exclude` throws rather than running with no exclusions, so the failure surfaces where Vitest loads the call site.",
+            "hash": "059c249e2857bed87cefcce02e894ce3a9a65970"
+          },
+          {
+            "description": "Hoist the pnpm-workspace resolver into nmr-core as a total API (#864)",
+            "body": "- Adds a `./workspace` subpath export to `@williamthorsen/nmr-core`, whose `resolveWorkspace` returns a pnpm workspace's package directories, or the cause of an empty result, as a value rather than as a thrown error.\n- Reports a `pnpm-workspace.yaml` that the YAML parser rejects under a cause of its own, `unreadable-manifest`, so `nmr -R build` names the syntax error and its remedy in place of failing with the parser's own error.\n- Removes `diagnoseEmptyWorkspace` and `EmptyWorkspaceDiagnosis` from `@williamthorsen/nmr/workspace`, since `resolveWorkspace` reports the same cause in one call.\n\nMigration: Replace each `diagnoseEmptyWorkspace` call with `resolveWorkspace`, which reports the cause on the empty result it already returns, and rename the `no-manifest` cause to `no-package`. `EmptyWorkspaceCause` also gains `unreadable-manifest`, so an exhaustive switch over it stops compiling until it handles the new member; every other export of `@williamthorsen/nmr/workspace` keeps its signature and its `UserError` throws.",
+            "breaking": true,
+            "migration": "Replace each `diagnoseEmptyWorkspace` call with `resolveWorkspace`, which reports the cause on the empty result it already returns, and rename the `no-manifest` cause to `no-package`. `EmptyWorkspaceCause` also gains `unreadable-manifest`, so an exhaustive switch over it stops compiling until it handles the new member; every other export of `@williamthorsen/nmr/workspace` keeps its signature and its `UserError` throws.",
+            "hash": "3196b84c9d5f037ead0e98ee8547fe71e80fb5c6"
+          },
+          {
+            "description": "Adopt the shared workspace resolver in release-kit, honoring exclusions (#865)",
+            "body": "- 🚨 **Breaking:** Narrows `no-pattern` in `@williamthorsen/nmr-core`'s `EmptyWorkspaceCause`, moving an absent or empty `packages` list onto `no-packages-list` and a value that is not a list of strings onto `unreadable-packages`.\n- Stops `release-kit` from tagging, publishing, and label-syncing a package that a `!` entry in `pnpm-workspace.yaml` excludes, which its previous `glob`-based resolution could not remove from the match.\n- Replaces the release of the root package alone with a failure naming the condition that emptied the workspace, when `pnpm-workspace.yaml` does not hold valid YAML or declares a workspace that does not resolve to a package.\n- Replaces the `no-pattern` remedy that `nmr` printed for a `pnpm-workspace.yaml` declaring no `packages` list, and for one declaring a `packages` value that is not a list of strings.\n- Raises `engines.node` to `>=24.16.0` in `@williamthorsen/nmr-core` and `@williamthorsen/release-kit`, because Node 24.0 through 24.15 ignores the `followSymlinks` option that the workspace matcher passes to `globSync` and drops a symlinked package directory from the resolution without reporting it.\n\nMigration: Add a `no-packages-list` arm and an `unreadable-packages` arm to any `switch` that covers `EmptyWorkspaceCause` exhaustively. `no-pattern` now names only a list that the resolver read in full and whose entries all failed to become a positive pattern, so move the handling of an absent `packages` key, an empty list, and a value that is not a list of strings out of the `no-pattern` arm into the two new ones. Route `no-packages-list` to single-package behavior rather than to a failure path: pnpm resolves such a manifest to the root package alone.",
+            "breaking": true,
+            "migration": "Add a `no-packages-list` arm and an `unreadable-packages` arm to any `switch` that covers `EmptyWorkspaceCause` exhaustively. `no-pattern` now names only a list that the resolver read in full and whose entries all failed to become a positive pattern, so move the handling of an absent `packages` key, an empty list, and a value that is not a list of strings out of the `no-pattern` arm into the two new ones. Route `no-packages-list` to single-package behavior rather than to a failure path: pnpm resolves such a manifest to the root package alone.",
+            "hash": "b1126866159402b8f027a9b82f2a19591a6e9e1b"
+          }
+        ]
+      },
+      {
+        "title": "♻️ Refactoring",
+        "audience": "dev",
+        "items": [
+          {
+            "description": "Align nmr's function names with naming conventions (#852)",
+            "body": "- Renames the functions in `packages/nmr` to align with naming conventions.",
+            "hash": "8a41a206ebe0374c844b20dca0eded7da6240133"
+          },
+          {
+            "description": "Suffix nmr's unit-measuring numbers and expand its abbreviations (#854)",
+            "body": "- Renames every unit-measuring number and abbreviated identifier under `packages/nmr/src`, adding `Ms`, `Bytes`, and `Offset` suffixes to the quantities and expanding `pkg`, `Len`, and `Idx` to `package`, `Length`, and `Index`. The renames are not part of any public API.",
+            "hash": "f8292228095a475b9a187ba64d700cb222f80305"
+          }
+        ]
+      },
+      {
+        "title": "🧪 Tests",
+        "audience": "dev",
+        "items": [
+          {
+            "description": "End nmr's test value names with the kind of the value (#851)",
+            "body": "- Aligns the name of every non-boolean value in `packages/nmr/src/**/__tests__` with naming conventions.\n- Renames `inline` to `inlineBody` in the repo-root `__tests__/reusable-workflow-runtime.app.unit.test.ts`.",
+            "hash": "d12494b2be31f0e73cda1afb7113e7508e9969ee"
+          }
+        ]
+      },
+      {
+        "title": "👷 CI",
+        "audience": "dev",
+        "items": [
+          {
+            "description": "Make the release workflow Corepack-agnostic (#842)",
+            "body": "- Removes the `Disable Corepack` step from `release.reusable.yaml`, which Node 25's removal of Corepack from the distribution would make fail with `command not found` once `actions/setup-node` moves past Node 24, and keeps `COREPACK_ENABLE_STRICT: '0'`, which guards the source-build path where the pnpm `packageManager` pin survives.\n- Adds a case to `reusable-workflow-runtime.app.unit.test.ts` that fails the test run on any reusable workflow whose `run:` commands invoke `corepack`, enumerated from `.github/workflows` so that a workflow added later is covered without editing a list.\n- Drops the Corepack recommendation from the README's pnpm prerequisite, which now points at pnpm's installation page, and from the fix text of the `.tool-versions does not list pnpm` check in nmr's readyup kit, which now names the `packageManager` field.",
+            "hash": "dd46b629c3b4448b701ded3301f6794bdea7af6b"
+          }
+        ]
+      }
+    ]
+  },
   {
     "version": "0.38.0",
     "date": "2026-09-18",

package/.readyup/kits/default.js

@@ -144,26 +144,26 @@ function hasTierInfix(filePath) {
 }
 var TEST_COLLECTION_EXCLUDE = [".git", "coverage", "dist", "node_modules"];
 var TIER_NAMES = ["unit", "tool", "localhost", "remote"];
-function collectTestFiles(dir, relativeDir, inTestDir, context) {
+function collectTestFiles(dir, relativeDir, isInTestDir, context) {
   const entries = readdirSync(dir, { withFileTypes: true });
   for (const entry of entries) {
     const relativePath = relativeDir === "" ? entry.name : `${relativeDir}/${entry.name}`;
     if (entry.isDirectory()) {
       if (context.pruned.has(entry.name)) continue;
-      collectTestFiles(path.join(dir, entry.name), relativePath, inTestDir || entry.name === TEST_DIR, context);
-    } else if (inTestDir !== context.misplaced && TEST_FILE_PATTERN.test(entry.name)) {
-      context.found.push(relativePath);
+      collectTestFiles(path.join(dir, entry.name), relativePath, isInTestDir || entry.name === TEST_DIR, context);
+    } else if (isInTestDir !== context.misplaced && TEST_FILE_PATTERN.test(entry.name)) {
+      context.foundPaths.push(relativePath);
     }
   }
 }
-function walkTestFiles(rootDir, { exclude = [] }, misplaced) {
+function walkTestFiles(rootDir, { excludedBasenames = [] }, misplaced) {
   const context = {
-    found: [],
+    foundPaths: [],
     misplaced,
-    pruned: /* @__PURE__ */ new Set([...TEST_COLLECTION_EXCLUDE, ...exclude])
+    pruned: /* @__PURE__ */ new Set([...TEST_COLLECTION_EXCLUDE, ...excludedBasenames])
   };
   collectTestFiles(rootDir, "", false, context);
-  return context.found.toSorted();
+  return context.foundPaths.toSorted();
 }
 
 // .readyup/kits/default.ts
@@ -210,7 +210,7 @@ var default_default = defineRdyKit({
           severity: "warn",
           quiet: true,
           check: toolVersionsHasNoPnpm,
-          fix: "Remove pnpm from .tool-versions \u2014 manage via packageManager field and corepack"
+          fix: "Remove pnpm from .tool-versions \u2014 manage via the packageManager field"
         },
         {
           name: "no package.json declares a pnpm field",
@@ -316,7 +316,7 @@ var default_default = defineRdyKit({
           name: "the test suite gates the test-file conventions",
           severity: "warn",
           check: () => testSuiteGatesTestFileConventions(),
-          fix: "Add a test of the repo's own under a __tests__ directory that declares the check: import { checkTestFileConventions } from '@williamthorsen/nmr/tests'; checkTestFileConventions(); -- passing `exclude` the directory names that the repo passes to `testCollectionExclude`. Without it no test run reports an untiered or misplaced test file, and this kit reports them in its place against nmr's built-in exclusions alone"
+          fix: "Add a test of the repo's own under a __tests__ directory that declares the check: import { checkTestFileConventions } from '@williamthorsen/nmr/tests'; checkTestFileConventions(); -- passing `excludedBasenames` the directory names that the repo passes to `testCollectionExclude`. Without it no test run reports an untiered or misplaced test file, and this kit reports them in its place against nmr's built-in exclusions alone"
         },
         {
           name: "every test file names its isolation tier",
@@ -330,7 +330,7 @@ var default_default = defineRdyKit({
           severity: "error",
           skip: () => describeConventionsGuardSkip(),
           check: () => everyTestFileSitsUnderTestsDir(),
-          fix: "Move each into a __tests__ directory, the only place from which the shared Vitest config collects. No project collects a file outside one, so it runs nowhere and reports nothing. For a file that is not a test, declare checkTestFileConventions with its directory in `exclude` instead, which this check then defers to"
+          fix: "Move each into a __tests__ directory, the only place from which the shared Vitest config collects. No project collects a file outside one, so it runs nowhere and reports nothing. For a file that is not a test, declare checkTestFileConventions with its directory in `excludedBasenames` instead, which this check then defers to"
         },
         {
           name: "no package re-exports the ancestor Vitest config",
@@ -424,7 +424,7 @@ function allWorkspacePackagesCanBuild() {
   const packagesDir = join(process.cwd(), "packages");
   if (!existsSync(packagesDir)) return true;
   const entries = readdirSync2(packagesDir, { withFileTypes: true });
-  const failing = [];
+  const failingPackages = [];
   for (const entry of entries) {
     if (!entry.isDirectory()) continue;
     const pkgPath = `packages/${entry.name}/package.json`;
@@ -433,30 +433,33 @@ function allWorkspacePackagesCanBuild() {
     const hasBuildOverride = /"build"\s*:/.test(content);
     const hasDefaultBuildInputs = fileExists(`packages/${entry.name}/tsconfig.json`) && existsSync(join(packagesDir, entry.name, "src"));
     if (!hasBuildOverride && !hasDefaultBuildInputs) {
-      failing.push(entry.name);
+      failingPackages.push(entry.name);
     }
   }
-  if (failing.length === 0) return true;
+  if (failingPackages.length === 0) return true;
   return {
     ok: false,
-    detail: `missing build override or tsconfig.json + src/: ${failing.join(", ")}`
+    detail: `missing build override or tsconfig.json + src/: ${failingPackages.join(", ")}`
   };
 }
 function checkNoMatchingFiles(patterns, cwd) {
-  const found = findFiles(patterns, cwd);
-  if (found.length === 0) return true;
-  return { ok: false, detail: formatPaths(found) };
+  const foundFiles = findFiles(patterns, cwd);
+  if (foundFiles.length === 0) return true;
+  return { ok: false, detail: formatPaths(foundFiles) };
 }
 function checkRootVitestConfig(baseName, exportName, cwd) {
   const matches = findFiles([`${baseName}.${CONFIG_EXTENSIONS}`], cwd);
   if (matches.length === 0) {
     return { ok: false, detail: `${baseName}.ts is missing` };
   }
-  const stale = matches.filter(
+  const staleConfigs = matches.filter(
     (relativePath) => !importsSharedExport(readFileIn(cwd, relativePath), exportName, SHARED_VITEST_MODULE)
   );
-  if (stale.length === 0) return true;
-  return { ok: false, detail: `does not import ${exportName} from ${SHARED_VITEST_MODULE}: ${stale.join(", ")}` };
+  if (staleConfigs.length === 0) return true;
+  return {
+    ok: false,
+    detail: `does not import ${exportName} from ${SHARED_VITEST_MODULE}: ${staleConfigs.join(", ")}`
+  };
 }
 function prettierConfigBuildsOnSharedConfig(cwd = process.cwd()) {
   const configs = findFiles(
@@ -467,18 +470,18 @@ function prettierConfigBuildsOnSharedConfig(cwd = process.cwd()) {
   if (configs.length === 0) {
     return { ok: false, detail: describeMissingPrettierConfig(cwd) };
   }
-  const stale = configs.filter(
+  const staleConfigs = configs.filter(
     (relativePath) => !importsSharedExport(readFileIn(cwd, relativePath), "definePrettierConfig", SHARED_PRETTIER_MODULE)
   );
-  if (stale.length === 0) return true;
+  if (staleConfigs.length === 0) return true;
   return {
     ok: false,
-    detail: `does not import definePrettierConfig from ${SHARED_PRETTIER_MODULE}: ${stale.join(", ")}`
+    detail: `does not import definePrettierConfig from ${SHARED_PRETTIER_MODULE}: ${staleConfigs.join(", ")}`
   };
 }
 function describeMissingPrettierConfig(cwd) {
-  const inert = findFiles(INERT_PRETTIER_CONFIGS, cwd);
-  if (inert.length > 0) return `holds no code to call the factory: ${inert.join(", ")}`;
+  const inertConfigs = findFiles(INERT_PRETTIER_CONFIGS, cwd);
+  if (inertConfigs.length > 0) return `holds no code to call the factory: ${inertConfigs.join(", ")}`;
   if (hasPrettierConfigKey(cwd)) {
     return 'holds no code to call the factory: the "prettier" key in package.json';
   }
@@ -488,8 +491,8 @@ function hasPrettierConfigKey(cwd) {
   const manifest = readFileIn(cwd, "package.json");
   if (manifest === void 0) return false;
   try {
-    const parsed = JSON.parse(manifest);
-    return isRecord(parsed) && parsed["prettier"] !== void 0;
+    const parsedManifest = JSON.parse(manifest);
+    return isRecord(parsedManifest) && parsedManifest["prettier"] !== void 0;
   } catch {
     return false;
   }
@@ -502,11 +505,11 @@ function discoverMemberWorkspaces() {
   }
 }
 async function everyBinTargetIsACommittedWrapper() {
-  const discovery = discoverMemberWorkspaces();
-  if (!discovery.ok) return discovery;
-  const tracked = await listTrackedFiles();
-  const trackedPaths = tracked === void 0 ? void 0 : new Set(tracked);
-  const offenders = discovery.workspaces.flatMap(
+  const workspaceDiscovery = discoverMemberWorkspaces();
+  if (!workspaceDiscovery.ok) return workspaceDiscovery;
+  const trackedFiles = await listTrackedFiles();
+  const trackedPaths = trackedFiles === void 0 ? void 0 : new Set(trackedFiles);
+  const offenders = workspaceDiscovery.workspaces.flatMap(
     (workspace) => readBinEntries(workspace).flatMap((entry) => {
       const defect = describeBinTargetDefect(workspace, entry, trackedPaths);
       return defect === void 0 ? [] : [`${describeBinEntry(workspace, entry)} (${defect})`];
@@ -516,16 +519,18 @@ async function everyBinTargetIsACommittedWrapper() {
   return { ok: false, detail: formatPaths(offenders) };
 }
 function everyBinWrapperTargetIsCoveredByFiles() {
-  const discovery = discoverMemberWorkspaces();
-  if (!discovery.ok) return discovery;
+  const workspaceDiscovery = discoverMemberWorkspaces();
+  if (!workspaceDiscovery.ok) return workspaceDiscovery;
   const cwd = process.cwd();
-  const offenders = discovery.workspaces.flatMap((workspace) => {
+  const offenders = workspaceDiscovery.workspaces.flatMap((workspace) => {
     const files = workspace.packageJson["files"];
     if (!Array.isArray(files)) return [];
-    const published = new Set(files.flatMap((entry) => typeof entry === "string" ? [readFirstSegment(entry)] : []));
+    const publishedSegments = new Set(
+      files.flatMap((entry) => typeof entry === "string" ? [readFirstSegment(entry)] : [])
+    );
     return readBinEntries(workspace).flatMap((entry) => {
       const target = readWrapperTarget(cwd, workspace, entry);
-      if (target === void 0 || published.has(readFirstSegment(target))) return [];
+      if (target === void 0 || publishedSegments.has(readFirstSegment(target))) return [];
       return [`${describeBinEntry(workspace, entry)} -> ${target}`];
     });
   });
@@ -566,25 +571,25 @@ function readWrapperTarget(cwd, workspace, entry) {
   return posix.normalize(posix.join(posix.dirname(entry.target), specifier));
 }
 function everyTestFileNamesItsTier(cwd = process.cwd()) {
-  const untiered = findUntieredTestFiles(cwd);
-  if (untiered.length === 0) return true;
-  return { ok: false, detail: formatPaths(untiered) };
+  const untieredFiles = findUntieredTestFiles(cwd);
+  if (untieredFiles.length === 0) return true;
+  return { ok: false, detail: formatPaths(untieredFiles) };
 }
 function everyTestFileSitsUnderTestsDir(cwd = process.cwd()) {
-  const misplaced = findMisplacedTestFiles(cwd);
-  if (misplaced.length === 0) return true;
-  return { ok: false, detail: formatPaths(misplaced) };
+  const misplacedFiles = findMisplacedTestFiles(cwd);
+  if (misplacedFiles.length === 0) return true;
+  return { ok: false, detail: formatPaths(misplacedFiles) };
 }
 function everyViteConfigHasVitestConfig() {
-  const discovery = discoverMemberWorkspaces();
-  if (!discovery.ok) return discovery;
-  const unpaired = discovery.workspaces.flatMap((workspace) => {
+  const workspaceDiscovery = discoverMemberWorkspaces();
+  if (!workspaceDiscovery.ok) return workspaceDiscovery;
+  const unpairedConfigs = workspaceDiscovery.workspaces.flatMap((workspace) => {
     const viteConfigs = findWorkspaceConfigs(workspace, VITE_CONFIG_PATTERN);
     if (viteConfigs.length === 0) return [];
     return findWorkspaceConfigs(workspace, VITEST_CONFIG_PATTERN).length > 0 ? [] : viteConfigs;
   });
-  if (unpaired.length === 0) return true;
-  return { ok: false, detail: formatPaths(unpaired) };
+  if (unpairedConfigs.length === 0) return true;
+  return { ok: false, detail: formatPaths(unpairedConfigs) };
 }
 function findFiles(patterns, cwd) {
   return globSync(patterns, { cwd, exclude: (path2) => SCAN_EXCLUDE_DIRS.has(basename(path2)) }).map((path2) => path2.split(sep).join("/")).toSorted();
@@ -597,11 +602,11 @@ function formatPaths(paths) {
 ${paths.map((path2) => `      ${path2}`).join("\n")}`;
 }
 function getMinVersion() {
-  const picked = { "version": "0.38.0" };
-  if (typeof picked["version"] !== "string") {
+  const pickedFields = { "version": "0.39.0" };
+  if (typeof pickedFields["version"] !== "string") {
     throw new TypeError("nmr/package.json: 'version' must be a string");
   }
-  return picked["version"];
+  return pickedFields["version"];
 }
 function hasSupportedEslintVersion() {
   return hasMinDevDependencyVersion("eslint", MIN_ESLINT_VERSION, {
@@ -630,13 +635,13 @@ function isReExportOnly(content) {
   return statements.length > 0 && statements.every((line) => RE_EXPORT_LINE_PATTERN.test(line));
 }
 function noPnpmFieldInPackageJson(cwd = process.cwd()) {
-  const declaring = findFiles(["**/package.json"], cwd).flatMap((relativePath) => {
+  const declaringFiles = findFiles(["**/package.json"], cwd).flatMap((relativePath) => {
     const keys = readPnpmFieldKeys(readFileIn(cwd, relativePath));
     if (keys === void 0) return [];
     return [keys.length > 0 ? `${relativePath} (${keys.join(", ")})` : relativePath];
   });
-  if (declaring.length === 0) return true;
-  return { ok: false, detail: formatPaths(declaring) };
+  if (declaringFiles.length === 0) return true;
+  return { ok: false, detail: formatPaths(declaringFiles) };
 }
 function noReExportOnlyVitestConfigs(cwd = process.cwd()) {
   const nonRootConfigs = findFiles([`**/${VITEST_CONFIG_PATTERN}`], cwd).filter((path2) => path2.includes("/"));
@@ -650,11 +655,11 @@ function noRedundantRootScripts() {
   const scripts = pkg["scripts"];
   if (!isRecord(scripts)) return true;
   const builtInNames = Object.keys(getDefaultRootScripts());
-  const redundant = Object.keys(scripts).filter((name) => builtInNames.includes(name));
-  if (redundant.length === 0) return true;
+  const redundantNames = Object.keys(scripts).filter((name) => builtInNames.includes(name));
+  if (redundantNames.length === 0) return true;
   return {
     ok: false,
-    detail: `redundant: ${redundant.join(", ")}`
+    detail: `redundant: ${redundantNames.join(", ")}`
   };
 }
 function noRetiredVitestConfigs(cwd = process.cwd()) {
@@ -665,14 +670,14 @@ function noRetiredVitestConfigs(cwd = process.cwd()) {
 }
 function noUnguardedLefthookInstall(cwd = process.cwd()) {
   const scripts = readRootScripts(cwd);
-  const unguarded = INSTALL_LIFECYCLE_SCRIPTS.flatMap((name) => {
+  const unguardedScripts = INSTALL_LIFECYCLE_SCRIPTS.flatMap((name) => {
     const command = scripts[name];
     if (typeof command !== "string") return [];
     const isUnguarded = LEFTHOOK_INSTALL_PATTERN.test(command) && !LEFTHOOK_CHECK_INSTALL_PATTERN.test(command);
     return isUnguarded ? [`${name}: ${command}`] : [];
   });
-  if (unguarded.length === 0) return true;
-  return { ok: false, detail: formatPaths(unguarded) };
+  if (unguardedScripts.length === 0) return true;
+  return { ok: false, detail: formatPaths(unguardedScripts) };
 }
 function noWorkspaceRunScriptReferences() {
   const packagesDir = join(process.cwd(), "packages");
@@ -698,27 +703,27 @@ function readFileIn(cwd, relativePath) {
 }
 function readPnpmFieldKeys(content) {
   if (content === void 0) return void 0;
-  let parsed;
+  let parsedManifest;
   try {
-    parsed = JSON.parse(content);
+    parsedManifest = JSON.parse(content);
   } catch {
     return void 0;
   }
-  if (!isRecord(parsed)) return void 0;
-  const pnpm = parsed["pnpm"];
+  if (!isRecord(parsedManifest)) return void 0;
+  const pnpm = parsedManifest["pnpm"];
   return isRecord(pnpm) ? Object.keys(pnpm).toSorted() : void 0;
 }
 function readRootScripts(cwd) {
   const content = readFileIn(cwd, "package.json");
   if (content === void 0) return {};
-  let parsed;
+  let parsedManifest;
   try {
-    parsed = JSON.parse(content);
+    parsedManifest = JSON.parse(content);
   } catch {
     return {};
   }
-  if (!isRecord(parsed)) return {};
-  const scripts = parsed["scripts"];
+  if (!isRecord(parsedManifest)) return {};
+  const scripts = parsedManifest["scripts"];
   return isRecord(scripts) ? scripts : {};
 }
 function resolvesVersionViaWorkspace(range) {
@@ -729,11 +734,11 @@ function tazeConfigBuildsOnSharedConfig(cwd = process.cwd()) {
   if (configs.length === 0) {
     return { ok: false, detail: describeMissingTazeConfig(cwd) };
   }
-  const stale = configs.filter(
+  const staleConfigs = configs.filter(
     (relativePath) => !importsSharedExport(readFileIn(cwd, relativePath), "defineConfig", SHARED_TAZE_MODULE)
   );
-  if (stale.length === 0) return true;
-  return { ok: false, detail: `does not import defineConfig from ${SHARED_TAZE_MODULE}: ${stale.join(", ")}` };
+  if (staleConfigs.length === 0) return true;
+  return { ok: false, detail: `does not import defineConfig from ${SHARED_TAZE_MODULE}: ${staleConfigs.join(", ")}` };
 }
 function tazeConfigAvoidsClobberedOptions(cwd = process.cwd()) {
   const configs = findFiles([`taze.config.${CONFIG_EXTENSIONS}`], cwd);
@@ -741,15 +746,15 @@ function tazeConfigAvoidsClobberedOptions(cwd = process.cwd()) {
   for (const relativePath of configs) {
     const content = readFileIn(cwd, relativePath);
     if (content === void 0) continue;
-    const discarded = CLOBBERED_TAZE_OPTIONS.filter(({ pattern }) => pattern.test(content)).map(({ key }) => key);
-    if (discarded.length > 0) findings.push(`${relativePath}: ${discarded.join(", ")}`);
+    const discardedKeys = CLOBBERED_TAZE_OPTIONS.filter(({ pattern }) => pattern.test(content)).map(({ key }) => key);
+    if (discardedKeys.length > 0) findings.push(`${relativePath}: ${discardedKeys.join(", ")}`);
   }
   if (findings.length === 0) return true;
   return { ok: false, detail: formatPaths(findings) };
 }
 function describeMissingTazeConfig(cwd) {
-  const inert = findFiles(INERT_TAZE_CONFIGS, cwd);
-  if (inert.length > 0) return `holds no code to call the factory: ${inert.join(", ")}`;
+  const inertConfigs = findFiles(INERT_TAZE_CONFIGS, cwd);
+  if (inertConfigs.length > 0) return `holds no code to call the factory: ${inertConfigs.join(", ")}`;
   return "taze.config.ts is missing";
 }
 function testSuiteGatesTestFileConventions(cwd = process.cwd()) {
@@ -777,9 +782,9 @@ function vitestConfigBuildsOnSharedConfig() {
   const cwd = process.cwd();
   const rootConfigs = findFiles([VITEST_CONFIG_PATTERN], cwd);
   if (rootConfigs.length === 0) return { ok: false, detail: "vitest.config.ts is missing" };
-  const discovery = discoverMemberWorkspaces();
-  if (!discovery.ok) return discovery;
-  const workspaceConfigs = discovery.workspaces.flatMap((workspace) => findWorkspaceConfigs(workspace, VITEST_CONFIG_PATTERN)).filter((relativePath) => !isOwnedByReExportCheck(cwd, relativePath));
+  const workspaceDiscovery = discoverMemberWorkspaces();
+  if (!workspaceDiscovery.ok) return workspaceDiscovery;
+  const workspaceConfigs = workspaceDiscovery.workspaces.flatMap((workspace) => findWorkspaceConfigs(workspace, VITEST_CONFIG_PATTERN)).filter((relativePath) => !isOwnedByReExportCheck(cwd, relativePath));
   const stale = [...rootConfigs, ...workspaceConfigs].filter(
     (relativePath) => !importsSharedExport(readFileIn(cwd, relativePath), "defineVitestConfig", SHARED_VITEST_MODULE)
   );

package/.readyup/manifest.json

@@ -11,12 +11,12 @@
       "esbuildVersion": "0.28.2",
       "inputs": [
         {
-          "hash": "488459c0",
+          "hash": "0ae7745f",
           "kind": "module",
           "path": "kits/default.ts"
         },
         {
-          "hash": "521b947a",
+          "hash": "30230373",
           "kind": "inline",
           "path": "../package.json",
           "paths": [
@@ -34,7 +34,7 @@
           "path": "../src/resolve-scripts.ts"
         },
         {
-          "hash": "2dfde614",
+          "hash": "beb92e94",
           "kind": "module",
           "path": "../src/tiers.ts"
         }
@@ -43,8 +43,8 @@
       "path": "kits/default.js",
       "readyupVersion": "0.38.0",
       "source": "kits/default.ts",
-      "sourceHash": "488459c0",
-      "targetHash": "cbef780a"
+      "sourceHash": "0ae7745f",
+      "targetHash": "06f47835"
     }
   ]
 }

package/CHANGELOG.md

@@ -2,6 +2,76 @@
 
 All notable changes to this project will be documented in this file.
 
+## 0.39.0 — 2026-09-22
+
+### 🎉 Features
+
+- Report why a workspace selects no package, and no-op nmr's own -R steps (#841)
+
+  - Drops a `-R` step that nmr composed into a script where the workspace holds no package, so `typecheck` and the five `test` scripts run there instead of failing the run.
+  - Adds a no-op verdict for a command that the drop leaves with no step at all.
+  - Replaces the single refusal for a workspace that selects no package with one message per cause.
+
+  Migration: In a repo that declares a workspace package pattern and deliberately holds no package, delete the `.config/nmr.config.ts` entries added to work around the `-R` refusal, the `build: ':'` override included.
+
+- 🚨 **Breaking:** Prefix the booleans and reject unrecognized Vitest option keys (#849)
+
+  - Renames two boolean options of `defineVitestConfig` and `defineRootVitestConfig`, the factories that build a repo's Vitest config: `isolateGit` becomes `shouldIsolateGit`, and `resolveFromSource` becomes `shouldResolveFromSource`.
+  - Makes both factories reject an option key outside the recognized set, naming the offending keys and the recognized ones, rather than ignoring a misspelled or retired key and letting its option fall back to the default with nothing in the run reporting it.
+  - Adds a retired-key check that runs ahead of the unrecognized-key check, reporting the new name for a config that still sets `isolateGit` or `resolveFromSource`.
+
+  Migration: In each options object passed to `defineVitestConfig` or `defineRootVitestConfig`, rename `isolateGit` to `shouldIsolateGit` and `resolveFromSource` to `shouldResolveFromSource`, and correct any other key that the factory now rejects. `defineRootVitestConfig` recognizes `monorepoRoot` on the last layer alone, so a config that sets it on an earlier layer now throws where the old code ignored it.
+
+- 🚨 **Breaking:** End nmr's source value names with the kind of the value (#850)
+
+  - Renames two fields of `TestFileConventionsReport`, `misplaced` and `untiered`, to `misplacedFiles` and `untieredFiles`, and renames `TestFileConventionsOptions`'s `exclude` to `excludedBasenames`, in the `@williamthorsen/nmr/tests` entry point.
+  - Adds a guard to `checkTestFileConventions` and `reportTestFileConventions` that throws a `TypeError` naming `excludedBasenames` when a call still sets `exclude`.
+
+  Migration: Read `report.misplacedFiles` and `report.untieredFiles` in place of `report.misplaced` and `report.untiered`, and pass `excludedBasenames` in place of `exclude`. A call that still sets `exclude` throws rather than running with no exclusions, so the failure surfaces where Vitest loads the call site.
+
+- 🚨 **Breaking:** Hoist the pnpm-workspace resolver into nmr-core as a total API (#864)
+
+  - Adds a `./workspace` subpath export to `@williamthorsen/nmr-core`, whose `resolveWorkspace` returns a pnpm workspace's package directories, or the cause of an empty result, as a value rather than as a thrown error.
+  - Reports a `pnpm-workspace.yaml` that the YAML parser rejects under a cause of its own, `unreadable-manifest`, so `nmr -R build` names the syntax error and its remedy in place of failing with the parser's own error.
+  - Removes `diagnoseEmptyWorkspace` and `EmptyWorkspaceDiagnosis` from `@williamthorsen/nmr/workspace`, since `resolveWorkspace` reports the same cause in one call.
+
+  Migration: Replace each `diagnoseEmptyWorkspace` call with `resolveWorkspace`, which reports the cause on the empty result it already returns, and rename the `no-manifest` cause to `no-package`. `EmptyWorkspaceCause` also gains `unreadable-manifest`, so an exhaustive switch over it stops compiling until it handles the new member; every other export of `@williamthorsen/nmr/workspace` keeps its signature and its `UserError` throws.
+
+- 🚨 **Breaking:** Adopt the shared workspace resolver in release-kit, honoring exclusions (#865)
+
+  - 🚨 **Breaking:** Narrows `no-pattern` in `@williamthorsen/nmr-core`'s `EmptyWorkspaceCause`, moving an absent or empty `packages` list onto `no-packages-list` and a value that is not a list of strings onto `unreadable-packages`.
+  - Stops `release-kit` from tagging, publishing, and label-syncing a package that a `!` entry in `pnpm-workspace.yaml` excludes, which its previous `glob`-based resolution could not remove from the match.
+  - Replaces the release of the root package alone with a failure naming the condition that emptied the workspace, when `pnpm-workspace.yaml` does not hold valid YAML or declares a workspace that does not resolve to a package.
+  - Replaces the `no-pattern` remedy that `nmr` printed for a `pnpm-workspace.yaml` declaring no `packages` list, and for one declaring a `packages` value that is not a list of strings.
+  - Raises `engines.node` to `>=24.16.0` in `@williamthorsen/nmr-core` and `@williamthorsen/release-kit`, because Node 24.0 through 24.15 ignores the `followSymlinks` option that the workspace matcher passes to `globSync` and drops a symlinked package directory from the resolution without reporting it.
+
+  Migration: Add a `no-packages-list` arm and an `unreadable-packages` arm to any `switch` that covers `EmptyWorkspaceCause` exhaustively. `no-pattern` now names only a list that the resolver read in full and whose entries all failed to become a positive pattern, so move the handling of an absent `packages` key, an empty list, and a value that is not a list of strings out of the `no-pattern` arm into the two new ones. Route `no-packages-list` to single-package behavior rather than to a failure path: pnpm resolves such a manifest to the root package alone.
+
+### ♻️ Refactoring
+
+- Align nmr's function names with naming conventions (#852)
+
+  - Renames the functions in `packages/nmr` to align with naming conventions.
+
+- Suffix nmr's unit-measuring numbers and expand its abbreviations (#854)
+
+  - Renames every unit-measuring number and abbreviated identifier under `packages/nmr/src`, adding `Ms`, `Bytes`, and `Offset` suffixes to the quantities and expanding `pkg`, `Len`, and `Idx` to `package`, `Length`, and `Index`. The renames are not part of any public API.
+
+### 🧪 Tests
+
+- End nmr's test value names with the kind of the value (#851)
+
+  - Aligns the name of every non-boolean value in `packages/nmr/src/**/__tests__` with naming conventions.
+  - Renames `inline` to `inlineBody` in the repo-root `__tests__/reusable-workflow-runtime.app.unit.test.ts`.
+
+### 👷 CI
+
+- Make the release workflow Corepack-agnostic (#842)
+
+  - Removes the `Disable Corepack` step from `release.reusable.yaml`, which Node 25's removal of Corepack from the distribution would make fail with `command not found` once `actions/setup-node` moves past Node 24, and keeps `COREPACK_ENABLE_STRICT: '0'`, which guards the source-build path where the pnpm `packageManager` pin survives.
+  - Adds a case to `reusable-workflow-runtime.app.unit.test.ts` that fails the test run on any reusable workflow whose `run:` commands invoke `corepack`, enumerated from `.github/workflows` so that a workflow added later is covered without editing a list.
+  - Drops the Corepack recommendation from the README's pnpm prerequisite, which now points at pnpm's installation page, and from the fix text of the `.tool-versions does not list pnpm` check in nmr's readyup kit, which now names the `packageManager` field.
+
 ## 0.38.0 — 2026-09-18
 
 ### 🎉 Features