nx 19.0.0-canary.20240420-9ca53b7 → 19.0.0-canary.20240423-b37bfdb

package/src/executors/run-commands/run-commands.impl.d.ts

@@ -22,6 +22,7 @@ export interface RunCommandsOptions extends Json {
     readyWhen?: string;
     cwd?: string;
     env?: Record<string, string>;
+    forwardAllArgs?: boolean;
     args?: string | string[];
     envFile?: string;
     __unparsed__: string[];

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

@@ -39,6 +39,7 @@ const propKeys = [
     'usePty',
     'streamOutput',
     'verbose',
+    'forwardAllArgs',
 ];
 async function default_1(options, context) {
     registerProcessListener();
@@ -135,7 +136,7 @@ function normalizeOptions(options) {
     options.parsedArgs = parseArgs(unparsedCommandArgs, options.unknownOptions, options.args);
     options.unparsedCommandArgs = unparsedCommandArgs;
     options.commands.forEach((c) => {
-        c.command = interpolateArgsIntoCommand(c.command, options, c.forwardAllArgs ?? true);
+        c.command = interpolateArgsIntoCommand(c.command, options, c.forwardAllArgs ?? options.forwardAllArgs ?? true);
     });
     return options;
 }

package/src/executors/run-commands/schema.json

@@ -135,6 +135,11 @@
         "$source": "unparsed"
       },
       "x-priority": "internal"
+    },
+    "forwardAllArgs": {
+      "type": "boolean",
+      "description": "Whether arguments should be forwarded when interpolation is not present.",
+      "default": true
     }
   },
   "additionalProperties": true,

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

@@ -388,7 +388,7 @@ function mergeTargetConfigurations(target, baseTarget, projectConfigSourceMap, s
     const { configurations: defaultConfigurations, options: defaultOptions, ...baseTargetProperties } = baseTarget ?? {};
     // 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);
+    const isCompatible = isCompatibleTarget(baseTarget ?? {}, 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