@shepai/cli 1.72.9 → 1.73.0

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,2CA8jBlF"}

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,8 @@ 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);
+    isOpenRef.current = isOpen;
     const onClose = useCallback(() => {
         router.push('/');
     }, [router]);
@@ -102,16 +110,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 +128,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 +226,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 +254,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;

package/dist/src/presentation/web/components/common/index.d.ts

@@ -1,5 +1,5 @@
-export { ControlCenterDrawer, type ControlCenterDrawerProps, computeDrawerView, type DrawerView, } from './control-center-drawer/index.js';
-export { AddRepositoryNode, type AddRepositoryNodeData, type AddRepositoryNodeType, } from './add-repository-node/index.js';
+export { computeDrawerView, type DrawerView } from './control-center-drawer/index.js';
+export { AddRepositoryButton } from './add-repository-button/index.js';
 export { ElapsedTime, formatElapsed } from './elapsed-time/index.js';
 export { EmptyState, type EmptyStateProps } from './empty-state/index.js';
 export { FeatureCreateDrawer, type FeatureCreateDrawerProps, type FeatureCreatePayload, } from './feature-create-drawer/index.js';
@@ -10,7 +10,6 @@ export { FeatureNode, featureNodeStateConfig, type FeatureNodeState, type Featur
 export { FeatureStatusBadges, type FeatureStatusBadgesProps } from './feature-status-badges/index.js';
 export { FeatureStatusGroup, type FeatureStatusGroupProps } from './feature-status-group/index.js';
 export { LoadingSkeleton } from './loading-skeleton/index.js';
-export { NotificationPermissionBanner } from './notification-permission-banner/index.js';
 export { RepositoryNode, type RepositoryNodeData, type RepositoryNodeType, } from './repository-node/index.js';
 export { PageHeader } from './page-header/index.js';
 export { PrdQuestionnaire, PrdQuestionnaireDrawer, type PrdQuestionnaireProps, type PrdQuestionnaireDrawerProps, type PrdQuestion, type PrdFinalAction, type PrdOption, type PrdQuestionnaireData, } from './prd-questionnaire/index.js';

package/dist/src/presentation/web/components/common/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/components/common/index.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,mBAAmB,EACnB,KAAK,wBAAwB,EAC7B,iBAAiB,EACjB,KAAK,UAAU,GAChB,MAAM,yBAAyB,CAAC;AACjC,OAAO,EACL,iBAAiB,EACjB,KAAK,qBAAqB,EAC1B,KAAK,qBAAqB,GAC3B,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,WAAW,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAC;AAC5D,OAAO,EAAE,UAAU,EAAE,KAAK,eAAe,EAAE,MAAM,eAAe,CAAC;AACjE,OAAO,EACL,mBAAmB,EACnB,KAAK,wBAAwB,EAC7B,KAAK,oBAAoB,GAC1B,MAAM,yBAAyB,CAAC;AACjC,OAAO,EAAE,aAAa,EAAE,KAAK,kBAAkB,EAAE,MAAM,kBAAkB,CAAC;AAC1E,OAAO,EAAE,eAAe,EAAE,KAAK,oBAAoB,EAAE,MAAM,qBAAqB,CAAC;AACjF,OAAO,EACL,iBAAiB,EACjB,KAAK,mBAAmB,EACxB,KAAK,mBAAmB,GACzB,MAAM,sCAAsC,CAAC;AAC9C,OAAO,EACL,WAAW,EACX,sBAAsB,EACtB,KAAK,gBAAgB,EACrB,KAAK,qBAAqB,EAC1B,KAAK,eAAe,EACpB,KAAK,eAAe,EACpB,KAAK,sBAAsB,GAC5B,MAAM,gBAAgB,CAAC;AACxB,OAAO,EAAE,mBAAmB,EAAE,KAAK,wBAAwB,EAAE,MAAM,yBAAyB,CAAC;AAC7F,OAAO,EAAE,kBAAkB,EAAE,KAAK,uBAAuB,EAAE,MAAM,wBAAwB,CAAC;AAC1F,OAAO,EAAE,eAAe,EAAE,MAAM,oBAAoB,CAAC;AACrD,OAAO,EAAE,4BAA4B,EAAE,MAAM,kCAAkC,CAAC;AAChF,OAAO,EACL,cAAc,EACd,KAAK,kBAAkB,EACvB,KAAK,kBAAkB,GACxB,MAAM,mBAAmB,CAAC;AAC3B,OAAO,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAC3C,OAAO,EACL,gBAAgB,EAChB,sBAAsB,EACtB,KAAK,qBAAqB,EAC1B,KAAK,2BAA2B,EAChC,KAAK,WAAW,EAChB,KAAK,cAAc,EACnB,KAAK,SAAS,EACd,KAAK,oBAAoB,GAC1B,MAAM,qBAAqB,CAAC;AAC7B,OAAO,EAAE,QAAQ,EAAE,KAAK,aAAa,EAAE,MAAM,aAAa,CAAC;AAC3D,OAAO,EACL,mBAAmB,EACnB,mBAAmB,EACnB,KAAK,wBAAwB,EAC7B,KAAK,wBAAwB,EAC7B,KAAK,uBAAuB,GAC7B,MAAM,yBAAyB,CAAC;AACjC,OAAO,EAAE,qBAAqB,EAAE,KAAK,0BAA0B,EAAE,MAAM,2BAA2B,CAAC;AACnG,OAAO,EAAE,cAAc,EAAE,KAAK,mBAAmB,EAAE,MAAM,oBAAoB,CAAC;AAC9E,OAAO,EAAE,oBAAoB,EAAE,KAAK,yBAAyB,EAAE,MAAM,0BAA0B,CAAC;AAChG,OAAO,EAAE,WAAW,EAAE,MAAM,gBAAgB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/components/common/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,UAAU,EAAE,MAAM,yBAAyB,CAAC;AAC7E,OAAO,EAAE,mBAAmB,EAAE,MAAM,yBAAyB,CAAC;AAC9D,OAAO,EAAE,WAAW,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAC;AAC5D,OAAO,EAAE,UAAU,EAAE,KAAK,eAAe,EAAE,MAAM,eAAe,CAAC;AACjE,OAAO,EACL,mBAAmB,EACnB,KAAK,wBAAwB,EAC7B,KAAK,oBAAoB,GAC1B,MAAM,yBAAyB,CAAC;AACjC,OAAO,EAAE,aAAa,EAAE,KAAK,kBAAkB,EAAE,MAAM,kBAAkB,CAAC;AAC1E,OAAO,EAAE,eAAe,EAAE,KAAK,oBAAoB,EAAE,MAAM,qBAAqB,CAAC;AACjF,OAAO,EACL,iBAAiB,EACjB,KAAK,mBAAmB,EACxB,KAAK,mBAAmB,GACzB,MAAM,sCAAsC,CAAC;AAC9C,OAAO,EACL,WAAW,EACX,sBAAsB,EACtB,KAAK,gBAAgB,EACrB,KAAK,qBAAqB,EAC1B,KAAK,eAAe,EACpB,KAAK,eAAe,EACpB,KAAK,sBAAsB,GAC5B,MAAM,gBAAgB,CAAC;AACxB,OAAO,EAAE,mBAAmB,EAAE,KAAK,wBAAwB,EAAE,MAAM,yBAAyB,CAAC;AAC7F,OAAO,EAAE,kBAAkB,EAAE,KAAK,uBAAuB,EAAE,MAAM,wBAAwB,CAAC;AAC1F,OAAO,EAAE,eAAe,EAAE,MAAM,oBAAoB,CAAC;AACrD,OAAO,EACL,cAAc,EACd,KAAK,kBAAkB,EACvB,KAAK,kBAAkB,GACxB,MAAM,mBAAmB,CAAC;AAC3B,OAAO,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAC3C,OAAO,EACL,gBAAgB,EAChB,sBAAsB,EACtB,KAAK,qBAAqB,EAC1B,KAAK,2BAA2B,EAChC,KAAK,WAAW,EAChB,KAAK,cAAc,EACnB,KAAK,SAAS,EACd,KAAK,oBAAoB,GAC1B,MAAM,qBAAqB,CAAC;AAC7B,OAAO,EAAE,QAAQ,EAAE,KAAK,aAAa,EAAE,MAAM,aAAa,CAAC;AAC3D,OAAO,EACL,mBAAmB,EACnB,mBAAmB,EACnB,KAAK,wBAAwB,EAC7B,KAAK,wBAAwB,EAC7B,KAAK,uBAAuB,GAC7B,MAAM,yBAAyB,CAAC;AACjC,OAAO,EAAE,qBAAqB,EAAE,KAAK,0BAA0B,EAAE,MAAM,2BAA2B,CAAC;AACnG,OAAO,EAAE,cAAc,EAAE,KAAK,mBAAmB,EAAE,MAAM,oBAAoB,CAAC;AAC9E,OAAO,EAAE,oBAAoB,EAAE,KAAK,yBAAyB,EAAE,MAAM,0BAA0B,CAAC;AAChG,OAAO,EAAE,WAAW,EAAE,MAAM,gBAAgB,CAAC"}

package/dist/src/presentation/web/components/common/index.js

@@ -1,5 +1,5 @@
-export { ControlCenterDrawer, computeDrawerView, } from './control-center-drawer/index.js';
-export { AddRepositoryNode, } from './add-repository-node/index.js';
+export { computeDrawerView } from './control-center-drawer/index.js';
+export { AddRepositoryButton } from './add-repository-button/index.js';
 export { ElapsedTime, formatElapsed } from './elapsed-time/index.js';
 export { EmptyState } from './empty-state/index.js';
 export { FeatureCreateDrawer, } from './feature-create-drawer/index.js';
@@ -10,7 +10,6 @@ export { FeatureNode, featureNodeStateConfig, } from './feature-node/index.js';
 export { FeatureStatusBadges } from './feature-status-badges/index.js';
 export { FeatureStatusGroup } from './feature-status-group/index.js';
 export { LoadingSkeleton } from './loading-skeleton/index.js';
-export { NotificationPermissionBanner } from './notification-permission-banner/index.js';
 export { RepositoryNode, } from './repository-node/index.js';
 export { PageHeader } from './page-header/index.js';
 export { PrdQuestionnaire, PrdQuestionnaireDrawer, } from './prd-questionnaire/index.js';

package/dist/src/presentation/web/components/common/repository-node/repository-node.js

@@ -23,7 +23,7 @@ export function RepositoryNode({ data }) {
         }
         : null);
     const isDeploymentActive = deployAction.status === 'Booting' || deployAction.status === 'Ready';
-    return (_jsxs("div", { className: "group relative", children: [data.showHandles ? (_jsx(Handle, { type: "target", position: Position.Left, isConnectable: false, className: "opacity-0!" })) : null, data.onDelete && data.id ? (_jsxs(_Fragment, { children: [_jsx("div", { className: "absolute top-1/2 -left-10 -translate-y-1/2 opacity-0 transition-opacity group-hover:opacity-100", children: _jsx(TooltipProvider, { children: _jsxs(Tooltip, { children: [_jsx(TooltipTrigger, { asChild: true, children: _jsx("button", { "aria-label": "Remove repository", "data-testid": "repository-node-delete-button", onClick: (e) => {
+    return (_jsxs("div", { className: cn('group relative', data.onDelete && data.id && 'pl-10'), children: [data.showHandles ? (_jsx(Handle, { type: "target", position: Position.Left, isConnectable: false, className: "opacity-0!" })) : null, data.onDelete && data.id ? (_jsxs(_Fragment, { children: [_jsx("div", { className: "absolute top-1/2 left-0 -translate-y-1/2 opacity-0 transition-opacity group-hover:opacity-100", children: _jsx(TooltipProvider, { children: _jsxs(Tooltip, { children: [_jsx(TooltipTrigger, { asChild: true, children: _jsx("button", { "aria-label": "Remove repository", "data-testid": "repository-node-delete-button", onClick: (e) => {
                                                 e.stopPropagation();
                                                 setConfirmOpen(true);
                                             }, className: "bg-card text-muted-foreground hover:border-destructive hover:text-destructive flex h-7 w-7 cursor-pointer items-center justify-center rounded-full border shadow-sm transition-colors", children: _jsx(Trash2, { className: "h-3.5 w-3.5" }) }) }), _jsx(TooltipContent, { children: "Remove repository" })] }) }) }), _jsx(Dialog, { open: confirmOpen, onOpenChange: setConfirmOpen, children: _jsxs(DialogContent, { className: "max-w-xs", children: [_jsxs(DialogHeader, { children: [_jsx(DialogTitle, { children: "Remove repository?" }), _jsxs(DialogDescription, { children: ["This will remove ", _jsx("strong", { children: data.name }), " and all its features from your workspace. The repository files on disk won't be affected."] })] }), _jsxs(DialogFooter, { className: "grid grid-cols-2 gap-2 sm:flex-none", children: [_jsx(DialogClose, { asChild: true, children: _jsx(Button, { variant: "outline", children: "Cancel" }) }), _jsx(Button, { variant: "destructive", onClick: () => {

package/dist/src/presentation/web/components/common/repository-node/use-repository-actions.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-repository-actions.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/repository-node/use-repository-actions.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,sBAAsB;IACrC,cAAc,EAAE,MAAM,CAAC;CACxB;AAED,MAAM,WAAW,sBAAsB;IACrC,SAAS,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC/B,WAAW,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IACjC,UAAU,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAChC,UAAU,EAAE,OAAO,CAAC;IACpB,YAAY,EAAE,OAAO,CAAC;IACtB,aAAa,EAAE,OAAO,CAAC;IACvB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;CAC5B;AAID,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,sBAAsB,GAAG,IAAI,GAAG,sBAAsB,CAkIjG"}
+{"version":3,"file":"use-repository-actions.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/repository-node/use-repository-actions.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,sBAAsB;IACrC,cAAc,EAAE,MAAM,CAAC;CACxB;AAED,MAAM,WAAW,sBAAsB;IACrC,SAAS,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC/B,WAAW,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IACjC,UAAU,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAChC,UAAU,EAAE,OAAO,CAAC;IACpB,YAAY,EAAE,OAAO,CAAC;IACtB,aAAa,EAAE,OAAO,CAAC;IACvB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;CAC5B;AAID,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,sBAAsB,GAAG,IAAI,GAAG,sBAAsB,CAyGjG"}