@shepai/cli 1.72.8 → 1.72.9-pr220.99e5e13

package/dist/src/presentation/web/hooks/use-notifications.js

@@ -1,5 +1,5 @@
 'use client';
-import { useCallback, useEffect, useMemo, useRef, useState } from 'react';
+import { useEffect, useMemo, useRef } from 'react';
 import { useRouter } from 'next/navigation';
 import { toast } from 'sonner';
 import { NotificationEventType, NotificationSeverity } from '../../../../packages/core/src/domain/generated/output.js';
@@ -27,15 +27,6 @@ function dispatchToast(event, navigate) {
         }),
     });
 }
-function dispatchBrowserNotification(event) {
-    if (globalThis.Notification?.permission !== 'granted') {
-        return;
-    }
-    new Notification(`Shep: ${event.featureName}`, {
-        body: event.message,
-        icon: '/favicon-light.svg',
-    });
-}
 const SEVERITY_TO_ACTION = {
     [NotificationSeverity.Success]: 'notification-success',
     [NotificationSeverity.Error]: 'notification-error',
@@ -55,11 +46,6 @@ export function useNotifications() {
         'notification-warning': warningSound,
         'notification-info': infoSound,
     }), [successSound, errorSound, warningSound, infoSound]);
-    const [browserPermissionState, setBrowserPermissionState] = useState(() => {
-        if (typeof globalThis.Notification === 'undefined')
-            return 'default';
-        return globalThis.Notification.permission;
-    });
     // Track how many events from the array we've already processed.
     // Using the array index (instead of lastEvent) prevents React batching
     // from silently dropping events when multiple SSE messages arrive together.
@@ -77,19 +63,8 @@ export function useNotifications() {
                 continue;
             }
             dispatchToast(event, (path) => router.push(path));
-            dispatchBrowserNotification(event);
             const actionName = SEVERITY_TO_ACTION[event.severity];
             soundsByAction[actionName]?.play();
         }
     }, [events, soundsByAction, router]);
-    const requestBrowserPermission = useCallback(async () => {
-        if (typeof globalThis.Notification === 'undefined')
-            return;
-        const result = await globalThis.Notification.requestPermission();
-        setBrowserPermissionState(result);
-    }, []);
-    return {
-        requestBrowserPermission,
-        browserPermissionState,
-    };
 }

package/dist/src/presentation/web/hooks/use-notifications.stories.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-notifications.stories.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-notifications.stories.tsx"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAqKvD,QAAA,MAAM,IAAI,EAAE,IAcX,CAAC;AAEF,eAAe,IAAI,CAAC;AACpB,KAAK,KAAK,GAAG,QAAQ,CAAC;AAEtB,eAAO,MAAM,OAAO,EAAE,KAErB,CAAC"}
+{"version":3,"file":"use-notifications.stories.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-notifications.stories.tsx"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAyIvD,QAAA,MAAM,IAAI,EAAE,IAcX,CAAC;AAEF,eAAe,IAAI,CAAC;AACpB,KAAK,KAAK,GAAG,QAAQ,CAAC;AAEtB,eAAO,MAAM,OAAO,EAAE,KAErB,CAAC"}

package/dist/src/presentation/web/hooks/use-notifications.stories.js

@@ -1,5 +1,5 @@
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
-import { useCallback, useMemo, useState } from 'react';
+import { useCallback, useMemo } from 'react';
 import { toast } from 'sonner';
 import { Toaster } from '../components/ui/sonner.js';
 import { Button } from '../components/ui/button.js';
@@ -24,46 +24,36 @@ function NotificationDemo() {
     const simulateNotification = useCallback((event) => {
         const method = event.severity ?? 'info';
         toast[method](event.featureName, { description: event.message });
-        if (globalThis.Notification?.permission === 'granted') {
-            new Notification(event.featureName, { body: event.message });
-        }
         const soundName = SEVERITY_SOUNDS[event.severity];
         soundsByName[soundName]?.play();
     }, [soundsByName]);
-    const [browserPermission, setBrowserPermission] = useState(typeof globalThis.Notification !== 'undefined' ? globalThis.Notification.permission : 'default');
-    const requestPermission = async () => {
-        if (typeof globalThis.Notification === 'undefined')
-            return;
-        const result = await globalThis.Notification.requestPermission();
-        setBrowserPermission(result);
-    };
-    return (_jsxs("div", { className: "flex w-[420px] flex-col gap-6", children: [_jsxs("div", { className: "flex flex-col gap-2", children: [_jsx("h3", { className: "text-sm font-semibold", children: "Browser Notification Permission" }), _jsxs("div", { className: "flex items-center gap-3", children: [_jsxs("span", { className: "text-muted-foreground text-sm", children: ["Status: ", _jsx("code", { className: "bg-muted rounded px-1.5 py-0.5", children: browserPermission })] }), _jsx(Button, { size: "sm", variant: "outline", onClick: requestPermission, children: "Request Permission" })] })] }), _jsxs("div", { className: "flex flex-col gap-2", children: [_jsx("h3", { className: "text-sm font-semibold", children: "Simulate Notification Events" }), _jsx("p", { className: "text-muted-foreground text-xs", children: "Click a button to fire a toast, browser notification, and sound." }), _jsxs("div", { className: "flex flex-wrap gap-2", children: [_jsx(Button, { size: "sm", variant: "outline", onClick: () => simulateNotification({
-                                    eventType: 'agent_started',
-                                    severity: 'info',
-                                    featureName: 'Auth Module',
-                                    message: 'Agent started running',
-                                }), children: "Agent Started" }), _jsx(Button, { size: "sm", variant: "outline", onClick: () => simulateNotification({
-                                    eventType: 'phase_completed',
-                                    severity: 'info',
-                                    featureName: 'Auth Module',
-                                    message: 'Completed analyze phase',
-                                    phaseName: 'analyze',
-                                }), children: "Phase Completed" }), _jsx(Button, { size: "sm", variant: "secondary", onClick: () => simulateNotification({
-                                    eventType: 'waiting_approval',
-                                    severity: 'warning',
-                                    featureName: 'Auth Module',
-                                    message: 'Waiting for user approval',
-                                }), children: "Waiting Approval" }), _jsx(Button, { size: "sm", onClick: () => simulateNotification({
-                                    eventType: 'agent_completed',
-                                    severity: 'success',
-                                    featureName: 'Auth Module',
-                                    message: 'Agent completed successfully',
-                                }), children: "Agent Completed" }), _jsx(Button, { size: "sm", variant: "destructive", onClick: () => simulateNotification({
-                                    eventType: 'agent_failed',
-                                    severity: 'error',
-                                    featureName: 'Auth Module',
-                                    message: 'Agent failed: token refresh error',
-                                }), children: "Agent Failed" })] })] })] }));
+    return (_jsx("div", { className: "flex w-[420px] flex-col gap-6", children: _jsxs("div", { className: "flex flex-col gap-2", children: [_jsx("h3", { className: "text-sm font-semibold", children: "Simulate Notification Events" }), _jsx("p", { className: "text-muted-foreground text-xs", children: "Click a button to fire a toast and sound." }), _jsxs("div", { className: "flex flex-wrap gap-2", children: [_jsx(Button, { size: "sm", variant: "outline", onClick: () => simulateNotification({
+                                eventType: 'agent_started',
+                                severity: 'info',
+                                featureName: 'Auth Module',
+                                message: 'Agent started running',
+                            }), children: "Agent Started" }), _jsx(Button, { size: "sm", variant: "outline", onClick: () => simulateNotification({
+                                eventType: 'phase_completed',
+                                severity: 'info',
+                                featureName: 'Auth Module',
+                                message: 'Completed analyze phase',
+                                phaseName: 'analyze',
+                            }), children: "Phase Completed" }), _jsx(Button, { size: "sm", variant: "secondary", onClick: () => simulateNotification({
+                                eventType: 'waiting_approval',
+                                severity: 'warning',
+                                featureName: 'Auth Module',
+                                message: 'Waiting for user approval',
+                            }), children: "Waiting Approval" }), _jsx(Button, { size: "sm", onClick: () => simulateNotification({
+                                eventType: 'agent_completed',
+                                severity: 'success',
+                                featureName: 'Auth Module',
+                                message: 'Agent completed successfully',
+                            }), children: "Agent Completed" }), _jsx(Button, { size: "sm", variant: "destructive", onClick: () => simulateNotification({
+                                eventType: 'agent_failed',
+                                severity: 'error',
+                                featureName: 'Auth Module',
+                                message: 'Agent failed: token refresh error',
+                            }), children: "Agent Failed" })] })] }) }));
 }
 const meta = {
     title: 'Hooks/useNotifications',

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

@@ -0,0 +1,52 @@
+/**
+ * derive-graph.ts
+ *
+ * Pure client-side derivation: domain Maps → React Flow nodes + edges.
+ * All edges are derived from domain relationships (repositoryPath matching,
+ * parentNodeId) — never stored independently.
+ */
+import type { Edge } from '@xyflow/react';
+import type { CanvasNodeType } from '../components/features/features-canvas/index.js';
+import type { FeatureNodeData } from '../components/common/feature-node/index.js';
+import type { RepositoryNodeData } from '../components/common/repository-node/index.js';
+/** A feature node entry stored in the domain Map. */
+export interface FeatureEntry {
+    nodeId: string;
+    data: FeatureNodeData;
+    /** If set, this feature connects to a parent feature via a dependency edge (no repo→feature edge). */
+    parentNodeId?: string;
+}
+/** A repository node entry stored in the domain Map. */
+export interface RepoEntry {
+    nodeId: string;
+    data: RepositoryNodeData;
+}
+/** Stable callbacks passed by the consumer, injected into derived node data. */
+export interface GraphCallbacks {
+    /** Called when the user triggers an action on a feature node (e.g., adds a sub-feature). */
+    onNodeAction?: (nodeId: string) => void;
+    /** Called when the user opens settings/details on a feature node. */
+    onNodeSettings?: (nodeId: string) => void;
+    /** Called when the user deletes a feature. */
+    onFeatureDelete?: (featureId: string) => void;
+    /** Called when the user clicks the "+" add-feature button on a repo node. */
+    onRepositoryAdd?: (repoNodeId: string) => void;
+    /** Called when the user clicks a repo node to navigate to its detail page. */
+    onRepositoryClick?: (repoNodeId: string) => void;
+    /** Called when the user deletes a repository. */
+    onRepositoryDelete?: (repositoryId: string) => void;
+}
+/**
+ * Derives React Flow nodes and edges from domain Maps.
+ *
+ * - Feature nodes without a parentNodeId → repo→feature edge (matched by repositoryPath)
+ * - Feature nodes without a parentNodeId but with no matching repo → virtual repo node
+ * - Feature nodes with a parentNodeId → dependency edge (parentNodeId→nodeId)
+ * - Pending feature nodes (creating state) from pendingMap → same rules, no action callbacks
+ * - Callbacks are injected into node data via closures
+ */
+export declare function deriveGraph(featureMap: Map<string, FeatureEntry>, repoMap: Map<string, RepoEntry>, pendingMap: Map<string, FeatureEntry>, callbacks?: GraphCallbacks): {
+    nodes: CanvasNodeType[];
+    edges: Edge[];
+};
+//# sourceMappingURL=derive-graph.d.ts.map

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

@@ -0,0 +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,KAAK,IAAI,CAAC;IAC9C,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;CACrD;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,CAyH5C"}

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

@@ -0,0 +1,126 @@
+/**
+ * derive-graph.ts
+ *
+ * Pure client-side derivation: domain Maps → React Flow nodes + edges.
+ * All edges are derived from domain relationships (repositoryPath matching,
+ * parentNodeId) — never stored independently.
+ */
+/**
+ * Derives React Flow nodes and edges from domain Maps.
+ *
+ * - Feature nodes without a parentNodeId → repo→feature edge (matched by repositoryPath)
+ * - Feature nodes without a parentNodeId but with no matching repo → virtual repo node
+ * - Feature nodes with a parentNodeId → dependency edge (parentNodeId→nodeId)
+ * - Pending feature nodes (creating state) from pendingMap → same rules, no action callbacks
+ * - Callbacks are injected into node data via closures
+ */
+export function deriveGraph(featureMap, repoMap, pendingMap, callbacks) {
+    const nodes = [];
+    const edges = [];
+    // 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);
+        }
+    }
+    // Virtual repo nodes created for orphaned features (not in repoMap)
+    const virtualRepos = new Map(); // path → virtualNodeId
+    function getOrCreateVirtualRepo(repositoryPath) {
+        if (virtualRepos.has(repositoryPath)) {
+            return virtualRepos.get(repositoryPath);
+        }
+        const virtualId = `virtual-repo-${repositoryPath}`;
+        const repoName = repositoryPath.split('/').filter(Boolean).at(-1) ?? repositoryPath;
+        nodes.push({
+            id: virtualId,
+            type: 'repositoryNode',
+            position: { x: 0, y: 0 },
+            data: { name: repoName, repositoryPath },
+        });
+        virtualRepos.set(repositoryPath, virtualId);
+        return virtualId;
+    }
+    // Add repo nodes (real ones)
+    for (const [nodeId, entry] of repoMap) {
+        const data = {
+            ...entry.data,
+            ...(callbacks?.onRepositoryAdd && {
+                onAdd: () => callbacks.onRepositoryAdd(nodeId),
+            }),
+            ...(callbacks?.onRepositoryClick && {
+                onClick: () => callbacks.onRepositoryClick(nodeId),
+            }),
+            ...(callbacks?.onRepositoryDelete && {
+                onDelete: callbacks.onRepositoryDelete,
+            }),
+        };
+        nodes.push({
+            id: nodeId,
+            type: 'repositoryNode',
+            position: { x: 0, y: 0 },
+            data,
+        });
+    }
+    // Process feature entries (both real and pending).
+    // Skip pendingMap entries that already exist in featureMap (reconcile may not
+    // have cleaned them up yet if the AI-generated name differs from the original).
+    const allFeatureEntries = [
+        ...[...featureMap.entries()].map(([id, e]) => [id, e, false]),
+        ...[...pendingMap.entries()]
+            .filter(([id]) => !featureMap.has(id))
+            .map(([id, e]) => [id, e, true]),
+    ];
+    for (const [nodeId, entry, isPending] of allFeatureEntries) {
+        const isCreating = entry.data.state === 'creating' || isPending;
+        const data = {
+            ...entry.data,
+            // Callbacks only for non-creating nodes
+            ...(!isCreating &&
+                callbacks?.onNodeAction && {
+                onAction: () => callbacks.onNodeAction(nodeId),
+            }),
+            ...(!isCreating &&
+                callbacks?.onNodeSettings && {
+                onSettings: () => callbacks.onNodeSettings(nodeId),
+            }),
+            ...(!isCreating &&
+                callbacks?.onFeatureDelete && {
+                onDelete: callbacks.onFeatureDelete,
+            }),
+        };
+        nodes.push({
+            id: nodeId,
+            type: 'featureNode',
+            position: { x: 0, y: 0 },
+            data,
+        });
+        // Edge derivation
+        if (entry.parentNodeId) {
+            // Dependency edge (parent→child feature)
+            edges.push({
+                id: `dep-${entry.parentNodeId}-${nodeId}`,
+                source: entry.parentNodeId,
+                target: nodeId,
+                type: 'dependencyEdge',
+            });
+        }
+        else {
+            // Repo→feature edge (matched by repositoryPath)
+            const repositoryPath = entry.data.repositoryPath;
+            const repoNodeId = repoByPath.get(repositoryPath) ?? getOrCreateVirtualRepo(repositoryPath);
+            edges.push({
+                id: `edge-${repoNodeId}-${nodeId}`,
+                source: repoNodeId,
+                target: nodeId,
+                style: { strokeDasharray: '5 5' },
+            });
+        }
+    }
+    // Inject showHandles based on whether there are any edges
+    const hasEdges = edges.length > 0;
+    for (const node of nodes) {
+        node.data.showHandles = hasEdges;
+    }
+    return { nodes, edges };
+}

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;AAmCF;;;;;;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,CAoK/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,CAoK/B"}

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

@@ -8,8 +8,7 @@ export const CANVAS_LAYOUT_DEFAULTS = {
 /** Known node-type dimensions for the canvas node types */
 const NODE_DIMENSIONS = {
     featureNode: { width: 288, height: 140 },
-    repositoryNode: { width: 288, height: 50 },
-    addRepositoryNode: { width: 224, height: 50 },
+    repositoryNode: { width: 328, height: 50 },
 };
 function getNodeSize(node, defaultSize) {
     // Prefer per-node size from data, then type-based lookup, then default