@shepai/cli 1.72.9-pr220.ce0911d → 1.72.9-pr220.f17b60a

package/dist/src/presentation/web/app/build-feature-node-data.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"build-feature-node-data.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/app/build-feature-node-data.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,sCAAsC,CAAC;AAK9E,OAAO,KAAK,EAAE,eAAe,EAAyB,MAAM,kCAAkC,CAAC;AAsB/F;;;GAGG;AACH,wBAAgB,oBAAoB,CAAC,OAAO,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,IAAI,GAAG,eAAe,CA+B5F"}
+{"version":3,"file":"build-feature-node-data.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/app/build-feature-node-data.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,sCAAsC,CAAC;AAM9E,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AAExE;;;GAGG;AACH,wBAAgB,oBAAoB,CAAC,OAAO,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,IAAI,GAAG,eAAe,CAuB5F"}

package/dist/src/presentation/web/app/build-feature-node-data.js

@@ -1,38 +1,14 @@
-import { deriveNodeState, deriveProgress, } from '../components/common/feature-node/derive-feature-state.js';
-/** Map domain SdlcLifecycle enum values to UI FeatureLifecyclePhase (1:1). */
-const lifecycleMap = {
-    Requirements: 'requirements',
-    Research: 'research',
-    Implementation: 'implementation',
-    Review: 'review',
-    'Deploy & QA': 'deploy',
-    Maintain: 'maintain',
-};
-/** Map agent graph node names (from agent_run.result) to UI lifecycle phases. */
-const nodeToLifecyclePhase = {
-    analyze: 'requirements',
-    requirements: 'requirements',
-    research: 'research',
-    plan: 'implementation',
-    implement: 'implementation',
-    merge: 'review',
-};
+import { deriveNodeState, deriveProgress, deriveLifecycle, } from '../components/common/feature-node/derive-feature-state.js';
 /**
  * Builds a FeatureNodeData object from a Feature entity and optional AgentRun.
  * This is the single-feature version of the logic in `buildGraphNodes`.
  */
 export function buildFeatureNodeData(feature, run) {
-    const agentNode = run?.result?.startsWith('node:') ? run.result.slice(5) : undefined;
-    const lifecycle = run?.status === 'completed'
-        ? 'maintain'
-        : ((agentNode ? nodeToLifecyclePhase[agentNode] : undefined) ??
-            lifecycleMap[feature.lifecycle] ??
-            'requirements');
     return {
         name: feature.name,
         description: feature.description ?? feature.slug,
         featureId: feature.id,
-        lifecycle,
+        lifecycle: deriveLifecycle(feature, run),
         repositoryPath: feature.repositoryPath,
         branch: feature.branch,
         specPath: feature.specPath,

package/dist/src/presentation/web/app/build-graph-nodes.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"build-graph-nodes.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/app/build-graph-nodes.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,sCAAsC,CAAC;AAK1F,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAC5E,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,eAAe,CAAC;AAuB1C,MAAM,WAAW,cAAc;IAC7B,OAAO,EAAE,OAAO,CAAC;IACjB,GAAG,EAAE,QAAQ,GAAG,IAAI,CAAC;CACtB;AAED;;;;;;;GAOG;AACH,wBAAgB,eAAe,CAC7B,YAAY,EAAE,UAAU,EAAE,EAC1B,gBAAgB,EAAE,cAAc,EAAE,GACjC;IAAE,KAAK,EAAE,cAAc,EAAE,CAAC;IAAC,KAAK,EAAE,IAAI,EAAE,CAAA;CAAE,CAiE5C"}
+{"version":3,"file":"build-graph-nodes.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/app/build-graph-nodes.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,sCAAsC,CAAC;AAM1F,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAC5E,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,eAAe,CAAC;AAG1C,MAAM,WAAW,cAAc;IAC7B,OAAO,EAAE,OAAO,CAAC;IACjB,GAAG,EAAE,QAAQ,GAAG,IAAI,CAAC;CACtB;AAED;;;;;;;GAOG;AACH,wBAAgB,eAAe,CAC7B,YAAY,EAAE,UAAU,EAAE,EAC1B,gBAAgB,EAAE,cAAc,EAAE,GACjC;IAAE,KAAK,EAAE,cAAc,EAAE,CAAC;IAAC,KAAK,EAAE,IAAI,EAAE,CAAA;CAAE,CAiE5C"}

package/dist/src/presentation/web/app/build-graph-nodes.js

@@ -1,22 +1,4 @@
-import { deriveNodeState, deriveProgress, } from '../components/common/feature-node/derive-feature-state.js';
-/** Map domain SdlcLifecycle enum values to UI FeatureLifecyclePhase (1:1). */
-const lifecycleMap = {
-    Requirements: 'requirements',
-    Research: 'research',
-    Implementation: 'implementation',
-    Review: 'review',
-    'Deploy & QA': 'deploy',
-    Maintain: 'maintain',
-};
-/** Map agent graph node names (from agent_run.result) to UI lifecycle phases. */
-const nodeToLifecyclePhase = {
-    analyze: 'requirements',
-    requirements: 'requirements',
-    research: 'research',
-    plan: 'implementation',
-    implement: 'implementation',
-    merge: 'review',
-};
+import { deriveNodeState, deriveProgress, deriveLifecycle, } from '../components/common/feature-node/derive-feature-state.js';
 /**
  * Builds React Flow nodes and edges from persisted repositories and features.
  *
@@ -95,12 +77,6 @@ function appendFeatureNodes(repoFeatures, repoNodeId, allFeaturesWithRuns, nodes
         return aTime - bTime;
     });
     sorted.forEach(({ feature, run }) => {
-        const agentNode = run?.result?.startsWith('node:') ? run.result.slice(5) : undefined;
-        const lifecycle = run?.status === 'completed'
-            ? 'maintain'
-            : ((agentNode ? nodeToLifecyclePhase[agentNode] : undefined) ??
-                lifecycleMap[feature.lifecycle] ??
-                'requirements');
         // Resolve blockedBy display name from parent feature
         let blockedBy;
         if (feature.parentId && feature.lifecycle === 'Blocked') {
@@ -113,7 +89,7 @@ function appendFeatureNodes(repoFeatures, repoNodeId, allFeaturesWithRuns, nodes
             name: feature.name,
             description: feature.description ?? feature.slug,
             featureId: feature.id,
-            lifecycle,
+            lifecycle: deriveLifecycle(feature, run),
             repositoryPath: feature.repositoryPath,
             branch: feature.branch,
             specPath: feature.specPath,

package/dist/src/presentation/web/components/common/control-center-drawer/create-drawer-client.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"create-drawer-client.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/create-drawer-client.tsx"],"names":[],"mappings":"AAQA,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,iEAAiE,CAAC;AAC3G,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,qCAAqC,CAAC;AAE5E,MAAM,WAAW,uBAAuB;IACtC,cAAc,EAAE,MAAM,CAAC;IACvB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,QAAQ,EAAE,mBAAmB,EAAE,CAAC;IAChC,gBAAgB,CAAC,EAAE,gBAAgB,CAAC;CACrC;AAED,wBAAgB,kBAAkB,CAAC,EACjC,cAAc,EACd,eAAe,EACf,QAAQ,EACR,gBAAgB,GACjB,EAAE,uBAAuB,2CA6EzB"}
+{"version":3,"file":"create-drawer-client.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/create-drawer-client.tsx"],"names":[],"mappings":"AAQA,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,iEAAiE,CAAC;AAC3G,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,qCAAqC,CAAC;AAE5E,MAAM,WAAW,uBAAuB;IACtC,cAAc,EAAE,MAAM,CAAC;IACvB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,QAAQ,EAAE,mBAAmB,EAAE,CAAC;IAChC,gBAAgB,CAAC,EAAE,gBAAgB,CAAC;CACrC;AAED,wBAAgB,kBAAkB,CAAC,EACjC,cAAc,EACd,eAAe,EACf,QAAQ,EACR,gBAAgB,GACjB,EAAE,uBAAuB,2CA2EzB"}

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

@@ -38,7 +38,6 @@ export function CreateDrawerClient({ repositoryPath, initialParentId, features,
             .then((result) => {
             if (result.error) {
                 toast.error(result.error);
-                window.dispatchEvent(new CustomEvent('shep:feature-create-failed'));
                 return;
             }
             // Dispatch event with real feature ID so control center adds it to featureMap
@@ -54,7 +53,6 @@ export function CreateDrawerClient({ repositoryPath, initialParentId, features,
         })
             .catch(() => {
             toast.error('Failed to create feature');
-            window.dispatchEvent(new CustomEvent('shep:feature-create-failed'));
             setIsSubmitting(false);
         });
     }, [router]);

package/dist/src/presentation/web/components/common/control-center-drawer/drawer-view.d.ts

@@ -40,6 +40,8 @@ export type DrawerView =
     type: 'repository';
     data: RepositoryNodeData;
 };
+/** Derives the feature-specific view type from node lifecycle + state. */
+export declare function deriveFeatureViewType(node: FeatureNodeData): 'feature' | 'prd-review' | 'tech-review' | 'merge-review';
 /**
  * Derives the active DrawerView from control-center state.
  * Priority: feature-create > selected-node (prd/tech/merge/feature) > repository

package/dist/src/presentation/web/components/common/control-center-drawer/drawer-view.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"drawer-view.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/drawer-view.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AACxE,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAC9E,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,iEAAiE,CAAC;AAC3G,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,qCAAqC,CAAC;AAE5E;;;GAGG;AACH,MAAM,MAAM,UAAU;AACpB,gGAAgG;AAC9F;IAAE,IAAI,EAAE,SAAS,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AAC5C,mEAAmE;GACjE;IAAE,IAAI,EAAE,YAAY,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AAC/C,yEAAyE;GACvE;IAAE,IAAI,EAAE,aAAa,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AAChD,8DAA8D;GAC5D;IAAE,IAAI,EAAE,cAAc,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AACjD,4BAA4B;GAC1B;IACE,IAAI,EAAE,gBAAgB,CAAC;IACvB,cAAc,EAAE,MAAM,CAAC;IACvB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,QAAQ,EAAE,mBAAmB,EAAE,CAAC;IAChC,gBAAgB,CAAC,EAAE,gBAAgB,CAAC;CACrC;AACH,8BAA8B;GAC5B;IAAE,IAAI,EAAE,YAAY,CAAC;IAAC,IAAI,EAAE,kBAAkB,CAAA;CAAE,CAAC;AAErD;;;GAGG;AACH,wBAAgB,iBAAiB,CAAC,EAChC,YAAY,EACZ,kBAAkB,EAClB,qBAAqB,EACrB,sBAAsB,EACtB,gBAAgB,EAChB,QAAQ,EACR,gBAAgB,GACjB,EAAE;IACD,YAAY,EAAE,eAAe,GAAG,IAAI,CAAC;IACrC,kBAAkB,EAAE,OAAO,CAAC;IAC5B,qBAAqB,EAAE,MAAM,CAAC;IAC9B,sBAAsB,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3C,gBAAgB,EAAE,kBAAkB,GAAG,IAAI,CAAC;IAC5C,QAAQ,EAAE,mBAAmB,EAAE,CAAC;IAChC,gBAAgB,EAAE,gBAAgB,GAAG,SAAS,CAAC;CAChD,GAAG,UAAU,GAAG,IAAI,CAgCpB"}
+{"version":3,"file":"drawer-view.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/drawer-view.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AACxE,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAC9E,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,iEAAiE,CAAC;AAC3G,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,qCAAqC,CAAC;AAE5E;;;GAGG;AACH,MAAM,MAAM,UAAU;AACpB,gGAAgG;AAC9F;IAAE,IAAI,EAAE,SAAS,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AAC5C,mEAAmE;GACjE;IAAE,IAAI,EAAE,YAAY,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AAC/C,yEAAyE;GACvE;IAAE,IAAI,EAAE,aAAa,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AAChD,8DAA8D;GAC5D;IAAE,IAAI,EAAE,cAAc,CAAC;IAAC,IAAI,EAAE,eAAe,CAAA;CAAE;AACjD,4BAA4B;GAC1B;IACE,IAAI,EAAE,gBAAgB,CAAC;IACvB,cAAc,EAAE,MAAM,CAAC;IACvB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,QAAQ,EAAE,mBAAmB,EAAE,CAAC;IAChC,gBAAgB,CAAC,EAAE,gBAAgB,CAAC;CACrC;AACH,8BAA8B;GAC5B;IAAE,IAAI,EAAE,YAAY,CAAC;IAAC,IAAI,EAAE,kBAAkB,CAAA;CAAE,CAAC;AAErD,0EAA0E;AAC1E,wBAAgB,qBAAqB,CACnC,IAAI,EAAE,eAAe,GACpB,SAAS,GAAG,YAAY,GAAG,aAAa,GAAG,cAAc,CAM3D;AAED;;;GAGG;AACH,wBAAgB,iBAAiB,CAAC,EAChC,YAAY,EACZ,kBAAkB,EAClB,qBAAqB,EACrB,sBAAsB,EACtB,gBAAgB,EAChB,QAAQ,EACR,gBAAgB,GACjB,EAAE;IACD,YAAY,EAAE,eAAe,GAAG,IAAI,CAAC;IACrC,kBAAkB,EAAE,OAAO,CAAC;IAC5B,qBAAqB,EAAE,MAAM,CAAC;IAC9B,sBAAsB,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3C,gBAAgB,EAAE,kBAAkB,GAAG,IAAI,CAAC;IAC5C,QAAQ,EAAE,mBAAmB,EAAE,CAAC;IAChC,gBAAgB,EAAE,gBAAgB,GAAG,SAAS,CAAC;CAChD,GAAG,UAAU,GAAG,IAAI,CAoBpB"}

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

@@ -1,3 +1,13 @@
+/** Derives the feature-specific view type from node lifecycle + state. */
+export function deriveFeatureViewType(node) {
+    if (node.lifecycle === 'requirements' && node.state === 'action-required')
+        return 'prd-review';
+    if (node.lifecycle === 'implementation' && node.state === 'action-required')
+        return 'tech-review';
+    if (node.lifecycle === 'review' && (node.state === 'action-required' || node.state === 'error'))
+        return 'merge-review';
+    return 'feature';
+}
 /**
  * Derives the active DrawerView from control-center state.
  * Priority: feature-create > selected-node (prd/tech/merge/feature) > repository
@@ -13,17 +23,7 @@ export function computeDrawerView({ selectedNode, isCreateDrawerOpen, pendingRep
         };
     }
     if (selectedNode) {
-        if (selectedNode.lifecycle === 'requirements' && selectedNode.state === 'action-required') {
-            return { type: 'prd-review', node: selectedNode };
-        }
-        if (selectedNode.lifecycle === 'implementation' && selectedNode.state === 'action-required') {
-            return { type: 'tech-review', node: selectedNode };
-        }
-        if (selectedNode.lifecycle === 'review' &&
-            (selectedNode.state === 'action-required' || selectedNode.state === 'error')) {
-            return { type: 'merge-review', node: selectedNode };
-        }
-        return { type: 'feature', node: selectedNode };
+        return { type: deriveFeatureViewType(selectedNode), node: selectedNode };
     }
     if (selectedRepoNode) {
         return { type: 'repository', data: selectedRepoNode };

package/dist/src/presentation/web/components/common/control-center-drawer/feature-drawer-client.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"feature-drawer-client.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/feature-drawer-client.tsx"],"names":[],"mappings":"AA2DA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAEhD,MAAM,WAAW,wBAAwB;IACvC,IAAI,EAAE,UAAU,CAAC;CAClB;AAED,wBAAgB,mBAAmB,CAAC,EAAE,IAAI,EAAE,WAAW,EAAE,EAAE,wBAAwB,2CAonBlF"}
+{"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"}

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

@@ -4,7 +4,7 @@ import { useState, useCallback, useEffect, useRef } from 'react';
 import { useRouter, usePathname } from 'next/navigation';
 import { toast } from 'sonner';
 import { Loader2, Trash2, ExternalLink, GitCommitHorizontal, Play, Square } from 'lucide-react';
-import { PrStatus, NotificationEventType } from '../../../../../../packages/core/src/domain/generated/output.js';
+import { PrStatus } from '../../../../../../packages/core/src/domain/generated/output.js';
 import { approveFeature } from '../../../app/actions/approve-feature.js';
 import { rejectFeature } from '../../../app/actions/reject-feature.js';
 import { getFeatureArtifact } from '../../../app/actions/get-feature-artifact.js';
@@ -34,7 +34,8 @@ import { TechReviewTabs } from '../../common/tech-review-tabs/index.js';
 import { MergeReview } from '../../common/merge-review/index.js';
 import { featureNodeStateConfig, lifecycleDisplayLabels } from '../../common/feature-node/index.js';
 import { useFeatureActions } from '../../common/feature-drawer/use-feature-actions.js';
-import { mapEventTypeToState, mapPhaseNameToLifecycle, } from '../../common/feature-node/derive-feature-state.js';
+import { resolveSseEventUpdates } from '../../common/feature-node/derive-feature-state.js';
+import { deriveFeatureViewType } from './drawer-view.js';
 export function FeatureDrawerClient({ view: initialView }) {
     const router = useRouter();
     const deleteSound = useSoundAction('delete');
@@ -59,28 +60,15 @@ export function FeatureDrawerClient({ view: initialView }) {
             return;
         const newEvents = events.slice(processedCountRef.current);
         processedCountRef.current = events.length;
-        // When WaitingApproval and PhaseCompleted arrive in the same poll batch,
-        // PhaseCompleted must be fully suppressed (both state AND lifecycle). Without this,
-        // a phase-timing PhaseCompleted (e.g. phaseName='research') can regress the lifecycle
-        // that WaitingApproval already set correctly (e.g. to 'implementation' for plan gate).
-        const waitingApprovalInBatch = newEvents.some((e) => e.featureId === featureNode.featureId &&
-            e.eventType === NotificationEventType.WaitingApproval);
-        for (const event of newEvents) {
-            if (event.featureId !== featureNode.featureId)
+        for (const update of resolveSseEventUpdates(newEvents)) {
+            if (update.featureId !== featureNode.featureId)
                 continue;
-            const isSuppressedPhaseCompleted = event.eventType === NotificationEventType.PhaseCompleted && waitingApprovalInBatch;
-            const newState = isSuppressedPhaseCompleted
-                ? undefined
-                : mapEventTypeToState(event.eventType);
-            const newLifecycle = isSuppressedPhaseCompleted
-                ? undefined
-                : mapPhaseNameToLifecycle(event.phaseName);
             // Trigger a server refresh to get the latest drawer view,
             // but skip when the user has unsaved changes or a rejection is in-flight
             // to avoid the refresh interfering with the reject handler's navigation.
             if (!isDirtyRef.current && !isRejectingRef.current)
                 router.refresh();
-            if (newState !== undefined || newLifecycle !== undefined) {
+            if (update.state !== undefined || update.lifecycle !== undefined) {
                 // Optimistically update the node data AND re-derive the view type so the
                 // drawer switches immediately (e.g. prd-review → feature when agent resumes)
                 // without waiting for the router.refresh() round-trip.
@@ -92,10 +80,10 @@ export function FeatureDrawerClient({ view: initialView }) {
                         return prev;
                     const updatedNode = {
                         ...prev.node,
-                        ...(newState !== undefined && { state: newState }),
-                        ...(newLifecycle !== undefined && { lifecycle: newLifecycle }),
+                        ...(update.state !== undefined && { state: update.state }),
+                        ...(update.lifecycle !== undefined && { lifecycle: update.lifecycle }),
                     };
-                    return { ...prev, type: deriveViewType(updatedNode), node: updatedNode };
+                    return { ...prev, type: deriveFeatureViewType(updatedNode), node: updatedNode };
                 });
             }
         }
@@ -317,6 +305,10 @@ export function FeatureDrawerClient({ view: initialView }) {
         }
         setChatInput('');
         toast.success(`${label} approved — agent resuming`);
+        // Optimistically update canvas node before SSE arrives (~500ms delay)
+        window.dispatchEvent(new CustomEvent('shep:feature-approved', {
+            detail: { featureId: reviewNode.featureId },
+        }));
         onClose();
     }, [reviewNode, onClose]);
     const handlePrdApprove = useCallback(async (_actionId) => {
@@ -341,6 +333,10 @@ export function FeatureDrawerClient({ view: initialView }) {
         }
         setChatInput('');
         toast.success('Requirements approved — agent resuming');
+        // Optimistically update canvas node before SSE arrives (~500ms delay)
+        window.dispatchEvent(new CustomEvent('shep:feature-approved', {
+            detail: { featureId: view.node.featureId },
+        }));
         setPrdSelections({});
         onClose();
     }, [view, prdData, prdSelections, onClose]);
@@ -433,13 +429,3 @@ function FeatureDetails({ data }) {
 function DetailRow({ label, value }) {
     return (_jsxs("div", { className: "flex flex-col gap-0.5", children: [_jsx("span", { className: "text-muted-foreground text-xs font-medium", children: label }), _jsx("span", { className: "text-sm", children: value })] }));
 }
-/** Mirrors computeDrawerView logic — derives view type from live node data. */
-function deriveViewType(node) {
-    if (node.lifecycle === 'requirements' && node.state === 'action-required')
-        return 'prd-review';
-    if (node.lifecycle === 'implementation' && node.state === 'action-required')
-        return 'tech-review';
-    if (node.lifecycle === 'review' && (node.state === 'action-required' || node.state === 'error'))
-        return 'merge-review';
-    return 'feature';
-}

package/dist/src/presentation/web/components/common/control-center-drawer/index.d.ts

@@ -1,3 +1,2 @@
-export { ControlCenterDrawer, type ControlCenterDrawerProps } from './control-center-drawer.js';
 export { computeDrawerView, type DrawerView } from './drawer-view.js';
 //# sourceMappingURL=index.d.ts.map

package/dist/src/presentation/web/components/common/control-center-drawer/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,KAAK,wBAAwB,EAAE,MAAM,yBAAyB,CAAC;AAC7F,OAAO,EAAE,iBAAiB,EAAE,KAAK,UAAU,EAAE,MAAM,eAAe,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,UAAU,EAAE,MAAM,eAAe,CAAC"}

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

@@ -1,2 +1 @@
-export { ControlCenterDrawer } from './control-center-drawer.js';
 export { computeDrawerView } from './drawer-view.js';

package/dist/src/presentation/web/components/common/feature-node/derive-feature-state.d.ts

@@ -28,5 +28,30 @@ export declare function deriveNodeState(feature: Feature, agentRun?: AgentRun |
 export declare function deriveProgress(feature: Feature): number;
 /** Maps a NotificationEventType to the corresponding FeatureNodeState for optimistic UI updates. */
 export declare function mapEventTypeToState(eventType: NotificationEventType): FeatureNodeState;
+/** Map domain SdlcLifecycle enum values to UI FeatureLifecyclePhase. */
+export declare const sdlcLifecycleMap: Record<string, FeatureLifecyclePhase>;
+/**
+ * Derives the UI lifecycle phase from a Feature + AgentRun.
+ * Shared by build-feature-node-data.ts and build-graph-nodes.ts.
+ */
+export declare function deriveLifecycle(feature: Feature, run: AgentRun | null): FeatureLifecyclePhase;
+export interface SseEventUpdate {
+    featureId: string;
+    state: FeatureNodeState | undefined;
+    lifecycle: FeatureLifecyclePhase | undefined;
+    eventType: NotificationEventType;
+    phaseName?: string;
+}
+/**
+ * Resolves state/lifecycle updates from a batch of SSE events, applying
+ * the WaitingApproval batch-suppression rule: when WaitingApproval and
+ * PhaseCompleted arrive in the same batch for a feature, PhaseCompleted
+ * is fully suppressed to prevent lifecycle regression.
+ */
+export declare function resolveSseEventUpdates(events: readonly {
+    featureId: string;
+    eventType: NotificationEventType;
+    phaseName?: string;
+}[]): SseEventUpdate[];
 export declare function mapPhaseNameToLifecycle(phaseName: string | undefined): FeatureLifecyclePhase | undefined;
 //# sourceMappingURL=derive-feature-state.d.ts.map

package/dist/src/presentation/web/components/common/feature-node/derive-feature-state.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"derive-feature-state.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-node/derive-feature-state.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAIL,qBAAqB,EACtB,MAAM,+BAA+B,CAAC;AACvC,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,+BAA+B,CAAC;AACvE,OAAO,KAAK,EAAE,gBAAgB,EAAE,qBAAqB,EAAE,MAAM,6BAA6B,CAAC;AAE3F;;;;;;;;;;GAUG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,OAAO,EAAE,QAAQ,CAAC,EAAE,QAAQ,GAAG,IAAI,GAAG,gBAAgB,CA4C9F;AAED;;;;;GAKG;AACH,wBAAgB,cAAc,CAAC,OAAO,EAAE,OAAO,GAAG,MAAM,CAYvD;AAED,oGAAoG;AACpG,wBAAgB,mBAAmB,CAAC,SAAS,EAAE,qBAAqB,GAAG,gBAAgB,CAkBtF;AAcD,wBAAgB,uBAAuB,CACrC,SAAS,EAAE,MAAM,GAAG,SAAS,GAC5B,qBAAqB,GAAG,SAAS,CAGnC"}
+{"version":3,"file":"derive-feature-state.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-node/derive-feature-state.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAIL,qBAAqB,EACtB,MAAM,+BAA+B,CAAC;AACvC,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,+BAA+B,CAAC;AACvE,OAAO,KAAK,EAAE,gBAAgB,EAAE,qBAAqB,EAAE,MAAM,6BAA6B,CAAC;AAE3F;;;;;;;;;;GAUG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,OAAO,EAAE,QAAQ,CAAC,EAAE,QAAQ,GAAG,IAAI,GAAG,gBAAgB,CA4C9F;AAED;;;;;GAKG;AACH,wBAAgB,cAAc,CAAC,OAAO,EAAE,OAAO,GAAG,MAAM,CAYvD;AAED,oGAAoG;AACpG,wBAAgB,mBAAmB,CAAC,SAAS,EAAE,qBAAqB,GAAG,gBAAgB,CAkBtF;AAED,wEAAwE;AACxE,eAAO,MAAM,gBAAgB,EAAE,MAAM,CAAC,MAAM,EAAE,qBAAqB,CAOlE,CAAC;AAcF;;;GAGG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,IAAI,GAAG,qBAAqB,CAQ7F;AAED,MAAM,WAAW,cAAc;IAC7B,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,gBAAgB,GAAG,SAAS,CAAC;IACpC,SAAS,EAAE,qBAAqB,GAAG,SAAS,CAAC;IAC7C,SAAS,EAAE,qBAAqB,CAAC;IACjC,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED;;;;;GAKG;AACH,wBAAgB,sBAAsB,CACpC,MAAM,EAAE,SAAS;IAAE,SAAS,EAAE,MAAM,CAAC;IAAC,SAAS,EAAE,qBAAqB,CAAC;IAAC,SAAS,CAAC,EAAE,MAAM,CAAA;CAAE,EAAE,GAC7F,cAAc,EAAE,CAmBlB;AAED,wBAAgB,uBAAuB,CACrC,SAAS,EAAE,MAAM,GAAG,SAAS,GAC5B,qBAAqB,GAAG,SAAS,CAGnC"}

package/dist/src/presentation/web/components/common/feature-node/derive-feature-state.js

@@ -31,7 +31,7 @@ export function deriveNodeState(feature, agentRun) {
             case AgentRunStatus.cancelled:
                 return 'blocked';
             case AgentRunStatus.completed:
-                return feature.lifecycle === SdlcLifecycle.Maintain ? 'done' : 'done';
+                return 'done';
             case AgentRunStatus.running:
             case AgentRunStatus.pending:
                 return 'running';
@@ -93,7 +93,16 @@ export function mapEventTypeToState(eventType) {
             return 'action-required';
     }
 }
-/** Maps an SSE event phaseName to a FeatureLifecyclePhase. Mirrors page.tsx nodeToLifecyclePhase. */
+/** Map domain SdlcLifecycle enum values to UI FeatureLifecyclePhase. */
+export const sdlcLifecycleMap = {
+    Requirements: 'requirements',
+    Research: 'research',
+    Implementation: 'implementation',
+    Review: 'review',
+    'Deploy & QA': 'deploy',
+    Maintain: 'maintain',
+};
+/** Map agent graph node names (from agent_run.result or SSE phaseName) to UI lifecycle phases. */
 const phaseNameToLifecycle = {
     analyze: 'requirements',
     requirements: 'requirements',
@@ -104,6 +113,40 @@ const phaseNameToLifecycle = {
     maintain: 'maintain',
     blocked: 'requirements',
 };
+/**
+ * Derives the UI lifecycle phase from a Feature + AgentRun.
+ * Shared by build-feature-node-data.ts and build-graph-nodes.ts.
+ */
+export function deriveLifecycle(feature, run) {
+    if (run?.status === 'completed')
+        return 'maintain';
+    const agentNode = run?.result?.startsWith('node:') ? run.result.slice(5) : undefined;
+    return ((agentNode ? phaseNameToLifecycle[agentNode] : undefined) ??
+        sdlcLifecycleMap[feature.lifecycle] ??
+        'requirements');
+}
+/**
+ * Resolves state/lifecycle updates from a batch of SSE events, applying
+ * the WaitingApproval batch-suppression rule: when WaitingApproval and
+ * PhaseCompleted arrive in the same batch for a feature, PhaseCompleted
+ * is fully suppressed to prevent lifecycle regression.
+ */
+export function resolveSseEventUpdates(events) {
+    const waitingApprovalFeatures = new Set(events
+        .filter((e) => e.eventType === NotificationEventType.WaitingApproval)
+        .map((e) => e.featureId));
+    return events.map((event) => {
+        const isSuppressed = event.eventType === NotificationEventType.PhaseCompleted &&
+            waitingApprovalFeatures.has(event.featureId);
+        return {
+            featureId: event.featureId,
+            state: isSuppressed ? undefined : mapEventTypeToState(event.eventType),
+            lifecycle: isSuppressed ? undefined : mapPhaseNameToLifecycle(event.phaseName),
+            eventType: event.eventType,
+            phaseName: event.phaseName,
+        };
+    });
+}
 export function mapPhaseNameToLifecycle(phaseName) {
     if (!phaseName)
         return undefined;

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

@@ -1,4 +1,4 @@
-export { ControlCenterDrawer, type ControlCenterDrawerProps, computeDrawerView, type DrawerView, } from './control-center-drawer/index.js';
+export { computeDrawerView, type DrawerView } from './control-center-drawer/index.js';
 export { AddRepositoryButton } from './add-repository-button/index.js';
 export { ElapsedTime, formatElapsed } from './elapsed-time/index.js';
 export { EmptyState, type EmptyStateProps } from './empty-state/index.js';

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

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

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

@@ -1,4 +1,4 @@
-export { ControlCenterDrawer, computeDrawerView, } from './control-center-drawer/index.js';
+export { computeDrawerView } from './control-center-drawer/index.js';
 export { AddRepositoryButton } from './add-repository-button/index.js';
 export { ElapsedTime, formatElapsed } from './elapsed-time/index.js';
 export { EmptyState } from './empty-state/index.js';

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;AAYjC,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,CAoVpB"}
+{"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"}

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

@@ -1,5 +1,5 @@
 'use client';
-import { useCallback, useEffect, useRef } from 'react';
+import { useCallback, useEffect, useMemo, useRef } from 'react';
 import { useRouter } from 'next/navigation';
 import { toast } from 'sonner';
 import { layoutWithDagre, CANVAS_LAYOUT_DEFAULTS, } from '../../../lib/layout-with-dagre.js';
@@ -9,8 +9,7 @@ import { deleteRepository } from '../../../app/actions/delete-repository.js';
 import { getFeatureMetadata } from '../../../app/actions/get-feature-metadata.js';
 import { useAgentEventsContext } from '../../../hooks/agent-events-provider.js';
 import { useSoundAction } from '../../../hooks/use-sound-action.js';
-import { mapEventTypeToState, mapPhaseNameToLifecycle, } from '../../common/feature-node/derive-feature-state.js';
-import { NotificationEventType } from '../../../../../../packages/core/src/domain/generated/output.js';
+import { mapEventTypeToState, resolveSseEventUpdates, } from '../../common/feature-node/derive-feature-state.js';
 import { useGraphState } from '../../../hooks/use-graph-state.js';
 /** Must match the message string emitted by the SSE route in agent-events/route.ts */
 const METADATA_UPDATED_MESSAGE = 'Feature metadata updated';
@@ -22,18 +21,18 @@ export function useControlCenterState(initialNodes, initialEdges) {
     const { nodes, edges, reconcile, updateFeature, addPendingFeature, removeFeature, restoreFeature, addRepository: addRepositoryToMap, removeRepository, replaceRepository, getFeatureRepositoryPath, getRepositoryData, setCallbacks, } = useGraphState(initialNodes, initialEdges);
     // 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 = initialNodes
+    const initialNodeKey = useMemo(() => initialNodes
         .map((n) => n.id)
         .sort()
-        .join(',');
-    const initialDataKey = initialNodes
+        .join(','), [initialNodes]);
+    const initialDataKey = useMemo(() => initialNodes
         .filter((n) => n.type === 'featureNode')
         .map((n) => {
         const d = n.data;
         return `${n.id}:${d.state}:${d.lifecycle}`;
     })
         .sort()
-        .join(',');
+        .join(','), [initialNodes]);
     const prevReconcileKey = useRef('');
     useEffect(() => {
         const key = `${initialNodeKey}|${initialDataKey}`;
@@ -82,33 +81,24 @@ export function useControlCenterState(initialNodes, initialEdges) {
             return;
         const newEvents = events.slice(processedEventCountRef.current);
         processedEventCountRef.current = events.length;
-        // When WaitingApproval and PhaseCompleted arrive in the same poll batch for a feature,
-        // PhaseCompleted must be fully suppressed (both state AND lifecycle). Without this,
-        // a phase-timing PhaseCompleted (e.g. phaseName='research') can regress the lifecycle
-        // that WaitingApproval already set correctly (e.g. to 'implementation' for plan gate).
-        // When PhaseCompleted arrives WITHOUT WaitingApproval (e.g. after approval), we allow
-        // it to update both state and lifecycle as a fallback for any missed AgentStarted event.
-        const waitingApprovalFeatures = new Set(newEvents
-            .filter((e) => e.eventType === NotificationEventType.WaitingApproval)
-            .map((e) => e.featureId));
-        for (const event of newEvents) {
-            const isSuppressedPhaseCompleted = event.eventType === NotificationEventType.PhaseCompleted &&
-                waitingApprovalFeatures.has(event.featureId);
-            const newState = isSuppressedPhaseCompleted
-                ? undefined
-                : mapEventTypeToState(event.eventType);
-            const newLifecycle = isSuppressedPhaseCompleted
-                ? undefined
-                : mapPhaseNameToLifecycle(event.phaseName);
-            const nodeId = `feat-${event.featureId}`;
-            if (newState !== undefined || newLifecycle !== undefined) {
-                updateFeature(nodeId, {
-                    ...(newState !== undefined && { state: newState }),
-                    ...(newLifecycle !== undefined && { lifecycle: newLifecycle }),
+        for (const { featureId, state, lifecycle } of resolveSseEventUpdates(newEvents)) {
+            if (state !== undefined || lifecycle !== undefined) {
+                updateFeature(`feat-${featureId}`, {
+                    ...(state !== undefined && { state }),
+                    ...(lifecycle !== undefined && { lifecycle }),
                 });
             }
         }
     }, [events, updateFeature]);
+    // Listen for optimistic approval events from the drawer (fires before SSE arrives)
+    useEffect(() => {
+        const handler = (e) => {
+            const { featureId } = e.detail;
+            updateFeature(`feat-${featureId}`, { state: 'running' });
+        };
+        window.addEventListener('shep:feature-approved', handler);
+        return () => window.removeEventListener('shep:feature-approved', handler);
+    }, [updateFeature]);
     // 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/components/features/features-canvas/features-canvas.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"features-canvas.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/features/features-canvas/features-canvas.tsx"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,UAAU,EAAE,IAAI,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAKlE,OAAO,KAAK,EAAE,eAAe,EAAmB,MAAM,kCAAkC,CAAC;AAEzF,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAG9E,MAAM,MAAM,cAAc,GAAG,eAAe,GAAG,kBAAkB,CAAC;AAElE,MAAM,WAAW,mBAAmB;IAClC,KAAK,EAAE,cAAc,EAAE,CAAC;IACxB,KAAK,EAAE,IAAI,EAAE,CAAC;IACd,iBAAiB,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAClC,aAAa,CAAC,EAAE,CAAC,OAAO,EAAE,UAAU,CAAC,cAAc,CAAC,EAAE,KAAK,IAAI,CAAC;IAChE,YAAY,CAAC,EAAE,MAAM,IAAI,CAAC;IAC1B,WAAW,CAAC,EAAE,CAAC,KAAK,EAAE,KAAK,CAAC,UAAU,EAAE,IAAI,EAAE,cAAc,KAAK,IAAI,CAAC;IACtE,WAAW,CAAC,EAAE,CAAC,KAAK,EAAE,KAAK,CAAC,UAAU,KAAK,IAAI,CAAC;IAChD,SAAS,CAAC,EAAE,CAAC,UAAU,EAAE,UAAU,KAAK,IAAI,CAAC;IAC7C,YAAY,CAAC,EAAE,MAAM,IAAI,CAAC;IAC1B,OAAO,CAAC,EAAE,KAAK,CAAC,SAAS,CAAC;IAC1B,UAAU,CAAC,EAAE,KAAK,CAAC,SAAS,CAAC;CAC9B;AAED,wBAAgB,cAAc,CAAC,EAC7B,KAAK,EACL,KAAK,EACL,iBAAiB,EACjB,aAAa,EACb,YAAY,EACZ,SAAS,EACT,WAAW,EACX,WAAW,EACX,YAAY,EACZ,OAAO,EACP,UAAU,GACX,EAAE,mBAAmB,2CAmGrB"}
+{"version":3,"file":"features-canvas.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/features/features-canvas/features-canvas.tsx"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,UAAU,EAAE,IAAI,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAKlE,OAAO,KAAK,EAAE,eAAe,EAAmB,MAAM,kCAAkC,CAAC;AAEzF,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAG9E,MAAM,MAAM,cAAc,GAAG,eAAe,GAAG,kBAAkB,CAAC;AAElE,MAAM,WAAW,mBAAmB;IAClC,KAAK,EAAE,cAAc,EAAE,CAAC;IACxB,KAAK,EAAE,IAAI,EAAE,CAAC;IACd,iBAAiB,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAClC,aAAa,CAAC,EAAE,CAAC,OAAO,EAAE,UAAU,CAAC,cAAc,CAAC,EAAE,KAAK,IAAI,CAAC;IAChE,YAAY,CAAC,EAAE,MAAM,IAAI,CAAC;IAC1B,WAAW,CAAC,EAAE,CAAC,KAAK,EAAE,KAAK,CAAC,UAAU,EAAE,IAAI,EAAE,cAAc,KAAK,IAAI,CAAC;IACtE,WAAW,CAAC,EAAE,CAAC,KAAK,EAAE,KAAK,CAAC,UAAU,KAAK,IAAI,CAAC;IAChD,SAAS,CAAC,EAAE,CAAC,UAAU,EAAE,UAAU,KAAK,IAAI,CAAC;IAC7C,YAAY,CAAC,EAAE,MAAM,IAAI,CAAC;IAC1B,OAAO,CAAC,EAAE,KAAK,CAAC,SAAS,CAAC;IAC1B,UAAU,CAAC,EAAE,KAAK,CAAC,SAAS,CAAC;CAC9B;AAED,wBAAgB,cAAc,CAAC,EAC7B,KAAK,EACL,KAAK,EACL,iBAAiB,EACjB,aAAa,EACb,YAAY,EACZ,SAAS,EACT,WAAW,EACX,WAAW,EACX,YAAY,EACZ,OAAO,EACP,UAAU,GACX,EAAE,mBAAmB,2CAkFrB"}

package/dist/src/presentation/web/components/features/features-canvas/features-canvas.js

@@ -1,6 +1,6 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
-import { useMemo, useCallback } from 'react';
+import { useMemo } from 'react';
 import { ReactFlow, Background, Controls } from '@xyflow/react';
 import { Plus } from 'lucide-react';
 import { Button } from '../../ui/button.js';
@@ -16,17 +16,6 @@ export function FeaturesCanvas({ nodes, edges, selectedFeatureId, onNodesChange,
     const edgeTypes = useMemo(() => ({
         dependencyEdge: DependencyEdge,
     }), []);
-    // Prevent a feature from having more than one source repository
-    const isValidConnection = useCallback((connection) => {
-        const sourceNode = nodes.find((n) => n.id === connection.source);
-        if (sourceNode?.type !== 'repositoryNode')
-            return true;
-        const targetAlreadyHasRepo = edges.some((e) => {
-            const edgeSourceNode = nodes.find((n) => n.id === e.source);
-            return edgeSourceNode?.type === 'repositoryNode' && e.target === connection.target;
-        });
-        return !targetAlreadyHasRepo;
-    }, [nodes, edges]);
     // Callbacks and showHandles are already injected into node.data by deriveGraph.
     // Only selectedFeatureId highlighting needs to be applied here.
     const enrichedNodes = useMemo(() => selectedFeatureId == null
@@ -41,5 +30,5 @@ export function FeaturesCanvas({ nodes, edges, selectedFeatureId, onNodesChange,
         }
         return (_jsx("div", { "data-testid": "features-canvas-empty", children: _jsx(EmptyState, { title: "No features yet", description: "Get started by creating your first feature.", action: _jsxs(Button, { onClick: onAddFeature, children: [_jsx(Plus, { className: "mr-2 h-4 w-4" }), "New Feature"] }) }) }));
     }
-    return (_jsx("div", { "data-testid": "features-canvas", "data-no-drawer-close": true, className: "pointer-events-auto h-full w-full", children: _jsxs(ReactFlow, { nodes: enrichedNodes, edges: edges, nodeTypes: nodeTypes, edgeTypes: edgeTypes, isValidConnection: isValidConnection, onConnect: onConnect, onNodesChange: onNodesChange, onNodeClick: onNodeClick, onPaneClick: onPaneClick, onMoveStart: onCanvasDrag, defaultViewport: { x: 30, y: 30, zoom: 0.85 }, nodesDraggable: false, nodesConnectable: false, elementsSelectable: false, children: [_jsx(Background, {}), _jsx(Controls, {}), toolbar] }) }));
+    return (_jsx("div", { "data-testid": "features-canvas", "data-no-drawer-close": true, className: "pointer-events-auto h-full w-full", children: _jsxs(ReactFlow, { nodes: enrichedNodes, edges: edges, nodeTypes: nodeTypes, edgeTypes: edgeTypes, onConnect: onConnect, onNodesChange: onNodesChange, onNodeClick: onNodeClick, onPaneClick: onPaneClick, onMoveStart: onCanvasDrag, defaultViewport: { x: 30, y: 30, zoom: 0.85 }, nodesDraggable: false, nodesConnectable: false, elementsSelectable: false, children: [_jsx(Background, {}), _jsx(Controls, {}), toolbar] }) }));
 }

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,CA2GpF"}
+{"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"}

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

@@ -75,10 +75,13 @@ export function useAgentEvents(options) {
                 subscribeToWorker(sw);
             }
             else {
-                sw.addEventListener('statechange', () => {
-                    if (sw.state === 'activated')
+                const onStateChange = () => {
+                    if (sw.state === 'activated') {
+                        sw.removeEventListener('statechange', onStateChange);
                         subscribeToWorker(sw);
-                });
+                    }
+                };
+                sw.addEventListener('statechange', onStateChange);
             }
         })
             .catch(() => {