@depup/nx 23.1.2-depup.1 → 23.2.0-depup.0

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
@@ -194,14 +196,6 @@ function createNxJsonFromTurboJson(turboJson) {
             nxJson.targetDefaults = targetDefaults;
         }
     }
-    /**
-     * The fact that cacheDir was in use suggests the user had a reason for deviating from the default.
-     * We can't know what that reason was, nor if it would still be applicable in Nx, but we can at least
-     * improve discoverability of the relevant Nx option by explicitly including it with its default value.
-     */
-    if (turboJson.cacheDir) {
-        nxJson.cacheDirectory = '.nx/cache';
-    }
     const defaultBase = (0, deduce_default_base_1.deduceDefaultBase)();
     // Do not add defaultBase if it is inferred to be the Nx default value of main
     if (defaultBase !== 'main') {
@@ -221,6 +215,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 +346,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 +359,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 +384,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 +393,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,7 +5,7 @@ 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");
@@ -20,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");
@@ -29,11 +30,14 @@ const utils_1 = require("./implementation/utils");
 const provenance_1 = require("../../utils/provenance");
 const handle_import_1 = require("../../utils/handle-import");
 const native_1 = require("../../native");
+const utils_2 = require("../../ai/utils");
+const permission_errors_1 = require("../../utils/permission-errors");
 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 ai_output_1 = require("./utils/ai-output");
+const is_sandbox_1 = require("../../utils/is-sandbox");
 async function initHandler(options, inner = false) {
     // Use environment variable to force local execution
     if (process.env.NX_USE_LOCAL === 'true' || inner) {
@@ -118,8 +122,15 @@ async function runInit(options, baseMeta) {
     }
     // AI agent mode: apply defaults for non-interactive operation
     const aiMode = (0, native_1.isAiAgent)();
+    const isSandboxed = (0, is_sandbox_1.isSandbox)();
     if (aiMode) {
         options.interactive = false; // Force non-interactive
+        if (isSandboxed) {
+            // During init, before configuring AI agents, the daemon and
+            // plugin socket connections are not available
+            process.env.NX_DAEMON = 'false';
+            process.env.NX_ISOLATE_PLUGINS = 'false';
+        }
         if (options.nxCloud === undefined) {
             options.nxCloud = false; // Default to skip Nx Cloud
         }
@@ -148,6 +159,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',
         });
@@ -161,21 +173,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());
@@ -184,6 +192,7 @@ async function runInit(options, baseMeta) {
                 version: '0.0.0',
                 private: true,
             });
+            (0, format_1.recordInitWrite)('package.json');
         }
         else {
             options.useDotNxInstallation = true;
@@ -198,14 +207,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';
     }
     /**
@@ -221,6 +226,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',
         });
@@ -295,6 +301,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);
             }
@@ -321,21 +330,38 @@ async function runInit(options, baseMeta) {
             await (0, configure_plugins_1.configurePlugins)(pluginsToInstall, updatePackageScripts, pmc, repoRoot, options.verbose);
         }
     }
-    const selectedAgents = await (0, ai_agent_prompts_1.determineAiAgents)(options.aiAgents, options.interactive && guided);
+    // AI agent mode configures the detected agent automatically (aiAgents is
+    // pre-filled above); interactive humans are always prompted — even in the
+    // minimum setup — since agent configuration also writes the sandbox
+    // allowances the Nx daemon needs when driven by an agent later.
+    const selectedAgents = await (0, ai_agent_prompts_1.determineAiAgents)(options.aiAgents, options.interactive);
     if (selectedAgents && selectedAgents.length > 0) {
-        const tree = new tree_1.FsTree(repoRoot, false);
-        const aiAgentsCallback = await (0, set_up_ai_agents_1.setupAiAgentsGenerator)(tree, {
-            directory: '.',
-            writeNxCloudRules: options.nxCloud !== false,
-            packageVersion: 'latest',
-            agents: [...selectedAgents],
-        });
-        const changes = tree.listChanges();
-        (0, tree_1.flushChanges)(repoRoot, changes);
-        if (aiAgentsCallback) {
-            const results = await aiAgentsCallback();
-            results.messages.forEach((m) => output_1.output.log(m));
-            results.errors.forEach((e) => output_1.output.error(e));
+        try {
+            const tree = new tree_1.FsTree(repoRoot, false);
+            const aiAgentsCallback = await (0, set_up_ai_agents_1.setupAiAgentsGenerator)(tree, {
+                directory: '.',
+                writeNxCloudRules: options.nxCloud !== false,
+                packageVersion: 'latest',
+                agents: [...selectedAgents],
+            });
+            const changes = tree.listChanges();
+            (0, tree_1.flushChanges)(repoRoot, changes);
+            if (aiAgentsCallback) {
+                const results = await aiAgentsCallback();
+                results.messages.forEach((m) => output_1.output.log(m));
+                results.errors.forEach((e) => output_1.output.error(e));
+            }
+        }
+        catch (e) {
+            if (!(0, permission_errors_1.isPermissionDenied)(e)) {
+                throw e;
+            }
+            // Don't fail the whole init over this — everything else succeeded.
+            // Warned rather than thrown so the rest of the init result still stands.
+            output_1.output.warn({
+                title: 'AI agent configuration could not be written from this process',
+                bodyLines: (0, utils_2.agentConfigWriteBlockedLines)(e),
+            });
         }
     }
     let nxCloudChoice;
@@ -373,6 +399,10 @@ async function runInit(options, baseMeta) {
             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)({
@@ -400,6 +430,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',
@@ -447,6 +478,7 @@ function parsePluginsFlag(value) {
 const npmPackageToPluginMap = {
     // Generic JS tools
     eslint: '@nx/eslint',
+    oxlint: '@nx/oxlint',
     storybook: '@nx/storybook',
     // Bundlers
     vite: '@nx/vite',
@@ -573,39 +605,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,