@pnpm/exec.commands 1100.0.9 → 1100.1.0

package/lib/create.d.ts

@@ -1,6 +1,7 @@
+import type { CommandHandlerMap } from '@pnpm/cli.command';
 import * as dlx from './dlx.js';
 export declare const commandNames: string[];
-export declare function handler(_opts: dlx.DlxCommandOptions, params: string[]): Promise<{
+export declare function handler(_opts: dlx.DlxCommandOptions, params: string[], commands?: CommandHandlerMap): Promise<{
     exitCode: number;
 } | string>;
 export declare function rcOptionsTypes(): Record<string, unknown>;

package/lib/create.js

@@ -3,7 +3,7 @@ import { PnpmError } from '@pnpm/error';
 import { renderHelp } from 'render-help';
 import * as dlx from './dlx.js';
 export const commandNames = ['create'];
-export async function handler(_opts, params) {
+export async function handler(_opts, params, commands) {
     // If the first argument is --help or -h, we show the help message.
     if (params[0] === '--help' || params[0] === '-h') {
         return help();
@@ -15,7 +15,7 @@ export async function handler(_opts, params) {
             'with <name> substituted for a package name.');
     }
     const createPackageName = convertToCreateName(packageName);
-    return dlx.handler(_opts, [createPackageName, ...packageArgs]);
+    return dlx.handler(_opts, [createPackageName, ...packageArgs], commands);
 }
 export function rcOptionsTypes() {
     return {};

package/lib/dlx.d.ts

@@ -1,3 +1,4 @@
+import type { CommandHandlerMap } from '@pnpm/cli.command';
 import { type Config } from '@pnpm/config.reader';
 import { add } from '@pnpm/installing.commands';
 import type { PnpmSettings, SupportedArchitectures } from '@pnpm/types';
@@ -12,7 +13,7 @@ export type DlxCommandOptions = {
     shellMode?: boolean;
     allowBuild?: string[];
 } & Pick<Config, 'extraBinPaths' | 'minimumReleaseAgeExclude' | 'registries' | 'reporter' | 'userAgent' | 'cacheDir' | 'dlxCacheMaxAge' | 'symlink'> & Omit<add.AddCommandOptions, 'rootProjectManifestDir'> & PnpmSettings;
-export declare function handler(opts: DlxCommandOptions, [command, ...args]: string[]): Promise<{
+export declare function handler(opts: DlxCommandOptions, [command, ...args]: string[], commands?: CommandHandlerMap): Promise<{
     exitCode: number;
     output?: string;
 }>;

package/lib/dlx.js

@@ -2,6 +2,7 @@ import fs, {} from 'node:fs';
 import path from 'node:path';
 import util from 'node:util';
 import { getBinsFromPackageManifest } from '@pnpm/bins.resolver';
+import { getAutomaticallyIgnoredBuilds } from '@pnpm/building.commands';
 import { resolveFromCatalog, } from '@pnpm/catalogs.resolver';
 import { OUTPUT_OPTIONS } from '@pnpm/cli.common-cli-options-help';
 import { docsUrl, readProjectManifestOnly } from '@pnpm/cli.utils';
@@ -21,6 +22,13 @@ import { symlinkDir } from 'symlink-dir';
 import { makeEnv } from './makeEnv.js';
 export const skipPackageManagerCheck = true;
 export const commandNames = ['dlx'];
+/**
+ * Test-only env var. When set, the dlx ignored-builds recovery path bypasses
+ * the TTY check and forwards `all: true` so `approve-builds` skips its
+ * multiselect and confirm prompts. Mirrors the same env var honored by
+ * `promptApproveGlobalBuilds` for global installs. Not for production use.
+ */
+const AUTO_APPROVE_FOR_TESTS_ENV = 'PNPM_AUTO_APPROVE_BUILDS_FOR_TESTS';
 export const shorthands = {
     c: '--shell-mode',
 };
@@ -67,7 +75,7 @@ export function help() {
         usages: ['pnpm dlx <command> [args...]'],
     });
 }
-export async function handler(opts, [command, ...args]) {
+export async function handler(opts, [command, ...args], commands) {
     if (!command && (!opts.package || opts.package.length === 0)) {
         return { exitCode: 1, output: help() };
     }
@@ -122,15 +130,22 @@ export async function handler(opts, [command, ...args]) {
         supportedArchitectures: opts.supportedArchitectures,
     });
     if (!cacheExists) {
+        const allowBuilds = Object.fromEntries([...resolvedPkgAliases, ...(opts.allowBuild ?? [])].map(pkg => [pkg, true]));
         try {
             fs.mkdirSync(cachedDir, { recursive: true });
             await add.handler({
                 ...opts,
+                // Mirror the global install flow: dlx prompts via `approve-builds`
+                // when transitive deps have skipped build scripts, so it must not let
+                // strictDepBuilds (the v11 default) turn that into a hard error.
+                // Without this, `pnpm dlx <pkg>` cannot launch packages whose bin
+                // depends on a postinstall step (e.g. native modules).
+                strictDepBuilds: false,
                 enableGlobalVirtualStore: opts.enableGlobalVirtualStore ?? true,
                 bin: path.join(cachedDir, 'node_modules/.bin'),
                 dir: cachedDir,
                 lockfileDir: cachedDir,
-                allowBuilds: Object.fromEntries([...resolvedPkgAliases, ...(opts.allowBuild ?? [])].map(pkg => [pkg, true])),
+                allowBuilds,
                 rootProjectManifestDir: cachedDir,
                 saveProd: true, // dlx will be looking for the package in the "dependencies" field!
                 saveDev: false,
@@ -139,6 +154,7 @@ export async function handler(opts, [command, ...args]) {
                 symlink: true,
                 workspaceDir: undefined,
             }, resolvedPkgs);
+            await promptApproveDlxBuilds({ cachedDir, allowBuilds, inheritedOpts: opts }, commands);
             try {
                 await symlinkDir(cachedDir, cacheLink, { overwrite: true });
             }
@@ -159,10 +175,15 @@ export async function handler(opts, [command, ...args]) {
             // directory swaps).  If another process completed the cache in the meantime,
             // use that instead of failing.
             const completedDir = getValidCacheDir(cacheLink, opts.dlxCacheMaxAge);
-            if (completedDir == null) {
+            if (completedDir != null) {
+                cachedDir = completedDir;
+            }
+            else {
+                // Drop the partially-populated cache so a subsequent dlx run starts
+                // clean instead of reusing a broken install.
+                await fs.promises.rm(cachedDir, { recursive: true, force: true });
                 throw err;
             }
-            cachedDir = completedDir;
         }
     }
     const binsDir = path.join(cachedDir, 'node_modules/.bin');
@@ -227,6 +248,47 @@ function scopeless(pkgName) {
     }
     return pkgName;
 }
+/**
+ * After a dlx install with `strictDepBuilds: false`, check whether any
+ * transitive dependencies had their build scripts skipped and, if so, run
+ * the same `approve-builds` flow that `pnpm add -g` uses. Mirrors
+ * `promptApproveGlobalBuilds` in @pnpm/global.commands.
+ *
+ * In non-interactive mode (no TTY, no commands map) this is a no-op and
+ * the install is persisted to the dlx cache with builds skipped — same
+ * behavior as `pnpm add -g` in CI. Users who need the skipped scripts
+ * to run can re-invoke dlx with `--allow-build=<pkg>`, which produces a
+ * different cache key and forces a fresh install.
+ */
+async function promptApproveDlxBuilds(opts, commands) {
+    if (!commands?.['approve-builds'])
+        return;
+    const autoApproveForTests = process.env[AUTO_APPROVE_FOR_TESTS_ENV] === '1';
+    if (!autoApproveForTests && !process.stdin.isTTY)
+        return;
+    const { automaticallyIgnoredBuilds } = await getAutomaticallyIgnoredBuilds({
+        dir: opts.cachedDir,
+        lockfileDir: opts.cachedDir,
+    });
+    if (!automaticallyIgnoredBuilds?.length)
+        return;
+    await commands['approve-builds']({
+        ...opts.inheritedOpts,
+        dir: opts.cachedDir,
+        lockfileDir: opts.cachedDir,
+        rootProjectManifestDir: opts.cachedDir,
+        modulesDir: undefined,
+        workspaceDir: undefined,
+        allProjects: undefined,
+        selectedProjectsGraph: undefined,
+        workspacePackagePatterns: undefined,
+        rootProjectManifest: undefined,
+        global: false,
+        pending: false,
+        allowBuilds: opts.allowBuilds,
+        all: autoApproveForTests ? true : undefined,
+    }, [], commands);
+}
 function findCache(opts) {
     const dlxCommandCacheDir = createDlxCommandCacheDir(opts);
     const cacheLink = path.join(dlxCommandCacheDir, 'pkg');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pnpm/exec.commands",
-  "version": "1100.0.9",
+  "version": "1100.1.0",
   "description": "Commands for running scripts",
   "keywords": [
     "pnpm",
@@ -39,6 +39,7 @@
     "which": "npm:@pnpm/which@^3.0.1",
     "write-json-file": "^7.0.0",
     "@pnpm/bins.resolver": "1100.0.2",
+    "@pnpm/building.commands": "1100.0.10",
     "@pnpm/catalogs.resolver": "1100.0.0",
     "@pnpm/cli.command": "1100.0.1",
     "@pnpm/cli.common-cli-options-help": "1100.0.1",
@@ -48,16 +49,16 @@
     "@pnpm/core-loggers": "1100.0.1",
     "@pnpm/crypto.hash": "1100.0.1",
     "@pnpm/deps.status": "1100.0.8",
-    "@pnpm/error": "1100.0.0",
     "@pnpm/engine.runtime.commands": "1100.0.9",
     "@pnpm/exec.lifecycle": "1100.0.5",
+    "@pnpm/error": "1100.0.0",
     "@pnpm/exec.pnpm-cli-runner": "1100.0.0",
     "@pnpm/installing.client": "1100.0.8",
-    "@pnpm/installing.commands": "1100.1.7",
+    "@pnpm/installing.commands": "1100.1.8",
     "@pnpm/pkg-manifest.reader": "1100.0.2",
     "@pnpm/resolving.parse-wanted-dependency": "1100.0.1",
-    "@pnpm/shell.path": "1100.0.1",
     "@pnpm/store.path": "1100.0.1",
+    "@pnpm/shell.path": "1100.0.1",
     "@pnpm/types": "1101.0.0",
     "@pnpm/workspace.injected-deps-syncer": "1100.0.6",
     "@pnpm/workspace.project-manifest-reader": "1100.0.3",
@@ -74,13 +75,13 @@
     "@types/which": "^3.0.4",
     "is-windows": "^1.0.2",
     "write-yaml-file": "^6.0.0",
+    "@pnpm/exec.commands": "1100.1.0",
     "@pnpm/engine.runtime.system-node-version": "1100.0.2",
-    "@pnpm/exec.commands": "1100.0.9",
-    "@pnpm/logger": "1100.0.0",
     "@pnpm/prepare": "1100.0.4",
     "@pnpm/test-ipc-server": "1100.0.0",
     "@pnpm/testing.command-defaults": "1100.0.1",
-    "@pnpm/workspace.projects-filter": "1100.0.7"
+    "@pnpm/workspace.projects-filter": "1100.0.7",
+    "@pnpm/logger": "1100.0.0"
   },
   "engines": {
     "node": ">=22.13"