@nx/dotnet 23.2.0-pr.36527.2252709 → 23.2.0-pr.36838.ef8e258

package/README.md

@@ -7,9 +7,9 @@
 
 <div style="text-align: center;">
 
-[![License](https://img.shields.io/npm/l/@nx/workspace.svg?style=flat-square)]()
+[![License](https://img.shields.io/npm/l/@nx/workspace.svg?style=flat-square)](<>)
 [![NPM Version](https://badge.fury.io/js/nx.svg)](https://www.npmjs.com/package/nx)
-[![Semantic Release](https://img.shields.io/badge/%20%20%F0%9F%93%A6%F0%9F%9A%80-semantic--release-e10079.svg?style=flat-square)]()
+[![Semantic Release](https://img.shields.io/badge/%20%20%F0%9F%93%A6%F0%9F%9A%80-semantic--release-e10079.svg?style=flat-square)](<>)
 [![Commitizen friendly](https://img.shields.io/badge/commitizen-friendly-brightgreen.svg)](http://commitizen.github.io/cz-cli/)
 [![Join us on the Official Nx Discord Server](https://img.shields.io/discord/1143497901675401286?label=discord)](https://go.nx.dev/community)
 [![Nx Sandboxing](https://staging.nx.app/workspaces/62d013ea0852fe0a2df74438/sandbox-badge.svg)](https://nx.dev/docs/features/ci-features/sandboxing)

package/dist/analyzer/analyzer-client.d.ts

@@ -1,26 +1,16 @@
 import { ProjectConfiguration } from '@nx/devkit';
+export declare const ANALYZER_CANCELLED_MESSAGE = "Analyzer run was cancelled";
+/**
+ * Cancel any in-flight analyzer process. Safe to call even if nothing is running.
+ */
+export declare function cancelPendingAnalysis(): void;
+export declare function getAnalysisTimeoutMs(): number;
 export interface AnalysisSuccessResult {
     nodesByFile: Record<string, ProjectConfiguration>;
     referencesByRoot: Record<string, {
         refs: string[];
         sourceConfigFile: string;
     }>;
-    /**
-     * Project roots whose targets were derived from C# sources rather than from
-     * project files alone — i.e. projects that produced atomized test targets.
-     *
-     * Absent (not empty) when nothing atomized, both so that output from an older
-     * analyzer still parses and so the common case is trivially detectable.
-     */
-    atomizedRoots?: string[];
-    /**
-     * Workspace-relative paths of sources that fed discovery but sit outside their
-     * project's directory — linked or shared files.
-     *
-     * `atomizedRoots` is hashed as a per-root glob, which by construction cannot
-     * reach these, so they are hashed alongside it.
-     */
-    atomizedExternalSources?: string[];
 }
 export interface AnalysisErrorResult {
     error: Error;
@@ -37,14 +27,6 @@ export interface DotNetAnalyzerOptions {
     restoreTargetName?: string;
     publishTargetName?: string;
     packTargetName?: string;
-    /**
-     * Name of the atomized parent target. Undefined disables test splitting
-     * entirely, which is the default.
-     */
-    testCiTargetName?: string;
-    /** Pre-derived group name; the analyzer does not compute one. */
-    testCiGroupName?: string;
-    testCiSplitBy?: 'class' | 'method';
 }
 /**
  * Get project analysis results for the given project files.

package/dist/analyzer/analyzer-client.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"analyzer-client.d.ts","sourceRoot":"","sources":["../../src/analyzer/analyzer-client.ts"],"names":[],"mappings":"AAIA,OAAO,EAGL,oBAAoB,EAErB,MAAM,YAAY,CAAC;AASpB,MAAM,WAAW,qBAAqB;IAEpC,WAAW,EAAE,MAAM,CAAC,MAAM,EAAE,oBAAoB,CAAC,CAAC;IAElD,gBAAgB,EAAE,MAAM,CACtB,MAAM,EACN;QAAE,IAAI,EAAE,MAAM,EAAE,CAAC;QAAC,gBAAgB,EAAE,MAAM,CAAA;KAAE,CAC7C,CAAC;IACF;;;;;;OAMG;IACH,aAAa,CAAC,EAAE,MAAM,EAAE,CAAC;IACzB;;;;;;OAMG;IACH,uBAAuB,CAAC,EAAE,MAAM,EAAE,CAAC;CACpC;AACD,MAAM,WAAW,mBAAmB;IAClC,KAAK,EAAE,KAAK,CAAC;CACd;AACD,MAAM,MAAM,cAAc,GAAG,qBAAqB,GAAG,mBAAmB,CAAC;AAEzE;;;GAGG;AACH,MAAM,WAAW,qBAAqB;IACpC,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB;;;OAGG;IACH,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,iEAAiE;IACjE,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,aAAa,CAAC,EAAE,OAAO,GAAG,QAAQ,CAAC;CACpC;AAuOD;;;;GAIG;AACH,wBAAsB,eAAe,CACnC,KAAK,EAAE,MAAM,EAAE,EACf,OAAO,CAAC,EAAE,qBAAqB,GAC9B,OAAO,CAAC,cAAc,CAAC,CA0EzB;AAED;;;;GAIG;AACH,wBAAgB,wBAAwB,IAAI,cAAc,CASzD;AAED;;GAEG;AACH,wBAAgB,UAAU,IAAI,IAAI,CAEjC;AAED,wBAAgB,qBAAqB,CACnC,MAAM,EAAE,cAAc,GACrB,MAAM,IAAI,mBAAmB,CAE/B;AAED,wBAAgB,uBAAuB,CACrC,MAAM,EAAE,cAAc,GACrB,MAAM,IAAI,qBAAqB,CAEjC"}
+{"version":3,"file":"analyzer-client.d.ts","sourceRoot":"","sources":["../../src/analyzer/analyzer-client.ts"],"names":[],"mappings":"AAEA,OAAO,EAIL,oBAAoB,EACrB,MAAM,YAAY,CAAC;AAmBpB,eAAO,MAAM,0BAA0B,+BAA+B,CAAC;AAIvE;;GAEG;AACH,wBAAgB,qBAAqB,IAAI,IAAI,CAK5C;AAED,wBAAgB,oBAAoB,IAAI,MAAM,CAS7C;AAED,MAAM,WAAW,qBAAqB;IAEpC,WAAW,EAAE,MAAM,CAAC,MAAM,EAAE,oBAAoB,CAAC,CAAC;IAElD,gBAAgB,EAAE,MAAM,CACtB,MAAM,EACN;QAAE,IAAI,EAAE,MAAM,EAAE,CAAC;QAAC,gBAAgB,EAAE,MAAM,CAAA;KAAE,CAC7C,CAAC;CACH;AACD,MAAM,WAAW,mBAAmB;IAClC,KAAK,EAAE,KAAK,CAAC;CACd;AACD,MAAM,MAAM,cAAc,GAAG,qBAAqB,GAAG,mBAAmB,CAAC;AAEzE;;;GAGG;AACH,MAAM,WAAW,qBAAqB;IACpC,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB;AA4MD;;;;GAIG;AACH,wBAAsB,eAAe,CACnC,KAAK,EAAE,MAAM,EAAE,EACf,OAAO,CAAC,EAAE,qBAAqB,GAC9B,OAAO,CAAC,cAAc,CAAC,CA+DzB;AAED;;;;GAIG;AACH,wBAAgB,wBAAwB,IAAI,cAAc,CASzD;AAED;;GAEG;AACH,wBAAgB,UAAU,IAAI,IAAI,CAEjC;AAED,wBAAgB,qBAAqB,CACnC,MAAM,EAAE,cAAc,GACrB,MAAM,IAAI,mBAAmB,CAE/B;AAED,wBAAgB,uBAAuB,CACrC,MAAM,EAAE,cAAc,GACrB,MAAM,IAAI,qBAAqB,CAEjC"}

package/dist/analyzer/analyzer-client.js

@@ -1,15 +1,44 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.ANALYZER_CANCELLED_MESSAGE = void 0;
+exports.cancelPendingAnalysis = cancelPendingAnalysis;
+exports.getAnalysisTimeoutMs = getAnalysisTimeoutMs;
 exports.analyzeProjects = analyzeProjects;
 exports.readCachedAnalysisResult = readCachedAnalysisResult;
 exports.clearCache = clearCache;
 exports.isAnalysisErrorResult = isAnalysisErrorResult;
 exports.isAnalysisSuccessResult = isAnalysisSuccessResult;
-const node_child_process_1 = require("node:child_process");
 const node_fs_1 = require("node:fs");
 const node_path_1 = require("node:path");
 const devkit_1 = require("@nx/devkit");
 const internal_1 = require("@nx/devkit/internal");
+const DEFAULT_ANALYSIS_TIMEOUT_SECONDS = (0, internal_1.isCI)() ? 600 : 120;
+// setTimeout silently clamps a delay past the 32-bit signed max to 1ms, which
+// would abort immediately — the opposite of what a large value asks for.
+const MAX_TIMEOUT_MS = 2 ** 31 - 1;
+// Thrown when a newer analysis supersedes an in-flight one. Callers swallow it
+// rather than surfacing an internal string to the user.
+exports.ANALYZER_CANCELLED_MESSAGE = 'Analyzer run was cancelled';
+let currentAbortController;
+/**
+ * Cancel any in-flight analyzer process. Safe to call even if nothing is running.
+ */
+function cancelPendingAnalysis() {
+    if (currentAbortController) {
+        currentAbortController.abort('cancelled');
+        currentAbortController = undefined;
+    }
+}
+function getAnalysisTimeoutMs() {
+    const envTimeout = process.env.NX_DOTNET_PROJECT_GRAPH_TIMEOUT;
+    if (envTimeout) {
+        const parsed = Number(envTimeout);
+        if (!Number.isNaN(parsed) && parsed > 0) {
+            return Math.min(parsed * 1000, MAX_TIMEOUT_MS);
+        }
+    }
+    return DEFAULT_ANALYSIS_TIMEOUT_SECONDS * 1000;
+}
 let cache = null;
 /**
  * Get the path to the msbuild-analyzer executable
@@ -37,64 +66,11 @@ function getAnalyzerPath() {
 async function calculateProjectFilesHash(files) {
     return await (0, internal_1.hashWithWorkspaceContext)(devkit_1.workspaceRoot, files);
 }
-/**
- * Joins a project root and a glob pattern, the way `{projectRoot}` inputs are
- * expanded for ordinary task hashing.
- */
-function rootGlob(root, pattern) {
-    const combined = `${root}/${pattern}`;
-    // Same problem Nx's own {projectRoot} substitution hits in task-hasher.ts:
-    // a workspace-root project's root is ".", and "./**" matches nothing.
-    return combined.startsWith('./') ? combined.slice(2) : combined;
-}
-/**
- * Hash the C# sources of every project that produced atomized targets.
- *
- * Atomized targets are derived from the test classes and methods declared in a
- * project's sources, but `calculateProjectFilesHash` only covers project files.
- * Without this, adding or removing a test would leave the cached analysis in
- * place and the new test would silently never get a target.
- *
- * Returns null when nothing atomized, which skips the work entirely — the point
- * being that a workspace which never opts in pays nothing for this.
- *
- * All roots go into a single glob group on purpose. The underlying native
- * implementation walks the whole file list once per group, so grouping per
- * project root would make this O(roots x files) instead of O(files).
- */
-async function computeSourceHash(result) {
-    if (!result.atomizedRoots?.length) {
-        return null;
-    }
-    const rootsHash = await (0, internal_1.hashWithWorkspaceContext)(devkit_1.workspaceRoot, result.atomizedRoots.map((root) => rootGlob(root, '**/*.cs')));
-    // Discovery reads MSBuild's Compile items, which can point outside the
-    // project. A linked test file would otherwise change the discovered units
-    // without changing this hash. Hashed by exact path rather than folded into
-    // the glob group above: these are literal paths, not patterns, and one
-    // containing a glob metacharacter (*, !, {, ...) would silently change
-    // which files the group matches.
-    const externalHashes = (result.atomizedExternalSources ?? []).map((relativePath) => (0, internal_1.hashFile)((0, node_path_1.join)(devkit_1.workspaceRoot, relativePath)) ?? '');
-    return externalHashes.length === 0
-        ? rootsHash
-        : (0, devkit_1.hashArray)([rootsHash, ...externalHashes]);
-}
-/**
- * Whether a cached analysis still reflects what is on disk.
- *
- * A null `sourceHash` means the analysis did not depend on C# sources, so the
- * project-files hash the entry is keyed by is already sufficient.
- */
-async function isSourceHashCurrent(sourceHash, result) {
-    if (sourceHash === null) {
-        return true;
-    }
-    return sourceHash === (await computeSourceHash(result));
-}
 /**
  * Run the msbuild-analyzer and return the results.
  * Uses stdin for large file lists to avoid ARG_MAX issues.
  */
-function runAnalyzer(files, options) {
+async function runAnalyzer(files, options) {
     if (files.length === 0) {
         return { nodesByFile: {}, referencesByRoot: {} };
     }
@@ -125,44 +101,106 @@ function runAnalyzer(files, options) {
     //     }
     //   }
     // }
+    // Nothing here may contain a double quote: on Windows `safeSpawn` routes a bare binary
+    // name through cmd.exe, whose quoting cannot express one, so it refuses the argument.
+    // That is why the options JSON goes over stdin instead (see below).
+    const args = [analyzerPath, devkit_1.workspaceRoot];
+    // Cancel any in-flight analyzer from a previous call, then create a fresh controller.
+    cancelPendingAnalysis();
+    const controller = new AbortController();
+    currentAbortController = controller;
+    const signal = controller.signal;
+    const timeoutMs = getAnalysisTimeoutMs();
+    const timeoutSeconds = timeoutMs / 1000;
+    const timer = setTimeout(() => controller.abort(), timeoutMs);
+    let stdout = '';
+    let stderr = '';
     try {
-        let output;
-        // Prepare CLI arguments
-        const args = [analyzerPath, devkit_1.workspaceRoot];
-        // Add plugin options as JSON string if provided
-        if (options) {
-            args.push(JSON.stringify(options));
-        }
-        // Use stdin mode for large file lists to avoid ARG_MAX issues. The analyzer
-        // partitions paths by filename, so we just stream everything in one block.
-        const input = files.join('\n');
-        const result = (0, node_child_process_1.spawnSync)('dotnet', args, {
-            input,
-            encoding: 'utf-8',
-            maxBuffer: 10 * 1024 * 1024, // 10MB buffer
-            stdio: ['pipe', 'pipe', 'pipe'], // stdin, stdout, stderr
-            windowsHide: true,
-            env,
+        await new Promise((resolve, reject) => {
+            const child = (0, internal_1.safeSpawn)('dotnet', args, {
+                stdio: ['pipe', 'pipe', 'pipe'],
+                windowsHide: true,
+                env,
+            });
+            // A plugin worker torn down by `nx reset` would otherwise orphan the analyzer.
+            (0, internal_1.killChildOnHostExit)(child);
+            // On abort, kill the process tree and settle immediately — a wedged
+            // process that outlives the kill signal would otherwise keep this
+            // promise pending and the abort error would never surface.
+            const onAbort = () => {
+                if (child.pid) {
+                    (0, internal_1.killProcessTreeGraceful)(child.pid).catch(() => { });
+                }
+                reject(new Error('Analyzer aborted'));
+            };
+            signal.addEventListener('abort', onAbort, { once: true });
+            child.stdout?.setEncoding('utf-8');
+            child.stderr?.setEncoding('utf-8');
+            child.stdout?.on('data', (data) => {
+                stdout += data;
+            });
+            child.stderr?.on('data', (data) => {
+                stderr += data;
+            });
+            child.on('error', (error) => {
+                signal.removeEventListener('abort', onAbort);
+                reject(error);
+            });
+            child.on('close', (code) => {
+                signal.removeEventListener('abort', onAbort);
+                if (code === 0) {
+                    resolve();
+                }
+                else {
+                    reject(new Error(`Analyzer exited with code ${code}: ${stderr}`));
+                }
+            });
+            // Stream the options JSON, then the file list, over stdin. The first line is
+            // always written, empty when there are no options, so the analyzer never has to
+            // infer whether one was sent. The analyzer partitions paths by filename, so we
+            // just send everything in one block.
+            // EPIPE if the analyzer dies early. Surfaced via `close` only when the
+            // child also exits non-zero; a stdin error with a clean exit means the
+            // analyzer read a truncated file list.
+            child.stdin?.on('error', () => { });
+            child.stdin?.end([options ? JSON.stringify(options) : '', ...files].join('\n'));
         });
-        if (result.error) {
-            throw result.error;
+    }
+    catch (error) {
+        if (signal.reason === 'cancelled') {
+            throw new Error(exports.ANALYZER_CANCELLED_MESSAGE);
         }
-        if (result.status !== 0) {
-            throw new Error(`Analyzer exited with code ${result.status}: ${result.stderr}`);
+        if (signal.aborted) {
+            throw new Error(`msbuild-analyzer timed out after ${timeoutSeconds} ${timeoutSeconds === 1 ? 'second' : 'seconds'}.\n` +
+                `  1. Set the environment variable NX_DOTNET_PROJECT_GRAPH_TIMEOUT to a higher value (in seconds) to increase the timeout.\n` +
+                `  2. If the issue persists, set NX_DOTNET_DISABLE=true to disable the .NET plugin entirely.`);
         }
-        // Output stderr (includes performance logs when NX_PERF_LOGGING=true)
-        if (result.stderr) {
-            console.error(result.stderr);
+        const err = error;
+        const stderrAlreadyInMessage = err.message.includes(stderr);
+        if (stderr && !stderrAlreadyInMessage) {
+            devkit_1.logger.error(`msbuild-analyzer error: ${stderr}`);
         }
-        output = result.stdout;
-        return JSON.parse(output);
+        throw new Error(`Failed to run msbuild-analyzer: ${err.message}${stderr && !stderrAlreadyInMessage ? `\n${stderr}` : ''}`);
     }
-    catch (error) {
-        const err = error;
-        if (err.stderr) {
-            devkit_1.logger.error(`msbuild-analyzer error: ${err.stderr}`);
+    finally {
+        clearTimeout(timer);
+        // Drop the settled controller so a later cancelPendingAnalysis() cannot
+        // abort a run that already finished.
+        if (currentAbortController === controller) {
+            currentAbortController = undefined;
         }
-        throw new Error(`Failed to run msbuild-analyzer: ${err.message}${err.stderr ? `\n${err.stderr}` : ''}`);
+    }
+    // Output stderr (includes performance logs when NX_PERF_LOGGING=true)
+    if (stderr) {
+        console.error(stderr);
+    }
+    try {
+        // parseJson over JSON.parse: on malformed output it reports line:column with
+        // a code frame, where JSON.parse gives only a character offset.
+        return (0, devkit_1.parseJson)(stdout);
+    }
+    catch (error) {
+        throw new Error(`Failed to parse msbuild-analyzer output: ${error.message}`);
     }
 }
 /**
@@ -172,47 +210,36 @@ function runAnalyzer(files, options) {
  */
 async function analyzeProjects(files, options) {
     const filesHash = await calculateProjectFilesHash(files);
-    const optionsHash = (0, internal_1.hashObject)(options);
     // Return cached results if the hash matches
     if (cache &&
         cache.hash === filesHash &&
-        cache.optionsHash === optionsHash &&
         // NOTE: We don't read from the cache here if it's an error result,
         // to allow retrying analysis in case of transient errors or errors fixed
         // that may not be reflected in the hash (like setting an env var).
-        isAnalysisSuccessResult(cache.result) &&
-        (await isSourceHashCurrent(cache.sourceHash, cache.result))) {
+        isAnalysisSuccessResult(cache.result)) {
         return cache.result;
     }
-    // The `-v2` marks the cache *value* shape, which now wraps the analysis
-    // alongside the hash of the sources it came from. Entries written by an
-    // earlier version live under the old name and are simply never read.
-    const analyzerCache = new internal_1.PluginCache((0, node_path_1.join)(internal_1.workspaceDataDirectory, `dotnet-v2-${optionsHash}.hash`));
-    const cachedEntry = analyzerCache.get(filesHash);
-    if (cachedEntry?.result &&
-        (await isSourceHashCurrent(cachedEntry.sourceHash, cachedEntry.result))) {
+    const optionsHash = (0, internal_1.hashObject)(options);
+    const analyzerCache = new internal_1.PluginCache((0, node_path_1.join)(internal_1.workspaceDataDirectory, `dotnet-${optionsHash}.hash`));
+    const cachedResult = analyzerCache.get(filesHash);
+    if (cachedResult) {
         // Update cache
         cache = {
             hash: filesHash,
-            optionsHash,
-            result: cachedEntry.result,
-            sourceHash: cachedEntry.sourceHash,
+            result: cachedResult,
         };
-        return cachedEntry.result;
+        return cachedResult;
     }
     // Run the analyzer
     try {
-        const result = runAnalyzer(files, options);
-        const sourceHash = await computeSourceHash(result);
+        const result = await runAnalyzer(files, options);
         // Update local cache
         cache = {
             hash: filesHash,
-            optionsHash,
             result,
-            sourceHash,
         };
         // Update persistent cache
-        analyzerCache.set(filesHash, { result, sourceHash });
+        analyzerCache.set(filesHash, result);
         analyzerCache.writeToDisk();
         return result;
     }
@@ -226,12 +253,14 @@ async function analyzeProjects(files, options) {
         const errorResult = {
             error: err,
         };
-        cache = {
-            hash: filesHash,
-            optionsHash,
-            result: errorResult,
-            sourceHash: null,
-        };
+        // A superseded run is not a result: caching it would hand the sentinel to
+        // createDependencies via readCachedAnalysisResult().
+        if (err.message !== exports.ANALYZER_CANCELLED_MESSAGE) {
+            cache = {
+                hash: filesHash,
+                result: errorResult,
+            };
+        }
         return errorResult;
     }
 }

package/dist/lib/MsbuildAnalyzer.deps.json

@@ -10,8 +10,7 @@
         "dependencies": {
           "Microsoft.Build": "17.0.0",
           "Microsoft.Build.Framework": "17.0.0",
-          "Microsoft.Build.Locator": "1.8.1",
-          "Microsoft.CodeAnalysis.CSharp": "4.11.0"
+          "Microsoft.Build.Locator": "1.8.1"
         },
         "runtime": {
           "MsbuildAnalyzer.dll": {}
@@ -22,9 +21,9 @@
           "Microsoft.Build.Framework": "17.0.0",
           "Microsoft.NET.StringTools": "1.0.0",
           "Microsoft.Win32.Registry": "4.3.0",
-          "System.Collections.Immutable": "8.0.0",
+          "System.Collections.Immutable": "5.0.0",
           "System.Configuration.ConfigurationManager": "4.7.0",
-          "System.Reflection.Metadata": "8.0.0",
+          "System.Reflection.Metadata": "1.6.0",
           "System.Security.Principal.Windows": "4.7.0",
           "System.Text.Encoding.CodePages": "4.0.1",
           "System.Text.Json": "5.0.2",
@@ -44,22 +43,6 @@
           }
         }
       },
-      "Microsoft.CodeAnalysis.Analyzers/3.3.4": {},
-      "Microsoft.CodeAnalysis.Common/4.11.0": {
-        "dependencies": {
-          "Microsoft.CodeAnalysis.Analyzers": "3.3.4",
-          "System.Collections.Immutable": "8.0.0",
-          "System.Reflection.Metadata": "8.0.0"
-        }
-      },
-      "Microsoft.CodeAnalysis.CSharp/4.11.0": {
-        "dependencies": {
-          "Microsoft.CodeAnalysis.Analyzers": "3.3.4",
-          "Microsoft.CodeAnalysis.Common": "4.11.0",
-          "System.Collections.Immutable": "8.0.0",
-          "System.Reflection.Metadata": "8.0.0"
-        }
-      },
       "Microsoft.NET.StringTools/1.0.0": {
         "dependencies": {
           "System.Memory": "4.5.4",
@@ -92,7 +75,7 @@
           "System.Runtime": "4.3.0"
         }
       },
-      "System.Collections.Immutable/8.0.0": {},
+      "System.Collections.Immutable/5.0.0": {},
       "System.Configuration.ConfigurationManager/4.7.0": {
         "dependencies": {
           "System.Security.Cryptography.ProtectedData": "4.7.0",
@@ -131,11 +114,7 @@
           "System.Runtime": "4.3.0"
         }
       },
-      "System.Reflection.Metadata/8.0.0": {
-        "dependencies": {
-          "System.Collections.Immutable": "8.0.0"
-        }
-      },
+      "System.Reflection.Metadata/1.6.0": {},
       "System.Reflection.Primitives/4.3.0": {
         "dependencies": {
           "Microsoft.NETCore.Platforms": "3.1.0",
@@ -269,27 +248,6 @@
       "path": "microsoft.build.locator/1.8.1",
       "hashPath": "microsoft.build.locator.1.8.1.nupkg.sha512"
     },
-    "Microsoft.CodeAnalysis.Analyzers/3.3.4": {
-      "type": "package",
-      "serviceable": true,
-      "sha512": "sha512-AxkxcPR+rheX0SmvpLVIGLhOUXAKG56a64kV9VQZ4y9gR9ZmPXnqZvHJnmwLSwzrEP6junUF11vuc+aqo5r68g==",
-      "path": "microsoft.codeanalysis.analyzers/3.3.4",
-      "hashPath": "microsoft.codeanalysis.analyzers.3.3.4.nupkg.sha512"
-    },
-    "Microsoft.CodeAnalysis.Common/4.11.0": {
-      "type": "package",
-      "serviceable": true,
-      "sha512": "sha512-djf8ujmqYImFgB04UGtcsEhHrzVqzHowS+EEl/Yunc5LdrYrZhGBWUTXoCF0NzYXJxtfuD+UVQarWpvrNc94Qg==",
-      "path": "microsoft.codeanalysis.common/4.11.0",
-      "hashPath": "microsoft.codeanalysis.common.4.11.0.nupkg.sha512"
-    },
-    "Microsoft.CodeAnalysis.CSharp/4.11.0": {
-      "type": "package",
-      "serviceable": true,
-      "sha512": "sha512-6XYi2EusI8JT4y2l/F3VVVS+ISoIX9nqHsZRaG6W5aFeJ5BEuBosHfT/ABb73FN0RZ1Z3cj2j7cL28SToJPXOw==",
-      "path": "microsoft.codeanalysis.csharp/4.11.0",
-      "hashPath": "microsoft.codeanalysis.csharp.4.11.0.nupkg.sha512"
-    },
     "Microsoft.NET.StringTools/1.0.0": {
       "type": "package",
       "serviceable": true,
@@ -332,12 +290,12 @@
       "path": "system.collections/4.3.0",
       "hashPath": "system.collections.4.3.0.nupkg.sha512"
     },
-    "System.Collections.Immutable/8.0.0": {
+    "System.Collections.Immutable/5.0.0": {
       "type": "package",
       "serviceable": true,
-      "sha512": "sha512-AurL6Y5BA1WotzlEvVaIDpqzpIPvYnnldxru8oXJU2yFxFUy3+pNXjXd1ymO+RA0rq0+590Q8gaz2l3Sr7fmqg==",
-      "path": "system.collections.immutable/8.0.0",
-      "hashPath": "system.collections.immutable.8.0.0.nupkg.sha512"
+      "sha512": "sha512-FXkLXiK0sVVewcso0imKQoOxjoPAj42R8HtjjbSjVPAzwDfzoyoznWxgA3c38LDbN9SJux1xXoXYAhz98j7r2g==",
+      "path": "system.collections.immutable/5.0.0",
+      "hashPath": "system.collections.immutable.5.0.0.nupkg.sha512"
     },
     "System.Configuration.ConfigurationManager/4.7.0": {
       "type": "package",
@@ -381,12 +339,12 @@
       "path": "system.reflection/4.3.0",
       "hashPath": "system.reflection.4.3.0.nupkg.sha512"
     },
-    "System.Reflection.Metadata/8.0.0": {
+    "System.Reflection.Metadata/1.6.0": {
       "type": "package",
       "serviceable": true,
-      "sha512": "sha512-ptvgrFh7PvWI8bcVqG5rsA/weWM09EnthFHR5SCnS6IN+P4mj6rE1lBDC4U8HL9/57htKAqy4KQ3bBj84cfYyQ==",
-      "path": "system.reflection.metadata/8.0.0",
-      "hashPath": "system.reflection.metadata.8.0.0.nupkg.sha512"
+      "sha512": "sha512-COC1aiAJjCoA5GBF+QKL2uLqEBew4JsCkQmoHKbN3TlOZKa2fKLz5CpiRQKDz0RsAOEGsVKqOD5bomsXq/4STQ==",
+      "path": "system.reflection.metadata/1.6.0",
+      "hashPath": "system.reflection.metadata.1.6.0.nupkg.sha512"
     },
     "System.Reflection.Primitives/4.3.0": {
       "type": "package",

package/dist/plugins/create-nodes.d.ts

@@ -1,50 +1,10 @@
-import { CreateNodes, ProjectConfiguration, TargetConfiguration } from '@nx/devkit';
+import { CreateNodes, TargetConfiguration } from '@nx/devkit';
 export type TargetConfigurationWithName = Partial<TargetConfiguration> & {
     /**
      * The name of the target. Defaults to the target type (e.g., 'build', 'test', etc.)
      */
     targetName?: string;
 };
-/**
- * Configuration for the test target, including the options that split it into
- * one task per test unit.
- */
-export type TestTargetConfiguration = TargetConfigurationWithName & {
-    /**
-     * Name of the target that runs the project's tests split across many tasks.
-     *
-     * Leave unset (the default) and nothing is split — the project keeps only its
-     * ordinary `test` target. Setting it turns on atomization for every project
-     * this plugin registration matches, so to split a single project, register
-     * `@nx/dotnet` a second time with an `include` pattern scoped to it.
-     *
-     * Requires Microsoft.Testing.Platform. Running the group needs Nx Cloud: Nx
-     * refuses the parent target without it, because expanding into every task on
-     * one machine is slower than not splitting at all. Individual tasks carry no
-     * such restriction and run locally.
-     */
-    ciTargetName?: string;
-    /**
-     * Overrides the name of the target group these tasks are collapsed into in Nx
-     * Console and the graph. Defaults to a name derived from `ciTargetName`
-     * (`test-ci` becomes `TEST (CI)`).
-     */
-    ciGroupName?: string;
-    /**
-     * Whether each task runs one test class or one test method. Defaults to
-     * `'class'`.
-     *
-     * `'method'` is worth opting into when each test method carries its own
-     * expensive fixture — spinning up a distributed application, a database, a
-     * browser — because then grouping methods into one task saves no setup work
-     * and only serializes tests that could have run on separate agents. For
-     * ordinary suites it multiplies the target count for no gain, since process
-     * startup dominates a fast test.
-     *
-     * `'method'` currently requires MSTest; see the plugin docs.
-     */
-    ciSplitBy?: 'class' | 'method';
-};
 /**
  * Configuration options for the @nx/dotnet plugin.
  *
@@ -85,11 +45,9 @@ export interface DotNetPluginOptions {
     build?: TargetConfigurationWithName | false;
     /**
      * Configuration for the test target.
-     * Use `targetName` to rename the target, `ciTargetName` to additionally split
-     * the tests across one task per test unit, and provide additional
-     * options/configurations to merge with the generated target.
+     * Use `targetName` to rename the target, and provide additional options/configurations to merge with the generated target.
      */
-    test?: TestTargetConfiguration | false;
+    test?: TargetConfigurationWithName | false;
     /**
      * Configuration for the clean target.
      * Use `targetName` to rename the target, and provide additional options/configurations to merge with the generated target.
@@ -121,10 +79,6 @@ export interface DotNetPluginOptions {
      */
     run?: TargetConfigurationWithName | false;
 }
-/**
- * Merge user-specified target configurations with the generated targets from the analyzer
- */
-export declare function mergeUserTargetConfigurations(node: ProjectConfiguration, options: DotNetPluginOptions): ProjectConfiguration;
 export declare const createNodes: CreateNodes<DotNetPluginOptions>;
 /**
  * @deprecated Use {@link createNodes} instead. This will be removed in Nx 24.

package/dist/plugins/create-nodes.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"create-nodes.d.ts","sourceRoot":"","sources":["../../src/plugins/create-nodes.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,WAAW,EAEX,oBAAoB,EACpB,mBAAmB,EACpB,MAAM,YAAY,CAAC;AAUpB,MAAM,MAAM,2BAA2B,GAAG,OAAO,CAAC,mBAAmB,CAAC,GAAG;IACvE;;OAEG;IACH,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB,CAAC;AAEF;;;GAGG;AACH,MAAM,MAAM,uBAAuB,GAAG,2BAA2B,GAAG;IAClE;;;;;;;;;;;;OAYG;IACH,YAAY,CAAC,EAAE,MAAM,CAAC;IAEtB;;;;OAIG;IACH,WAAW,CAAC,EAAE,MAAM,CAAC;IAErB;;;;;;;;;;;;OAYG;IACH,SAAS,CAAC,EAAE,OAAO,GAAG,QAAQ,CAAC;CAChC,CAAC;AAEF;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AACH,MAAM,WAAW,mBAAmB;IAClC;;;OAGG;IACH,KAAK,CAAC,EAAE,2BAA2B,GAAG,KAAK,CAAC;IAC5C;;;;;OAKG;IACH,IAAI,CAAC,EAAE,uBAAuB,GAAG,KAAK,CAAC;IACvC;;;OAGG;IACH,KAAK,CAAC,EAAE,2BAA2B,GAAG,KAAK,CAAC;IAC5C;;;OAGG;IACH,OAAO,CAAC,EAAE,2BAA2B,GAAG,KAAK,CAAC;IAC9C;;;OAGG;IACH,OAAO,CAAC,EAAE,2BAA2B,GAAG,KAAK,CAAC;IAC9C;;;OAGG;IACH,IAAI,CAAC,EAAE,2BAA2B,GAAG,KAAK,CAAC;IAC3C;;;OAGG;IACH,KAAK,CAAC,EAAE,2BAA2B,GAAG,KAAK,CAAC;IAC5C;;;OAGG;IACH,GAAG,CAAC,EAAE,2BAA2B,GAAG,KAAK,CAAC;CAC3C;AAaD;;GAEG;AACH,wBAAgB,6BAA6B,CAC3C,IAAI,EAAE,oBAAoB,EAC1B,OAAO,EAAE,mBAAmB,GAC3B,oBAAoB,CA4EtB;AAED,eAAO,MAAM,WAAW,EAAE,WAAW,CAAC,mBAAmB,CA8FxD,CAAC;AAEF;;GAEG;AACH,eAAO,MAAM,aAAa,kCAAc,CAAC"}
+{"version":3,"file":"create-nodes.d.ts","sourceRoot":"","sources":["../../src/plugins/create-nodes.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,WAAW,EAGX,mBAAmB,EACpB,MAAM,YAAY,CAAC;AAQpB,MAAM,MAAM,2BAA2B,GAAG,OAAO,CAAC,mBAAmB,CAAC,GAAG;IACvE;;OAEG;IACH,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB,CAAC;AAEF;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AACH,MAAM,WAAW,mBAAmB;IAClC;;;OAGG;IACH,KAAK,CAAC,EAAE,2BAA2B,GAAG,KAAK,CAAC;IAC5C;;;OAGG;IACH,IAAI,CAAC,EAAE,2BAA2B,GAAG,KAAK,CAAC;IAC3C;;;OAGG;IACH,KAAK,CAAC,EAAE,2BAA2B,GAAG,KAAK,CAAC;IAC5C;;;OAGG;IACH,OAAO,CAAC,EAAE,2BAA2B,GAAG,KAAK,CAAC;IAC9C;;;OAGG;IACH,OAAO,CAAC,EAAE,2BAA2B,GAAG,KAAK,CAAC;IAC9C;;;OAGG;IACH,IAAI,CAAC,EAAE,2BAA2B,GAAG,KAAK,CAAC;IAC3C;;;OAGG;IACH,KAAK,CAAC,EAAE,2BAA2B,GAAG,KAAK,CAAC;IAC5C;;;OAGG;IACH,GAAG,CAAC,EAAE,2BAA2B,GAAG,KAAK,CAAC;CAC3C;AAqFD,eAAO,MAAM,WAAW,EAAE,WAAW,CAAC,mBAAmB,CAqFxD,CAAC;AAEF;;GAEG;AACH,eAAO,MAAM,aAAa,kCAAc,CAAC"}

package/dist/plugins/create-nodes.js

@@ -1,7 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.createNodesV2 = exports.createNodes = void 0;
-exports.mergeUserTargetConfigurations = mergeUserTargetConfigurations;
 const devkit_1 = require("@nx/devkit");
 const analyzer_client_1 = require("../analyzer/analyzer-client");
 const internal_1 = require("@nx/devkit/internal");
@@ -38,13 +37,8 @@ function mergeUserTargetConfigurations(node, options) {
             delete mergedTargets[defaultTargetName];
             continue;
         }
-        // Use empty object as default when option is not provided.
-        //
-        // The ci* keys configure how the analyzer generates targets rather than
-        // being target configuration themselves, so they are pulled out here
-        // alongside targetName — otherwise they would be merged onto the generated
-        // target as junk properties.
-        const { targetName, ciTargetName: _ciTargetName, ciGroupName: _ciGroupName, ciSplitBy: _ciSplitBy, ...userSpecifiedConfig } = targetOption ?? {};
+        // Use empty object as default when option is not provided
+        const { targetName, ...userSpecifiedConfig } = targetOption ?? {};
         const actualTargetName = targetName ?? defaultTargetName;
         // Find the generated target - it might be under the default name or the user-specified name
         const generatedTarget = mergedTargets[actualTargetName] ?? mergedTargets[defaultTargetName];
@@ -69,7 +63,6 @@ function mergeUserTargetConfigurations(node, options) {
     return {
         ...node,
         targets: mergedTargets,
-        ...(node.metadata ? { metadata: node.metadata } : {}),
     };
 }
 exports.createNodes = [
@@ -79,23 +72,8 @@ exports.createNodes = [
         try {
             // Normalize options to handle undefined (when plugin is registered as string)
             const normalizedOptions = options ?? {};
-            const testOptions = normalizedOptions.test === false ? undefined : normalizedOptions.test;
-            // Only meaningful when the test target itself is enabled; splitting a
-            // target that will be deleted would generate targets referring to it.
-            const ciTargetName = testOptions?.ciTargetName;
             // Extract target names from new format and create options for analyzer
             const analyzerOptions = {
-                // Derived here rather than in the analyzer so there is one
-                // implementation of the naming convention shared with every other Nx
-                // plugin that splits tests.
-                testCiTargetName: ciTargetName,
-                testCiGroupName: ciTargetName
-                    ? (testOptions?.ciGroupName ??
-                        (0, internal_1.deriveGroupNameFromTarget)(ciTargetName))
-                    : undefined,
-                testCiSplitBy: ciTargetName
-                    ? (testOptions?.ciSplitBy ?? 'class')
-                    : undefined,
                 buildTargetName: (normalizedOptions.build && normalizedOptions.build.targetName) ||
                     'build',
                 testTargetName: (normalizedOptions.test && normalizedOptions.test.targetName) ||
@@ -114,6 +92,14 @@ exports.createNodes = [
             };
             const result = await (0, analyzer_client_1.analyzeProjects)([...configFilePaths], analyzerOptions);
             if ((0, analyzer_client_1.isAnalysisErrorResult)(result)) {
+                if (result.error.message === analyzer_client_1.ANALYZER_CANCELLED_MESSAGE) {
+                    // Superseded by a newer analysis — silently return empty rather than
+                    // failing the user's command with an internal sentinel.
+                    // Safe only because the daemon drops this compute at its
+                    // `stalePostCreateNodes` guard before createDependencies runs; that one
+                    // would throw on the cache this run deliberately did not write.
+                    return [];
+                }
                 throw result.error;
             }
             const { nodesByFile } = result;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/dotnet",
-  "version": "23.2.0-pr.36527.2252709",
+  "version": "23.2.0-pr.36838.ef8e258",
   "private": false,
   "description": "The Nx Plugin for .NET containing graph support for working with .NET projects in an Nx workspace.",
   "keywords": [
@@ -49,7 +49,7 @@
   "dependencies": {
     "tslib": "^2.3.0",
     "ignore": "^7.0.5",
-    "@nx/devkit": "23.2.0-pr.36527.2252709"
+    "@nx/devkit": "23.2.0-pr.36838.ef8e258"
   },
   "devDependencies": {
     "@types/jest": "30.0.0",
@@ -58,7 +58,7 @@
     "memfs": "^4.9.2",
     "ts-jest": "^29.4.7",
     "typescript": "~6.0.3",
-    "nx": "23.2.0-pr.36527.2252709"
+    "nx": "23.2.0-pr.36838.ef8e258"
   },
   "publishConfig": {
     "access": "public"