@shepai/cli 1.107.1 → 1.107.2

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

@@ -10,7 +10,6 @@ import { rejectFeature } from '../../../app/actions/reject-feature.js';
 import { getFeatureArtifact } from '../../../app/actions/get-feature-artifact.js';
 import { getResearchArtifact } from '../../../app/actions/get-research-artifact.js';
 import { getMergeReviewData } from '../../../app/actions/get-merge-review-data.js';
-import { deleteFeature } from '../../../app/actions/delete-feature.js';
 import { useFeatureFlags } from '../../../hooks/feature-flags-context.js';
 import { useSoundAction } from '../../../hooks/use-sound-action.js';
 import { useGuardedDrawerClose } from '../../../hooks/drawer-close-guard.js';
@@ -33,7 +32,6 @@ import { useDrawerSync } from './use-drawer-sync.js';
 export function FeatureDrawerClient({ view: initialView, urlTab }) {
     const featureFlags = useFeatureFlags();
     const router = useRouter();
-    const deleteSound = useSoundAction('delete');
     const rejectSound = useSoundAction('reject');
     // Track the view locally so SSE events can update the drawer type in real-time
     const [view, setView] = useState(initialView);
@@ -255,24 +253,16 @@ export function FeatureDrawerClient({ view: initialView, urlTab }) {
     const handleMergeApprove = useCallback(() => handleSimpleApprove('Merge'), [handleSimpleApprove]);
     const handleDelete = useCallback(async (featureId, cleanup) => {
         setIsDeleting(true);
-        try {
-            const result = await deleteFeature(featureId, cleanup);
-            if (result.error) {
-                toast.error(result.error);
-                return;
-            }
-            deleteSound.play();
-            toast.success('Feature deleted successfully');
-            router.push('/');
-            router.refresh();
-        }
-        catch {
-            toast.error('Failed to delete feature');
-        }
-        finally {
-            setIsDeleting(false);
-        }
-    }, [router, deleteSound]);
+        // Close the delete dialog and drawer before the server action so the
+        // user sees immediate feedback. We dispatch a DOM event so the canvas
+        // control center can run its own optimistic delete flow (deleting state,
+        // mutation guard, node removal) in parallel.
+        setDeleteDialogOpen(false);
+        window.dispatchEvent(new CustomEvent('shep:feature-delete-requested', {
+            detail: { featureId, cleanup },
+        }));
+        router.push('/');
+    }, [router]);
     const handleRetry = useCallback(async (featureId) => {
         const result = await resumeFeature(featureId);
         if (result.error) {

package/dist/src/presentation/web/components/common/merge-review/diff-view.js

@@ -26,5 +26,5 @@ function FileDiffItem({ file }) {
 export function DiffView({ fileDiffs }) {
     if (fileDiffs.length === 0)
         return null;
-    return (_jsxs("div", { className: "border-border rounded-lg border", children: [_jsx("div", { className: "px-4 py-3", children: _jsxs("div", { className: "mb-2 flex items-center gap-2", children: [_jsx(FileText, { className: "text-muted-foreground h-4 w-4" }), _jsx("span", { className: "text-foreground text-xs font-semibold", children: "Changed Files" }), _jsxs("span", { className: "text-muted-foreground text-[10px]", children: ["(", fileDiffs.length, ")"] })] }) }), _jsx("div", { className: "border-border border-t", children: fileDiffs.map((file) => (_jsx(FileDiffItem, { file: file }, file.path))) })] }));
+    return (_jsxs("div", { className: "border-border rounded-lg border", children: [_jsx("div", { className: "px-4 py-3", children: _jsxs("div", { className: "mb-2 flex items-center gap-2", children: [_jsx(FileText, { className: "text-muted-foreground h-4 w-4" }), _jsx("span", { className: "text-foreground text-xs font-semibold", children: "Changed Files" }), _jsxs("span", { className: "text-muted-foreground text-[10px]", children: ["(", fileDiffs.length, ")"] })] }) }), _jsx("div", { className: "border-border border-t", children: fileDiffs.map((file) => (_jsx(FileDiffItem, { file: file }, `${file.status}-${file.path}`))) })] }));
 }

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

@@ -1 +1 @@
-{"version":3,"file":"control-center-inner.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/features/control-center/control-center-inner.tsx"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,IAAI,EAAY,MAAM,eAAe,CAAC;AAGpD,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAsB5E,UAAU,uBAAuB;IAC/B,YAAY,EAAE,cAAc,EAAE,CAAC;IAC/B,YAAY,EAAE,IAAI,EAAE,CAAC;CACtB;AAED,wBAAgB,kBAAkB,CAAC,EAAE,YAAY,EAAE,YAAY,EAAE,EAAE,uBAAuB,2CAuTzF"}
+{"version":3,"file":"control-center-inner.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/features/control-center/control-center-inner.tsx"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,IAAI,EAAY,MAAM,eAAe,CAAC;AAGpD,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAsB5E,UAAU,uBAAuB;IAC/B,YAAY,EAAE,cAAc,EAAE,CAAC;IAC/B,YAAY,EAAE,IAAI,EAAE,CAAC;CACtB;AAED,wBAAgB,kBAAkB,CAAC,EAAE,YAAY,EAAE,YAAY,EAAE,EAAE,uBAAuB,2CAoUzF"}

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

@@ -174,6 +174,18 @@ export function ControlCenterInner({ initialNodes, initialEdges }) {
         window.addEventListener('shep:feature-created', handler);
         return () => window.removeEventListener('shep:feature-created', handler);
     }, [nodes, createFeatureNode]);
+    // Listen for delete requests from the feature drawer (fires when the user
+    // confirms delete inside the drawer). Delegates to handleDeleteFeature so
+    // the canvas gets optimistic state, mutation guard, and node removal.
+    useEffect(() => {
+        const handler = (e) => {
+            const { featureId, cleanup } = e
+                .detail;
+            handleDeleteFeature(featureId, cleanup);
+        };
+        window.addEventListener('shep:feature-delete-requested', handler);
+        return () => window.removeEventListener('shep:feature-delete-requested', handler);
+    }, [handleDeleteFeature]);
     // Wire callbacks into derived node data (via ref — no re-render).
     useEffect(() => {
         setCallbacks({

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;AAcjC,OAAO,EAAiB,KAAK,cAAc,EAAE,MAAM,yBAAyB,CAAC;AAM7E,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;QAAE,QAAQ,EAAE,OAAO,CAAC;QAAC,QAAQ,EAAE,MAAM,CAAA;KAAE,CAAC;IAC/E,YAAY,EAAE,CAAC,SAAS,EAAE,eAAe,KAAK,IAAI,CAAC;IACnD,mBAAmB,EAAE,CAAC,SAAS,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,OAAO,KAAK,IAAI,CAAC;IACpE,kBAAkB,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IAChD,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;AAQD,wBAAgB,qBAAqB,CACnC,YAAY,EAAE,cAAc,EAAE,EAC9B,YAAY,EAAE,IAAI,EAAE,GACnB,kBAAkB,CA6bpB"}
+{"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;AAcjC,OAAO,EAAiB,KAAK,cAAc,EAAE,MAAM,yBAAyB,CAAC;AAM7E,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;QAAE,QAAQ,EAAE,OAAO,CAAC;QAAC,QAAQ,EAAE,MAAM,CAAA;KAAE,CAAC;IAC/E,YAAY,EAAE,CAAC,SAAS,EAAE,eAAe,KAAK,IAAI,CAAC;IACnD,mBAAmB,EAAE,CAAC,SAAS,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,OAAO,KAAK,IAAI,CAAC;IACpE,kBAAkB,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IAChD,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;AAQD,wBAAgB,qBAAqB,CACnC,YAAY,EAAE,cAAc,EAAE,EAC9B,YAAY,EAAE,IAAI,EAAE,GACnB,kBAAkB,CAudpB"}

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

@@ -23,7 +23,7 @@ export function useControlCenterState(initialNodes, initialEdges) {
     const router = useRouter();
     const deleteSound = useSoundAction('delete');
     const createSound = useSoundAction('create');
-    const { nodes, edges, reconcile, updateFeature, addPendingFeature, removeFeature, restoreFeature, addRepository: addRepositoryToMap, removeRepository, replaceRepository, getFeatureRepositoryPath, getRepositoryData, getRepoMapSize, setCallbacks, } = useGraphState(initialNodes, initialEdges);
+    const { nodes, edges, reconcile, updateFeature, addPendingFeature, removeFeature, restoreFeature, addRepository: addRepositoryToMap, removeRepository, replaceRepository, getFeatureRepositoryPath, getRepositoryData, getRepoMapSize, setCallbacks, beginMutation, endMutation, isMutating, } = useGraphState(initialNodes, initialEdges);
     // Refs for stable access to latest nodes/edges without callback recreation
     const nodesRef = useRef(nodes);
     const edgesRef = useRef(edges);
@@ -156,6 +156,10 @@ export function useControlCenterState(initialNodes, initialEdges) {
     useEffect(() => {
         log.debug(`polling enabled (${POLL_INTERVAL_MS}ms interval)`);
         const timer = setInterval(async () => {
+            // Skip fetch entirely while a mutation is in-flight — the response
+            // would contain pre-mutation data that reconcile would discard anyway.
+            if (isMutating())
+                return;
             try {
                 // Use a plain fetch instead of a server action so the poll
                 // doesn't trigger the Next.js "Rendering…" indicator.
@@ -173,7 +177,7 @@ export function useControlCenterState(initialNodes, initialEdges) {
             log.debug('polling disabled');
             clearInterval(timer);
         };
-    }, [reconcile]);
+    }, [reconcile, isMutating]);
     // onNodesChange is a no-op: nodes are derived from domain Maps.
     // Since nodesDraggable=false and elementsSelectable=false, only React Flow's
     // internal replace/dimensions changes come through — we ignore them all.
@@ -205,6 +209,7 @@ export function useControlCenterState(initialNodes, initialEdges) {
     const handleRetryFeature = useCallback((featureId) => {
         const nodeId = `feat-${featureId}`;
         // Optimistic: switch to running state
+        beginMutation();
         updateFeature(nodeId, { state: 'running' });
         resumeFeature(featureId)
             .then((result) => {
@@ -219,8 +224,9 @@ export function useControlCenterState(initialNodes, initialEdges) {
             .catch(() => {
             updateFeature(nodeId, { state: 'error' });
             toast.error('Failed to resume feature');
-        });
-    }, [updateFeature]);
+        })
+            .finally(() => endMutation());
+    }, [updateFeature, beginMutation, endMutation]);
     const handleDeleteFeature = useCallback((featureId, cleanup) => {
         const nodeId = `feat-${featureId}`;
         // Collect all descendant feature node IDs (children, grandchildren, etc.)
@@ -244,6 +250,7 @@ export function useControlCenterState(initialNodes, initialEdges) {
             }
         }
         // Optimistic: show "deleting" state on parent AND all descendants
+        beginMutation();
         updateFeature(nodeId, { state: 'deleting' });
         for (const childId of descendants) {
             updateFeature(childId, { state: 'deleting' });
@@ -276,8 +283,9 @@ export function useControlCenterState(initialNodes, initialEdges) {
                     updateFeature(nid, { state: prevState });
             }
             toast.error('Failed to delete feature');
-        });
-    }, [router, deleteSound, updateFeature, removeFeature]);
+        })
+            .finally(() => endMutation());
+    }, [router, deleteSound, updateFeature, removeFeature, beginMutation, endMutation]);
     const handleDeleteRepository = useCallback(async (repositoryId) => {
         const repoNodeId = `repo-${repositoryId}`;
         // Find children of this repo via edges
@@ -299,6 +307,7 @@ export function useControlCenterState(initialNodes, initialEdges) {
             }
         };
         // Optimistic: remove repo + children
+        beginMutation();
         removeRepository(repoNodeId);
         for (const childId of childFeatureIds) {
             removeFeature(childId);
@@ -316,6 +325,9 @@ export function useControlCenterState(initialNodes, initialEdges) {
             toast.error('Failed to remove repository');
             rollback();
         }
+        finally {
+            endMutation();
+        }
     }, [
         deleteSound,
         removeRepository,
@@ -323,6 +335,8 @@ export function useControlCenterState(initialNodes, initialEdges) {
         addRepositoryToMap,
         restoreFeature,
         getRepositoryData,
+        beginMutation,
+        endMutation,
     ]);
     const handleLayout = useCallback((direction) => {
         // Layout is applied via reconcile on next server prop update.
@@ -340,6 +354,7 @@ export function useControlCenterState(initialNodes, initialEdges) {
             .replace(/[\\/]+$/, '')
             .split(/[\\/]/)
             .pop() ?? path;
+        beginMutation();
         addRepositoryToMap(tempId, {
             name: repoName,
             repositoryPath: path,
@@ -366,9 +381,18 @@ export function useControlCenterState(initialNodes, initialEdges) {
             .catch(() => {
             removeRepository(tempId);
             toast.error('Failed to add repository');
-        });
+        })
+            .finally(() => endMutation());
         return { wasEmpty, repoPath: path };
-    }, [addRepositoryToMap, removeRepository, replaceRepository, createSound, getRepoMapSize]);
+    }, [
+        addRepositoryToMap,
+        removeRepository,
+        replaceRepository,
+        createSound,
+        getRepoMapSize,
+        beginMutation,
+        endMutation,
+    ]);
     return {
         nodes,
         edges,

package/dist/src/presentation/web/hooks/use-graph-state.d.ts

@@ -12,6 +12,7 @@ export interface UseGraphStateReturn {
     /**
      * Reconcile domain Maps with fresh server data (new initialNodes/initialEdges).
      * Preserves pending (creating) nodes unless a matching real feature appears.
+     * No-ops when a mutation is in-flight (see beginMutation/endMutation).
      */
     reconcile: (newNodes: CanvasNodeType[], newEdges: Edge[]) => void;
     /** Update a feature's state/lifecycle/name (e.g., from SSE events). */
@@ -38,6 +39,19 @@ export interface UseGraphStateReturn {
     getRepoMapSize: () => number;
     /** Update callbacks injected into node data (does NOT trigger re-render). */
     setCallbacks: (callbacks: GraphCallbacks) => void;
+    /**
+     * Signal that an optimistic mutation has started. While any mutation is
+     * in-flight, `reconcile` becomes a no-op so stale poll data cannot
+     * overwrite optimistic state. Calls are ref-counted — nest freely.
+     */
+    beginMutation: () => void;
+    /**
+     * Signal that an optimistic mutation has resolved. Adds a one-poll-interval
+     * cooldown (default 3 s) so the next poll fetches post-mutation data.
+     */
+    endMutation: (cooldownMs?: number) => void;
+    /** Whether a mutation is currently in-flight (for polling skip logic). */
+    isMutating: () => boolean;
 }
 type FeatureDataUpdates = Partial<Pick<FeatureNodeData, 'state' | 'lifecycle' | 'name' | 'description'>>;
 export declare function useGraphState(initialNodes: CanvasNodeType[], initialEdges: Edge[]): UseGraphStateReturn;

package/dist/src/presentation/web/hooks/use-graph-state.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-graph-state.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-graph-state.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,IAAI,EAAY,MAAM,eAAe,CAAC;AACpD,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAC5E,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AACxE,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAC9E,OAAO,EAEL,KAAK,YAAY,EAEjB,KAAK,cAAc,EACpB,MAAM,oBAAoB,CAAC;AAG5B,YAAY,EAAE,cAAc,EAAE,MAAM,oBAAoB,CAAC;AAEzD,MAAM,WAAW,mBAAmB;IAClC,mDAAmD;IACnD,KAAK,EAAE,cAAc,EAAE,CAAC;IACxB,mDAAmD;IACnD,KAAK,EAAE,IAAI,EAAE,CAAC;IACd;;;OAGG;IACH,SAAS,EAAE,CAAC,QAAQ,EAAE,cAAc,EAAE,EAAE,QAAQ,EAAE,IAAI,EAAE,KAAK,IAAI,CAAC;IAClE,uEAAuE;IACvE,aAAa,EAAE,CAAC,aAAa,EAAE,MAAM,EAAE,OAAO,EAAE,kBAAkB,KAAK,IAAI,CAAC;IAC5E,kEAAkE;IAClE,iBAAiB,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,eAAe,EAAE,YAAY,CAAC,EAAE,MAAM,KAAK,IAAI,CAAC;IAC1F,yDAAyD;IACzD,oBAAoB,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,IAAI,CAAC;IAC/C,sDAAsD;IACtD,aAAa,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,IAAI,CAAC;IACxC,uDAAuD;IACvD,cAAc,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,YAAY,KAAK,IAAI,CAAC;IAC9D,2DAA2D;IAC3D,aAAa,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,kBAAkB,KAAK,IAAI,CAAC;IAClE,gCAAgC;IAChC,gBAAgB,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,IAAI,CAAC;IAC3C,yDAAyD;IACzD,iBAAiB,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,kBAAkB,KAAK,IAAI,CAAC;IACtF,gEAAgE;IAChE,wBAAwB,EAAE,CAAC,aAAa,EAAE,MAAM,KAAK,MAAM,GAAG,SAAS,CAAC;IACxE,yDAAyD;IACzD,iBAAiB,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,kBAAkB,GAAG,SAAS,CAAC;IACtE,+EAA+E;IAC/E,cAAc,EAAE,MAAM,MAAM,CAAC;IAC7B,6EAA6E;IAC7E,YAAY,EAAE,CAAC,SAAS,EAAE,cAAc,KAAK,IAAI,CAAC;CACnD;AAuCD,KAAK,kBAAkB,GAAG,OAAO,CAC/B,IAAI,CAAC,eAAe,EAAE,OAAO,GAAG,WAAW,GAAG,MAAM,GAAG,aAAa,CAAC,CACtE,CAAC;AA8BF,wBAAgB,aAAa,CAC3B,YAAY,EAAE,cAAc,EAAE,EAC9B,YAAY,EAAE,IAAI,EAAE,GACnB,mBAAmB,CAuXrB"}
+{"version":3,"file":"use-graph-state.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-graph-state.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,IAAI,EAAY,MAAM,eAAe,CAAC;AACpD,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAC5E,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AACxE,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAC9E,OAAO,EAEL,KAAK,YAAY,EAEjB,KAAK,cAAc,EACpB,MAAM,oBAAoB,CAAC;AAG5B,YAAY,EAAE,cAAc,EAAE,MAAM,oBAAoB,CAAC;AAEzD,MAAM,WAAW,mBAAmB;IAClC,mDAAmD;IACnD,KAAK,EAAE,cAAc,EAAE,CAAC;IACxB,mDAAmD;IACnD,KAAK,EAAE,IAAI,EAAE,CAAC;IACd;;;;OAIG;IACH,SAAS,EAAE,CAAC,QAAQ,EAAE,cAAc,EAAE,EAAE,QAAQ,EAAE,IAAI,EAAE,KAAK,IAAI,CAAC;IAClE,uEAAuE;IACvE,aAAa,EAAE,CAAC,aAAa,EAAE,MAAM,EAAE,OAAO,EAAE,kBAAkB,KAAK,IAAI,CAAC;IAC5E,kEAAkE;IAClE,iBAAiB,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,eAAe,EAAE,YAAY,CAAC,EAAE,MAAM,KAAK,IAAI,CAAC;IAC1F,yDAAyD;IACzD,oBAAoB,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,IAAI,CAAC;IAC/C,sDAAsD;IACtD,aAAa,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,IAAI,CAAC;IACxC,uDAAuD;IACvD,cAAc,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,YAAY,KAAK,IAAI,CAAC;IAC9D,2DAA2D;IAC3D,aAAa,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,kBAAkB,KAAK,IAAI,CAAC;IAClE,gCAAgC;IAChC,gBAAgB,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,IAAI,CAAC;IAC3C,yDAAyD;IACzD,iBAAiB,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,kBAAkB,KAAK,IAAI,CAAC;IACtF,gEAAgE;IAChE,wBAAwB,EAAE,CAAC,aAAa,EAAE,MAAM,KAAK,MAAM,GAAG,SAAS,CAAC;IACxE,yDAAyD;IACzD,iBAAiB,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,kBAAkB,GAAG,SAAS,CAAC;IACtE,+EAA+E;IAC/E,cAAc,EAAE,MAAM,MAAM,CAAC;IAC7B,6EAA6E;IAC7E,YAAY,EAAE,CAAC,SAAS,EAAE,cAAc,KAAK,IAAI,CAAC;IAClD;;;;OAIG;IACH,aAAa,EAAE,MAAM,IAAI,CAAC;IAC1B;;;OAGG;IACH,WAAW,EAAE,CAAC,UAAU,CAAC,EAAE,MAAM,KAAK,IAAI,CAAC;IAC3C,0EAA0E;IAC1E,UAAU,EAAE,MAAM,OAAO,CAAC;CAC3B;AAuCD,KAAK,kBAAkB,GAAG,OAAO,CAC/B,IAAI,CAAC,eAAe,EAAE,OAAO,GAAG,WAAW,GAAG,MAAM,GAAG,aAAa,CAAC,CACtE,CAAC;AA8BF,wBAAgB,aAAa,CAC3B,YAAY,EAAE,cAAc,EAAE,EAC9B,YAAY,EAAE,IAAI,EAAE,GACnB,mBAAmB,CA6ZrB"}

package/dist/src/presentation/web/hooks/use-graph-state.js

@@ -1,5 +1,5 @@
 'use client';
-import { useState, useMemo, useCallback, useRef } from 'react';
+import { useState, useMemo, useCallback, useRef, useEffect } from 'react';
 import { deriveGraph, } from '../lib/derive-graph.js';
 import { layoutWithDagre, CANVAS_LAYOUT_DEFAULTS } from '../lib/layout-with-dagre.js';
 /** Parse initialNodes + initialEdges into domain Maps. */
@@ -71,6 +71,10 @@ export function useGraphState(initialNodes, initialEdges) {
     // This prevents a stale poll (that fetched data before the server action completed)
     // from wiping a repo that the client has already added.
     const protectedRepoIdsRef = useRef(new Set());
+    // Mutation guard: ref-counted counter. While > 0, reconcile is a no-op
+    // so stale poll data cannot overwrite optimistic state.
+    const mutationCountRef = useRef(0);
+    const mutationTimersRef = useRef(new Set());
     // Track 'deleting' features that the server no longer returns.
     // Retained for one extra reconcile cycle so the user sees the deleting state
     // before the node disappears from the canvas.
@@ -154,6 +158,11 @@ export function useGraphState(initialNodes, initialEdges) {
     }, [derived]);
     // --- Mutations ---
     const reconcile = useCallback((newNodes, newEdges) => {
+        // Skip reconciliation while an optimistic mutation is in-flight.
+        // Stale poll data fetched before the mutation completed would overwrite
+        // the optimistic state, causing flicker (nodes disappearing/reappearing).
+        if (mutationCountRef.current > 0)
+            return;
         const { featureMap: newFeatureMap, repoMap: newRepoMap } = parseMaps(newNodes, newEdges);
         // Precompute match keys so we don't scan newFeatureMap.values() per pending entry
         const realFeatureKeys = new Set([...newFeatureMap.values()].map((e) => `${e.data.name}\0${e.data.repositoryPath}`));
@@ -355,6 +364,28 @@ export function useGraphState(initialNodes, initialEdges) {
     const setCallbacks = useCallback((callbacks) => {
         callbacksRef.current = callbacks;
     }, []);
+    const beginMutation = useCallback(() => {
+        mutationCountRef.current++;
+    }, []);
+    const endMutation = useCallback((cooldownMs = 3_000) => {
+        // Delay decrement by one poll interval so the next fetch after the
+        // mutation returns post-mutation data, not a stale in-flight response.
+        const timer = setTimeout(() => {
+            mutationCountRef.current = Math.max(0, mutationCountRef.current - 1);
+            mutationTimersRef.current.delete(timer);
+        }, cooldownMs);
+        mutationTimersRef.current.add(timer);
+    }, []);
+    const isMutating = useCallback(() => mutationCountRef.current > 0, []);
+    // Cleanup cooldown timers on unmount
+    useEffect(() => {
+        const timers = mutationTimersRef.current;
+        return () => {
+            for (const timer of timers)
+                clearTimeout(timer);
+            timers.clear();
+        };
+    }, []);
     return {
         nodes,
         edges,
@@ -371,5 +402,8 @@ export function useGraphState(initialNodes, initialEdges) {
         getRepositoryData,
         getRepoMapSize,
         setCallbacks,
+        beginMutation,
+        endMutation,
+        isMutating,
     };
 }

package/dist/src/presentation/web/lib/derive-graph.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"derive-graph.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/lib/derive-graph.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,eAAe,CAAC;AAC1C,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAC5E,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AACxE,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAC9E,qDAAqD;AACrD,MAAM,WAAW,YAAY;IAC3B,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,eAAe,CAAC;IACtB,sGAAsG;IACtG,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB;AAED,wDAAwD;AACxD,MAAM,WAAW,SAAS;IACxB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,kBAAkB,CAAC;CAC1B;AAED,gFAAgF;AAChF,MAAM,WAAW,cAAc;IAC7B,4FAA4F;IAC5F,YAAY,CAAC,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,IAAI,CAAC;IACxC,qEAAqE;IACrE,cAAc,CAAC,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,IAAI,CAAC;IAC1C,8CAA8C;IAC9C,eAAe,CAAC,EAAE,CAAC,SAAS,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,OAAO,KAAK,IAAI,CAAC;IACjE,6EAA6E;IAC7E,eAAe,CAAC,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,IAAI,CAAC;IAC/C,8EAA8E;IAC9E,iBAAiB,CAAC,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,IAAI,CAAC;IACjD,iDAAiD;IACjD,kBAAkB,CAAC,EAAE,CAAC,YAAY,EAAE,MAAM,KAAK,IAAI,CAAC;IACpD,qDAAqD;IACrD,cAAc,CAAC,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;CAC9C;AAED;;;;;;;;GAQG;AACH,wBAAgB,WAAW,CACzB,UAAU,EAAE,GAAG,CAAC,MAAM,EAAE,YAAY,CAAC,EACrC,OAAO,EAAE,GAAG,CAAC,MAAM,EAAE,SAAS,CAAC,EAC/B,UAAU,EAAE,GAAG,CAAC,MAAM,EAAE,YAAY,CAAC,EACrC,SAAS,CAAC,EAAE,cAAc,GACzB;IAAE,KAAK,EAAE,cAAc,EAAE,CAAC;IAAC,KAAK,EAAE,IAAI,EAAE,CAAA;CAAE,CAqI5C"}
+{"version":3,"file":"derive-graph.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/lib/derive-graph.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,eAAe,CAAC;AAC1C,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAC5E,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AACxE,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAC9E,qDAAqD;AACrD,MAAM,WAAW,YAAY;IAC3B,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,eAAe,CAAC;IACtB,sGAAsG;IACtG,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB;AAED,wDAAwD;AACxD,MAAM,WAAW,SAAS;IACxB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,kBAAkB,CAAC;CAC1B;AAED,gFAAgF;AAChF,MAAM,WAAW,cAAc;IAC7B,4FAA4F;IAC5F,YAAY,CAAC,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,IAAI,CAAC;IACxC,qEAAqE;IACrE,cAAc,CAAC,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,IAAI,CAAC;IAC1C,8CAA8C;IAC9C,eAAe,CAAC,EAAE,CAAC,SAAS,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,OAAO,KAAK,IAAI,CAAC;IACjE,6EAA6E;IAC7E,eAAe,CAAC,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,IAAI,CAAC;IAC/C,8EAA8E;IAC9E,iBAAiB,CAAC,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,IAAI,CAAC;IACjD,iDAAiD;IACjD,kBAAkB,CAAC,EAAE,CAAC,YAAY,EAAE,MAAM,KAAK,IAAI,CAAC;IACpD,qDAAqD;IACrD,cAAc,CAAC,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;CAC9C;AAED;;;;;;;;GAQG;AACH,wBAAgB,WAAW,CACzB,UAAU,EAAE,GAAG,CAAC,MAAM,EAAE,YAAY,CAAC,EACrC,OAAO,EAAE,GAAG,CAAC,MAAM,EAAE,SAAS,CAAC,EAC/B,UAAU,EAAE,GAAG,CAAC,MAAM,EAAE,YAAY,CAAC,EACrC,SAAS,CAAC,EAAE,cAAc,GACzB;IAAE,KAAK,EAAE,cAAc,EAAE,CAAC;IAAC,KAAK,EAAE,IAAI,EAAE,CAAA;CAAE,CAwI5C"}

package/dist/src/presentation/web/lib/derive-graph.js

@@ -17,11 +17,13 @@
 export function deriveGraph(featureMap, repoMap, pendingMap, callbacks) {
     const nodes = [];
     const edges = [];
+    // Normalize path separators so Windows backslash paths match forward-slash paths
+    const normalizePath = (p) => p.replace(/\\/g, '/');
     // Build a lookup: repositoryPath → repoNodeId
     const repoByPath = new Map();
     for (const [nodeId, entry] of repoMap) {
         if (entry.data.repositoryPath) {
-            repoByPath.set(entry.data.repositoryPath, nodeId);
+            repoByPath.set(normalizePath(entry.data.repositoryPath), nodeId);
         }
     }
     // Virtual repo nodes created for orphaned features (not in repoMap)
@@ -119,7 +121,7 @@ export function deriveGraph(featureMap, repoMap, pendingMap, callbacks) {
         }
         else {
             // Repo→feature edge (matched by repositoryPath)
-            const repositoryPath = entry.data.repositoryPath;
+            const repositoryPath = normalizePath(entry.data.repositoryPath);
             const repoNodeId = repoByPath.get(repositoryPath) ?? getOrCreateVirtualRepo(repositoryPath);
             edges.push({
                 id: `edge-${repoNodeId}-${nodeId}`,