@shepai/cli 1.72.9-pr218.000ada0 → 1.72.9-pr220.e1bfec0

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

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