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

package/web/.next/standalone/src/presentation/web/components/features/features-canvas/features-canvas.stories.tsx

@@ -4,7 +4,6 @@ import { FeaturesCanvas } from './features-canvas';
 import type { CanvasNodeType } from './features-canvas';
 import type { FeatureNodeType } from '@/components/common/feature-node';
 import type { RepositoryNodeType } from '@/components/common/repository-node';
-import type { AddRepositoryNodeType } from '@/components/common/add-repository-node';
 import { layoutWithDagre } from '@/lib/layout-with-dagre';
 
 const meta: Meta<typeof FeaturesCanvas> = {
@@ -229,8 +228,6 @@ export const Interactive: Story = {
       { id: 'e3-4', source: 'node-3', target: 'node-4' },
     ],
     onAddFeature: () => undefined,
-    onNodeAction: () => undefined,
-    onNodeSettings: () => undefined,
   },
 };
 
@@ -243,16 +240,8 @@ const repoNode: RepositoryNodeType = {
   },
 };
 
-const addRepoNode: AddRepositoryNodeType = {
-  id: 'add-repo',
-  type: 'addRepositoryNode',
-  position: { x: 50, y: 302 },
-  data: {},
-};
-
 const repoFeatureNodes: CanvasNodeType[] = [
   repoNode,
-  addRepoNode,
   {
     id: 'feat-1',
     type: 'featureNode',
@@ -296,8 +285,6 @@ export const ConnectedRepositoryMultipleFeatures: Story = {
   args: {
     nodes: repoFeatureNodes,
     edges: repoFeatureEdges,
-    onNodeAction: () => undefined,
-    onNodeSettings: () => undefined,
   },
 };
 
@@ -310,12 +297,6 @@ const singleRepoSingleFeatureNodes: CanvasNodeType[] = [
     position: { x: 50, y: 127 },
     data: { name: 'shep-ai/cli' },
   },
-  {
-    id: 'add-repo',
-    type: 'addRepositoryNode',
-    position: { x: 50, y: 267 },
-    data: {},
-  },
   {
     id: 'feat-1',
     type: 'featureNode',
@@ -339,8 +320,6 @@ export const ConnectedRepositorySingleFeature: Story = {
     edges: [
       { id: 'e-repo-f1', source: 'repo-1', target: 'feat-1', type: 'straight', ...dashedEdge },
     ],
-    onNodeAction: () => undefined,
-    onNodeSettings: () => undefined,
   },
 };
 
@@ -353,12 +332,6 @@ const singleRepoMultiFeatNodes: CanvasNodeType[] = [
     position: { x: 50, y: 162 },
     data: { name: 'shep-ai/cli' },
   },
-  {
-    id: 'add-repo',
-    type: 'addRepositoryNode',
-    position: { x: 50, y: 450 },
-    data: {},
-  },
   {
     id: 'feat-1',
     type: 'featureNode',
@@ -414,8 +387,6 @@ export const SingleRepositoryMultipleFeatures: Story = {
       { id: 'e-r1-f2', source: 'repo-1', target: 'feat-2', ...dashedEdge },
       { id: 'e-r1-f3', source: 'repo-1', target: 'feat-3', ...dashedEdge },
     ],
-    onNodeAction: () => undefined,
-    onNodeSettings: () => undefined,
   },
 };
 
@@ -434,12 +405,6 @@ const multiRepoNodes: CanvasNodeType[] = [
     position: { x: 50, y: 270 },
     data: { name: 'shep-ai/web' },
   },
-  {
-    id: 'add-repo',
-    type: 'addRepositoryNode',
-    position: { x: 50, y: 470 },
-    data: {},
-  },
   {
     id: 'feat-1',
     type: 'featureNode',
@@ -479,8 +444,6 @@ export const MultipleRepositories: Story = {
       { id: 'e-r1-f1', source: 'repo-1', target: 'feat-1', ...dashedEdge },
       { id: 'e-r2-f2', source: 'repo-2', target: 'feat-2', ...dashedEdge },
     ],
-    onNodeAction: () => undefined,
-    onNodeSettings: () => undefined,
   },
 };
 
@@ -505,12 +468,6 @@ const mixedRepoFeatureNodesRaw: CanvasNodeType[] = [
     position: { x: 0, y: 0 },
     data: { name: 'shep-ai/api' },
   },
-  {
-    id: 'add-repo',
-    type: 'addRepositoryNode',
-    position: { x: 0, y: 0 },
-    data: {},
-  },
   {
     id: 'feat-1',
     type: 'featureNode',
@@ -593,8 +550,6 @@ export const MultipleRepositoriesMixedConnections: Story = {
   args: {
     nodes: mixedLayoutedNodes,
     edges: mixedLayoutedEdges,
-    onNodeAction: () => undefined,
-    onNodeSettings: () => undefined,
   },
 };
 
@@ -603,8 +558,5 @@ export const InteractiveWithRepository: Story = {
     nodes: repoFeatureNodes,
     edges: repoFeatureEdges,
     onAddFeature: () => undefined,
-    onNodeAction: () => undefined,
-    onNodeSettings: () => undefined,
-    onRepositoryAdd: () => undefined,
   },
 };

package/web/.next/standalone/src/presentation/web/components/features/features-canvas/features-canvas.tsx

@@ -1,7 +1,7 @@
 'use client';
 
-import { useCallback, useMemo } from 'react';
-import { ReactFlow, Background, Controls, ReactFlowProvider } from '@xyflow/react';
+import { useMemo, useCallback } from 'react';
+import { ReactFlow, Background, Controls } from '@xyflow/react';
 import type { Connection, Edge, NodeChange } from '@xyflow/react';
 import { Plus } from 'lucide-react';
 import { Button } from '@/components/ui/button';
@@ -10,11 +10,9 @@ import { FeatureNode } from '@/components/common/feature-node';
 import type { FeatureNodeType, FeatureNodeData } from '@/components/common/feature-node';
 import { RepositoryNode } from '@/components/common/repository-node';
 import type { RepositoryNodeType } from '@/components/common/repository-node';
-import { AddRepositoryNode } from '@/components/common/add-repository-node';
-import type { AddRepositoryNodeType } from '@/components/common/add-repository-node';
 import { DependencyEdge } from './dependency-edge';
 
-export type CanvasNodeType = FeatureNodeType | RepositoryNodeType | AddRepositoryNodeType;
+export type CanvasNodeType = FeatureNodeType | RepositoryNodeType;
 
 export interface FeaturesCanvasProps {
   nodes: CanvasNodeType[];
@@ -22,16 +20,9 @@ export interface FeaturesCanvasProps {
   selectedFeatureId?: string | null;
   onNodesChange?: (changes: NodeChange<CanvasNodeType>[]) => void;
   onAddFeature?: () => void;
-  onNodeAction?: (nodeId: string) => void;
-  onNodeSettings?: (nodeId: string) => void;
   onNodeClick?: (event: React.MouseEvent, node: CanvasNodeType) => void;
   onPaneClick?: (event: React.MouseEvent) => void;
-  onRepositoryAdd?: (repoNodeId: string) => void;
-  onRepositoryClick?: (nodeId: string) => void;
-  onRepositoryDelete?: (repositoryId: string) => void;
-  onFeatureDelete?: (featureId: string) => void;
   onConnect?: (connection: Connection) => void;
-  onRepositorySelect?: (path: string) => void;
   onCanvasDrag?: () => void;
   toolbar?: React.ReactNode;
   emptyState?: React.ReactNode;
@@ -43,16 +34,9 @@ export function FeaturesCanvas({
   selectedFeatureId,
   onNodesChange,
   onAddFeature,
-  onNodeAction,
-  onNodeSettings,
   onConnect,
   onNodeClick,
   onPaneClick,
-  onRepositoryAdd,
-  onRepositoryClick,
-  onRepositoryDelete,
-  onFeatureDelete,
-  onRepositorySelect,
   onCanvasDrag,
   toolbar,
   emptyState,
@@ -61,7 +45,6 @@ export function FeaturesCanvas({
     () => ({
       featureNode: FeatureNode,
       repositoryNode: RepositoryNode,
-      addRepositoryNode: AddRepositoryNode,
     }),
     []
   );
@@ -89,48 +72,19 @@ export function FeaturesCanvas({
     [nodes, edges]
   );
 
+  // Callbacks and showHandles are already injected into node.data by deriveGraph.
+  // Only selectedFeatureId highlighting needs to be applied here.
   const enrichedNodes = useMemo(
     () =>
-      nodes.map((node) => ({
-        ...node,
-        // React Flow passes `selected` as a prop to the node component
-        ...(node.type === 'featureNode' && {
-          selected:
-            selectedFeatureId != null &&
-            (node.data as FeatureNodeData).featureId === selectedFeatureId,
-        }),
-        data: {
-          ...node.data,
-          showHandles: edges.length > 0,
-          ...(node.type === 'featureNode' && (node.data as FeatureNodeData).state !== 'creating'
-            ? {
-                onAction: onNodeAction ? () => onNodeAction(node.id) : undefined,
-                onSettings: onNodeSettings ? () => onNodeSettings(node.id) : undefined,
-                onDelete: onFeatureDelete,
-              }
-            : {}),
-          ...(node.type === 'repositoryNode' && {
-            onAdd: onRepositoryAdd ? () => onRepositoryAdd(node.id) : undefined,
-            onClick: onRepositoryClick ? () => onRepositoryClick(node.id) : undefined,
-            onDelete: onRepositoryDelete,
-          }),
-          ...(node.type === 'addRepositoryNode' && {
-            onSelect: onRepositorySelect ? (path: string) => onRepositorySelect(path) : undefined,
-          }),
-        },
-      })) as CanvasNodeType[],
-    [
-      nodes,
-      edges.length,
-      selectedFeatureId,
-      onNodeAction,
-      onNodeSettings,
-      onFeatureDelete,
-      onRepositoryAdd,
-      onRepositoryClick,
-      onRepositoryDelete,
-      onRepositorySelect,
-    ]
+      selectedFeatureId == null
+        ? nodes
+        : (nodes.map((node) =>
+            node.type === 'featureNode' &&
+            (node.data as FeatureNodeData).featureId === selectedFeatureId
+              ? { ...node, selected: true }
+              : node
+          ) as CanvasNodeType[]),
+    [nodes, selectedFeatureId]
   );
 
   if (nodes.length === 0) {
@@ -163,28 +117,26 @@ export function FeaturesCanvas({
       data-no-drawer-close
       className="pointer-events-auto h-full w-full"
     >
-      <ReactFlowProvider>
-        <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}
-        >
-          <Background />
-          <Controls />
-          {toolbar}
-        </ReactFlow>
-      </ReactFlowProvider>
+      <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}
+      >
+        <Background />
+        <Controls />
+        {toolbar}
+      </ReactFlow>
     </div>
   );
 }

package/web/.next/standalone/src/presentation/web/components/layouts/app-shell/app-shell.tsx

@@ -4,7 +4,7 @@ import { useCallback, type ReactNode } from 'react';
 import { useRouter } from 'next/navigation';
 import { SidebarProvider, SidebarInset } from '@/components/ui/sidebar';
 import { AppSidebar } from '@/components/layouts/app-sidebar';
-import { AddRepositoryButton } from '@/components/common/add-repository-node';
+import { AddRepositoryButton } from '@/components/common/add-repository-button';
 import { ThemeToggle } from '@/components/common/theme-toggle';
 import { SoundToggle } from '@/components/common/sound-toggle';
 import { AgentEventsProvider } from '@/hooks/agent-events-provider';

package/web/.next/standalone/src/presentation/web/components/layouts/app-sidebar/app-sidebar.tsx

@@ -37,17 +37,9 @@ export interface AppSidebarProps {
   features: FeatureItem[];
   onNewFeature?: () => void;
   onFeatureClick?: (featureId: string) => void;
-  onFeaturesFolderClick?: () => void;
-  onFeaturesMenuClick?: () => void;
 }
 
-export function AppSidebar({
-  features,
-  onNewFeature,
-  onFeatureClick,
-  onFeaturesFolderClick,
-  onFeaturesMenuClick,
-}: AppSidebarProps) {
+export function AppSidebar({ features, onNewFeature, onFeatureClick }: AppSidebarProps) {
   const pathname = usePathname();
   const { state } = useSidebar();
   const collapsed = state === 'collapsed';
@@ -109,11 +101,7 @@ export function AppSidebar({
               expandedVisible ? 'opacity-100' : 'opacity-0',
             ].join(' ')}
           >
-            <SidebarSectionHeader
-              label="Features"
-              onFolderClick={onFeaturesFolderClick}
-              onMenuClick={onFeaturesMenuClick}
-            />
+            <SidebarSectionHeader label="Features" />
             <ScrollArea>
               {grouped.map(({ key, label, items }) =>
                 items.length > 0 ? (

package/web/.next/standalone/src/presentation/web/components/ui/scroll-area.tsx

@@ -18,7 +18,7 @@ function ScrollArea({
     >
       <ScrollAreaPrimitive.Viewport
         data-slot="scroll-area-viewport"
-        className="focus-visible:ring-ring/50 size-full rounded-[inherit] transition-[color,box-shadow] outline-none focus-visible:ring-[3px] focus-visible:outline-1"
+        className="focus-visible:ring-ring/50 size-full rounded-[inherit] transition-[color,box-shadow] outline-none focus-visible:ring-[3px] focus-visible:outline-1 [&>div]:block!"
       >
         {children}
       </ScrollAreaPrimitive.Viewport>