@williamthorsen/nmr 0.15.0 → 0.17.0

package/AGENTS.md

@@ -38,7 +38,7 @@ Some root scripts (e.g. `lint`, `typecheck`, `test`) expand to `nmr root:X && pn
 
 ## Managed build
 
-The default `compile` script runs `nmr-compile`, a standalone bin that esbuild-compiles a package's `src` to `dist/esm`, rewriting `~/` (package-root) import aliases and `.ts`→`.js` specifiers, and skipping work when inputs are unchanged. There is no repo-local build script to maintain; `nmr build` runs `compile` then `generate-typings`. To find or debug the build, look to `nmr-compile`, not a `config/build.ts` in the consuming repo.
+The default `compile` script runs `nmr-compile`, a standalone bin that uses the TypeScript compiler API to emit a package's `src` to `dist/esm` as `.js` and `.d.ts` in one pass, rewriting relative `.ts`→`.js` specifiers and tsconfig `paths` aliases in both outputs, and skipping work when inputs are unchanged. `typescript` is a peer dependency (`>=5.7.0`). There is no repo-local build script to maintain, and no separate typings step; `nmr build` runs `compile` alone. To find or debug the build, look to `nmr-compile`, not a `config/build.ts` in the consuming repo.
 
 ## Override behaviors
 

package/README.md

@@ -3,27 +3,13 @@
 Context-aware script runner for PNPM monorepos. Ships an `nmr` (node-monorepo run) binary that provides centralized, consistent script execution across workspace packages and the monorepo root.
 
 <!-- section:release-notes -->
-## Release notes — v0.15.0 (2026-06-27)
+## Release notes — v0.17.0 (2026-07-01)
 
 ### 🎉 Features
 
-- Centralize the per-package build as an nmr-compile bin (#419)
+- 🚨 **Breaking:** Rebuild nmr-compile on a unified tsc emit (#455)
 
-  Introduces `nmr-compile`, a single command shipped with `@williamthorsen/nmr` that compiles each workspace package and now backs the default build. Consuming repos can delete their own per-package build script and pick up future build fixes just by upgrading nmr. Repeated builds with unchanged source now reliably skip recompiling instead of occasionally rebuilding for no reason, and import aliases now resolve correctly in symlinked checkouts.
-
-### 🐛 Bug fixes
-
-- Write the build cache only after a successful compile (#421)
-
-  Fixes an issue where a package compile that failed partway — from a crash, disk error, or transient build failure — could leave stale or incomplete output that the next build skipped over as unchanged. The next build now retries the failed compile, so recovering no longer requires a manual `nmr clean` or a source-file edit.
-
-- Stop sync-agent-files --check failing on version-only bumps (#424)
-
-  Fixes an issue where `nmr sync-agent-files --check` failed after an nmr upgrade even when the managed agent guidance was identical. The check now passes whenever the guidance content is current, even if the version has changed.
-
-- Remove inapplicable bootstrap fallback from agent guidance (#425)
-
-  Fixes an issue where the agent guidance bundled with nmr told coding agents to run a recovery step that exists only in the nmr repo itself.
+  `nmr` now compiles a package's JavaScript and type declarations in one step, rewriting every import form — static, re-export, dynamic `import()`, bare side-effect, and tsconfig `paths` aliases — to runnable `.js` in both outputs. It now requires TypeScript 5.7 or newer as a peer dependency.
 <!-- /section:release-notes -->
 
 ## Installation
@@ -40,7 +26,7 @@ From a package directory:
 
 ```bash
 nmr test          # Run tests for the current package
-nmr build         # Compile + generate typings
+nmr build         # Compile to .js and .d.ts in one pass
 nmr check         # Typecheck, format check, lint check, and tests
 ```
 
@@ -83,11 +69,11 @@ Scripts resolve through three tiers. Higher tiers override lower ones:
 
 Given the `build` command for a package that defines its own build script:
 
-| Tier | Source                  | Value                             | Wins? |
-| ---- | ----------------------- | --------------------------------- | ----- |
-| 1    | Built-in default        | `['compile', 'generate-typings']` | —     |
-| 2    | `.config/nmr.config.ts` | _(not set)_                       | —     |
-| 3    | `package.json` scripts  | `"tsx custom-build.ts"`           | ✓     |
+| Tier | Source                  | Value                   | Wins? |
+| ---- | ----------------------- | ----------------------- | ----- |
+| 1    | Built-in default        | `['compile']`           | —     |
+| 2    | `.config/nmr.config.ts` | _(not set)_             | —     |
+| 3    | `package.json` scripts  | `"tsx custom-build.ts"` | ✓     |
 
 If no per-package override exists, the highest-tier value that is set wins. Set a script to `""` in `package.json` to skip it for that package.
 
@@ -98,8 +84,8 @@ If no per-package override exists, the highest-tier value that is set wins. Set
 Script values can be `string` or `string[]`. Arrays expand to chained `nmr` sub-invocations:
 
 ```ts
-// "build": ["compile", "generate-typings"]
-// expands to: nmr compile && nmr generate-typings
+// "fix": ["lint", "fmt"]
+// expands to: nmr lint && nmr fmt
 ```
 
 ## Configuration
@@ -153,34 +139,35 @@ These scripts are available out of the box. Repo-wide config (tier 2) and per-pa
 
 ### Workspace scripts
 
-| Command            | Runs                                                     |
-| ------------------ | -------------------------------------------------------- |
-| `build`            | `compile`, `generate-typings`                            |
-| `check`            | `typecheck`, `fmt:check`, `lint:check`, `test`           |
-| `check:strict`     | `typecheck`, `fmt:check`, `lint:strict`, `test:coverage` |
-| `clean`            | `pnpm exec rimraf dist/*`                                |
-| `compile`          | `nmr-compile`                                            |
-| `fix`              | `lint`, `fmt`                                            |
-| `fix:check`        | `fmt:check`, `lint:check`                                |
-| `fmt`              | `prettier --list-different --write .`                    |
-| `fmt:check`        | `prettier --check .`                                     |
-| `generate-typings` | `tsc --project tsconfig.generate-typings.json`           |
-| `lint`             | `eslint --fix .`                                         |
-| `lint:check`       | `eslint .`                                               |
-| `lint:strict`      | `strict-lint`                                            |
-| `test`             | `pnpm exec vitest`                                       |
-| `test:coverage`    | `pnpm exec vitest --coverage`                            |
-| `test:watch`       | `pnpm exec vitest --watch`                               |
-| `typecheck`        | `tsgo --noEmit`                                          |
-| `view-coverage`    | `open coverage/index.html`                               |
+| Command         | Runs                                                             |
+| --------------- | ---------------------------------------------------------------- |
+| `attw`          | `attw --pack --profile esm-only`                                 |
+| `build`         | `compile`                                                        |
+| `check`         | `typecheck`, `fmt:check`, `lint:check`, `test`                   |
+| `check:strict`  | `typecheck`, `fmt:check`, `lint:strict`, `test:coverage`, `attw` |
+| `clean`         | `pnpm exec rimraf dist/*`                                        |
+| `compile`       | `nmr-compile`                                                    |
+| `fix`           | `lint`, `fmt`                                                    |
+| `fix:check`     | `fmt:check`, `lint:check`                                        |
+| `fmt`           | `prettier --list-different --write .`                            |
+| `fmt:check`     | `prettier --check .`                                             |
+| `lint`          | `eslint --fix .`                                                 |
+| `lint:check`    | `eslint .`                                                       |
+| `lint:strict`   | `strict-lint`                                                    |
+| `test`          | `pnpm exec vitest`                                               |
+| `test:coverage` | `pnpm exec vitest --coverage`                                    |
+| `test:watch`    | `pnpm exec vitest --watch`                                       |
+| `typecheck`     | `tsgo --noEmit`                                                  |
+| `view-coverage` | `open coverage/index.html`                                       |
 
 #### Integration test variant
 
-Packages with a `vitest.integration.config.ts` file get different test commands. Use the `--int-test` flag to select this variant.
+A package gets this variant automatically when it contains a `vitest.integration.config.ts` file: `test` and `test:coverage` run the standalone (non-integration) suite, and integration tests run only when invoked explicitly. Activation is detected per package, so it applies under the recursive `nmr ci` fan-out. Opting in requires **both** `vitest.integration.config.ts` and `vitest.standalone.config.ts`, since the standalone scripts reference the latter.
 
 | Command            | Runs                                                               |
 | ------------------ | ------------------------------------------------------------------ |
 | `test`             | `pnpm exec vitest --config=vitest.standalone.config.ts`            |
+| `test:all`         | `pnpm exec vitest`                                                 |
 | `test:coverage`    | `pnpm exec vitest --config=vitest.standalone.config.ts --coverage` |
 | `test:integration` | `pnpm exec vitest --config=vitest.integration.config.ts`           |
 | `test:watch`       | `pnpm exec vitest --config=vitest.standalone.config.ts --watch`    |
@@ -197,11 +184,12 @@ Packages with a `vitest.integration.config.ts` file get different test commands.
 
 #### Check and quality
 
-| Command             | Runs                                                                          |
-| ------------------- | ----------------------------------------------------------------------------- |
-| `check`             | `typecheck`, `fmt:check`, `lint:check`, `test`                                |
-| `check:agent-files` | `nmr-sync-agent-files --check`                                                |
-| `check:strict`      | `typecheck`, `fmt:check`, `lint:strict`, `test:coverage`, `check:agent-files` |
+| Command             | Runs                                                                                  |
+| ------------------- | ------------------------------------------------------------------------------------- |
+| `attw`              | `pnpm --recursive exec nmr attw`                                                      |
+| `check`             | `typecheck`, `fmt:check`, `lint:check`, `test`                                        |
+| `check:agent-files` | `nmr-sync-agent-files --check`                                                        |
+| `check:strict`      | `typecheck`, `fmt:check`, `lint:strict`, `test:coverage`, `attw`, `check:agent-files` |
 
 #### Fix
 
@@ -294,14 +282,13 @@ nmr [flags] <command> [args...]
 | `-q, --quiet`            | Suppress info messages; show full output on failure | —       |
 | `-?, --help`             | Show available commands                             | —       |
 | `-V, --version`          | Show version number                                 | —       |
-| `--int-test`             | Use integration test scripts                        | —       |
 
 ### Examples
 
 ```bash
 # From a package directory
 nmr test                    # Run workspace test script
-nmr build                   # Compile + generate typings
+nmr build                   # Compile to .js and .d.ts in one pass
 
 # From the monorepo root
 nmr test                    # Root tests + recursive workspace tests
@@ -358,7 +345,9 @@ nmr sync-pnpm-version
 
 ### `nmr-compile`
 
-Compile a single package's `src` tree to `dist/esm` with esbuild. Rewrites `~/` (package-root) import aliases and `.ts`→`.js` specifiers to match the emitted output, and skips the build when no input has changed (a content-and-path hash is cached in `dist/esm/.cache`). This is the default `compile` script — run it from a package directory.
+Compile a single package's `src` tree to `dist/esm` with the TypeScript compiler API, emitting `.js` and `.d.ts` in one pass. Because the compiler parses each source file, every relative import form — static, re-export, dynamic `import()`, and bare side-effect — is rewritten from `.ts` to `.js` in both outputs, and `.ts` occurrences inside strings and comments are left intact. tsconfig `paths` aliases are resolved to runnable relative `.js` specifiers in both outputs, sourced from the package's tsconfig. The build is skipped when no input has changed (a content-and-path hash is cached in `dist/esm/.cache`). This is the default `compile` script — run it from a package directory.
+
+`typescript` is a peer dependency (`>=5.7.0`, required for `rewriteRelativeImportExtensions`); the consuming repo provides it. Relative imports in source must carry explicit `.ts` extensions for them to be rewritten.
 
 ```bash
 nmr-compile

package/dist/esm/.cache

@@ -1 +1 @@
-d2f01850cfe4f42bf00cb8b8eaf3d3fab438916c419957842d927d22fb9d67cf
+49f79592247d14646b1445473aa445a8bcbeafb478c6dc9949eb31cee8e2a627

package/dist/esm/cli-build.js

@@ -1,7 +1,8 @@
 import { buildPackage } from "./commands/build.js";
 try {
-  await buildPackage(process.cwd());
-} catch (error) {
-  console.error(error instanceof Error ? error.message : String(error));
-  process.exit(1);
+    await buildPackage(process.cwd());
+}
+catch (error) {
+    process.stderr.write(`${error instanceof Error ? error.message : String(error)}\n`);
+    process.exitCode = 1;
 }

package/dist/esm/cli-ensure-prepublish-hooks.js

@@ -1,62 +1,43 @@
+import { parseArgsOrExit, reportError } from '@williamthorsen/nmr-core';
 import { DEFAULT_HOOK, ensurePrepublishHooks } from "./commands/ensure-prepublish-hooks.js";
 import { findMonorepoRoot } from "./context.js";
-let fix = false;
-let dryRun = false;
-let command;
-const args = process.argv.slice(2);
-for (let i = 0; i < args.length; i++) {
-  const arg = args[i];
-  switch (arg) {
-    case "--fix":
-      fix = true;
-      break;
-    case "--dry-run":
-      dryRun = true;
-      break;
-    case "--command":
-      i++;
-      command = args[i];
-      if (!command) {
-        console.error("--command requires a value");
-        process.exit(1);
-      }
-      break;
-    default:
-      console.error(`Unknown argument: ${arg}`);
-      process.exit(1);
-  }
-}
+const flagSchema = {
+    fix: { long: '--fix', type: 'boolean' },
+    dryRun: { long: '--dry-run', type: 'boolean' },
+    command: { long: '--command', type: 'string' },
+};
+const { fix, dryRun, command } = parseArgsOrExit(process.argv.slice(2), flagSchema).flags;
 try {
-  const monorepoRoot = findMonorepoRoot();
-  const hookCommand = command ?? DEFAULT_HOOK;
-  const options = command ? { fix, dryRun, command } : { fix, dryRun };
-  const result = ensurePrepublishHooks(monorepoRoot, options);
-  for (const pkg of result.packages) {
-    if (pkg.isPrivate) {
-      continue;
+    const monorepoRoot = findMonorepoRoot();
+    const hookCommand = command ?? DEFAULT_HOOK;
+    const options = command ? { fix, dryRun, command } : { fix, dryRun };
+    const result = ensurePrepublishHooks(monorepoRoot, options);
+    for (const pkg of result.packages) {
+        if (pkg.isPrivate) {
+            continue;
+        }
+        switch (pkg.action) {
+            case 'ok':
+                console.info(`✓ ${pkg.packageName}: prepublishOnly = "${pkg.prepublishOnly}"`);
+                break;
+            case 'missing':
+                console.warn(`✗ ${pkg.packageName}: missing prepublishOnly`);
+                break;
+            case 'fixed':
+                console.info(`✓ ${pkg.packageName}: added prepublishOnly = "${hookCommand}"`);
+                break;
+            case 'would-fix':
+                console.info(`~ ${pkg.packageName}: would add prepublishOnly = "${hookCommand}"`);
+                break;
+        }
     }
-    switch (pkg.action) {
-      case "ok":
-        console.info(`\u2713 ${pkg.packageName}: prepublishOnly = "${pkg.prepublishOnly}"`);
-        break;
-      case "missing":
-        console.warn(`\u2717 ${pkg.packageName}: missing prepublishOnly`);
-        break;
-      case "fixed":
-        console.info(`\u2713 ${pkg.packageName}: added prepublishOnly = "${hookCommand}"`);
-        break;
-      case "would-fix":
-        console.info(`~ ${pkg.packageName}: would add prepublishOnly = "${hookCommand}"`);
-        break;
+    if (result.hasFailures) {
+        const missing = result.packages.filter((p) => p.action === 'missing').length;
+        process.stderr.write(`\n${missing} package(s) missing prepublishOnly. Use --fix to add it.\n`);
+        process.exitCode = 1;
     }
-  }
-  if (result.hasFailures) {
-    const missing = result.packages.filter((p) => p.action === "missing").length;
-    console.error(`
-${missing} package(s) missing prepublishOnly. Use --fix to add it.`);
-    process.exit(1);
-  }
-} catch (error) {
-  console.error(error instanceof Error ? error.message : String(error));
-  process.exit(1);
+}
+catch (error) {
+    reportError(error instanceof Error ? error.message : String(error));
+    process.exitCode = 1;
 }

package/dist/esm/cli-report-overrides.js

@@ -1,9 +1,11 @@
+import { reportError } from '@williamthorsen/nmr-core';
 import { reportOverrides } from "./commands/report-overrides.js";
 import { findMonorepoRoot } from "./context.js";
 try {
-  const monorepoRoot = findMonorepoRoot();
-  reportOverrides(monorepoRoot);
-} catch (error) {
-  console.error(error instanceof Error ? error.message : String(error));
-  process.exit(1);
+    const monorepoRoot = findMonorepoRoot();
+    reportOverrides(monorepoRoot);
+}
+catch (error) {
+    reportError(error instanceof Error ? error.message : String(error));
+    process.exitCode = 1;
 }

package/dist/esm/cli-sync-agent-files.js

@@ -1,38 +1,29 @@
-import { parseArgs, translateParseError } from "@williamthorsen/nmr-core";
+import { parseArgsOrExit, reportError } from '@williamthorsen/nmr-core';
 import { check, sync } from "./commands/sync-agent-files.js";
 import { findMonorepoRoot } from "./context.js";
 const flagSchema = {
-  check: { long: "--check", type: "boolean" }
+    check: { long: '--check', type: 'boolean' },
 };
-const { flags } = parseArgsOrExit(process.argv.slice(2));
+const { flags } = parseArgsOrExit(process.argv.slice(2), flagSchema);
 try {
-  runCommand(flags.check);
-} catch (error) {
-  console.error(error instanceof Error ? error.message : String(error));
-  process.exit(1);
+    runCommand(flags.check);
 }
-function parseArgsOrExit(argv) {
-  try {
-    return parseArgs(argv, flagSchema);
-  } catch (error) {
-    console.error(`Error: ${translateParseError(error)}`);
+catch (error) {
+    reportError(error instanceof Error ? error.message : String(error));
     process.exit(1);
-  }
 }
 function runCommand(checkOnly) {
-  const monorepoRoot = findMonorepoRoot();
-  if (!checkOnly) {
-    const { path: destinationPath, packageSpecifier, changed } = sync(monorepoRoot);
-    console.info(
-      changed ? `\u2713 Wrote ${destinationPath} (${packageSpecifier})` : `\u2713 ${destinationPath} already up to date`
-    );
-    process.exit(0);
-  }
-  const result = check(monorepoRoot);
-  if (result.ok) {
-    console.info("\u2713 .agents/nmr/AGENTS.md is in sync");
-    process.exit(0);
-  }
-  console.error(result.reason);
-  process.exit(1);
+    const monorepoRoot = findMonorepoRoot();
+    if (!checkOnly) {
+        const { path: destinationPath, packageSpecifier, changed } = sync(monorepoRoot);
+        console.info(changed ? `✓ Wrote ${destinationPath} (${packageSpecifier})` : `✓ ${destinationPath} already up to date`);
+        process.exit(0);
+    }
+    const result = check(monorepoRoot);
+    if (result.ok) {
+        console.info('✓ .agents/nmr/AGENTS.md is in sync');
+        process.exit(0);
+    }
+    process.stderr.write(`${result.reason}\n`);
+    process.exit(1);
 }

package/dist/esm/cli-sync-pnpm-version.js

@@ -1,9 +1,11 @@
+import { reportError } from '@williamthorsen/nmr-core';
 import { syncPnpmVersion } from "./commands/sync-pnpm-version.js";
 import { findMonorepoRoot } from "./context.js";
 try {
-  const monorepoRoot = findMonorepoRoot();
-  syncPnpmVersion(monorepoRoot);
-} catch (error) {
-  console.error(error instanceof Error ? error.message : String(error));
-  process.exit(1);
+    const monorepoRoot = findMonorepoRoot();
+    syncPnpmVersion(monorepoRoot);
+}
+catch (error) {
+    reportError(error instanceof Error ? error.message : String(error));
+    process.exitCode = 1;
 }

package/dist/esm/cli.js

@@ -1,16 +1,16 @@
-import process from "node:process";
+import process from 'node:process';
 import { runCli } from "./runCli.js";
 try {
-  const { exitCode } = await runCli({
-    args: process.argv.slice(2),
-    cwd: process.cwd(),
-    env: process.env,
-    stdout: process.stdout,
-    stderr: process.stderr
-  });
-  process.exit(exitCode);
-} catch (error) {
-  process.stderr.write(`${error instanceof Error ? error.stack ?? error.message : String(error)}
-`);
-  process.exit(1);
+    const { exitCode } = await runCli({
+        args: process.argv.slice(2),
+        cwd: process.cwd(),
+        env: process.env,
+        stdout: process.stdout,
+        stderr: process.stderr,
+    });
+    process.exitCode = exitCode;
+}
+catch (error) {
+    process.stderr.write(`${error instanceof Error ? (error.stack ?? error.message) : String(error)}\n`);
+    process.exitCode = 1;
 }

package/dist/esm/commands/build.d.ts

@@ -1,15 +1,9 @@
-import { type Format, type Platform } from 'esbuild';
 export interface BuildOptions {
-    aliases?: Record<string, string>;
     entryGlobs?: string[];
     ignore?: string[];
     outdir?: string;
     cacheFile?: string;
-    format?: Format;
-    platform?: Platform;
-    target?: string;
 }
 export declare function buildPackage(packageDir: string, options?: BuildOptions): Promise<void>;
-export declare function computeBuildHash(packageDir: string, files: string[], outputConfig: object): Promise<string>;
-export declare function resolveAliasImports(code: string, fileDir: string, aliasMap: Record<string, string>, packageDir: string): string;
-export declare function rewriteTsImportExtensions(code: string): string;
+export declare function computeBuildHash(packageDir: string, files: string[], emitConfig: object): Promise<string>;
+export declare function resolveTsconfigChain(packageDir: string, configFileName?: string): string[];