fix(misc): do not add includedScripts unless really needed when runni…

…ng nx add (#22180)

packages/devkit/src/utils/update-package-scripts.spec.ts

@@ -327,6 +327,41 @@ describe('updatePackageScripts', () => {
     ]);
   });
 
+  it('should not set "nx.includedScripts" when no script matched an inferred target', async () => {
+    tree.write('vite.config.ts', '');
+    writeJson(tree, 'package.json', {
+      name: 'app1',
+      scripts: {
+        serve: 'vite',
+        coverage: 'vitest run --coverage',
+        foo: 'echo "foo"',
+      },
+    });
+
+    await updatePackageScripts(tree, [
+      '**/{vite,vitest}.config.{js,ts,mjs,mts,cjs,cts}',
+      () => ({
+        projects: {
+          app1: {
+            targets: {
+              build: { command: 'vite build' },
+              serve: { command: 'vite serve' },
+              test: { command: 'vitest run' },
+            },
+          },
+        },
+      }),
+    ]);
+
+    const packageJson = readJson<PackageJson>(tree, 'package.json');
+    expect(packageJson.scripts).toStrictEqual({
+      serve: 'vite',
+      coverage: 'nx test --coverage',
+      foo: 'echo "foo"',
+    });
+    expect(packageJson.nx).toBeUndefined();
+  });
+
   it('should exclude replaced package.json scripts from nx if they are initially included', async () => {
     tree.write('next.config.js', '');
     writeJson(tree, 'package.json', {

packages/devkit/src/utils/update-package-scripts.ts

@@ -175,20 +175,25 @@ async function processProject(
     }
   }
 
-  packageJson.nx ??= {};
   if (process.env.NX_RUNNING_NX_INIT === 'true') {
     // running `nx init` so we want to exclude everything by default
+    packageJson.nx ??= {};
     packageJson.nx.includedScripts = [];
-  } else {
+  } else if (replacedTargets.size) {
     /**
      * Running `nx add`. In this case we want to:
      * - if `includedScripts` is already set: exclude scripts that match inferred targets that were used to replace a script
      * - if `includedScripts` is not set: set `includedScripts` with all scripts except the ones that match an inferred target that was used to replace a script
      */
-    packageJson.nx.includedScripts ??= Object.keys(packageJson.scripts);
-    packageJson.nx.includedScripts = packageJson.nx.includedScripts.filter(
+    const includedScripts =
+      packageJson.nx?.includedScripts ?? Object.keys(packageJson.scripts);
+    const filteredScripts = includedScripts.filter(
       (s) => !replacedTargets.has(s)
     );
+    if (filteredScripts.length !== includedScripts.length) {
+      packageJson.nx ??= {};
+      packageJson.nx.includedScripts = filteredScripts;
+    }
   }
 
   writeJson(tree, packageJsonPath, packageJson);