nx 18.0.3 → 18.0.5

package/src/core/graph/runtime.js

@@ -1 +1 @@
-(()=>{"use strict";var e,r={},t={};function o(e){var n=t[e];if(void 0!==n)return n.exports;var i=t[e]={id:e,loaded:!1,exports:{}};return r[e].call(i.exports,i,i.exports,o),i.loaded=!0,i.exports}o.m=r,e=[],o.O=(r,t,n,i)=>{if(!t){var l=1/0;for(f=0;f<e.length;f++){for(var[t,n,i]=e[f],a=!0,u=0;u<t.length;u++)(!1&i||l>=i)&&Object.keys(o.O).every((e=>o.O[e](t[u])))?t.splice(u--,1):(a=!1,i<l&&(l=i));if(a){e.splice(f--,1);var d=n();void 0!==d&&(r=d)}}return r}i=i||0;for(var f=e.length;f>0&&e[f-1][2]>i;f--)e[f]=e[f-1];e[f]=[t,n,i]},o.n=e=>{var r=e&&e.__esModule?()=>e.default:()=>e;return o.d(r,{a:r}),r},o.d=(e,r)=>{for(var t in r)o.o(r,t)&&!o.o(e,t)&&Object.defineProperty(e,t,{enumerable:!0,get:r[t]})},o.g=function(){if("object"==typeof globalThis)return globalThis;try{return this||new Function("return this")()}catch(e){if("object"==typeof window)return window}}(),o.o=(e,r)=>Object.prototype.hasOwnProperty.call(e,r),o.r=e=>{"undefined"!=typeof Symbol&&Symbol.toStringTag&&Object.defineProperty(e,Symbol.toStringTag,{value:"Module"}),Object.defineProperty(e,"__esModule",{value:!0})},o.nmd=e=>(e.paths=[],e.children||(e.children=[]),e),(()=>{var e={2:0};o.O.j=r=>0===e[r];var r=(r,t)=>{var n,i,[l,a,u]=t,d=0;if(l.some((r=>0!==e[r]))){for(n in a)o.o(a,n)&&(o.m[n]=a[n]);if(u)var f=u(o)}for(r&&r(t);d<l.length;d++)i=l[d],o.o(e,i)&&e[i]&&e[i][0](),e[i]=0;return o.O(f)},t=self.webpackChunk=self.webpackChunk||[];t.forEach(r.bind(null,0)),t.push=r.bind(null,t.push.bind(t))})()})();
+(()=>{"use strict";var e,r={},t={};function o(e){var n=t[e];if(void 0!==n)return n.exports;var i=t[e]={id:e,loaded:!1,exports:{}};return r[e].call(i.exports,i,i.exports,o),i.loaded=!0,i.exports}o.m=r,e=[],o.O=(r,t,n,i)=>{if(!t){var l=1/0;for(f=0;f<e.length;f++){for(var[t,n,i]=e[f],a=!0,u=0;u<t.length;u++)(!1&i||l>=i)&&Object.keys(o.O).every((e=>o.O[e](t[u])))?t.splice(u--,1):(a=!1,i<l&&(l=i));if(a){e.splice(f--,1);var d=n();void 0!==d&&(r=d)}}return r}i=i||0;for(var f=e.length;f>0&&e[f-1][2]>i;f--)e[f]=e[f-1];e[f]=[t,n,i]},o.n=e=>{var r=e&&e.__esModule?()=>e.default:()=>e;return o.d(r,{a:r}),r},o.d=(e,r)=>{for(var t in r)o.o(r,t)&&!o.o(e,t)&&Object.defineProperty(e,t,{enumerable:!0,get:r[t]})},o.g=function(){if("object"==typeof globalThis)return globalThis;try{return this||new Function("return this")()}catch(e){if("object"==typeof window)return window}}(),o.o=(e,r)=>Object.prototype.hasOwnProperty.call(e,r),o.r=e=>{"undefined"!=typeof Symbol&&Symbol.toStringTag&&Object.defineProperty(e,Symbol.toStringTag,{value:"Module"}),Object.defineProperty(e,"__esModule",{value:!0})},o.nmd=e=>(e.paths=[],e.children||(e.children=[]),e),(()=>{var e={666:0};o.O.j=r=>0===e[r];var r=(r,t)=>{var n,i,[l,a,u]=t,d=0;if(l.some((r=>0!==e[r]))){for(n in a)o.o(a,n)&&(o.m[n]=a[n]);if(u)var f=u(o)}for(r&&r(t);d<l.length;d++)i=l[d],o.o(e,i)&&e[i]&&e[i][0](),e[i]=0;return o.O(f)},t=self.webpackChunk=self.webpackChunk||[];t.forEach(r.bind(null,0)),t.push=r.bind(null,t.push.bind(t))})()})();

package/src/core/graph/styles.js

@@ -1 +1 @@
-"use strict";(self.webpackChunk=self.webpackChunk||[]).push([[3],{33767:(s,e,k)=>{k.r(e)}},s=>{var e;e=33767,s(s.s=e)}]);
+"use strict";(self.webpackChunk=self.webpackChunk||[]).push([[532],{33767:()=>{}},s=>{var e;e=33767,s(s.s=e)}]);

package/src/executors/run-commands/run-commands.impl.js

@@ -65,7 +65,7 @@ async function runInParallel(options, context) {
     if (options.readyWhen) {
         const r = await Promise.race(procs);
         if (!r.result) {
-            process.stderr.write(`Warning: run-commands command "${r.command}" exited with non-zero status code`);
+            process.stderr.write(`Warning: command "${r.command}" exited with non-zero status code`);
             return false;
         }
         else {
@@ -77,7 +77,7 @@ async function runInParallel(options, context) {
         const failed = r.filter((v) => !v.result);
         if (failed.length > 0) {
             failed.forEach((f) => {
-                process.stderr.write(`Warning: run-commands command "${f.command}" exited with non-zero status code`);
+                process.stderr.write(`Warning: command "${f.command}" exited with non-zero status code`);
             });
             return false;
         }
@@ -107,7 +107,7 @@ async function runSerially(options, context) {
     for (const c of options.commands) {
         const success = await createProcess(c, undefined, options.color, calculateCwd(options.cwd, context), options.env ?? {}, false);
         if (!success) {
-            process.stderr.write(`Warning: run-commands command "${c.command}" exited with non-zero status code`);
+            process.stderr.write(`Warning: command "${c.command}" exited with non-zero status code`);
             return false;
         }
     }

package/src/migrations/update-17-0-0/rm-default-collection-npm-scope.js

@@ -3,7 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const format_changed_files_with_prettier_if_available_1 = require("../../generators/internal-utils/format-changed-files-with-prettier-if-available");
 const nx_json_1 = require("../../generators/utils/nx-json");
 const json_1 = require("../../generators/utils/json");
-const logger_1 = require("../../utils/logger");
+const output_1 = require("../../utils/output");
 const path_1 = require("../../utils/path");
 async function update(tree) {
     if (!tree.exists('nx.json')) {
@@ -37,7 +37,7 @@ function warnNpmScopeHasChanged(tree, nxJson) {
     }
     const packageJsonName = (0, json_1.readJson)(tree, 'package.json').name;
     if (newScope) {
-        logger_1.logger.warn({
+        output_1.output.warn({
             title: 'npmScope has been removed from nx.json',
             bodyLines: [
                 'This will now be read from package.json',
@@ -50,7 +50,7 @@ function warnNpmScopeHasChanged(tree, nxJson) {
     }
     else {
         // There is no scope in package.json
-        logger_1.logger.warn({
+        output_1.output.warn({
             title: 'npmScope has been removed from nx.json',
             bodyLines: [
                 'This will now be read from package.json',

package/src/plugins/js/utils/register.d.ts

@@ -21,14 +21,10 @@ export declare function registerTsProject(tsConfigPath: string): () => void;
  * @returns cleanup function
  * @deprecated This signature will be removed in Nx v19. You should pass the full path to the tsconfig in the first argument.
  */
-export declare function registerTsProject(path: string, configFilename: string, 
-/**
- * @deprecated Do not use this.. it's a 1-off exception
- */
-useForSwcEnvironmentVariable?: boolean): any;
-export declare function getSwcTranspiler(compilerOptions: CompilerOptions, tsConfigPath?: string): (...args: unknown[]) => unknown;
+export declare function registerTsProject(path: string, configFilename: string): any;
+export declare function getSwcTranspiler(compilerOptions: CompilerOptions): (...args: unknown[]) => unknown;
 export declare function getTsNodeTranspiler(compilerOptions: CompilerOptions): (...args: unknown[]) => unknown;
-export declare function getTranspiler(compilerOptions: CompilerOptions, tsConfigPath?: string): () => (...args: unknown[]) => unknown;
+export declare function getTranspiler(compilerOptions: CompilerOptions): () => (...args: unknown[]) => unknown;
 /**
  * Register ts-node or swc-node given a set of compiler options.
  *
@@ -37,7 +33,7 @@ export declare function getTranspiler(compilerOptions: CompilerOptions, tsConfig
  *
  * @returns cleanup method
  */
-export declare function registerTranspiler(compilerOptions: CompilerOptions, tsConfigPath?: string): () => void;
+export declare function registerTranspiler(compilerOptions: CompilerOptions): () => void;
 /**
  * @param tsConfigPath Adds the paths from a tsconfig file into node resolutions
  * @returns cleanup function

package/src/plugins/js/utils/register.js

@@ -3,22 +3,16 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.getTsNodeCompilerOptions = exports.registerTsConfigPaths = exports.registerTranspiler = exports.getTranspiler = exports.getTsNodeTranspiler = exports.getSwcTranspiler = exports.registerTsProject = void 0;
 const path_1 = require("path");
 const logger_1 = require("../../../utils/logger");
-const fs_1 = require("fs");
-const workspace_root_1 = require("../../../utils/workspace-root");
 const swcNodeInstalled = packageIsInstalled('@swc-node/register');
 const tsNodeInstalled = packageIsInstalled('ts-node/register');
 let ts;
 let isTsEsmLoaderRegistered = false;
-function registerTsProject(path, configFilename, 
-/**
- * @deprecated Do not use this.. it's a 1-off exception
- */
-useForSwcEnvironmentVariable) {
+function registerTsProject(path, configFilename) {
     const tsConfigPath = configFilename ? (0, path_1.join)(path, configFilename) : path;
     const compilerOptions = readCompilerOptions(tsConfigPath);
     const cleanupFunctions = [
         registerTsConfigPaths(tsConfigPath),
-        registerTranspiler(compilerOptions, useForSwcEnvironmentVariable ? tsConfigPath : undefined),
+        registerTranspiler(compilerOptions),
     ];
     // Add ESM support for `.ts` files.
     // NOTE: There is no cleanup function for this, as it's not possible to unregister the loader.
@@ -39,19 +33,10 @@ useForSwcEnvironmentVariable) {
     };
 }
 exports.registerTsProject = registerTsProject;
-function getSwcTranspiler(compilerOptions, tsConfigPath) {
+function getSwcTranspiler(compilerOptions) {
     // These are requires to prevent it from registering when it shouldn't
     const register = require('@swc-node/register/register')
         .register;
-    if (tsConfigPath) {
-        process.env.SWC_NODE_PROJECT = tsConfigPath;
-    }
-    else {
-        let rootTsConfig = (0, path_1.join)(workspace_root_1.workspaceRoot, 'tsconfig.base.json');
-        if ((0, fs_1.existsSync)(rootTsConfig)) {
-            process.env.SWC_NODE_PROJECT = rootTsConfig;
-        }
-    }
     const cleanupFn = register(compilerOptions);
     return typeof cleanupFn === 'function' ? cleanupFn : () => { };
 }
@@ -75,18 +60,19 @@ function getTsNodeTranspiler(compilerOptions) {
     };
 }
 exports.getTsNodeTranspiler = getTsNodeTranspiler;
-function getTranspiler(compilerOptions, tsConfigPath) {
+function getTranspiler(compilerOptions) {
     const preferTsNode = process.env.NX_PREFER_TS_NODE === 'true';
     if (!ts) {
         ts = require('typescript');
     }
     compilerOptions.lib = ['es2021'];
     compilerOptions.module = ts.ModuleKind.CommonJS;
+    compilerOptions.moduleResolution = ts.ModuleResolutionKind.Node10;
     compilerOptions.target = ts.ScriptTarget.ES2021;
     compilerOptions.inlineSourceMap = true;
     compilerOptions.skipLibCheck = true;
     if (swcNodeInstalled && !preferTsNode) {
-        return () => getSwcTranspiler(compilerOptions, tsConfigPath);
+        return () => getSwcTranspiler(compilerOptions);
     }
     // We can fall back on ts-node if it's available
     if (tsNodeInstalled) {
@@ -102,9 +88,9 @@ exports.getTranspiler = getTranspiler;
  *
  * @returns cleanup method
  */
-function registerTranspiler(compilerOptions, tsConfigPath) {
+function registerTranspiler(compilerOptions) {
     // Function to register transpiler that returns cleanup function
-    const transpiler = getTranspiler(compilerOptions, tsConfigPath);
+    const transpiler = getTranspiler(compilerOptions);
     if (!transpiler) {
         warnNoTranspiler();
         return () => { };

package/src/plugins/target-defaults/target-defaults-plugin.d.ts

@@ -1,6 +1,5 @@
-import { ProjectConfiguration } from '../../config/workspace-json-project-json';
+import { TargetConfiguration } from '../../config/workspace-json-project-json';
 import { NxPluginV2 } from '../../utils/nx-plugin';
-import { PackageJson } from '../../utils/package-json';
 /**
  * This symbol marks that a target provides information which should modify a target already registered
  * on the project via other plugins. If the target has not already been registered, and this symbol is true,
@@ -17,7 +16,7 @@ export declare const TargetDefaultsPlugin: NxPluginV2;
  * being scrapped. By adding enough information from the project.json / package.json,
  * we can make sure that the target after merging is compatible with the defined target.
  */
-export declare function getTargetInfo(target: string, projectJson: Pick<ProjectConfiguration, 'targets'>, packageJson: Pick<PackageJson, 'scripts' | 'nx'>): {
+export declare function getTargetInfo(target: string, projectJsonTargets: Record<string, TargetConfiguration>, packageJsonTargets: Record<string, TargetConfiguration>): {
     command: string;
     executor?: undefined;
     options?: undefined;

package/src/plugins/target-defaults/target-defaults-plugin.js

@@ -6,6 +6,7 @@ const node_fs_1 = require("node:fs");
 const node_path_1 = require("node:path");
 const fileutils_1 = require("../../utils/fileutils");
 const globs_1 = require("../../utils/globs");
+const package_json_1 = require("../../utils/package-json");
 const package_json_workspaces_1 = require("../package-json-workspaces");
 /**
  * This symbol marks that a target provides information which should modify a target already registered
@@ -37,18 +38,12 @@ exports.TargetDefaultsPlugin = {
             }
             const projectJson = readJsonOrNull((0, node_path_1.join)(ctx.workspaceRoot, root, 'project.json'));
             const packageJson = readJsonOrNull((0, node_path_1.join)(ctx.workspaceRoot, root, 'package.json'));
-            const includedScripts = packageJson?.nx?.includedScripts;
+            const packageJsonTargets = (0, package_json_1.readTargetsFromPackageJson)(packageJson);
             const projectDefinedTargets = new Set([
-                ...Object.keys(packageJson?.scripts ?? {}).filter((script) => {
-                    if (includedScripts) {
-                        return includedScripts.includes(script);
-                    }
-                    return true;
-                }),
                 ...Object.keys(projectJson?.targets ?? {}),
-                ...Object.keys(packageJson?.nx?.targets ?? {}),
+                ...(packageJson ? Object.keys(packageJsonTargets) : []),
             ]);
-            const executorToTargetMap = getExecutorToTargetMap(packageJson, projectJson);
+            const executorToTargetMap = getExecutorToTargetMap(packageJsonTargets, projectJson?.targets);
             const modifiedTargets = {};
             for (const defaultSpecifier in targetDefaults) {
                 const targetNames = executorToTargetMap.get(defaultSpecifier) ?? new Set();
@@ -59,7 +54,7 @@ exports.TargetDefaultsPlugin = {
                     if (!modifiedTargets[targetName] || targetName !== defaultSpecifier) {
                         const defaults = JSON.parse(JSON.stringify(targetDefaults[defaultSpecifier]));
                         modifiedTargets[targetName] = {
-                            ...getTargetInfo(targetName, projectJson, packageJson),
+                            ...getTargetInfo(targetName, projectJson?.targets, packageJsonTargets),
                             ...defaults,
                         };
                     }
@@ -80,17 +75,14 @@ exports.TargetDefaultsPlugin = {
         },
     ],
 };
-function getExecutorToTargetMap(packageJson, projectJson) {
+function getExecutorToTargetMap(packageJsonTargets, projectJsonTargets) {
     const executorToTargetMap = new Map();
     const targets = Object.keys({
-        ...projectJson?.targets,
-        ...packageJson?.scripts,
-        ...packageJson?.nx?.targets,
+        ...projectJsonTargets,
+        ...packageJsonTargets,
     });
     for (const target of targets) {
-        const executor = projectJson?.targets?.[target]?.executor ??
-            packageJson?.nx?.targets?.[target]?.executor ??
-            'nx:run-script';
+        const executor = getTargetExecutor(target, projectJsonTargets, packageJsonTargets);
         const targetsForExecutor = executorToTargetMap.get(executor) ?? new Set();
         targetsForExecutor.add(target);
         executorToTargetMap.set(executor, targetsForExecutor);
@@ -114,10 +106,10 @@ function readJsonOrNull(path) {
  * being scrapped. By adding enough information from the project.json / package.json,
  * we can make sure that the target after merging is compatible with the defined target.
  */
-function getTargetInfo(target, projectJson, packageJson) {
-    const projectJsonTarget = projectJson?.targets?.[target];
-    const packageJsonTarget = packageJson?.nx?.targets?.[target];
-    const executor = getTargetExecutor(target, projectJson, packageJson);
+function getTargetInfo(target, projectJsonTargets, packageJsonTargets) {
+    const projectJsonTarget = projectJsonTargets?.[target];
+    const packageJsonTarget = packageJsonTargets?.[target];
+    const executor = getTargetExecutor(target, projectJsonTargets, packageJsonTargets);
     const targetOptions = {
         ...packageJsonTarget?.options,
         ...projectJsonTarget?.options,
@@ -162,17 +154,17 @@ function getTargetInfo(target, projectJson, packageJson) {
     return {};
 }
 exports.getTargetInfo = getTargetInfo;
-function getTargetExecutor(target, projectJson, packageJson) {
-    const projectJsonTarget = projectJson?.targets?.[target];
-    const packageJsonTarget = packageJson?.nx?.targets?.[target];
-    const packageJsonScript = packageJson?.scripts?.[target];
-    if (projectJsonTarget?.command) {
-        return 'nx:run-commands';
+function getTargetExecutor(target, projectJsonTargets, packageJsonTargets) {
+    const projectJsonTargetConfiguration = projectJsonTargets?.[target];
+    const packageJsonTargetConfiguration = packageJsonTargets?.[target];
+    if (!projectJsonTargetConfiguration && packageJsonTargetConfiguration) {
+        return packageJsonTargetConfiguration?.executor ?? 'nx:run-script';
     }
-    if (!projectJsonTarget?.executor &&
-        !packageJsonTarget?.executor &&
-        packageJsonScript) {
-        return 'nx:run-script';
+    if (projectJsonTargetConfiguration?.executor) {
+        return projectJsonTargetConfiguration.executor;
+    }
+    if (projectJsonTargetConfiguration?.command) {
+        return 'nx:run-commands';
     }
-    return projectJsonTarget?.executor ?? packageJsonTarget?.executor;
+    return null;
 }

package/src/project-graph/utils/project-configuration-utils.js

@@ -108,13 +108,13 @@ skipCommandNormalization) {
                 !matchingProject.targets?.[targetName]) {
                 continue;
             }
+            const mergedTarget = mergeTargetConfigurations(skipCommandNormalization
+                ? target
+                : resolveCommandSyntacticSugar(target, project.root), matchingProject.targets?.[targetName], sourceMap, sourceInformation, `targets.${targetName}`);
             // We don't want the symbol to live on past the merge process
-            if (target?.[target_defaults_plugin_1.ONLY_MODIFIES_EXISTING_TARGET])
-                delete target?.[target_defaults_plugin_1.ONLY_MODIFIES_EXISTING_TARGET];
-            updatedProjectConfiguration.targets[targetName] =
-                mergeTargetConfigurations(skipCommandNormalization
-                    ? target
-                    : resolveCommandSyntacticSugar(target, project.root), matchingProject.targets?.[targetName], sourceMap, sourceInformation, `targets.${targetName}`);
+            if (mergedTarget?.[target_defaults_plugin_1.ONLY_MODIFIES_EXISTING_TARGET])
+                delete mergedTarget?.[target_defaults_plugin_1.ONLY_MODIFIES_EXISTING_TARGET];
+            updatedProjectConfiguration.targets[targetName] = mergedTarget;
         }
     }
     projectRootMap.set(updatedProjectConfiguration.root, updatedProjectConfiguration);
@@ -288,6 +288,13 @@ function mergeTargetConfigurations(target, baseTarget, projectConfigSourceMap, s
     // Target is "compatible", e.g. executor is defined only once or is the same
     // in both places. This means that it is likely safe to merge
     const isCompatible = isCompatibleTarget(baseTargetProperties, target);
+    // If the targets are not compatible, we would normally overwrite the old target
+    // with the new one. However, we have a special case for targets that have the
+    // ONLY_MODIFIES_EXISTING_TARGET symbol set. This prevents the merged target
+    // equaling info that should have only been used to modify the existing target.
+    if (!isCompatible && target[target_defaults_plugin_1.ONLY_MODIFIES_EXISTING_TARGET]) {
+        return baseTarget;
+    }
     if (!isCompatible && projectConfigSourceMap) {
         // if the target is not compatible, we will simply override the options
         // we have to delete old entries from the source map

package/src/tasks-runner/life-cycles/dynamic-run-many-terminal-output-life-cycle.js

@@ -9,6 +9,8 @@ const output_1 = require("../../utils/output");
 const pretty_time_1 = require("./pretty-time");
 const formatting_utils_1 = require("./formatting-utils");
 const view_logs_utils_1 = require("./view-logs-utils");
+const LEFT_PAD = `  `;
+const EXTENDED_LEFT_PAD = `    `;
 /**
  * The following function is responsible for creating a life cycle with dynamic
  * outputs, meaning previous outputs can be rewritten or modified as new outputs
@@ -74,7 +76,7 @@ async function createRunManyDynamicOutputRenderer({ projectNames, tasks, args, o
         // Create vertical breathing room for cursor position under the pinned footer
         lines.push('');
         for (const line of lines) {
-            output_1.output.overwriteLine(output_1.output.X_PADDING + line);
+            output_1.output.overwriteLine(line);
         }
         pinnedFooterNumLines = lines.length + additionalLines;
         // clear any possible text below the cursor's position
@@ -143,10 +145,10 @@ async function createRunManyDynamicOutputRenderer({ projectNames, tasks, args, o
         const runningTasks = taskRows.filter((row) => row.status === 'running');
         const remainingTasks = totalTasks - totalCompletedTasks;
         if (runningTasks.length > 0) {
-            additionalFooterRows.push(output_1.output.dim(`   ${output_1.output.colors.cyan(figures.arrowRight)}    Executing ${runningTasks.length}/${remainingTasks} remaining tasks${runningTasks.length > 1 ? ' in parallel' : ''}...`));
+            additionalFooterRows.push(output_1.output.dim(`${LEFT_PAD}${output_1.output.colors.cyan(figures.arrowRight)}    Executing ${runningTasks.length}/${remainingTasks} remaining tasks${runningTasks.length > 1 ? ' in parallel' : ''}...`));
             additionalFooterRows.push('');
             for (const runningTask of runningTasks) {
-                additionalFooterRows.push(`   ${output_1.output.dim.cyan(cli_spinners_1.dots.frames[currentFrame])}    ${output_1.output.formatCommand(runningTask.task.id)}`);
+                additionalFooterRows.push(`${LEFT_PAD}${output_1.output.dim.cyan(cli_spinners_1.dots.frames[currentFrame])}    ${output_1.output.formatCommand(runningTask.task.id)}`);
             }
             /**
              * Reduce layout thrashing by ensuring that there is a relatively consistent
@@ -170,21 +172,20 @@ async function createRunManyDynamicOutputRenderer({ projectNames, tasks, args, o
             additionalFooterRows.push('');
         }
         if (totalSuccessfulTasks > 0) {
-            additionalFooterRows.push(`   ${output_1.output.colors.green(figures.tick)}    ${totalSuccessfulTasks}${`/${totalCompletedTasks}`} succeeded ${output_1.output.dim(`[${totalCachedTasks} read from cache]`)}`);
+            additionalFooterRows.push(`${LEFT_PAD}${output_1.output.colors.green(figures.tick)}    ${totalSuccessfulTasks}${`/${totalCompletedTasks}`} succeeded ${output_1.output.dim(`[${totalCachedTasks} read from cache]`)}`);
         }
         if (totalFailedTasks > 0) {
-            additionalFooterRows.push(`   ${output_1.output.colors.red(figures.cross)}    ${totalFailedTasks}${`/${totalCompletedTasks}`} failed`);
+            additionalFooterRows.push(`${LEFT_PAD}${output_1.output.colors.red(figures.cross)}    ${totalFailedTasks}${`/${totalCompletedTasks}`} failed`);
         }
         moveCursorToStartOfPinnedFooter();
         if (additionalFooterRows.length > 1) {
             const text = `Running ${(0, formatting_utils_1.formatTargetsAndProjects)(projectNames, targets, tasks)}`;
             const taskOverridesRows = [];
             if (Object.keys(overrides).length > 0) {
-                const leftPadding = `${output_1.output.X_PADDING}       `;
                 taskOverridesRows.push('');
-                taskOverridesRows.push(`${leftPadding}${output_1.output.dim.cyan('With additional flags:')}`);
+                taskOverridesRows.push(`${EXTENDED_LEFT_PAD}${output_1.output.dim.cyan('With additional flags:')}`);
                 Object.entries(overrides)
-                    .map(([flag, value]) => output_1.output.dim.cyan((0, formatting_utils_1.formatFlags)(leftPadding, flag, value)))
+                    .map(([flag, value]) => output_1.output.dim.cyan((0, formatting_utils_1.formatFlags)(EXTENDED_LEFT_PAD, flag, value)))
                     .forEach((arg) => taskOverridesRows.push(arg));
             }
             const pinnedFooterLines = [
@@ -221,11 +222,10 @@ async function createRunManyDynamicOutputRenderer({ projectNames, tasks, args, o
             const text = `Successfully ran ${(0, formatting_utils_1.formatTargetsAndProjects)(projectNames, targets, tasks)}`;
             const taskOverridesRows = [];
             if (Object.keys(overrides).length > 0) {
-                const leftPadding = `${output_1.output.X_PADDING}       `;
                 taskOverridesRows.push('');
-                taskOverridesRows.push(`${leftPadding}${output_1.output.dim.green('With additional flags:')}`);
+                taskOverridesRows.push(`${EXTENDED_LEFT_PAD}${output_1.output.dim.green('With additional flags:')}`);
                 Object.entries(overrides)
-                    .map(([flag, value]) => output_1.output.dim.green((0, formatting_utils_1.formatFlags)(leftPadding, flag, value)))
+                    .map(([flag, value]) => output_1.output.dim.green((0, formatting_utils_1.formatFlags)(EXTENDED_LEFT_PAD, flag, value)))
                     .forEach((arg) => taskOverridesRows.push(arg));
             }
             const pinnedFooterLines = [
@@ -233,7 +233,7 @@ async function createRunManyDynamicOutputRenderer({ projectNames, tasks, args, o
                 ...taskOverridesRows,
             ];
             if (totalCachedTasks > 0) {
-                pinnedFooterLines.push(output_1.output.dim(`${os_1.EOL}   Nx read the output from the cache instead of running the command for ${totalCachedTasks} out of ${totalTasks} tasks.`));
+                pinnedFooterLines.push(output_1.output.dim(`${os_1.EOL}Nx read the output from the cache instead of running the command for ${totalCachedTasks} out of ${totalTasks} tasks.`));
             }
             renderPinnedFooter(pinnedFooterLines, 'green');
         }
@@ -241,11 +241,10 @@ async function createRunManyDynamicOutputRenderer({ projectNames, tasks, args, o
             const text = `Ran ${(0, formatting_utils_1.formatTargetsAndProjects)(projectNames, targets, tasks)}`;
             const taskOverridesRows = [];
             if (Object.keys(overrides).length > 0) {
-                const leftPadding = `${output_1.output.X_PADDING}       `;
                 taskOverridesRows.push('');
-                taskOverridesRows.push(`${leftPadding}${output_1.output.dim.red('With additional flags:')}`);
+                taskOverridesRows.push(`${EXTENDED_LEFT_PAD}${output_1.output.dim.red('With additional flags:')}`);
                 Object.entries(overrides)
-                    .map(([flag, value]) => output_1.output.dim.red((0, formatting_utils_1.formatFlags)(leftPadding, flag, value)))
+                    .map(([flag, value]) => output_1.output.dim.red((0, formatting_utils_1.formatFlags)(EXTENDED_LEFT_PAD, flag, value)))
                     .forEach((arg) => taskOverridesRows.push(arg));
             }
             const numFailedToPrint = 5;
@@ -254,15 +253,15 @@ async function createRunManyDynamicOutputRenderer({ projectNames, tasks, args, o
                 output_1.output.applyNxPrefix('red', output_1.output.colors.red(text) + output_1.output.dim.white(` (${timeTakenText})`)),
                 ...taskOverridesRows,
                 '',
-                output_1.output.dim(`   ${output_1.output.dim(figures.tick)}    ${totalSuccessfulTasks}${`/${totalCompletedTasks}`} succeeded ${output_1.output.dim(`[${totalCachedTasks} read from cache]`)}`),
+                output_1.output.dim(`${LEFT_PAD}${output_1.output.dim(figures.tick)}    ${totalSuccessfulTasks}${`/${totalCompletedTasks}`} succeeded ${output_1.output.dim(`[${totalCachedTasks} read from cache]`)}`),
                 '',
-                `   ${output_1.output.colors.red(figures.cross)}    ${totalFailedTasks}${`/${totalCompletedTasks}`} targets failed, including the following:`,
+                `${LEFT_PAD}${output_1.output.colors.red(figures.cross)}    ${totalFailedTasks}${`/${totalCompletedTasks}`} targets failed, including the following:`,
                 `${failedTasksForPrinting
-                    .map((t) => `        ${output_1.output.colors.red('-')} ${output_1.output.formatCommand(t.toString())}`)
+                    .map((t) => `${EXTENDED_LEFT_PAD}${output_1.output.colors.red('-')} ${output_1.output.formatCommand(t.toString())}`)
                     .join('\n ')}`,
             ];
             if (failedTasks.size > numFailedToPrint) {
-                failureSummaryRows.push(output_1.output.dim(`        ...and ${failedTasks.size - numFailedToPrint} more...`));
+                failureSummaryRows.push(output_1.output.dim(`${EXTENDED_LEFT_PAD}...and ${failedTasks.size - numFailedToPrint} more...`));
             }
             failureSummaryRows.push(...(0, view_logs_utils_1.viewLogsFooterRows)(failedTasks.size));
             renderPinnedFooter(failureSummaryRows, 'red');
@@ -314,8 +313,7 @@ async function createRunManyDynamicOutputRenderer({ projectNames, tasks, args, o
 }
 exports.createRunManyDynamicOutputRenderer = createRunManyDynamicOutputRenderer;
 function writeCompletedTaskResultLine(line) {
-    const additionalXPadding = '   ';
-    output_1.output.overwriteLine(output_1.output.X_PADDING + additionalXPadding + line);
+    output_1.output.overwriteLine(LEFT_PAD + line);
 }
 /**
  * There's not much we can do in order to "neaten up" the outputs of
@@ -326,7 +324,6 @@ function writeCompletedTaskResultLine(line) {
 function writeCommandOutputBlock(commandOutput) {
     commandOutput = commandOutput || '';
     commandOutput = commandOutput.trimStart();
-    const additionalXPadding = '      ';
     const lines = commandOutput.split(os_1.EOL);
     let totalTrailingEmptyLines = 0;
     for (let i = lines.length - 1; i >= 0; i--) {
@@ -341,5 +338,5 @@ function writeCommandOutputBlock(commandOutput) {
     }
     lines.push('');
     // Indent the command output to make it look more "designed" in the context of the dynamic output
-    lines.forEach((l) => output_1.output.overwriteLine(`${output_1.output.X_PADDING}${additionalXPadding}${l}`));
+    lines.forEach((l) => output_1.output.overwriteLine(`${EXTENDED_LEFT_PAD}${l}`));
 }

package/src/tasks-runner/life-cycles/dynamic-run-one-terminal-output-life-cycle.js

@@ -9,6 +9,8 @@ const output_1 = require("../../utils/output");
 const pretty_time_1 = require("./pretty-time");
 const formatting_utils_1 = require("./formatting-utils");
 const view_logs_utils_1 = require("./view-logs-utils");
+const LEFT_PAD = `  `;
+const EXTENDED_LEFT_PAD = `    `;
 /**
  * The following function is responsible for creating a life cycle with dynamic
  * outputs, meaning previous outputs can be rewritten or modified as new outputs
@@ -57,7 +59,7 @@ async function createRunOneDynamicOutputRenderer({ initiatingProject, tasks, arg
     const moveCursorToStartOfDependentTargetLines = () => {
         readline.moveCursor(process.stdout, 0, -dependentTargetsNumLines);
     };
-    const renderLines = (lines, dividerColor = 'cyan', renderDivider = true, skipPadding = false) => {
+    const renderLines = (lines, dividerColor = 'cyan', renderDivider = true) => {
         let additionalLines = 0;
         if (renderDivider) {
             const dividerLines = output_1.output.getVerticalSeparatorLines(dividerColor);
@@ -68,7 +70,7 @@ async function createRunOneDynamicOutputRenderer({ initiatingProject, tasks, arg
             lines.push('');
         }
         for (const line of lines) {
-            output_1.output.overwriteLine((skipPadding ? '' : output_1.output.X_PADDING) + line);
+            output_1.output.overwriteLine(line);
         }
         dependentTargetsNumLines = lines.length + additionalLines;
         // clear any possible text below the cursor's position
@@ -86,7 +88,7 @@ async function createRunOneDynamicOutputRenderer({ initiatingProject, tasks, arg
         switch (state) {
             case 'EXECUTING_DEPENDENT_TARGETS':
                 if (totalFailedTasks === 0) {
-                    linesToRender.push(`   ${output_1.output.colors.cyan(cli_spinners_1.dots.frames[dependentTargetsCurrentFrame])}    ${output_1.output.dim(`Nx is waiting on ${remainingDependentTasksNotFromInitiatingProject} dependent project tasks before running tasks from`)} ${initiatingProject}${output_1.output.dim('...')}`);
+                    linesToRender.push(`${LEFT_PAD}${output_1.output.colors.cyan(cli_spinners_1.dots.frames[dependentTargetsCurrentFrame])}    ${output_1.output.dim(`Nx is waiting on ${remainingDependentTasksNotFromInitiatingProject} dependent project tasks before running tasks from`)} ${initiatingProject}${output_1.output.dim('...')}`);
                     if (totalSuccessfulTasks > 0) {
                         linesToRender.push('');
                     }
@@ -94,14 +96,14 @@ async function createRunOneDynamicOutputRenderer({ initiatingProject, tasks, arg
                 break;
         }
         if (totalFailedTasks > 0) {
-            linesToRender.push(output_1.output.colors.red.dim(`   ${output_1.output.colors.red(figures.cross)}    ${totalFailedTasks}${`/${totalCompletedTasks}`} dependent project tasks failed (see below)`));
+            linesToRender.push(output_1.output.colors.red.dim(`${LEFT_PAD}${output_1.output.colors.red(figures.cross)}    ${totalFailedTasks}${`/${totalCompletedTasks}`} dependent project tasks failed (see below)`));
         }
         if (totalSuccessfulTasks > 0) {
-            linesToRender.push(output_1.output.dim(`   ${output_1.output.dim(figures.tick)}    ${totalSuccessfulTasks}${`/${totalCompletedTasks}`} dependent project tasks succeeded ${output_1.output.dim(`[${totalCachedTasks} read from cache]`)}`));
+            linesToRender.push(output_1.output.dim(`${LEFT_PAD}${output_1.output.dim(figures.tick)}    ${totalSuccessfulTasks}${`/${totalCompletedTasks}`} dependent project tasks succeeded ${output_1.output.dim(`[${totalCachedTasks} read from cache]`)}`));
         }
         moveCursorToStartOfDependentTargetLines();
         if (linesToRender.length > 1) {
-            renderLines(linesToRender, 'gray', renderDivider && state !== 'EXECUTING_DEPENDENT_TARGETS', true);
+            renderLines(linesToRender, 'gray', renderDivider && state !== 'EXECUTING_DEPENDENT_TARGETS');
         }
         else {
             renderLines([]);
@@ -120,7 +122,7 @@ async function createRunOneDynamicOutputRenderer({ initiatingProject, tasks, arg
                 renderDependentTargets(false);
                 if (totalDependentTasksNotFromInitiatingProject > 0) {
                     output_1.output.addNewline();
-                    process.stdout.write(`   ${output_1.output.dim('Hint: you can run the command with')} --verbose ${output_1.output.dim('to see the full dependent project outputs')}` + os_1.EOL);
+                    process.stdout.write(`${LEFT_PAD}${output_1.output.dim('Hint: you can run the command with')} --verbose ${output_1.output.dim('to see the full dependent project outputs')}` + os_1.EOL);
                     output_1.output.addVerticalSeparator('gray');
                 }
             }
@@ -176,11 +178,10 @@ async function createRunOneDynamicOutputRenderer({ initiatingProject, tasks, arg
             const text = `Successfully ran ${(0, formatting_utils_1.formatTargetsAndProjects)([initiatingProject], [targetName], tasks)}`;
             const taskOverridesLines = [];
             if (Object.keys(overrides).length > 0) {
-                const leftPadding = `${output_1.output.X_PADDING}       `;
                 taskOverridesLines.push('');
-                taskOverridesLines.push(`${leftPadding}${output_1.output.dim.green('With additional flags:')}`);
+                taskOverridesLines.push(`${EXTENDED_LEFT_PAD}${output_1.output.dim.green('With additional flags:')}`);
                 Object.entries(overrides)
-                    .map(([flag, value]) => output_1.output.dim.green((0, formatting_utils_1.formatFlags)(leftPadding, flag, value)))
+                    .map(([flag, value]) => output_1.output.dim.green((0, formatting_utils_1.formatFlags)(EXTENDED_LEFT_PAD, flag, value)))
                     .forEach((arg) => taskOverridesLines.push(arg));
             }
             const pinnedFooterLines = [
@@ -188,7 +189,7 @@ async function createRunOneDynamicOutputRenderer({ initiatingProject, tasks, arg
                 ...taskOverridesLines,
             ];
             if (totalCachedTasks > 0) {
-                pinnedFooterLines.push(output_1.output.dim(`${os_1.EOL}   Nx read the output from the cache instead of running the command for ${totalCachedTasks} out of ${totalTasks} tasks.`));
+                pinnedFooterLines.push(output_1.output.dim(`${os_1.EOL}Nx read the output from the cache instead of running the command for ${totalCachedTasks} out of ${totalTasks} tasks.`));
             }
             renderLines(pinnedFooterLines, 'green');
         }
@@ -200,11 +201,10 @@ async function createRunOneDynamicOutputRenderer({ initiatingProject, tasks, arg
             }
             const taskOverridesLines = [];
             if (Object.keys(overrides).length > 0) {
-                const leftPadding = `${output_1.output.X_PADDING}       `;
                 taskOverridesLines.push('');
-                taskOverridesLines.push(`${leftPadding}${output_1.output.dim.red('With additional flags:')}`);
+                taskOverridesLines.push(`${EXTENDED_LEFT_PAD}${output_1.output.dim.red('With additional flags:')}`);
                 Object.entries(overrides)
-                    .map(([flag, value]) => output_1.output.dim.red((0, formatting_utils_1.formatFlags)(leftPadding, flag, value)))
+                    .map(([flag, value]) => output_1.output.dim.red((0, formatting_utils_1.formatFlags)(EXTENDED_LEFT_PAD, flag, value)))
                     .forEach((arg) => taskOverridesLines.push(arg));
             }
             const viewLogs = (0, view_logs_utils_1.viewLogsFooterRows)(totalFailedTasks);
@@ -212,8 +212,8 @@ async function createRunOneDynamicOutputRenderer({ initiatingProject, tasks, arg
                 output_1.output.applyNxPrefix('red', output_1.output.colors.red(text) + output_1.output.dim(` (${timeTakenText})`)),
                 ...taskOverridesLines,
                 '',
-                `   ${output_1.output.colors.red(figures.cross)}    ${totalFailedTasks}${`/${totalCompletedTasks}`} failed`,
-                `   ${output_1.output.dim(figures.tick)}    ${totalSuccessfulTasks}${`/${totalCompletedTasks}`} succeeded ${output_1.output.dim(`[${totalCachedTasks} read from cache]`)}`,
+                `${LEFT_PAD}${output_1.output.colors.red(figures.cross)}    ${totalFailedTasks}${`/${totalCompletedTasks}`} failed`,
+                `${LEFT_PAD}${output_1.output.dim(figures.tick)}    ${totalSuccessfulTasks}${`/${totalCompletedTasks}`} succeeded ${output_1.output.dim(`[${totalCachedTasks} read from cache]`)}`,
                 ...viewLogs,
             ], 'red');
         }

package/src/tasks-runner/life-cycles/static-run-many-terminal-output-life-cycle.js

@@ -27,7 +27,7 @@ class StaticRunManyTerminalOutputLifeCycle {
             output_1.output.logSingleLine(`No projects with ${(0, formatting_utils_1.formatTargetsAndProjects)(this.projectNames, this.args.targets, this.tasks)} were run`);
             return;
         }
-        const bodyLines = this.projectNames.map((affectedProject) => ` ${output_1.output.dim('-')} ${affectedProject}`);
+        const bodyLines = this.projectNames.map((affectedProject) => `${output_1.output.dim('-')} ${affectedProject}`);
         if (Object.keys(this.taskOverrides).length > 0) {
             bodyLines.push('');
             bodyLines.push(`${output_1.output.dim('With additional flags:')}`);

package/src/tasks-runner/life-cycles/static-run-one-terminal-output-life-cycle.d.ts

@@ -19,6 +19,7 @@ export declare class StaticRunOneTerminalOutputLifeCycle implements LifeCycle {
     constructor(initiatingProject: string, projectNames: string[], tasks: Task[], args: {
         targets?: string[];
         configuration?: string;
+        verbose?: boolean;
     });
     startCommand(): void;
     endCommand(): void;

package/src/tasks-runner/life-cycles/static-run-one-terminal-output-life-cycle.js

@@ -78,12 +78,20 @@ class StaticRunOneTerminalOutputLifeCycle {
         }
     }
     printTaskTerminalOutput(task, status, terminalOutput) {
-        if (status === 'success' ||
+        const args = (0, utils_1.getPrintableCommandArgsForTask)(task);
+        if (this.args.verbose ||
+            status === 'success' ||
             status === 'failure' ||
             task.target.project === this.initiatingProject) {
-            const args = (0, utils_1.getPrintableCommandArgsForTask)(task);
             output_1.output.logCommandOutput(args.join(' '), status, terminalOutput);
         }
+        else {
+            /**
+             * Do not show the terminal output in the case where it is not the initiating project and verbose is not set,
+             * but still print the command that was run and its status (so that cache hits can still be traced).
+             */
+            output_1.output.logCommandOutput(args.join(' '), status, '');
+        }
     }
 }
 exports.StaticRunOneTerminalOutputLifeCycle = StaticRunOneTerminalOutputLifeCycle;