@nx/devkit 23.2.0-beta.10 → 23.2.0-beta.12

package/dist/src/generators/plugin-migrations/executor-to-plugin-migrator.js

@@ -1,11 +1,24 @@
 "use strict";
-var _ExecutorToPluginMigrator_instances, _ExecutorToPluginMigrator_projectGraph, _ExecutorToPluginMigrator_executor, _ExecutorToPluginMigrator_pluginPath, _ExecutorToPluginMigrator_pluginOptionsBuilder, _ExecutorToPluginMigrator_postTargetTransformer, _ExecutorToPluginMigrator_skipTargetFilter, _ExecutorToPluginMigrator_skipProjectFilter, _ExecutorToPluginMigrator_specificProjectToMigrate, _ExecutorToPluginMigrator_logger, _ExecutorToPluginMigrator_nxJson, _ExecutorToPluginMigrator_targetDefaultsForExecutor, _ExecutorToPluginMigrator_targetAndProjectsToMigrate, _ExecutorToPluginMigrator_createNodes, _ExecutorToPluginMigrator_createNodesV2, _ExecutorToPluginMigrator_createNodesResultsForTargets, _ExecutorToPluginMigrator_skippedProjects, _ExecutorToPluginMigrator_init, _ExecutorToPluginMigrator_migrateTarget, _ExecutorToPluginMigrator_migrateProject, _ExecutorToPluginMigrator_mergeInputs, _ExecutorToPluginMigrator_getTargetAndProjectsToMigrate, _ExecutorToPluginMigrator_getTargetDefaultsForExecutor, _ExecutorToPluginMigrator_getCreatedTargetForProjectRoot, _ExecutorToPluginMigrator_getCreateNodesResults;
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.NoTargetsToMigrateError = void 0;
+exports.stableStringify = stableStringify;
+exports.collectMigrationScope = collectMigrationScope;
+exports.computeResidualByProject = computeResidualByProject;
+exports.computeStrictCommon = computeStrictCommon;
+exports.subtractCommon = subtractCommon;
+exports.removeDeadExecutorTargetDefault = removeDeadExecutorTargetDefault;
+exports.appendPluginScopedTargetDefault = appendPluginScopedTargetDefault;
+exports.removeHoistedTargetDefault = removeHoistedTargetDefault;
+exports.packageJsonAuthorsTargetIdentity = packageJsonAuthorsTargetIdentity;
+exports.isRegistrationOfPlugin = isRegistrationOfPlugin;
+exports.hoistChangesExistingTargetDefaults = hoistChangesExistingTargetDefaults;
 exports.readTargetDefaultsForExecutor = readTargetDefaultsForExecutor;
 exports.migrateProjectExecutorsToPlugin = migrateProjectExecutorsToPlugin;
 exports.migrateProjectExecutorsToPluginV1 = migrateProjectExecutorsToPluginV1;
-const tslib_1 = require("tslib");
+exports.inferOncePerOptionSet = inferOncePerOptionSet;
+exports.generatedIncludeRoots = generatedIncludeRoots;
+exports.harvestConfigurationErrors = harvestConfigurationErrors;
+exports.buildOwnerRootByPath = buildOwnerRootByPath;
 const minimatch_1 = require("minimatch");
 const node_assert_1 = require("node:assert");
 const posix_1 = require("node:path/posix");
@@ -13,81 +26,255 @@ const devkit_exports_1 = require("nx/src/devkit-exports");
 const devkit_internals_1 = require("nx/src/devkit-internals");
 const executor_options_utils_1 = require("../executor-options-utils");
 const target_defaults_utils_1 = require("../target-defaults-utils");
+const batch_conversion_session_1 = require("./batch-conversion-session");
+const conversion_warnings_1 = require("./conversion-warnings");
 const plugin_migration_utils_1 = require("./plugin-migration-utils");
-class ExecutorToPluginMigrator {
-    constructor(tree, projectGraph, executor, pluginPath, pluginOptionsBuilder, postTargetTransformer, createNodes, createNodesV2, specificProjectToMigrate, filters, logger) {
-        _ExecutorToPluginMigrator_instances.add(this);
-        _ExecutorToPluginMigrator_projectGraph.set(this, void 0);
-        _ExecutorToPluginMigrator_executor.set(this, void 0);
-        _ExecutorToPluginMigrator_pluginPath.set(this, void 0);
-        _ExecutorToPluginMigrator_pluginOptionsBuilder.set(this, void 0);
-        _ExecutorToPluginMigrator_postTargetTransformer.set(this, void 0);
-        _ExecutorToPluginMigrator_skipTargetFilter.set(this, void 0);
-        _ExecutorToPluginMigrator_skipProjectFilter.set(this, void 0);
-        _ExecutorToPluginMigrator_specificProjectToMigrate.set(this, void 0);
-        _ExecutorToPluginMigrator_logger.set(this, void 0);
-        _ExecutorToPluginMigrator_nxJson.set(this, void 0);
-        _ExecutorToPluginMigrator_targetDefaultsForExecutor.set(this, void 0);
-        _ExecutorToPluginMigrator_targetAndProjectsToMigrate.set(this, void 0);
-        _ExecutorToPluginMigrator_createNodes.set(this, void 0);
-        _ExecutorToPluginMigrator_createNodesV2.set(this, void 0);
-        _ExecutorToPluginMigrator_createNodesResultsForTargets.set(this, void 0);
-        _ExecutorToPluginMigrator_skippedProjects.set(this, void 0);
-        this.tree = tree;
-        tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_projectGraph, projectGraph, "f");
-        tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_executor, executor, "f");
-        tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_pluginPath, pluginPath, "f");
-        tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_pluginOptionsBuilder, pluginOptionsBuilder, "f");
-        tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_postTargetTransformer, postTargetTransformer, "f");
-        tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_createNodes, createNodes, "f");
-        tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_createNodesV2, createNodesV2, "f");
-        tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_specificProjectToMigrate, specificProjectToMigrate, "f");
-        tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_skipProjectFilter, filters?.skipProjectFilter ?? ((...args) => false), "f");
-        tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_skipTargetFilter, filters?.skipTargetFilter ?? ((...args) => false), "f");
-        tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_logger, logger ?? devkit_exports_1.logger, "f");
-    }
-    async run() {
-        await tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_instances, "m", _ExecutorToPluginMigrator_init).call(this);
-        if (tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_targetAndProjectsToMigrate, "f").size > 0) {
-            for (const targetName of tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_targetAndProjectsToMigrate, "f").keys()) {
-                await tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_instances, "m", _ExecutorToPluginMigrator_migrateTarget).call(this, targetName);
-            }
-        }
-        return tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_targetAndProjectsToMigrate, "f");
-    }
-}
-_ExecutorToPluginMigrator_projectGraph = new WeakMap(), _ExecutorToPluginMigrator_executor = new WeakMap(), _ExecutorToPluginMigrator_pluginPath = new WeakMap(), _ExecutorToPluginMigrator_pluginOptionsBuilder = new WeakMap(), _ExecutorToPluginMigrator_postTargetTransformer = new WeakMap(), _ExecutorToPluginMigrator_skipTargetFilter = new WeakMap(), _ExecutorToPluginMigrator_skipProjectFilter = new WeakMap(), _ExecutorToPluginMigrator_specificProjectToMigrate = new WeakMap(), _ExecutorToPluginMigrator_logger = new WeakMap(), _ExecutorToPluginMigrator_nxJson = new WeakMap(), _ExecutorToPluginMigrator_targetDefaultsForExecutor = new WeakMap(), _ExecutorToPluginMigrator_targetAndProjectsToMigrate = new WeakMap(), _ExecutorToPluginMigrator_createNodes = new WeakMap(), _ExecutorToPluginMigrator_createNodesV2 = new WeakMap(), _ExecutorToPluginMigrator_createNodesResultsForTargets = new WeakMap(), _ExecutorToPluginMigrator_skippedProjects = new WeakMap(), _ExecutorToPluginMigrator_instances = new WeakSet(), _ExecutorToPluginMigrator_init = async function _ExecutorToPluginMigrator_init() {
-    const nxJson = (0, devkit_exports_1.readNxJson)(this.tree);
-    nxJson.plugins ??= [];
-    tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_nxJson, nxJson, "f");
-    tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_targetAndProjectsToMigrate, new Map(), "f");
-    tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_createNodesResultsForTargets, new Map(), "f");
-    tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_skippedProjects, new Set(), "f");
-    tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_instances, "m", _ExecutorToPluginMigrator_getTargetDefaultsForExecutor).call(this);
-    tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_instances, "m", _ExecutorToPluginMigrator_getTargetAndProjectsToMigrate).call(this);
-    await tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_instances, "m", _ExecutorToPluginMigrator_getCreateNodesResults).call(this);
-}, _ExecutorToPluginMigrator_migrateTarget = async function _ExecutorToPluginMigrator_migrateTarget(targetName) {
-    for (const projectName of tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_targetAndProjectsToMigrate, "f").get(targetName)) {
-        await tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_instances, "m", _ExecutorToPluginMigrator_migrateProject).call(this, projectName, targetName);
-    }
-}, _ExecutorToPluginMigrator_migrateProject = async function _ExecutorToPluginMigrator_migrateProject(projectName, targetName) {
-    const projectFromGraph = tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_projectGraph, "f").nodes[projectName];
-    const projectConfig = (0, devkit_exports_1.readProjectConfiguration)(this.tree, projectName);
-    const createdTarget = tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_instances, "m", _ExecutorToPluginMigrator_getCreatedTargetForProjectRoot).call(this, targetName, projectFromGraph.data.root);
-    let projectTarget = projectConfig.targets[targetName];
-    projectTarget = (0, devkit_internals_1.mergeTargetConfigurations)(projectTarget, tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_targetDefaultsForExecutor, "f"));
-    delete projectTarget.executor;
-    (0, plugin_migration_utils_1.deleteMatchingProperties)(projectTarget, createdTarget);
-    if (projectTarget.inputs && createdTarget.inputs) {
-        tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_instances, "m", _ExecutorToPluginMigrator_mergeInputs).call(this, projectTarget, createdTarget);
-    }
-    projectTarget = await tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_postTargetTransformer, "f").call(this, projectTarget, this.tree, { projectName, root: projectFromGraph.data.root }, { ...createdTarget, name: targetName });
-    if (projectTarget.options &&
-        Object.keys(projectTarget.options).length === 0) {
-        delete projectTarget.options;
-    }
-    if (Object.keys(projectTarget).length > 0) {
-        projectConfig.targets[targetName] = projectTarget;
+function stableStringify(value) {
+    if (value === null || typeof value !== 'object') {
+        return JSON.stringify(value) ?? 'undefined';
+    }
+    if (Array.isArray(value)) {
+        return `[${value.map(stableStringify).join(',')}]`;
+    }
+    const keys = Object.keys(value).sort();
+    return `{${keys
+        .map((k) => JSON.stringify(k) +
+        ':' +
+        stableStringify(value[k]))
+        .join(',')}}`;
+}
+/**
+ * Phase 0: Collect (once). Fold `forEachExecutorOptions` over every
+ * migration/executor into a single scope object, applying the skip filters with
+ * the exact same warn-vs-throw semantics the migrator used before (a
+ * `specificProjectToMigrate` skip throws instead of warning). This is the single
+ * authority for filtering; downstream phases only read the returned maps.
+ */
+function collectMigrationScope(tree, projectGraph, migrations, defaultPluginOptions, specificProjectToMigrate, logger) {
+    const log = logger ?? devkit_exports_1.logger;
+    const pluginOptionsByProject = new Map();
+    // Same content as pluginOptionsByProject, accumulated in walk order so the
+    // conflict check below sees options claimed by earlier targets.
+    // pluginOptionsByProject itself is populated in the grouping loop to keep
+    // its target-grouped insertion order, which addPluginRegistrations turns
+    // into the registration order in nx.json.
+    const claimedOptionsByProject = new Map();
+    const executorScopes = [];
+    const optionSetGroupsByKey = new Map();
+    for (const migration of migrations) {
+        const skipProjectFilter = migration.skipProjectFilter ?? ((..._args) => false);
+        const skipTargetFilter = migration.skipTargetFilter ?? ((..._args) => false);
+        for (const executor of migration.executors) {
+            const targetAndProjects = new Map();
+            const inferenceOptionSetIdsByTarget = new Map();
+            const inferenceOptionsByTarget = new Map();
+            // Fresh per executor to preserve the previous per-migrator semantics
+            // (each executor got its own migrator + skipped set).
+            const skippedForExecutor = new Set();
+            (0, executor_options_utils_1.forEachExecutorOptions)(tree, executor, (options, projectName, targetName, configurationName) => {
+                if (skippedForExecutor.has(projectName) || configurationName) {
+                    return;
+                }
+                if (specificProjectToMigrate &&
+                    projectName !== specificProjectToMigrate) {
+                    return;
+                }
+                const skipProjectReason = skipProjectFilter(projectGraph.nodes[projectName].data);
+                if (skipProjectReason) {
+                    skippedForExecutor.add(projectName);
+                    const errorMsg = `The "${projectName}" project cannot be migrated. ${skipProjectReason}`;
+                    if (specificProjectToMigrate) {
+                        throw new Error(errorMsg);
+                    }
+                    log.warn(errorMsg);
+                    return;
+                }
+                const skipTargetReason = skipTargetFilter(options, projectGraph.nodes[projectName].data);
+                if (skipTargetReason) {
+                    const errorMsg = `The ${targetName} target on project "${projectName}" cannot be migrated. ${skipTargetReason}`;
+                    if (specificProjectToMigrate) {
+                        throw new Error(errorMsg);
+                    }
+                    log.warn(errorMsg);
+                    return;
+                }
+                if (!inferenceOptionsByTarget.has(targetName)) {
+                    inferenceOptionsByTarget.set(targetName, migration.targetPluginOptionMapper(targetName));
+                }
+                const inferenceOptions = inferenceOptionsByTarget.get(targetName);
+                // One registration cannot map different target names through the
+                // same option
+                const existing = claimedOptionsByProject.get(projectName);
+                const takenOption = Object.keys(inferenceOptions).find((option) => existing?.[option] !== undefined &&
+                    !isDeepEqual(existing[option], inferenceOptions[option]));
+                if (takenOption && !migration.allowSharedOptionOverwrite) {
+                    const errorMsg = `The ${targetName} target on project "${projectName}" cannot be migrated. The "${takenOption}" plugin option is already set to "${existing[takenOption]}" by another target of the project, and the plugin can only infer one target per option.`;
+                    if (specificProjectToMigrate) {
+                        throw new Error(errorMsg);
+                    }
+                    log.warn(`${errorMsg} The target keeps its current configuration; rerun the migration once the other target is inferred to convert it.`);
+                    return;
+                }
+                claimedOptionsByProject.set(projectName, {
+                    ...(existing ?? {}),
+                    ...inferenceOptions,
+                });
+                if (!targetAndProjects.has(targetName)) {
+                    targetAndProjects.set(targetName, new Set());
+                }
+                targetAndProjects.get(targetName).add(projectName);
+            });
+            for (const [targetName, projs] of targetAndProjects) {
+                const inferenceOptions = inferenceOptionsByTarget.get(targetName);
+                const key = stableStringify(inferenceOptions);
+                if (!optionSetGroupsByKey.has(key)) {
+                    optionSetGroupsByKey.set(key, {
+                        id: optionSetGroupsByKey.size,
+                        options: inferenceOptions,
+                        targetNames: new Set(),
+                        migratedRoots: new Set(),
+                    });
+                }
+                const optionSetGroup = optionSetGroupsByKey.get(key);
+                optionSetGroup.targetNames.add(targetName);
+                inferenceOptionSetIdsByTarget.set(targetName, optionSetGroup.id);
+                for (const project of projs) {
+                    optionSetGroup.migratedRoots.add(projectGraph.nodes[project].data.root);
+                    pluginOptionsByProject.set(project, {
+                        ...(pluginOptionsByProject.get(project) ?? {}),
+                        ...inferenceOptions,
+                    });
+                }
+            }
+            if (targetAndProjects.size === 0) {
+                continue;
+            }
+            executorScopes.push({
+                executor,
+                migration,
+                targetAndProjects,
+                inferenceOptionSetIdsByTarget,
+            });
+        }
+    }
+    // Apply default plugin options (registration options only).
+    for (const [project, options] of pluginOptionsByProject) {
+        pluginOptionsByProject.set(project, {
+            ...defaultPluginOptions,
+            ...options,
+        });
+    }
+    return {
+        pluginOptionsByProject,
+        optionSetGroups: [...optionSetGroupsByKey.values()],
+        executorScopes,
+    };
+}
+function stripInferredTarget(fullInferredTarget) {
+    const stripped = structuredClone(fullInferredTarget);
+    delete stripped.command;
+    delete stripped.options?.cwd;
+    return stripped;
+}
+function getFullInferredTarget(inferredTargetsByOptionSet, optionSetId, projectRoot, targetName) {
+    const inferredByRoot = inferredTargetsByOptionSet.get(optionSetId);
+    const inferredTargetsByName = inferredByRoot?.get(projectRoot);
+    if (!inferredTargetsByName) {
+        throw new Error(`The nx plugin did not find a project inside ${projectRoot}. File an issue at https://github.com/nrwl/nx with information about your project structure.`);
+    }
+    const inferredTarget = inferredTargetsByName.get(targetName);
+    if (!inferredTarget) {
+        // The plugin found the project but did not infer a "${targetName}" target
+        // for it under this option set: the migration and inference disagree on
+        // the target name.
+        throw new Error(`The nx plugin found a project inside ${projectRoot} but did not infer a "${targetName}" target for it. File an issue at https://github.com/nrwl/nx with information about your project structure.`);
+    }
+    return inferredTarget;
+}
+/**
+ * Phase 2: Per-project residual (in-memory, no inference). For each
+ * `(project, target)` computes the residual exactly as the previous engine did
+ * (`mergeTargetConfigurations` with the executor target defaults ->
+ * `deleteMatchingProperties` -> input merge -> the plugin's
+ * `postTargetTransformer`), plus `baselineFinal = merge(residual, inferred)` as
+ * the equivalence oracle. Does NOT write project.json.
+ */
+async function computeResidualByProject(tree, projectGraph, scope, inferredTargetsByOptionSet, nxJson, projectConfigsByName = (0, devkit_exports_1.getProjects)(tree)) {
+    const residualByProject = new Map();
+    for (const executorScope of scope.executorScopes) {
+        const targetDefaultsForExecutor = readTargetDefaultsForExecutor(executorScope.executor, nxJson.targetDefaults) ?? {};
+        for (const [targetName, projectNames] of executorScope.targetAndProjects) {
+            const optionSetId = executorScope.inferenceOptionSetIdsByTarget.get(targetName);
+            if (optionSetId === undefined) {
+                throw new Error(`Could not find an inference option set for target "${targetName}".`);
+            }
+            for (const projectName of projectNames) {
+                const root = projectGraph.nodes[projectName].data.root;
+                const fullInferredTarget = getFullInferredTarget(inferredTargetsByOptionSet, optionSetId, root, targetName);
+                const strippedInferredTarget = stripInferredTarget(fullInferredTarget);
+                const projectConfig = readCachedProjectConfiguration(tree, projectConfigsByName, projectName);
+                const preMigrationTarget = structuredClone(projectConfig.targets[targetName]);
+                // Later steps mutate the merged target, which may retain nested
+                // references from either input
+                let projectTarget = (0, devkit_internals_1.mergeTargetConfigurations)(structuredClone(projectConfig.targets[targetName]), structuredClone(targetDefaultsForExecutor));
+                delete projectTarget.executor;
+                (0, plugin_migration_utils_1.deleteMatchingProperties)(projectTarget, strippedInferredTarget);
+                if (projectTarget.inputs && strippedInferredTarget.inputs) {
+                    mergeInputs(projectTarget, strippedInferredTarget);
+                }
+                projectTarget = await executorScope.migration.postTargetTransformer(projectTarget, tree, { projectName, root }, { ...strippedInferredTarget, name: targetName });
+                if (projectTarget.options &&
+                    Object.keys(projectTarget.options).length === 0) {
+                    delete projectTarget.options;
+                }
+                const residual = projectTarget;
+                const baselineFinal = (0, devkit_internals_1.mergeTargetConfigurations)(structuredClone(residual), structuredClone(fullInferredTarget));
+                if (!residualByProject.has(projectName)) {
+                    residualByProject.set(projectName, new Map());
+                }
+                residualByProject
+                    .get(projectName)
+                    .set(targetName, { residual, baselineFinal, preMigrationTarget });
+            }
+        }
+    }
+    return residualByProject;
+}
+/**
+ * Phase 3 (single-project-mode variant): write the full residual into each
+ * project.json, exactly reproducing the previous per-(project, target) write
+ * sequence. Single-project mode never centralizes, so this is its permanent
+ * write path.
+ */
+function writeResiduals(tree, projectConfigsByName, projectGraph, scope, residualByProject, logger) {
+    for (const executorScope of scope.executorScopes) {
+        for (const [targetName, projectNames] of executorScope.targetAndProjects) {
+            for (const projectName of projectNames) {
+                const entry = residualByProject.get(projectName)?.get(targetName);
+                if (!entry) {
+                    continue;
+                }
+                entry.keptPreMigration = !writeResidualTarget(tree, projectConfigsByName, projectName, targetName, structuredClone(entry.residual), logger);
+            }
+        }
+    }
+}
+/**
+ * Write a single residual target into project.json (or delete it if empty).
+ * Returns false, writing nothing, when package.json authors the target's
+ * identity: without the explicit executor an included same-name script (or an
+ * `nx.targets` executor/command) would replace the inferred target in the
+ * default plugin layer, so the pre-migration target is kept as is.
+ */
+function writeResidualTarget(tree, projectConfigsByName, projectName, targetName, residual, logger) {
+    const projectConfig = readCachedProjectConfiguration(tree, projectConfigsByName, projectName);
+    const identitySource = packageJsonAuthorsTargetIdentity(tree, projectConfig.root, targetName);
+    if (identitySource) {
+        (logger ?? devkit_exports_1.logger).warn((0, conversion_warnings_1.keptPreMigrationTargetWarning)(targetName, projectName, identitySource));
+        return false;
+    }
+    if (Object.keys(residual).length > 0) {
+        projectConfig.targets[targetName] = residual;
     }
     else {
         delete projectConfig.targets[targetName];
@@ -96,8 +283,633 @@ _ExecutorToPluginMigrator_projectGraph = new WeakMap(), _ExecutorToPluginMigrato
         projectConfig['// targets'] =
             `to see all targets run: nx show project ${projectName} --web`;
     }
-    (0, devkit_exports_1.updateProjectConfiguration)(this.tree, projectName, projectConfig);
-}, _ExecutorToPluginMigrator_mergeInputs = function _ExecutorToPluginMigrator_mergeInputs(target, inferredTarget) {
+    (0, devkit_exports_1.updateProjectConfiguration)(tree, projectName, projectConfig);
+    // `updateProjectConfiguration` writes a package-based project (no
+    // project.json) by spreading the config over package.json `nx`, and deletes an
+    // empty `targets` from this very (cached) object before the spread. Two
+    // consequences: the old `nx.targets` entry (executor included) survives in
+    // package.json, and the cache can no longer take a later write (the Phase 4
+    // revert). Delete the entry here and keep the cache writable.
+    if (!projectConfig.targets) {
+        projectConfig.targets = {};
+        deletePackageJsonTarget(tree, projectConfig.root, targetName);
+    }
+    return true;
+}
+function deletePackageJsonTarget(tree, root, targetName) {
+    const packageJsonPath = (0, posix_1.join)(root, 'package.json');
+    const packageJson = (0, devkit_exports_1.readJson)(tree, packageJsonPath);
+    if (!packageJson.nx?.targets?.[targetName]) {
+        return;
+    }
+    delete packageJson.nx.targets[targetName];
+    if (Object.keys(packageJson.nx.targets).length === 0) {
+        delete packageJson.nx.targets;
+    }
+    (0, devkit_exports_1.writeJson)(tree, packageJsonPath, packageJson);
+}
+function readCachedProjectConfiguration(tree, projectConfigsByName, projectName) {
+    const projectConfig = projectConfigsByName.get(projectName);
+    if (projectConfig) {
+        return projectConfig;
+    }
+    const readProjectConfig = (0, devkit_exports_1.readProjectConfiguration)(tree, projectName);
+    projectConfigsByName.set(projectName, readProjectConfig);
+    return readProjectConfig;
+}
+function isDeepEqual(a, b) {
+    try {
+        (0, node_assert_1.deepStrictEqual)(a, b);
+        return true;
+    }
+    catch {
+        return false;
+    }
+}
+/**
+ * Phase 3: the strict-common residual across ALL migrated projects for a
+ * target: the values that are deep-equal across every project's residual.
+ * Granularity: whole value for top-level target props (`inputs`, `outputs`,
+ * `cache`, `dependsOn`, `configurations`, ...); per-key for `options`. A key is
+ * common only when EVERY residual carries it with an identical value.
+ */
+function computeStrictCommon(residuals) {
+    if (residuals.length === 0) {
+        return {};
+    }
+    const [first, ...rest] = residuals;
+    const common = {};
+    for (const key of Object.keys(first)) {
+        if (key === 'options') {
+            continue;
+        }
+        const value = first[key];
+        const isCommon = rest.every((residual) => {
+            const r = residual;
+            return key in r && isDeepEqual(r[key], value);
+        });
+        if (isCommon) {
+            common[key] = structuredClone(value);
+        }
+    }
+    const firstOptions = first.options;
+    if (firstOptions && typeof firstOptions === 'object') {
+        const commonOptions = {};
+        for (const optKey of Object.keys(firstOptions)) {
+            const optValue = firstOptions[optKey];
+            const isCommon = rest.every((residual) => {
+                const options = residual.options;
+                return (options &&
+                    typeof options === 'object' &&
+                    optKey in options &&
+                    isDeepEqual(options[optKey], optValue));
+            });
+            if (isCommon) {
+                commonOptions[optKey] = structuredClone(optValue);
+            }
+        }
+        if (Object.keys(commonOptions).length > 0) {
+            common.options = commonOptions;
+        }
+    }
+    return common;
+}
+/** `residual` with every property that the strict-common config carries removed. */
+function subtractCommon(residual, common) {
+    const deviation = structuredClone(residual);
+    for (const key of Object.keys(common)) {
+        if (key === 'options') {
+            continue;
+        }
+        const d = deviation;
+        if (key in d &&
+            isDeepEqual(d[key], common[key])) {
+            delete d[key];
+        }
+    }
+    if (common.options && deviation.options) {
+        const commonOptions = common.options;
+        const deviationOptions = deviation.options;
+        for (const optKey of Object.keys(commonOptions)) {
+            if (optKey in deviationOptions &&
+                isDeepEqual(deviationOptions[optKey], commonOptions[optKey])) {
+                delete deviationOptions[optKey];
+            }
+        }
+        if (Object.keys(deviationOptions).length === 0) {
+            delete deviation.options;
+        }
+    }
+    return deviation;
+}
+/**
+ * Remove the now-dead executor-keyed target default that Phase 2 inlined into
+ * every migrated project (mirrors `readTargetDefaultsForExecutor`'s match: the
+ * unfiltered entry keyed directly by the executor string).
+ */
+function removeDeadExecutorTargetDefault(nxJson, executor) {
+    (0, target_defaults_utils_1.updateTargetDefault)(nxJson, { executor }, (_config, info) => info.key === executor && info.filter === undefined ? null : undefined);
+}
+/**
+ * Append the hoisted common as a plugin-scoped entry after whatever value the
+ * key already holds. Existing entries, the workspace catch-all and any
+ * user-authored filtered entries, are never modified, so targets outside this
+ * plugin resolve exactly what they resolved before the migration. The entry is
+ * appended (never merged into an existing one) so the verification pass can
+ * revert precisely this entry and nothing else.
+ */
+function appendPluginScopedTargetDefault(nxJson, targetName, pluginPath, common) {
+    const entry = {
+        filter: { plugin: pluginPath },
+        ...structuredClone(common),
+    };
+    nxJson.targetDefaults ??= {};
+    const existing = nxJson.targetDefaults[targetName];
+    nxJson.targetDefaults[targetName] =
+        existing === undefined
+            ? [entry]
+            : Array.isArray(existing)
+                ? [...existing, entry]
+                : [existing, entry];
+    return entry;
+}
+/**
+ * Remove a previously appended plugin-scoped entry, collapsing the value back
+ * to the plain object form when only a lone unfiltered entry remains. The
+ * appended entry survives an `updateNxJson`/`readNxJson` round trip only by
+ * value, so the last deep-equal occurrence (append order puts ours last) is
+ * the one removed.
+ */
+function removeHoistedTargetDefault(nxJson, targetName, entry) {
+    const value = nxJson.targetDefaults?.[targetName];
+    if (!Array.isArray(value)) {
+        return;
+    }
+    const entryKey = stableStringify(entry);
+    const remaining = [...value];
+    for (let i = remaining.length - 1; i >= 0; i--) {
+        if (stableStringify(remaining[i]) === entryKey) {
+            remaining.splice(i, 1);
+            break;
+        }
+    }
+    if (remaining.length === 0) {
+        delete nxJson.targetDefaults[targetName];
+    }
+    else if (remaining.length === 1 && remaining[0].filter === undefined) {
+        const { filter: _filter, ...config } = remaining[0];
+        nxJson.targetDefaults[targetName] = config;
+    }
+    else {
+        nxJson.targetDefaults[targetName] = remaining;
+    }
+}
+/**
+ * Which `package.json` signal, if any, authors an identity for `targetName` in
+ * the DEFAULT plugin layer. The package-json plugin turns every included script into
+ * an `nx:run-script` target and honors `nx.targets`; either way the target gains
+ * an `executor`/`command` in a default layer, which makes Nx's
+ * `resolveSourcePlugin` refuse a `filter: { plugin }` targetDefault for it.
+ * The hoist uses this to keep the full residual per project instead of silently
+ * dropping the centralized keys; the residual write uses it to keep the
+ * pre-migration target instead of letting the package.json identity take the
+ * target over.
+ *
+ * Read through the Tree so this sees the same in-memory package.json the rest of
+ * the generator reads and writes, rather than a possibly-stale copy on disk.
+ */
+function packageJsonAuthorsTargetIdentity(tree, root, targetName) {
+    if (!root) {
+        return undefined;
+    }
+    const packageJsonPath = (0, posix_1.join)(root, 'package.json');
+    if (!tree.exists(packageJsonPath)) {
+        return undefined;
+    }
+    let packageJson;
+    try {
+        packageJson = (0, devkit_exports_1.readJson)(tree, packageJsonPath);
+    }
+    catch {
+        // Nx reads this file with the same jsonc-tolerant parser (`//` comments,
+        // trailing commas), so a failure here means inference cannot be trusted for
+        // it either. Fail closed: treat the identity as authored rather than hoist
+        // on a guess.
+        return 'unparseable';
+    }
+    const scripts = packageJson?.scripts ?? {};
+    // `readTargetsFromPackageJson` turns each *included* script into an
+    // `nx:run-script` target (identity). `nx.includedScripts`, when present,
+    // restricts which scripts become targets. Normalize a malformed (non-array)
+    // value to the default (all scripts) instead of letting `.includes` throw an
+    // uncaught TypeError mid-generator.
+    const nxIncludedScripts = packageJson?.nx?.includedScripts;
+    const includedScripts = Array.isArray(nxIncludedScripts)
+        ? nxIncludedScripts
+        : Object.keys(scripts);
+    if (includedScripts.includes(targetName)) {
+        return 'script';
+    }
+    // The `nx.targets` check below reads PRE-migration state. In a package-based
+    // workspace the target being migrated lives in `nx.targets` itself (that is
+    // where its executor is authored), so reading it here would flag every project
+    // as authoring its own identity, exclude them all, and centralize nothing.
+    // A package-based project's residual is written back to `nx.targets` without
+    // the migrated executor (a residual that still carries an executor/command is
+    // excluded by the caller before this gate), so post-migration it authors no
+    // identity and the hoisted `filter:{plugin}` default resolves. Gate on the
+    // same `project.json` signal `updateProjectConfiguration` uses to decide where
+    // config lives: only trust an `nx.targets` identity when the project keeps its
+    // config in `project.json` (there a package.json `nx.targets` entry is
+    // genuinely separate from the migrated target).
+    if (!tree.exists((0, posix_1.join)(root, 'project.json'))) {
+        return undefined;
+    }
+    // An `nx.targets` entry authors identity only when it says how to run.
+    const nxTarget = packageJson?.nx?.targets?.[targetName];
+    return nxTarget != null &&
+        (nxTarget.executor !== undefined || nxTarget.command !== undefined)
+        ? 'nxTargets'
+        : undefined;
+}
+function isRegistrationOfPlugin(registration, pluginPath) {
+    return typeof registration === 'string'
+        ? registration === pluginPath
+        : registration.plugin === pluginPath;
+}
+/**
+ * Whether every nx.json `plugins` entry after the migrated plugin's first
+ * registration is also the migrated plugin. Only then is the migrated plugin
+ * guaranteed to merge last for the targets it infers, keeping the
+ * executor/command attribution (and with it `filter: { plugin }`
+ * target-default resolution) with this plugin. No registration at all fails
+ * closed.
+ */
+function pluginRegistrationsFormTail(nxJson, pluginPath) {
+    const plugins = nxJson.plugins ?? [];
+    const isMigratedPlugin = (plugin) => isRegistrationOfPlugin(plugin, pluginPath);
+    const firstIndex = plugins.findIndex(isMigratedPlugin);
+    if (firstIndex === -1) {
+        return false;
+    }
+    return plugins.slice(firstIndex + 1).every(isMigratedPlugin);
+}
+/**
+ * The effective executor each migrated (project, target) pair resolves to
+ * after the migration: the INFERRED target's executor, with a top-level
+ * `command` resolving to `nx:run-commands` (mirrors
+ * `resolveCommandSyntacticSugar`). Keyed by `"<project>\t<target>"`. Input to
+ * the target-default preflight, inline and at batch finalize.
+ */
+function computeInferredExecutorByPair(scope, projectGraph, inferredTargetsByOptionSet) {
+    const inferredExecutorByPair = new Map();
+    for (const executorScope of scope.executorScopes) {
+        for (const [targetName, projectNames] of executorScope.targetAndProjects) {
+            const optionSetId = executorScope.inferenceOptionSetIdsByTarget.get(targetName);
+            for (const projectName of projectNames) {
+                const root = projectGraph.nodes[projectName]?.data.root;
+                const inferredTarget = getFullInferredTarget(inferredTargetsByOptionSet, optionSetId, root, targetName);
+                const inferredExecutor = inferredTarget.executor ??
+                    (inferredTarget.command ? 'nx:run-commands' : undefined);
+                inferredExecutorByPair.set(`${projectName}\t${targetName}`, inferredExecutor);
+            }
+        }
+    }
+    return inferredExecutorByPair;
+}
+/**
+ * Whether appending the plugin-scoped `targetDefaults[targetName]` entry would
+ * change what the existing target defaults resolve to for any eligible migrated
+ * pair. Two hazards, both invisible to the Phase 4 verification (it merges no
+ * target defaults):
+ *
+ * - Key displacement: an exact target-name key takes precedence over a glob
+ *   key (`build-*`), so appending one can silently stop a glob default from
+ *   contributing to the migrated targets.
+ * - Executor masking: an executor-keyed default for the plugin's INFERRED
+ *   executor (e.g. `nx:run-commands` for command-based inferred targets) takes
+ *   precedence over the exact key, so the appended entry would never resolve
+ *   and its keys would be silently dropped.
+ *
+ * The check resolves the defaults for each pair without and with the
+ * hypothetical entry, through the production reader. The hoist is a pure
+ * "residual moved into a default" only when the with-entry resolution equals
+ * the without-entry resolution with the common merged on top; anything else
+ * changes behavior, so the target keeps its full residuals.
+ */
+function hoistChangesExistingTargetDefaults(targetDefaults, targetName, common, pluginPath, eligiblePairs, projectNodesByName) {
+    const hypotheticalNxJson = {
+        targetDefaults: structuredClone(targetDefaults) ?? {},
+    };
+    appendPluginScopedTargetDefault(hypotheticalNxJson, targetName, pluginPath, common);
+    // Per-pair context only matters to `filter.projects` entries under a key
+    // this target's lookup can select (its own name, one of the pairs' effective
+    // executors, or a glob matching the name); everything else resolves purely
+    // from the pair's effective executor, so pairs sharing one resolve
+    // identically and only the first needs checking. The appended entry carries
+    // no `projects` filter, so scanning the current map covers the hypothetical
+    // one too.
+    const pairExecutors = new Set();
+    for (const { inferredExecutor } of eligiblePairs) {
+        if (inferredExecutor !== undefined) {
+            pairExecutors.add(inferredExecutor);
+        }
+    }
+    const hasRelevantProjectScopedEntry = Object.entries(targetDefaults ?? {}).some(([key, value]) => {
+        // The glob arm mirrors the reader's own selection test exactly; a bare
+        // `minimatch` would over-match keys the reader never treats as globs
+        // (e.g. `?`, which is not in Nx's glob character set).
+        const keyParticipates = key === targetName ||
+            pairExecutors.has(key) ||
+            ((0, devkit_internals_1.isGlobPattern)(key) && (0, minimatch_1.minimatch)(targetName, key));
+        return (keyParticipates &&
+            (Array.isArray(value) ? value : [value ?? {}]).some((entry) => entry.filter?.projects !== undefined));
+    });
+    const checkedContexts = new Set();
+    for (const { projectName, inferredExecutor } of eligiblePairs) {
+        const contextKey = hasRelevantProjectScopedEntry
+            ? `${projectName}\t${inferredExecutor}`
+            : `${inferredExecutor}`;
+        if (checkedContexts.has(contextKey)) {
+            continue;
+        }
+        checkedContexts.add(contextKey);
+        const opts = {
+            projectName,
+            projectNode: projectNodesByName[projectName],
+            // The migrated pair's residual carries no executor/command, so the
+            // specified layer (this plugin) owns the identity attribution.
+            sourcePlugin: pluginPath,
+        };
+        const before = (0, devkit_internals_1.readTargetDefaultsForTarget)(targetName, targetDefaults, inferredExecutor, opts);
+        const after = (0, devkit_internals_1.readTargetDefaultsForTarget)(targetName, hypotheticalNxJson.targetDefaults, inferredExecutor, opts);
+        // The reader merges a key's later entries on top of earlier ones, and the
+        // appended entry is last, so an unchanged resolution is exactly the prior
+        // result with the common merged over it.
+        const expected = before === null
+            ? structuredClone(common)
+            : (0, devkit_internals_1.mergeTargetConfigurations)(structuredClone(common), structuredClone(before));
+        // stableStringify rather than isDeepEqual: deepStrictEqual also compares
+        // prototype identity, which rejects structurally equal objects here.
+        if (stableStringify(after ?? {}) !== stableStringify(expected)) {
+            return true;
+        }
+    }
+    return false;
+}
+/**
+ * Phase 3 (centralized variant): hoist the strict-common residual per target
+ * into `nx.json` `targetDefaults[targetName]` as a plugin-scoped array entry,
+ * remove the dead executor-keyed entries, and write only per-project
+ * deviations to project.json. Used for whole-workspace migrations;
+ * single-project mode keeps the full residual. Hoisting also requires the
+ * plugin's registrations to form the tail of nx.json `plugins` (see
+ * `pluginRegistrationsFormTail`), and the appended
+ * entry to leave existing target-default resolution unchanged (see
+ * `hoistChangesExistingTargetDefaults`); otherwise the affected projects keep
+ * their full residuals.
+ *
+ * Returns the appended entry per target so the verification pass can revert a
+ * hoist that turns out to reach a target the migration did not migrate.
+ */
+function hoistCommonAndWrite(tree, projectConfigsByName, scope, residualByProject, pluginPath, projectGraph, inferredTargetsByOptionSet, inferredExecutors, pluginPreRegistered, logger) {
+    // Group residuals by target name across all migrated projects, tracking which
+    // projects contribute each target so we can inspect their default-layer config.
+    const residualsByTarget = new Map();
+    const projectsByTarget = new Map();
+    for (const [projectName, targetMap] of residualByProject) {
+        for (const [targetName, entry] of targetMap) {
+            if (!residualsByTarget.has(targetName)) {
+                residualsByTarget.set(targetName, []);
+                projectsByTarget.set(targetName, []);
+            }
+            residualsByTarget.get(targetName).push(entry.residual);
+            projectsByTarget.get(targetName).push(projectName);
+        }
+    }
+    const commonByTarget = new Map();
+    // Projects whose target identity is authored in a DEFAULT layer, per target.
+    // Nx's `resolveSourcePlugin` refuses a `filter: { plugin }` default for such a
+    // target (see nx/.../project-configuration/target-defaults.ts), so those
+    // projects can't inherit the hoisted keys and must carry the full config
+    // themselves. The partition is PER-PROJECT: one project with authored identity
+    // no longer blocks centralization for its siblings.
+    const excludedProjectsByTarget = new Map();
+    // package.json can reintroduce a migrated executor at resolution after the
+    // explicit target is rewritten, invisible to the liveness scan below (the
+    // package config is ignored when a sibling project.json exists).
+    let packageJsonAuthoredIdentity = false;
+    // Deterministic target-name order for stable nx.json output.
+    for (const targetName of [...residualsByTarget.keys()].sort()) {
+        const residuals = residualsByTarget.get(targetName);
+        const projects = projectsByTarget.get(targetName) ?? [];
+        // Partition the target's projects. A target's identity is authored in a
+        // default layer when either:
+        //   - the project.json residual carries `executor`/`command` (e.g. @nx/detox
+        //     stamps a per-project `command`), or
+        //   - the package-json plugin authors it: a package.json script byte-equal
+        //     to the target name becomes an `nx:run-script` target (and `nx.targets`
+        //     is honored).
+        // Such projects are EXCLUDED from the hoist (they keep the full residual,
+        // or the pre-migration target when package.json authors the identity, see
+        // `writeResidualTarget`); the rest are hoist-eligible.
+        const excludedProjects = new Set();
+        const eligibleResiduals = [];
+        for (let i = 0; i < projects.length; i++) {
+            const projectName = projects[i];
+            const residual = residuals[i];
+            const packageJsonAuthored = packageJsonAuthorsTargetIdentity(tree, projectConfigsByName.get(projectName)?.root, targetName);
+            if (packageJsonAuthored) {
+                packageJsonAuthoredIdentity = true;
+            }
+            const identityAuthored = residual.executor !== undefined ||
+                residual.command !== undefined ||
+                packageJsonAuthored;
+            if (identityAuthored) {
+                excludedProjects.add(projectName);
+            }
+            else {
+                eligibleResiduals.push(residual);
+            }
+        }
+        excludedProjectsByTarget.set(targetName, excludedProjects);
+        // Centralization only pays off when at least two ELIGIBLE projects share the
+        // target; the common is computed over eligible residuals only, so an excluded
+        // project's config never leaks into the shared default.
+        const common = eligibleResiduals.length >= 2
+            ? computeStrictCommon(eligibleResiduals)
+            : {};
+        commonByTarget.set(targetName, common);
+    }
+    // Drop the given targets' centralization (they keep full residuals) and say
+    // why. The retention is lossless: the resolved configuration is unchanged,
+    // only the deduplication into `targetDefaults` is skipped.
+    const retainResiduals = (targetNames, reason) => {
+        for (const targetName of targetNames) {
+            commonByTarget.set(targetName, {});
+        }
+        (logger ?? devkit_exports_1.logger).warn((0, conversion_warnings_1.retainedResidualsWarning)(targetNames, reason));
+    };
+    const centralizableTargets = () => [...commonByTarget.entries()]
+        .filter(([, common]) => Object.keys(common).length > 0)
+        .map(([targetName]) => targetName)
+        .sort();
+    // A specified plugin registered AFTER the migrated plugin merges later, so a
+    // same-name target it authors takes the executor/command attribution;
+    // `resolveSourcePlugin` then names that plugin and rejects this plugin's
+    // `filter: { plugin }` default, silently dropping the hoisted keys. The
+    // verification pass loads only the migrated plugin, so it cannot observe
+    // this. Registrations are written before this runs, so the plugins array is
+    // final for THIS generator invocation: hoist only when the migrated plugin's
+    // registrations form its tail; otherwise keep the full residuals (the
+    // previous engine's output). A batch runner invoking several conversions
+    // against one Tree appends more registrations after this one returns, where
+    // this tail check would pass on a plugins array that is not final; such
+    // invocations never reach this hoist, since a batch child defers
+    // centralization entirely to the finalize pass (see BatchConversionSession).
+    if (!pluginRegistrationsFormTail((0, devkit_exports_1.readNxJson)(tree), pluginPath)) {
+        const skippedTargets = centralizableTargets();
+        if (skippedTargets.length > 0) {
+            retainResiduals(skippedTargets, `another plugin is registered after ${pluginPath} in nx.json and may take over those targets`);
+        }
+    }
+    // Target-default preflight: even with the plugin merging last, appending the
+    // exact target-name entry can change what the EXISTING defaults resolve to
+    // for the migrated pairs (see `hoistChangesExistingTargetDefaults`). Checked
+    // per pair with the production reader; a rejected target keeps its full
+    // residuals. In-memory only, no inference or graph pass.
+    const preflightTargets = centralizableTargets();
+    if (preflightTargets.length > 0) {
+        const currentTargetDefaults = (0, devkit_exports_1.readNxJson)(tree).targetDefaults;
+        // The pair's effective executor decides whether an executor-keyed default
+        // outranks the appended exact key.
+        const inferredExecutorByPair = computeInferredExecutorByPair(scope, projectGraph, inferredTargetsByOptionSet);
+        const nonExactNameTargets = [];
+        const rejectedTargets = [];
+        for (const targetName of preflightTargets) {
+            // A `targetDefaults` key resolves as an EXECUTOR key for any target
+            // whose effective executor equals it (executor strings are plain
+            // strings, no `:` required), and as a glob key when it contains glob
+            // characters. So a candidate name that is executor-like (`a:b`),
+            // glob-like, or equal to ANY executor this plugin's inference emits
+            // cannot be hoisted: the appended key would apply the common to OTHER
+            // targets of this plugin, invisible to the per-pair check below (it
+            // resolves only the migrated pairs) and to the Phase 4 exposure revert
+            // (it looks only for this target name). Rejecting here also keeps the
+            // surviving candidates non-interacting, so checking each against the
+            // current map alone stays sound.
+            if (!(0, target_defaults_utils_1.isExactTargetNameKey)(targetName) ||
+                inferredExecutors.has(targetName)) {
+                nonExactNameTargets.push(targetName);
+                continue;
+            }
+            const excludedProjects = excludedProjectsByTarget.get(targetName) ?? new Set();
+            const eligiblePairs = (projectsByTarget.get(targetName) ?? [])
+                .filter((projectName) => !excludedProjects.has(projectName))
+                .map((projectName) => ({
+                projectName,
+                inferredExecutor: inferredExecutorByPair.get(`${projectName}\t${targetName}`),
+            }));
+            if (hoistChangesExistingTargetDefaults(currentTargetDefaults, targetName, commonByTarget.get(targetName), pluginPath, eligiblePairs, projectGraph.nodes)) {
+                rejectedTargets.push(targetName);
+            }
+        }
+        if (nonExactNameTargets.length > 0) {
+            retainResiduals(nonExactNameTargets, 'the target name would resolve as an executor or glob targetDefaults key and could apply to other targets');
+        }
+        if (rejectedTargets.length > 0) {
+            retainResiduals(rejectedTargets, 'centralization would change which existing targetDefaults apply');
+        }
+    }
+    // Write per-project deviations first so migrated targets drop their executor
+    // before we decide whether an executor-keyed target default is still needed.
+    for (const executorScope of scope.executorScopes) {
+        for (const [targetName, projectNames] of executorScope.targetAndProjects) {
+            const common = commonByTarget.get(targetName) ?? {};
+            const excludedProjects = excludedProjectsByTarget.get(targetName) ?? new Set();
+            for (const projectName of projectNames) {
+                const entry = residualByProject.get(projectName)?.get(targetName);
+                if (!entry) {
+                    continue;
+                }
+                // Excluded projects keep the FULL residual: the plugin-scoped default
+                // won't resolve for them, so nothing may be subtracted. Eligible projects
+                // drop the hoisted common and keep only their deviation.
+                const toWrite = excludedProjects.has(projectName)
+                    ? structuredClone(entry.residual)
+                    : subtractCommon(entry.residual, common);
+                entry.keptPreMigration = !writeResidualTarget(tree, projectConfigsByName, projectName, targetName, toWrite, logger);
+            }
+        }
+    }
+    // Update nx.json targetDefaults: hoist commons, drop dead executor entries.
+    const nxJson = (0, devkit_exports_1.readNxJson)(tree);
+    nxJson.targetDefaults ??= {};
+    const hoistedByTarget = new Map();
+    for (const targetName of [...commonByTarget.keys()].sort()) {
+        const common = commonByTarget.get(targetName);
+        if (Object.keys(common).length === 0) {
+            continue;
+        }
+        hoistedByTarget.set(targetName, appendPluginScopedTargetDefault(nxJson, targetName, pluginPath, common));
+    }
+    // A pre-existing registration of this plugin (or a plugin after it) can win
+    // a migrated pair's identity after the explicit target is rewritten, and a
+    // package-authored identity resurfaces the same way. A registration this
+    // migration ADDS is appended last and wins identity itself, so only fresh
+    // registrations remove dead executor entries; otherwise fail open.
+    // `pluginPreRegistered` is captured by the caller BEFORE registrations are
+    // written (they now precede this).
+    if (!pluginPreRegistered && !packageJsonAuthoredIdentity) {
+        // An executor-keyed targetDefault applies to any RESOLVED target carrying
+        // the executor, so liveness needs every source of one: post-write explicit
+        // targets, graph targets of untouched pairs (migrated pairs still show the
+        // pre-write executor), and everything this plugin's own inference emits.
+        const liveExecutors = new Set(inferredExecutors);
+        for (const projectConfig of projectConfigsByName.values()) {
+            for (const target of Object.values(projectConfig.targets ?? {})) {
+                if (target.executor) {
+                    liveExecutors.add(target.executor);
+                }
+            }
+        }
+        for (const [projectName, node] of Object.entries(projectGraph.nodes)) {
+            for (const [targetName, target] of Object.entries(node.data.targets ?? {})) {
+                if (target.executor &&
+                    !residualByProject.get(projectName)?.has(targetName)) {
+                    liveExecutors.add(target.executor);
+                }
+            }
+        }
+        for (const executorScope of scope.executorScopes) {
+            if (!liveExecutors.has(executorScope.executor)) {
+                removeDeadExecutorTargetDefault(nxJson, executorScope.executor);
+            }
+        }
+    }
+    if (nxJson.targetDefaults &&
+        Object.keys(nxJson.targetDefaults).length === 0) {
+        delete nxJson.targetDefaults;
+    }
+    (0, devkit_exports_1.updateNxJson)(tree, nxJson);
+    // A per-project exclusion is otherwise silent (unlike a Phase 4 revert, which
+    // warns): the excluded projects simply keep their per-project configuration
+    // and nothing is centralized for them. Surface it so a partial or total
+    // non-centralization is never indistinguishable from "centralization did not
+    // apply".
+    const excludedTargets = [...excludedProjectsByTarget.entries()].filter(([, projects]) => projects.size > 0);
+    if (excludedTargets.length > 0) {
+        const excludedProjectNames = [
+            ...new Set(excludedTargets.flatMap(([, projects]) => [...projects])),
+        ].sort();
+        const targetNames = excludedTargets
+            .map(([targetName]) => targetName)
+            .sort();
+        (logger ?? devkit_exports_1.logger).warn((0, conversion_warnings_1.excludedProjectsWarning)(excludedProjectNames, targetNames));
+    }
+    return hoistedByTarget;
+}
+function mergeInputs(target, inferredTarget) {
     const isInputInferred = (input) => {
         return inferredTarget.inputs.some((inferredInput) => {
             try {
@@ -132,73 +944,7 @@ _ExecutorToPluginMigrator_projectGraph = new WeakMap(), _ExecutorToPluginMigrato
             ...inferredTargetExternalDependencyInput.externalDependencies,
         ]));
     }
-}, _ExecutorToPluginMigrator_getTargetAndProjectsToMigrate = function _ExecutorToPluginMigrator_getTargetAndProjectsToMigrate() {
-    (0, executor_options_utils_1.forEachExecutorOptions)(this.tree, tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_executor, "f"), (options, projectName, targetName, configurationName) => {
-        if (tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_skippedProjects, "f").has(projectName) || configurationName) {
-            return;
-        }
-        if (tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_specificProjectToMigrate, "f") &&
-            projectName !== tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_specificProjectToMigrate, "f")) {
-            return;
-        }
-        const skipProjectReason = tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_skipProjectFilter, "f").call(this, tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_projectGraph, "f").nodes[projectName].data);
-        if (skipProjectReason) {
-            tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_skippedProjects, "f").add(projectName);
-            const errorMsg = `The "${projectName}" project cannot be migrated. ${skipProjectReason}`;
-            if (tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_specificProjectToMigrate, "f")) {
-                throw new Error(errorMsg);
-            }
-            tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_logger, "f").warn(errorMsg);
-            return;
-        }
-        const skipTargetReason = tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_skipTargetFilter, "f").call(this, options, tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_projectGraph, "f").nodes[projectName].data);
-        if (skipTargetReason) {
-            const errorMsg = `The ${targetName} target on project "${projectName}" cannot be migrated. ${skipTargetReason}`;
-            if (tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_specificProjectToMigrate, "f")) {
-                throw new Error(errorMsg);
-            }
-            else {
-                tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_logger, "f").warn(errorMsg);
-            }
-            return;
-        }
-        if (tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_targetAndProjectsToMigrate, "f").has(targetName)) {
-            tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_targetAndProjectsToMigrate, "f").get(targetName).add(projectName);
-        }
-        else {
-            tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_targetAndProjectsToMigrate, "f").set(targetName, new Set([projectName]));
-        }
-    });
-}, _ExecutorToPluginMigrator_getTargetDefaultsForExecutor = function _ExecutorToPluginMigrator_getTargetDefaultsForExecutor() {
-    tslib_1.__classPrivateFieldSet(this, _ExecutorToPluginMigrator_targetDefaultsForExecutor, structuredClone(readTargetDefaultsForExecutor(tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_executor, "f"), tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_nxJson, "f").targetDefaults) ?? {}), "f");
-}, _ExecutorToPluginMigrator_getCreatedTargetForProjectRoot = function _ExecutorToPluginMigrator_getCreatedTargetForProjectRoot(targetName, projectRoot) {
-    const entry = Object.entries(tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_createNodesResultsForTargets, "f").get(targetName)?.projects ?? {}).find(([root]) => root === projectRoot);
-    if (!entry) {
-        throw new Error(`The nx plugin did not find a project inside ${projectRoot}. File an issue at https://github.com/nrwl/nx with information about your project structure.`);
-    }
-    const createdProject = entry[1];
-    const createdTarget = structuredClone(createdProject.targets[targetName]);
-    delete createdTarget.command;
-    delete createdTarget.options?.cwd;
-    return createdTarget;
-}, _ExecutorToPluginMigrator_getCreateNodesResults = async function _ExecutorToPluginMigrator_getCreateNodesResults() {
-    if (tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_targetAndProjectsToMigrate, "f").size === 0) {
-        return;
-    }
-    global.NX_GRAPH_CREATION = true;
-    try {
-        for (const targetName of tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_targetAndProjectsToMigrate, "f").keys()) {
-            const result = await getCreateNodesResultsForPlugin(this.tree, {
-                plugin: tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_pluginPath, "f"),
-                options: tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_pluginOptionsBuilder, "f").call(this, targetName),
-            }, tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_pluginPath, "f"), tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_createNodes, "f"), tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_createNodesV2, "f"), tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_nxJson, "f"));
-            tslib_1.__classPrivateFieldGet(this, _ExecutorToPluginMigrator_createNodesResultsForTargets, "f").set(targetName, result);
-        }
-    }
-    finally {
-        global.NX_GRAPH_CREATION = false;
-    }
-};
+}
 class NoTargetsToMigrateError extends Error {
     constructor() {
         super('Could not find any targets to migrate.');
@@ -229,101 +975,739 @@ async function migrateProjectExecutorsToPluginV1(tree, projectGraph, pluginPath,
     const projects = await migrateProjects(tree, projectGraph, pluginPath, createNodes, undefined, defaultPluginOptions, migrations, specificProjectToMigrate);
     return projects;
 }
-async function migrateProjects(tree, projectGraph, pluginPath, createNodes, createNodesV2, defaultPluginOptions, migrations, specificProjectToMigrate, logger) {
-    const projects = new Map();
-    const spinner = devkit_internals_1.globalSpinner.start(`Calculating migration scope...`, pluginPath);
-    for (const migration of migrations) {
-        for (const executor of migration.executors) {
-            const migrator = new ExecutorToPluginMigrator(tree, projectGraph, executor, pluginPath, migration.targetPluginOptionMapper, migration.postTargetTransformer, createNodes, createNodesV2, specificProjectToMigrate, {
-                skipProjectFilter: migration.skipProjectFilter,
-                skipTargetFilter: migration.skipTargetFilter,
-            }, logger);
-            const result = await migrator.run();
-            // invert the result to have a map of projects to their targets
-            for (const [target, projectList] of result.entries()) {
-                for (const project of projectList) {
-                    if (!projects.has(project)) {
-                        projects.set(project, {});
+/**
+ * Phase 1: Infer (once per distinct option set). Runs a whole-workspace
+ * inference per distinct plugin-option set (usually one) instead of once per
+ * target and once per project. Builds `inferredTargetsByOptionSet` (option set
+ * id -> project root -> target name -> FULL inferred target; residual
+ * computation strips `command` / `options.cwd` at the point of use), which
+ * Phase 2 (`computeResidualByProject`) reads to compute residuals and
+ * `baselineFinal`; plus the matched config files owned by an inferred project
+ * root, which Phase 3's registration step reads for analytic include coverage.
+ */
+async function inferOncePerOptionSet(tree, pluginPath, createNodes, createNodesV2, nxJson, scope) {
+    const inferredTargetsByOptionSet = new Map();
+    const rawMatchedFiles = new Set();
+    const inferredRoots = new Set();
+    const erroredConfigFiles = new Set();
+    const inferredExecutors = new Set();
+    if (scope.optionSetGroups.length === 0) {
+        return {
+            inferredTargetsByOptionSet,
+            rawMatchedConfigFiles: [],
+            matchedConfigFiles: [],
+            erroredConfigFiles: [],
+            inferredExecutors,
+            inferredRoots,
+        };
+    }
+    global.NX_GRAPH_CREATION = true;
+    try {
+        for (const group of scope.optionSetGroups) {
+            const inferredByRoot = new Map();
+            inferredTargetsByOptionSet.set(group.id, inferredByRoot);
+            const { result, erroredConfigFiles: groupErroredConfigFiles } = await getCreateNodesResultsForPlugin(tree, { plugin: pluginPath, options: group.options }, pluginPath, createNodes, createNodesV2, nxJson);
+            for (const file of groupErroredConfigFiles) {
+                erroredConfigFiles.add(file);
+            }
+            // The plugin's glob pattern is option-independent, so every option set
+            // matches the same config files; union defensively.
+            for (const file of result?.matchingProjectFiles ?? []) {
+                rawMatchedFiles.add(file);
+            }
+            for (const [root, projectConfig] of Object.entries(result?.projects ?? {})) {
+                // Every root the plugin produces a project for. Config files whose
+                // owning root is NOT one of these produce no project and can be safely
+                // excluded by an `include` without changing the inferred set: this is
+                // why include-necessity must be judged against inferred roots, not the
+                // raw glob (which also matches package.json/project.json/etc.).
+                inferredRoots.add(root);
+                // Executor-keyed targetDefaults can apply to ANY inferred target name,
+                // so the Phase 3 dead-entry removal and the target-default preflight
+                // must see every EFFECTIVE executor: a command-based target resolves
+                // to `nx:run-commands` (resolveCommandSyntacticSugar).
+                for (const target of Object.values(projectConfig.targets ?? {})) {
+                    if (target.executor) {
+                        inferredExecutors.add(target.executor);
+                    }
+                    else if (target.command) {
+                        inferredExecutors.add('nx:run-commands');
                     }
-                    projects.set(project, {
-                        ...projects.get(project),
-                        ...migration.targetPluginOptionMapper(target),
-                    });
+                }
+                // Retain clones only for this option set's migrated roots: Phase 2
+                // reads them for migrated (project, target) pairs alone, so keeping
+                // every inferred root would retain up to (option sets x roots) targets
+                // for nothing.
+                if (!group.migratedRoots.has(root)) {
+                    continue;
+                }
+                for (const targetName of group.targetNames) {
+                    const inferredTarget = projectConfig.targets?.[targetName];
+                    if (!inferredTarget) {
+                        continue;
+                    }
+                    // Store the FULL inferred (command-based) target. Residual
+                    // computation strips `command`/`options.cwd` at the point of use;
+                    // `baselineFinal` (the equivalence oracle) needs the full form.
+                    if (!inferredByRoot.has(root)) {
+                        inferredByRoot.set(root, new Map());
+                    }
+                    inferredByRoot
+                        .get(root)
+                        .set(targetName, structuredClone(inferredTarget));
                 }
             }
         }
     }
-    // apply default options
-    for (const [project, pluginOptions] of projects.entries()) {
-        projects.set(project, {
-            ...defaultPluginOptions,
-            ...pluginOptions,
-        });
+    finally {
+        global.NX_GRAPH_CREATION = false;
     }
-    await addPluginRegistrations(tree, projects, pluginPath, createNodes, createNodesV2, defaultPluginOptions, projectGraph, spinner);
-    spinner.succeed(`Migrated configuration for ${projects.size} project(s).\n`);
-    return projects;
+    // Keep only config files owned by an inferred project root (i.e. files that
+    // actually contribute a project). Include-coverage is decided against these.
+    // Ownership is an O(path depth) ancestor walk against the root Set, not an
+    // O(roots) scan per file: both sets scale with project count.
+    const matchedConfigFiles = [...rawMatchedFiles].filter((file) => isFileOwnedByAnyRoot(file, inferredRoots));
+    return {
+        inferredTargetsByOptionSet,
+        // Every config file the plugin's glob matched, independent of whether this
+        // pass inferred a project from it. The registration step needs the full set
+        // to decide include coverage against non-migrated PROJECT roots: a project
+        // the plugin infers only on a later pass (e.g. cache/error asymmetry) is
+        // absent from `matchedConfigFiles` but its config is still globbed here.
+        rawMatchedConfigFiles: [...rawMatchedFiles],
+        matchedConfigFiles,
+        // Config files the plugin's glob matched but could not infer this pass.
+        // These produced no project (so they're absent from `matchedConfigFiles`),
+        // yet the registration step must still keep its `include` scoped when one
+        // sits outside the migrated roots, otherwise the plugin is widened to that
+        // root, re-hits the same failure at verification, and reverts the hoist.
+        erroredConfigFiles: [...erroredConfigFiles],
+        inferredExecutors,
+        inferredRoots,
+    };
 }
-async function addPluginRegistrations(tree, projects, pluginPath, createNodes, createNodesV2, defaultPluginOptions, projectGraph, spinner) {
+/**
+ * Whether `file` (workspace-relative) is owned by any root in `roots`. Walks the
+ * file's ancestor directories and checks Set membership: O(path depth), not
+ * O(roots). Equivalent to `roots.some((root) => isFileUnderRoot(file, root))`:
+ * the root project (`.`) owns workspace-root-level files (no path separator),
+ * and a nested root owns the file when it is (or is an ancestor directory of)
+ * the file.
+ */
+function isFileOwnedByAnyRoot(file, roots) {
+    if (roots.has(file)) {
+        return true;
+    }
+    const firstSlash = file.indexOf('/');
+    if (firstSlash === -1) {
+        // A workspace-root-level file is owned only by the root project.
+        return roots.has('.');
+    }
+    let dir = file;
+    let slash = dir.lastIndexOf('/');
+    while (slash !== -1) {
+        dir = dir.slice(0, slash);
+        if (roots.has(dir)) {
+            return true;
+        }
+        slash = dir.lastIndexOf('/');
+    }
+    return false;
+}
+// Minimatch metacharacters plus extglob prefix operators (`!+@()`). A path
+// segment containing any of these has glob semantics and cannot be reduced to a
+// literal project root.
+// Not reused from `isGlobPattern` (nx/src/utils/globs.ts): its `GLOB_CHARACTERS`
+// set omits `?`, `!`, `+`, and `@`, so it would treat an extglob/negation as a
+// literal root and wrongly widen the include.
+const GLOB_METACHARACTERS = /[*?[\]{}!+@()]/;
+/**
+ * The set of project roots a generated `include` list scopes to, or `undefined`
+ * if any entry is not one of the two shapes this generator emits: `*` (the root
+ * project) or a literal root followed by a trailing globstar segment (a nested
+ * root). A user-authored include or any `exclude` falls back to the glob engine.
+ *
+ * The globstar branch only qualifies when the root prefix is a LITERAL path.
+ * A prefix carrying glob metacharacters (a wildcard, brace, or extglob segment
+ * before the trailing globstar) is not a shape this generator emits and can't be
+ * reduced to root ownership by string equality, so it falls back to the glob
+ * engine.
+ */
+function generatedIncludeRoots(include) {
+    const roots = new Set();
+    for (const glob of include) {
+        if (glob === '*') {
+            roots.add('.');
+        }
+        else if (glob.endsWith('/**/*') && glob.length > '/**/*'.length) {
+            const root = glob.slice(0, -'/**/*'.length);
+            if (GLOB_METACHARACTERS.test(root)) {
+                return undefined;
+            }
+            roots.add(root);
+        }
+        else {
+            return undefined;
+        }
+    }
+    return roots;
+}
+/**
+ * Whether a registration's `include` globs already cover every config file the
+ * plugin globs that is owned by an inferred project root (so the registration
+ * can be left unscoped). Plugin inference is a pure function of the matched
+ * config-file set, so this answers what the old per-project
+ * `arePluginIncludesRequired` re-inference computed without running any
+ * additional inference, differing only on matched files that contribute no
+ * project: those are invisible here, so an `include` the old diff-based check
+ * kept for their sake is now deleted, letting future config files in such
+ * locations infer eagerly. The Phase 4 verification pass guards the current
+ * behavioral outcome either way.
+ *
+ * Every glob this generator emits scopes to a single root (`*` for the root
+ * project, a nested-root globstar for the rest), so coverage reduces to root
+ * ownership with no glob engine: an O(files * path depth) walk instead of
+ * running minimatch (whose `#matchGlobstar`/`slashSplit` dominate at scale) per
+ * (file, glob). Only user-authored includes or any `exclude` fall through.
+ */
+function includeCoversAllConfigFiles(include, exclude, configFiles) {
+    if (!include || include.length === 0) {
+        return true;
+    }
+    if (!exclude || exclude.length === 0) {
+        const includeRoots = generatedIncludeRoots(include);
+        if (includeRoots) {
+            return configFiles.every((file) => isFileOwnedByAnyRoot(file, includeRoots));
+        }
+    }
+    // Fallback for user-authored includes / any exclude: defer to the SAME matcher
+    // Nx uses to bind config files to a registration. A hand-rolled
+    // `include.some() && !exclude.some()` cannot express an ordered-override
+    // negation like `["packages/**/*", "!packages/legacy/**/*"]`: `.some()` reports
+    // every file "covered" and deletes the include, which widens the registration
+    // so the fenced-off configs get inferred (and executed). `findMatchingConfigFiles`
+    // applies include/exclude with Nx's exact ordered semantics, so coverage holds
+    // only when every config file survives the same filter Nx would run.
+    return ((0, devkit_internals_1.findMatchingConfigFiles)(configFiles, include, exclude ?? []).length ===
+        configFiles.length);
+}
+/**
+ * Harvest the diagnostic messages and the errored config-file paths from a
+ * `ProjectConfigurationsError`. `ProjectConfigurationsError.errors` is a closed
+ * 5-member union; the two members that name a failing config file are
+ * `AggregateCreateNodesError` (a `[file, error]` list) and `MergeNodesError` (a
+ * single `.file`). `ProjectsWithNoNameError` / `MultipleProjectsWithSameNameError`
+ * are artifacts of running with no `project.json` layer (nothing supplies names),
+ * so they are dropped; `WorkspaceValidityError` carries no file and is exempt.
+ */
+function harvestConfigurationErrors(e) {
+    const entries = [];
+    for (const error of e.errors) {
+        if ((0, devkit_internals_1.isProjectsWithNoNameError)(error) ||
+            (0, devkit_internals_1.isMultipleProjectsWithSameNameError)(error)) {
+            continue;
+        }
+        const message = error.message ?? String(error);
+        if ((0, devkit_internals_1.isAggregateCreateNodesError)(error)) {
+            entries.push({
+                message,
+                files: error.errors
+                    .map(([file]) => file)
+                    .filter((file) => !!file),
+                pluginIndex: error.pluginIndex,
+            });
+        }
+        else if ((0, devkit_internals_1.isMergeNodesError)(error)) {
+            entries.push({
+                message,
+                files: error.file ? [error.file] : [],
+                pluginIndex: error.pluginIndex,
+            });
+        }
+        else if ((0, devkit_internals_1.isWorkspaceValidityError)(error)) {
+            // Carries no config file and no plugin attribution.
+            entries.push({ message, files: [], pluginIndex: undefined });
+        }
+        else {
+            // Exhaustiveness guard: `ProjectConfigurationsError.errors` is a closed
+            // union. If a sixth member is added, this assignment stops compiling until
+            // it is handled here.
+            const _exhaustive = error;
+            void _exhaustive;
+        }
+    }
+    return {
+        messages: entries.map((entry) => entry.message),
+        erroredConfigFiles: [...new Set(entries.flatMap((entry) => entry.files))],
+        entries,
+    };
+}
+/**
+ * Attribution map for errored-config ownership: every root the engine knows
+ * owns a project, i.e. graph project roots plus the roots the plugin inference
+ * produced (a project discovered from a config file alone has no graph node;
+ * graph roots win a collision). Keys are normalized for the
+ * `findProjectForPath` walk; values are the raw roots the per-target
+ * migrated-root sets hold.
+ */
+function buildOwnerRootByPath(graphRoots, inferredRoots) {
+    const normalizeRoot = (root) => root === '' ? '.' : root.endsWith('/') ? root.slice(0, -1) : root;
+    const ownerRootByPath = new Map();
+    for (const root of graphRoots) {
+        ownerRootByPath.set(normalizeRoot(root), root);
+    }
+    for (const root of inferredRoots) {
+        if (!ownerRootByPath.has(normalizeRoot(root))) {
+            ownerRootByPath.set(normalizeRoot(root), root);
+        }
+    }
+    return ownerRootByPath;
+}
+/**
+ * Phase 4: a single verification inference pass over the whole workspace with
+ * the updated `nx.json` plugin registrations. One
+ * `retrieveProjectConfigurations` call runs every registration for this plugin
+ * (the plugin's `createNodes` executes once per registration group). The
+ * equivalence oracle + fallback in `verifyAndFallback` consume the result.
+ */
+async function runVerificationPass(tree, pluginPath, createNodes, createNodesV2) {
     const nxJson = (0, devkit_exports_1.readNxJson)(tree);
-    // collect createNodes results for each project before adding the plugins
-    const createNodesResults = new Map();
+    const registrations = (nxJson.plugins ?? []).filter((plugin) => plugin === pluginPath ||
+        (typeof plugin !== 'string' && plugin.plugin === pluginPath));
+    if (registrations.length === 0) {
+        return { result: undefined, errors: [], erroredConfigFiles: [] };
+    }
     global.NX_GRAPH_CREATION = true;
     try {
-        let index = 0;
-        for (const [project, options] of projects.entries()) {
-            index++;
-            spinner.updateText(`${index}/${projects.size} - Parsing "${project}" configuration...`);
-            const projectConfigs = await getCreateNodesResultsForPlugin(tree, { plugin: pluginPath, options }, pluginPath, createNodes, createNodesV2, nxJson);
-            createNodesResults.set(project, projectConfigs);
+        const plugins = registrations.map((registration) => new devkit_internals_1.LoadedNxPlugin({ createNodes, createNodesV2, name: pluginPath }, registration));
+        return {
+            result: await (0, devkit_internals_1.retrieveProjectConfigurations)({ specifiedPlugins: plugins, defaultPlugins: [] }, tree.root, nxJson),
+            errors: [],
+            erroredConfigFiles: [],
+        };
+    }
+    catch (e) {
+        if (e instanceof devkit_internals_1.ProjectConfigurationsError) {
+            // Verify against the partial result, but keep the causes: they are the
+            // only diagnostic for why affected projects fall back or a hoist reverts.
+            // Recording the errored config files is what makes the revert fail-closed:
+            // a hoist is reverted when an errored file lies outside every migrated root.
+            const { messages, erroredConfigFiles } = harvestConfigurationErrors(e);
+            return {
+                result: e.partialProjectConfigurationsResult,
+                errors: messages,
+                erroredConfigFiles,
+            };
         }
+        throw e;
     }
     finally {
         global.NX_GRAPH_CREATION = false;
     }
-    const arePluginIncludesRequired = async (project, pluginConfiguration) => {
-        global.NX_GRAPH_CREATION = true;
-        let result;
-        try {
-            result = await getCreateNodesResultsForPlugin(tree, pluginConfiguration, pluginPath, createNodes, createNodesV2, nxJson);
+}
+/**
+ * Phase 4: run the single verification inference pass, then apply the
+ * equivalence oracle. `retrieveProjectConfigurations` already merges
+ * `targetDefaults` into the inferred targets, so the real post-migration
+ * effective config for a target is approximated by `merge(project.json
+ * deviation, verified inferred+targetDefaults)`. It must deep-equal
+ * `baselineFinal` (the previous engine's migrated effective config, from Phase
+ * 2). CAVEAT: this pass runs with no DEFAULT layer (neither `project.json` nor
+ * `package.json`), so it cannot observe cases where a default layer authors a
+ * target's identity and thereby changes whether a `targetDefaults` entry applies
+ * at all, e.g. a `filter: { plugin }` entry is rejected by
+ * `resolveSourcePlugin` once the target carries `executor` / `command` in a
+ * default layer. That identity can come from the project.json residual itself OR
+ * from the package-json plugin (a script or `nx.targets` entry byte-equal to the
+ * target name). Those targets are kept per-project at the hoist site (they never
+ * reach a plugin-scoped entry), not caught here. Any project that fails,
+ * or that the intended target no longer infers for at all, is restored to the
+ * exact pre-centralization migration output (a full residual; an empty
+ * residual removes the target), and every fallback is summarized in a single
+ * `logger.warn` that asks for manual review rather than asserting equivalence
+ * the pass could not establish.
+ */
+async function verifyAndFallback(tree, projectGraph, pluginPath, createNodes, createNodesV2, residualByProject, projectConfigsByName, hoistedByTarget, inferredRoots, singleProjectMode, logger) {
+    // Single-project mode never hoists, so there is nothing to reconcile, and
+    // no reason to pay for a whole-workspace verification inference.
+    if (singleProjectMode) {
+        return;
+    }
+    const { result: verifyResult, errors: verificationErrors, erroredConfigFiles, } = await runVerificationPass(tree, pluginPath, createNodes, createNodesV2);
+    if (!verifyResult) {
+        return;
+    }
+    // The hoist's contract is: behavioral equivalence for migrated targets, no
+    // effect on anything else. A root this plugin infers the target for that was
+    // NOT migrated (inferred-only, or covered by a pre-existing registration)
+    // would inherit the plugin-scoped default, so revert that target's hoist
+    // entirely: drop the appended entry and restore every full residual.
+    const migratedRootsByTarget = new Map();
+    for (const [projectName, targetMap] of residualByProject) {
+        const root = projectGraph.nodes[projectName]?.data?.root;
+        for (const targetName of targetMap.keys()) {
+            if (!migratedRootsByTarget.has(targetName)) {
+                migratedRootsByTarget.set(targetName, new Set());
+            }
+            migratedRootsByTarget.get(targetName).add(root);
         }
-        finally {
-            global.NX_GRAPH_CREATION = false;
+    }
+    const ownerRootByPath = buildOwnerRootByPath(Object.values(projectGraph.nodes).map((node) => node.data.root), inferredRoots);
+    const revertedTargets = new Set();
+    for (const [targetName] of hoistedByTarget) {
+        const migratedRoots = migratedRootsByTarget.get(targetName) ?? new Set();
+        const reachesNonMigratedRoot = Object.entries(verifyResult.projects ?? {}).some(([root, projectConfig]) => projectConfig.targets?.[targetName] !== undefined &&
+            !migratedRoots.has(root));
+        // Fail closed on inference errors. `reachesNonMigratedRoot` is keyed on the
+        // partial result, so an errored root never appears there, yet it may be an
+        // inferred-only root that would inherit the plugin-scoped default once its
+        // config is fixed. Attribute each errored file to the CLOSEST known
+        // project root (not any migrated ancestor: a nested non-migrated project
+        // must not be absorbed by the migrated project above it) and revert unless
+        // that root is one of this target's migrated roots (errors on migrated
+        // roots are handled by the per-project divergence oracle below).
+        const erroredOutsideMigratedRoots = erroredConfigFiles.some((file) => {
+            const ownerRoot = (0, devkit_internals_1.findProjectForPath)(file, ownerRootByPath);
+            return ownerRoot == null || !migratedRoots.has(ownerRoot);
+        });
+        if (reachesNonMigratedRoot || erroredOutsideMigratedRoots) {
+            revertedTargets.add(targetName);
         }
-        const originalResults = createNodesResults.get(project);
-        return !deepEqual(originalResults, result);
-    };
-    let index = 0;
+    }
+    if (revertedTargets.size > 0) {
+        const nxJson = (0, devkit_exports_1.readNxJson)(tree);
+        for (const targetName of revertedTargets) {
+            removeHoistedTargetDefault(nxJson, targetName, hoistedByTarget.get(targetName));
+        }
+        (0, devkit_exports_1.updateNxJson)(tree, nxJson);
+        for (const [projectName, targetMap] of residualByProject) {
+            for (const [targetName, entry] of targetMap) {
+                if (!revertedTargets.has(targetName) || entry.keptPreMigration) {
+                    continue;
+                }
+                writeResidualTarget(tree, projectConfigsByName, projectName, targetName, structuredClone(entry.residual), logger);
+            }
+        }
+        // A revert can be triggered by an inference error the pass could not
+        // inspect; surface those errors so the incomplete verification is not silent.
+        (logger ?? devkit_exports_1.logger).warn((0, conversion_warnings_1.revertedTargetsWarning)([...revertedTargets].sort(), verificationErrors));
+    }
+    const divergent = [];
+    const missing = [];
+    for (const [projectName, targetMap] of residualByProject) {
+        const root = projectGraph.nodes[projectName]?.data?.root;
+        for (const [targetName, entry] of targetMap) {
+            if (revertedTargets.has(targetName) || entry.keptPreMigration) {
+                // Restored to the full pre-centralization residual above (or never
+                // migrated): already the intended output, nothing left to verify.
+                continue;
+            }
+            const verifiedInferred = verifyResult.projects?.[root]?.targets?.[targetName];
+            let equivalent = false;
+            if (verifiedInferred) {
+                const projectConfig = readCachedProjectConfiguration(tree, projectConfigsByName, projectName);
+                const deviation = projectConfig.targets?.[targetName] ?? {};
+                const postMigrationFinal = (0, devkit_internals_1.mergeTargetConfigurations)(structuredClone(deviation), structuredClone(verifiedInferred));
+                equivalent = isDeepEqual(postMigrationFinal, entry.baselineFinal);
+            }
+            if (!equivalent) {
+                // Drop this project's hoist: restore its full residual as an explicit
+                // project.json override (which wins over the shared targetDefaults).
+                const written = writeResidualTarget(tree, projectConfigsByName, projectName, targetName, structuredClone(entry.residual), logger);
+                const pair = `${projectName} > ${targetName}`;
+                if (verifiedInferred) {
+                    divergent.push(pair);
+                }
+                else {
+                    missing.push({
+                        pair,
+                        root,
+                        removed: written && (0, conversion_warnings_1.isDroppedTarget)(entry.residual),
+                    });
+                }
+            }
+        }
+    }
+    const anyFallback = divergent.length > 0 || missing.length > 0;
+    if (anyFallback) {
+        // Surface the pass's own errors only when a target vanished from the
+        // verification result: the one fallback class an inference error can
+        // explain. Divergence fallbacks have a verified config; appending
+        // unrelated pass errors to them would misattribute the cause.
+        (logger ?? devkit_exports_1.logger).warn((0, conversion_warnings_1.unverifiedPairsWarning)(divergent, missing, missing.length > 0 ? verificationErrors : []));
+    }
+    // Whenever the verification pass reported errors they must appear in at least
+    // one warning. The revert warning always carries them; the fallback warning
+    // carries them only for a target missing from the verification result (a
+    // divergence fallback has a verified config, so attaching unrelated errors to
+    // it would misattribute the cause). Fire this standalone warning whenever the
+    // errors were surfaced by neither, i.e. nothing reverted AND no fallback
+    // carried them (no fallback at all, or only divergence fallbacks). Otherwise a
+    // broken config file leaves the pass unable to see the whole workspace with no
+    // trace. (`verificationErrors` here has already had the no-project-name noise
+    // filtered out, so it means inference genuinely broke.)
+    const errorsSurfacedByFallbackWarning = missing.length > 0;
+    if (verificationErrors.length > 0 &&
+        revertedTargets.size === 0 &&
+        !errorsSurfacedByFallbackWarning) {
+        // Only claim the migrated targets matched when nothing fell back; a
+        // divergence fallback means at least one did not.
+        (logger ?? devkit_exports_1.logger).warn((0, conversion_warnings_1.verificationErrorsWarning)(verificationErrors, anyFallback));
+    }
+}
+/**
+ * Recover the default option keys a plugin fills into its options object during
+ * `createNodes` (Phase 1 mutated each option-set object in place if the plugin
+ * does so). A key qualifies only if it is not one of our own
+ * `defaultPluginOptions` and appears with an identical value across every
+ * option set: the signature of a plugin default fill. (A mapper-provided key
+ * that is constant across every option set is indistinguishable and also
+ * qualifies; harmless, since per-project options are spread over these.)
+ * Reproduces the previous engine's incidental option enrichment without extra
+ * inference.
+ */
+function derivePluginFilledDefaults(optionSetGroups, defaultPluginOptions) {
+    const filled = {};
+    if (optionSetGroups.length === 0) {
+        return filled;
+    }
+    const defaults = (defaultPluginOptions ?? {});
+    const seen = new Map();
+    for (const group of optionSetGroups) {
+        for (const [key, value] of Object.entries((group.options ?? {}))) {
+            if (key in defaults) {
+                continue;
+            }
+            const existing = seen.get(key);
+            if (!existing) {
+                seen.set(key, { value, count: 1, consistent: true });
+            }
+            else {
+                existing.count++;
+                if (stableStringify(existing.value) !== stableStringify(value)) {
+                    existing.consistent = false;
+                }
+            }
+        }
+    }
+    for (const [key, entry] of seen) {
+        if (entry.consistent && entry.count === optionSetGroups.length) {
+            filled[key] = entry.value;
+        }
+    }
+    return filled;
+}
+/**
+ * Deferred (batch) variant of Phase 3's hoist + Phase 4: instead of running
+ * them, capture everything the batch finalize pass needs to run them once over
+ * the whole batch. The session clones the mutable structures at staging time.
+ */
+function stageDeferredPlan(batchStaging, pluginPath, createNodes, createNodesV2, logger, pluginPreRegistered, scope, projectGraph, residualByProject, inferredTargetsByOptionSet, inferredExecutors, inferredRoots, matchedConfigFiles, erroredConfigFiles) {
+    const rootByProject = new Map();
+    const graphNodeByProject = new Map();
+    for (const projectName of residualByProject.keys()) {
+        rootByProject.set(projectName, projectGraph.nodes[projectName]?.data.root);
+        const node = projectGraph.nodes[projectName];
+        if (node) {
+            graphNodeByProject.set(projectName, node);
+        }
+    }
+    const graphRoots = new Set();
+    for (const node of Object.values(projectGraph.nodes)) {
+        graphRoots.add(node.data.root);
+    }
+    // Executors of the pre-migration graph targets, for the finalize liveness
+    // scan (mirrors the inline scan in `hoistCommonAndWrite`, which reads them
+    // from the same graph).
+    const graphExecutorByPair = new Map();
+    for (const [projectName, node] of Object.entries(projectGraph.nodes)) {
+        for (const [targetName, target] of Object.entries(node.data.targets ?? {})) {
+            if (target.executor) {
+                graphExecutorByPair.set(`${projectName}\t${targetName}`, target.executor);
+            }
+        }
+    }
+    batchStaging.stagePlan({
+        pluginPath,
+        createNodes,
+        createNodesV2,
+        logger,
+        pluginPreRegistered,
+        residualByProject,
+        rootByProject,
+        graphNodeByProject,
+        graphRoots,
+        inferredExecutorByPair: computeInferredExecutorByPair(scope, projectGraph, inferredTargetsByOptionSet),
+        inferredExecutors,
+        inferredRoots,
+        matchedConfigFiles,
+        erroredConfigFiles,
+        migratedExecutors: scope.executorScopes.map((executorScope) => executorScope.executor),
+        graphExecutorByPair,
+    });
+}
+async function migrateProjects(tree, projectGraph, pluginPath, createNodes, createNodesV2, defaultPluginOptions, migrations, specificProjectToMigrate, logger) {
+    const projects = new Map();
+    const spinner = devkit_internals_1.globalSpinner.start(`Calculating migration scope...`, pluginPath);
+    // Phase 0: Collect (once). Scope + filtering + per-project options.
+    const scope = collectMigrationScope(tree, projectGraph, migrations, defaultPluginOptions, specificProjectToMigrate, logger);
+    const projectConfigsByName = (0, devkit_exports_1.getProjects)(tree);
+    // Phase 1: Infer (once per distinct option set) for the whole workspace.
+    const nxJson = (0, devkit_exports_1.readNxJson)(tree);
+    const { inferredTargetsByOptionSet, matchedConfigFiles, erroredConfigFiles: erroredConfigFilesFromInference, rawMatchedConfigFiles, inferredExecutors, inferredRoots, } = await inferOncePerOptionSet(tree, pluginPath, createNodes, createNodesV2, nxJson, scope);
+    // Phase 2: Per-project residual (in-memory, no re-inference). Also captures
+    // `baselineFinal` (the equivalence oracle consumed in Phase 4).
+    const residualByProject = await computeResidualByProject(tree, projectGraph, scope, inferredTargetsByOptionSet, nxJson, projectConfigsByName);
+    // Some plugins' `createNodes` normalize their options object in place (e.g.
+    // `options.devTargetName ??= 'dev'`). The previous engine inferred once per
+    // project passing the project's registration-options object by reference, so
+    // those objects picked up the plugin's default keys. Phase 1 runs the same
+    // inference on each option-set object, so recover exactly those plugin-filled
+    // default keys (without any extra inference) and merge them back so the
+    // emitted registration options stay identical.
+    const pluginFilledDefaults = derivePluginFilledDefaults(scope.optionSetGroups, defaultPluginOptions);
+    for (const [project, options] of scope.pluginOptionsByProject) {
+        projects.set(project, {
+            ...pluginFilledDefaults,
+            ...options,
+        });
+    }
+    // Phase 3: one-shot plugin registration + analytic include computation.
+    // Runs before the hoist: the hoist is gated on the FINAL position of this
+    // plugin's registrations in the plugins array (a later registration of
+    // another plugin can take a target's identity over), so the array must be in
+    // its final state first. The pre-registration state is captured here for the
+    // hoist's dead-entry removal gate, which needs to know whether this
+    // migration added the registration.
+    const pluginPreRegistered = ((0, devkit_exports_1.readNxJson)(tree).plugins ?? []).some((plugin) => isRegistrationOfPlugin(plugin, pluginPath));
+    addPluginRegistrations(tree, projects, pluginPath, defaultPluginOptions, projectGraph, spinner, matchedConfigFiles, erroredConfigFilesFromInference, rawMatchedConfigFiles);
+    // An open batch session (`infer-targets` with several plugins selected)
+    // defers centralization: full residuals are written now and the hoist,
+    // dead-default cleanup, and verification run once at the batch finalize
+    // pass, over every conversion's staged plan. Single-project mode never
+    // centralizes, so it stays on its inline path even inside a batch.
+    const batchStaging = specificProjectToMigrate
+        ? undefined
+        : (0, batch_conversion_session_1.getActiveBatchStaging)(tree);
+    // Phase 3: Derive strict-common + write. Single-project mode never hoists
+    // (would leak shared config to sibling projects), so it keeps the full
+    // residual in project.json, as does a deferred batch conversion (the batch
+    // finalize pass hoists later); whole-workspace mode hoists the common config
+    // to a plugin-scoped `targetDefaults` entry and writes only per-project
+    // deviations.
+    let hoistedByTarget = new Map();
+    if (specificProjectToMigrate || batchStaging) {
+        writeResiduals(tree, projectConfigsByName, projectGraph, scope, residualByProject, logger);
+    }
+    else {
+        hoistedByTarget = hoistCommonAndWrite(tree, projectConfigsByName, scope, residualByProject, pluginPath, projectGraph, inferredTargetsByOptionSet, inferredExecutors, pluginPreRegistered, logger);
+    }
+    if (batchStaging) {
+        stageDeferredPlan(batchStaging, pluginPath, createNodes, createNodesV2, logger, pluginPreRegistered, scope, projectGraph, residualByProject, inferredTargetsByOptionSet, inferredExecutors, inferredRoots, matchedConfigFiles, erroredConfigFilesFromInference);
+    }
+    else {
+        // Phase 4: single verification inference pass + equivalence oracle. Any
+        // project whose centralized config cannot be verified as equivalent falls
+        // back to a full project.json override (summarized in one logger.warn).
+        await verifyAndFallback(tree, projectGraph, pluginPath, createNodes, createNodesV2, residualByProject, projectConfigsByName, hoistedByTarget, inferredRoots, Boolean(specificProjectToMigrate), logger);
+    }
+    spinner.succeed(`Migrated configuration for ${projects.size} project(s).\n`);
+    return projects;
+}
+function addPluginRegistrations(tree, projects, pluginPath, defaultPluginOptions, projectGraph, spinner, 
+// The matched config files owned by an inferred project root (the filtered
+// subset from the Phase 1 inference: files outside any inferred root
+// contribute no project and are deliberately excluded). Used to decide
+// analytically whether a registration's `include` globs already cover
+// everything (so it can be left unscoped).
+matchedConfigFiles, 
+// Config files the Phase 1 inference could not load. These produce no project
+// (so they're absent from `matchedConfigFiles`), but an errored file OUTSIDE
+// the migrated roots must still keep the `include` scoped: leaving the plugin
+// unscoped would widen it onto that root, where the verification pass re-hits
+// the failure and reverts the whole hoist. A harmless project-free config (a
+// shared/base config that loads fine) is NOT errored, so it does not keep the
+// include: the plugin may safely apply workspace-wide over it.
+erroredConfigFiles, 
+// The FULL set of config files the plugin's glob matched, regardless of what
+// this migration's inference pass produced a project from. Used to detect
+// config files owned by NON-MIGRATED projects that the plugin would infer once
+// the registration is workspace-wide, see below.
+rawMatchedConfigFiles) {
+    const nxJson = (0, devkit_exports_1.readNxJson)(tree);
+    // Config files owned by a NON-MIGRATED project keep the `include` scoped.
+    // `matchedConfigFiles` only lists roots THIS pass inferred; a non-migrated
+    // project the plugin infers only later (cache/error asymmetry) is absent from
+    // it, so an unscoped registration would let the verification pass infer that
+    // root and `reachesNonMigratedRoot` would revert the whole hoist. Judging
+    // coverage against project-graph membership, not this pass's inference,
+    // closes that hole. (A config owned by NO project, e.g. a shared/base config,
+    // is not counted, so it does not force a scoped include.)
+    const migratedRoots = new Set();
+    for (const project of projects.keys()) {
+        const root = projectGraph.nodes[project]?.data?.root;
+        if (root) {
+            migratedRoots.add(root);
+        }
+    }
+    const nonMigratedProjectRoots = new Set();
+    for (const node of Object.values(projectGraph.nodes)) {
+        const root = node?.data?.root;
+        if (root && !migratedRoots.has(root)) {
+            nonMigratedProjectRoots.add(root);
+        }
+    }
+    const nonMigratedProjectConfigFiles = rawMatchedConfigFiles.filter((file) => isFileOwnedByAnyRoot(file, nonMigratedProjectRoots));
+    // Fold errored files (broken configs the plugin would fail to infer) and
+    // non-migrated project configs (roots the plugin would infer workspace-wide)
+    // into the coverage set: an unscoped `include` is chosen only when neither lies
+    // outside the migrated roots.
+    const coverageConfigFiles = [
+        ...matchedConfigFiles,
+        ...erroredConfigFiles,
+        ...nonMigratedProjectConfigFiles,
+    ];
+    const registrationGroups = new Map();
     for (const [project, options] of projects.entries()) {
+        const projectIncludeGlob = projectGraph.nodes[project].data.root === '.'
+            ? '*'
+            : (0, posix_1.join)(projectGraph.nodes[project].data.root, '**/*');
+        const key = stableStringify(options);
+        if (!registrationGroups.has(key)) {
+            registrationGroups.set(key, { options, include: [] });
+        }
+        registrationGroups.get(key).include.push(projectIncludeGlob);
+    }
+    let index = 0;
+    for (const { options, include } of registrationGroups.values()) {
         index++;
-        spinner.updateText(`${index}/${projects.size} - Applying "${project}" configuration...`);
+        spinner.updateText(`${index}/${registrationGroups.size} - Applying plugin configuration...`);
         const existingPlugin = nxJson.plugins?.find((plugin) => typeof plugin !== 'string' &&
             plugin.plugin === pluginPath &&
             Object.keys(options).every((key) => plugin.options[key] === options[key] ||
                 (plugin.options[key] === undefined &&
                     options[key] === defaultPluginOptions[key])));
-        const projectIncludeGlob = projectGraph.nodes[project].data.root === '.'
-            ? '*'
-            : (0, posix_1.join)(projectGraph.nodes[project].data.root, '**/*');
         if (!existingPlugin) {
             nxJson.plugins ??= [];
             const plugin = {
                 plugin: pluginPath,
                 options,
-                include: [projectIncludeGlob],
+                include: [...include],
             };
-            if (!(await arePluginIncludesRequired(project, plugin))) {
+            if (includeCoversAllConfigFiles(plugin.include, plugin.exclude, coverageConfigFiles)) {
                 delete plugin.include;
             }
             nxJson.plugins.push(plugin);
         }
         else if (existingPlugin.include) {
-            if (!existingPlugin.include.some((include) => (0, minimatch_1.minimatch)(projectIncludeGlob, include, { dot: true }))) {
-                existingPlugin.include.push(projectIncludeGlob);
-                if (!(await arePluginIncludesRequired(project, existingPlugin))) {
-                    delete existingPlugin.include;
+            for (const projectIncludeGlob of include) {
+                if (!existingPlugin.include.some((include) => (0, minimatch_1.minimatch)(projectIncludeGlob, include, { dot: true }))) {
+                    existingPlugin.include.push(projectIncludeGlob);
                 }
             }
+            if (includeCoversAllConfigFiles(existingPlugin.include, existingPlugin.exclude, coverageConfigFiles)) {
+                delete existingPlugin.include;
+            }
         }
     }
     spinner.updateText(`Migrations done`);
@@ -331,6 +1715,7 @@ async function addPluginRegistrations(tree, projects, pluginPath, createNodes, c
 }
 async function getCreateNodesResultsForPlugin(tree, pluginConfiguration, pluginPath, createNodes, createNodesV2, nxJson) {
     let projectConfigs;
+    let erroredConfigFiles = [];
     try {
         const plugin = new devkit_internals_1.LoadedNxPlugin({ createNodes, createNodesV2, name: pluginPath }, pluginConfiguration);
         projectConfigs = await (0, devkit_internals_1.retrieveProjectConfigurations)({ specifiedPlugins: [plugin], defaultPlugins: [] }, tree.root, nxJson);
@@ -338,33 +1723,15 @@ async function getCreateNodesResultsForPlugin(tree, pluginConfiguration, pluginP
     catch (e) {
         if (e instanceof devkit_internals_1.ProjectConfigurationsError) {
             projectConfigs = e.partialProjectConfigurationsResult;
+            // Capture the config files this pass could not infer. The registration
+            // step keeps its `include` scoped to migrated roots when an errored file
+            // sits outside them, so the plugin is never widened to a root it would
+            // fail to infer (which the verification pass would otherwise revert).
+            erroredConfigFiles = harvestConfigurationErrors(e).erroredConfigFiles;
         }
         else {
             throw e;
         }
     }
-    return projectConfigs;
-}
-// Checks if two objects are structurely equal, without caring
-// about the order of the keys.
-function deepEqual(a, b, logKey = '') {
-    const aKeys = Object.keys(a);
-    const bKeys = new Set(Object.keys(b));
-    if (aKeys.length !== bKeys.size) {
-        return false;
-    }
-    for (const key of aKeys) {
-        if (!bKeys.has(key)) {
-            return false;
-        }
-        if (typeof a[key] === 'object' && typeof b[key] === 'object') {
-            if (!deepEqual(a[key], b[key], logKey + '.' + key)) {
-                return false;
-            }
-        }
-        else if (a[key] !== b[key]) {
-            return false;
-        }
-    }
-    return true;
+    return { result: projectConfigs, erroredConfigFiles };
 }