@contentful/experience-design-system-cli 2.12.1-dev-build-589b615.0 → 2.12.1-dev-build-0b7de73.0

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contentful/experience-design-system-cli",
-  "version": "2.12.1-dev-build-589b615.0",
+  "version": "2.12.1-dev-build-0b7de73.0",
   "description": "Contentful Experiences design system import CLI",
   "license": "MIT",
   "type": "module",

package/dist/src/analyze/select-agent/command.js

@@ -167,6 +167,7 @@ async function selectBatch(agent, model, batch, total, verbose, skillPathOverrid
         const call = calls.find((toolCall) => toolCall.name === component.name);
         if (!call) {
             process.stderr.write(`  ${pos}  ${c.bold(component.name)}  ${c.yellow('no tool call')}\n`);
+            emitProgress(item, 'failed', 'no-tool-call-from-agent');
             results.push({
                 componentKey: componentKey(component),
                 componentName: component.name,

package/dist/src/import/command.js

@@ -7,6 +7,7 @@ import { DEFAULT_CONFIGURED_HOST, toConfiguredHost } from '../host-utils.js';
 import { replayRun, modifyRun } from '../runs/replay-helpers.js';
 import { resolvePromptFlags } from './print-prompt.js';
 import { shouldShowRunPicker } from '../runs/run-picker-mount.js';
+import { dispatchPickerSelection } from './picker-dispatch.js';
 export function registerImportCommand(program) {
     program
         .command('import')
@@ -220,15 +221,20 @@ export function registerImportCommand(program) {
                 isTTY: !!process.stdin.isTTY,
             });
             let pickerSelection = null;
+            // Ink `unmount` handle captured after `render(...)` below so the
+            // picker callback can tear down the wizard cleanly. Prior to this
+            // fix the callback did `setImmediate(() => process.exit(0))`, which
+            // killed the process before `dispatchPickerSelection` could run.
+            let unmountInk = null;
             const pickerProps = {};
             if (pickerDecision.shouldShow) {
                 pickerProps.initialRuns = pickerDecision.runs;
                 pickerProps.onRunPicked = (selection) => {
                     pickerSelection = selection;
-                    setImmediate(() => process.exit(0));
+                    unmountInk?.();
                 };
             }
-            const { waitUntilExit } = render(createElement(WizardApp, {
+            const { waitUntilExit, unmount } = render(createElement(WizardApp, {
                 initialSpaceId: creds.spaceId,
                 initialEnvironmentId: creds.environmentId || 'master',
                 initialCmaToken: creds.cmaToken,
@@ -250,43 +256,24 @@ export function registerImportCommand(program) {
                 ...(opts.rawTokens ? { initialRawTokensPath: resolve(opts.rawTokens) } : {}),
                 ...pickerProps,
             }));
-            try {
-                await waitUntilExit();
-            }
-            catch {
-                /* Ink throws on process.exit; swallow so picker dispatch can run. */
-            }
+            unmountInk = unmount;
+            await waitUntilExit();
             // ── Picker dispatch ─────────────────────────────────────────────
-            // If the operator picked a run, route into the existing entry
-            // points so credential resolution and mutex checks stay in one
-            // place. The --modify path resolves the run record via
-            // resolveRunTarget and threads its session IDs through
-            // launchModifyWizard (see runs/replay-helpers.ts).
+            // If the operator picked a run, route into replayRun / modifyRun.
+            // dispatchPickerSelection lives in ./picker-dispatch.ts so this
+            // decision is testable without an Ink runtime.
             if (pickerSelection) {
-                const sel = pickerSelection;
-                if (sel.action === 'push' && sel.runId) {
-                    await replayRun({
-                        runIdOrPath: sel.runId,
-                        ...(opts.spaceId ? { spaceId: opts.spaceId } : {}),
-                        ...(opts.environmentId ? { environmentId: opts.environmentId } : {}),
-                        ...(opts.cmaToken ? { cmaToken: opts.cmaToken } : {}),
-                        ...(opts.host ? { host: opts.host } : {}),
-                        interactive: !!process.stdout.isTTY,
-                        ...(opts.force ? { force: true } : {}),
-                    });
-                    return;
-                }
-                if (sel.action === 'modify' && sel.runId) {
-                    await modifyRun({
-                        runIdOrPath: sel.runId,
-                        ...(opts.outDir ? { outDir: opts.outDir } : {}),
-                        ...(opts.overwrite ? { overwrite: true } : {}),
-                        ...(opts.saveAsNew ? { saveAsNew: true } : {}),
-                        ...(opts.force ? { force: true } : {}),
-                    });
-                    return;
-                }
-                // action === 'new' falls through — the wizard already advanced.
+                await dispatchPickerSelection(pickerSelection, {
+                    ...(opts.spaceId ? { spaceId: opts.spaceId } : {}),
+                    ...(opts.environmentId ? { environmentId: opts.environmentId } : {}),
+                    ...(opts.cmaToken ? { cmaToken: opts.cmaToken } : {}),
+                    ...(opts.host ? { host: opts.host } : {}),
+                    interactive: !!process.stdout.isTTY,
+                    ...(opts.outDir ? { outDir: opts.outDir } : {}),
+                    ...(opts.overwrite ? { overwrite: true } : {}),
+                    ...(opts.saveAsNew ? { saveAsNew: true } : {}),
+                    ...(opts.force ? { force: true } : {}),
+                }, { replayRun, modifyRun });
             }
             return;
         }

package/dist/src/import/picker-dispatch.d.ts

@@ -0,0 +1,18 @@
+import type { RunPickerSelection } from '../runs/run-picker.js';
+import type { replayRun as replayRunFn, modifyRun as modifyRunFn } from '../runs/replay-helpers.js';
+export type PickerDispatchOptions = {
+    spaceId?: string;
+    environmentId?: string;
+    cmaToken?: string;
+    host?: string;
+    interactive?: boolean;
+    outDir?: string;
+    overwrite?: boolean;
+    saveAsNew?: boolean;
+    force?: boolean;
+};
+export type PickerDispatchDeps = {
+    replayRun: typeof replayRunFn;
+    modifyRun: typeof modifyRunFn;
+};
+export declare function dispatchPickerSelection(selection: RunPickerSelection, opts: PickerDispatchOptions, deps: PickerDispatchDeps): Promise<void>;

package/dist/src/import/picker-dispatch.js

@@ -0,0 +1,29 @@
+// Route a resolved RunPickerSelection into replayRun / modifyRun. Extracted
+// from command.ts so the dispatch decision is testable without an Ink runtime
+// and so the picker callback in command.ts is a simple state-capture (no
+// process.exit shenanigans that would kill the process before dispatch runs).
+export async function dispatchPickerSelection(selection, opts, deps) {
+    if (selection.action === 'push' && selection.runId) {
+        await deps.replayRun({
+            runIdOrPath: selection.runId,
+            ...(opts.spaceId ? { spaceId: opts.spaceId } : {}),
+            ...(opts.environmentId ? { environmentId: opts.environmentId } : {}),
+            ...(opts.cmaToken ? { cmaToken: opts.cmaToken } : {}),
+            ...(opts.host ? { host: opts.host } : {}),
+            ...(opts.interactive !== undefined ? { interactive: opts.interactive } : {}),
+            ...(opts.force ? { force: true } : {}),
+        });
+        return;
+    }
+    if (selection.action === 'modify' && selection.runId) {
+        await deps.modifyRun({
+            runIdOrPath: selection.runId,
+            ...(opts.outDir ? { outDir: opts.outDir } : {}),
+            ...(opts.overwrite ? { overwrite: true } : {}),
+            ...(opts.saveAsNew ? { saveAsNew: true } : {}),
+            ...(opts.force ? { force: true } : {}),
+        });
+        return;
+    }
+    // action === 'new' → no-op; the wizard already advanced past the picker.
+}

package/dist/src/import/tui/WizardApp.d.ts

@@ -19,7 +19,7 @@ export declare function buildSelectAgentArgs(opts: {
 export type AutoFilterProgress = {
     n: number;
     total: number;
-    decision: 'accepted' | 'rejected';
+    decision: 'accepted' | 'rejected' | 'failed';
     name: string;
     reason: string;
 };

package/dist/src/import/tui/WizardApp.js

@@ -40,6 +40,7 @@ import { readTokensFromPath, hasBreakingChangesWithImpact } from '../../apply/ma
 import { buildManifest } from '@contentful/experience-design-system-types';
 import { openPipelineDb, loadCDFComponents, loadScopeComponents, seedCDFFromPreviewResponse, seedDefaultsFromChangedItems, backfillUnclassifiedProps, } from '../../session/db.js';
 import { ScopeGateHost } from './scope-gate-host.js';
+import { mergeAiDecisions } from './merge-ai-decisions.js';
 import { FinalReviewHost } from './final-review-host.js';
 import { runScopeGate } from './runScopeGate.js';
 import { buildAutoFilterErrorTail } from './auto-filter-error.js';
@@ -83,7 +84,7 @@ export function parseAutoFilterProgressLine(line) {
     const counterMatch = /^(\d+)\/(\d+)$/.exec(counter);
     if (!counterMatch)
         return null;
-    if (decision !== 'accepted' && decision !== 'rejected')
+    if (decision !== 'accepted' && decision !== 'rejected' && decision !== 'failed')
         return null;
     if (!name)
         return null;
@@ -1399,6 +1400,11 @@ export function WizardApp({ initialSpaceId = '', initialEnvironmentId = 'master'
                 finally {
                     db.close();
                 }
+                // INTEG-4318: overlay the streamed auto-filter decisions (from
+                // stderr progress lines) onto DB-loaded rows so 'failed' components
+                // (LLM omitted a tool call in a batch) surface in the scope-gate
+                // instead of silently defaulting to included.
+                components = mergeAiDecisions(components, state.aiDecisions);
                 return (_jsx(ScopeGateHost, { components: components, autoAccept: autoAcceptScope, aiFilterStatus: state.aiFilterStatus, aiFilterProgress: state.aiFilterProgress, aiFilterError: state.aiFilterError, onCancelAutoFilter: cancelAutoFilter, onConfirm: (decisions) => {
                         void runScopeGate({
                             sessionId,

package/dist/src/import/tui/merge-ai-decisions.d.ts

@@ -0,0 +1,5 @@
+import type { ScopeComponent } from './steps/ScopeGateStep.js';
+export declare function mergeAiDecisions(components: ReadonlyArray<ScopeComponent>, aiDecisions: Record<string, {
+    decision: 'accepted' | 'rejected' | 'failed';
+    reason: string;
+}>): ScopeComponent[];

package/dist/src/import/tui/merge-ai-decisions.js

@@ -0,0 +1,22 @@
+// INTEG-4318: overlay streamed auto-filter decisions (from the select-agent
+// child's stderr progress lines) onto components loaded from raw_components.
+// Only fills gaps where the DB has no decision — the DB row is authoritative
+// once the child persists its status. This is what lets the scope-gate see
+// a 'failed' status for components where the LLM omitted a tool call in a
+// batch (the select-agent does not persist a 'failed' status to the DB).
+export function mergeAiDecisions(components, aiDecisions) {
+    return components.map((component) => {
+        if (component.aiDecision !== null && component.aiDecision !== undefined) {
+            return component;
+        }
+        const streamed = aiDecisions[component.name];
+        if (!streamed) {
+            return component;
+        }
+        return {
+            ...component,
+            aiDecision: streamed.decision,
+            aiReason: streamed.reason,
+        };
+    });
+}

package/dist/src/import/tui/steps/ScopeGateStep.d.ts

@@ -2,7 +2,7 @@ import React from 'react';
 export type ScopeComponent = {
     name: string;
     componentId: string;
-    aiDecision?: 'accepted' | 'rejected' | null;
+    aiDecision?: 'accepted' | 'rejected' | 'failed' | null;
     aiReason?: string | null;
 };
 export type ScopeGateStepProps = {

package/dist/src/import/tui/steps/ScopeGateStep.js

@@ -20,7 +20,11 @@ function truncateReason(reason) {
     return reason.slice(0, REASON_DISPLAY_MAX - 1).trimEnd() + '…';
 }
 function isAiFlagged(row) {
-    return row.aiDecision === 'rejected';
+    // INTEG-4318: `failed` means the LLM omitted a decision for this component
+    // (e.g. batch under-emit). Surface these in the AI-recommended-exclusions
+    // section so the operator sees them and can override — silent inclusion was
+    // the bug.
+    return row.aiDecision === 'rejected' || row.aiDecision === 'failed';
 }
 export function ScopeGateStep({ components, onConfirm, onQuit, aiFilterStatus = 'idle', aiFilterProgress = null, aiFilterError = null, onCancelAutoFilter, }) {
     // Pilot-2026-06-25: scope-gate UX overhaul — single unified list.
@@ -60,7 +64,10 @@ export function ScopeGateStep({ components, onConfirm, onQuit, aiFilterStatus =
             return false;
         if (userUnExcluded.has(row.name))
             return true;
-        return row.aiDecision !== 'rejected';
+        // INTEG-4318: exclude on 'rejected' AND 'failed'. Missing/null aiDecision
+        // (auto-filter not run, or component never seen) still defaults to
+        // included so --no-auto-filter and skip-credentials flows are unchanged.
+        return row.aiDecision !== 'rejected' && row.aiDecision !== 'failed';
     };
     const partition = () => {
         const accepted = [];
@@ -216,7 +223,7 @@ export function ScopeGateStep({ components, onConfirm, onQuit, aiFilterStatus =
     const showCancelledBanner = aiFilterStatus === 'cancelled';
     const showFailedBanner = aiFilterStatus === 'failed';
     const allRejected = aiFilterStatus === 'complete' && total > 0 && flatList.every((c) => !isIncluded(c));
-    return (_jsxs(Box, { flexDirection: "column", gap: 1, paddingX: 2, paddingY: 1, children: [_jsx(Text, { color: "green", children: "\u2713 Extraction complete" }), _jsxs(Text, { dimColor: true, children: ["Found ", total, " component", total === 1 ? '' : 's', ". Pick which ones to import. Generation runs only on the included set."] }), showRunningHeader && (_jsx(Box, { flexDirection: "column", marginTop: 1, children: _jsxs(Text, { color: "cyan", children: ["[AI filtering (", aiFilterProgress.done, "/", aiFilterProgress.total, ")\u2026] ", _jsx(Text, { dimColor: true, children: "[q] cancels" })] }) })), showCancelledBanner && (_jsx(Box, { marginTop: 1, children: _jsxs(Text, { color: "yellow", children: ["AI auto-filter cancelled", aiFilterProgress ? ` at ${aiFilterProgress.done}/${aiFilterProgress.total}` : '', ". Review remaining manually."] }) })), showFailedBanner && (_jsx(Box, { marginTop: 1, children: _jsxs(Text, { color: "yellow", children: ["AI auto-filter failed: ", aiFilterError ?? 'unknown error', ". Continuing without AI suggestions."] }) })), reasonPanelOpen && flatList[cursor]?.aiDecision === 'rejected' && (_jsxs(Box, { flexDirection: "column", borderStyle: "single", borderColor: "gray", paddingX: 1, marginTop: 1, children: [_jsx(Text, { dimColor: true, bold: true, children: `AI rejection reason: ${flatList[cursor].name}` }), _jsx(Text, { children: flatList[cursor].aiReason ?? '<no reason given>' }), _jsx(Text, { dimColor: true, children: "[s] close \u00B7 [Esc] close" })] })), allRejected ? (_jsx(Box, { marginTop: 1, children: _jsx(Text, { color: "yellow", children: "AI excluded all components \u2014 press [a] to override or [q] to quit" }) })) : (_jsxs(Box, { flexDirection: "column", marginTop: 1, children: [above > 0 && _jsxs(Text, { dimColor: true, children: ["\u2191 ", above, " above"] }), visible.map((c, vi) => {
+    return (_jsxs(Box, { flexDirection: "column", gap: 1, paddingX: 2, paddingY: 1, children: [_jsx(Text, { color: "green", children: "\u2713 Extraction complete" }), _jsxs(Text, { dimColor: true, children: ["Found ", total, " component", total === 1 ? '' : 's', ". Pick which ones to import. Generation runs only on the included set."] }), showRunningHeader && (_jsx(Box, { flexDirection: "column", marginTop: 1, children: _jsxs(Text, { color: "cyan", children: ["[AI filtering (", aiFilterProgress.done, "/", aiFilterProgress.total, ")\u2026] ", _jsx(Text, { dimColor: true, children: "[q] cancels" })] }) })), showCancelledBanner && (_jsx(Box, { marginTop: 1, children: _jsxs(Text, { color: "yellow", children: ["AI auto-filter cancelled", aiFilterProgress ? ` at ${aiFilterProgress.done}/${aiFilterProgress.total}` : '', ". Review remaining manually."] }) })), showFailedBanner && (_jsx(Box, { marginTop: 1, children: _jsxs(Text, { color: "yellow", children: ["AI auto-filter failed: ", aiFilterError ?? 'unknown error', ". Continuing without AI suggestions."] }) })), reasonPanelOpen && flatList[cursor] !== undefined && isAiFlagged(flatList[cursor]) && (_jsxs(Box, { flexDirection: "column", borderStyle: "single", borderColor: "gray", paddingX: 1, marginTop: 1, children: [_jsx(Text, { dimColor: true, bold: true, children: `AI rejection reason: ${flatList[cursor].name}` }), _jsx(Text, { children: flatList[cursor].aiReason ?? '<no reason given>' }), _jsx(Text, { dimColor: true, children: "[s] close \u00B7 [Esc] close" })] })), allRejected ? (_jsx(Box, { marginTop: 1, children: _jsx(Text, { color: "yellow", children: "AI excluded all components \u2014 press [a] to override or [q] to quit" }) })) : (_jsxs(Box, { flexDirection: "column", marginTop: 1, children: [above > 0 && _jsxs(Text, { dimColor: true, children: ["\u2191 ", above, " above"] }), visible.map((c, vi) => {
                         const i = vi + scrollOffset;
                         const isCursor = i === cursor;
                         const included = isIncluded(c);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contentful/experience-design-system-cli",
-  "version": "2.12.1-dev-build-589b615.0",
+  "version": "2.12.1-dev-build-0b7de73.0",
   "description": "Contentful Experiences design system import CLI",
   "license": "MIT",
   "type": "module",
@@ -37,7 +37,7 @@
     "svelte": "^5.56.4",
     "ts-morph": "^27.0.2",
     "typescript": "^5.9.3",
-    "@contentful/experience-design-system-types": "2.12.1-dev-build-589b615.0"
+    "@contentful/experience-design-system-types": "2.12.1-dev-build-0b7de73.0"
   },
   "devDependencies": {
     "@tsconfig/node24": "^24.0.3",