@shepai/cli 1.72.9 → 1.73.0-pr222.ef53005

package/dist/src/presentation/web/components/common/control-center-drawer/create-drawer-client.js

@@ -5,10 +5,8 @@ import { useRouter, usePathname } from 'next/navigation';
 import { toast } from 'sonner';
 import { createFeature } from '../../../app/actions/create-feature.js';
 import { FeatureCreateDrawer } from '../../common/feature-create-drawer/index.js';
-import { useSoundAction } from '../../../hooks/use-sound-action.js';
 export function CreateDrawerClient({ repositoryPath, initialParentId, features, workflowDefaults, }) {
     const router = useRouter();
-    const createSound = useSoundAction('create');
     const [isSubmitting, setIsSubmitting] = useState(false);
     // Derive open state from the URL. Next.js parallel routes preserve slot
     // content during soft navigation, so this component is NOT unmounted when
@@ -32,34 +30,31 @@ export function CreateDrawerClient({ repositoryPath, initialParentId, features,
     }, [router]);
     const onSubmit = useCallback((data) => {
         setIsSubmitting(true);
-        // Dispatch event for optimistic canvas update before navigating
-        window.dispatchEvent(new CustomEvent('shep:feature-created', {
-            detail: {
-                name: data.name,
-                description: data.description,
-                repositoryPath: data.repositoryPath,
-                parentId: data.parentId,
-            },
-        }));
         // Close the drawer immediately for responsive UI
         router.push('/');
-        // Fire server action in the background
+        // Server action Phase 1 returns fast with real feature ID (DB record created)
+        // Phase 2 (metadata, worktree, agent) runs in background on server
         createFeature(data)
             .then((result) => {
             if (result.error) {
                 toast.error(result.error);
-                // Dispatch rollback event
-                window.dispatchEvent(new CustomEvent('shep:feature-create-failed'));
                 return;
             }
-            createSound.play();
+            // Dispatch event with real feature ID so control center adds it to featureMap
+            window.dispatchEvent(new CustomEvent('shep:feature-created', {
+                detail: {
+                    featureId: result.feature.id,
+                    name: result.feature.name,
+                    description: result.feature.description,
+                    repositoryPath: result.feature.repositoryPath,
+                    parentId: data.parentId,
+                },
+            }));
         })
             .catch(() => {
             toast.error('Failed to create feature');
-            window.dispatchEvent(new CustomEvent('shep:feature-create-failed'));
-            // Reset on error so the user can retry without navigating away
             setIsSubmitting(false);
         });
-    }, [router, createSound]);
+    }, [router]);
     return (_jsx(FeatureCreateDrawer, { open: isOpen, onClose: onClose, onSubmit: onSubmit, repositoryPath: repositoryPath, features: features, workflowDefaults: workflowDefaults, initialParentId: initialParentId, isSubmitting: isSubmitting }));
 }

package/dist/src/presentation/web/components/common/control-center-drawer/drawer-view.d.ts

@@ -40,6 +40,8 @@ export type DrawerView =
     type: 'repository';
     data: RepositoryNodeData;
 };
+/** Derives the feature-specific view type from node lifecycle + state. */
+export declare function deriveFeatureViewType(node: FeatureNodeData): 'feature' | 'prd-review' | 'tech-review' | 'merge-review';
 /**
  * Derives the active DrawerView from control-center state.
  * Priority: feature-create > selected-node (prd/tech/merge/feature) > repository

package/dist/src/presentation/web/components/common/control-center-drawer/drawer-view.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"drawer-view.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/drawer-view.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AACxE,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAC9E,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,iEAAiE,CAAC;AAC3G,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,qCAAqC,CAAC;AAE5E;;;GAGG;AACH,MAAM,MAAM,UAAU;AACpB,gGAAgG;AAC9F;IAAE,IAAI,EAAE,SAAS,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AAC5C,mEAAmE;GACjE;IAAE,IAAI,EAAE,YAAY,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AAC/C,yEAAyE;GACvE;IAAE,IAAI,EAAE,aAAa,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AAChD,8DAA8D;GAC5D;IAAE,IAAI,EAAE,cAAc,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AACjD,4BAA4B;GAC1B;IACE,IAAI,EAAE,gBAAgB,CAAC;IACvB,cAAc,EAAE,MAAM,CAAC;IACvB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,QAAQ,EAAE,mBAAmB,EAAE,CAAC;IAChC,gBAAgB,CAAC,EAAE,gBAAgB,CAAC;CACrC;AACH,8BAA8B;GAC5B;IAAE,IAAI,EAAE,YAAY,CAAC;IAAC,IAAI,EAAE,kBAAkB,CAAA;CAAE,CAAC;AAErD;;;GAGG;AACH,wBAAgB,iBAAiB,CAAC,EAChC,YAAY,EACZ,kBAAkB,EAClB,qBAAqB,EACrB,sBAAsB,EACtB,gBAAgB,EAChB,QAAQ,EACR,gBAAgB,GACjB,EAAE;IACD,YAAY,EAAE,eAAe,GAAG,IAAI,CAAC;IACrC,kBAAkB,EAAE,OAAO,CAAC;IAC5B,qBAAqB,EAAE,MAAM,CAAC;IAC9B,sBAAsB,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3C,gBAAgB,EAAE,kBAAkB,GAAG,IAAI,CAAC;IAC5C,QAAQ,EAAE,mBAAmB,EAAE,CAAC;IAChC,gBAAgB,EAAE,gBAAgB,GAAG,SAAS,CAAC;CAChD,GAAG,UAAU,GAAG,IAAI,CAgCpB"}
+{"version":3,"file":"drawer-view.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/drawer-view.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AACxE,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAC9E,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,iEAAiE,CAAC;AAC3G,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,qCAAqC,CAAC;AAE5E;;;GAGG;AACH,MAAM,MAAM,UAAU;AACpB,gGAAgG;AAC9F;IAAE,IAAI,EAAE,SAAS,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AAC5C,mEAAmE;GACjE;IAAE,IAAI,EAAE,YAAY,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AAC/C,yEAAyE;GACvE;IAAE,IAAI,EAAE,aAAa,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AAChD,8DAA8D;GAC5D;IAAE,IAAI,EAAE,cAAc,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AACjD,4BAA4B;GAC1B;IACE,IAAI,EAAE,gBAAgB,CAAC;IACvB,cAAc,EAAE,MAAM,CAAC;IACvB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,QAAQ,EAAE,mBAAmB,EAAE,CAAC;IAChC,gBAAgB,CAAC,EAAE,gBAAgB,CAAC;CACrC;AACH,8BAA8B;GAC5B;IAAE,IAAI,EAAE,YAAY,CAAC;IAAC,IAAI,EAAE,kBAAkB,CAAA;CAAE,CAAC;AAErD,0EAA0E;AAC1E,wBAAgB,qBAAqB,CACnC,IAAI,EAAE,eAAe,GACpB,SAAS,GAAG,YAAY,GAAG,aAAa,GAAG,cAAc,CAM3D;AAED;;;GAGG;AACH,wBAAgB,iBAAiB,CAAC,EAChC,YAAY,EACZ,kBAAkB,EAClB,qBAAqB,EACrB,sBAAsB,EACtB,gBAAgB,EAChB,QAAQ,EACR,gBAAgB,GACjB,EAAE;IACD,YAAY,EAAE,eAAe,GAAG,IAAI,CAAC;IACrC,kBAAkB,EAAE,OAAO,CAAC;IAC5B,qBAAqB,EAAE,MAAM,CAAC;IAC9B,sBAAsB,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3C,gBAAgB,EAAE,kBAAkB,GAAG,IAAI,CAAC;IAC5C,QAAQ,EAAE,mBAAmB,EAAE,CAAC;IAChC,gBAAgB,EAAE,gBAAgB,GAAG,SAAS,CAAC;CAChD,GAAG,UAAU,GAAG,IAAI,CAoBpB"}

package/dist/src/presentation/web/components/common/control-center-drawer/drawer-view.js

@@ -1,3 +1,13 @@
+/** Derives the feature-specific view type from node lifecycle + state. */
+export function deriveFeatureViewType(node) {
+    if (node.lifecycle === 'requirements' && node.state === 'action-required')
+        return 'prd-review';
+    if (node.lifecycle === 'implementation' && node.state === 'action-required')
+        return 'tech-review';
+    if (node.lifecycle === 'review' && (node.state === 'action-required' || node.state === 'error'))
+        return 'merge-review';
+    return 'feature';
+}
 /**
  * Derives the active DrawerView from control-center state.
  * Priority: feature-create > selected-node (prd/tech/merge/feature) > repository
@@ -13,17 +23,7 @@ export function computeDrawerView({ selectedNode, isCreateDrawerOpen, pendingRep
         };
     }
     if (selectedNode) {
-        if (selectedNode.lifecycle === 'requirements' && selectedNode.state === 'action-required') {
-            return { type: 'prd-review', node: selectedNode };
-        }
-        if (selectedNode.lifecycle === 'implementation' && selectedNode.state === 'action-required') {
-            return { type: 'tech-review', node: selectedNode };
-        }
-        if (selectedNode.lifecycle === 'review' &&
-            (selectedNode.state === 'action-required' || selectedNode.state === 'error')) {
-            return { type: 'merge-review', node: selectedNode };
-        }
-        return { type: 'feature', node: selectedNode };
+        return { type: deriveFeatureViewType(selectedNode), node: selectedNode };
     }
     if (selectedRepoNode) {
         return { type: 'repository', data: selectedRepoNode };

package/dist/src/presentation/web/components/common/control-center-drawer/feature-drawer-client.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"feature-drawer-client.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/feature-drawer-client.tsx"],"names":[],"mappings":"AA2DA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAEhD,MAAM,WAAW,wBAAwB;IACvC,IAAI,EAAE,UAAU,CAAC;CAClB;AAED,wBAAgB,mBAAmB,CAAC,EAAE,IAAI,EAAE,WAAW,EAAE,EAAE,wBAAwB,2CAimBlF"}
+{"version":3,"file":"feature-drawer-client.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/feature-drawer-client.tsx"],"names":[],"mappings":"AAyDA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAGhD,MAAM,WAAW,wBAAwB;IACvC,IAAI,EAAE,UAAU,CAAC;CAClB;AAED,wBAAgB,mBAAmB,CAAC,EAAE,IAAI,EAAE,WAAW,EAAE,EAAE,wBAAwB,2CAykBlF"}

package/dist/src/presentation/web/components/common/control-center-drawer/feature-drawer-client.js

@@ -34,7 +34,9 @@ import { TechReviewTabs } from '../../common/tech-review-tabs/index.js';
 import { MergeReview } from '../../common/merge-review/index.js';
 import { featureNodeStateConfig, lifecycleDisplayLabels } from '../../common/feature-node/index.js';
 import { useFeatureActions } from '../../common/feature-drawer/use-feature-actions.js';
-import { mapEventTypeToState, mapPhaseNameToLifecycle, } from '../../common/feature-node/derive-feature-state.js';
+import { resolveSseEventUpdates } from '../../common/feature-node/derive-feature-state.js';
+import { deriveFeatureViewType } from './drawer-view.js';
+import { useArtifactFetch } from './use-artifact-fetch.js';
 export function FeatureDrawerClient({ view: initialView }) {
     const router = useRouter();
     const deleteSound = useSoundAction('delete');
@@ -55,36 +57,40 @@ export function FeatureDrawerClient({ view: initialView }) {
     const { events } = useAgentEventsContext();
     const processedCountRef = useRef(0);
     useEffect(() => {
+        // Clamp cursor if events were pruned
+        if (processedCountRef.current > events.length) {
+            processedCountRef.current = 0;
+        }
         if (!featureNode || events.length <= processedCountRef.current)
             return;
         const newEvents = events.slice(processedCountRef.current);
         processedCountRef.current = events.length;
-        for (const event of newEvents) {
-            if (event.featureId !== featureNode.featureId)
+        for (const update of resolveSseEventUpdates(newEvents)) {
+            if (update.featureId !== featureNode.featureId)
                 continue;
-            const newState = mapEventTypeToState(event.eventType);
-            const newLifecycle = mapPhaseNameToLifecycle(event.phaseName);
             // Trigger a server refresh to get the latest drawer view,
-            // but skip when the user has unsaved changes or a rejection is in-flight
-            // to avoid the refresh interfering with the reject handler's navigation.
-            if (!isDirtyRef.current && !isRejectingRef.current)
+            // but skip when the drawer is closed, the user has unsaved changes,
+            // or a rejection is in-flight to avoid unnecessary refreshes.
+            if (isOpenRef.current && !isDirtyRef.current && !isRejectingRef.current)
                 router.refresh();
-            // Optimistically update the node data for immediate UI feedback
-            setView((prev) => {
-                if (prev.type !== 'feature' &&
-                    prev.type !== 'prd-review' &&
-                    prev.type !== 'tech-review' &&
-                    prev.type !== 'merge-review')
-                    return prev;
-                return {
-                    ...prev,
-                    node: {
+            if (update.state !== undefined || update.lifecycle !== undefined) {
+                // Optimistically update the node data AND re-derive the view type so the
+                // drawer switches immediately (e.g. prd-review → feature when agent resumes)
+                // without waiting for the router.refresh() round-trip.
+                setView((prev) => {
+                    if (prev.type !== 'feature' &&
+                        prev.type !== 'prd-review' &&
+                        prev.type !== 'tech-review' &&
+                        prev.type !== 'merge-review')
+                        return prev;
+                    const updatedNode = {
                         ...prev.node,
-                        state: newState,
-                        ...(newLifecycle !== undefined && { lifecycle: newLifecycle }),
-                    },
-                };
-            });
+                        ...(update.state !== undefined && { state: update.state }),
+                        ...(update.lifecycle !== undefined && { lifecycle: update.lifecycle }),
+                    };
+                    return { ...prev, type: deriveFeatureViewType(updatedNode), node: updatedNode };
+                });
+            }
         }
     }, [events, featureNode, router]);
     // Derive open state from the URL. Next.js parallel routes preserve slot
@@ -93,6 +99,17 @@ export function FeatureDrawerClient({ view: initialView }) {
     // the close animation when the path no longer matches a feature route.
     const pathname = usePathname();
     const isOpen = pathname.startsWith('/feature/');
+    const isOpenRef = useRef(isOpen);
+    const wasOpenRef = useRef(isOpen);
+    // When the drawer re-opens, force a server refresh so the view is always fresh.
+    // SSE skips router.refresh() while the drawer is closed, so data can go stale.
+    useEffect(() => {
+        if (isOpen && !wasOpenRef.current) {
+            router.refresh();
+        }
+        wasOpenRef.current = isOpen;
+    }, [isOpen, router]);
+    isOpenRef.current = isOpen;
     const onClose = useCallback(() => {
         router.push('/');
     }, [router]);
@@ -102,16 +119,12 @@ export function FeatureDrawerClient({ view: initialView }) {
     const [prdData, setPrdData] = useState(null);
     const [prdSelections, setPrdSelections] = useState({});
     const [prdDefaultSelections, setPrdDefaultSelections] = useState({});
-    const [isLoadingPrd, setIsLoadingPrd] = useState(false);
     // ── Tech state ─────────────────────────────────────────────────────────
     const [techData, setTechData] = useState(null);
-    const [isLoadingTech, setIsLoadingTech] = useState(false);
     // ── Product decisions state (for tech review Product tab) ─────────────
     const [techProductData, setTechProductData] = useState(undefined);
-    const [isLoadingTechProduct, setIsLoadingTechProduct] = useState(false);
     // ── Merge state ────────────────────────────────────────────────────────
     const [mergeData, setMergeData] = useState(null);
-    const [isLoadingMerge, setIsLoadingMerge] = useState(false);
     // ── Delete state ───────────────────────────────────────────────────────
     const [isDeleting, setIsDeleting] = useState(false);
     // ── Shared reject state ────────────────────────────────────────────────
@@ -124,130 +137,48 @@ export function FeatureDrawerClient({ view: initialView }) {
     }, [viewType]);
     // ── Data fetching ─────────────────────────────────────────────────────
     const prdFeatureId = view.type === 'prd-review' ? view.node.featureId : null;
-    useEffect(() => {
+    const isLoadingPrd = useArtifactFetch(prdFeatureId, getFeatureArtifact, (result) => {
+        if (result.error) {
+            toast.error(result.error);
+            return;
+        }
+        if (result.questionnaire) {
+            setPrdData(result.questionnaire);
+            const defaults = {};
+            for (const q of result.questionnaire.questions) {
+                const recommended = q.options.find((o) => o.recommended);
+                if (recommended)
+                    defaults[q.id] = recommended.id;
+            }
+            setPrdSelections(defaults);
+            setPrdDefaultSelections(defaults);
+        }
+    }, () => {
         setPrdSelections({});
         setPrdDefaultSelections({});
         setPrdData(null);
-        if (!prdFeatureId)
-            return;
-        let cancelled = false;
-        setIsLoadingPrd(true);
-        getFeatureArtifact(prdFeatureId)
-            .then((result) => {
-            if (cancelled)
-                return;
-            if (result.error) {
-                toast.error(result.error);
-                return;
-            }
-            if (result.questionnaire) {
-                setPrdData(result.questionnaire);
-                const defaults = {};
-                for (const q of result.questionnaire.questions) {
-                    const recommended = q.options.find((o) => o.recommended);
-                    if (recommended)
-                        defaults[q.id] = recommended.id;
-                }
-                setPrdSelections(defaults);
-                setPrdDefaultSelections(defaults);
-            }
-        })
-            .catch(() => {
-            if (!cancelled)
-                toast.error('Failed to load questionnaire');
-        })
-            .finally(() => {
-            if (!cancelled)
-                setIsLoadingPrd(false);
-        });
-        return () => {
-            cancelled = true;
-        };
-    }, [prdFeatureId]);
+    }, 'Failed to load questionnaire');
     const techFeatureId = view.type === 'tech-review' ? view.node.featureId : null;
-    useEffect(() => {
-        setTechData(null);
-        if (!techFeatureId)
+    const isLoadingTech = useArtifactFetch(techFeatureId, getResearchArtifact, (result) => {
+        if (result.error) {
+            toast.error(result.error);
             return;
-        let cancelled = false;
-        setIsLoadingTech(true);
-        getResearchArtifact(techFeatureId)
-            .then((result) => {
-            if (cancelled)
-                return;
-            if (result.error) {
-                toast.error(result.error);
-                return;
-            }
-            if (result.techDecisions)
-                setTechData(result.techDecisions);
-        })
-            .catch(() => {
-            if (!cancelled)
-                toast.error('Failed to load tech decisions');
-        })
-            .finally(() => {
-            if (!cancelled)
-                setIsLoadingTech(false);
-        });
-        return () => {
-            cancelled = true;
-        };
-    }, [techFeatureId]);
-    useEffect(() => {
-        setTechProductData(undefined);
-        if (!techFeatureId)
-            return;
-        let cancelled = false;
-        setIsLoadingTechProduct(true);
-        getFeatureArtifact(techFeatureId)
-            .then((result) => {
-            if (cancelled)
-                return;
-            if (result.productDecisions) {
-                setTechProductData(result.productDecisions);
-            }
-        })
-            .catch(() => {
-            // Silent failure — the product tab is supplementary
-        })
-            .finally(() => {
-            if (!cancelled)
-                setIsLoadingTechProduct(false);
-        });
-        return () => {
-            cancelled = true;
-        };
-    }, [techFeatureId]);
+        }
+        if (result.techDecisions)
+            setTechData(result.techDecisions);
+    }, () => setTechData(null), 'Failed to load tech decisions');
+    const isLoadingTechProduct = useArtifactFetch(techFeatureId, getFeatureArtifact, (result) => {
+        if (result.productDecisions)
+            setTechProductData(result.productDecisions);
+    }, () => setTechProductData(undefined));
     const mergeFeatureId = view.type === 'merge-review' ? view.node.featureId : null;
-    useEffect(() => {
-        setMergeData(null);
-        if (!mergeFeatureId)
+    const isLoadingMerge = useArtifactFetch(mergeFeatureId, getMergeReviewData, (result) => {
+        if ('error' in result) {
+            toast.error(result.error);
             return;
-        let cancelled = false;
-        setIsLoadingMerge(true);
-        getMergeReviewData(mergeFeatureId)
-            .then((result) => {
-            if (cancelled)
-                return;
-            if ('error' in result) {
-                toast.error(result.error);
-                return;
-            }
-            setMergeData(result);
-        })
-            .catch(() => {
-            if (!cancelled)
-                toast.error('Failed to load merge review data');
-        })
-            .finally(() => {
-            if (!cancelled)
-                setIsLoadingMerge(false);
-        });
-        return () => {
-            cancelled = true;
-        };
-    }, [mergeFeatureId]);
+        }
+        setMergeData(result);
+    }, () => setMergeData(null), 'Failed to load merge review data');
     // ── Close guard ──────────────────────────────────────────────────────
     const isChatDirty = chatInput.trim().length > 0;
     const isPrdDirty = view.type === 'prd-review' &&
@@ -304,6 +235,10 @@ export function FeatureDrawerClient({ view: initialView }) {
         }
         setChatInput('');
         toast.success(`${label} approved — agent resuming`);
+        // Optimistically update canvas node before SSE arrives (~500ms delay)
+        window.dispatchEvent(new CustomEvent('shep:feature-approved', {
+            detail: { featureId: reviewNode.featureId },
+        }));
         onClose();
     }, [reviewNode, onClose]);
     const handlePrdApprove = useCallback(async (_actionId) => {
@@ -328,6 +263,10 @@ export function FeatureDrawerClient({ view: initialView }) {
         }
         setChatInput('');
         toast.success('Requirements approved — agent resuming');
+        // Optimistically update canvas node before SSE arrives (~500ms delay)
+        window.dispatchEvent(new CustomEvent('shep:feature-approved', {
+            detail: { featureId: view.node.featureId },
+        }));
         setPrdSelections({});
         onClose();
     }, [view, prdData, prdSelections, onClose]);

package/dist/src/presentation/web/components/common/control-center-drawer/index.d.ts

@@ -1,3 +1,2 @@
-export { ControlCenterDrawer, type ControlCenterDrawerProps } from './control-center-drawer.js';
 export { computeDrawerView, type DrawerView } from './drawer-view.js';
 //# sourceMappingURL=index.d.ts.map

package/dist/src/presentation/web/components/common/control-center-drawer/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,KAAK,wBAAwB,EAAE,MAAM,yBAAyB,CAAC;AAC7F,OAAO,EAAE,iBAAiB,EAAE,KAAK,UAAU,EAAE,MAAM,eAAe,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,UAAU,EAAE,MAAM,eAAe,CAAC"}

package/dist/src/presentation/web/components/common/control-center-drawer/index.js

@@ -1,2 +1 @@
-export { ControlCenterDrawer } from './control-center-drawer.js';
 export { computeDrawerView } from './drawer-view.js';

package/dist/src/presentation/web/components/common/control-center-drawer/use-artifact-fetch.d.ts

@@ -0,0 +1,13 @@
+/**
+ * Generic hook for fetching drawer artifact data when a feature ID changes.
+ * Handles loading state, cancellation on unmount/re-fetch, and error toasting.
+ *
+ * @param featureId - The feature ID to fetch for, or null to reset.
+ * @param fetcher - Async function that fetches the artifact data.
+ * @param onSuccess - Called with the fetch result when successful and not cancelled.
+ * @param onReset - Called immediately when featureId changes (before fetching).
+ * @param errorMessage - Optional toast message shown on fetch failure.
+ * @returns Whether the fetch is currently in progress.
+ */
+export declare function useArtifactFetch<TResult>(featureId: string | null, fetcher: (id: string) => Promise<TResult>, onSuccess: (result: TResult) => void, onReset: () => void, errorMessage?: string): boolean;
+//# sourceMappingURL=use-artifact-fetch.d.ts.map

package/dist/src/presentation/web/components/common/control-center-drawer/use-artifact-fetch.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"use-artifact-fetch.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/use-artifact-fetch.ts"],"names":[],"mappings":"AAKA;;;;;;;;;;GAUG;AACH,wBAAgB,gBAAgB,CAAC,OAAO,EACtC,SAAS,EAAE,MAAM,GAAG,IAAI,EACxB,OAAO,EAAE,CAAC,EAAE,EAAE,MAAM,KAAK,OAAO,CAAC,OAAO,CAAC,EACzC,SAAS,EAAE,CAAC,MAAM,EAAE,OAAO,KAAK,IAAI,EACpC,OAAO,EAAE,MAAM,IAAI,EACnB,YAAY,CAAC,EAAE,MAAM,GACpB,OAAO,CAiCT"}

package/dist/src/presentation/web/components/common/control-center-drawer/use-artifact-fetch.js

@@ -0,0 +1,46 @@
+'use client';
+import { useState, useEffect, useRef } from 'react';
+import { toast } from 'sonner';
+/**
+ * Generic hook for fetching drawer artifact data when a feature ID changes.
+ * Handles loading state, cancellation on unmount/re-fetch, and error toasting.
+ *
+ * @param featureId - The feature ID to fetch for, or null to reset.
+ * @param fetcher - Async function that fetches the artifact data.
+ * @param onSuccess - Called with the fetch result when successful and not cancelled.
+ * @param onReset - Called immediately when featureId changes (before fetching).
+ * @param errorMessage - Optional toast message shown on fetch failure.
+ * @returns Whether the fetch is currently in progress.
+ */
+export function useArtifactFetch(featureId, fetcher, onSuccess, onReset, errorMessage) {
+    const [isLoading, setIsLoading] = useState(false);
+    // Use refs to avoid stale callbacks in the effect without triggering re-runs.
+    const onSuccessRef = useRef(onSuccess);
+    const onResetRef = useRef(onReset);
+    onSuccessRef.current = onSuccess;
+    onResetRef.current = onReset;
+    useEffect(() => {
+        onResetRef.current();
+        if (!featureId)
+            return;
+        let cancelled = false;
+        setIsLoading(true);
+        fetcher(featureId)
+            .then((result) => {
+            if (!cancelled)
+                onSuccessRef.current(result);
+        })
+            .catch(() => {
+            if (!cancelled && errorMessage)
+                toast.error(errorMessage);
+        })
+            .finally(() => {
+            if (!cancelled)
+                setIsLoading(false);
+        });
+        return () => {
+            cancelled = true;
+        };
+    }, [featureId, fetcher, errorMessage]);
+    return isLoading;
+}

package/dist/src/presentation/web/components/common/feature-drawer/use-feature-actions.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-feature-actions.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-drawer/use-feature-actions.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,mBAAmB;IAClC,cAAc,EAAE,MAAM,CAAC;IACvB,MAAM,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,MAAM,WAAW,mBAAmB;IAClC,SAAS,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC/B,WAAW,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IACjC,eAAe,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IACrC,UAAU,EAAE,OAAO,CAAC;IACpB,YAAY,EAAE,OAAO,CAAC;IACtB,YAAY,EAAE,OAAO,CAAC;IACtB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;CAC3B;AAID,wBAAgB,iBAAiB,CAAC,KAAK,EAAE,mBAAmB,GAAG,IAAI,GAAG,mBAAmB,CA+GxF"}
+{"version":3,"file":"use-feature-actions.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-drawer/use-feature-actions.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,mBAAmB;IAClC,cAAc,EAAE,MAAM,CAAC;IACvB,MAAM,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,MAAM,WAAW,mBAAmB;IAClC,SAAS,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC/B,WAAW,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IACjC,eAAe,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IACrC,UAAU,EAAE,OAAO,CAAC;IACpB,YAAY,EAAE,OAAO,CAAC;IACtB,YAAY,EAAE,OAAO,CAAC;IACtB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;CAC3B;AAID,wBAAgB,iBAAiB,CAAC,KAAK,EAAE,mBAAmB,GAAG,IAAI,GAAG,mBAAmB,CA8GxF"}

package/dist/src/presentation/web/components/common/feature-drawer/use-feature-actions.js

@@ -14,18 +14,15 @@ export function useFeatureActions(input) {
     const ideTimerRef = useRef(null);
     const shellTimerRef = useRef(null);
     const specsTimerRef = useRef(null);
-    // Clear timers on unmount
+    // Clear timers on unmount — read .current inside cleanup so we get the
+    // actual timer value at teardown time, not the always-null value at mount.
     useEffect(() => {
-        const ideTimer = ideTimerRef.current;
-        const shellTimer = shellTimerRef.current;
-        const specsTimer = specsTimerRef.current;
+        const refs = [ideTimerRef, shellTimerRef, specsTimerRef];
         return () => {
-            if (ideTimer)
-                clearTimeout(ideTimer);
-            if (shellTimer)
-                clearTimeout(shellTimer);
-            if (specsTimer)
-                clearTimeout(specsTimer);
+            for (const ref of refs) {
+                if (ref.current)
+                    clearTimeout(ref.current);
+            }
         };
     }, []);
     const performAction = useCallback(async (action, setLoading, setError, timerRef, isLoading) => {

package/dist/src/presentation/web/components/common/feature-node/derive-feature-state.d.ts

@@ -28,5 +28,30 @@ export declare function deriveNodeState(feature: Feature, agentRun?: AgentRun |
 export declare function deriveProgress(feature: Feature): number;
 /** Maps a NotificationEventType to the corresponding FeatureNodeState for optimistic UI updates. */
 export declare function mapEventTypeToState(eventType: NotificationEventType): FeatureNodeState;
+/** Map domain SdlcLifecycle enum values to UI FeatureLifecyclePhase. */
+export declare const sdlcLifecycleMap: Record<string, FeatureLifecyclePhase>;
+/**
+ * Derives the UI lifecycle phase from a Feature + AgentRun.
+ * Shared by build-feature-node-data.ts and build-graph-nodes.ts.
+ */
+export declare function deriveLifecycle(feature: Feature, run: AgentRun | null): FeatureLifecyclePhase;
+export interface SseEventUpdate {
+    featureId: string;
+    state: FeatureNodeState | undefined;
+    lifecycle: FeatureLifecyclePhase | undefined;
+    eventType: NotificationEventType;
+    phaseName?: string;
+}
+/**
+ * Resolves state/lifecycle updates from a batch of SSE events, applying
+ * the WaitingApproval batch-suppression rule: when WaitingApproval and
+ * PhaseCompleted arrive in the same batch for a feature, PhaseCompleted
+ * is fully suppressed to prevent lifecycle regression.
+ */
+export declare function resolveSseEventUpdates(events: readonly {
+    featureId: string;
+    eventType: NotificationEventType;
+    phaseName?: string;
+}[]): SseEventUpdate[];
 export declare function mapPhaseNameToLifecycle(phaseName: string | undefined): FeatureLifecyclePhase | undefined;
 //# sourceMappingURL=derive-feature-state.d.ts.map

package/dist/src/presentation/web/components/common/feature-node/derive-feature-state.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"derive-feature-state.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-node/derive-feature-state.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAIL,qBAAqB,EACtB,MAAM,+BAA+B,CAAC;AACvC,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,+BAA+B,CAAC;AACvE,OAAO,KAAK,EAAE,gBAAgB,EAAE,qBAAqB,EAAE,MAAM,6BAA6B,CAAC;AAE3F;;;;;;;;;;GAUG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,OAAO,EAAE,QAAQ,CAAC,EAAE,QAAQ,GAAG,IAAI,GAAG,gBAAgB,CA4C9F;AAED;;;;;GAKG;AACH,wBAAgB,cAAc,CAAC,OAAO,EAAE,OAAO,GAAG,MAAM,CAYvD;AAED,oGAAoG;AACpG,wBAAgB,mBAAmB,CAAC,SAAS,EAAE,qBAAqB,GAAG,gBAAgB,CAkBtF;AAcD,wBAAgB,uBAAuB,CACrC,SAAS,EAAE,MAAM,GAAG,SAAS,GAC5B,qBAAqB,GAAG,SAAS,CAGnC"}
+{"version":3,"file":"derive-feature-state.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-node/derive-feature-state.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAIL,qBAAqB,EACtB,MAAM,+BAA+B,CAAC;AACvC,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,+BAA+B,CAAC;AACvE,OAAO,KAAK,EAAE,gBAAgB,EAAE,qBAAqB,EAAE,MAAM,6BAA6B,CAAC;AAE3F;;;;;;;;;;GAUG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,OAAO,EAAE,QAAQ,CAAC,EAAE,QAAQ,GAAG,IAAI,GAAG,gBAAgB,CA4C9F;AAED;;;;;GAKG;AACH,wBAAgB,cAAc,CAAC,OAAO,EAAE,OAAO,GAAG,MAAM,CAYvD;AAED,oGAAoG;AACpG,wBAAgB,mBAAmB,CAAC,SAAS,EAAE,qBAAqB,GAAG,gBAAgB,CAkBtF;AAED,wEAAwE;AACxE,eAAO,MAAM,gBAAgB,EAAE,MAAM,CAAC,MAAM,EAAE,qBAAqB,CAOlE,CAAC;AAcF;;;GAGG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,IAAI,GAAG,qBAAqB,CAQ7F;AAED,MAAM,WAAW,cAAc;IAC7B,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,gBAAgB,GAAG,SAAS,CAAC;IACpC,SAAS,EAAE,qBAAqB,GAAG,SAAS,CAAC;IAC7C,SAAS,EAAE,qBAAqB,CAAC;IACjC,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED;;;;;GAKG;AACH,wBAAgB,sBAAsB,CACpC,MAAM,EAAE,SAAS;IAAE,SAAS,EAAE,MAAM,CAAC;IAAC,SAAS,EAAE,qBAAqB,CAAC;IAAC,SAAS,CAAC,EAAE,MAAM,CAAA;CAAE,EAAE,GAC7F,cAAc,EAAE,CAmBlB;AAED,wBAAgB,uBAAuB,CACrC,SAAS,EAAE,MAAM,GAAG,SAAS,GAC5B,qBAAqB,GAAG,SAAS,CAGnC"}

package/dist/src/presentation/web/components/common/feature-node/derive-feature-state.js

@@ -31,7 +31,7 @@ export function deriveNodeState(feature, agentRun) {
             case AgentRunStatus.cancelled:
                 return 'blocked';
             case AgentRunStatus.completed:
-                return feature.lifecycle === SdlcLifecycle.Maintain ? 'done' : 'done';
+                return 'done';
             case AgentRunStatus.running:
             case AgentRunStatus.pending:
                 return 'running';
@@ -93,7 +93,16 @@ export function mapEventTypeToState(eventType) {
             return 'action-required';
     }
 }
-/** Maps an SSE event phaseName to a FeatureLifecyclePhase. Mirrors page.tsx nodeToLifecyclePhase. */
+/** Map domain SdlcLifecycle enum values to UI FeatureLifecyclePhase. */
+export const sdlcLifecycleMap = {
+    Requirements: 'requirements',
+    Research: 'research',
+    Implementation: 'implementation',
+    Review: 'review',
+    'Deploy & QA': 'deploy',
+    Maintain: 'maintain',
+};
+/** Map agent graph node names (from agent_run.result or SSE phaseName) to UI lifecycle phases. */
 const phaseNameToLifecycle = {
     analyze: 'requirements',
     requirements: 'requirements',
@@ -104,6 +113,40 @@ const phaseNameToLifecycle = {
     maintain: 'maintain',
     blocked: 'requirements',
 };
+/**
+ * Derives the UI lifecycle phase from a Feature + AgentRun.
+ * Shared by build-feature-node-data.ts and build-graph-nodes.ts.
+ */
+export function deriveLifecycle(feature, run) {
+    if (run?.status === 'completed')
+        return 'maintain';
+    const agentNode = run?.result?.startsWith('node:') ? run.result.slice(5) : undefined;
+    return ((agentNode ? phaseNameToLifecycle[agentNode] : undefined) ??
+        sdlcLifecycleMap[feature.lifecycle] ??
+        'requirements');
+}
+/**
+ * Resolves state/lifecycle updates from a batch of SSE events, applying
+ * the WaitingApproval batch-suppression rule: when WaitingApproval and
+ * PhaseCompleted arrive in the same batch for a feature, PhaseCompleted
+ * is fully suppressed to prevent lifecycle regression.
+ */
+export function resolveSseEventUpdates(events) {
+    const waitingApprovalFeatures = new Set(events
+        .filter((e) => e.eventType === NotificationEventType.WaitingApproval)
+        .map((e) => e.featureId));
+    return events.map((event) => {
+        const isSuppressed = event.eventType === NotificationEventType.PhaseCompleted &&
+            waitingApprovalFeatures.has(event.featureId);
+        return {
+            featureId: event.featureId,
+            state: isSuppressed ? undefined : mapEventTypeToState(event.eventType),
+            lifecycle: isSuppressed ? undefined : mapPhaseNameToLifecycle(event.phaseName),
+            eventType: event.eventType,
+            phaseName: event.phaseName,
+        };
+    });
+}
 export function mapPhaseNameToLifecycle(phaseName) {
     if (!phaseName)
         return undefined;