nx 23.2.0-beta.0 → 23.2.0-beta.10

package/dist/src/command-line/init/implementation/format.js

@@ -0,0 +1,114 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.recordInitWrite = recordInitWrite;
+exports.formatInitWrites = formatInitWrites;
+const tslib_1 = require("tslib");
+const node_fs_1 = require("node:fs");
+const path = tslib_1.__importStar(require("node:path"));
+const chunkify_1 = require("../../../utils/chunkify");
+const formatters_1 = require("../../../utils/formatters");
+const oxfmt_1 = require("../../../utils/formatters/oxfmt");
+const prettier_1 = require("../../../utils/formatters/prettier");
+const output_1 = require("../../../utils/output");
+/**
+ * Paths `nx init` has created or modified.
+ *
+ * Module state because init's flows write from many call sites across
+ * several files, a number behind early returns; the alternative is threading
+ * a collector through every one.
+ */
+const writtenFiles = new Set();
+/**
+ * Records a file for `formatInitWrites`. Only for files a formatter handles -
+ * `.gitignore` and the vendored `.nx` scripts are left out rather than
+ * filtered later.
+ *
+ * Two recording helpers are shared with `nx import`, which drains before each
+ * commit amend; draining after one would leave the formatting uncommitted.
+ * The drain is a no-op when empty.
+ */
+function recordInitWrite(filePath) {
+    writtenFiles.add(filePath);
+}
+/**
+ * Formats what init just wrote and nothing else: an existing repo's other
+ * files are not init's to reformat.
+ *
+ * Never fatal - the repo is already initialised, so a formatter that cannot
+ * run costs a `nx format` the user can run themselves, not their setup.
+ */
+async function formatInitWrites(repoRoot, 
+// `nx import` drains this set too, and a user who never ran `nx init` should
+// not be told `nx init` failed.
+command = 'nx init') {
+    const recorded = [...writtenFiles];
+    // Cleared unconditionally: a second init in the same process must not
+    // reformat the first one's files.
+    writtenFiles.clear();
+    if (process.env.NX_SKIP_FORMAT === 'true' || recorded.length === 0) {
+        return;
+    }
+    // Repo-relative with `repoRoot` as cwd, so paths stay short and resolve
+    // against the repo. That also pins oxfmt's `.editorconfig` lookup, which it
+    // does once from the working directory; it does not pin config discovery.
+    //
+    // De-duplicated *after* normalising: the same file is routinely recorded
+    // both as `'nx.json'` and as `join(repoRoot, 'nx.json')`, which are two Set
+    // keys for one file.
+    const files = [
+        ...new Set(recorded.map((file) => path
+            .relative(repoRoot, path.resolve(repoRoot, file))
+            .split(path.sep)
+            .join('/'))),
+    ].filter((file) => file &&
+        !file.startsWith('../') &&
+        !path.isAbsolute(file) &&
+        (0, node_fs_1.existsSync)(path.join(repoRoot, file)));
+    if (files.length === 0) {
+        return;
+    }
+    // `detectFormatter` reads package.json, so it belongs inside the try - a
+    // workspace whose package.json cannot be parsed must not fail an init that
+    // already finished.
+    //
+    // Chunked as `nx format` chunks: the Angular flow records a `project.json`
+    // per project. The prettier path is sized against its quoted length; oxfmt
+    // goes through execFile and gets raw paths.
+    let formatter = null;
+    try {
+        formatter = (0, formatters_1.detectFormatter)(repoRoot);
+        if (!formatter) {
+            return;
+        }
+        if (formatter === 'oxfmt') {
+            for (const chunk of (0, chunkify_1.chunkify)(files)) {
+                if (chunk.length) {
+                    (0, oxfmt_1.writeWithOxfmt)(chunk, repoRoot);
+                }
+            }
+        }
+        else {
+            // oxfmt needs no equivalent filter - it silently skips file types it
+            // does not handle and exits 0. prettier still formats the rest of the
+            // batch, but exits 2 on one unsupported file, which would end a
+            // successful init with a spurious warning and prettier's own stderr.
+            const supported = await (0, prettier_1.filterToPrettierSupportedFiles)(files);
+            for (const chunk of (0, chunkify_1.chunkify)(supported, undefined, (pattern) => (0, prettier_1.quoteForShell)(pattern).length)) {
+                if (chunk.length) {
+                    (0, prettier_1.writeWithPrettier)(chunk, repoRoot);
+                }
+            }
+        }
+    }
+    catch (e) {
+        output_1.output.warn({
+            title: formatter
+                ? `Could not format the files ${command} wrote with ${formatter}.`
+                : `Could not work out which formatter to use for the files ${command} wrote.`,
+            bodyLines: [
+                ...(e?.message ? [e.message] : []),
+                'Run "nx format:write" to format them.',
+            ],
+        });
+    }
+}

package/dist/src/command-line/init/implementation/utils.js

@@ -26,6 +26,7 @@ const acknowledge_build_scripts_1 = require("../../../utils/acknowledge-build-sc
 const path_2 = require("../../../utils/path");
 const versions_1 = require("../../../utils/versions");
 const fs_1 = require("fs");
+const format_1 = require("./format");
 const connect_to_nx_cloud_1 = require("../../../nx-cloud/generators/connect-to-nx-cloud/connect-to-nx-cloud");
 const connect_to_nx_cloud_2 = require("../../nx-cloud/connect/connect-to-nx-cloud");
 const deduce_default_base_1 = require("./deduce-default-base");
@@ -71,6 +72,7 @@ function createNxJsonFile(repoRoot, topologicalTargets, cacheableOperations, scr
         nxJson.defaultBase ??= defaultBase;
     }
     (0, fileutils_1.writeJsonFile)(nxJsonPath, nxJson);
+    (0, format_1.recordInitWrite)(nxJsonPath);
 }
 /**
  * Locate-by-target upsert against an in-memory `targetDefaults` map. Merges
@@ -221,6 +223,7 @@ function addDepsToPackageJson(repoRoot, packageManager, additionalPackages) {
         }
     }
     (0, fileutils_1.writeJsonFile)(path, json);
+    (0, format_1.recordInitWrite)(path);
     // nx has a postinstall script, which pnpm 11+ refuses to install
     // unacknowledged.
     (0, acknowledge_build_scripts_1.acknowledgeBuildScripts)(repoRoot, packageManager, { nx: true });
@@ -351,6 +354,7 @@ function setNeverConnectToCloud(repoRoot) {
     const nxJson = (0, fileutils_1.readJsonFile)(nxJsonPath);
     nxJson.neverConnectToCloud = true;
     (0, fileutils_1.writeJsonFile)(nxJsonPath, nxJson);
+    (0, format_1.recordInitWrite)(nxJsonPath);
 }
 function addVsCodeRecommendedExtensions(repoRoot, extensions) {
     const vsCodeExtensionsPath = (0, path_1.join)(repoRoot, '.vscode/extensions.json');
@@ -363,9 +367,11 @@ function addVsCodeRecommendedExtensions(repoRoot, extensions) {
             }
         });
         (0, fileutils_1.writeJsonFile)(vsCodeExtensionsPath, vsCodeExtensionsJson);
+        (0, format_1.recordInitWrite)(vsCodeExtensionsPath);
     }
     else {
         (0, fileutils_1.writeJsonFile)(vsCodeExtensionsPath, { recommendations: extensions });
+        (0, format_1.recordInitWrite)(vsCodeExtensionsPath);
     }
 }
 function markRootPackageJsonAsNxProjectLegacy(repoRoot, cacheableScripts, pmc) {
@@ -386,6 +392,7 @@ function markRootPackageJsonAsNxProjectLegacy(repoRoot, cacheableScripts, pmc) {
         }
     }
     (0, fileutils_1.writeJsonFile)(`package.json`, json);
+    (0, format_1.recordInitWrite)('package.json');
 }
 function markPackageJsonAsNxProject(packageJsonPath) {
     const json = (0, fileutils_1.readJsonFile)(packageJsonPath);
@@ -394,6 +401,7 @@ function markPackageJsonAsNxProject(packageJsonPath) {
     }
     json.nx = {};
     (0, fileutils_1.writeJsonFile)(packageJsonPath, json);
+    (0, format_1.recordInitWrite)(packageJsonPath);
 }
 function printFinalMessage({ learnMoreLink, appendLines, }) {
     output_1.output.success({

package/dist/src/command-line/init/init-v1.js

@@ -2,7 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.initHandler = initHandler;
 const child_process_1 = require("child_process");
-const enquirer_1 = require("enquirer");
+const prompt_helpers_1 = require("../../utils/prompt-helpers");
 const fs_1 = require("fs");
 const semver_1 = require("semver");
 const add_nx_to_monorepo_1 = require("./implementation/add-nx-to-monorepo");
@@ -14,6 +14,7 @@ const child_process_2 = require("../../utils/child-process");
 const fileutils_1 = require("../../utils/fileutils");
 const versions_1 = require("../../utils/versions");
 const utils_1 = require("./implementation/utils");
+const format_1 = require("./implementation/format");
 async function initHandler(options) {
     // strip the 'init' command itself so we don't forward it
     const args = process.argv.slice(3).join(' ');
@@ -23,11 +24,13 @@ async function initHandler(options) {
     }
     if (options.useDotNxInstallation === true) {
         setupDotNxInstallation(version);
+        await (0, format_1.formatInitWrites)(process.cwd());
     }
     else if ((0, fs_1.existsSync)('package.json')) {
         const packageJson = (0, fileutils_1.readJsonFile)('package.json');
         if ((0, fs_1.existsSync)('angular.json')) {
             await (0, angular_1.addNxToAngularCliRepo)(options);
+            await (0, format_1.formatInitWrites)(process.cwd());
             (0, utils_1.printFinalMessage)({
                 learnMoreLink: 'https://nx.dev/recipes/angular/migration/angular',
             });
@@ -35,6 +38,7 @@ async function initHandler(options) {
         }
         else if (isNestCLI(packageJson)) {
             await (0, add_nx_to_nest_1.addNxToNest)(options, packageJson);
+            await (0, format_1.formatInitWrites)(process.cwd());
             (0, utils_1.printFinalMessage)({
                 learnMoreLink: 'https://nx.dev/recipes/adopting-nx/adding-to-monorepo',
             });
@@ -42,37 +46,30 @@ async function initHandler(options) {
         }
         else if ((0, utils_1.isMonorepo)(packageJson)) {
             await (0, add_nx_to_monorepo_1.addNxToMonorepo)({ ...options, legacy: true });
+            await (0, format_1.formatInitWrites)(process.cwd());
             (0, utils_1.printFinalMessage)({
                 learnMoreLink: 'https://nx.dev/recipes/adopting-nx/adding-to-monorepo',
             });
         }
         else {
             await (0, add_nx_to_npm_repo_1.addNxToNpmRepo)({ ...options, legacy: true });
+            await (0, format_1.formatInitWrites)(process.cwd());
             (0, utils_1.printFinalMessage)({
                 learnMoreLink: 'https://nx.dev/recipes/adopting-nx/adding-to-existing-project',
             });
         }
     }
     else {
-        const useDotNxFolder = await (0, enquirer_1.prompt)([
-            {
-                name: 'useDotNxFolder',
-                type: 'autocomplete',
-                message: 'Where should your workspace be created?',
-                choices: [
-                    {
-                        name: 'In a new folder under this directory',
-                        value: 'false',
-                    },
-                    {
-                        name: 'In this directory',
-                        value: 'true',
-                    },
-                ],
-            },
-        ]).then((r) => r.useDotNxFolder === 'true');
+        const useDotNxFolder = (await (0, prompt_helpers_1.selectPrompt)({
+            message: 'Where should your workspace be created?',
+            choices: [
+                { value: 'false', label: 'In a new folder under this directory' },
+                { value: 'true', label: 'In this directory' },
+            ],
+        })) === 'true';
         if (useDotNxFolder) {
             setupDotNxInstallation(version);
+            await (0, format_1.formatInitWrites)(process.cwd());
         }
         else {
             (0, child_process_1.execSync)(`npx --yes create-nx-workspace@${version} ${args}`, {

package/dist/src/command-line/init/init-v2.js

@@ -5,12 +5,13 @@ exports.getPluginReason = getPluginReason;
 exports.detectPlugins = detectPlugins;
 const fs_1 = require("fs");
 const path_1 = require("path");
-const enquirer_1 = require("enquirer");
+const prompt_helpers_1 = require("../../utils/prompt-helpers");
 const semver_1 = require("semver");
 const nx_json_1 = require("../../config/nx-json");
 const fileutils_1 = require("../../utils/fileutils");
 const get_package_name_from_import_path_1 = require("../../utils/get-package-name-from-import-path");
 const output_1 = require("../../utils/output");
+const package_json_1 = require("../../utils/package-json");
 const package_manager_1 = require("../../utils/package-manager");
 const versions_1 = require("../../utils/versions");
 const workspace_context_1 = require("../../utils/workspace-context");
@@ -19,6 +20,7 @@ const configure_plugins_1 = require("./configure-plugins");
 const ai_agent_prompts_1 = require("./ai-agent-prompts");
 const set_up_ai_agents_1 = require("../../ai/set-up-ai-agents/set-up-ai-agents");
 const tree_1 = require("../../generators/tree");
+const format_1 = require("./implementation/format");
 const add_nx_to_monorepo_1 = require("./implementation/add-nx-to-monorepo");
 const add_nx_to_npm_repo_1 = require("./implementation/add-nx-to-npm-repo");
 const add_nx_to_turborepo_1 = require("./implementation/add-nx-to-turborepo");
@@ -26,14 +28,12 @@ const angular_1 = require("./implementation/angular");
 const add_nx_scripts_1 = require("./implementation/dot-nx/add-nx-scripts");
 const utils_1 = require("./implementation/utils");
 const provenance_1 = require("../../utils/provenance");
-const devkit_internals_1 = require("../../devkit-internals");
 const handle_import_1 = require("../../utils/handle-import");
 const native_1 = require("../../native");
 const detect_ai_agent_1 = require("../../ai/detect-ai-agent");
 const ab_testing_1 = require("../../utils/ab-testing");
 const analytics_prompt_1 = require("../../utils/analytics-prompt");
 const is_ci_1 = require("../../utils/is-ci");
-const package_manager_2 = require("../../utils/package-manager");
 const ai_output_1 = require("./utils/ai-output");
 async function initHandler(options, inner = false) {
     // Use environment variable to force local execution
@@ -43,7 +43,7 @@ async function initHandler(options, inner = false) {
     let cleanup;
     try {
         await (0, provenance_1.ensurePackageHasProvenance)('nx', 'latest');
-        const packageInstallResults = (0, devkit_internals_1.installPackageToTmp)('nx', 'latest', (0, package_manager_2.detectPackageManager)(process.cwd()));
+        const packageInstallResults = (0, package_json_1.installPackageToTmp)('nx', 'latest', (0, package_manager_1.detectPackageManager)(process.cwd()));
         cleanup = packageInstallResults.cleanup;
         let modulePath = require.resolve('nx/src/command-line/init/init-v2.js', {
             paths: [packageInstallResults.tempDir],
@@ -95,7 +95,7 @@ async function initHandlerImpl(options) {
     const baseMeta = {
         nodeVersion: process.versions.node,
         os: process.platform,
-        packageManager: (0, package_manager_2.detectPackageManager)(),
+        packageManager: (0, package_manager_1.detectPackageManager)(),
         aiAgent: (0, native_1.isAiAgent)(),
         isCI: (0, is_ci_1.isCI)(),
     };
@@ -149,6 +149,7 @@ async function runInit(options, baseMeta) {
             ...options,
             integrated: !!options.integrated,
         });
+        await (0, format_1.formatInitWrites)(process.cwd());
         (0, utils_1.printFinalMessage)({
             learnMoreLink: 'https://nx.dev/technologies/angular/migration/angular',
         });
@@ -162,21 +163,17 @@ async function runInit(options, baseMeta) {
         options.interactive &&
         !aiMode &&
         process.stdin.isTTY) {
-        const setupMode = await (0, enquirer_1.prompt)([
-            {
-                type: 'select',
-                name: 'setupMode',
-                message: 'How would you like to set up Nx in this directory?',
-                choices: [
-                    {
-                        name: '.nx installation (recommended for non-JavaScript projects)',
-                    },
-                    {
-                        name: 'package.json installation (recommended for JavaScript/TypeScript projects)',
-                    },
-                ],
-            },
-        ]).then((r) => r.setupMode);
+        const setupMode = await (0, prompt_helpers_1.selectPrompt)({
+            message: 'How would you like to set up Nx in this directory?',
+            choices: [
+                {
+                    value: '.nx installation (recommended for non-JavaScript projects)',
+                },
+                {
+                    value: 'package.json installation (recommended for JavaScript/TypeScript projects)',
+                },
+            ],
+        });
         if (setupMode.startsWith('package.json')) {
             // Create a minimal package.json so the JS/TS workflow takes over
             const workspaceName = (0, path_1.basename)(process.cwd());
@@ -185,6 +182,7 @@ async function runInit(options, baseMeta) {
                 version: '0.0.0',
                 private: true,
             });
+            (0, format_1.recordInitWrite)('package.json');
         }
         else {
             options.useDotNxInstallation = true;
@@ -199,14 +197,10 @@ async function runInit(options, baseMeta) {
     // AI mode defaults to minimum setup, humans can choose
     let guided = !aiMode; // Default to minimum (false) for AI, guided (true) for humans
     if (options.interactive && !(_isTurborepo || _isNonJs)) {
-        const setupType = await (0, enquirer_1.prompt)([
-            {
-                type: 'select',
-                name: 'setupPreference',
-                message: 'Would you like a minimum or guided setup?',
-                choices: [{ name: 'Minimum' }, { name: 'Guided' }],
-            },
-        ]).then((r) => r.setupPreference);
+        const setupType = await (0, prompt_helpers_1.selectPrompt)({
+            message: 'Would you like a minimum or guided setup?',
+            choices: [{ value: 'Minimum' }, { value: 'Guided' }],
+        });
         guided = setupType === 'Guided';
     }
     /**
@@ -222,6 +216,7 @@ async function runInit(options, baseMeta) {
         await (0, add_nx_to_turborepo_1.addNxToTurborepo)({
             interactive: options.interactive,
         });
+        await (0, format_1.formatInitWrites)(process.cwd());
         (0, utils_1.printFinalMessage)({
             learnMoreLink: 'https://nx.dev/recipes/adopting-nx/from-turborepo',
         });
@@ -296,6 +291,9 @@ async function runInit(options, baseMeta) {
                     reason: getPluginReason(name),
                 }));
                 (0, ai_output_1.logProgress)('detecting', `Detected ${detectedPluginNames.length} plugin(s): ${detectedPluginNames.join(', ')}`);
+                // This path exits without reaching the drain below, and `nx.json` has
+                // already been written by now.
+                await (0, format_1.formatInitWrites)(repoRoot);
                 (0, ai_output_1.writeAiOutput)((0, ai_output_1.buildNeedsInputResult)(detectedPlugins));
                 process.exit(0);
             }
@@ -368,12 +366,16 @@ async function runInit(options, baseMeta) {
             analyticsPrompt,
             nodeVersion: process.versions.node,
             os: process.platform,
-            packageManager: (0, package_manager_2.detectPackageManager)(),
+            packageManager: (0, package_manager_1.detectPackageManager)(),
             aiAgent: aiMode,
             isCI: (0, is_ci_1.isCI)(),
             pluginsInstalled: pluginsToInstall.join(','),
         },
     });
+    // Before the AI record, not after: the formatters write to stdout
+    // (`--list-different`, oxfmt's summary), which would otherwise land after the
+    // NDJSON result and corrupt the tail an agent parses.
+    await (0, format_1.formatInitWrites)(repoRoot);
     // Output success result for AI agents
     if (aiMode) {
         (0, ai_output_1.writeAiOutput)((0, ai_output_1.buildSuccessResult)({
@@ -401,6 +403,7 @@ async function runInit(options, baseMeta) {
 function getPluginReason(plugin) {
     const reasonMap = {
         '@nx/eslint': 'eslint detected in dependencies',
+        '@nx/oxlint': 'oxlint detected in dependencies',
         '@nx/storybook': 'storybook detected in dependencies',
         '@nx/vite': 'vite detected in dependencies',
         '@nx/vitest': 'vitest detected in dependencies',
@@ -448,6 +451,7 @@ function parsePluginsFlag(value) {
 const npmPackageToPluginMap = {
     // Generic JS tools
     eslint: '@nx/eslint',
+    oxlint: '@nx/oxlint',
     storybook: '@nx/storybook',
     // Bundlers
     vite: '@nx/vite',
@@ -574,39 +578,18 @@ async function detectPlugins(nxJson, packageJson, interactive, includeAngularCli
             `Add these Nx plugins to integrate with the tools used in your workspace.`,
         ],
     });
-    const pluginsToInstall = await (0, enquirer_1.prompt)([
-        {
-            name: 'plugins',
-            type: 'multiselect',
-            message: `Which plugins would you like to add? Press <Space> to select and <Enter> to submit.`,
-            choices: plugins.map((p) => ({ name: p, value: p })),
-            /**
-             * limit is missing from the interface but it limits the amount of options shown
-             */
-            limit: process.stdout.rows - 4, // 4 leaves room for the header above, the prompt and some whitespace
-        },
-    ]).then((r) => r.plugins);
+    const pluginsToInstall = await (0, prompt_helpers_1.multiselectPrompt)({
+        message: `Which plugins would you like to add? Press <Space> to select and <Enter> to submit.`,
+        choices: plugins,
+    });
     if (pluginsToInstall?.length === 0)
         return {
             plugins: [],
             updatePackageScripts: false,
         };
     const updatePackageScripts = (0, fs_1.existsSync)('package.json') &&
-        (await (0, enquirer_1.prompt)([
-            {
-                name: 'updatePackageScripts',
-                type: 'autocomplete',
-                message: `Do you want to start using Nx in your package.json scripts?`,
-                choices: [
-                    {
-                        name: 'Yes',
-                    },
-                    {
-                        name: 'No',
-                    },
-                ],
-                initial: 0,
-            },
-        ]).then((r) => r.updatePackageScripts === 'Yes'));
+        (await (0, prompt_helpers_1.confirmationPrompt)({
+            message: `Do you want to start using Nx in your package.json scripts?`,
+        }));
     return { plugins: pluginsToInstall, updatePackageScripts };
 }

package/dist/src/command-line/migrate/agentic/definitions.js

@@ -4,7 +4,7 @@ exports.AGENT_DEFINITIONS = exports.opencodeDefinition = exports.codexDefinition
 exports.getAgentDefinition = getAgentDefinition;
 const os_1 = require("os");
 const path_1 = require("path");
-const handoff_1 = require("./handoff");
+const types_1 = require("./types");
 // --- Claude Code ---------------------------------------------------------
 function claudeCodeWellKnownPaths() {
     if (process.platform === 'win32') {
@@ -13,21 +13,48 @@ function claudeCodeWellKnownPaths() {
     }
     return [(0, path_1.join)((0, os_1.homedir)(), '.claude', 'local', 'claude')];
 }
-// Pre-authorizes the handoff write: Claude Code's default permission mode
-// asks before file writes it has no allow rule for, so without this each step
-// ends with an approval prompt for nx's own handoff scratch. Prefix-less
-// patterns resolve against the session cwd (pinned to the workspace root
-// below); `Edit` covers corrections to an already-written handoff.
-const CLAUDE_CODE_HANDOFF_ALLOWED_TOOLS = `Write(${handoff_1.MIGRATE_RUNS_RELATIVE_DIR}/**),Edit(${handoff_1.MIGRATE_RUNS_RELATIVE_DIR}/**)`;
+/**
+ * The run directory names Nx will interpolate into a permission rule. A rule
+ * is a gitignore pattern, which reads `*?[]\` as syntax, and `--allowedTools`
+ * splits its value on commas and spaces, so a name carrying any of those
+ * would change what the rule authorizes rather than what it names. Run
+ * directory names derive from a migration's own version string, which nothing
+ * upstream constrains to this alphabet. The leading character rejects `.`
+ * and `..`.
+ */
+const RULE_SAFE_RUN_DIR_NAME = /^[A-Za-z0-9][A-Za-z0-9._+-]*$/;
+/**
+ * Pre-authorizes the handoff write. Claude Code's default permission mode
+ * asks before a file write it has no allow rule for, so without this every
+ * step ends on an approval prompt for Nx's own handoff scratch.
+ *
+ * The rule reaches one run's handoffs and nothing else. Its own run
+ * directory also holds state Nx wrote and reads back, and the sibling
+ * directories belong to other runs, including an orchestrated run whose
+ * handoffs decide how its steps settle. `Edit` is the only tool name file
+ * rules are matched against, and it covers creating the file as well as
+ * correcting one already written. Prefix-less patterns resolve against the
+ * session cwd, pinned to the workspace root below.
+ *
+ * Returns null for a name it cannot express, which costs the approval prompt
+ * this exists to avoid. That beats widening the rule, and nothing narrower is
+ * available: Claude Code has no escape for a literal path.
+ */
+function claudeCodeHandoffAllowedTools(runDirName) {
+    if (!RULE_SAFE_RUN_DIR_NAME.test(runDirName)) {
+        return null;
+    }
+    return `Edit(${types_1.MIGRATE_RUNS_RELATIVE_DIR}/${runDirName}/${types_1.HANDOFFS_DIR_NAME}/**)`;
+}
 function claudeCodeBuildInteractive(ctx) {
+    const allowedTools = claudeCodeHandoffAllowedTools(ctx.runDirName);
     return {
         // `--allowedTools` is variadic (space/comma separated): a positional
         // placed right after its value gets swallowed as another rule. The rules
         // must stay in one comma-joined element with a non-variadic flag
         // (`--system-prompt`) between them and the user prompt.
         args: [
-            '--allowedTools',
-            CLAUDE_CODE_HANDOFF_ALLOWED_TOOLS,
+            ...(allowedTools ? ['--allowedTools', allowedTools] : []),
             '--system-prompt',
             ctx.systemContext,
             ctx.userPrompt,

package/dist/src/command-line/migrate/agentic/handoff-gitignore.d.ts

@@ -1,33 +1,30 @@
-export declare function isHandoffGitignoreMigration(m: {
-    package: string;
-    name: string;
-}): boolean;
 /**
- * Under `--agentic`, the runner writes per-run scratch under
- * `.nx/migrate-runs/<run-id>/`. The v23 migration
+ * Both the agentic runner and the orchestrator write per-run scratch under
+ * `.nx/migrate-runs/<run-id>/`: handoff files in both cases, plus the durable
+ * run state and its plan snapshots for the orchestrator. The v23 migration
  * `23-0-0-add-migrate-runs-to-git-ignore` adds `.nx/migrate-runs` to
- * `.gitignore`; without intervention it would run in its declared slot
- * (typically late), so earlier per-migration commits would absorb the
- * scratch into the user-visible diff.
+ * `.gitignore`; in its declared slot (typically late) earlier per-migration
+ * commits would absorb the scratch into the user-visible diff.
  *
  * Two paths cover the leak, with no overlap:
  *
- *   1. HOIST — handled by the sort comparator in `executeMigrations`. When
- *      the v23 migration is in the queue, it sorts to position 0 and runs
- *      first via the normal migration runner (with its own log line and
- *      commit). Fully traceable in `git log`.
+ *   1. HOIST: `sortMigrations`, which `executeMigrations` applies, sorts the
+ *      v23 migration to position 0 when it is in the queue, so it runs first
+ *      through the normal runner with its own log line and commit. A
+ *      single-migration worker run needs no hoisting: the requested migration
+ *      is the entire queue.
  *
- *   2. INLINE FALLBACK — this function. When the migration is NOT in the
- *      queue AND the highest target version is < v23 (intra-pre-v23
- *      `--agentic` run), the migration won't run at all. Apply its body
- *      inline against an `FsTree` and commit it as a standalone preflight
- *      commit (or leave in the working tree under `--no-create-commits`).
+ *   2. INLINE FALLBACK, this function. When the migration is NOT in the queue
+ *      AND the highest target version is < v23 (intra-pre-v23 `--agentic`
+ *      run) it will never run, so apply its body inline against an `FsTree`
+ *      and commit it as a standalone preflight commit (or leave it in the
+ *      working tree under `--no-create-commits`).
  *
- * When the migration is not in the queue AND target >= v23, the user is
- * past v23 already. They had the entry historically; if it's gone, that's
- * a conscious removal we respect.
+ * Not in the queue AND target >= v23 means the user is already past v23. They
+ * had the entry historically; if it is gone, that is a conscious removal we
+ * respect.
  */
-export declare function applyAgenticHandoffGitignoreFallback({ migrations, installedNxVersion, effectiveCreateCommits, commitPrefix, root, }: {
+export declare function applyAgenticHandoffGitignoreFallback({ migrations, installedNxVersion, effectiveCreateCommits, commitPrefix, root, applyWhenPlanned, commitStandalone, }: {
     migrations: ReadonlyArray<{
         package: string;
         name: string;
@@ -43,4 +40,22 @@ export declare function applyAgenticHandoffGitignoreFallback({ migrations, insta
     effectiveCreateCommits: boolean;
     commitPrefix: string;
     root: string;
+    /**
+     * Apply the entry even when the hoisted migration is in the plan. The
+     * classic loop can defer to that migration because its run scratch appears
+     * only after migration 1 has run; the orchestrator creates its run dir at
+     * init, before any migration, so it needs the entry immediately. A planned
+     * migration also means the missing entry is not a conscious removal, so the
+     * v23 cutoff does not apply.
+     */
+    applyWhenPlanned?: boolean;
+    /**
+     * Commit the applied entry as its own preflight commit (the default). The
+     * orchestrator suppresses this: it applies the fallback before its init
+     * checkpoint so the checkpoint's `git add -A` cannot sweep in older
+     * scratch, and in that ordering a standalone commit here would carry the
+     * user's pre-existing changes along with the entry. The checkpoint that
+     * follows captures both instead.
+     */
+    commitStandalone?: boolean;
 }): Promise<void>;