@shepai/cli 1.72.9-pr220.f17b60a → 1.73.0

package/dist/src/presentation/web/app/layout.js

@@ -36,5 +36,5 @@ export default function RootLayout({ children, }) {
                 } catch (e) {}
               })();
             `,
-                    } }) }), _jsxs("body", { className: "min-h-screen antialiased", children: [_jsx(AppShell, { children: children }), _jsx(Toaster, {})] })] }));
+                    } }) }), _jsxs("body", { className: "min-h-screen antialiased", children: [_jsx(AppShell, { children: children }), _jsx(Toaster, { position: "bottom-center" })] })] }));
 }

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":"AAyDA,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,2CA6mBlF"}
+{"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

@@ -36,6 +36,7 @@ import { featureNodeStateConfig, lifecycleDisplayLabels } from '../../common/fea
 import { useFeatureActions } from '../../common/feature-drawer/use-feature-actions.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');
@@ -56,6 +57,10 @@ 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);
@@ -64,9 +69,9 @@ export function FeatureDrawerClient({ view: initialView }) {
             if (update.featureId !== featureNode.featureId)
                 continue;
             // 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();
             if (update.state !== undefined || update.lifecycle !== undefined) {
                 // Optimistically update the node data AND re-derive the view type so the
@@ -94,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]);
@@ -103,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 ────────────────────────────────────────────────
@@ -125,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' &&

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/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"}

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

@@ -16,28 +16,27 @@ export function useRepositoryActions(input) {
     const folderTimerRef = useRef(null);
     // Clear timers on unmount
     useEffect(() => {
-        const ideTimer = ideTimerRef.current;
-        const shellTimer = shellTimerRef.current;
-        const folderTimer = folderTimerRef.current;
+        const ideRef = ideTimerRef;
+        const shellRef = shellTimerRef;
+        const folderRef = folderTimerRef;
         return () => {
-            if (ideTimer)
-                clearTimeout(ideTimer);
-            if (shellTimer)
-                clearTimeout(shellTimer);
-            if (folderTimer)
-                clearTimeout(folderTimer);
+            if (ideRef.current)
+                clearTimeout(ideRef.current);
+            if (shellRef.current)
+                clearTimeout(shellRef.current);
+            if (folderRef.current)
+                clearTimeout(folderRef.current);
         };
     }, []);
-    const performToolbarAction = useCallback(async (action, setLoading, setError, timerRef, isLoading) => {
+    const performAction = useCallback(async (action, setLoading, setError, timerRef, isLoading) => {
         if (!input || isLoading)
             return;
-        // Clear any existing timer
         if (timerRef.current)
             clearTimeout(timerRef.current);
         setLoading(true);
         setError(null);
         try {
-            const result = await action({ repositoryPath: input.repositoryPath });
+            const result = await action();
             if (!result.success) {
                 const errorMessage = result.error ?? 'An unexpected error occurred';
                 setError(errorMessage);
@@ -53,33 +52,9 @@ export function useRepositoryActions(input) {
             setLoading(false);
         }
     }, [input]);
-    const performFolderAction = useCallback(async (action, setLoading, setError, timerRef, isLoading) => {
-        if (!input || isLoading)
-            return;
-        if (timerRef.current)
-            clearTimeout(timerRef.current);
-        setLoading(true);
-        setError(null);
-        try {
-            const result = await action(input.repositoryPath);
-            if (!result.success) {
-                const errorMessage = result.error ?? 'An unexpected error occurred';
-                setError(errorMessage);
-                timerRef.current = setTimeout(() => setError(null), ERROR_CLEAR_DELAY);
-            }
-        }
-        catch (err) {
-            const errorMessage = err instanceof Error ? err.message : 'An unexpected error occurred';
-            setError(errorMessage);
-            timerRef.current = setTimeout(() => setError(null), ERROR_CLEAR_DELAY);
-        }
-        finally {
-            setLoading(false);
-        }
-    }, [input]);
-    const handleOpenIde = useCallback(() => performToolbarAction(openIde, setIdeLoading, setIdeError, ideTimerRef, ideLoading), [performToolbarAction, ideLoading]);
-    const handleOpenShell = useCallback(() => performToolbarAction(openShell, setShellLoading, setShellError, shellTimerRef, shellLoading), [performToolbarAction, shellLoading]);
-    const handleOpenFolder = useCallback(() => performFolderAction(openFolder, setFolderLoading, setFolderError, folderTimerRef, folderLoading), [performFolderAction, folderLoading]);
+    const handleOpenIde = useCallback(() => performAction(() => openIde({ repositoryPath: input.repositoryPath }), setIdeLoading, setIdeError, ideTimerRef, ideLoading), [performAction, ideLoading, input]);
+    const handleOpenShell = useCallback(() => performAction(() => openShell({ repositoryPath: input.repositoryPath }), setShellLoading, setShellError, shellTimerRef, shellLoading), [performAction, shellLoading, input]);
+    const handleOpenFolder = useCallback(() => performAction(() => openFolder(input.repositoryPath), setFolderLoading, setFolderError, folderTimerRef, folderLoading), [performAction, folderLoading, input]);
     return {
         openInIde: handleOpenIde,
         openInShell: handleOpenShell,

package/dist/src/presentation/web/components/features/control-center/control-center-inner.js

@@ -18,19 +18,18 @@ export function ControlCenterInner({ initialNodes, initialEdges }) {
     const { nodes, edges, onNodesChange, handleConnect, handleAddRepository, handleDeleteFeature, handleDeleteRepository, createFeatureNode, setCallbacks, } = useControlCenterState(initialNodes, initialEdges);
     // Publish sidebar features to context whenever feature node data changes
     const { setFeatures: setSidebarFeatures } = useSidebarFeaturesContext();
+    const featureNodes = useMemo(() => nodes.filter((n) => n.type === 'featureNode'), [nodes]);
     const sidebarKey = useMemo(() => {
-        return nodes
-            .filter((n) => n.type === 'featureNode')
+        return featureNodes
             .map((n) => {
             const d = n.data;
-            return `${d.featureId}:${d.state}:${d.runtime ?? ''}:${d.startedAt ?? ''}`;
+            return `${d.featureId}:${d.state}:${d.name}`;
         })
             .sort()
             .join(',');
-    }, [nodes]);
+    }, [featureNodes]);
     useEffect(() => {
-        const sidebarItems = nodes
-            .filter((n) => n.type === 'featureNode')
+        const sidebarItems = featureNodes
             .map((n) => {
             const d = n.data;
             const status = mapNodeStateToSidebarStatus(d.state);
@@ -46,7 +45,7 @@ export function ControlCenterInner({ initialNodes, initialEdges }) {
         })
             .filter(Boolean);
         setSidebarFeatures(sidebarItems);
-    }, [sidebarKey, nodes, setSidebarFeatures]);
+    }, [sidebarKey, featureNodes, setSidebarFeatures]);
     // ── URL-based navigation handlers ────────────────────────────────────
     const handleNodeClick = useCallback((_event, node) => {
         if (node.type === 'featureNode') {

package/dist/src/presentation/web/components/features/control-center/use-control-center-state.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-control-center-state.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/features/control-center/use-control-center-state.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,UAAU,EAAE,IAAI,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAClE,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AACxE,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAC9E,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAC5E,OAAO,EAGL,KAAK,eAAe,EACrB,MAAM,yBAAyB,CAAC;AAWjC,OAAO,EAAiB,KAAK,cAAc,EAAE,MAAM,yBAAyB,CAAC;AAG7E,MAAM,WAAW,kBAAkB;IACjC,KAAK,EAAE,cAAc,EAAE,CAAC;IACxB,KAAK,EAAE,IAAI,EAAE,CAAC;IACd,aAAa,EAAE,CAAC,OAAO,EAAE,UAAU,CAAC,cAAc,CAAC,EAAE,KAAK,IAAI,CAAC;IAC/D,aAAa,EAAE,CAAC,UAAU,EAAE,UAAU,KAAK,IAAI,CAAC;IAChD,mBAAmB,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,CAAC;IAC5C,YAAY,EAAE,CAAC,SAAS,EAAE,eAAe,KAAK,IAAI,CAAC;IACnD,mBAAmB,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IACjD,sBAAsB,EAAE,CAAC,YAAY,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IAChE,iBAAiB,EAAE,CACjB,YAAY,EAAE,MAAM,GAAG,IAAI,EAC3B,YAAY,CAAC,EAAE,OAAO,CAAC,eAAe,CAAC,EACvC,QAAQ,CAAC,EAAE,MAAM,KACd,MAAM,CAAC;IACZ,wDAAwD;IACxD,wBAAwB,EAAE,CAAC,aAAa,EAAE,MAAM,KAAK,MAAM,GAAG,SAAS,CAAC;IACxE,gDAAgD;IAChD,iBAAiB,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,kBAAkB,GAAG,SAAS,CAAC;IACtE,0EAA0E;IAC1E,YAAY,EAAE,CAAC,SAAS,EAAE,cAAc,KAAK,IAAI,CAAC;CACnD;AAOD,wBAAgB,qBAAqB,CACnC,YAAY,EAAE,cAAc,EAAE,EAC9B,YAAY,EAAE,IAAI,EAAE,GACnB,kBAAkB,CA+UpB"}
+{"version":3,"file":"use-control-center-state.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/features/control-center/use-control-center-state.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,UAAU,EAAE,IAAI,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAClE,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AACxE,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAC9E,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAC5E,OAAO,EAGL,KAAK,eAAe,EACrB,MAAM,yBAAyB,CAAC;AAWjC,OAAO,EAAiB,KAAK,cAAc,EAAE,MAAM,yBAAyB,CAAC;AAG7E,MAAM,WAAW,kBAAkB;IACjC,KAAK,EAAE,cAAc,EAAE,CAAC;IACxB,KAAK,EAAE,IAAI,EAAE,CAAC;IACd,aAAa,EAAE,CAAC,OAAO,EAAE,UAAU,CAAC,cAAc,CAAC,EAAE,KAAK,IAAI,CAAC;IAC/D,aAAa,EAAE,CAAC,UAAU,EAAE,UAAU,KAAK,IAAI,CAAC;IAChD,mBAAmB,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,CAAC;IAC5C,YAAY,EAAE,CAAC,SAAS,EAAE,eAAe,KAAK,IAAI,CAAC;IACnD,mBAAmB,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IACjD,sBAAsB,EAAE,CAAC,YAAY,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IAChE,iBAAiB,EAAE,CACjB,YAAY,EAAE,MAAM,GAAG,IAAI,EAC3B,YAAY,CAAC,EAAE,OAAO,CAAC,eAAe,CAAC,EACvC,QAAQ,CAAC,EAAE,MAAM,KACd,MAAM,CAAC;IACZ,wDAAwD;IACxD,wBAAwB,EAAE,CAAC,aAAa,EAAE,MAAM,KAAK,MAAM,GAAG,SAAS,CAAC;IACxE,gDAAgD;IAChD,iBAAiB,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,kBAAkB,GAAG,SAAS,CAAC;IACtE,0EAA0E;IAC1E,YAAY,EAAE,CAAC,SAAS,EAAE,cAAc,KAAK,IAAI,CAAC;CACnD;AAOD,wBAAgB,qBAAqB,CACnC,YAAY,EAAE,cAAc,EAAE,EAC9B,YAAY,EAAE,IAAI,EAAE,GACnB,kBAAkB,CAgWpB"}

package/dist/src/presentation/web/components/features/control-center/use-control-center-state.js

@@ -19,6 +19,13 @@ export function useControlCenterState(initialNodes, initialEdges) {
     const deleteSound = useSoundAction('delete');
     const createSound = useSoundAction('create');
     const { nodes, edges, reconcile, updateFeature, addPendingFeature, removeFeature, restoreFeature, addRepository: addRepositoryToMap, removeRepository, replaceRepository, getFeatureRepositoryPath, getRepositoryData, setCallbacks, } = useGraphState(initialNodes, initialEdges);
+    // Refs for stable access to latest nodes/edges without callback recreation
+    const nodesRef = useRef(nodes);
+    const edgesRef = useRef(edges);
+    useEffect(() => {
+        nodesRef.current = nodes;
+        edgesRef.current = edges;
+    }, [nodes, edges]);
     // Sync server props into domain Maps when initialNodes/initialEdges change.
     // Keyed by a stable string so we don't re-reconcile on every render.
     const initialNodeKey = useMemo(() => initialNodes
@@ -77,6 +84,10 @@ export function useControlCenterState(initialNodes, initialEdges) {
     // SSE effect: only state + lifecycle updates
     const processedEventCountRef = useRef(0);
     useEffect(() => {
+        // Clamp cursor if events were pruned
+        if (processedEventCountRef.current > events.length) {
+            processedEventCountRef.current = 0;
+        }
         if (events.length <= processedEventCountRef.current)
             return;
         const newEvents = events.slice(processedEventCountRef.current);
@@ -103,6 +114,10 @@ export function useControlCenterState(initialNodes, initialEdges) {
     const metadataFetchedRef = useRef(new Set());
     const processedMetadataCountRef = useRef(0);
     useEffect(() => {
+        // Clamp cursor if events were pruned
+        if (processedMetadataCountRef.current > events.length) {
+            processedMetadataCountRef.current = 0;
+        }
         if (events.length <= processedMetadataCountRef.current)
             return;
         const newEvents = events.slice(processedMetadataCountRef.current);
@@ -156,7 +171,7 @@ export function useControlCenterState(initialNodes, initialEdges) {
     const handleDeleteFeature = useCallback((featureId) => {
         const nodeId = `feat-${featureId}`;
         // Find the current entry for rollback
-        const prevEntry = nodes
+        const prevEntry = nodesRef.current
             .filter((n) => n.id === nodeId)
             .map((n) => ({
             nodeId: n.id,
@@ -180,16 +195,16 @@ export function useControlCenterState(initialNodes, initialEdges) {
                 restoreFeature(nodeId, prevEntry);
             toast.error('Failed to delete feature');
         });
-    }, [nodes, router, deleteSound, removeFeature, restoreFeature]);
+    }, [router, deleteSound, removeFeature, restoreFeature]);
     const handleDeleteRepository = useCallback(async (repositoryId) => {
         const repoNodeId = `repo-${repositoryId}`;
         // Find children of this repo via edges
-        const childFeatureIds = new Set(edges.filter((e) => e.source === repoNodeId).map((e) => e.target));
+        const childFeatureIds = new Set(edgesRef.current.filter((e) => e.source === repoNodeId).map((e) => e.target));
         // Snapshot for rollback
         const prevRepoData = getRepositoryData(repoNodeId);
         const childSnapshots = new Map();
         for (const childId of childFeatureIds) {
-            const childNode = nodes.find((n) => n.id === childId);
+            const childNode = nodesRef.current.find((n) => n.id === childId);
             if (childNode) {
                 childSnapshots.set(childId, { nodeId: childId, data: childNode.data });
             }
@@ -221,8 +236,6 @@ export function useControlCenterState(initialNodes, initialEdges) {
             rollback();
         }
     }, [
-        edges,
-        nodes,
         deleteSound,
         removeRepository,
         removeFeature,
@@ -233,9 +246,12 @@ export function useControlCenterState(initialNodes, initialEdges) {
     const handleLayout = useCallback((direction) => {
         // Layout is applied via reconcile on next server prop update.
         // For immediate re-layout, we apply dagre and trigger a reconcile-like update.
-        const result = layoutWithDagre(nodes, edges, { ...CANVAS_LAYOUT_DEFAULTS, direction });
+        const result = layoutWithDagre(nodesRef.current, edgesRef.current, {
+            ...CANVAS_LAYOUT_DEFAULTS,
+            direction,
+        });
         reconcile(result.nodes, result.edges);
-    }, [nodes, edges, reconcile]);
+    }, [reconcile]);
     const handleAddRepository = useCallback((path) => {
         const tempId = `repo-temp-${Date.now()}`;
         const repoName = path

package/dist/src/presentation/web/hooks/agent-events-provider.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"agent-events-provider.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/agent-events-provider.tsx"],"names":[],"mappings":"AAEA,OAAO,EAA6B,KAAK,SAAS,EAAE,MAAM,OAAO,CAAC;AAClE,OAAO,EAEL,KAAK,qBAAqB,EAC1B,KAAK,oBAAoB,EAC1B,MAAM,oBAAoB,CAAC;AAI5B,UAAU,wBAAyB,SAAQ,qBAAqB;IAC9D,QAAQ,EAAE,SAAS,CAAC;CACrB;AAED;;;GAGG;AACH,wBAAgB,mBAAmB,CAAC,EAAE,QAAQ,EAAE,KAAK,EAAE,EAAE,wBAAwB,2CAIhF;AAED,wBAAgB,qBAAqB,IAAI,oBAAoB,CAM5D"}
+{"version":3,"file":"agent-events-provider.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/agent-events-provider.tsx"],"names":[],"mappings":"AAEA,OAAO,EAAsC,KAAK,SAAS,EAAE,MAAM,OAAO,CAAC;AAC3E,OAAO,EAEL,KAAK,qBAAqB,EAC1B,KAAK,oBAAoB,EAC1B,MAAM,oBAAoB,CAAC;AAI5B,UAAU,wBAAyB,SAAQ,qBAAqB;IAC9D,QAAQ,EAAE,SAAS,CAAC;CACrB;AAED;;;GAGG;AACH,wBAAgB,mBAAmB,CAAC,EAAE,QAAQ,EAAE,KAAK,EAAE,EAAE,wBAAwB,2CAShF;AAED,wBAAgB,qBAAqB,IAAI,oBAAoB,CAM5D"}

package/dist/src/presentation/web/hooks/agent-events-provider.js

@@ -1,6 +1,6 @@
 'use client';
 import { jsx as _jsx } from "react/jsx-runtime";
-import { createContext, useContext } from 'react';
+import { createContext, useContext, useMemo } from 'react';
 import { useAgentEvents, } from './use-agent-events.js';
 const AgentEventsContext = createContext(null);
 /**
@@ -8,7 +8,8 @@ const AgentEventsContext = createContext(null);
  * Wrap the app once; use `useAgentEventsContext()` to read.
  */
 export function AgentEventsProvider({ children, runId }) {
-    const value = useAgentEvents({ runId });
+    const { events, lastEvent, connectionStatus } = useAgentEvents({ runId });
+    const value = useMemo(() => ({ events, lastEvent, connectionStatus }), [events, lastEvent, connectionStatus]);
     return _jsx(AgentEventsContext.Provider, { value: value, children: children });
 }
 export function useAgentEventsContext() {

package/dist/src/presentation/web/hooks/use-agent-events.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-agent-events.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-agent-events.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,sCAAsC,CAAC;AAE9E,MAAM,MAAM,gBAAgB,GAAG,WAAW,GAAG,YAAY,GAAG,cAAc,CAAC;AAE3E,MAAM,WAAW,qBAAqB;IACpC,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED,MAAM,WAAW,oBAAoB;IACnC,MAAM,EAAE,iBAAiB,EAAE,CAAC;IAC5B,SAAS,EAAE,iBAAiB,GAAG,IAAI,CAAC;IACpC,gBAAgB,EAAE,gBAAgB,CAAC;CACpC;AAID;;;;;;GAMG;AACH,wBAAgB,cAAc,CAAC,OAAO,CAAC,EAAE,qBAAqB,GAAG,oBAAoB,CA+GpF"}
+{"version":3,"file":"use-agent-events.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-agent-events.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,sCAAsC,CAAC;AAE9E,MAAM,MAAM,gBAAgB,GAAG,WAAW,GAAG,YAAY,GAAG,cAAc,CAAC;AAE3E,MAAM,WAAW,qBAAqB;IACpC,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED,MAAM,WAAW,oBAAoB;IACnC,MAAM,EAAE,iBAAiB,EAAE,CAAC;IAC5B,SAAS,EAAE,iBAAiB,GAAG,IAAI,CAAC;IACpC,gBAAgB,EAAE,gBAAgB,CAAC;CACpC;AAMD;;;;;;GAMG;AACH,wBAAgB,cAAc,CAAC,OAAO,CAAC,EAAE,qBAAqB,GAAG,oBAAoB,CAoHpF"}

package/dist/src/presentation/web/hooks/use-agent-events.js

@@ -1,6 +1,8 @@
 'use client';
 import { useCallback, useEffect, useRef, useState } from 'react';
 const SW_PATH = '/agent-events-sw.js';
+const MAX_EVENTS = 500;
+const PRUNE_KEEP = 250;
 /**
  * Hook that receives real-time agent notification events via a Service Worker.
  *
@@ -20,7 +22,12 @@ export function useAgentEvents(options) {
             return;
         if (msg.type === 'notification') {
             const parsed = msg.data;
-            setEvents((prev) => [...prev, parsed]);
+            // eslint-disable-next-line no-console
+            console.log('[SSE] event received:', parsed.eventType, parsed);
+            setEvents((prev) => {
+                const next = [...prev, parsed];
+                return next.length > MAX_EVENTS ? next.slice(-PRUNE_KEEP) : next;
+            });
             setLastEvent(parsed);
         }
         else if (msg.type === 'status') {
@@ -146,7 +153,10 @@ function connectDirectEventSource(runId, setEvents, setLastEvent, setConnectionS
         };
         es.addEventListener('notification', ((event) => {
             const parsed = JSON.parse(event.data);
-            setEvents((prev) => [...prev, parsed]);
+            setEvents((prev) => {
+                const next = [...prev, parsed];
+                return next.length > MAX_EVENTS ? next.slice(-PRUNE_KEEP) : next;
+            });
             setLastEvent(parsed);
         }));
     }

package/dist/src/presentation/web/hooks/use-deploy-action.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-deploy-action.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-deploy-action.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,sCAAsC,CAAC;AAM5E,MAAM,WAAW,iBAAiB;IAChC,QAAQ,EAAE,MAAM,CAAC;IACjB,UAAU,EAAE,SAAS,GAAG,YAAY,CAAC;IACrC,cAAc,EAAE,MAAM,CAAC;IACvB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,iBAAiB;IAChC,MAAM,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC5B,IAAI,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC1B,aAAa,EAAE,OAAO,CAAC;IACvB,WAAW,EAAE,OAAO,CAAC;IACrB,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,MAAM,EAAE,eAAe,GAAG,IAAI,CAAC;IAC/B,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;CACpB;AAeD,wBAAgB,eAAe,CAAC,KAAK,EAAE,iBAAiB,GAAG,IAAI,GAAG,iBAAiB,CAgKlF"}
+{"version":3,"file":"use-deploy-action.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-deploy-action.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,sCAAsC,CAAC;AAM5E,MAAM,WAAW,iBAAiB;IAChC,QAAQ,EAAE,MAAM,CAAC;IACjB,UAAU,EAAE,SAAS,GAAG,YAAY,CAAC;IACrC,cAAc,EAAE,MAAM,CAAC;IACvB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,iBAAiB;IAChC,MAAM,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC5B,IAAI,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC1B,aAAa,EAAE,OAAO,CAAC;IACvB,WAAW,EAAE,OAAO,CAAC;IACrB,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,MAAM,EAAE,eAAe,GAAG,IAAI,CAAC;IAC/B,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;CACpB;AAeD,wBAAgB,eAAe,CAAC,KAAK,EAAE,iBAAiB,GAAG,IAAI,GAAG,iBAAiB,CAwKlF"}