@williamthorsen/nmr 0.36.0 → 0.38.0

package/.meta/changelog.json

@@ -1,4 +1,117 @@
 [
+  {
+    "version": "0.38.0",
+    "date": "2026-09-18",
+    "sections": [
+      {
+        "title": "🎉 Features",
+        "audience": "all",
+        "items": [
+          {
+            "description": "Give nmr plain output in pipes and CI logs (#838)",
+            "body": "- Stops `nmr` from printing emoji to a pipe or a CI log: When the output stream is not a terminal or `CI` is set, a verdict reads `PASS`, `FAIL`, `SKIP`, or `NOOP` and a command icon is omitted, so that a log can be searched for a verdict.\n- Adds `--output-style <auto|plain|rich>` to `nmr` and the `NMR_OUTPUT_STYLE` variable, each forcing either style; the flag outranks the variable, a value that names no style exits 1 with a usage error, and neither enters a check-cache key: A rich run and a plain run share a recorded pass.\n- Replaces several rich glyphs, taking the status markers from the set that release-kit and v11y-check print so that one status has one glyph across the three tools: a recalled pass shows ⏩ in place of ⏭️, a skipped empty override shows ⚪ in place of ⛔, a warning and the `--log` refusal show 🟠 in place of ⚠️ and 📭, the `--log` header shows 💾 in place of 📼, and `nmr-ensure-prepublish-hooks` reports ✅ and ❌ in place of ✓ and ✗.",
+            "hash": "693b3a7f57def3d299ced240c19bda90f6253bdc"
+          }
+        ]
+      },
+      {
+        "title": "🪦 Removed",
+        "audience": "all",
+        "items": [
+          {
+            "description": "Rename the declinesArgs step declaration to shouldDeclineArguments (#822)",
+            "body": "- Makes a `.config/nmr.config.ts` that still declares `declinesArgs` on a composite script's `{ run, declinesArgs }` step fail to load, with an error that names `shouldDeclineArguments` as the replacement.\n- Renames the matching field of the `StepSpec` type exported by `@williamthorsen/nmr/config` and `@williamthorsen/nmr/scripts`.\n\nMigration: Rename `declinesArgs` to `shouldDeclineArguments` in each composite script step of `.config/nmr.config.ts`, and in any code that sets or reads the field on a `StepSpec`.",
+            "breaking": true,
+            "migration": "Rename `declinesArgs` to `shouldDeclineArguments` in each composite script step of `.config/nmr.config.ts`, and in any code that sets or reads the field on a `StepSpec`.",
+            "hash": "b12333390e731c1a9d7055d293b686028c89ad26"
+          }
+        ]
+      }
+    ]
+  },
+  {
+    "version": "0.37.0",
+    "date": "2026-09-15",
+    "sections": [
+      {
+        "title": "🎉 Features",
+        "audience": "all",
+        "items": [
+          {
+            "description": "Guard lefthook install in root prepare and report unguarded installs (#798)",
+            "body": "- Adds a `warn` check to the readyup kit of `@williamthorsen/nmr` that reports a root script run by `pnpm install`, such as `prepare` or `postinstall`, whose command calls `lefthook install` without `lefthook check-install`.\n- Includes in the check's suggested fix the guarded form `lefthook check-install || lefthook install`, which skips rewriting `.git/hooks` while `lefthook.yml` is unchanged since the last install, so that a repo using that form can run `pnpm install` without write access to `.git/hooks`, as inside the Claude Code sandbox, and under pnpm's default `verifyDepsBeforeRun` avoids a reinstall before every `pnpm exec`.",
+            "hash": "bd624dbe72c843266986f725411e067ca5127275"
+          },
+          {
+            "description": "Report misplaced test files and a missing guard from the readyup kit (#804)",
+            "body": "- Adds an `error` check to the `@williamthorsen/nmr` readyup kit that reports each `*.test.ts` or `*.test.tsx` file outside a `__tests__` directory, which no Vitest project collects and which therefore runs nowhere.\n- Adds a `warn` check that passes once a test file under `__tests__` imports `checkTestFileConventions` from `@williamthorsen/nmr/tests`, and whose fix shows the one-line guard test to add.\n- Makes the kit skip both test-file checks once it finds that guard, because the guard reports the same files and applies the repo's `exclude` list, which the kit cannot read.",
+            "hash": "233e66c24dd77d64c041b7170fefeb608f0f53d6"
+          }
+        ]
+      },
+      {
+        "title": "🐛 Bug fixes",
+        "audience": "all",
+        "items": [
+          {
+            "description": "Fail when a -F pattern selects no workspace (#785)",
+            "body": "- Fixes a run that reported success without doing anything. `nmr -F <pattern> <command>` printed nothing and exited 0 when the pattern matched no package. It now exits 1, names the rejected pattern, and explains the likely cause.\n- Covers three further selections that ran nothing and exited 0: an exclusion that leaves only the workspace root, such as `nmr -F '!./packages/*'`; `-R` in a workspace that declares no package; and an empty `-F ''`.\n\nMigration: Replace any `-F` pattern that names a directory with the package's manifest `name`. A filter that selects nothing now fails the run instead of passing in silence.",
+            "migration": "Replace any `-F` pattern that names a directory with the package's manifest `name`. A filter that selects nothing now fails the run instead of passing in silence.",
+            "hash": "542fe79b7fe8acdcc237841b85e7bea7f466bab2"
+          },
+          {
+            "description": "Fix test runs failing on a dependency with a TypeScript source export (#793)",
+            "body": "- Fixes the issue that, with `resolveFromSource` at its default of `true`, every test in a file failed with `Stripping types is currently unsupported for files under node_modules` when the file's imports reached a dependency whose `exports` map declares a `source` condition pointing at TypeScript.\n- Limits the source resolution enabled by `resolveFromSource` to packages whose real path lies outside every `node_modules`, which covers workspace packages and packages linked into the repo from elsewhere.\n- Makes an import of a workspace package fail when that package's `source` condition names a missing file, instead of resolving to the build output without warning.\n- Makes conditions that a config adds to `resolve.conditions` or `ssr.resolve.conditions` extend Vite's defaults when `resolveFromSource` is `false`, instead of replacing them.\n\nMigration: Point each workspace package's `source` condition at a file that exists, or remove the condition. Build a workspace package before running a test that reaches it through an externalized dependency (one that Vitest loads through Node instead of transforming): Node resolves that import without the `source` condition, so it loads the package's build output.",
+            "migration": "Point each workspace package's `source` condition at a file that exists, or remove the condition. Build a workspace package before running a test that reaches it through an externalized dependency (one that Vitest loads through Node instead of transforming): Node resolves that import without the `source` condition, so it loads the package's build output.",
+            "hash": "cd196ee47a17a891732b583ed706137ab3500507"
+          },
+          {
+            "description": "Prevent a recursive or multi-package run from hanging under pnpm 12 (#814)",
+            "body": "- Fixes a hang under pnpm 12: In a terminal, `nmr -R <command>`, a root command that runs in every package such as `nmr build`, and a `-F` pattern that selects several packages printed nothing and waited until Ctrl+C when a package command read keyboard input or changed the terminal's mode, as the spinner in `next build` does.\n- Gives no terminal input to each package of a `-R` run or a multi-package `-F` run, including a dependency-chain filter such as `-F 'app...'`, and keeps it for a `-F` filter that selects one package, which an interactive command such as `nmr -F <pkg> test:watch` needs.",
+            "hash": "c3a3305875fe2bd086b6f0cbb948186846fffb3d"
+          }
+        ]
+      },
+      {
+        "title": "🧪 Tests",
+        "audience": "dev",
+        "items": [
+          {
+            "description": "Run the descriptor-inheritance test on Linux and skip it in a sandbox (#803)",
+            "body": "- Stops `nmr ci` from failing in every Claude Code sandbox session on macOS by skipping the suite in `inherit-descriptor.tool.test.ts` when `SANDBOX_RUNTIME` is set, because the sandbox does not let `script(1)` open a pseudo-terminal.",
+            "hash": "fbdef48ba26d405a04241ab420c7f7eeafcdab12"
+          }
+        ]
+      },
+      {
+        "title": "⚙️ Tooling",
+        "audience": "dev",
+        "items": [
+          {
+            "description": "Upgrade eslint-config-typescript to v17 and adopt its ignore lists (#797)",
+            "body": "- Declares `vitest` as `catalog:` in every workspace that imports it, with the pnpm catalog pinning 4.1.11, because v17 no longer exempts `vitest` from `n/no-extraneous-import`.\n- Builds the global ignores in `eslint.config.ts` from the published `commonIgnores` and `toolIgnores`, which also exclude `pnpm-lock.yaml` and each `.readyup/manifest.json`, so the `eslint --fix` in `nmr lint` can no longer rewrite a kit manifest and make the next build's `rdy verify` fail.",
+            "hash": "b201669444763ad5bef85730b114098ef352e09b"
+          }
+        ]
+      },
+      {
+        "title": "📚 Documentation",
+        "audience": "dev",
+        "items": [
+          {
+            "description": "Trim the README and move reference detail into shipped topic docs (#800)",
+            "body": "- Deletes the default script registry tables and the CLI flag table, which `nmr --help` prints, and the migration sections, which `CHANGELOG.md` records, rather than moving them.\n- Adds `docs` to the files published with `@williamthorsen/nmr`, and changes the closing pointer in `agents/guidance/rulebooks/nmr.md` to name each topic's document under `node_modules/@williamthorsen/nmr/docs/` in place of `README.md`.",
+            "hash": "4758385382a0c720bbae60eed93b7718b16fbd6b"
+          },
+          {
+            "description": "Trim the release-kit README and move reference detail into shipped docs (#802)",
+            "body": "- Adds `docs` to the published `files` of `@williamthorsen/release-kit` and a `!docs/*.v*.md` exclusion to both it and `@williamthorsen/nmr`, because `pnpm pack` would otherwise include the release-notes previews that a local checkout holds.\n- Adds `--version` to the options that `release-kit --help` lists, and points the hints for the removed `releaseNotes.shouldCreateGithubRelease` field at `release-kit init` instead of the README.",
+            "hash": "bce2e74071cffd44e64e8bf9e6d2e6a2adaaf363"
+          }
+        ]
+      }
+    ]
+  },
   {
     "version": "0.36.0",
     "date": "2026-09-08",

package/.readyup/kits/default.js

@@ -1,18 +1,18 @@
 /** @noformat -- @generated. Do not edit. Compiled by rdy. */
 /* eslint-disable */
-export const __readyupVersion = "0.35.0";
+export const __readyupVersion = "0.38.0";
 
 
 // .readyup/kits/default.ts
 import { existsSync, globSync, readdirSync as readdirSync2 } from "node:fs";
 import { basename, dirname, join, posix, sep } from "node:path";
 
-// ../../node_modules/.pnpm/@williamthorsen+toolbelt.errors@0.7.0/node_modules/@williamthorsen/toolbelt.errors/dist/esm/4-release/isError.js
+// ../../node_modules/.pnpm/@williamthorsen+toolbelt.errors@0.8.1/node_modules/@williamthorsen/toolbelt.errors/dist/esm/4-release/isError.js
 function isError(error) {
   return error instanceof Error || Object.prototype.toString.call(error) === "[object Error]";
 }
 
-// ../../node_modules/.pnpm/@williamthorsen+toolbelt.errors@0.7.0/node_modules/@williamthorsen/toolbelt.errors/dist/esm/4-release/describeError.js
+// ../../node_modules/.pnpm/@williamthorsen+toolbelt.errors@0.8.1/node_modules/@williamthorsen/toolbelt.errors/dist/esm/4-release/describeError.js
 function describeError(error) {
   try {
     if (isError(error) && typeof error.message === "string" && error.message !== "") {
@@ -41,8 +41,8 @@ import {
 
 // src/default-scripts.ts
 var GATE_PROJECTS = "--project unit --project tool";
-var TYPECHECK_STEP = { run: "typecheck", declinesArgs: true };
-var ROOT_TYPECHECK_STEP = { run: "root:typecheck", declinesArgs: true };
+var TYPECHECK_STEP = { run: "typecheck", shouldDeclineArguments: true };
+var ROOT_TYPECHECK_STEP = { run: "root:typecheck", shouldDeclineArguments: true };
 var workspaceScripts = {
   build: ["compile"],
   check: [TYPECHECK_STEP, "fmt:check", "lint:check", "test"],
@@ -77,7 +77,7 @@ var rootScripts = {
   "check:strict": [TYPECHECK_STEP, "fmt:check", "lint:strict", "test"],
   // Excludes the audit, which in CI has a workflow of its own. The build is what the narrowed check runs
   // against, so it declines the arguments rather than being narrowed by them.
-  ci: [{ run: "build", declinesArgs: true }, "check:strict"],
+  ci: [{ run: "build", shouldDeclineArguments: true }, "check:strict"],
   clean: "nmr-clean",
   fix: ["lint", "fmt"],
   "fix:check": ["fmt:check", "lint:check"],
@@ -88,7 +88,7 @@ var rootScripts = {
   "lint:strict": "strict-lint",
   // The audit costs seconds and `ci` costs minutes, so the cheap gate fails first. The audit reads the
   // dependency tree, which no argument narrowing the code under test says anything about.
-  prepush: [{ run: "audit", declinesArgs: true }, "ci"],
+  prepush: [{ run: "audit", shouldDeclineArguments: true }, "ci"],
   "report-overrides": "nmr-report-overrides",
   "root:check": [ROOT_TYPECHECK_STEP, "fmt:check", "root:lint:check", "root:test"],
   "root:lint": "eslint --fix --ignore-pattern 'packages/**' .",
@@ -110,7 +110,7 @@ var rootScripts = {
   "test:watch": `vitest ${GATE_PROJECTS} --watch`,
   // Neither step is narrowable, so `nmr typecheck <file>` is rejected rather than checking that file under
   // default options at the root and hunting for it in every package.
-  typecheck: [ROOT_TYPECHECK_STEP, { run: "-R typecheck", declinesArgs: true }],
+  typecheck: [ROOT_TYPECHECK_STEP, { run: "-R typecheck", shouldDeclineArguments: true }],
   // The command is a string because neither half names an nmr command: both are binaries, and a composite
   // element can name only a command.
   upgrade: "nmr-report-overrides && nmr-taze --recursive"
@@ -129,9 +129,15 @@ var TEST_EXTENSIONS = "{ts,tsx}";
 var TEST_GLOB_PREFIX = `**/${TEST_DIR}/**`;
 var TEST_FILE_PATTERN = /\.test\.tsx?$/;
 var ALL_TEST_PATTERNS = [`${TEST_GLOB_PREFIX}/*.test.${TEST_EXTENSIONS}`];
+function findMisplacedTestFiles(rootDir, options = {}) {
+  return walkTestFiles(rootDir, options, true);
+}
 function findTestFiles(rootDir, options = {}) {
   return walkTestFiles(rootDir, options, false);
 }
+function findUntieredTestFiles(rootDir, options = {}) {
+  return findTestFiles(rootDir, options).filter((file) => !hasTierInfix(file));
+}
 function hasTierInfix(filePath) {
   const tiers = TIER_NAMES;
   return tiers.includes(path.basename(filePath).split(".").at(-3) ?? "");
@@ -252,6 +258,14 @@ var default_default = defineRdyKit({
           check: noRedundantRootScripts,
           fix: "Remove scripts from root package.json that nmr provides as built-in root scripts \u2014 invoke via nmr directly"
         },
+        // -- Git hooks -----------------------------------------------------------
+        {
+          name: "no root install script runs lefthook install unguarded",
+          severity: "warn",
+          quiet: true,
+          check: () => noUnguardedLefthookInstall(),
+          fix: "Guard each listed script as `lefthook check-install || lefthook install`, which writes hooks only when they are missing or stale. A bare `lefthook install` rewrites `.git/hooks` on every install, so when that directory is not writable, as in an agent's sandbox, `pnpm install` fails and pnpm's `verifyDepsBeforeRun` then reinstalls before every `pnpm exec`"
+        },
         // -- Workspace build readiness -------------------------------------------
         {
           name: "all workspace packages can build",
@@ -298,12 +312,26 @@ var default_default = defineRdyKit({
           check: () => everyViteConfigHasVitestConfig(),
           fix: "Add a vitest.config.ts calling defineVitestConfig() from @williamthorsen/nmr/vitest beside each listed vite.config -- Vitest stops its config search at the first directory holding either name, so the Vite config otherwise wins and the projects model is never reached"
         },
+        {
+          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"
+        },
         {
           name: "every test file names its isolation tier",
           severity: "error",
+          skip: () => describeConventionsGuardSkip(),
           check: () => everyTestFileNamesItsTier(),
           fix: `Rename each to <subject>[.<aspect>].<tier>.test.ts, naming one of ${TIER_NAMES.join(", ")}. Use tool for a test that reaches a program the environment supplies, which is where a retired .int. or .integration. file belongs. Only the segment before .test. selects a project, so an untiered file runs under the residual unit project and reports success`
         },
+        {
+          name: "every test file sits under a __tests__ directory",
+          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"
+        },
         {
           name: "no package re-exports the ancestor Vitest config",
           severity: "recommend",
@@ -368,6 +396,7 @@ var SHARED_VITEST_MODULE = "@williamthorsen/nmr/vitest";
 var SHARED_PRETTIER_MODULE = "@williamthorsen/nmr/prettier";
 var INERT_PRETTIER_CONFIGS = [".prettierrc", ".prettierrc.{json,json5,yaml,yml,toml}"];
 var SHARED_TAZE_MODULE = "@williamthorsen/nmr/taze";
+var SHARED_TESTS_MODULE = "@williamthorsen/nmr/tests";
 var INERT_TAZE_CONFIGS = [".tazerc", ".tazerc.json", "taze.config.json"];
 var CLOBBERED_TAZE_OPTIONS = [
   { key: "concurrency", pattern: /\bconcurrency\s*:/ },
@@ -377,6 +406,17 @@ var CLOBBERED_TAZE_OPTIONS = [
   { key: "requestTimeout", pattern: /\brequestTimeout\s*:/ }
 ];
 var RE_EXPORT_LINE_PATTERN = /^export\s*(?:\{\s*default\s*}|\*)\s*from\s*['"]\.\.\/[^'"]*['"];?$/;
+var INSTALL_LIFECYCLE_SCRIPTS = [
+  "pnpm:devPreinstall",
+  "preinstall",
+  "install",
+  "postinstall",
+  "preprepare",
+  "prepare",
+  "postprepare"
+];
+var LEFTHOOK_CHECK_INSTALL_PATTERN = /\blefthook\s+check-install\b/;
+var LEFTHOOK_INSTALL_PATTERN = /\blefthook\s+install\b/;
 var MIN_ESLINT_VERSION = "10.0.0";
 var MIN_STRICT_LINT_VERSION = "9.3.0";
 var WORKSPACE_VERSION_MARKERS = ["catalog:", "workspace:"];
@@ -526,10 +566,15 @@ function readWrapperTarget(cwd, workspace, entry) {
   return posix.normalize(posix.join(posix.dirname(entry.target), specifier));
 }
 function everyTestFileNamesItsTier(cwd = process.cwd()) {
-  const untiered = findTestFiles(cwd).filter((path2) => !hasTierInfix(path2));
+  const untiered = findUntieredTestFiles(cwd);
   if (untiered.length === 0) return true;
   return { ok: false, detail: formatPaths(untiered) };
 }
+function everyTestFileSitsUnderTestsDir(cwd = process.cwd()) {
+  const misplaced = findMisplacedTestFiles(cwd);
+  if (misplaced.length === 0) return true;
+  return { ok: false, detail: formatPaths(misplaced) };
+}
 function everyViteConfigHasVitestConfig() {
   const discovery = discoverMemberWorkspaces();
   if (!discovery.ok) return discovery;
@@ -552,7 +597,7 @@ function formatPaths(paths) {
 ${paths.map((path2) => `      ${path2}`).join("\n")}`;
 }
 function getMinVersion() {
-  const picked = { "version": "0.36.0" };
+  const picked = { "version": "0.38.0" };
   if (typeof picked["version"] !== "string") {
     throw new TypeError("nmr/package.json: 'version' must be a string");
   }
@@ -618,6 +663,17 @@ function noRetiredVitestConfigs(cwd = process.cwd()) {
     cwd
   );
 }
+function noUnguardedLefthookInstall(cwd = process.cwd()) {
+  const scripts = readRootScripts(cwd);
+  const unguarded = 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) };
+}
 function noWorkspaceRunScriptReferences() {
   const packagesDir = join(process.cwd(), "packages");
   if (!existsSync(packagesDir)) return true;
@@ -652,6 +708,19 @@ function readPnpmFieldKeys(content) {
   const pnpm = parsed["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;
+  try {
+    parsed = JSON.parse(content);
+  } catch {
+    return {};
+  }
+  if (!isRecord(parsed)) return {};
+  const scripts = parsed["scripts"];
+  return isRecord(scripts) ? scripts : {};
+}
 function resolvesVersionViaWorkspace(range) {
   return WORKSPACE_VERSION_MARKERS.some((marker) => range.startsWith(marker));
 }
@@ -683,6 +752,22 @@ function describeMissingTazeConfig(cwd) {
   if (inert.length > 0) return `holds no code to call the factory: ${inert.join(", ")}`;
   return "taze.config.ts is missing";
 }
+function testSuiteGatesTestFileConventions(cwd = process.cwd()) {
+  if (findConventionsGuard(cwd) !== void 0) return true;
+  return {
+    ok: false,
+    detail: `no test file under __tests__ imports checkTestFileConventions from ${SHARED_TESTS_MODULE}`
+  };
+}
+function describeConventionsGuardSkip() {
+  const guard = findConventionsGuard(process.cwd());
+  return guard === void 0 ? false : `reported by checkTestFileConventions in ${guard}`;
+}
+function findConventionsGuard(cwd) {
+  return findTestFiles(cwd).find(
+    (relativePath) => importsSharedExport(readFileIn(cwd, relativePath), "checkTestFileConventions", SHARED_TESTS_MODULE)
+  );
+}
 function toolVersionsHasNoPnpm() {
   const content = readFile(".tool-versions");
   if (content === void 0) return true;
@@ -709,15 +794,18 @@ export {
   everyBinTargetIsACommittedWrapper,
   everyBinWrapperTargetIsCoveredByFiles,
   everyTestFileNamesItsTier,
+  everyTestFileSitsUnderTestsDir,
   everyViteConfigHasVitestConfig,
   hasSupportedEslintVersion,
   hasSupportedStrictLintVersion,
   noPnpmFieldInPackageJson,
   noReExportOnlyVitestConfigs,
   noRetiredVitestConfigs,
+  noUnguardedLefthookInstall,
   prettierConfigBuildsOnSharedConfig,
   tazeConfigAvoidsClobberedOptions,
   tazeConfigBuildsOnSharedConfig,
+  testSuiteGatesTestFileConventions,
   vitestConfigBuildsOnSharedConfig,
   vitestRootConfigBuildsOnSharedConfig
 };

package/.readyup/manifest.json

@@ -3,7 +3,7 @@
   "kits": [
     {
       "bundledDependencies": {
-        "@williamthorsen/toolbelt.errors": "0.7.0"
+        "@williamthorsen/toolbelt.errors": "0.8.1"
       },
       "checklists": [
         "nmr"
@@ -11,12 +11,12 @@
       "esbuildVersion": "0.28.2",
       "inputs": [
         {
-          "hash": "5c3cbdb9",
+          "hash": "488459c0",
           "kind": "module",
           "path": "kits/default.ts"
         },
         {
-          "hash": "6f709fcd",
+          "hash": "521b947a",
           "kind": "inline",
           "path": "../package.json",
           "paths": [
@@ -24,7 +24,7 @@
           ]
         },
         {
-          "hash": "74bd7994",
+          "hash": "06bffeed",
           "kind": "module",
           "path": "../src/default-scripts.ts"
         },
@@ -41,10 +41,10 @@
       ],
       "name": "default",
       "path": "kits/default.js",
-      "readyupVersion": "0.35.0",
+      "readyupVersion": "0.38.0",
       "source": "kits/default.ts",
-      "sourceHash": "5c3cbdb9",
-      "targetHash": "c24a46ec"
+      "sourceHash": "488459c0",
+      "targetHash": "cbef780a"
     }
   ]
 }

package/CHANGELOG.md

@@ -2,6 +2,88 @@
 
 All notable changes to this project will be documented in this file.
 
+## 0.38.0 — 2026-09-18
+
+### 🎉 Features
+
+- Give nmr plain output in pipes and CI logs (#838)
+
+  - Stops `nmr` from printing emoji to a pipe or a CI log: When the output stream is not a terminal or `CI` is set, a verdict reads `PASS`, `FAIL`, `SKIP`, or `NOOP` and a command icon is omitted, so that a log can be searched for a verdict.
+  - Adds `--output-style <auto|plain|rich>` to `nmr` and the `NMR_OUTPUT_STYLE` variable, each forcing either style; the flag outranks the variable, a value that names no style exits 1 with a usage error, and neither enters a check-cache key: A rich run and a plain run share a recorded pass.
+  - Replaces several rich glyphs, taking the status markers from the set that release-kit and v11y-check print so that one status has one glyph across the three tools: a recalled pass shows ⏩ in place of ⏭️, a skipped empty override shows ⚪ in place of ⛔, a warning and the `--log` refusal show 🟠 in place of ⚠️ and 📭, the `--log` header shows 💾 in place of 📼, and `nmr-ensure-prepublish-hooks` reports ✅ and ❌ in place of ✓ and ✗.
+
+### 🪦 Removed
+
+- 🚨 **Breaking:** Rename the declinesArgs step declaration to shouldDeclineArguments (#822)
+
+  - Makes a `.config/nmr.config.ts` that still declares `declinesArgs` on a composite script's `{ run, declinesArgs }` step fail to load, with an error that names `shouldDeclineArguments` as the replacement.
+  - Renames the matching field of the `StepSpec` type exported by `@williamthorsen/nmr/config` and `@williamthorsen/nmr/scripts`.
+
+  Migration: Rename `declinesArgs` to `shouldDeclineArguments` in each composite script step of `.config/nmr.config.ts`, and in any code that sets or reads the field on a `StepSpec`.
+
+## 0.37.0 — 2026-09-15
+
+### 🎉 Features
+
+- Guard lefthook install in root prepare and report unguarded installs (#798)
+
+  - Adds a `warn` check to the readyup kit of `@williamthorsen/nmr` that reports a root script run by `pnpm install`, such as `prepare` or `postinstall`, whose command calls `lefthook install` without `lefthook check-install`.
+  - Includes in the check's suggested fix the guarded form `lefthook check-install || lefthook install`, which skips rewriting `.git/hooks` while `lefthook.yml` is unchanged since the last install, so that a repo using that form can run `pnpm install` without write access to `.git/hooks`, as inside the Claude Code sandbox, and under pnpm's default `verifyDepsBeforeRun` avoids a reinstall before every `pnpm exec`.
+
+- Report misplaced test files and a missing guard from the readyup kit (#804)
+
+  - Adds an `error` check to the `@williamthorsen/nmr` readyup kit that reports each `*.test.ts` or `*.test.tsx` file outside a `__tests__` directory, which no Vitest project collects and which therefore runs nowhere.
+  - Adds a `warn` check that passes once a test file under `__tests__` imports `checkTestFileConventions` from `@williamthorsen/nmr/tests`, and whose fix shows the one-line guard test to add.
+  - Makes the kit skip both test-file checks once it finds that guard, because the guard reports the same files and applies the repo's `exclude` list, which the kit cannot read.
+
+### 🐛 Bug fixes
+
+- Fail when a -F pattern selects no workspace (#785)
+
+  - Fixes a run that reported success without doing anything. `nmr -F <pattern> <command>` printed nothing and exited 0 when the pattern matched no package. It now exits 1, names the rejected pattern, and explains the likely cause.
+  - Covers three further selections that ran nothing and exited 0: an exclusion that leaves only the workspace root, such as `nmr -F '!./packages/*'`; `-R` in a workspace that declares no package; and an empty `-F ''`.
+
+  Migration: Replace any `-F` pattern that names a directory with the package's manifest `name`. A filter that selects nothing now fails the run instead of passing in silence.
+
+- Fix test runs failing on a dependency with a TypeScript source export (#793)
+
+  - Fixes the issue that, with `resolveFromSource` at its default of `true`, every test in a file failed with `Stripping types is currently unsupported for files under node_modules` when the file's imports reached a dependency whose `exports` map declares a `source` condition pointing at TypeScript.
+  - Limits the source resolution enabled by `resolveFromSource` to packages whose real path lies outside every `node_modules`, which covers workspace packages and packages linked into the repo from elsewhere.
+  - Makes an import of a workspace package fail when that package's `source` condition names a missing file, instead of resolving to the build output without warning.
+  - Makes conditions that a config adds to `resolve.conditions` or `ssr.resolve.conditions` extend Vite's defaults when `resolveFromSource` is `false`, instead of replacing them.
+
+  Migration: Point each workspace package's `source` condition at a file that exists, or remove the condition. Build a workspace package before running a test that reaches it through an externalized dependency (one that Vitest loads through Node instead of transforming): Node resolves that import without the `source` condition, so it loads the package's build output.
+
+- Prevent a recursive or multi-package run from hanging under pnpm 12 (#814)
+
+  - Fixes a hang under pnpm 12: In a terminal, `nmr -R <command>`, a root command that runs in every package such as `nmr build`, and a `-F` pattern that selects several packages printed nothing and waited until Ctrl+C when a package command read keyboard input or changed the terminal's mode, as the spinner in `next build` does.
+  - Gives no terminal input to each package of a `-R` run or a multi-package `-F` run, including a dependency-chain filter such as `-F 'app...'`, and keeps it for a `-F` filter that selects one package, which an interactive command such as `nmr -F <pkg> test:watch` needs.
+
+### 🧪 Tests
+
+- Run the descriptor-inheritance test on Linux and skip it in a sandbox (#803)
+
+  - Stops `nmr ci` from failing in every Claude Code sandbox session on macOS by skipping the suite in `inherit-descriptor.tool.test.ts` when `SANDBOX_RUNTIME` is set, because the sandbox does not let `script(1)` open a pseudo-terminal.
+
+### ⚙️ Tooling
+
+- Upgrade eslint-config-typescript to v17 and adopt its ignore lists (#797)
+
+  - Declares `vitest` as `catalog:` in every workspace that imports it, with the pnpm catalog pinning 4.1.11, because v17 no longer exempts `vitest` from `n/no-extraneous-import`.
+  - Builds the global ignores in `eslint.config.ts` from the published `commonIgnores` and `toolIgnores`, which also exclude `pnpm-lock.yaml` and each `.readyup/manifest.json`, so the `eslint --fix` in `nmr lint` can no longer rewrite a kit manifest and make the next build's `rdy verify` fail.
+
+### 📚 Documentation
+
+- Trim the README and move reference detail into shipped topic docs (#800)
+
+  - Deletes the default script registry tables and the CLI flag table, which `nmr --help` prints, and the migration sections, which `CHANGELOG.md` records, rather than moving them.
+  - Adds `docs` to the files published with `@williamthorsen/nmr`, and changes the closing pointer in `agents/guidance/rulebooks/nmr.md` to name each topic's document under `node_modules/@williamthorsen/nmr/docs/` in place of `README.md`.
+
+- Trim the release-kit README and move reference detail into shipped docs (#802)
+
+  - Adds `docs` to the published `files` of `@williamthorsen/release-kit` and a `!docs/*.v*.md` exclusion to both it and `@williamthorsen/nmr`, because `pnpm pack` would otherwise include the release-notes previews that a local checkout holds.
+  - Adds `--version` to the options that `release-kit --help` lists, and points the hints for the removed `releaseNotes.shouldCreateGithubRelease` field at `release-kit init` instead of the README.
+
 ## 0.36.0 — 2026-09-08
 
 ### 🎉 Features