nx 23.0.0 → 23.0.2

package/dist/src/utils/min-release-age/behavior/pnpm.js

@@ -2,13 +2,10 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.readPnpmPolicy = readPnpmPolicy;
 exports.pickPnpmVersion = pickPnpmVersion;
-const fs_1 = require("fs");
-const os_1 = require("os");
-const path_1 = require("path");
+const child_process_1 = require("child_process");
 const semver_1 = require("semver");
 const constants_1 = require("../constants");
 const errors_1 = require("../errors");
-const npmrc_1 = require("../npmrc");
 const pick_1 = require("../pick");
 // Ordered newest-bound-first so the first matching row applies.
 const PNPM_BEHAVIOR_ROWS = [
@@ -19,7 +16,6 @@ const PNPM_BEHAVIOR_ROWS = [
         strictAutoOnWhenExplicit: true,
         excludeGrammar: 'v2-globs-unions',
         writesExcludes: true,
-        uppercaseEnv: true,
     },
     {
         minVersion: '11.1.0',
@@ -28,7 +24,6 @@ const PNPM_BEHAVIOR_ROWS = [
         strictAutoOnWhenExplicit: true,
         excludeGrammar: 'v2-globs-unions',
         writesExcludes: false,
-        uppercaseEnv: true,
     },
     {
         minVersion: '11.0.4',
@@ -37,7 +32,6 @@ const PNPM_BEHAVIOR_ROWS = [
         strictAutoOnWhenExplicit: true,
         excludeGrammar: 'v2-globs-unions',
         writesExcludes: false,
-        uppercaseEnv: false,
     },
     {
         minVersion: '11.0.0',
@@ -46,7 +40,6 @@ const PNPM_BEHAVIOR_ROWS = [
         strictAutoOnWhenExplicit: false,
         excludeGrammar: 'v2-globs-unions',
         writesExcludes: false,
-        uppercaseEnv: false,
     },
     {
         minVersion: '10.19.0',
@@ -55,7 +48,6 @@ const PNPM_BEHAVIOR_ROWS = [
         strictAutoOnWhenExplicit: false,
         excludeGrammar: 'v2-globs-unions',
         writesExcludes: false,
-        uppercaseEnv: false,
     },
     {
         minVersion: '10.16.0',
@@ -64,29 +56,78 @@ const PNPM_BEHAVIOR_ROWS = [
         strictAutoOnWhenExplicit: false,
         excludeGrammar: 'v1-exact-names',
         writesExcludes: false,
-        uppercaseEnv: false,
     },
 ];
 function findBehaviorRow(pmVersion) {
     return PNPM_BEHAVIOR_ROWS.find((row) => (0, semver_1.gte)(pmVersion, row.minVersion));
 }
-// Builds a resolved cooldown window, tagging it with the surface label used in
-// messaging. windowExplicit is false only for the invisible built-in default.
-function okWindow(windowMinutes, excludes, source, windowExplicit = true) {
+// Asks pnpm for its own resolved config rather than re-deriving how pnpm layers
+// .npmrc / env / pnpm-workspace.yaml / global config across versions. Returns
+// null if pnpm cannot be run or its output is not parseable JSON, so the caller
+// defers to a real install.
+function readPnpmConfigList(root) {
+    let raw;
+    try {
+        raw = (0, child_process_1.execSync)('pnpm config list --json', {
+            cwd: root,
+            encoding: 'utf-8',
+            windowsHide: true,
+            stdio: ['ignore', 'pipe', 'ignore'],
+        });
+    }
+    catch {
+        return null;
+    }
+    try {
+        const parsed = JSON.parse(raw);
+        return parsed && typeof parsed === 'object'
+            ? parsed
+            : null;
+    }
+    catch {
+        return null;
+    }
+}
+// Extracts the cooldown keys from pnpm's reported config. pnpm reports the
+// kebab-case keys; the exclude list comes back as a JSON array (set in a yaml
+// surface) or a comma-joined string (set via .npmrc / env).
+function parseCooldownConfig(config) {
     return {
-        kind: 'ok',
-        windowMinutes,
-        excludes,
-        sourceDescription: `pnpm minimumReleaseAge (${windowMinutes} min, ${source})`,
-        windowExplicit,
+        windowMinutes: toNumber(config['minimum-release-age']) ?? undefined,
+        excludes: parseExcludeValue(config['minimum-release-age-exclude']),
+        strictExplicit: toBoolean(config['minimum-release-age-strict']),
+        ignoreMissingTimeExplicit: toBoolean(config['minimum-release-age-ignore-missing-time']),
     };
 }
+function parseExcludeValue(value) {
+    const entries = Array.isArray(value)
+        ? value.map((entry) => String(entry))
+        : typeof value === 'string'
+            ? value.split(',')
+            : [];
+    return entries
+        .map((entry) => entry.trim())
+        .filter((entry) => entry.length > 0);
+}
+function toBoolean(value) {
+    if (typeof value === 'boolean') {
+        return value;
+    }
+    if (value === 'true') {
+        return true;
+    }
+    if (value === 'false') {
+        return false;
+    }
+    return undefined;
+}
 /**
- * Reads pnpm's effective cooldown configuration once. The surface set and
- * precedence depend on the pnpm major: v10 layers npm-conf surfaces
- * (workspace yaml > npm_config env > project .npmrc > user .npmrc); v11
- * layers pnpm-native surfaces (pnpm_config / PNPM_CONFIG env >
- * pnpm-workspace.yaml > global config.yaml > built-in 1440-minute default).
+ * Reads pnpm's effective cooldown configuration once by asking pnpm itself
+ * (`pnpm config list --json`) instead of re-implementing how it layers config
+ * surfaces (a moving target across pnpm versions). The window and excludes come
+ * straight from pnpm; the strict default, exclude grammar, and loose-fallback
+ * semantics stay version-keyed because pnpm does not report those resolved
+ * values.
  */
 async function readPnpmPolicy(root, pmVersion) {
     // A pnpm major newer than the table knows can behave differently; defer to a
@@ -101,39 +142,39 @@ async function readPnpmPolicy(root, pmVersion) {
     if (!row) {
         return { outcome: 'inactive' };
     }
-    let resolution;
-    let strictExplicit;
-    let ignoreMissingTimeExplicit;
-    if (row.major === 11) {
-        const read = readV11Surfaces(root, row);
-        resolution = read.window;
-        strictExplicit = read.strictExplicit;
-        ignoreMissingTimeExplicit = read.ignoreMissingTime;
-    }
-    else {
-        resolution = readV10Surfaces(root);
-        // v10 strict is hardcoded; no surface toggles it.
-        strictExplicit = undefined;
-    }
-    if (!resolution) {
-        return { outcome: 'inactive' };
-    }
-    if (resolution.kind !== 'ok') {
-        return { outcome: 'ambiguous', reason: resolution.reason };
-    }
-    const { windowMinutes, excludes, sourceDescription, windowExplicit } = resolution;
-    if (!Number.isFinite(windowMinutes)) {
+    const config = readPnpmConfigList(root);
+    if (config === null) {
+        // Could not read pnpm's resolved config; defer to a real install rather than
+        // guess the policy.
         return {
             outcome: 'ambiguous',
-            reason: `Invalid pnpm minimumReleaseAge value: ${String(windowMinutes)}.`,
+            reason: 'Unable to read the pnpm minimum-release-age configuration.',
         };
     }
+    const { windowMinutes: explicitWindow, excludes, strictExplicit, ignoreMissingTimeExplicit, } = parseCooldownConfig(config);
+    // An explicitly configured window (from whatever surface pnpm honors) wins;
+    // otherwise v11 falls back to its built-in 1440-minute (1 day) default, while
+    // v10 has no default (no cooldown).
+    let windowMinutes;
+    let windowExplicit;
+    if (explicitWindow !== undefined) {
+        windowMinutes = explicitWindow;
+        windowExplicit = true;
+    }
+    else if (row.major === 11) {
+        windowMinutes = 1440;
+        windowExplicit = false;
+    }
+    else {
+        return { outcome: 'inactive' };
+    }
     // 0 disables; negatives push the cutoff into the future -> everything passes.
     if (windowMinutes <= 0) {
         return { outcome: 'inactive' };
     }
     const windowMs = windowMinutes * constants_1.MS_PER_MINUTE;
     const cutoffMs = Date.now() - windowMs;
+    const sourceDescription = `pnpm minimumReleaseAge (${windowMinutes} min)`;
     const strict = resolveStrict(row, strictExplicit, windowExplicit);
     // v10 always errors on missing time; v11 defaults to skip unless a surface
     // explicitly disabled it.
@@ -179,269 +220,6 @@ function resolveStrict(row, strictExplicit, windowExplicit) {
     // so it stays loose - the normal pnpm 11 state.
     return row.strictAutoOnWhenExplicit && windowExplicit;
 }
-// --- v11 surfaces -----------------------------------------------------------
-function readV11Surfaces(root, row) {
-    const env = process.env;
-    const keySet = envKeySetForMajor(row);
-    const wsRead = readYamlWindow((0, path_1.join)(root, 'pnpm-workspace.yaml'));
-    if (wsRead && wsRead.kind === 'invalid') {
-        return {
-            window: wsRead,
-            strictExplicit: undefined,
-            ignoreMissingTime: undefined,
-        };
-    }
-    const globalRead = readYamlWindow((0, path_1.join)(getConfigDir(env), 'config.yaml'));
-    if (globalRead && globalRead.kind === 'invalid') {
-        return {
-            window: globalRead,
-            strictExplicit: undefined,
-            ignoreMissingTime: undefined,
-        };
-    }
-    const wsYaml = wsRead && wsRead.kind === 'ok' ? wsRead : null;
-    const globalYaml = globalRead && globalRead.kind === 'ok' ? globalRead : null;
-    // pnpm v11 resolves each cooldown key independently across surfaces:
-    // env > pnpm-workspace.yaml > global config.yaml > built-in default.
-    const strictExplicit = readEnvBoolean(env, keySet, 'minimum_release_age_strict') ??
-        wsYaml?.strict ??
-        globalYaml?.strict;
-    const ignoreMissingTime = readEnvBoolean(env, keySet, 'minimum_release_age_ignore_missing_time') ??
-        wsYaml?.ignoreMissingTime ??
-        globalYaml?.ignoreMissingTime;
-    const envExcludes = readEnvArray(env, keySet, 'minimum_release_age_exclude');
-    if (envExcludes === 'invalid') {
-        return {
-            window: {
-                kind: 'invalid',
-                reason: 'Invalid pnpm minimumReleaseAgeExclude entry.',
-            },
-            strictExplicit: undefined,
-            ignoreMissingTime: undefined,
-        };
-    }
-    const excludes = envExcludes ?? wsYaml?.excludes ?? globalYaml?.excludes ?? [];
-    const envWindow = readEnvNumber(env, keySet, 'minimum_release_age');
-    // pnpm drops a NaN env value rather than erroring; treat 'invalid' as unset
-    // and fall through to lower surfaces.
-    if (envWindow !== undefined && envWindow !== 'invalid') {
-        return {
-            window: okWindow(envWindow, excludes, 'env'),
-            strictExplicit,
-            ignoreMissingTime,
-        };
-    }
-    if (wsYaml && wsYaml.windowMinutes !== undefined) {
-        return {
-            window: okWindow(wsYaml.windowMinutes, excludes, 'pnpm-workspace.yaml'),
-            strictExplicit,
-            ignoreMissingTime,
-        };
-    }
-    if (globalYaml && globalYaml.windowMinutes !== undefined) {
-        return {
-            window: okWindow(globalYaml.windowMinutes, excludes, 'global config.yaml'),
-            strictExplicit,
-            ignoreMissingTime,
-        };
-    }
-    // Built-in 1440-minute (1 day) default, injected programmatically and
-    // invisible to `pnpm config get`. Loose unless some surface set strict.
-    return {
-        window: okWindow(1440, excludes, 'default', false),
-        strictExplicit,
-        ignoreMissingTime,
-    };
-}
-// --- v10 surfaces -----------------------------------------------------------
-function readV10Surfaces(root) {
-    // v10 layers config via @pnpm/npm-conf, per key:
-    // workspace yaml > npm_config_* env > project .npmrc > user .npmrc.
-    const wsRead = readYamlWindow((0, path_1.join)(root, 'pnpm-workspace.yaml'));
-    if (wsRead && wsRead.kind === 'invalid') {
-        return wsRead;
-    }
-    const wsYaml = wsRead && wsRead.kind === 'ok' ? wsRead : null;
-    const env = process.env;
-    const keySet = { prefix: 'npm_config_', uppercase: false };
-    const rawEnvWindow = readEnvNumber(env, keySet, 'minimum_release_age');
-    // npm-conf drops values that fail the Number type; treat as unset.
-    const envWindow = rawEnvWindow === 'invalid' ? undefined : rawEnvWindow;
-    const projectNpmrc = readNpmrcSurface((0, path_1.join)(root, '.npmrc'));
-    const userNpmrc = readNpmrcSurface((0, path_1.join)((0, os_1.homedir)(), '.npmrc'));
-    // v10 reads npm_config_*, which never JSON-parses, so readEnvArray can only
-    // yield a single-entry array or undefined here - never 'invalid'.
-    const envExcludes = readEnvArray(env, keySet, 'minimum_release_age_exclude');
-    const excludes = wsYaml?.excludes ??
-        (envExcludes === 'invalid' ? undefined : envExcludes) ??
-        projectNpmrc?.excludes ??
-        userNpmrc?.excludes ??
-        [];
-    if (wsYaml && wsYaml.windowMinutes !== undefined) {
-        return okWindow(wsYaml.windowMinutes, excludes, 'pnpm-workspace.yaml');
-    }
-    if (envWindow !== undefined) {
-        return okWindow(envWindow, excludes, 'env');
-    }
-    for (const npmrc of [projectNpmrc, userNpmrc]) {
-        if (npmrc && npmrc.windowMinutes !== undefined) {
-            return okWindow(npmrc.windowMinutes, excludes, '.npmrc');
-        }
-    }
-    return null;
-}
-function readYamlRaw(path) {
-    if (!(0, fs_1.existsSync)(path)) {
-        return null;
-    }
-    try {
-        const { load } = require('@zkochan/js-yaml');
-        return load((0, fs_1.readFileSync)(path, 'utf-8')) ?? {};
-    }
-    catch {
-        // The file exists but is unparseable. An absent file falls through to lower
-        // surfaces; a corrupt one can't be reasoned about, so signal it and let the
-        // caller defer to a real install (matching npm/yarn/bun on a read failure).
-        return 'invalid';
-    }
-}
-function readYamlWindow(path) {
-    const doc = readYamlRaw(path);
-    if (doc === 'invalid') {
-        return { kind: 'invalid', reason: `Unable to parse ${(0, path_1.basename)(path)}.` };
-    }
-    if (!doc) {
-        return null;
-    }
-    // Each key is read independently; pnpm merges surfaces per key.
-    const excludes = readArrayKey(doc, 'minimumReleaseAgeExclude');
-    const strict = readBooleanKey(doc, 'minimumReleaseAgeStrict');
-    const ignoreMissingTime = readBooleanKey(doc, 'minimumReleaseAgeIgnoreMissingTime');
-    const raw = doc['minimumReleaseAge'];
-    if (raw === undefined || raw === null) {
-        return { kind: 'ok', excludes, strict, ignoreMissingTime };
-    }
-    const num = toNumber(raw);
-    if (num === null) {
-        return {
-            kind: 'invalid',
-            reason: `Invalid pnpm minimumReleaseAge value: ${String(raw)}.`,
-        };
-    }
-    return {
-        kind: 'ok',
-        windowMinutes: num,
-        excludes,
-        strict,
-        ignoreMissingTime,
-    };
-}
-function readNpmrcSurface(path) {
-    const entries = (0, npmrc_1.readNpmrcEntries)(path);
-    if (entries === null) {
-        return null;
-    }
-    // v10 reads the kebab-case keys via npm-conf; camelCase in .npmrc is never
-    // honored. Only the two cooldown keys are relevant here.
-    let windowMinutes;
-    let excludes;
-    for (const { key, value: rawValue } of entries) {
-        const value = stripQuotes(rawValue);
-        if (key === 'minimum-release-age') {
-            const num = toNumber(value);
-            if (num !== null) {
-                windowMinutes = num;
-            }
-        }
-        else if (key === 'minimum-release-age-exclude') {
-            // npm-conf accumulates repeated ini keys into an array.
-            (excludes ??= []).push(value);
-        }
-    }
-    return { windowMinutes, excludes };
-}
-// pnpm mirrors getConfigDir: XDG_CONFIG_HOME, else per-platform default.
-function getConfigDir(env) {
-    if (env.XDG_CONFIG_HOME) {
-        return (0, path_1.join)(env.XDG_CONFIG_HOME, 'pnpm');
-    }
-    if (process.platform === 'darwin') {
-        return (0, path_1.join)((0, os_1.homedir)(), 'Library/Preferences/pnpm');
-    }
-    if (process.platform !== 'win32') {
-        return (0, path_1.join)((0, os_1.homedir)(), '.config/pnpm');
-    }
-    if (env.LOCALAPPDATA) {
-        return (0, path_1.join)(env.LOCALAPPDATA, 'pnpm/config');
-    }
-    return (0, path_1.join)((0, os_1.homedir)(), '.config/pnpm');
-}
-function envKeySetForMajor(row) {
-    return row.major === 11
-        ? { prefix: 'pnpm_config_', uppercase: row.uppercaseEnv }
-        : { prefix: 'npm_config_', uppercase: false };
-}
-function envKeys(keySet, suffix) {
-    const keys = [`${keySet.prefix}${suffix}`];
-    if (keySet.uppercase) {
-        keys.push(`${keySet.prefix.toUpperCase()}${suffix.toUpperCase()}`);
-    }
-    return keys;
-}
-function readEnvRaw(env, keySet, suffix) {
-    for (const key of envKeys(keySet, suffix)) {
-        if (env[key] !== undefined) {
-            return env[key];
-        }
-    }
-    return undefined;
-}
-function readEnvNumber(env, keySet, suffix) {
-    const raw = readEnvRaw(env, keySet, suffix);
-    if (raw === undefined) {
-        return undefined;
-    }
-    const num = toNumber(raw);
-    // pnpm parses env values via Number(); NaN drops the key.
-    return num === null ? 'invalid' : num;
-}
-function readEnvBoolean(env, keySet, suffix) {
-    const raw = readEnvRaw(env, keySet, suffix);
-    // pnpm's Boolean env schema (config/reader/src/env.ts) yields only true/false
-    // for the literals; anything else drops the key, so the value falls through to
-    // lower surfaces / defaults instead of coercing to false.
-    if (raw === 'true') {
-        return true;
-    }
-    if (raw === 'false') {
-        return false;
-    }
-    return undefined;
-}
-function readEnvArray(env, keySet, suffix) {
-    const raw = readEnvRaw(env, keySet, suffix);
-    if (raw === undefined) {
-        return undefined;
-    }
-    // pnpm v11's [String, Array] env schema tries a JSON array first, then falls
-    // back to the raw value as a single entry. v10 (nopt) never JSON-parses.
-    if (keySet.prefix === 'pnpm_config_') {
-        try {
-            const parsed = JSON.parse(raw);
-            if (Array.isArray(parsed)) {
-                // pnpm passes the array verbatim, then errors at install
-                // (ERR_PNPM_INVALID_MINIMUM_RELEASE_AGE_EXCLUDE) on any non-string
-                // element; signal invalid so the caller defers to a real install.
-                return parsed.every((e) => typeof e === 'string')
-                    ? parsed
-                    : 'invalid';
-            }
-        }
-        catch { }
-    }
-    return [raw];
-}
-// --- value helpers ----------------------------------------------------------
 function toNumber(value) {
     if (typeof value === 'number') {
         return Number.isFinite(value) ? value : null;
@@ -456,36 +234,6 @@ function toNumber(value) {
     }
     return null;
 }
-function stripQuotes(value) {
-    if ((value.startsWith('"') && value.endsWith('"')) ||
-        (value.startsWith("'") && value.endsWith("'"))) {
-        return value.slice(1, -1);
-    }
-    return value;
-}
-function readArrayKey(doc, key) {
-    const value = doc[key];
-    if (Array.isArray(value)) {
-        return value.map((v) => String(v));
-    }
-    if (typeof value === 'string') {
-        return [value];
-    }
-    return undefined;
-}
-function readBooleanKey(doc, key) {
-    const value = doc[key];
-    if (typeof value === 'boolean') {
-        return value;
-    }
-    if (value === 'true') {
-        return true;
-    }
-    if (value === 'false') {
-        return false;
-    }
-    return undefined;
-}
 /**
  * Builds the exclude predicate matching pnpm's version-policy semantics.
  * Returns null when any entry is invalid (pnpm would error on it), so the

package/dist/src/utils/package-json.d.ts

@@ -63,6 +63,7 @@ export interface PackageJson {
         ignoredOptionalDependencies?: string[];
     };
     overrides?: PackageOverride;
+    allowScripts?: Record<string, boolean>;
     bin?: Record<string, string> | string;
     workspaces?: string[] | {
         packages: string[];

package/dist/src/utils/package-manager.d.ts

@@ -78,9 +78,12 @@ export declare function modifyYarnRcToFitNewDirectory(contents: string): string;
 /**
  * We copy pnpm-workspace.yaml to the temporary directory so the workspace's
  * registry, auth and release-age settings still apply, and so `pnpm add -w`
- * recognizes the directory as a workspace root. `packages` (member globs) and
- * `patchedDependencies` (relative patch paths) only resolve in the real
- * workspace, so they are dropped.
+ * recognizes the directory as a workspace root. `patchedDependencies` (relative
+ * patch paths) only resolves in the real workspace, so it is dropped. The
+ * `packages` field is always set to a self-reference (`['.']`) - the temp dir is
+ * a single-package workspace - whether or not the source had one, since pnpm
+ * <10.5 (and corepack's bundled default pnpm) reject a workspace manifest whose
+ * `packages` field is missing or empty.
  *
  * Exported for testing - not meant to be used outside of this file.
  *

package/dist/src/utils/package-manager.js

@@ -326,9 +326,12 @@ function modifyYarnRcToFitNewDirectory(contents) {
 /**
  * We copy pnpm-workspace.yaml to the temporary directory so the workspace's
  * registry, auth and release-age settings still apply, and so `pnpm add -w`
- * recognizes the directory as a workspace root. `packages` (member globs) and
- * `patchedDependencies` (relative patch paths) only resolve in the real
- * workspace, so they are dropped.
+ * recognizes the directory as a workspace root. `patchedDependencies` (relative
+ * patch paths) only resolves in the real workspace, so it is dropped. The
+ * `packages` field is always set to a self-reference (`['.']`) - the temp dir is
+ * a single-package workspace - whether or not the source had one, since pnpm
+ * <10.5 (and corepack's bundled default pnpm) reject a workspace manifest whose
+ * `packages` field is missing or empty.
  *
  * Exported for testing - not meant to be used outside of this file.
  *
@@ -337,10 +340,11 @@ function modifyYarnRcToFitNewDirectory(contents) {
  */
 function modifyPnpmWorkspaceYamlToFitNewDirectory(contents) {
     const doc = (0, yaml_1.parseDocument)(contents);
-    if (doc.contents) {
-        doc.delete('packages');
-        doc.delete('patchedDependencies');
-    }
+    // Set unconditionally so an empty/comments-only source (null doc.contents)
+    // still gets a packages field; doc.set creates the root map.
+    doc.set('packages', ['.']);
+    // Relative patch paths don't resolve in the temp dir.
+    doc.delete('patchedDependencies');
     return doc.toString();
 }
 function copyPackageManagerConfigurationFiles(root, destination) {
@@ -499,8 +503,15 @@ options) {
     // Quote the spec so range operators (e.g. `>=0.0.0`) are not parsed as shell
     // redirections.
     const spec = version ? `${pkg}@${version}` : pkg;
+    // npm enforces `devEngines.packageManager` on every command, even a read-only
+    // `view`; in a yarn/bun workspace we run npm, so a non-npm pin with
+    // `onFail: error` aborts the lookup. force downgrades that to a warning.
+    // Scoped to npm so a `pnpm view` spawn is left untouched.
     const { stdout } = await execAsync(`${pm} view "${spec}" ${args}`, {
         windowsHide: true,
+        ...(pm === 'npm'
+            ? { env: { ...process.env, npm_config_force: 'true' } }
+            : {}),
     });
     return stdout.toString().trim();
 }
@@ -516,6 +527,9 @@ async function packageRegistryPack(cwd, pkg, version) {
     const { stdout } = await execAsync(`${pm} pack ${pkg}@${version}`, {
         cwd,
         windowsHide: true,
+        // npm enforces `devEngines.packageManager` even on `pack`; force keeps the
+        // download working in workspaces that pin a non-npm manager (onFail: error).
+        env: { ...process.env, npm_config_force: 'true' },
     });
     const tarballPath = stdout.trim();
     return { tarballPath };

package/dist/src/utils/spinner.d.ts

@@ -1,7 +1,19 @@
 export declare const SHOULD_SHOW_SPINNERS: boolean;
+export interface StartSpinnerOptions {
+    /**
+     * When `true`, the text passed to `start`, `succeed`, and `fail` is NOT
+     * emitted in non-TTY environments. By default (`false`), the text is logged
+     * via `console.warn` so progress information isn't lost in non-interactive
+     * environments. Set to `true` when completion is reported through a
+     * different mechanism (e.g. a batched logger).
+     *
+     * Defaults to `false`.
+     */
+    skipNonTtyLogging?: boolean;
+}
 declare class SpinnerManager {
     #private;
-    start(text?: string, prefix?: string): SpinnerManager;
+    start(text?: string, prefix?: string, opts?: StartSpinnerOptions): SpinnerManager;
     succeed(text?: string): void;
     stop(): void;
     fail(text?: string): void;

package/dist/src/utils/spinner.js

@@ -1,5 +1,5 @@
 "use strict";
-var _SpinnerManager_instances, _SpinnerManager_ora, _SpinnerManager_prefix, _SpinnerManager_createOra;
+var _SpinnerManager_instances, _SpinnerManager_ora, _SpinnerManager_prefix, _SpinnerManager_skipNonTtyLogging, _SpinnerManager_createOra, _SpinnerManager_handleNonTty;
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.globalSpinner = exports.SHOULD_SHOW_SPINNERS = void 0;
 const tslib_1 = require("tslib");
@@ -11,9 +11,11 @@ class SpinnerManager {
         _SpinnerManager_instances.add(this);
         _SpinnerManager_ora.set(this, void 0);
         _SpinnerManager_prefix.set(this, void 0);
+        _SpinnerManager_skipNonTtyLogging.set(this, false);
     }
-    start(text, prefix) {
-        if (!exports.SHOULD_SHOW_SPINNERS) {
+    start(text, prefix, opts) {
+        tslib_1.__classPrivateFieldSet(this, _SpinnerManager_skipNonTtyLogging, opts?.skipNonTtyLogging ?? false, "f");
+        if (tslib_1.__classPrivateFieldGet(this, _SpinnerManager_instances, "m", _SpinnerManager_handleNonTty).call(this, text)) {
             return this;
         }
         if (prefix !== undefined) {
@@ -30,12 +32,18 @@ class SpinnerManager {
         return this;
     }
     succeed(text) {
+        if (tslib_1.__classPrivateFieldGet(this, _SpinnerManager_instances, "m", _SpinnerManager_handleNonTty).call(this, text)) {
+            return;
+        }
         tslib_1.__classPrivateFieldGet(this, _SpinnerManager_ora, "f")?.succeed(text);
     }
     stop() {
         tslib_1.__classPrivateFieldGet(this, _SpinnerManager_ora, "f")?.stop();
     }
     fail(text) {
+        if (tslib_1.__classPrivateFieldGet(this, _SpinnerManager_instances, "m", _SpinnerManager_handleNonTty).call(this, text)) {
+            return;
+        }
         tslib_1.__classPrivateFieldGet(this, _SpinnerManager_ora, "f")?.fail(text);
     }
     updateText(text) {
@@ -50,12 +58,20 @@ class SpinnerManager {
         return tslib_1.__classPrivateFieldGet(this, _SpinnerManager_ora, "f")?.isSpinning ?? false;
     }
 }
-_SpinnerManager_ora = new WeakMap(), _SpinnerManager_prefix = new WeakMap(), _SpinnerManager_instances = new WeakSet(), _SpinnerManager_createOra = function _SpinnerManager_createOra(text) {
+_SpinnerManager_ora = new WeakMap(), _SpinnerManager_prefix = new WeakMap(), _SpinnerManager_skipNonTtyLogging = new WeakMap(), _SpinnerManager_instances = new WeakSet(), _SpinnerManager_createOra = function _SpinnerManager_createOra(text) {
     tslib_1.__classPrivateFieldSet(this, _SpinnerManager_ora, (0, ora_1.default)({
         text: text,
         prefixText: tslib_1.__classPrivateFieldGet(this, _SpinnerManager_prefix, "f"),
         hideCursor: false,
         discardStdin: false,
     }), "f");
+}, _SpinnerManager_handleNonTty = function _SpinnerManager_handleNonTty(text) {
+    if (exports.SHOULD_SHOW_SPINNERS) {
+        return false;
+    }
+    if (!tslib_1.__classPrivateFieldGet(this, _SpinnerManager_skipNonTtyLogging, "f") && text) {
+        console.warn(text);
+    }
+    return true;
 };
 exports.globalSpinner = new SpinnerManager();