@shepai/cli 1.114.0-pr361.c9914c5 → 1.115.0

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;AAuB5E,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,2CA4VzF"}
+{"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;AAuB5E,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,2CAqUzF"}

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

@@ -124,22 +124,16 @@ export function ControlCenterInner({ initialNodes, initialEdges }) {
         const handler = (e) => {
             const path = e.detail.path;
             const { wasEmpty, repoPath } = handleAddRepository(path);
-            // Always center the viewport on the new repo, ignoring any stored
-            // pan/zoom position.  This guarantees the repo is visible after adding
-            // — even if the user previously panned far away or deleted everything.
-            resetViewport();
-            // Use a short delay to allow React to flush the state update and
-            // the showCanvas latch to transition, ensuring ReactFlow has the
-            // new nodes rendered before fitView calculates bounds.
-            requestAnimationFrame(() => {
-                fitView(AUTO_FOCUS_OPTIONS);
-                if (wasEmpty) {
+            if (wasEmpty) {
+                // Wait for next render so the repo node exists in the DOM, then auto-focus
+                setTimeout(() => {
+                    fitView(AUTO_FOCUS_OPTIONS);
                     // Open the create-feature drawer after the fitView animation completes
                     drawerTimerRef.current = setTimeout(() => {
                         guardedNavigate(() => router.push(`/create?repo=${encodeURIComponent(repoPath)}`));
                     }, AUTO_FOCUS_DRAWER_DELAY_MS);
-                }
-            });
+                }, 0);
+            }
         };
         window.addEventListener('shep:add-repository', handler);
         return () => {
@@ -148,7 +142,7 @@ export function ControlCenterInner({ initialNodes, initialEdges }) {
                 clearTimeout(drawerTimerRef.current);
             }
         };
-    }, [handleAddRepository, fitView, guardedNavigate, router, resetViewport]);
+    }, [handleAddRepository, fitView, guardedNavigate, router]);
     // Listen for create events from the create drawer (with real feature ID from server)
     useEffect(() => {
         const handler = (e) => {
@@ -210,21 +204,6 @@ export function ControlCenterInner({ initialNodes, initialEdges }) {
         handleRepositoryClick,
         handleDeleteRepository,
     ]);
-    // Auto-center whenever the canvas transitions from empty to having nodes.
-    // This covers both initial page load AND re-adding repos after deleting
-    // everything — the view always centers so the repo is visible, ignoring
-    // any previously stored pan/zoom position.
-    const hadNodesRef = useRef(nodes.length > 0);
-    useEffect(() => {
-        const hasNodes = nodes.length > 0;
-        if (hasNodes && !hadNodesRef.current) {
-            // Went from empty → has nodes — center the view
-            resetViewport();
-            // Yield to React so nodes are rendered in the DOM before fitView calculates bounds.
-            setTimeout(() => fitView(AUTO_FOCUS_OPTIONS), 0);
-        }
-        hadNodesRef.current = hasNodes;
-    }, [nodes.length, fitView, resetViewport]);
     const handleMoveEnd = useCallback((_event, viewport) => {
         handleViewportChange(viewport);
     }, [handleViewportChange]);

package/dist/src/presentation/web/hooks/use-viewport-persistence.d.ts

@@ -5,8 +5,6 @@ export declare const DEBOUNCE_MS = 500;
 export declare const DEFAULT_VIEWPORT: Viewport;
 export interface UseViewportPersistenceResult {
     defaultViewport: Viewport;
-    /** True when a valid viewport was restored from localStorage. */
-    hasSavedViewport: boolean;
     onMoveEnd: (viewport: Viewport) => void;
     resetViewport: () => Viewport;
 }

package/dist/src/presentation/web/hooks/use-viewport-persistence.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-viewport-persistence.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-viewport-persistence.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAC;AAE9C,YAAY,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAC;AAE9C,eAAO,MAAM,WAAW,yBAAyB,CAAC;AAClD,eAAO,MAAM,WAAW,MAAM,CAAC;AAE/B,eAAO,MAAM,gBAAgB,EAAE,QAAuC,CAAC;AAEvE,MAAM,WAAW,4BAA4B;IAC3C,eAAe,EAAE,QAAQ,CAAC;IAC1B,iEAAiE;IACjE,gBAAgB,EAAE,OAAO,CAAC;IAC1B,SAAS,EAAE,CAAC,QAAQ,EAAE,QAAQ,KAAK,IAAI,CAAC;IACxC,aAAa,EAAE,MAAM,QAAQ,CAAC;CAC/B;AA8BD,wBAAgB,sBAAsB,IAAI,4BAA4B,CA8CrE"}
+{"version":3,"file":"use-viewport-persistence.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-viewport-persistence.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAC;AAE9C,YAAY,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAC;AAE9C,eAAO,MAAM,WAAW,yBAAyB,CAAC;AAClD,eAAO,MAAM,WAAW,MAAM,CAAC;AAE/B,eAAO,MAAM,gBAAgB,EAAE,QAAuC,CAAC;AAEvE,MAAM,WAAW,4BAA4B;IAC3C,eAAe,EAAE,QAAQ,CAAC;IAC1B,SAAS,EAAE,CAAC,QAAQ,EAAE,QAAQ,KAAK,IAAI,CAAC;IACxC,aAAa,EAAE,MAAM,QAAQ,CAAC;CAC/B;AA6BD,wBAAgB,sBAAsB,IAAI,4BAA4B,CAmCrE"}

package/dist/src/presentation/web/hooks/use-viewport-persistence.js

@@ -1,5 +1,5 @@
 'use client';
-import { useCallback, useEffect, useRef, useState } from 'react';
+import { useCallback, useEffect, useRef } from 'react';
 export const STORAGE_KEY = 'shep-canvas-viewport';
 export const DEBOUNCE_MS = 500;
 export const DEFAULT_VIEWPORT = { x: 30, y: 30, zoom: 0.85 };
@@ -20,20 +20,16 @@ function readViewport() {
     try {
         const raw = localStorage.getItem(STORAGE_KEY);
         if (raw == null)
-            return { viewport: DEFAULT_VIEWPORT, fromStorage: false };
+            return DEFAULT_VIEWPORT;
         const parsed = JSON.parse(raw);
-        if (isValidViewport(parsed))
-            return { viewport: parsed, fromStorage: true };
-        return { viewport: DEFAULT_VIEWPORT, fromStorage: false };
+        return isValidViewport(parsed) ? parsed : DEFAULT_VIEWPORT;
     }
     catch {
-        return { viewport: DEFAULT_VIEWPORT, fromStorage: false };
+        return DEFAULT_VIEWPORT;
     }
 }
 export function useViewportPersistence() {
-    const stored = useRef(readViewport()).current;
-    const [defaultViewport, setDefaultViewport] = useState(stored.viewport);
-    const hasSavedViewport = stored.fromStorage;
+    const defaultViewport = useRef(readViewport()).current;
     const timerRef = useRef(null);
     useEffect(() => {
         return () => {
@@ -56,22 +52,13 @@ export function useViewportPersistence() {
         }, DEBOUNCE_MS);
     }, []);
     const resetViewport = useCallback(() => {
-        // Cancel any pending debounced save so it cannot overwrite the reset
-        if (timerRef.current != null) {
-            clearTimeout(timerRef.current);
-            timerRef.current = null;
-        }
         try {
             localStorage.removeItem(STORAGE_KEY);
         }
         catch {
             // Silently ignore localStorage errors
         }
-        // Update the defaultViewport so that if ReactFlow re-mounts (e.g. after
-        // showCanvas toggles), it initializes at the default position instead of
-        // the stale saved viewport.
-        setDefaultViewport(DEFAULT_VIEWPORT);
         return DEFAULT_VIEWPORT;
     }, []);
-    return { defaultViewport, hasSavedViewport, onMoveEnd, resetViewport };
+    return { defaultViewport, onMoveEnd, resetViewport };
 }

package/dist/src/presentation/web/lib/layout-with-dagre.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"layout-with-dagre.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/lib/layout-with-dagre.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,IAAI,EAAE,IAAI,EAAE,MAAM,eAAe,CAAC;AAEhD,MAAM,MAAM,eAAe,GAAG,IAAI,GAAG,IAAI,GAAG,IAAI,GAAG,IAAI,CAAC;AAExD,MAAM,WAAW,aAAa;IAC5B,iEAAiE;IACjE,SAAS,CAAC,EAAE,eAAe,CAAC;IAC5B,0DAA0D;IAC1D,QAAQ,CAAC,EAAE;QAAE,KAAK,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,CAAC;IAC7C,qDAAqD;IACrD,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,wEAAwE;IACxE,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB;AAED,+DAA+D;AAC/D,eAAO,MAAM,sBAAsB,EAAE,aAIpC,CAAC;AAkCF;;;;;;GAMG;AACH,wBAAgB,eAAe,CAAC,CAAC,SAAS,IAAI,EAC5C,KAAK,EAAE,CAAC,EAAE,EACV,KAAK,EAAE,IAAI,EAAE,EACb,IAAI,GAAE,aAAkB,GACvB;IAAE,KAAK,EAAE,CAAC,EAAE,CAAC;IAAC,KAAK,EAAE,IAAI,EAAE,CAAA;CAAE,CA8L/B"}
+{"version":3,"file":"layout-with-dagre.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/lib/layout-with-dagre.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,IAAI,EAAE,IAAI,EAAE,MAAM,eAAe,CAAC;AAEhD,MAAM,MAAM,eAAe,GAAG,IAAI,GAAG,IAAI,GAAG,IAAI,GAAG,IAAI,CAAC;AAExD,MAAM,WAAW,aAAa;IAC5B,iEAAiE;IACjE,SAAS,CAAC,EAAE,eAAe,CAAC;IAC5B,0DAA0D;IAC1D,QAAQ,CAAC,EAAE;QAAE,KAAK,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,CAAC;IAC7C,qDAAqD;IACrD,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,wEAAwE;IACxE,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB;AAED,+DAA+D;AAC/D,eAAO,MAAM,sBAAsB,EAAE,aAIpC,CAAC;AAkCF;;;;;;GAMG;AACH,wBAAgB,eAAe,CAAC,CAAC,SAAS,IAAI,EAC5C,KAAK,EAAE,CAAC,EAAE,EACV,KAAK,EAAE,IAAI,EAAE,EACb,IAAI,GAAE,aAAkB,GACvB;IAAE,KAAK,EAAE,CAAC,EAAE,CAAC;IAAC,KAAK,EAAE,IAAI,EAAE,CAAA;CAAE,CAqL/B"}

package/dist/src/presentation/web/lib/layout-with-dagre.js

@@ -78,10 +78,6 @@ export function layoutWithDagre(nodes, edges, opts = {}) {
     // Add disconnected nodes to centerMap so they participate in root positioning.
     // Use the same primary-axis position as connected roots (rank 0) so they align
     // visually with repos that have features attached.
-    // When there are NO connected nodes (e.g. repos with no features), use half the
-    // node's own dimension so the leading edge sits at the canvas origin — preventing
-    // nodes from rendering off-screen with the default viewport.
-    const hasCenterMap = centerMap.size > 0;
     let rootPrimaryAxis = 0;
     for (const c of centerMap.values()) {
         rootPrimaryAxis = isHorizontal ? c.cx : c.cy;
@@ -89,14 +85,9 @@ export function layoutWithDagre(nodes, edges, opts = {}) {
     }
     for (const node of disconnectedNodes) {
         const size = getNodeSize(node, nodeSize);
-        const primaryAxisCenter = hasCenterMap
-            ? rootPrimaryAxis
-            : isHorizontal
-                ? size.width / 2
-                : size.height / 2;
         centerMap.set(node.id, {
-            cx: isHorizontal ? primaryAxisCenter : 0,
-            cy: isHorizontal ? 0 : primaryAxisCenter,
+            cx: isHorizontal ? rootPrimaryAxis : 0,
+            cy: isHorizontal ? 0 : rootPrimaryAxis,
             w: size.width,
             h: size.height,
         });