@bobfrankston/npmglobalize 1.0.194 → 1.0.196

package/README.md

@@ -454,6 +454,19 @@ Workspace mode is auto-detected when run from a root with `"private": true` and
                 target. Fixes arborist "Cannot read properties of null"
                 crashes caused by sibling file: deps with nested
                 node_modules. Suggested automatically when the error hits.
+-ts7-report, -deprecation-report
+                Report-only: scan this package and its file: deps for
+                compilerOptions removed in TypeScript 7 and list a migration
+                to-do. Writes nothing.
+-tsfix, -ts7-fix
+                One-off utility, NOT part of the main flow: apply the TS7
+                tsconfig migration to a package (and its file: deps) and exit —
+                no build, no commit, no push, no publish. Intended as a
+                temporary tool for fixing a local/subdirectory tsconfig in
+                place. The normal release flow already applies the same
+                migration automatically when a build hits a TS7 deprecation
+                error, so day-to-day you never need this flag.
+                    npmglobalize <path> -tsfix
 -show           Show package.json dependency changes
 -package, -pkg  Update package.json scripts to use npmglobalize (see below)
 -h, -help       Show help
@@ -577,14 +590,43 @@ Before transforming or publishing anything, `npmglobalize` builds `file:` depend
 
 For each project visited (the target and every transitive `file:` dep):
 
-- If `tsconfig.json` is missing or has `"noEmit": true` → **skip** (not a TypeScript build).
-- If `tsconfig.json` exists but `package.json` has no `"build"` script → **prompt** to add `"build": "tsc"`. Decline and that project is skipped.
+- If `tsconfig.json` is missing or has `"noEmit": true` → **skip** (not a TypeScript build), unless the `build` script runs `importgen` — a plain-JS browser app still needs its import map regenerated.
+- If `tsconfig.json` exists but `package.json` has no `"build"` script → **prompt** to add `"build": "tsc"` (plus a `tsc -p <dir>` per sub-project — see below). Decline and that project is skipped.
 - Otherwise → run `npm run build`. A failure halts the cascade unless `-force` is passed.
 
 Cycle-safe via a shared visited set; each project is built at most once per run.
 
 This complements the existing publish cascade (which ensures version refs are correct) by closing the build-freshness gap that `npm install` alone left open.
 
+#### Import maps (`importgen`) as a build step
+
+Browser projects that use [`importgen`](https://www.npmjs.com/package/@bobfrankston/importgen) have historically regenerated their import map from `.vscode/tasks.json`, which only runs when VS Code opens the folder — so a command-line or CI build could publish a stale map. `npmglobalize` treats the import map as a build product and moves the step into the package's own `build` script, where every build path picks it up.
+
+Before building each project, it checks whether the project is an importgen project, in this order:
+
+1. `.vscode/tasks.json` has a task whose `command` is `importgen` (the HTML file, if the task names one, is reused).
+2. A root-level `.htm`/`.html` file already contains a `<script type="importmap">` block (`index.html`, `default.html`, `default.htm` are checked first, so a stray `temp.htm` doesn't win).
+3. `importgen` is in `dependencies`/`devDependencies` **and** the project has one of those HTML files — the HTML requirement keeps packages that merely *use* importgen as a library from matching.
+
+If a signal matches and the `build` script doesn't already run `importgen`, you're prompted to rewrite it — e.g. `"build": "tsc"` → `"build": "importgen default.htm && tsc"`. The HTML file is named explicitly so importgen doesn't have to guess. Decline and `"importgen": false` is written to `.globalize.json5`, which suppresses the prompt for good; `-noimportgen` does the same from the command line.
+
+Once wired, the freshness check gains a second condition: a project whose build runs importgen is only considered up to date if the generated HTML is at least as new as `package.json`. Adding a dependency changes the import map without touching any `.ts` file, which the source-vs-output comparison alone would miss.
+
+#### Sub-projects (a second `tsconfig.json` in a sub-directory)
+
+A package can hold more than one TypeScript project. The common shape is a service worker in `Sw/` with its own `tsconfig.json` (`lib: ["WebWorker"]`, its own `outDir`) that the root `tsconfig.json` lists under `exclude` — so a bare `"build": "tsc"` compiles everything *except* the service worker, and the stale `sw2.js` ships. As with import maps, these have historically been built only by a second `.vscode/tasks.json` watcher, which runs on folder open and nowhere else.
+
+Before building, `npmglobalize` looks for sub-projects, in this order:
+
+1. `.vscode/tasks.json` has a task that runs `tsc` with `"options": { "cwd": "${workspaceFolder}/Sw" }` or with `-p`/`--project` naming a sub-directory (authoritative — it's how the project is actually built today; the task's `label` is quoted back in the prompt).
+2. An immediate sub-directory containing its own `tsconfig.json`. Build output and vendored trees (`node_modules`, `prev`, `dist`, `built`, `out`, `wwwroot`, `coverage`, `temp`, `preflight`, dot-directories) are never scanned.
+
+Nothing is detected when the root `tsconfig.json` uses project `references` — that build graph belongs to `tsc -b` and isn't second-guessed.
+
+When the `build` script is `tsc`-driven and doesn't already compile a detected sub-project, you're prompted to append it — `"build": "tsc"` → `"build": "tsc && tsc -p Sw"`, or for an importgen project `"build": "importgen default.htm && tsc && tsc -p Sw"`. Already-wired scripts are recognized in any of their spellings (`tsc -p Sw`, `tsc --project ./Sw/tsconfig.json`, `cd Sw && tsc`), and the directory is emitted with its real on-disk casing so the script still works on WSL and CI. Decline and `"subProjects": false` is written to `.globalize.json5`, suppressing the prompt for good; a `"subProjects": ["Sw"]` array pins the list instead of detecting it.
+
+Sub-projects the build actually compiles are also folded into the freshness check — the package rebuilds when a sub-project's sources are newer than its output. A sub-project that emits *outside* its own directory (`"outDir": ".."`, the usual service-worker case) always reports stale: comparing its sources against the whole package would prove nothing.
+
 #### TypeScript 6 `types` auto-fix
 
 TypeScript 6 dropped the legacy behavior of auto-including every installed `@types/*` package. A `tsconfig.json` with no explicit `compilerOptions.types` then loses the Node globals (`process`, `Buffer`, …) and the build fails with `TS2591`.

package/cli.js

@@ -56,6 +56,9 @@ Dependency Options:
                            public visibility to already-published deps in the cascade.
                            Use when a public consumer's deps weren't all marked public.
   -no-prescan, -nps        Skip upfront dep-graph prescan
+  -force-build, -fb        Rebuild file: deps even when their output looks up
+                           to date (freshness check compares .ts/tsconfig
+                           mtimes against emitted .js/.d.ts)
   -force-publish           Republish dependencies even if version exists
   -fix                     Run npm audit fix after transformation
   -no-use-paths, -nup      Declare this package standalone — file: deps shouldn't
@@ -70,7 +73,9 @@ Install Options:
   -freeze        Freeze node_modules (replace symlinks with real copies for network shares)
   -nofreeze      Disable freeze
   -importgen     Run importgen to update import maps before publishing
-  -noimportgen   Disable importgen
+                 (auto-detected from .vscode/tasks.json or an import map in the
+                 project's HTML — npmglobalize offers to add it to the build script)
+  -noimportgen   Disable importgen (recorded in .globalize.json5, stops the prompt)
   -once          Don't persist flags to .globalize.json5
 
 Mode Options:
@@ -122,10 +127,12 @@ Other Options:
                   moduleResolution, target es3, deprecated flags). Lists a
                   migration to-do list and exits. Writes nothing.
   -tsfix, -ts7-fix
-                  Fix-only: apply the TS7 tsconfig migration to this package
-                  and its file: deps, then exit. Only tsconfig files change —
-                  no build, no commit, no push, no publish. Use on any local
-                  directory: npmglobalize <path> -tsfix
+                  One-off utility, not the main flow: apply the TS7 tsconfig
+                  migration to this package and its file: deps, then exit.
+                  Only tsconfig files change — no build, no commit, no push,
+                  no publish. The release flow auto-applies the same migration
+                  on a TS7 build error; use this only for a temporary in-place
+                  fix of a local directory: npmglobalize <path> -tsfix
   -show           Show package.json dependency changes
   -package, -pkg  Update package.json scripts to use npmglobalize
   -h, -help       Show this help
@@ -243,6 +250,10 @@ function parseArgs(args) {
             case '-force':
                 options.force = true;
                 break;
+            case '-force-build':
+            case '-fb':
+                options.forceBuild = true;
+                break;
             case '-files':
                 options.files = true;
                 options.explicitKeys.add('files');
@@ -555,14 +566,14 @@ export async function main() {
     // whose source has changed since their last build.
     if (!cliOptions.cleanup) {
         await ensureFileDepModules(cwd, !!cliOptions.verbose);
-        const depsOk = await buildFileDepsTopologically(cwd, { verbose: !!cliOptions.verbose, force: !!cliOptions.force });
+        const depsOk = await buildFileDepsTopologically(cwd, { verbose: !!cliOptions.verbose, force: !!cliOptions.force, forceBuild: !!cliOptions.forceBuild });
         if (!depsOk && !cliOptions.force) {
             printBuildSummary();
             process.exit(1);
         }
         if (!depsOk)
             console.log(colors.warn('Continuing with --force despite dep build failure...'));
-        const targetOk = await buildProject(cwd, { verbose: !!cliOptions.verbose, force: !!cliOptions.force });
+        const targetOk = await buildProject(cwd, { verbose: !!cliOptions.verbose, force: !!cliOptions.force, forceBuild: !!cliOptions.forceBuild });
         if (!targetOk) {
             if (!cliOptions.force) {
                 printBuildSummary();

package/lib.d.ts

@@ -101,6 +101,11 @@ export interface GlobalizeOptions {
     once?: boolean;
     /** Run importgen to update import maps before publishing */
     importgen?: boolean;
+    /** Sub-directories with their own tsconfig.json that `build` must also compile
+     *  (e.g. a service worker in `Sw/`). undefined: auto-detect from
+     *  .vscode/tasks.json + nested tsconfigs. An array pins the list.
+     *  false: never ask about sub-projects here. */
+    subProjects?: string[] | false;
     /** Use filesystem paths for `file:` deps (default true). Set false to
      *  mark a package as publishable/installable even when sibling checkouts
      *  are absent. Currently declarative (recorded in config and displayed);
@@ -108,6 +113,8 @@ export interface GlobalizeOptions {
     usePaths?: boolean;
     /** Local install only — skip transform/publish, just npm install -g . */
     local?: boolean;
+    /** Build every file: dep even when outputs look up to date (skips the freshness check) */
+    forceBuild?: boolean;
     /** Freeze node_modules: replace symlinks/junctions with real copies for network share use */
     freeze?: boolean;
     /** Before `npm pack`, delete `node_modules/` inside each `file:` dep target.
@@ -311,13 +318,62 @@ export declare function missingDeps(pkgDir: string, pkg: any): string[];
  *  re-run" (partial-sync) cases. Also covers `cwd` itself on the first call.
  *  Cycle-safe via the shared `visited` set. */
 export declare function ensureFileDepModules(cwd: string, verbose?: boolean, visited?: Set<string>): Promise<void>;
+/** Cheap freshness check so the build cascade can skip packages whose output is
+ *  already current. Returns true only when provably up to date:
+ *  - outDir projects: newest source (.ts/.tsx/.mts/.cts + tsconfig*.json) must
+ *    not be newer than the newest file under outDir.
+ *  - side-by-side projects: every source must have an emitted sibling
+ *    (.js/.jsx/.mjs/.cjs or declaration) at least as new.
+ *  Conservative: project references, allowJs, missing outputs, or unreadable
+ *  tsconfig all report stale (→ build). package.json mtime is deliberately
+ *  ignored — npmglobalize itself rewrites it around every publish, which would
+ *  otherwise force a rebuild on every run. */
+export declare function isBuildUpToDate(cwd: string): boolean;
+/** Detect whether a package is an importgen project — a browser app whose HTML
+ *  carries a generated `<script type="importmap">`. Signals, in order:
+ *    1. `.vscode/tasks.json` runs `importgen` (how these projects are driven
+ *       today: VS Code starts an importgen watcher on folder open),
+ *    2. a root HTML file already contains an import map,
+ *    3. importgen is a declared dependency.
+ *  Returns the HTML file importgen should update plus the signal that matched,
+ *  or null when this isn't an importgen project. */
+export declare function detectImportgen(cwd: string): {
+    htmlFile?: string;
+    reason: string;
+} | null;
+/** Wire `importgen` into the package's own `build` script when the project turns
+ *  out to be an importgen project. The import map is a build product, so it
+ *  belongs in `npm run build` — that way it regenerates everywhere (VS Code, CI,
+ *  a bare command line, npmglobalize) instead of only when VS Code happens to
+ *  start its watcher on folder open. Declining records `importgen: false` in
+ *  .globalize.json5 so it stops asking. */
+export declare function ensureImportgenInBuild(cwd: string): Promise<void>;
+/** Sub-directories that need their own `tsc -p <dir>` pass. A package can hold
+ *  more than one tsconfig — a service worker compiled with `lib: WebWorker`, say,
+ *  which the root tsconfig `exclude`s — and the root `tsc` silently skips them.
+ *  Signals, in order:
+ *    1. `.vscode/tasks.json` runs tsc with `options.cwd` or `-p` pointing at a
+ *       sub-dir (authoritative: it's how the developer actually builds today),
+ *    2. an immediate sub-directory holding its own tsconfig.json.
+ *  Returns nothing when the root tsconfig uses project `references` — `tsc -b`
+ *  owns that build graph and shouldn't be second-guessed. */
+export declare function detectSubProjects(cwd: string): Array<{
+    dir: string;
+    reason: string;
+}>;
+/** Append `tsc -p <dir>` for every sub-project the existing build script misses.
+ *  Only touches tsc-driven builds — a bundler-driven `build` has its own idea of
+ *  what to compile. Declining records `subProjects: false` in .globalize.json5. */
+export declare function ensureSubProjectsInBuild(cwd: string): Promise<void>;
 /** Build a single project: detect tsconfig, prompt to add `build: tsc` if a
  *  TypeScript project lacks a build script, run `npm run build`, record
  *  failures. Returns true if build succeeded (or was skipped because no
- *  tsconfig / noEmit / no build script after declining the prompt). */
+ *  tsconfig / noEmit / no build script after declining the prompt / output
+ *  already up to date). Pass `forceBuild` to skip the freshness check. */
 export declare function buildProject(cwd: string, opts?: {
     verbose?: boolean;
     force?: boolean;
+    forceBuild?: boolean;
 }): Promise<boolean>;
 /** Walk `file:` deps depth-first (deps before consumers) and build each one
  *  that has a tsconfig. Mirrors `ensureFileDepModules`'s traversal but invokes
@@ -327,6 +383,7 @@ export declare function buildProject(cwd: string, opts?: {
 export declare function buildFileDepsTopologically(cwd: string, opts?: {
     verbose?: boolean;
     force?: boolean;
+    forceBuild?: boolean;
 }, visited?: Set<string>): Promise<boolean>;
 /** A single TS7-readiness problem found in a package's effective tsconfig. */
 export interface Ts7Finding {

package/lib.js

@@ -299,6 +299,10 @@ export function writeConfig(dir, config, explicitKeys) {
                 comment = '  // Resolve file: deps from sibling checkouts (set false for standalone packages)';
             else if (key === 'allowTs')
                 comment = '  // Keep .ts source (and *.map, tsconfig.json) in the published npm tarball';
+            else if (key === 'importgen')
+                comment = '  // Regenerate browser import maps (false = never ask about importgen here)';
+            else if (key === 'subProjects')
+                comment = '  // Sub-dirs with their own tsconfig that build must compile (false = never ask)';
             lines.push(`  "${key}": ${jsonValue}${comma}${comment}`);
         });
         lines.push('');
@@ -321,6 +325,8 @@ export function writeConfig(dir, config, explicitKeys) {
     lines.push('  // "freeze": false              // Freeze node_modules (replace symlinks with real copies)');
     lines.push('  // "usePaths": true              // Resolve file: deps from siblings; false = use latest npm version (standalone)');
     lines.push('  // "allowTs": false              // Include .ts source in npm tarball (auto-true for noEmit projects)');
+    lines.push('  // "importgen": auto             // Detected from .vscode/tasks.json / import map in HTML; false = never ask');
+    lines.push('  // "subProjects": auto           // Sub-dirs with their own tsconfig (e.g. ["Sw"]); detected from .vscode/tasks.json; false = never ask');
     lines.push('}');
     fs.writeFileSync(configPath, lines.join('\n') + '\n');
 }
@@ -1720,6 +1726,41 @@ export function printPnpmSuggestionSummary() {
         console.error(l);
     console.error('');
 }
+/** Run `npm install` in `dir`; on failure, offer to delete `node_modules/`
+ *  and `package-lock.json` and retry once — the standard fix for arborist's
+ *  null-`package` crash on stale lockfile / junction state — before falling
+ *  back to the pnpm suggestion banner. Returns true if install succeeded.
+ *  `rmSync` unlinks junctions without descending, so file: sibling targets
+ *  are never touched. */
+async function npmInstallWithCleanRetry(dir, verbose, context) {
+    let r = await runCommandAsync('npm', ['install'], { cwd: dir, silent: !verbose });
+    if (r.success)
+        return true;
+    console.error(colors.red(`  ✗ npm install failed in ${dir}`));
+    if (r.stderr)
+        console.error(colors.dim(r.stderr.split('\n').slice(0, 5).join('\n')));
+    if (await confirm(`  Delete node_modules/ and package-lock.json in ${dir} and retry npm install?`, true)) {
+        try {
+            fs.rmSync(path.join(dir, 'node_modules'), { recursive: true, force: true });
+        }
+        catch { /* best-effort */ }
+        try {
+            fs.rmSync(path.join(dir, 'package-lock.json'), { force: true });
+        }
+        catch { /* best-effort */ }
+        console.log(colors.yellow(`↻ clean reinstall in ${dir}`));
+        r = await runCommandAsync('npm', ['install'], { cwd: dir, silent: !verbose });
+        if (r.success) {
+            console.log(colors.green(`  ✓ clean reinstall succeeded in ${dir}`));
+            return true;
+        }
+        console.error(colors.red(`  ✗ npm install still failing in ${dir} after clean reinstall`));
+        if (r.stderr)
+            console.error(colors.dim(r.stderr.split('\n').slice(0, 5).join('\n')));
+    }
+    await suggestPnpmOnInstallFailure(context);
+    return false;
+}
 /** Wait for a package version to appear on the npm registry.
  *  First-time publishes (brand-new package name) take much longer to
  *  propagate than version bumps — npm has no cached metadata to update,
@@ -1877,13 +1918,7 @@ export async function ensureFileDepModules(cwd, verbose = false, visited = new S
             catch { /* best-effort */ }
         }
         console.log(colors.yellow(`↻ installing node_modules in ${pkg?.name || abs} (${formatMissingReason(cwdMissing)})`));
-        const r = await runCommandAsync('npm', ['install'], { cwd: abs, silent: !verbose });
-        if (!r.success) {
-            console.error(colors.red(`  ✗ npm install failed in ${abs}`));
-            if (r.stderr)
-                console.error(colors.dim(r.stderr.split('\n').slice(0, 5).join('\n')));
-            await suggestPnpmOnInstallFailure(`ensureFileDepModules (cwd): ${abs}`);
-        }
+        await npmInstallWithCleanRetry(abs, verbose, `ensureFileDepModules (cwd): ${abs}`);
     }
     for (const key of ['dependencies', 'devDependencies']) {
         const deps = pkg?.[key];
@@ -1915,13 +1950,7 @@ export async function ensureFileDepModules(cwd, verbose = false, visited = new S
                     catch { /* best-effort */ }
                 }
                 console.log(colors.yellow(`↻ restoring node_modules in ${name} (${target}) (${formatMissingReason(targetMissing)})`));
-                const r = await runCommandAsync('npm', ['install'], { cwd: target, silent: !verbose });
-                if (!r.success) {
-                    console.error(colors.red(`  ✗ npm install failed in ${target}`));
-                    if (r.stderr)
-                        console.error(colors.dim(r.stderr.split('\n').slice(0, 5).join('\n')));
-                    await suggestPnpmOnInstallFailure(`ensureFileDepModules (file: dep ${name}): ${target}`);
-                }
+                await npmInstallWithCleanRetry(target, verbose, `ensureFileDepModules (file: dep ${name}): ${target}`);
             }
             await ensureFileDepModules(target, verbose, visited);
         }
@@ -2244,11 +2273,505 @@ function migrateTsconfigDeprecations(cwd) {
     }
     return changed;
 }
+/** Newest mtime of any regular file under `dir` (recursive). Symlinks/junctions
+ *  are not followed — the file: dep webs here link back into sibling checkouts. */
+function newestMtimeUnder(dir) {
+    let newest = 0;
+    let entries;
+    try {
+        entries = fs.readdirSync(dir, { withFileTypes: true });
+    }
+    catch {
+        return 0;
+    }
+    for (const e of entries) {
+        if (e.isSymbolicLink())
+            continue;
+        const full = path.join(dir, e.name);
+        if (e.isDirectory()) {
+            newest = Math.max(newest, newestMtimeUnder(full));
+        }
+        else if (e.isFile()) {
+            try {
+                newest = Math.max(newest, fs.statSync(full).mtimeMs);
+            }
+            catch { /* ignore */ }
+        }
+    }
+    return newest;
+}
+/** Cheap freshness check so the build cascade can skip packages whose output is
+ *  already current. Returns true only when provably up to date:
+ *  - outDir projects: newest source (.ts/.tsx/.mts/.cts + tsconfig*.json) must
+ *    not be newer than the newest file under outDir.
+ *  - side-by-side projects: every source must have an emitted sibling
+ *    (.js/.jsx/.mjs/.cjs or declaration) at least as new.
+ *  Conservative: project references, allowJs, missing outputs, or unreadable
+ *  tsconfig all report stale (→ build). package.json mtime is deliberately
+ *  ignored — npmglobalize itself rewrites it around every publish, which would
+ *  otherwise force a rebuild on every run. */
+export function isBuildUpToDate(cwd) {
+    let tsconfig;
+    try {
+        tsconfig = JSON5.parse(fs.readFileSync(path.join(cwd, 'tsconfig.json'), 'utf-8'));
+    }
+    catch {
+        return false;
+    }
+    const co = tsconfig.compilerOptions || {};
+    if (tsconfig.references || co.allowJs || co.checkJs)
+        return false;
+    const outDir = typeof co.outDir === 'string' ? path.resolve(cwd, co.outDir) : null;
+    // tsconfig "exclude" dirs aren't compiled, so their .ts files never get
+    // outputs — don't count them as sources. Match on the literal path prefix
+    // (glob tails like "tests/**" reduce to "tests").
+    const excludes = (Array.isArray(tsconfig.exclude) ? tsconfig.exclude : [])
+        .filter((x) => typeof x === 'string')
+        .map((x) => x.replace(/^\.\//, '').replace(/[\\/]?\*.*$/, '').replace(/[\\/]+$/, '').toLowerCase())
+        .filter((x) => x.length > 0 && !x.includes('*'));
+    const sources = [];
+    const collect = (dir, rel) => {
+        let entries;
+        try {
+            entries = fs.readdirSync(dir, { withFileTypes: true });
+        }
+        catch {
+            return;
+        }
+        for (const e of entries) {
+            if (e.isSymbolicLink())
+                continue;
+            const full = path.join(dir, e.name);
+            const relPath = rel ? `${rel}/${e.name.toLowerCase()}` : e.name.toLowerCase();
+            if (e.isDirectory()) {
+                const lower = e.name.toLowerCase();
+                if (lower === 'node_modules' || lower === 'prev' || e.name.startsWith('.'))
+                    continue;
+                if (outDir && path.resolve(full) === outDir)
+                    continue;
+                if (excludes.includes(relPath))
+                    continue;
+                collect(full, relPath);
+            }
+            else if (/\.(ts|tsx|mts|cts)$/.test(e.name) && !/\.d\.(ts|mts|cts)$/.test(e.name)) {
+                if (excludes.includes(relPath))
+                    continue;
+                sources.push(full);
+            }
+        }
+    };
+    collect(cwd, '');
+    if (sources.length === 0)
+        return true;
+    const mtime = (p) => {
+        try {
+            return fs.statSync(p).mtimeMs;
+        }
+        catch {
+            return -1;
+        }
+    };
+    // Any tsconfig*.json edit (own or extended base in the package) invalidates the build.
+    let configTime = 0;
+    try {
+        for (const name of fs.readdirSync(cwd)) {
+            if (/^tsconfig.*\.json$/.test(name))
+                configTime = Math.max(configTime, mtime(path.join(cwd, name)));
+        }
+    }
+    catch { /* ignore */ }
+    if (outDir) {
+        const newestOut = newestMtimeUnder(outDir);
+        if (newestOut === 0)
+            return false;
+        const newestSrc = Math.max(configTime, ...sources.map(mtime));
+        return newestSrc <= newestOut;
+    }
+    // Side-by-side emit: each source needs an output sibling at least as new.
+    let newestOut = 0;
+    for (const src of sources) {
+        const m = src.match(/\.(ts|tsx|mts|cts)$/);
+        const base = src.slice(0, -m[0].length);
+        const candidates = m[1] === 'mts' ? ['.mjs', '.d.mts']
+            : m[1] === 'cts' ? ['.cjs', '.d.cts']
+                : ['.js', '.jsx', '.d.ts'];
+        let outTime = -1;
+        for (const ext of candidates)
+            outTime = Math.max(outTime, mtime(base + ext));
+        if (outTime < 0)
+            return false;
+        if (mtime(src) > outTime)
+            return false;
+        newestOut = Math.max(newestOut, outTime);
+    }
+    return configTime <= newestOut;
+}
+/** HTML files importgen looks for when none is named, in its preference order. */
+const IMPORTGEN_HTML_CANDIDATES = ['index.html', 'default.html', 'default.htm'];
+/** Resolve the HTML file importgen would update in `cwd`. */
+function findImportgenHtml(cwd, preferred) {
+    if (preferred && fs.existsSync(path.join(cwd, preferred)))
+        return preferred;
+    for (const name of IMPORTGEN_HTML_CANDIDATES) {
+        if (fs.existsSync(path.join(cwd, name)))
+            return name;
+    }
+    return undefined;
+}
+/** True when a script string invokes importgen (`importgen`, `npx importgen`,
+ *  `importgen default.htm`, …) — used to tell a wired build from an unwired one. */
+function scriptRunsImportgen(script) {
+    return /(^|[\s/\\&|])importgen(\s|$)/.test(script);
+}
+/** Detect whether a package is an importgen project — a browser app whose HTML
+ *  carries a generated `<script type="importmap">`. Signals, in order:
+ *    1. `.vscode/tasks.json` runs `importgen` (how these projects are driven
+ *       today: VS Code starts an importgen watcher on folder open),
+ *    2. a root HTML file already contains an import map,
+ *    3. importgen is a declared dependency.
+ *  Returns the HTML file importgen should update plus the signal that matched,
+ *  or null when this isn't an importgen project. */
+export function detectImportgen(cwd) {
+    // 1. .vscode/tasks.json — JSON5 because VS Code allows comments/trailing commas
+    try {
+        const tasks = JSON5.parse(fs.readFileSync(path.join(cwd, '.vscode', 'tasks.json'), 'utf-8'));
+        for (const task of Array.isArray(tasks?.tasks) ? tasks.tasks : []) {
+            const command = typeof task?.command === 'string' ? task.command : '';
+            const args = Array.isArray(task?.args) ? task.args.filter((a) => typeof a === 'string') : [];
+            if (!scriptRunsImportgen(command))
+                continue;
+            const named = args.find(a => /\.html?$/i.test(a));
+            return { htmlFile: findImportgenHtml(cwd, named), reason: '.vscode/tasks.json runs importgen' };
+        }
+    }
+    catch {
+        // No tasks.json, or unparsable — fall through to the other signals
+    }
+    // 2. A root HTML file that already holds an import map. Check importgen's own
+    //    candidates first so `default.htm` wins over a stray `temp.htm` copy.
+    let htmlFiles = [];
+    try {
+        htmlFiles = fs.readdirSync(cwd).filter(f => /\.html?$/i.test(f));
+    }
+    catch {
+        // Unreadable directory — nothing to detect
+    }
+    htmlFiles.sort((a, b) => {
+        const rank = (f) => {
+            const i = IMPORTGEN_HTML_CANDIDATES.indexOf(f.toLowerCase());
+            return i === -1 ? IMPORTGEN_HTML_CANDIDATES.length : i;
+        };
+        return rank(a) - rank(b) || a.localeCompare(b);
+    });
+    for (const name of htmlFiles) {
+        try {
+            const html = fs.readFileSync(path.join(cwd, name), 'utf-8');
+            if (/<script\s+type=["']importmap["']/i.test(html)) {
+                return { htmlFile: name, reason: `${name} contains an import map` };
+            }
+        }
+        catch {
+            // Unreadable file — try the next one
+        }
+    }
+    // 3. importgen declared as a dependency AND an HTML file to inject into.
+    //    The HTML requirement matters: a package can depend on importgen as a
+    //    library (npmglobalize itself does) without being a browser app.
+    const htmlFile = findImportgenHtml(cwd);
+    if (htmlFile) {
+        try {
+            const pkg = readPackageJson(cwd);
+            for (const key of ['dependencies', 'devDependencies']) {
+                const deps = pkg?.[key];
+                if (!deps || typeof deps !== 'object')
+                    continue;
+                if (Object.keys(deps).some(n => n === 'importgen' || n.endsWith('/importgen'))) {
+                    return { htmlFile, reason: `importgen is in ${key}` };
+                }
+            }
+        }
+        catch {
+            // No package.json — caller handles that
+        }
+    }
+    return null;
+}
+/** Wire `importgen` into the package's own `build` script when the project turns
+ *  out to be an importgen project. The import map is a build product, so it
+ *  belongs in `npm run build` — that way it regenerates everywhere (VS Code, CI,
+ *  a bare command line, npmglobalize) instead of only when VS Code happens to
+ *  start its watcher on folder open. Declining records `importgen: false` in
+ *  .globalize.json5 so it stops asking. */
+export async function ensureImportgenInBuild(cwd) {
+    const config = readConfig(cwd);
+    if (config.importgen === false)
+        return;
+    let pkg;
+    try {
+        pkg = readPackageJson(cwd);
+    }
+    catch {
+        return;
+    }
+    const buildScript = typeof pkg.scripts?.build === 'string' ? pkg.scripts.build : '';
+    if (scriptRunsImportgen(buildScript))
+        return; // already wired
+    const detected = config.importgen === true
+        ? { htmlFile: findImportgenHtml(cwd), reason: 'importgen: true in .globalize.json5' }
+        : detectImportgen(cwd);
+    if (!detected)
+        return;
+    const name = pkg.name || path.basename(cwd);
+    // Name the HTML file explicitly: importgen's own search order may pick a
+    // different file than the one this project actually ships.
+    const command = detected.htmlFile ? `importgen ${detected.htmlFile}` : 'importgen';
+    const newScript = buildScript ? `${command} && ${buildScript}` : command;
+    console.log(colors.yellow(`${name} looks like an importgen project (${detected.reason}) but its build script doesn't run importgen.`));
+    const addIt = await confirm(`Set "build": "${newScript}" in ${name}'s package.json?`, true);
+    if (!addIt) {
+        writeConfig(cwd, { ...config, importgen: false }, new Set(['importgen']));
+        console.log(colors.dim(`  Recorded importgen: false in .globalize.json5 — won't ask again.`));
+        return;
+    }
+    if (!pkg.scripts)
+        pkg.scripts = {};
+    pkg.scripts.build = newScript;
+    writePackageJson(cwd, pkg);
+    console.log(colors.green(`✓ Added importgen to ${name}'s build script`));
+}
+/** True when the generated import map is at least as new as the package.json it
+ *  is generated from. `isBuildUpToDate` only compares .ts sources to emitted
+ *  .js, so it can't see a dependency added to package.json — which is exactly
+ *  what changes an import map. */
+function isImportMapUpToDate(cwd) {
+    const htmlFile = findImportgenHtml(cwd, detectImportgen(cwd)?.htmlFile);
+    if (!htmlFile)
+        return false; // can't tell which file — let the build run
+    const mtime = (p) => { try {
+        return fs.statSync(p).mtimeMs;
+    }
+    catch {
+        return 0;
+    } };
+    const htmlTime = mtime(path.join(cwd, htmlFile));
+    return htmlTime > 0 && htmlTime >= mtime(path.join(cwd, 'package.json'));
+}
+/** Directories never worth scanning for a nested tsconfig — build output, vendored
+ *  code, and the prev/ archive all carry stray tsconfigs that aren't sub-projects. */
+const SUBPROJECT_SKIP_DIRS = new Set([
+    'node_modules', 'prev', 'built', 'build', 'dist', 'out', 'output',
+    'wwwroot', 'coverage', 'temp', 'tmp', 'preflight'
+]);
+/** True when a script string invokes `tsc` (`tsc`, `npx tsc`, `tsc -p Sw`, …). */
+function scriptRunsTsc(script) {
+    return /(^|[\s&|(])(npx\s+)?tsc(\s|$)/.test(script);
+}
+/** Normalize a path as written in a script/task (quotes, backslashes, `./`,
+ *  a trailing `tsconfig.json`) down to a comparable relative directory. */
+function normalizeProjectPath(raw) {
+    return raw
+        .replace(/^["']|["']$/g, '')
+        .replace(/\$\{workspaceFolder\}/gi, '')
+        .replace(/\\/g, '/')
+        .replace(/^\.\//, '')
+        .replace(/\/tsconfig\.json$/i, '')
+        .replace(/^\/+|\/+$/g, '');
+}
+/** Match `rel` against what's actually on disk so the emitted script uses the
+ *  real casing (`Sw`, not `sw`) — Windows compares case-insensitively, but the
+ *  script also has to run on WSL/CI where it doesn't. Returns null if absent. */
+function resolveOnDiskCase(base, rel) {
+    const parts = rel.split('/').filter(Boolean);
+    let dir = base;
+    const out = [];
+    for (const part of parts) {
+        let entries;
+        try {
+            entries = fs.readdirSync(dir);
+        }
+        catch {
+            return null;
+        }
+        const match = entries.find(e => e.toLowerCase() === part.toLowerCase());
+        if (!match)
+            return null;
+        out.push(match);
+        dir = path.join(dir, match);
+    }
+    return out.length ? out.join('/') : null;
+}
+/** True when `script` already compiles the tsconfig in `dir` — `tsc -p Sw`,
+ *  `tsc --project ./Sw/tsconfig.json`, or `cd Sw && tsc`. */
+function scriptBuildsSubProject(script, dir) {
+    const target = normalizeProjectPath(dir).toLowerCase();
+    if (!target)
+        return false;
+    for (const re of [/(?:^|\s)(?:-p|--project)[\s=]+("[^"]+"|'[^']+'|\S+)/g,
+        /(?:^|[\s&|(])cd\s+("[^"]+"|'[^']+'|\S+)/g]) {
+        for (const m of script.matchAll(re)) {
+            if (normalizeProjectPath(m[1]).toLowerCase() === target)
+                return true;
+        }
+    }
+    return false;
+}
+/** Sub-directories that need their own `tsc -p <dir>` pass. A package can hold
+ *  more than one tsconfig — a service worker compiled with `lib: WebWorker`, say,
+ *  which the root tsconfig `exclude`s — and the root `tsc` silently skips them.
+ *  Signals, in order:
+ *    1. `.vscode/tasks.json` runs tsc with `options.cwd` or `-p` pointing at a
+ *       sub-dir (authoritative: it's how the developer actually builds today),
+ *    2. an immediate sub-directory holding its own tsconfig.json.
+ *  Returns nothing when the root tsconfig uses project `references` — `tsc -b`
+ *  owns that build graph and shouldn't be second-guessed. */
+export function detectSubProjects(cwd) {
+    try {
+        const root = JSON5.parse(fs.readFileSync(path.join(cwd, 'tsconfig.json'), 'utf-8'));
+        if (Array.isArray(root?.references) && root.references.length > 0)
+            return [];
+    }
+    catch {
+        // No root tsconfig (or unparsable) — nested ones still count
+    }
+    const found = new Map(); // on-disk relative dir → reason
+    const add = (raw, reason) => {
+        const rel = normalizeProjectPath(raw);
+        if (!rel || rel.startsWith('..') || path.isAbsolute(rel))
+            return;
+        if (rel.split('/').some(p => SUBPROJECT_SKIP_DIRS.has(p.toLowerCase()) || p.startsWith('.')))
+            return;
+        const onDisk = resolveOnDiskCase(cwd, rel);
+        if (!onDisk)
+            return;
+        if (!fs.existsSync(path.join(cwd, onDisk, 'tsconfig.json')))
+            return;
+        if (!found.has(onDisk))
+            found.set(onDisk, reason);
+    };
+    // 1. .vscode/tasks.json — JSON5 because VS Code allows comments/trailing commas
+    try {
+        const tasks = JSON5.parse(fs.readFileSync(path.join(cwd, '.vscode', 'tasks.json'), 'utf-8'));
+        for (const task of Array.isArray(tasks?.tasks) ? tasks.tasks : []) {
+            const command = typeof task?.command === 'string' ? task.command : '';
+            const args = Array.isArray(task?.args) ? task.args.filter((a) => typeof a === 'string') : [];
+            if (!scriptRunsTsc([command, ...args].join(' ')))
+                continue;
+            const label = typeof task?.label === 'string' ? task.label : 'a tsc task';
+            const projIdx = args.findIndex(a => a === '-p' || a === '--project');
+            if (projIdx !== -1 && args[projIdx + 1]) {
+                add(args[projIdx + 1], `.vscode/tasks.json "${label}" builds it`);
+            }
+            else if (typeof task?.options?.cwd === 'string') {
+                add(task.options.cwd, `.vscode/tasks.json "${label}" runs tsc there`);
+            }
+        }
+    }
+    catch {
+        // No tasks.json, or unparsable — fall through to the filesystem scan
+    }
+    // 2. Immediate sub-directories with their own tsconfig.json
+    try {
+        for (const e of fs.readdirSync(cwd, { withFileTypes: true })) {
+            if (!e.isDirectory() || e.isSymbolicLink())
+                continue;
+            if (e.name.startsWith('.') || SUBPROJECT_SKIP_DIRS.has(e.name.toLowerCase()))
+                continue;
+            if (fs.existsSync(path.join(cwd, e.name, 'tsconfig.json'))) {
+                add(e.name, `${e.name}/tsconfig.json`);
+            }
+        }
+    }
+    catch {
+        // Unreadable directory — nothing more to detect
+    }
+    return [...found].map(([dir, reason]) => ({ dir, reason }));
+}
+/** The sub-projects to wire, honoring `.globalize.json5`: `false` disables the
+ *  whole check, an array pins the list instead of detecting it. */
+function configuredSubProjects(cwd, config) {
+    if (config.subProjects === false)
+        return [];
+    if (Array.isArray(config.subProjects)) {
+        return config.subProjects
+            .map(d => ({ dir: resolveOnDiskCase(cwd, normalizeProjectPath(d)) || d, reason: 'listed in .globalize.json5' }))
+            .filter(s => fs.existsSync(path.join(cwd, s.dir, 'tsconfig.json')));
+    }
+    return detectSubProjects(cwd);
+}
+/** Quote a path for a package.json script only when it needs it. */
+function quoteScriptPath(p) {
+    return /[\s&|<>]/.test(p) ? `"${p}"` : p;
+}
+/** Append `tsc -p <dir>` for every sub-project the existing build script misses.
+ *  Only touches tsc-driven builds — a bundler-driven `build` has its own idea of
+ *  what to compile. Declining records `subProjects: false` in .globalize.json5. */
+export async function ensureSubProjectsInBuild(cwd) {
+    const config = readConfig(cwd);
+    if (config.subProjects === false)
+        return;
+    let pkg;
+    try {
+        pkg = readPackageJson(cwd);
+    }
+    catch {
+        return;
+    }
+    const buildScript = typeof pkg.scripts?.build === 'string' ? pkg.scripts.build.trim() : '';
+    if (!buildScript || !scriptRunsTsc(buildScript))
+        return;
+    const missing = configuredSubProjects(cwd, config)
+        .filter(s => !scriptBuildsSubProject(buildScript, s.dir));
+    if (missing.length === 0)
+        return;
+    const name = pkg.name || path.basename(cwd);
+    const newScript = [buildScript, ...missing.map(s => `tsc -p ${quoteScriptPath(s.dir)}`)].join(' && ');
+    // Default to yes only when tasks.json says the developer already builds these
+    // — a bare nested tsconfig (tests, examples) is a weaker signal, and this
+    // prompt rewrites a build script that currently works.
+    const strong = missing.every(s => s.reason.startsWith('.vscode/tasks.json') || s.reason.startsWith('listed in'));
+    console.log(colors.yellow(`${name}'s build script doesn't compile ${missing.length === 1 ? 'a sub-project' : 'its sub-projects'}:`));
+    for (const s of missing)
+        console.log(colors.yellow(`  ${s.dir}/ — ${s.reason}`));
+    const addIt = await confirm(`Set "build": "${newScript}" in ${name}'s package.json?`, strong);
+    if (!addIt) {
+        writeConfig(cwd, { ...config, subProjects: false }, new Set(['subProjects']));
+        console.log(colors.dim(`  Recorded subProjects: false in .globalize.json5 — won't ask again.`));
+        return;
+    }
+    pkg.scripts.build = newScript;
+    writePackageJson(cwd, pkg);
+    console.log(colors.green(`✓ Added ${missing.map(s => `tsc -p ${s.dir}`).join(' && ')} to ${name}'s build script`));
+}
+/** Freshness for the sub-projects the build script actually compiles. A
+ *  sub-project that emits outside its own directory (a service worker with
+ *  `outDir: ".."`, say) reports stale: `isBuildUpToDate` would then be comparing
+ *  its sources against the whole package, which proves nothing. */
+function areSubProjectsUpToDate(cwd, subs) {
+    for (const s of subs) {
+        const dir = path.join(cwd, s.dir);
+        let co;
+        try {
+            co = JSON5.parse(fs.readFileSync(path.join(dir, 'tsconfig.json'), 'utf-8')).compilerOptions || {};
+        }
+        catch {
+            return false;
+        }
+        if (typeof co.outDir === 'string' && path.relative(dir, path.resolve(dir, co.outDir)).startsWith('..'))
+            return false;
+        if (!isBuildUpToDate(dir))
+            return false;
+    }
+    return true;
+}
 /** Build a single project: detect tsconfig, prompt to add `build: tsc` if a
  *  TypeScript project lacks a build script, run `npm run build`, record
  *  failures. Returns true if build succeeded (or was skipped because no
- *  tsconfig / noEmit / no build script after declining the prompt). */
+ *  tsconfig / noEmit / no build script after declining the prompt / output
+ *  already up to date). Pass `forceBuild` to skip the freshness check. */
 export async function buildProject(cwd, opts = {}) {
+    // Do this before the tsconfig check: an importgen project may be plain JS
+    // with no tsconfig at all, and still need its import map regenerated.
+    await ensureImportgenInBuild(cwd);
     let shouldBuild = false;
     try {
         const tsconfigPath = path.join(cwd, 'tsconfig.json');
@@ -2259,24 +2782,63 @@ export async function buildProject(cwd, opts = {}) {
     catch {
         // No tsconfig — skip
     }
-    if (!shouldBuild)
+    let earlyPkg;
+    try {
+        earlyPkg = readPackageJson(cwd);
+    }
+    catch {
         return true;
-    const pkg = readPackageJson(cwd);
-    if (!pkg.scripts?.build) {
+    }
+    const runsImportgen = scriptRunsImportgen(typeof earlyPkg.scripts?.build === 'string' ? earlyPkg.scripts.build : '');
+    // A build script that regenerates the import map is worth running even when
+    // there's nothing for tsc to emit.
+    if (!shouldBuild && !runsImportgen)
+        return true;
+    const pkg = earlyPkg;
+    const existingBuild = typeof pkg.scripts?.build === 'string' ? pkg.scripts.build.trim() : '';
+    // A build script ensureImportgenInBuild just created for a package that had
+    // none still needs its tsc pass — treat importgen-only as "no build yet".
+    const importgenOnly = !!existingBuild && existingBuild.split(/\s*&&\s*/).every(scriptRunsImportgen);
+    if (shouldBuild && (!existingBuild || importgenOnly)) {
+        // Compile the sub-projects (service worker, …) in the same breath: the
+        // root tsc excludes them, so a bare "tsc" would leave them unbuilt.
+        const subs = configuredSubProjects(cwd, readConfig(cwd));
+        const tscParts = ['tsc', ...subs.map(s => `tsc -p ${quoteScriptPath(s.dir)}`)];
+        const newScript = [...(existingBuild ? [existingBuild] : []), ...tscParts].join(' && ');
         console.log(colors.yellow(`TypeScript project has no "build" script in ${pkg.name || cwd}`));
-        const addIt = await confirm(`Add "build": "tsc" to ${pkg.name || path.basename(cwd)}'s package.json?`, true);
+        for (const s of subs)
+            console.log(colors.yellow(`  sub-project ${s.dir}/ — ${s.reason}`));
+        const addIt = await confirm(`Add "build": "${newScript}" to ${pkg.name || path.basename(cwd)}'s package.json?`, true);
         if (addIt) {
             if (!pkg.scripts)
                 pkg.scripts = {};
-            pkg.scripts.build = 'tsc';
+            pkg.scripts.build = newScript;
             writePackageJson(cwd, pkg);
-            console.log(colors.green(`✓ Added "build": "tsc" to ${pkg.name || path.basename(cwd)}`));
+            console.log(colors.green(`✓ Added "build": "${newScript}" to ${pkg.name || path.basename(cwd)}`));
         }
         else {
             return true;
         }
     }
+    else if (existingBuild) {
+        await ensureSubProjectsInBuild(cwd);
+        try {
+            const refreshed = readPackageJson(cwd);
+            if (typeof refreshed.scripts?.build === 'string')
+                pkg.scripts.build = refreshed.scripts.build;
+        }
+        catch { /* keep the script we already have */ }
+    }
     ensureTsconfigNodeTypes(cwd);
+    // Only the sub-projects the final build script really compiles count toward
+    // freshness — one the user declined to wire in isn't this build's business.
+    const finalBuild = typeof pkg.scripts?.build === 'string' ? pkg.scripts.build : '';
+    const wiredSubs = detectSubProjects(cwd).filter(s => scriptBuildsSubProject(finalBuild, s.dir));
+    if (!opts.forceBuild && isBuildUpToDate(cwd) && areSubProjectsUpToDate(cwd, wiredSubs)
+        && (!runsImportgen || isImportMapUpToDate(cwd))) {
+        console.log(colors.dim(`– Build up to date (${pkg.name || path.basename(cwd)})`));
+        return true;
+    }
     console.log(`Building ${pkg.name || cwd}...`);
     let buildResult = await runCommandAsync('npm', ['run', 'build'], { cwd, silent: true });
     if (!buildResult.success) {
@@ -2549,13 +3111,7 @@ export async function ensureWorkspaceDepModules(rootDir, members, verbose = fals
         return;
     const list = [...allMissing];
     console.log(colors.yellow(`↻ installing workspace node_modules in ${rootPkg?.name || path.basename(root)} (${formatMissingReason(list)})`));
-    const r = await runCommandAsync('npm', ['install'], { cwd: root, silent: !verbose });
-    if (!r.success) {
-        console.error(colors.red(`  ✗ npm install failed in ${root}`));
-        if (r.stderr)
-            console.error(colors.dim(r.stderr.split('\n').slice(0, 5).join('\n')));
-        await suggestPnpmOnInstallFailure(`ensureWorkspaceDepModules: ${root}`);
-    }
+    await npmInstallWithCleanRetry(root, verbose, `ensureWorkspaceDepModules: ${root}`);
 }
 /** Extract the lines a human needs from a failed npm command's output: the
  *  `npm error` lines minus stack frames and brace-only object-dump fragments,
@@ -5045,8 +5601,10 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
             console.log('  [dry-run] Would run: npm run build');
         }
     }
-    // Run importgen if enabled (update import maps in HTML files)
-    if (options.importgen) {
+    // Run importgen if explicitly enabled and the build script isn't already
+    // doing it (the CLI wires it into `build` for detected importgen projects).
+    const buildRunsImportgen = scriptRunsImportgen(typeof pkg.scripts?.build === 'string' ? pkg.scripts.build : '');
+    if (options.importgen && !buildRunsImportgen) {
         try {
             if (!dryRun) {
                 const igResult = runImportgen(cwd);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/npmglobalize",
-  "version": "1.0.194",
+  "version": "1.0.196",
   "description": "Transform file: dependencies to npm versions for publishing",
   "main": "index.js",
   "type": "module",