@contentful/experience-design-system-cli 2.12.4-dev-build-ff57340.0 → 2.13.1-dev-build-fa478b3.0

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

@@ -8,8 +8,7 @@ import { StatusBar } from '../../../analyze/select/tui/components/StatusBar.js';
 import { FinalizeDialog } from '../../../analyze/select/tui/components/FinalizeDialog.js';
 import { QuitDialog } from '../../../analyze/select/tui/components/QuitDialog.js';
 import { useImmediateInput } from '../../../analyze/select/tui/hooks/useImmediateInput.js';
-import { openPipelineDb, loadCDFComponents, storeCDFComponents, loadComponentReviewMetadata, loadComponentRationale, loadSlotCycles, storeSlotCycles, } from '../../../session/db.js';
-import { formatCyclePathSegments, findSlotCycles, suggestCycleBreakEdge } from '../../../analyze/cycle-detection.js';
+import { openPipelineDb, loadCDFComponents, storeCDFComponents, loadComponentReviewMetadata, loadComponentRationale, } from '../../../session/db.js';
 import { RationalePanel } from '../../../analyze/select/tui/components/RationalePanel.js';
 import { ComponentRationalePanel } from '../../../analyze/select/tui/components/ComponentRationalePanel.js';
 import { applyPreviewAnnotations } from '../../../analyze/select/preview-annotations.js';
@@ -25,22 +24,12 @@ import { computeNextScrollOffset } from '../../../analyze/select/tui/hooks/scrol
  *
  * Within each tier (empty / non-empty) we tie-break alphabetically by `key`.
  */
-export function sortComponentsForSidebar(components, cycleParticipants) {
-    const isEmpty = (entry) => Object.keys(entry.$properties ?? {}).length === 0 && Object.keys(entry.$slots ?? {}).length === 0;
-    // Tier order: cycle members first (they block push — surface loudest),
-    // then empty (soft warning), then everything else. Ties broken alpha.
-    const tier = (c) => {
-        if (cycleParticipants?.has(c.key))
-            return 0;
-        if (isEmpty(c.entry))
-            return 1;
-        return 2;
-    };
+export function sortComponentsForSidebar(components) {
     return [...components].sort((a, b) => {
-        const at = tier(a);
-        const bt = tier(b);
-        if (at !== bt)
-            return at - bt;
+        const aEmpty = Object.keys(a.entry.$properties ?? {}).length === 0;
+        const bEmpty = Object.keys(b.entry.$properties ?? {}).length === 0;
+        if (aEmpty !== bEmpty)
+            return aEmpty ? -1 : 1;
         return a.key.localeCompare(b.key);
     });
 }
@@ -86,12 +75,6 @@ export function GenerateReviewStep({ extractSessionId, onFinalize, onQuit, liveP
     // Tracks the first `g` of a potential `gg` double-tap (jumps to top in
     // JSON-view + panel-focused state). Reset on any non-`g` key.
     const pendingGRef = useRef(false);
-    // INTEG-4401: slot-dependency cycles loaded from the session DB. Non-empty
-    // triggers sidebar (cycle) badges, banner + [c] detail-panel affordance,
-    // and (at push time) a hard block.
-    const [slotCycles, setSlotCycles] = useState([]);
-    const [showCyclePanel, setShowCyclePanel] = useState(false);
-    const [cyclePanelScroll, setCyclePanelScroll] = useState(0);
     const handleLivePreviewResult = (response) => {
         if (!response)
             return;
@@ -123,10 +106,8 @@ export function GenerateReviewStep({ extractSessionId, onFinalize, onQuit, liveP
         async function load() {
             const db = openPipelineDb();
             let cdfComponents;
-            let cycles = [];
             try {
                 cdfComponents = loadCDFComponents(db, extractSessionId);
-                cycles = loadSlotCycles(db, extractSessionId);
             }
             finally {
                 db.close();
@@ -141,12 +122,7 @@ export function GenerateReviewStep({ extractSessionId, onFinalize, onQuit, liveP
                 entry,
                 status: 'needs-review',
             }));
-            const cycleParticipants = new Set();
-            for (const c of cycles)
-                for (const p of c.path)
-                    cycleParticipants.add(p);
-            setSlotCycles(cycles);
-            setComponents(sortComponentsForSidebar(reviewEntries, cycleParticipants));
+            setComponents(sortComponentsForSidebar(reviewEntries));
             setLoading(false);
         }
         load().catch((e) => {
@@ -253,66 +229,6 @@ export function GenerateReviewStep({ extractSessionId, onFinalize, onQuit, liveP
         }
         onFinalize(acceptedCount, explicitlyRejected.length, unresolved.length);
     };
-    /**
-     * INTEG-4401 (Fix 3/4): re-run cycle detection against the current in-memory
-     * component state and, if the result differs from the persisted `slotCycles`
-     * state, update both React state and the session DB. Called from:
-     *  - `handleEditSave` after a FieldEditor save mutates a slot's
-     *    `$allowedComponents`, so the banner / sidebar badges / [c] panel reflect
-     *    reality instead of the stale extract-time snapshot.
-     *  - The `r` reject keystroke, since dropping a component from the manifest
-     *    can collapse cycles that routed through it.
-     *
-     * Rejected components are excluded from the graph — they will never ship, so
-     * they cannot contribute to a real push-time cycle even if their slot config
-     * still references other components locally.
-     *
-     * Cheap for typical N: findSlotCycles is O((V+E)(C+1)) and most manifests
-     * have zero or a handful of cycles. Wrapped in try/catch so a malformed
-     * $slots shape can't crash the render pipeline.
-     */
-    const recomputeCycles = (currentComponents) => {
-        try {
-            const graph = currentComponents
-                .filter((c) => c.status !== 'rejected')
-                .map((c) => ({
-                name: c.key,
-                slots: Object.entries(c.entry.$slots ?? {}).map(([slotName, slotDef]) => ({
-                    name: slotName,
-                    allowedComponents: Array.isArray(slotDef?.$allowedComponents)
-                        ? slotDef.$allowedComponents.filter((v) => typeof v === 'string')
-                        : [],
-                })),
-            }));
-            const rawCycles = findSlotCycles(graph);
-            const next = rawCycles.map((cycle) => ({
-                path: cycle.path,
-                edges: cycle.edges,
-                suggestedBreak: cycle.edges.length > 0 ? suggestCycleBreakEdge(cycle, rawCycles) : null,
-            }));
-            // Cheap structural equality via JSON.stringify — cycle count is tiny and
-            // this only fires on user actions, not on every render.
-            const prevSerialized = JSON.stringify(slotCycles);
-            const nextSerialized = JSON.stringify(next);
-            if (prevSerialized === nextSerialized)
-                return;
-            setSlotCycles(next);
-            const db = openPipelineDb();
-            try {
-                storeSlotCycles(db, extractSessionId, next);
-            }
-            finally {
-                db.close();
-            }
-            // If the new cycle set is empty, clear any lingering finalize error the
-            // [F] guard may have surfaced. Otherwise leave it alone.
-            if (next.length === 0)
-                setFinalizeError(null);
-        }
-        catch {
-            // Defensive: swallow — never let cycle detection crash the review UI.
-        }
-    };
     const handleEditSave = () => {
         const current = components[selectedIdx];
         if (!current)
@@ -328,11 +244,7 @@ export function GenerateReviewStep({ extractSessionId, onFinalize, onQuit, liveP
                 setSaveError('Invalid CDF entry: must have $type: "component" and $properties object');
                 return;
             }
-            let updatedComponents = [];
-            setComponents((prev) => {
-                updatedComponents = prev.map((c, i) => i === selectedIdx ? { ...c, entry, status: c.status === 'needs-review' ? 'accepted' : c.status } : c);
-                return updatedComponents;
-            });
+            setComponents((prev) => prev.map((c, i) => i === selectedIdx ? { ...c, entry, status: c.status === 'needs-review' ? 'accepted' : c.status } : c));
             setDraftValue('');
             setSaveError(null);
             const db = openPipelineDb();
@@ -342,12 +254,6 @@ export function GenerateReviewStep({ extractSessionId, onFinalize, onQuit, liveP
             finally {
                 db.close();
             }
-            // INTEG-4401 (Fix 3/4): recompute cycles against the freshly-edited
-            // component set so banner/badges/[c]-panel stay in sync. The FieldEditor
-            // picker filters cycle-forming candidates, but free-text edits + JSON
-            // pastes still slip through, and edits that BREAK an existing cycle
-            // won't propagate without this call.
-            recomputeCycles(updatedComponents);
             // Feature 2: re-fire the live preview now that pipeline.db reflects
             // the new state. The hook owns debounce + cred-missing short-circuit.
             livePreviewHook.trigger();
@@ -376,31 +282,6 @@ export function GenerateReviewStep({ extractSessionId, onFinalize, onQuit, liveP
             }
             return;
         }
-        // INTEG-4401: slot-cycle detail panel. Same modal-swallow rules as
-        // showRemovedPanel; q/Esc close, ↑↓ scroll.
-        if (showCyclePanel) {
-            if (input === 'c' || input === 'q' || key.escape) {
-                setShowCyclePanel(false);
-                setCyclePanelScroll(0);
-                return;
-            }
-            if (key.upArrow || input === 'k') {
-                setCyclePanelScroll((v) => Math.max(0, v - 1));
-                return;
-            }
-            if (key.downArrow || input === 'j') {
-                setCyclePanelScroll((v) => v + 1);
-                return;
-            }
-            return;
-        }
-        // Open the cycle panel from sidebar-focused state when there is at
-        // least one cycle to display.
-        if (input === 'c' && sidebarFocused && slotCycles.length > 0) {
-            setShowCyclePanel(true);
-            setCyclePanelScroll(0);
-            return;
-        }
         // `d` opens the panel only when live-preview is enabled and there is at
         // least one removed component to display. Sidebar-focused only so it
         // doesn't collide with FieldEditor input.
@@ -530,14 +411,6 @@ export function GenerateReviewStep({ extractSessionId, onFinalize, onQuit, liveP
             return;
         }
         if (input === 'F') {
-            // INTEG-4401: refuse to open the finalize dialog while any slot cycle
-            // is unresolved. The push-time hard block (`assertNoSlotCycles`) would
-            // catch this downstream, but showing an inline banner here saves the
-            // operator from confirming the dialog only to hit a stderr crash.
-            if (slotCycles.length > 0) {
-                setFinalizeError('Cannot finalize — resolve slot dependency cycle(s) first (press [c] for detail)');
-                return;
-            }
             setShowFinalize(true);
             return;
         }
@@ -547,14 +420,7 @@ export function GenerateReviewStep({ extractSessionId, onFinalize, onQuit, liveP
             return;
         }
         if (input === 'r') {
-            // INTEG-4401 (Fix 4): rejecting removes the component from the effective
-            // manifest, so recompute cycles across the reduced graph — any cycle
-            // that routed through this component collapses. We build the "next"
-            // components array inline (mirroring updateStatus) so recomputeCycles
-            // sees the post-update graph without waiting for a render tick.
-            const next = components.map((c, i) => i === selectedIdx ? { ...c, status: 'rejected' } : c);
-            setComponents(next);
-            recomputeCycles(next);
+            updateStatus(selectedIdx, 'rejected');
             return;
         }
         if (input === 'A') {
@@ -607,89 +473,37 @@ export function GenerateReviewStep({ extractSessionId, onFinalize, onQuit, liveP
     }
     const selected = components[selectedIdx] ?? null;
     const selectedJson = selected ? JSON.stringify({ [selected.key]: selected.entry }, null, 2) : '';
-    const isEmpty = (c) => Object.keys(c.entry.$properties).length === 0 && Object.keys(c.entry.$slots ?? {}).length === 0;
+    // A component with zero classified $properties is a real defensibility issue —
+    // it can't be pushed to Contentful (no fields). Surface it in the sidebar via
+    // the existing warning-color path (yellow) and a "(empty)" suffix so the user
+    // can see what went wrong. They can manually add props in FieldEditor or
+    // explicitly reject the component.
+    const isEmpty = (c) => Object.keys(c.entry.$properties).length === 0;
     const emptyCount = components.filter(isEmpty).length;
-    // INTEG-4401: cycle-participant set drives sidebar `(cycle)` badges plus
-    // the banner counts. Recomputed on every render — cheap for typical N.
-    const cycleParticipantSet = new Set();
-    for (const cycle of slotCycles)
-        for (const p of cycle.path)
-            cycleParticipantSet.add(p);
-    const sidebarSuffix = (c) => {
-        if (cycleParticipantSet.has(c.key))
-            return ' (cycle)';
-        if (isEmpty(c))
-            return ' (empty)';
-        return '';
-    };
     const sidebarItems = components.map((c) => ({
         id: c.key,
-        name: `${c.key}${sidebarSuffix(c)}`,
+        name: isEmpty(c) ? `${c.key} (empty)` : c.key,
         status: c.status,
         previewAnnotation: previewAnnotations.get(c.key),
         extractionConfidence: null,
         needsReview: false,
         validationErrorCount: 0,
-        validationWarningCount: sidebarSuffix(c) !== '' ? 1 : 0,
+        validationWarningCount: isEmpty(c) ? 1 : 0,
     }));
-    // Account for the "(cycle)" / "(empty)" suffix added to badged component
-    // names so the sidebar doesn't truncate them.
-    const longestName = components.reduce((m, c) => Math.max(m, c.key.length + sidebarSuffix(c).length), 0);
+    // Account for the "(empty)" suffix added to zero-prop component names so the
+    // sidebar doesn't truncate it.
+    const longestName = components.reduce((m, c) => Math.max(m, c.key.length + (isEmpty(c) ? ' (empty)'.length : 0)), 0);
     // +5 = border (1) + status icon (1) + badge column (1) + space (1) + border (1).
     // The badge column is reserved even when no annotation is present so the
     // sidebar width doesn't jitter as live-preview annotations flip in/out.
     const sidebarWidth = Math.min(Math.max(longestName + 5, 14), 30);
     const panelWidth = Math.max(10, terminalWidth - sidebarWidth - 4);
-    // INTEG-4401: project-wide slot graph passed to FieldEditor so its
-    // $allowedComponents picker can filter out cycle-forming candidates. Built
-    // from every review entry's $slots — includes accepted, rejected, and
-    // needs-review components so the graph reflects what will actually be
-    // pushed. The FieldEditor replaces its own entry in-simulation with its
-    // live editor state, so pending edits are always reflected.
-    const projectSlotGraph = components.map((c) => ({
-        name: c.key,
-        slots: Object.entries(c.entry.$slots ?? {}).map(([slotName, slotDef]) => ({
-            name: slotName,
-            allowedComponents: Array.isArray(slotDef?.$allowedComponents)
-                ? slotDef.$allowedComponents.filter((v) => typeof v === 'string')
-                : [],
-        })),
-    }));
     const accepted = components.filter((c) => c.status === 'accepted').length;
     const rejected = components.filter((c) => c.status === 'rejected').length;
     const needsReview = components.filter((c) => c.status === 'needs-review').length;
     const propCount = selected ? Object.keys(selected.entry.$properties).length : 0;
     const slotCount = selected?.entry.$slots ? Object.keys(selected.entry.$slots).length : 0;
-    return (_jsxs(Box, { flexDirection: "column", children: [showFinalize && (_jsx(FinalizeDialog, { accepted: accepted, rejected: rejected, needsReview: needsReview, onConfirm: handleFinalizeConfirm, onCancel: () => setShowFinalize(false) })), showQuit && _jsx(QuitDialog, { hasUnsavedDrafts: false, onConfirm: onQuit, onCancel: () => setShowQuit(false) }), showRemovedPanel && !dialogOpen && (_jsxs(Box, { flexDirection: "column", borderStyle: "round", borderColor: "cyan", paddingX: 1, children: [_jsx(Text, { bold: true, color: "cyan", children: `Removed components (${removedComponents.length})` }), _jsx(Text, { dimColor: true, children: "these will be DELETED from the target space" }), _jsx(Text, { children: " " }), removedComponents.map((rc) => (_jsx(Text, { children: `- ${rc.name}${rc.id ? `  (${rc.id})` : ''}` }, rc.id))), _jsx(Text, { children: " " }), _jsx(Text, { dimColor: true, children: "press d or Esc to close" })] })), showCyclePanel &&
-                !dialogOpen &&
-                (() => {
-                    // Materialize the full panel body as a flat list of Text lines,
-                    // then slice by cyclePanelScroll so ↑↓ can walk arbitrarily long
-                    // content. Each cycle contributes 3-4 lines: heading, path,
-                    // suggested fix (if any), and a blank separator.
-                    const PANEL_H = 20;
-                    const lines = [];
-                    lines.push(_jsx(Text, { bold: true, color: "yellow", children: `SLOT DEPENDENCY CYCLES (${slotCycles.length})` }, "cyc-title"));
-                    lines.push(_jsx(Text, { dimColor: true, children: 'push will fail until these are resolved' }, "cyc-sub"));
-                    lines.push(_jsx(Text, { children: " " }, "cyc-space"));
-                    slotCycles.forEach((cycle, idx) => {
-                        const nodeCount = new Set(cycle.path).size;
-                        lines.push(_jsx(Text, { bold: true, children: `▸ Cycle ${idx + 1} (${nodeCount} component${nodeCount === 1 ? '' : 's'}):` }, `cyc-h-${idx}`));
-                        // Colorize slots (cyan, bracketed) distinctly from components so
-                        // the operator can visually parse the alternating structure.
-                        // Brackets on slot names ensure the distinction survives when
-                        // ANSI is stripped (logs, redirected output).
-                        const segs = formatCyclePathSegments(cycle, 16);
-                        lines.push(_jsxs(Text, { children: ['    ', segs.map((seg, si) => seg.kind === 'slot' ? (_jsx(Text, { color: "cyan", children: seg.text }, si)) : seg.kind === 'arrow' ? (_jsx(Text, { dimColor: true, children: seg.text }, si)) : (_jsx(Text, { children: seg.text }, si)))] }, `cyc-p-${idx}`));
-                        if (cycle.suggestedBreak) {
-                            const b = cycle.suggestedBreak;
-                            lines.push(_jsx(Text, { dimColor: true, children: `    Suggested fix: remove '${b.toComponent}' from ${b.fromComponent}.$slots.${b.slotName}.$allowedComponents` }, `cyc-f-${idx}`));
-                        }
-                        lines.push(_jsx(Text, { children: " " }, `cyc-s-${idx}`));
-                    });
-                    const visible = lines.slice(cyclePanelScroll, cyclePanelScroll + PANEL_H);
-                    return (_jsxs(Box, { flexDirection: "column", borderStyle: "round", borderColor: "yellow", paddingX: 1, children: [visible, _jsx(Text, { dimColor: true, children: '[↑↓/j/k] scroll  [c/q/Esc] close' })] }));
-                })(), !dialogOpen &&
+    return (_jsxs(Box, { flexDirection: "column", children: [showFinalize && (_jsx(FinalizeDialog, { accepted: accepted, rejected: rejected, needsReview: needsReview, onConfirm: handleFinalizeConfirm, onCancel: () => setShowFinalize(false) })), showQuit && _jsx(QuitDialog, { hasUnsavedDrafts: false, onConfirm: onQuit, onCancel: () => setShowQuit(false) }), showRemovedPanel && !dialogOpen && (_jsxs(Box, { flexDirection: "column", borderStyle: "round", borderColor: "cyan", paddingX: 1, children: [_jsx(Text, { bold: true, color: "cyan", children: `Removed components (${removedComponents.length})` }), _jsx(Text, { dimColor: true, children: "these will be DELETED from the target space" }), _jsx(Text, { children: " " }), removedComponents.map((rc) => (_jsx(Text, { children: `- ${rc.name}${rc.id ? `  (${rc.id})` : ''}` }, rc.id))), _jsx(Text, { children: " " }), _jsx(Text, { dimColor: true, children: "press d or Esc to close" })] })), !dialogOpen &&
                 livePreview &&
                 (() => {
                     // Pilot-2026-06-23 R2: at-a-glance diff summary at the top of the
@@ -712,10 +526,7 @@ export function GenerateReviewStep({ extractSessionId, onFinalize, onQuit, liveP
                     if (!hasCounts)
                         return null;
                     return (_jsxs(Box, { children: [_jsx(Text, { children: 'Preview: ' }), _jsx(Text, { color: "green", children: `${counts.new} new` }), _jsx(Text, { children: ' · ' }), _jsx(Text, { color: "yellow", children: `${counts.changed} changed` }), _jsx(Text, { children: ' · ' }), _jsx(Text, { dimColor: true, children: `${counts.removed} removed` }), removedComponents.length > 0 && _jsx(Text, { dimColor: true, children: ' ([d] removed list)' }), _jsx(Text, { children: ' · ' }), _jsx(Text, { color: "red", bold: true, children: `${counts.breaking} breaking` })] }));
-                })(), !dialogOpen && slotCycles.length > 0 && !showCyclePanel && (_jsxs(Box, { flexDirection: "column", children: [_jsx(Text, { color: "yellow", children: `⚠ ${slotCycles.length} slot dependency cycle${slotCycles.length === 1 ? '' : 's'} detected — push will fail` }), slotCycles.slice(0, 3).map((cycle, idx) => {
-                        const segs = formatCyclePathSegments(cycle);
-                        return (_jsxs(Text, { color: "yellow", children: ['  Cycle: ', segs.map((seg, si) => seg.kind === 'slot' ? (_jsx(Text, { color: "cyan", children: seg.text }, si)) : seg.kind === 'arrow' ? (_jsx(Text, { dimColor: true, children: seg.text }, si)) : (_jsx(Text, { color: "yellow", children: seg.text }, si)))] }, `cyc-banner-${idx}`));
-                    }), slotCycles.length > 3 && _jsx(Text, { color: "yellow", children: `  …${slotCycles.length - 3} more` }), _jsx(Text, { dimColor: true, children: '  press [c] for detail' })] })), !dialogOpen && emptyCount > 0 && (_jsx(Text, { color: "yellow", children: `⚠ ${emptyCount} component${emptyCount === 1 ? '' : 's'} had no classifiable props — review with care` })), !dialogOpen && finalizeError && _jsx(Text, { color: "red", children: `⚠ ${finalizeError}` }), !dialogOpen && (_jsxs(Box, { children: [_jsx(Sidebar, { components: sidebarItems, selectedId: selected?.key ?? null, focused: sidebarFocused, scrollOffset: sidebarScrollOffset, visibleCount: VISIBLE_COUNT, onSelect: (id) => {
+                })(), !dialogOpen && emptyCount > 0 && (_jsx(Text, { color: "yellow", children: `⚠ ${emptyCount} component${emptyCount === 1 ? '' : 's'} had no classifiable props — review with care` })), !dialogOpen && finalizeError && _jsx(Text, { color: "red", children: `⚠ ${finalizeError}` }), !dialogOpen && (_jsxs(Box, { children: [_jsx(Sidebar, { components: sidebarItems, selectedId: selected?.key ?? null, focused: sidebarFocused, scrollOffset: sidebarScrollOffset, visibleCount: VISIBLE_COUNT, onSelect: (id) => {
                             const idx = components.findIndex((c) => c.key === id);
                             if (idx >= 0) {
                                 setSelectedIdx(idx);
@@ -764,12 +575,11 @@ export function GenerateReviewStep({ extractSessionId, onFinalize, onQuit, liveP
                                     }, onToggleSourceExternal: () => {
                                         setPanelOpen('source');
                                         setPanelScrollOffset(() => 0);
-                                    }, onTextEntryActiveChange: setTextEntryActive, projectSlotGraph: projectSlotGraph, currentComponentName: selected.key }, selected.key)), saveError && _jsx(Text, { color: "red", children: '✗ ' + saveError }), _jsxs(Text, { dimColor: true, children: [sidebarFocused
+                                    }, onTextEntryActiveChange: setTextEntryActive }, selected.key)), saveError && _jsx(Text, { color: "red", children: '✗ ' + saveError }), _jsxs(Text, { dimColor: true, children: [sidebarFocused
                                             ? '  [a] accept  [r] reject  [A] accept all  [J] ' +
                                                 (showJson ? 'hide JSON' : 'show JSON') +
                                                 '  [F] finalize  [e/Tab] focus panel' +
                                                 (livePreview && removedComponents.length > 0 ? '  [d] removed list' : '') +
-                                                (slotCycles.length > 0 ? '  [c] cycles' : '') +
                                                 '  [q] quit'
                                             : showJson
                                                 ? '  [j/k] scroll  [Ctrl+u/d] half-page  [gg/G] top/bottom  [Tab] focus list'

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

@@ -1,15 +1,5 @@
 import React from 'react';
 import type { ServerPreviewResponse } from '@contentful/experience-design-system-types';
-export interface PreviewDiffLine {
-    key: string;
-    color: 'green' | 'red' | 'yellow' | 'gray';
-    text: string;
-}
-/**
- * Pure builder for the wizard preview diff lines. Extracted from the render
- * layer so it can be unit-tested independently of Ink.
- */
-export declare function buildPreviewDiffLines(preview: ServerPreviewResponse): PreviewDiffLine[];
 type WizardPreviewStepProps = {
     preview: ServerPreviewResponse;
     spaceId: string;

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

@@ -1,99 +1,9 @@
-import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
+import { jsxs as _jsxs, jsx as _jsx, Fragment as _Fragment } from "react/jsx-runtime";
 import { useState, useMemo } from 'react';
 import { Box, Text, useStdout } from 'ink';
 import { useImmediateInput } from '../../../analyze/select/tui/hooks/useImmediateInput.js';
 import { hasBreakingChangesWithImpact } from '../../../apply/manifest.js';
 import { computeComponentDiffLines } from './preview-diff.js';
-/**
- * Pure builder for the wizard preview diff lines. Extracted from the render
- * layer so it can be unit-tested independently of Ink.
- */
-export function buildPreviewDiffLines(preview) {
-    const lines = [];
-    const { components, tokens } = preview;
-    for (const item of components.new) {
-        const raw = item;
-        const name = raw.key ?? raw.$name ?? 'unknown';
-        lines.push({ key: `comp-new-${name}`, color: 'green', text: ` + ${name}` });
-        const slots = (raw.$slots ?? {});
-        for (const slotName of Object.keys(slots).sort()) {
-            lines.push({
-                key: `comp-new-${name}-slot-${slotName}`,
-                color: 'green',
-                text: `   slot: ${slotName}`,
-            });
-            const allowed = slots[slotName]?.['$allowedComponents'];
-            if (Array.isArray(allowed) && allowed.length > 0) {
-                const names = allowed.filter((n) => typeof n === 'string');
-                lines.push({
-                    key: `comp-new-${name}-slot-${slotName}-allow`,
-                    color: 'green',
-                    text: `     allowedComponents: [${names.join(', ')}]`,
-                });
-            }
-        }
-    }
-    for (const item of components.removed) {
-        lines.push({ key: `comp-rm-${item.name}`, color: 'red', text: ` - ${item.name}` });
-    }
-    for (const item of components.changed) {
-        lines.push({
-            key: `comp-h-${item.current.name}`,
-            color: 'yellow',
-            text: ` ~ ${item.current.name}${item.hasPendingDraftChanges ? ' ⚡ has pending draft changes' : ''}`,
-        });
-        if (item.changeClassification?.classification === 'breaking') {
-            const reasons = item.changeClassification.breakingChanges
-                .map((bc) => `${bc.propertyId}: ${bc.reason}`)
-                .join(', ');
-            lines.push({ key: `comp-b-${item.current.name}`, color: 'red', text: ` ⚠ BREAKING: ${reasons}` });
-        }
-        const diffLines = computeComponentDiffLines(item.current, item.proposed, item.changeClassification);
-        for (const d of diffLines) {
-            lines.push({ key: `comp-d-${item.current.name}-${d.key}`, color: d.color, text: ` ${d.text}` });
-        }
-        // Also enumerate slots + allowedComponents from the proposed side so the
-        // reviewer sees the shape they're pushing, not just the delta.
-        const proposedSlots = item.proposed['$slots'] ?? {};
-        for (const slotName of Object.keys(proposedSlots).sort()) {
-            const allowed = proposedSlots[slotName]?.['$allowedComponents'];
-            if (Array.isArray(allowed) && allowed.length > 0) {
-                const names = allowed.filter((n) => typeof n === 'string');
-                const key = `comp-d-${item.current.name}-slot-${slotName}-allow-list`;
-                if (!lines.some((l) => l.key === key)) {
-                    lines.push({
-                        key,
-                        color: 'gray',
-                        text: `   slot ${slotName} allowedComponents: [${names.join(', ')}]`,
-                    });
-                }
-            }
-        }
-    }
-    for (const item of tokens.new) {
-        const raw = item;
-        const name = raw.name ?? raw.path ?? 'unknown';
-        lines.push({ key: `tok-new-${name}`, color: 'green', text: ` + ${name}` });
-    }
-    for (const item of tokens.removed) {
-        lines.push({ key: `tok-rm-${item.name}`, color: 'red', text: ` - ${item.name}` });
-    }
-    for (const item of tokens.changed) {
-        const tokenName = item.current.name;
-        lines.push({
-            key: `tok-h-${tokenName}`,
-            color: 'yellow',
-            text: ` ~ ${tokenName}${item.hasPendingDraftChanges ? ' ⚡ has pending draft changes' : ''}`,
-        });
-        if (item.changeClassification?.classification === 'breaking') {
-            const reasons = item.changeClassification.breakingChanges
-                .map((bc) => `${bc.propertyId}: ${bc.reason}`)
-                .join(', ');
-            lines.push({ key: `tok-b-${tokenName}`, color: 'red', text: ` ⚠ BREAKING: ${reasons}` });
-        }
-    }
-    return lines;
-}
 export function WizardPreviewStep({ preview, spaceId, environmentId, stepNumber, totalSteps, onConfirm, onEdit, onSaveFiles, onQuit, }) {
     const breakingWithImpact = hasBreakingChangesWithImpact(preview);
     const [diffExpanded, setDiffExpanded] = useState(false);
@@ -104,10 +14,56 @@ export function WizardPreviewStep({ preview, spaceId, environmentId, stepNumber,
     const allDiffLines = useMemo(() => {
         if (!diffExpanded)
             return [];
-        return buildPreviewDiffLines(preview).map((line) => ({
-            key: line.key,
-            element: (_jsx(Text, { color: line.color === 'gray' ? undefined : line.color, dimColor: line.color === 'gray', children: line.text })),
-        }));
+        const lines = [];
+        const { components, tokens } = preview;
+        for (const item of components.new) {
+            const name = item.key ??
+                item.$name ??
+                'unknown';
+            lines.push({ key: `comp-new-${name}`, element: _jsxs(Text, { color: "green", children: [" + ", name] }) });
+        }
+        for (const item of components.removed) {
+            lines.push({ key: `comp-rm-${item.name}`, element: _jsxs(Text, { color: "red", children: [" - ", item.name] }) });
+        }
+        for (const item of components.changed) {
+            lines.push({
+                key: `comp-h-${item.current.name}`,
+                element: (_jsxs(Text, { color: "yellow", children: [' ', "~ ", item.current.name, item.hasPendingDraftChanges ? _jsx(Text, { color: "yellow", children: " \u26A1 has pending draft changes" }) : null] })),
+            });
+            if (item.changeClassification?.classification === 'breaking') {
+                const reasons = item.changeClassification.breakingChanges
+                    .map((bc) => `${bc.propertyId}: ${bc.reason}`)
+                    .join(', ');
+                lines.push({ key: `comp-b-${item.current.name}`, element: _jsxs(Text, { color: "red", children: [" \u26A0 BREAKING: ", reasons] }) });
+            }
+            const diffLines = computeComponentDiffLines(item.current, item.proposed, item.changeClassification);
+            for (const d of diffLines) {
+                lines.push({ key: `comp-d-${item.current.name}-${d.key}`, element: _jsxs(Text, { color: d.color, children: [" ", d.text] }) });
+            }
+        }
+        for (const item of tokens.new) {
+            const name = item.name ??
+                item.path ??
+                'unknown';
+            lines.push({ key: `tok-new-${name}`, element: _jsxs(Text, { color: "green", children: [" + ", name] }) });
+        }
+        for (const item of tokens.removed) {
+            lines.push({ key: `tok-rm-${item.name}`, element: _jsxs(Text, { color: "red", children: [" - ", item.name] }) });
+        }
+        for (const item of tokens.changed) {
+            const tokenName = item.current.name;
+            lines.push({
+                key: `tok-h-${tokenName}`,
+                element: (_jsxs(Text, { color: "yellow", children: [' ', "~ ", tokenName, item.hasPendingDraftChanges ? _jsx(Text, { color: "yellow", children: " \u26A1 has pending draft changes" }) : null] })),
+            });
+            if (item.changeClassification?.classification === 'breaking') {
+                const reasons = item.changeClassification.breakingChanges
+                    .map((bc) => `${bc.propertyId}: ${bc.reason}`)
+                    .join(', ');
+                lines.push({ key: `tok-b-${tokenName}`, element: _jsxs(Text, { color: "red", children: [" \u26A0 BREAKING: ", reasons] }) });
+            }
+        }
+        return lines;
     }, [diffExpanded, preview]);
     const maxScroll = Math.max(0, allDiffLines.length - viewportHeight);
     useImmediateInput((input, key) => {

package/dist/src/import/tui/steps/preview-diff.js

@@ -59,7 +59,6 @@ export function computeComponentDiffLines(current, proposed, changeClassificatio
     const currentSlots = new Set(current.slots);
     const proposedSlots = (proposed['$slots'] ?? {});
     const proposedSlotNames = new Set(Object.keys(proposedSlots));
-    const currentSlotAllowed = current.currentSlotAllowed ?? {};
     for (const name of [...proposedSlotNames].sort()) {
         if (!currentSlots.has(name)) {
             lines.push({ key: `slot-${name}-add`, color: 'green', text: `+ slot: ${name}` });
@@ -70,55 +69,8 @@ export function computeComponentDiffLines(current, proposed, changeClassificatio
             lines.push({ key: `slot-${name}-rm`, color: 'red', text: `- slot: ${name}` });
         }
     }
-    // $allowedComponents diffs — for each slot present in both sides, or newly-added
-    // with a non-empty allowedComponents list.
-    for (const name of [...proposedSlotNames].sort()) {
-        const nextAllowed = normalizeAllowedComponents(proposedSlots[name]?.['$allowedComponents']);
-        const prevAllowed = normalizeAllowedComponents(currentSlotAllowed[name]);
-        const prevExists = currentSlots.has(name);
-        if (!prevExists) {
-            if (nextAllowed.length > 0) {
-                lines.push({
-                    key: `slot-${name}-allow-new`,
-                    color: 'green',
-                    text: `+ slot ${name} allowedComponents: [${nextAllowed.join(', ')}]`,
-                });
-            }
-            continue;
-        }
-        if (arraysEqual(prevAllowed, nextAllowed))
-            continue;
-        lines.push({
-            key: `slot-${name}-allow-old`,
-            color: 'red',
-            text: `- slot ${name} allowedComponents: [${prevAllowed.join(', ')}]`,
-        });
-        lines.push({
-            key: `slot-${name}-allow-new`,
-            color: 'green',
-            text: `+ slot ${name} allowedComponents: [${nextAllowed.join(', ')}]`,
-        });
-    }
     return lines;
 }
-function normalizeAllowedComponents(value) {
-    if (!Array.isArray(value))
-        return [];
-    const out = [];
-    for (const item of value) {
-        if (typeof item === 'string' && item.length > 0)
-            out.push(item);
-    }
-    return out.sort();
-}
-function arraysEqual(a, b) {
-    if (a.length !== b.length)
-        return false;
-    for (let i = 0; i < a.length; i++)
-        if (a[i] !== b[i])
-            return false;
-    return true;
-}
 function normalizeType(type) {
     switch (type.toLowerCase()) {
         case 'boolean':

package/dist/src/session/db.d.ts

@@ -3,7 +3,6 @@ import type { RawComponentDefinition } from '../types.js';
 import type { CDFComponentEntry, DTCGTokenEntry, DTCGTokenGroup } from '@contentful/experience-design-system-types';
 import type { ToolCall, TokenToolCall } from '../generate/agent-runner.js';
 import type { ComponentTypeSummary } from '@contentful/experience-design-system-types';
-import type { SlotCycle, SlotEdge } from '../analyze/cycle-detection.js';
 export type StepStatus = 'pending' | 'complete' | 'failed' | 'interrupted';
 export type CommandName = 'analyze extract' | 'analyze select' | 'generate components' | 'generate tokens' | 'generate edit' | 'apply preview' | 'apply select' | 'apply push' | 'print components' | 'print tokens' | 'import';
 export interface SessionRow {
@@ -177,14 +176,6 @@ export declare function lookupCacheByEntity(db: DatabaseSync, entityType: 'compo
 export declare function storeCache(db: DatabaseSync, inputHash: string, entityType: 'component' | 'token_set', entityId: string, sourceSessionId: string, humanEdited: boolean, promptHash?: string): void;
 export declare function storeScannedFiles(db: DatabaseSync, sessionId: string, filePaths: string[]): void;
 export declare function loadScannedFiles(db: DatabaseSync, sessionId: string): string[];
-export declare function storeSlotCycles(db: DatabaseSync, sessionId: string, cycles: Array<SlotCycle & {
-    suggestedBreak?: SlotEdge | null;
-}>): void;
-export interface StoredSlotCycle extends SlotCycle {
-    suggestedBreak: SlotEdge | null;
-}
-export declare function loadSlotCycles(db: DatabaseSync, sessionId: string): StoredSlotCycle[];
-export declare function clearSlotCycles(db: DatabaseSync, sessionId: string): void;
 export declare function markCacheHumanEdited(db: DatabaseSync, entityType: 'component' | 'token_set', entityId: string): void;
 export declare function copyComponentFromCache(db: DatabaseSync, sourceSessionId: string, targetSessionId: string, componentId: string): void;
 export declare function copyTokensFromCache(db: DatabaseSync, sourceSessionId: string, targetSessionId: string): void;