fix(core): script-based targets should be able to be modified in a pr…

…oject.json file
nrwl · Aug 6, 2024 · df522d1 · df522d1
1 parent a3869a8
commit df522d1
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 1 deletion.
diff --git a/packages/nx/src/plugins/package-json/create-nodes.spec.ts b/packages/nx/src/plugins/package-json/create-nodes.spec.ts
@@ -570,13 +570,17 @@ describe('nx package.json workspaces plugin', () => {
             name: 'root',
             scripts: {
               build: 'echo build',
+              test: 'echo test',
             },
           }),
           'packages/a/project.json': JSON.stringify({
             targets: {
               'something-other-than-build': {
                 command: 'echo something-other-than-build',
               },
+              test: {
+                dependsOn: ['build-native'],
+              },
             },
           }),
         },
@@ -597,6 +601,7 @@ describe('nx package.json workspaces plugin', () => {
                     "targetGroups": {
                       "NPM Scripts": [
                         "build",
+                        "test",
                       ],
                     },
                   },
@@ -624,6 +629,16 @@ describe('nx package.json workspaces plugin', () => {
                       "executor": "@nx/js:release-publish",
                       "options": {},
                     },
+                    "test": {
+                      "executor": "nx:run-script",
+                      "metadata": {
+                        "runCommand": "npm run test",
+                        "scriptContent": "echo test",
+                      },
+                      "options": {
+                        "script": "test",
+                      },
+                    },
                   },
                 },
               },

diff --git a/packages/nx/src/plugins/package-json/create-nodes.ts b/packages/nx/src/plugins/package-json/create-nodes.ts
@@ -151,7 +151,13 @@ export function buildProjectConfigurationFromPackageJson(
 
   if (siblingProjectJson) {
     for (const target of Object.keys(siblingProjectJson?.targets ?? {})) {
-      delete packageJson.scripts?.[target];
+      const { executor, command, options } = siblingProjectJson.targets[target];
+      if (
+        command ||
+        (executor && executor !== 'nx:run-script' && options.script !== target)
+      ) {
+        delete packageJson.scripts?.[target];
+      }
     }
   }