@shepai/cli 1.122.0 → 1.122.1-pr361.35a9abd

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/feature-agent-graph.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"feature-agent-graph.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/feature-agent-graph.ts"],"names":[],"mappings":"AAAA,OAAO,EAA0B,KAAK,mBAAmB,EAAE,MAAM,sBAAsB,CAAC;AACxF,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AAOpG,OAAO,EAAmB,KAAK,aAAa,EAAE,MAAM,6BAA6B,CAAC;AASlF,OAAO,EAAE,sBAAsB,EAAE,KAAK,iBAAiB,EAAE,MAAM,YAAY,CAAC;AAE5E;;;GAGG;AACH,MAAM,WAAW,qBAAqB;IACpC,QAAQ,EAAE,cAAc,CAAC;IACzB,aAAa,CAAC,EAAE,IAAI,CAAC,aAAa,EAAE,UAAU,CAAC,CAAC;CACjD;AA0HD;;;;;;;;;;;;GAYG;AACH,wBAAgB,uBAAuB,CACrC,cAAc,EAAE,qBAAqB,GAAG,cAAc,EACtD,YAAY,CAAC,EAAE,mBAAmB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;qBAwEnC"}
+{"version":3,"file":"feature-agent-graph.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/feature-agent-graph.ts"],"names":[],"mappings":"AAAA,OAAO,EAA0B,KAAK,mBAAmB,EAAE,MAAM,sBAAsB,CAAC;AACxF,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AAOpG,OAAO,EAAmB,KAAK,aAAa,EAAE,MAAM,6BAA6B,CAAC;AAclF,OAAO,EAAE,sBAAsB,EAAE,KAAK,iBAAiB,EAAE,MAAM,YAAY,CAAC;AAE5E;;;GAGG;AACH,MAAM,WAAW,qBAAqB;IACpC,QAAQ,EAAE,cAAc,CAAC;IACzB,aAAa,CAAC,EAAE,IAAI,CAAC,aAAa,EAAE,UAAU,CAAC,CAAC;CACjD;AAwID;;;;;;;;;;;;GAYG;AACH,wBAAgB,uBAAuB,CACrC,cAAc,EAAE,qBAAqB,GAAG,cAAc,EACtD,YAAY,CAAC,EAAE,mBAAmB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;qBAgFnC"}

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/feature-agent-graph.js

@@ -11,7 +11,7 @@ import { createRepairNode } from './nodes/repair.node.js';
 import { validateSpecAnalyze, validateSpecRequirements } from './nodes/schemas/spec.schema.js';
 import { validateResearch } from './nodes/schemas/research.schema.js';
 import { validatePlan, validateTasks } from './nodes/schemas/plan.schema.js';
-import { readSpecFile, safeYamlLoad, createNodeLogger } from './nodes/node-helpers.js';
+import { readSpecFile, safeYamlLoad, createNodeLogger, getCompletedPhases, } from './nodes/node-helpers.js';
 // Re-export state types for consumers
 export { FeatureAgentAnnotation } from './state.js';
 /**
@@ -54,6 +54,18 @@ function createPlanTasksValidator() {
     const log = createNodeLogger('validate:plan+tasks');
     return async (state) => {
         log.activate();
+        // Skip validation when the successor phase (implement) is already completed.
+        // This proves validation already passed and the pipeline progressed beyond it.
+        // Same logic as createValidateNode's successorPhase guard.
+        if (getCompletedPhases(state.specDir).includes('implement')) {
+            log.info("Successor phase 'implement' already completed, skipping validation");
+            return {
+                lastValidationTarget: 'plan.yaml+tasks.yaml',
+                lastValidationErrors: [],
+                validationRetries: 0,
+                messages: ["[validate:plan+tasks] SKIP (successor 'implement' already completed)"],
+            };
+        }
         log.info('Validating plan.yaml and tasks.yaml');
         const allErrors = [];
         // Validate plan.yaml
@@ -145,9 +157,14 @@ export function createFeatureAgentGraph(depsOrExecutor, checkpointer) {
         .addNode('plan', createPlanNode(executor))
         .addNode('implement', createImplementNode(executor))
         // --- Validate nodes ---
-        .addNode('validate_spec_analyze', createValidateNode('spec.yaml', validateSpecAnalyze))
-        .addNode('validate_spec_requirements', createValidateNode('spec.yaml', validateSpecRequirements))
-        .addNode('validate_research', createValidateNode('research.yaml', validateResearch))
+        // Each validate node receives its SUCCESSOR phase name (the phase that runs
+        // after validation passes). On resume, validation is skipped only when the
+        // successor is already completed — proving this validation already passed.
+        // Using the successor (not the producer) avoids skipping validation when the
+        // producer ran successfully but validation itself failed.
+        .addNode('validate_spec_analyze', createValidateNode('spec.yaml', validateSpecAnalyze, 'requirements'))
+        .addNode('validate_spec_requirements', createValidateNode('spec.yaml', validateSpecRequirements, 'research'))
+        .addNode('validate_research', createValidateNode('research.yaml', validateResearch, 'plan'))
         .addNode('validate_plan_tasks', createPlanTasksValidator())
         // --- Repair nodes ---
         .addNode('repair_spec_analyze', createRepairNode('spec.yaml', executor))

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/validate.node.d.ts

@@ -6,7 +6,13 @@ type SchemaValidator = (data: unknown) => ValidationResult;
  *
  * On success: resets validationRetries to 0, clears errors.
  * On failure: increments validationRetries, sets errors for repair node.
+ *
+ * @param successorPhase - The phase that runs AFTER this validation passes.
+ *   If provided, skip validation when the successor is already completed — this
+ *   proves the validation already passed in a prior run and the pipeline progressed
+ *   beyond it. Checking the successor (not the producer) avoids skipping validation
+ *   when the producer ran but validation itself failed.
  */
-export declare function createValidateNode(filename: string, schema: SchemaValidator): (state: FeatureAgentState) => Promise<Partial<FeatureAgentState>>;
+export declare function createValidateNode(filename: string, schema: SchemaValidator, successorPhase?: string): (state: FeatureAgentState) => Promise<Partial<FeatureAgentState>>;
 export {};
 //# sourceMappingURL=validate.node.d.ts.map

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/validate.node.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"validate.node.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/validate.node.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AACrD,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAGhE,KAAK,eAAe,GAAG,CAAC,IAAI,EAAE,OAAO,KAAK,gBAAgB,CAAC;AAE3D;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAChC,QAAQ,EAAE,MAAM,EAChB,MAAM,EAAE,eAAe,GACtB,CAAC,KAAK,EAAE,iBAAiB,KAAK,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CAuDnE"}
+{"version":3,"file":"validate.node.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/validate.node.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AACrD,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAQhE,KAAK,eAAe,GAAG,CAAC,IAAI,EAAE,OAAO,KAAK,gBAAgB,CAAC;AAE3D;;;;;;;;;;;GAWG;AACH,wBAAgB,kBAAkB,CAChC,QAAQ,EAAE,MAAM,EAChB,MAAM,EAAE,eAAe,EACvB,cAAc,CAAC,EAAE,MAAM,GACtB,CAAC,KAAK,EAAE,iBAAiB,KAAK,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CAyEnE"}

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/validate.node.js

@@ -1,14 +1,36 @@
-import { readSpecFile, safeYamlLoad, createNodeLogger } from './node-helpers.js';
+import { readSpecFile, safeYamlLoad, createNodeLogger, getCompletedPhases, } from './node-helpers.js';
 /**
  * Factory that creates a validate node for a specific YAML file.
  *
  * On success: resets validationRetries to 0, clears errors.
  * On failure: increments validationRetries, sets errors for repair node.
+ *
+ * @param successorPhase - The phase that runs AFTER this validation passes.
+ *   If provided, skip validation when the successor is already completed — this
+ *   proves the validation already passed in a prior run and the pipeline progressed
+ *   beyond it. Checking the successor (not the producer) avoids skipping validation
+ *   when the producer ran but validation itself failed.
  */
-export function createValidateNode(filename, schema) {
+export function createValidateNode(filename, schema, successorPhase) {
     const log = createNodeLogger(`validate:${filename}`);
     return async (state) => {
         log.activate();
+        // Skip validation when the successor phase is already completed.
+        // This means: the validation already passed in a prior run AND the next phase
+        // ran successfully. On resume, LangGraph replays nodes from the last checkpoint.
+        // The producer node skips (via getCompletedPhases), but if we only checked the
+        // producer we'd also skip validation when it previously FAILED (since
+        // markPhaseComplete runs before validation). Checking the successor is safe:
+        // if the successor completed, this validation must have passed to get there.
+        if (successorPhase && getCompletedPhases(state.specDir).includes(successorPhase)) {
+            log.info(`Successor phase '${successorPhase}' already completed, skipping validation`);
+            return {
+                lastValidationTarget: filename,
+                lastValidationErrors: [],
+                validationRetries: 0,
+                messages: [`[validate:${filename}] SKIP (successor '${successorPhase}' already completed)`],
+            };
+        }
         log.info(`Validating ${filename}`);
         const content = readSpecFile(state.specDir, filename);
         if (!content) {

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":"AAuCA,OAAO,KAAK,EAAE,UAAU,EAAE,aAAa,EAAE,MAAM,eAAe,CAAC;AAI/D,MAAM,WAAW,wBAAwB;IACvC,IAAI,EAAE,UAAU,CAAC;IACjB,8FAA8F;IAC9F,MAAM,CAAC,EAAE,aAAa,CAAC;CACxB;AAED,wBAAgB,mBAAmB,CAAC,EAAE,IAAI,EAAE,WAAW,EAAE,MAAM,EAAE,EAAE,wBAAwB,2CA+jB1F"}
+{"version":3,"file":"feature-drawer-client.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/feature-drawer-client.tsx"],"names":[],"mappings":"AAuCA,OAAO,KAAK,EAAE,UAAU,EAAE,aAAa,EAAE,MAAM,eAAe,CAAC;AAI/D,MAAM,WAAW,wBAAwB;IACvC,IAAI,EAAE,UAAU,CAAC;IACjB,8FAA8F;IAC9F,MAAM,CAAC,EAAE,aAAa,CAAC;CACxB;AAED,wBAAgB,mBAAmB,CAAC,EAAE,IAAI,EAAE,WAAW,EAAE,MAAM,EAAE,EAAE,wBAAwB,2CAqkB1F"}

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

@@ -194,6 +194,10 @@ export function FeatureDrawerClient({ view: initialView, urlTab }) {
             if (result.iterationWarning) {
                 toast.warning(`Iteration ${result.iteration} — consider approving or adjusting feedback to avoid excessive iterations`);
             }
+            // Optimistically update canvas node before SSE arrives (~500ms delay)
+            window.dispatchEvent(new CustomEvent('shep:feature-approved', {
+                detail: { featureId: featureNode.featureId },
+            }));
             onClose();
             onDone?.();
         }

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,2CA2UzF"}
+{"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,2CAyXzF"}

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

@@ -8,7 +8,7 @@ import { useSidebarFeaturesContext, mapNodeStateToSidebarStatus, } from '../../.
 import { useSelectedFeatureId } from '../../../hooks/use-selected-feature-id.js';
 import { useSoundAction } from '../../../hooks/use-sound-action.js';
 import { useDrawerCloseGuard } from '../../../hooks/drawer-close-guard.js';
-import { useViewportPersistence } from '../../../hooks/use-viewport-persistence.js';
+import { useViewportPersistence, DEFAULT_VIEWPORT } from '../../../hooks/use-viewport-persistence.js';
 import { ControlCenterEmptyState } from './control-center-empty-state.js';
 import { useControlCenterState } from './use-control-center-state.js';
 const AUTO_FOCUS_OPTIONS = {
@@ -23,7 +23,7 @@ export function ControlCenterInner({ initialNodes, initialEdges }) {
     const selectedFeatureId = useSelectedFeatureId();
     const clickSound = useSoundAction('click');
     const { guardedNavigate } = useDrawerCloseGuard();
-    const { fitView } = useReactFlow();
+    const { fitView, setViewport } = useReactFlow();
     const drawerTimerRef = useRef(null);
     const { defaultViewport, onMoveEnd: handleViewportChange, resetViewport, } = useViewportPersistence();
     const { nodes, edges, onNodesChange, handleConnect, handleAddRepository, handleDeleteFeature, handleRetryFeature, handleStartFeature, handleStopFeature, handleDeleteRepository, createFeatureNode, setCallbacks, } = useControlCenterState(initialNodes, initialEdges);
@@ -119,20 +119,27 @@ export function ControlCenterInner({ initialNodes, initialEdges }) {
             guardedNavigate(() => router.push('/'));
         }
     }, [router, pathname, guardedNavigate]);
+    // ── Deferred fitView mechanism ──────────────────────────────────────
+    // Instead of calling fitView from event handlers (which races with the
+    // showCanvas latch and React render cycles), we set a ref flag and let
+    // a useEffect consume it once showCanvas is true and nodes are rendered.
+    const pendingFitViewRef = useRef(false);
+    const pendingDrawerRepoRef = useRef(null);
     // Listen for global "add repository" events from the top bar button
     useEffect(() => {
         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();
+            // Immediately snap the in-memory viewport so the user never sees
+            // the old far-away position (defaultViewport only applies on mount).
+            setViewport(DEFAULT_VIEWPORT);
+            pendingFitViewRef.current = true;
             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);
+                pendingDrawerRepoRef.current = repoPath;
             }
         };
         window.addEventListener('shep:add-repository', handler);
@@ -142,7 +149,7 @@ export function ControlCenterInner({ initialNodes, initialEdges }) {
                 clearTimeout(drawerTimerRef.current);
             }
         };
-    }, [handleAddRepository, fitView, guardedNavigate, router]);
+    }, [handleAddRepository, resetViewport, setViewport]);
     // Listen for create events from the create drawer (with real feature ID from server)
     useEffect(() => {
         const handler = (e) => {
@@ -208,6 +215,21 @@ 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 — request a deferred fitView
+            resetViewport();
+            setViewport(DEFAULT_VIEWPORT);
+            pendingFitViewRef.current = true;
+        }
+        hadNodesRef.current = hasNodes;
+    }, [nodes.length, resetViewport, setViewport]);
     const handleMoveEnd = useCallback((_event, viewport) => {
         handleViewportChange(viewport);
     }, [handleViewportChange]);
@@ -238,6 +260,26 @@ export function ControlCenterInner({ initialNodes, initialEdges }) {
                 clearTimeout(latchTimerRef.current);
         };
     }, [hasRepositories, showCanvas]);
+    // ── Consume the pending fitView flag ──────────────────────────────────
+    // This effect runs after React renders with showCanvas=true and nodes
+    // available, guaranteeing ReactFlow has the nodes in the DOM before
+    // fitView calculates bounds.  Using requestAnimationFrame ensures the
+    // browser has flushed layout so ReactFlow can measure node dimensions.
+    useEffect(() => {
+        if (!pendingFitViewRef.current || !showCanvas || nodes.length === 0)
+            return;
+        pendingFitViewRef.current = false;
+        const drawerRepo = pendingDrawerRepoRef.current;
+        pendingDrawerRepoRef.current = null;
+        requestAnimationFrame(() => {
+            fitView(AUTO_FOCUS_OPTIONS);
+            if (drawerRepo) {
+                drawerTimerRef.current = setTimeout(() => {
+                    guardedNavigate(() => router.push(`/create?repo=${encodeURIComponent(drawerRepo)}`));
+                }, AUTO_FOCUS_DRAWER_DELAY_MS);
+            }
+        });
+    }, [showCanvas, nodes.length, fitView, guardedNavigate, router]);
     // Pulse the "+" button when there's a single repo with no features and the
     // create-feature drawer is not open — draws attention to the next action.
     const isCreateDrawerOpen = pathname.startsWith('/create');

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;AAgBjC,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,EAAE,aAAa,CAAC,EAAE,OAAO,KAAK,IAAI,CAAC;IAC7F,kBAAkB,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IAChD,kBAAkB,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IAChD,iBAAiB,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IAC/C,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,CAygBpB"}
+{"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;AAgBjC,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,EAAE,aAAa,CAAC,EAAE,OAAO,KAAK,IAAI,CAAC;IAC7F,kBAAkB,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IAChD,kBAAkB,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IAChD,iBAAiB,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IAC/C,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,CA6gBpB"}

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

@@ -115,15 +115,19 @@ export function useControlCenterState(initialNodes, initialEdges) {
             }
         }
     }, [events, updateFeature]);
-    // Listen for optimistic approval events from the drawer (fires before SSE arrives)
+    // Listen for optimistic approval/rejection events from the drawer (fires before SSE arrives).
+    // Uses beginMutation + endMutation to prevent stale poll/reconcile data from overwriting
+    // the optimistic 'running' state during the transition window (avoids brief "Something went wrong" flash).
     useEffect(() => {
         const handler = (e) => {
             const { featureId } = e.detail;
+            beginMutation();
             updateFeature(`feat-${featureId}`, { state: 'running' });
+            endMutation();
         };
         window.addEventListener('shep:feature-approved', handler);
         return () => window.removeEventListener('shep:feature-approved', handler);
-    }, [updateFeature]);
+    }, [updateFeature, beginMutation, endMutation]);
     // Separate effect: fetch metadata (name + description) when SSE reports it changed
     const metadataFetchedRef = useRef(new Set());
     const processedMetadataCountRef = useRef(0);

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

@@ -5,6 +5,8 @@ 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,SAAS,EAAE,CAAC,QAAQ,EAAE,QAAQ,KAAK,IAAI,CAAC;IACxC,aAAa,EAAE,MAAM,QAAQ,CAAC;CAC/B;AA6BD,wBAAgB,sBAAsB,IAAI,4BAA4B,CAmCrE"}
+{"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"}

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

@@ -1,5 +1,5 @@
 'use client';
-import { useCallback, useEffect, useRef } from 'react';
+import { useCallback, useEffect, useRef, useState } 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,16 +20,20 @@ function readViewport() {
     try {
         const raw = localStorage.getItem(STORAGE_KEY);
         if (raw == null)
-            return DEFAULT_VIEWPORT;
+            return { viewport: DEFAULT_VIEWPORT, fromStorage: false };
         const parsed = JSON.parse(raw);
-        return isValidViewport(parsed) ? parsed : DEFAULT_VIEWPORT;
+        if (isValidViewport(parsed))
+            return { viewport: parsed, fromStorage: true };
+        return { viewport: DEFAULT_VIEWPORT, fromStorage: false };
     }
     catch {
-        return DEFAULT_VIEWPORT;
+        return { viewport: DEFAULT_VIEWPORT, fromStorage: false };
     }
 }
 export function useViewportPersistence() {
-    const defaultViewport = useRef(readViewport()).current;
+    const stored = useRef(readViewport()).current;
+    const [defaultViewport, setDefaultViewport] = useState(stored.viewport);
+    const hasSavedViewport = stored.fromStorage;
     const timerRef = useRef(null);
     useEffect(() => {
         return () => {
@@ -52,13 +56,22 @@ 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, onMoveEnd, resetViewport };
+    return { defaultViewport, hasSavedViewport, 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,CAqL/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,CA8L/B"}

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

@@ -78,6 +78,10 @@ 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;
@@ -85,9 +89,14 @@ 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 ? rootPrimaryAxis : 0,
-            cy: isHorizontal ? 0 : rootPrimaryAxis,
+            cx: isHorizontal ? primaryAxisCenter : 0,
+            cy: isHorizontal ? 0 : primaryAxisCenter,
             w: size.width,
             h: size.height,
         });