fix(core): overwrite inferred script target when nx prop defines executor or command (#35227)

AgentEnder · nx-cloud[bot] · FrozenPandaz · commit 13d69ef2ed9c · 2026-04-09T17:51:23.000-04:00
## Current Behavior When a `package.json` has both a script entry and an `nx.targets` entry for the same target name, and the `nx.targets` entry uses command shorthand (`command: "tsc"`) or an explicit executor, the two targets are merged together. This produces an invalid hybrid target that has both `executor: "nx:run-script"` (from the inferred script target) and the `command` property (from the nx prop), causing the node to fail to merge into the project graph. ## Expected Behavior When the `nx.targets` entry specifies how to run (via `executor` or `command`), it should completely overwrite the inferred script target instead of merging with it. Targets without `executor` or `command` (e.g., just adding `outputs` or `dependsOn`) should continue to merge as before. ## Related Issue(s) Fixes NXC-3923 --------- Co-authored-by: nx-cloud[bot] <71083854+nx-cloud[bot]@users.noreply.github.com> (cherry picked from commit 7edc7b1)
diff --git a/packages/nx/src/utils/package-json.spec.ts b/packages/nx/src/utils/package-json.spec.ts
@@ -76,6 +76,16 @@ describe('installPackageToTmp', () => {
 });
 
 describe('readTargetsFromPackageJson', () => {
+  beforeEach(() => {
+    jest
+      .spyOn(pacakgeManager, 'getPackageManagerCommand')
+      .mockReturnValue({ run: (script) => `npm run ${script}` } as any);
+  });
+
+  afterEach(() => {
+    jest.restoreAllMocks();
+  });
+
   const packageJson: PackageJson = {
     name: 'my-app',
     version: '0.0.0',
@@ -319,6 +329,33 @@ describe('readTargetsFromPackageJson', () => {
     `);
   });
 
+  it('should override script target when nx target uses command shorthand', () => {
+    const result = readTargetsFromPackageJson(
+      {
+        name: 'my-other-app',
+        version: '',
+        scripts: {
+          build: 'echo 1',
+        },
+        nx: {
+          targets: {
+            build: {
+              command: 'echo 2',
+            },
+          },
+        },
+      },
+      {},
+      workspaceRoot,
+      '/root'
+    );
+    expect(result.build).toMatchInlineSnapshot(`
+      {
+        "command": "echo 2",
+      }
+    `);
+  });
+
   it('should override script if provided in options', () => {
     const result = readTargetsFromPackageJson(
       {
diff --git a/packages/nx/src/utils/package-json.ts b/packages/nx/src/utils/package-json.ts
@@ -228,10 +228,15 @@ export function readTargetsFromPackageJson(
     res[script] = buildTargetFromScript(script, scripts, packageManagerCommand);
   }
   for (const targetName in nx?.targets) {
-    res[targetName] = mergeTargetConfigurations(
-      nx?.targets[targetName],
-      res[targetName]
-    );
+    const nxTarget = nx.targets[targetName];
+    // If the nx target specifies how to run (via executor or command shorthand),
+    // it's incompatible with the inferred nx:run-script target from scripts,
+    // so overwrite instead of merge.
+    if (res[targetName] && (nxTarget.executor || nxTarget.command)) {
+      res[targetName] = nxTarget;
+    } else {
+      res[targetName] = mergeTargetConfigurations(nxTarget, res[targetName]);
+    }
   }
 
   /**
