@ohos-ports/nx 23.2.0-beta.0 → 23.2.1-beta.0

package/dist/src/command-line/migrate/agentic/run-step.d.ts

@@ -1,68 +0,0 @@
-import { FileChange } from '../../../generators/tree';
-import { AgenticPromptMode } from './prompts/system-prompt';
-import { EnabledResolvedAgentic } from './types';
-/**
- * Context describing the deterministic (generator) half of a migration, used
- * to seed the agent's prompt with what just ran. Required for hybrid prompt
- * migrations (`buildHybridPromptUserPrompt`) and for generic-validation runs.
- */
-export interface AgenticPromptImplContext {
-    logs: string;
-    changes: FileChange[];
-    agentContext: string[];
-    hasDiffContext: boolean;
-}
-export interface AgenticStepResult {
-    /** Agent's handoff summary, or a placeholder when the user marked complete. */
-    summary: string;
-    /**
-     * True when the agent didn't write a valid handoff and the user told nx to
-     * continue anyway. The caller uses this to swap the outcome label
-     * accordingly.
-     */
-    ambiguous: boolean;
-}
-/**
- * `runStep` travels in the context rather than being imported by its call
- * sites, so this module loads only where the context is built, behind an
- * agentic-enabled gate.
- */
-export interface AgenticRunContext {
-    agentic: EnabledResolvedAgentic;
-    runDir: string;
-    runStep: typeof runAgenticPromptStep;
-}
-export interface RunAgenticPromptStepInput {
-    root: string;
-    migration: {
-        package: string;
-        name: string;
-        version: string;
-        description?: string;
-        prompt?: string;
-    };
-    /**
-     * Path to the migration's documentation file, resolved by the orchestrator -
-     * workspace-relative, or absolute when the file resolves outside the
-     * workspace. Omitted when the migration declares no `documentation` or it
-     * can't be resolved.
-     */
-    documentationPath?: string;
-    agentic: EnabledResolvedAgentic;
-    runDir: string;
-    installDepsIfChanged: () => Promise<void>;
-    implContext?: AgenticPromptImplContext;
-    mode?: AgenticPromptMode;
-}
-/**
- * Spawns the configured AI agent against a migration step, awaits its handoff,
- * and translates the outcome into a structured result the executor can branch
- * on. Throws on failure / abort; returns a result with `ambiguous` set when
- * the agent exited without writing a handoff and the user chose to continue.
- *
- * `installDepsIfChanged` is a callback rather than a `ChangedDepInstaller`
- * instance so this module stays decoupled from the executor's internal state.
- * The structural `migration` type captures only the fields read here, keeping
- * the file free of cross-imports with the orchestrator.
- */
-export declare function runAgenticPromptStep(input: RunAgenticPromptStepInput): Promise<AgenticStepResult>;

package/dist/src/command-line/migrate/agentic/run-step.js

@@ -1,124 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.runAgenticPromptStep = runAgenticPromptStep;
-const tslib_1 = require("tslib");
-const path_1 = require("path");
-const pc = tslib_1.__importStar(require("picocolors"));
-const child_process_1 = require("../../../utils/child-process");
-const logger_1 = require("../../../utils/logger");
-const package_manager_1 = require("../../../utils/package-manager");
-const migrate_output_1 = require("../migrate-output");
-const handoff_1 = require("./handoff");
-const generic_validation_1 = require("./prompts/generic-validation");
-const hybrid_prompt_migration_1 = require("./prompts/hybrid-prompt-migration");
-const prompt_migration_1 = require("./prompts/prompt-migration");
-const system_prompt_1 = require("./prompts/system-prompt");
-const definitions_1 = require("./definitions");
-const runner_1 = require("./runner");
-/**
- * Spawns the configured AI agent against a migration step, awaits its handoff,
- * and translates the outcome into a structured result the executor can branch
- * on. Throws on failure / abort; returns a result with `ambiguous` set when
- * the agent exited without writing a handoff and the user chose to continue.
- *
- * `installDepsIfChanged` is a callback rather than a `ChangedDepInstaller`
- * instance so this module stays decoupled from the executor's internal state.
- * The structural `migration` type captures only the fields read here, keeping
- * the file free of cross-imports with the orchestrator.
- */
-async function runAgenticPromptStep(input) {
-    const { root, migration, agentic, runDir, installDepsIfChanged, implContext, documentationPath, mode = 'author', } = input;
-    const handoffFilePath = (0, handoff_1.stepHandoffPath)(runDir, migration);
-    // The system prompt tells the agent the parent dir exists, so the agent
-    // doesn't defensively `mkdir -p` (which triggers a workspace-permission
-    // prompt in agents like Claude Code every run).
-    (0, handoff_1.mkdirSafely)((0, path_1.dirname)(handoffFilePath), `handoff directory for ${migration.name}`);
-    const pm = (0, package_manager_1.detectPackageManager)(root);
-    const systemContext = (0, system_prompt_1.buildSystemPrompt)({
-        workspaceRoot: root,
-        handoffFileAbsolutePath: handoffFilePath,
-        packageManager: pm,
-        nxInvocation: (0, child_process_1.getRunNxBaseCommand)((0, package_manager_1.getPackageManagerCommand)(pm, root), root),
-        mode,
-    });
-    let userPrompt;
-    if (mode === 'generic-validation') {
-        if (!implContext) {
-            throw new Error(`Internal error: generic-validation mode requires impl context (logs, changes, agentContext, hasDiffContext) but none was provided.`);
-        }
-        userPrompt = (0, generic_validation_1.buildGenericValidationUserPrompt)({
-            package: migration.package,
-            name: migration.name,
-            version: migration.version,
-            description: migration.description,
-            handoffFileAbsolutePath: handoffFilePath,
-            documentationPath,
-            impl: implContext,
-        });
-    }
-    else {
-        const promptCtx = {
-            package: migration.package,
-            name: migration.name,
-            version: migration.version,
-            description: migration.description,
-            promptPath: migration.prompt,
-            handoffFileAbsolutePath: handoffFilePath,
-            documentationPath,
-        };
-        userPrompt = implContext
-            ? (0, hybrid_prompt_migration_1.buildHybridPromptUserPrompt)({ ...promptCtx, impl: implContext })
-            : (0, prompt_migration_1.buildPromptMigrationUserPrompt)(promptCtx);
-    }
-    const definition = (0, definitions_1.getAgentDefinition)(agentic.selectedAgent.id);
-    if (!definition) {
-        throw new Error(`No agent definition registered for "${agentic.selectedAgent.id}".`);
-    }
-    const phase = mode === 'generic-validation' ? 'Validating' : 'Running prompt';
-    logger_1.logger.info(pc.dim(`→ ${phase} with ${agentic.selectedAgent.displayName}…`));
-    const outcome = await (0, runner_1.runAgentic)({
-        detected: agentic.selectedAgent,
-        definition,
-        invocationContext: {
-            systemContext,
-            userPrompt,
-            workspaceRoot: root,
-            runDirName: (0, path_1.basename)(runDir),
-        },
-        handoffFilePath,
-    });
-    // Some agent TUIs leave cursor/SGR state behind on exit. Reset before our
-    // own log lines so the outcome line lands clean instead of overlaid on the
-    // agent's trailing status bar.
-    (0, migrate_output_1.resetSgrAfterAgent)();
-    switch (outcome.kind) {
-        case 'success':
-            await installDepsIfChanged();
-            return { summary: outcome.summary, ambiguous: false };
-        case 'ambiguous-continue':
-            await installDepsIfChanged();
-            return {
-                summary: 'No handoff file was written; marked complete by user.',
-                ambiguous: true,
-            };
-        case 'failed': {
-            const failLabel = mode === 'generic-validation' ? 'Validation failed' : 'Failed';
-            logger_1.logger.info(`${pc.red('✗')} ${failLabel}: ${outcome.summary}`);
-            throw new Error(`Prompt migration ${migration.package}: ${migration.name} failed.`);
-        }
-        case 'ambiguous-abort':
-            // When Ctrl+C masked an underlying crash, the runner forwards the
-            // pre-rendered cause lines so we surface them before "Aborted by
-            // user" — otherwise the user sees only "aborted" with no signal that
-            // anything also crashed. When the abort came from the user picking
-            // "abort" at the prompt, the cause was already shown there.
-            if (outcome.causeSummary && outcome.causeSummary.length > 0) {
-                logger_1.logger.info(pc.dim('The agent run ended without a usable handoff:'));
-                for (const line of outcome.causeSummary) {
-                    logger_1.logger.info(pc.dim(`  ${line}`));
-                }
-            }
-            logger_1.logger.info(`${pc.red('✗')} Aborted by user.`);
-            throw new Error(`Prompt migration ${migration.package}: ${migration.name} was aborted by user.`);
-    }
-}

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

@@ -1,33 +0,0 @@
-import { SpawnOptions } from 'child_process';
-import { AgentDefinition, DetectedInstalledAgent, HandoffOutcome, InvocationContext } from './types';
-export interface RunAgenticArgs {
-    detected: DetectedInstalledAgent;
-    definition: AgentDefinition;
-    invocationContext: InvocationContext;
-    handoffFilePath: string;
-    /** Override the handoff-file poll interval (test seam). */
-    handoffPollIntervalMs?: number;
-    /** Override the SIGINT-to-SIGTERM grace period (test seam). */
-    gracefulExitMs?: number;
-    /** Override the post-force-kill safety bound (test seam). */
-    forceKillWaitMs?: number;
-}
-/**
- * Spawns the selected agent with `stdio: 'inherit'`, swallows SIGINT while the
- * child is alive, waits for it to exit, and resolves the run's outcome from
- * the handoff file (or the user when the file is missing).
- */
-export declare function runAgentic(args: RunAgenticArgs): Promise<HandoffOutcome>;
-/**
- * Node's `spawn` cannot directly execute `.cmd` / `.bat` shims on Windows;
- * `which` resolves to those when an agent was installed via npm. Wrap them in
- * a `cmd.exe /d /s /c` invocation with `windowsVerbatimArguments` so quoting
- * follows the cmd.exe convention rather than Node's default cooking.
- *
- * On non-Windows or for non-shim binaries this is a passthrough.
- */
-export declare function adaptSpawnForWindowsShim(binary: string, args: readonly string[], options: SpawnOptions): {
-    binary: string;
-    args: string[];
-    options: SpawnOptions;
-};

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

@@ -1,444 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.runAgentic = runAgentic;
-exports.adaptSpawnForWindowsShim = adaptSpawnForWindowsShim;
-const child_process_1 = require("child_process");
-const path_1 = require("path");
-const output_1 = require("../../../utils/output");
-const migrate_analytics_1 = require("../migrate-analytics");
-const safe_prompt_1 = require("../safe-prompt");
-const handoff_1 = require("./handoff");
-// How long to wait for the agent to exit gracefully after sending SIGINT once
-// a valid handoff has been written. Long enough for an interactive agent to
-// finish its current render and clean up; short enough that a frozen child
-// still gets escalated to SIGTERM in a sensible time.
-const AGENT_GRACEFUL_EXIT_MS = 5_000;
-/**
- * Spawns the selected agent with `stdio: 'inherit'`, swallows SIGINT while the
- * child is alive, waits for it to exit, and resolves the run's outcome from
- * the handoff file (or the user when the file is missing).
- */
-async function runAgentic(args) {
-    const { detected, definition, invocationContext, handoffFilePath, handoffPollIntervalMs, gracefulExitMs = AGENT_GRACEFUL_EXIT_MS, forceKillWaitMs = FORCE_KILL_WAIT_MS, } = args;
-    const spec = definition.buildInteractive(invocationContext);
-    const adapted = adaptSpawnForWindowsShim(detected.binary, spec.args, {
-        stdio: 'inherit',
-        cwd: spec.cwd ?? invocationContext.workspaceRoot,
-        env: spec.env ? { ...process.env, ...spec.env } : process.env,
-        windowsHide: true,
-    });
-    let child;
-    // Local alias so `@nx/workspace-require-windows-hide` recognizes the
-    // options arg as a tracked Identifier rather than giving up on a
-    // member-expression skip — keeps the lint rule strict on other call sites.
-    const spawnOptions = adapted.options;
-    try {
-        child = (0, child_process_1.spawn)(adapted.binary, adapted.args, spawnOptions);
-    }
-    catch (err) {
-        return resolveFromHandoffOrPrompt(handoffFilePath, false, {
-            spawnError: err instanceof Error ? err.message : String(err),
-        });
-    }
-    // Counts SIGINTs while the agent runs. Non-zero after exit means the
-    // user pressed Ctrl+C → skip the ambiguous abort/continue prompt. Prompting
-    // in the TTY state left by a SIGINT-killed child misbehaved under enquirer
-    // (ERR_USE_AFTER_CLOSE, setRawMode EIO) from async chains `await` could not
-    // catch. Whether @clack/prompts shares that fault is unverified, so the skip
-    // stays; reproducing it needs a real terminal.
-    let userInterruptCount = 0;
-    const swallowSigint = () => {
-        userInterruptCount++;
-    };
-    process.on('SIGINT', swallowSigint);
-    const handoffWatchAbort = new AbortController();
-    const exitPromise = waitForExit(child);
-    const handoffPromise = (0, handoff_1.waitForValidHandoff)(handoffFilePath, {
-        signal: handoffWatchAbort.signal,
-        intervalMs: handoffPollIntervalMs,
-    });
-    let exitInfo = {};
-    try {
-        const winner = await Promise.race([
-            exitPromise.then((info) => {
-                exitInfo = info;
-                return 'exit';
-            }),
-            // Rejection handler swallows the abort-triggered rejection from the
-            // `finally` block after the race has settled — otherwise Node logs it
-            // as an unhandled rejection.
-            handoffPromise.then(() => 'handoff', () => 'exit'),
-        ]);
-        if (winner === 'handoff') {
-            await closeAgentSession(child, exitPromise, gracefulExitMs, forceKillWaitMs);
-            // Extra safety bound so the orchestrator can't hang if the child
-            // stays stuck after SIGKILL / taskkill (e.g. D-state on a hung NFS
-            // read, or taskkill returning before the process actually exits).
-            await raceWithTimeout(exitPromise, forceKillWaitMs);
-        }
-    }
-    finally {
-        handoffWatchAbort.abort();
-        process.removeListener('SIGINT', swallowSigint);
-        restoreTermiosAfterAgent();
-    }
-    return resolveFromHandoffOrPrompt(handoffFilePath, userInterruptCount > 0, exitInfoToCause(exitInfo));
-}
-function exitInfoToCause(info) {
-    const cause = {};
-    // Include `code === 0` so the prompt can distinguish "agent exited cleanly
-    // without writing a handoff" (likely the user closed the agent on purpose,
-    // or the agent terminated without invoking the handoff step) from "agent
-    // was killed before it could write" (signal) or "agent crashed" (non-zero
-    // code). Without this distinction every clean-exit-no-handoff collapses
-    // into the same uninformative ambiguous-prompt body.
-    if (info.code !== undefined && info.code !== null) {
-        cause.exitCode = info.code;
-    }
-    if (info.signal)
-        cause.exitSignal = info.signal;
-    if (info.error)
-        cause.exitError = info.error.message;
-    return cause;
-}
-function restoreTermiosAfterAgent() {
-    if (process.platform === 'win32')
-        return;
-    if (!process.stdin.isTTY)
-        return;
-    try {
-        // `stty sane` resets termios to a known cooked state via the kernel —
-        // independent of Node's libuv mode tracking (Node's setRawMode(false)
-        // short-circuits when libuv's per-handle mode is already NORMAL, even
-        // if the OS-level termios was changed out-of-band by the agent).
-        (0, child_process_1.execSync)('stty sane < /dev/tty', {
-            stdio: ['ignore', 'ignore', 'ignore'],
-            windowsHide: true,
-        });
-        // Carriage-return + clear to end of screen, to wipe any agent TUI
-        // cells below our row that subsequent log lines won't overwrite
-        // (e.g. a status footer past where our text wraps).
-        process.stdout.write('\r\x1B[J');
-    }
-    catch {
-        // best-effort — if stty isn't on PATH or /dev/tty isn't accessible,
-        // the worst case is the pre-existing staircase + cell-bleed output.
-    }
-}
-// Safety bound after force-kill. SIGKILL normally reaps in microseconds;
-// the bound exists for uninterruptible kernel calls or taskkill returning
-// before the process actually exits.
-const FORCE_KILL_WAIT_MS = 500;
-/**
- * Stops the agent process after a successful handoff. Platform-branched:
- *
- * - POSIX: SIGINT (graceful, equivalent to user Ctrl+C) → wait
- *   `gracefulExitMs` for the child to exit → SIGKILL → wait
- *   `FORCE_KILL_WAIT_MS` (bounded) → return. SIGTERM is intentionally
- *   skipped: a process that ignores SIGINT for 5s will hit the same
- *   handler on SIGTERM, the extra step only delays the inevitable.
- *
- * - Windows: skip SIGINT entirely. `child.kill('*')` on Windows is a
- *   `TerminateProcess` call regardless of the signal name (Windows has
- *   no POSIX signals), and on the `cmd.exe /d /s /c "..."` shim path it
- *   would terminate cmd.exe while leaving the agent orphaned (parent
- *   death doesn't cascade to children on Windows). `taskkill /T /F`
- *   walks the process tree and kills cmd.exe AND the agent atomically;
- *   that's the only reliable shutdown path here. `taskkill` failures
- *   (binary missing, race with already-dead pid) are swallowed; the
- *   safety bound returns regardless.
- */
-async function closeAgentSession(child, exitPromise, gracefulExitMs, forceKillWaitMs) {
-    if (child.exitCode !== null || child.signalCode !== null)
-        return;
-    if (process.platform === 'win32') {
-        await forceKillWindowsTree(child, exitPromise, forceKillWaitMs);
-        return;
-    }
-    // POSIX path.
-    try {
-        child.kill('SIGINT');
-    }
-    catch {
-        // child already gone between the check above and here
-        return;
-    }
-    let escalation;
-    try {
-        await Promise.race([
-            exitPromise,
-            new Promise((resolve) => {
-                escalation = setTimeout(resolve, gracefulExitMs);
-            }),
-        ]);
-    }
-    finally {
-        if (escalation)
-            clearTimeout(escalation);
-    }
-    if (child.exitCode !== null || child.signalCode !== null)
-        return;
-    // Graceful timeout elapsed without the agent exiting. SIGKILL is
-    // uncatchable; bound the post-kill wait so a pathological uninterruptible
-    // syscall can't hang us forever.
-    try {
-        child.kill('SIGKILL');
-    }
-    catch {
-        /* child already gone */
-    }
-    await raceWithTimeout(exitPromise, forceKillWaitMs);
-}
-async function forceKillWindowsTree(child, exitPromise, forceKillWaitMs) {
-    const pid = child.pid;
-    // `child.pid` is undefined only when spawn itself failed; the early-return
-    // guard in `closeAgentSession` should short-circuit that path. Reaching
-    // here without a pid means a narrow race between handoff-detection and
-    // error-event propagation — without a pid we can't taskkill, so wait
-    // briefly and return.
-    if (pid !== undefined) {
-        try {
-            (0, child_process_1.execSync)(`taskkill /T /F /PID ${pid}`, {
-                stdio: 'ignore',
-                windowsHide: true,
-                // Bound so a hung Windows shell can't block the orchestrator.
-                timeout: 2_000,
-            });
-        }
-        catch {
-            /* taskkill missing, pid already dead, or timed out — fall through */
-        }
-    }
-    await raceWithTimeout(exitPromise, forceKillWaitMs);
-}
-async function raceWithTimeout(promise, timeoutMs) {
-    let timer;
-    try {
-        await Promise.race([
-            promise,
-            new Promise((resolve) => {
-                timer = setTimeout(resolve, timeoutMs);
-            }),
-        ]);
-    }
-    finally {
-        if (timer)
-            clearTimeout(timer);
-    }
-}
-// Merge window so a paired exit + error both land in the same ExitInfo
-// (e.g. error from IPC followed by exit when the process actually
-// terminates). For error-only paths like spawn ENOENT — where Node fires
-// error but never exit — this timer is the SOLE settlement mechanism.
-const EXIT_MERGE_WINDOW_MS = 10;
-function waitForExit(child) {
-    return new Promise((resolve) => {
-        const info = {};
-        let pending = null;
-        let settled = false;
-        const settle = () => {
-            if (settled)
-                return;
-            settled = true;
-            if (pending)
-                clearTimeout(pending);
-            resolve(info);
-        };
-        const onFirst = () => {
-            if (settled || pending)
-                return;
-            pending = setTimeout(settle, EXIT_MERGE_WINDOW_MS);
-        };
-        child.on('exit', (code, signal) => {
-            info.code = code;
-            info.signal = signal;
-            onFirst();
-        });
-        // `error` fires when spawn itself fails (e.g. binary disappeared between
-        // detection and run) OR alongside `exit` when the process started but
-        // emitted an error event later. Treat both as exit with no handoff so
-        // the ambiguous flow kicks in; field-merge so we don't drop the loser's
-        // contribution when both fire.
-        child.on('error', (error) => {
-            info.error = error;
-            onFirst();
-        });
-    });
-}
-async function resolveFromHandoffOrPrompt(handoffFilePath, userInterrupted = false, cause = {}) {
-    const read = (0, handoff_1.readHandoffWithReason)(handoffFilePath);
-    if (read.ok === true) {
-        return {
-            kind: read.handoff.status,
-            summary: read.handoff.summary,
-            extras: read.handoff.extras,
-        };
-    }
-    const fullCause = {
-        ...cause,
-        handoff: { reason: read.reason, detail: read.detail },
-    };
-    if (userInterrupted) {
-        // User pressed Ctrl+C. Don't show the abort/continue prompt — they
-        // already told us what they want. Prompting in the TTY state left by a
-        // SIGINT-killed child also tripped enquirer's setRawMode-EIO and
-        // ERR_USE_AFTER_CLOSE bugs; whether @clack/prompts shares that fault is
-        // unverified, so the skip stays. The orchestrator's standard failure
-        // cascade surfaces the abort outcome.
-        //
-        // Forward the underlying cause as pre-rendered summary lines so the
-        // caller can log it before "Aborted by user" — a Ctrl+C that masked
-        // a SEPARATE crash still needs to show the user what crashed. Scrub
-        // fields that are just the Ctrl+C itself reverberating: exit code
-        // 130 (SIGINT) / 143 (SIGTERM from our escalation) and signals
-        // SIGINT / SIGTERM reflect the user's own keystroke (and our
-        // graceful-exit handling of it); surfacing them as "agent crashed"
-        // would be noise. Anything else — code 1, code 137 (OOM), an
-        // unrelated signal — is a separate diagnostic worth keeping. Note
-        // that `spawnError` is structurally impossible here: the spawn-throw
-        // path returns directly without registering the SIGINT listener, so
-        // `userInterrupted` can never be true on that branch.
-        const exitWasCtrlC = cause.exitCode === 130 ||
-            cause.exitCode === 143 ||
-            cause.exitSignal === 'SIGINT' ||
-            cause.exitSignal === 'SIGTERM';
-        const userScrubbed = {
-            exitError: cause.exitError,
-            exitCode: exitWasCtrlC ? undefined : cause.exitCode,
-            exitSignal: exitWasCtrlC ? undefined : cause.exitSignal,
-            handoff: read.reason === 'missing'
-                ? undefined
-                : { reason: read.reason, detail: read.detail },
-        };
-        const causeSummary = describeAmbiguousCause(userScrubbed);
-        return {
-            kind: 'ambiguous-abort',
-            ...(causeSummary.length > 0 && { causeSummary }),
-        };
-    }
-    return promptAmbiguous(fullCause);
-}
-/**
- * Node's `spawn` cannot directly execute `.cmd` / `.bat` shims on Windows;
- * `which` resolves to those when an agent was installed via npm. Wrap them in
- * a `cmd.exe /d /s /c` invocation with `windowsVerbatimArguments` so quoting
- * follows the cmd.exe convention rather than Node's default cooking.
- *
- * On non-Windows or for non-shim binaries this is a passthrough.
- */
-function adaptSpawnForWindowsShim(binary, args, options) {
-    if (process.platform !== 'win32') {
-        return { binary, args: [...args], options };
-    }
-    const ext = (0, path_1.extname)(binary).toLowerCase();
-    if (ext !== '.cmd' && ext !== '.bat') {
-        return { binary, args: [...args], options };
-    }
-    const cmdLine = [escapeCmdCommand(binary), ...args.map(escapeCmdArg)].join(' ');
-    return {
-        binary: process.env.comspec || 'cmd.exe',
-        // Outer pair of quotes is required so cmd.exe /c does not strip the inner
-        // quotes around the binary path.
-        args: ['/d', '/s', '/c', `"${cmdLine}"`],
-        options: { ...options, windowsVerbatimArguments: true },
-    };
-}
-const CMD_META_CHARS = /([()\][%!^"`<>&|;, ])/g;
-// Backslash-escape embedded quotes per MS C runtime convention, wrap in
-// quotes, then caret-escape cmd.exe metacharacters.
-function escapeCmdArg(arg) {
-    const quoted = `"${arg
-        .replace(/(\\*)"/g, '$1$1\\"')
-        .replace(/(\\*)$/, '$1$1')}"`;
-    return quoted.replace(CMD_META_CHARS, '^$1');
-}
-function escapeCmdCommand(arg) {
-    // cmd.exe interprets the command portion through an extra parsing pass;
-    // apply the caret-escape twice so the .cmd shim sees the original.
-    return escapeCmdArg(arg).replace(CMD_META_CHARS, '^$1');
-}
-async function promptAmbiguous(cause) {
-    // stderr blank line so the spacer lands in the same stream as output.warn
-    // and the prompt — buffered stdout could otherwise reorder it and glue the
-    // prompt to the agent's trailing exit message.
-    process.stderr.write('\n');
-    // Cause rendered ABOVE the prompt rather than inlined into its message:
-    // enquirer's select redraw used different math for clear (wrap-aware) vs
-    // restore (raw \n-split count), so a multi-line message could leave orphaned
-    // cells on arrow-key re-renders. Kept single-line because that constraint is
-    // unverified under @clack/prompts, not because the cause still applies.
-    const causeLines = describeAmbiguousCause(cause);
-    if (causeLines.length > 0) {
-        output_1.output.warn({
-            title: 'The agent run ended without a usable handoff',
-            bodyLines: causeLines,
-        });
-    }
-    // Cancelling exits 130 from inside the prompt; any other rejection is an
-    // abort.
-    try {
-        const choice = await (0, safe_prompt_1.migrateChoice)({
-            message: 'How should nx migrate proceed?',
-            choices: [
-                { value: 'abort', label: 'Treat as failed — abort the run' },
-                {
-                    value: 'continue',
-                    label: 'Treat as completed — mark done and continue',
-                },
-            ],
-        });
-        (0, migrate_analytics_1.reportMigratePrompt)('ambiguous_agent_outcome', choice);
-        return choice === 'continue'
-            ? { kind: 'ambiguous-continue' }
-            : { kind: 'ambiguous-abort' };
-    }
-    catch {
-        return { kind: 'ambiguous-abort' };
-    }
-}
-function describeAmbiguousCause(cause) {
-    const lines = [];
-    if (cause.spawnError) {
-        lines.push(`Could not spawn the agent: ${cause.spawnError}`);
-    }
-    if (cause.exitError) {
-        lines.push(`Agent process emitted an error: ${cause.exitError}`);
-    }
-    if (cause.exitCode !== undefined && cause.exitCode !== null) {
-        // Distinguish clean-exit-without-handoff (likely a deliberate close by
-        // the user, or the agent ended its session without invoking the handoff
-        // step) from a non-zero crash, so the prompt body doesn't collapse them
-        // into the same uninformative "exited with code N" line.
-        lines.push(cause.exitCode === 0
-            ? 'Agent exited cleanly (code 0) without writing a handoff.'
-            : `Agent exited with code ${cause.exitCode}.`);
-    }
-    if (cause.exitSignal) {
-        lines.push(`Agent was terminated by signal ${cause.exitSignal}.`);
-    }
-    // "No handoff file was written." is redundant when another cause line
-    // already implies it (spawn error, exit code/signal, process error).
-    // Independent diagnostics (read-error, parse-error, shape-mismatch) always
-    // surface — they're not implied by the exit shape.
-    const handoffMissingIsRedundant = !!cause.spawnError ||
-        !!cause.exitError ||
-        cause.exitCode !== undefined ||
-        cause.exitSignal !== undefined;
-    switch (cause.handoff?.reason) {
-        case 'missing':
-            if (!handoffMissingIsRedundant) {
-                lines.push('No handoff file was written.');
-            }
-            break;
-        case 'read-error':
-            lines.push(`Handoff file could not be read${cause.handoff.detail ? `: ${cause.handoff.detail}` : '.'}`);
-            break;
-        case 'parse-error':
-            lines.push(`Handoff file contained invalid JSON${cause.handoff.detail ? `: ${cause.handoff.detail}` : '.'}`);
-            break;
-        case 'shape-mismatch':
-            lines.push('Handoff file was missing required fields or had an unexpected shape.');
-            break;
-    }
-    return lines;
-}