@shepai/cli 1.72.7 → 1.72.8

package/web/.next/standalone/src/presentation/web/components/common/base-drawer/base-drawer.tsx

@@ -12,6 +12,8 @@ import {
   DrawerHeader,
   DrawerFooter,
   DrawerOverlay,
+  DrawerTitle,
+  DrawerDescription,
 } from '@/components/ui/drawer';
 import { Separator } from '@/components/ui/separator';
 import { Tooltip, TooltipContent, TooltipProvider, TooltipTrigger } from '@/components/ui/tooltip';
@@ -36,6 +38,7 @@ export interface BaseDrawerProps extends VariantProps<typeof drawerVariants> {
   modal?: boolean;
   /** When true, clicking anywhere outside the drawer closes it, ignoring `data-no-drawer-close` guards. */
   dismissOnOutsideClick?: boolean;
+  title?: string;
   header?: React.ReactNode;
   children: React.ReactNode;
   footer?: React.ReactNode;
@@ -49,6 +52,7 @@ export function BaseDrawer({
   onClose,
   modal = false,
   dismissOnOutsideClick = false,
+  title = 'Drawer',
   size,
   header,
   children,
@@ -101,8 +105,16 @@ export function BaseDrawer({
         showCloseButton={false}
         className={cn(drawerVariants({ size }), className)}
         data-testid={testId}
-        onInteractOutside={(e) => e.preventDefault()}
+        onInteractOutside={modal ? undefined : (e) => e.preventDefault()}
       >
+        {/* Visually hidden title & description required by Radix Dialog for accessibility */}
+        <DrawerTitle asChild>
+          <span className="sr-only">{title}</span>
+        </DrawerTitle>
+        <DrawerDescription asChild>
+          <span className="sr-only">{title}</span>
+        </DrawerDescription>
+
         {/* Close button */}
         <button
           type="button"

package/web/.next/standalone/src/presentation/web/components/features/index.ts

@@ -1,4 +1,4 @@
 export { FeaturesCanvas, type FeaturesCanvasProps } from './features-canvas';
 export { ToolsPageClient, type ToolsPageClientProps } from './tools';
 export { ToolCard, type ToolCardProps } from './tools';
-export { InstallInstructions, type InstallInstructionsProps } from './tools';
+export { ToolDetailDrawer, type ToolDetailDrawerProps } from './tools';

package/web/.next/standalone/src/presentation/web/components/features/tools/index.ts

@@ -1,3 +1,3 @@
 export { ToolsPageClient, type ToolsPageClientProps } from './tools-page-client';
 export { ToolCard, type ToolCardProps } from './tool-card';
-export { InstallInstructions, type InstallInstructionsProps } from './install-instructions';
+export { ToolDetailDrawer, type ToolDetailDrawerProps } from './tool-detail-drawer';

package/web/.next/standalone/src/presentation/web/components/features/tools/tool-card.stories.tsx

@@ -16,6 +16,9 @@ const installedIdeTool: ToolItem = {
   documentationUrl: 'https://cursor.sh',
   installCommand: 'brew install --cask cursor',
   status: { status: 'available', toolName: 'cursor' },
+  author: 'Anysphere',
+  website: 'https://cursor.com',
+  platforms: ['linux', 'darwin', 'win32'],
 };
 
 const installedNoLaunchTool: ToolItem = {
@@ -32,6 +35,9 @@ const installedNoLaunchTool: ToolItem = {
   documentationUrl: 'https://claude.ai',
   installCommand: 'npm install -g @anthropic-ai/claude-code',
   status: { status: 'available', toolName: 'claude-code' },
+  author: 'Anthropic',
+  website: 'https://claude.ai/code',
+  platforms: ['linux', 'darwin'],
 };
 
 const missingAutoInstallTool: ToolItem = {
@@ -48,6 +54,9 @@ const missingAutoInstallTool: ToolItem = {
   documentationUrl: 'https://aider.chat',
   installCommand: 'pip install aider-chat',
   status: { status: 'missing', toolName: 'aider' },
+  author: 'Paul Gauthier',
+  website: 'https://aider.chat',
+  platforms: ['linux', 'darwin', 'win32'],
 };
 
 const missingManualTool: ToolItem = {
@@ -64,6 +73,9 @@ const missingManualTool: ToolItem = {
   documentationUrl: 'https://code.visualstudio.com',
   installCommand: 'brew install --cask visual-studio-code',
   status: { status: 'missing', toolName: 'vscode' },
+  author: 'Microsoft',
+  website: 'https://code.visualstudio.com',
+  platforms: ['linux', 'darwin', 'win32'],
 };
 
 const cliAgentTool: ToolItem = {
@@ -80,6 +92,9 @@ const cliAgentTool: ToolItem = {
   documentationUrl: 'https://cursor.sh',
   installCommand: 'curl https://cursor.com/install -fsS | bash',
   status: { status: 'available', toolName: 'cursor-cli' },
+  author: 'Anysphere',
+  website: 'https://cursor.com',
+  platforms: ['linux', 'darwin', 'win32'],
 };
 
 const vcsTool: ToolItem = {
@@ -96,6 +111,9 @@ const vcsTool: ToolItem = {
   documentationUrl: 'https://cli.github.com/manual/',
   installCommand: 'brew install gh',
   status: { status: 'available', toolName: 'gh' },
+  author: 'GitHub',
+  website: 'https://cli.github.com',
+  platforms: ['linux', 'darwin', 'win32'],
 };
 
 const meta: Meta<typeof ToolCard> = {

package/web/.next/standalone/src/presentation/web/components/features/tools/tool-card.tsx

@@ -2,7 +2,6 @@
 
 import { useState, useTransition } from 'react';
 import {
-  ExternalLink,
   Loader2,
   Rocket,
   Download,
@@ -10,12 +9,14 @@ import {
   Terminal,
   GitBranch,
   CircleX,
-  Check,
+  Circle,
+  CheckCircle2,
   Package,
 } from 'lucide-react';
 import { Button } from '@/components/ui/button';
+import { Tooltip, TooltipContent, TooltipProvider, TooltipTrigger } from '@/components/ui/tooltip';
 import { cn } from '@/lib/utils';
-import { InstallInstructions } from './install-instructions';
+import { ToolDetailDrawer } from './tool-detail-drawer';
 import type { ToolItem } from '@shepai/core/application/use-cases/tools/list-tools.use-case';
 
 export interface ToolCardProps {
@@ -32,34 +33,39 @@ const TAG_CONFIG: Record<string, { label: string; icon: typeof Monitor }> = {
 };
 
 export function ToolCard({ tool, onRefresh, className }: ToolCardProps) {
-  const [installDrawerOpen, setInstallDrawerOpen] = useState(false);
+  const [drawerOpen, setDrawerOpen] = useState(false);
+  const [autoStartInstall, setAutoStartInstall] = useState(false);
   const [isPending, startTransition] = useTransition();
 
   const isInstalled = tool.status.status === 'available';
   const isError = tool.status.status === 'error';
   const canLaunch = isInstalled && Boolean(tool.openDirectory);
 
-  function handleLaunch() {
+  function handleLaunch(e: React.MouseEvent) {
+    e.stopPropagation();
     startTransition(async () => {
       await fetch(`/api/tools/${tool.id}/launch`, { method: 'POST' });
     });
   }
 
-  function handleAutoInstall() {
-    startTransition(async () => {
-      const res = await fetch(`/api/tools/${tool.id}/install`, { method: 'POST' });
-      if (res.ok && onRefresh) {
-        await onRefresh();
-      }
-    });
+  function handleCardClick() {
+    setAutoStartInstall(false);
+    setDrawerOpen(true);
+  }
+
+  function handleInstallClick(e: React.MouseEvent) {
+    e.stopPropagation();
+    setAutoStartInstall(tool.autoInstall);
+    setDrawerOpen(true);
   }
 
   return (
     <>
       <div
         data-testid="tool-card"
+        onClick={handleCardClick}
         className={cn(
-          'bg-card group flex h-30 w-full flex-col rounded-lg border p-3 shadow-sm transition-shadow hover:shadow-md',
+          'bg-card group flex h-30 w-full cursor-pointer flex-col rounded-lg border p-3 transition-shadow hover:shadow-md',
           className
         )}
       >
@@ -89,7 +95,7 @@ export function ToolCard({ tool, onRefresh, className }: ToolCardProps) {
               return (
                 <span
                   key={tag}
-                  className="bg-muted text-muted-foreground inline-flex items-center gap-0.5 rounded px-1.5 py-0.5 text-[10px] font-medium"
+                  className="text-muted-foreground/70 inline-flex items-center gap-0.5 text-[9px]"
                 >
                   <TagIcon className="h-2.5 w-2.5" />
                   {config.label}
@@ -117,72 +123,74 @@ export function ToolCard({ tool, onRefresh, className }: ToolCardProps) {
                 {tool.status.errorMessage ?? 'Error'}
               </span>
             ) : isInstalled ? (
-              <span className="flex items-center gap-1 text-[11px] text-emerald-600 dark:text-emerald-400">
-                <Check className="h-3 w-3" />
+              <span className="flex items-center gap-1 text-[10px] text-emerald-600 dark:text-emerald-400">
+                <CheckCircle2 className="h-3 w-3" />
                 Installed
               </span>
             ) : (
-              <span className="text-muted-foreground text-[10px]">Not installed</span>
+              <span className="text-muted-foreground flex items-center gap-1 text-[10px]">
+                <Circle className="h-3 w-3" />
+                Not installed
+              </span>
             )}
             {tool.required ? (
-              <span className="text-muted-foreground text-[10px] italic">Required</span>
+              <TooltipProvider>
+                <Tooltip>
+                  <TooltipTrigger asChild>
+                    <span className="rounded bg-amber-100 px-1.5 py-0.5 text-[9px] font-medium text-amber-700 dark:bg-amber-900/50 dark:text-amber-400">
+                      Required
+                    </span>
+                  </TooltipTrigger>
+                  <TooltipContent side="top">
+                    This tool is required for Shep to function properly
+                  </TooltipContent>
+                </Tooltip>
+              </TooltipProvider>
             ) : null}
           </div>
 
-          {/* Action button */}
-          {isInstalled && canLaunch ? (
-            <Button
-              size="sm"
-              variant="ghost"
-              onClick={handleLaunch}
-              disabled={isPending}
-              aria-label={`Launch ${tool.name}`}
-              data-testid="tool-card-launch-button"
-              className="hover:bg-primary hover:text-primary-foreground h-7 cursor-pointer rounded-md px-3 text-xs"
-            >
-              {isPending ? (
-                <Loader2 className="mr-1 h-3 w-3 animate-spin" />
-              ) : (
-                <Rocket className="mr-1 h-3 w-3" />
-              )}
-              Launch
-            </Button>
-          ) : !isInstalled && !isError && tool.autoInstall ? (
-            <Button
-              size="sm"
-              onClick={handleAutoInstall}
-              disabled={isPending}
-              aria-label={`Install ${tool.name}`}
-              data-testid="tool-card-install-button"
-              className="h-7 cursor-pointer rounded-md px-3 text-xs"
-            >
-              {isPending ? (
-                <Loader2 className="mr-1 h-3 w-3 animate-spin" />
-              ) : (
+          {/* Action buttons */}
+          <div className="flex items-center gap-1">
+            {isInstalled && canLaunch ? (
+              <Button
+                size="sm"
+                variant="outline"
+                onClick={handleLaunch}
+                disabled={isPending}
+                aria-label={`Launch ${tool.name}`}
+                data-testid="tool-card-launch-button"
+                className="h-7 cursor-pointer rounded-md px-3 text-xs"
+              >
+                {isPending ? (
+                  <Loader2 className="mr-1 h-3 w-3 animate-spin" />
+                ) : (
+                  <Rocket className="mr-1 h-3 w-3" />
+                )}
+                Launch
+              </Button>
+            ) : !isInstalled && !isError ? (
+              <Button
+                size="sm"
+                variant="default"
+                onClick={handleInstallClick}
+                aria-label={`Install ${tool.name}`}
+                data-testid="tool-card-install-button"
+                className="h-7 cursor-pointer rounded-md px-3 text-xs"
+              >
                 <Download className="mr-1 h-3 w-3" />
-              )}
-              {isPending ? 'Installing…' : 'Install'}
-            </Button>
-          ) : !isInstalled && !isError ? (
-            <Button
-              size="sm"
-              variant="outline"
-              onClick={() => setInstallDrawerOpen(true)}
-              aria-label={`View install instructions for ${tool.name}`}
-              data-testid="tool-card-manual-install-button"
-              className="h-7 cursor-pointer rounded-md px-3 text-xs"
-            >
-              <ExternalLink className="mr-1 h-3 w-3" />
-              Install
-            </Button>
-          ) : null}
+                Install
+              </Button>
+            ) : null}
+          </div>
         </div>
       </div>
 
-      <InstallInstructions
+      <ToolDetailDrawer
         tool={tool}
-        open={installDrawerOpen}
-        onOpenChange={setInstallDrawerOpen}
+        open={drawerOpen}
+        onClose={() => setDrawerOpen(false)}
+        onRefresh={onRefresh}
+        autoStart={autoStartInstall}
       />
     </>
   );

package/web/.next/standalone/src/presentation/web/components/features/tools/tool-detail-drawer.stories.tsx

@@ -0,0 +1,132 @@
+import type { Meta, StoryObj } from '@storybook/react';
+import { ToolDetailDrawer } from './tool-detail-drawer';
+import type { ToolItem } from '@shepai/core/application/use-cases/tools/list-tools.use-case';
+
+const baseTool: ToolItem = {
+  id: 'tmux',
+  name: 'tmux',
+  summary: 'Terminal multiplexer',
+  description:
+    'tmux is a terminal multiplexer for Unix-like operating systems. It allows multiple terminal sessions to be accessed simultaneously in a single window.',
+  tags: ['terminal'],
+  iconUrl: 'https://cdn.simpleicons.org/tmux',
+  autoInstall: true,
+  required: false,
+  openDirectory: 'tmux new-session -c {dir}',
+  documentationUrl: 'https://github.com/tmux/tmux/wiki',
+  installCommand: 'sudo apt-get install -y tmux',
+  status: { status: 'missing', toolName: 'tmux' },
+  author: 'tmux team',
+  website: 'https://tmux.github.io',
+  platforms: ['linux', 'darwin'],
+};
+
+const meta: Meta<typeof ToolDetailDrawer> = {
+  title: 'Features/ToolDetailDrawer',
+  component: ToolDetailDrawer,
+  parameters: {
+    layout: 'centered',
+  },
+  tags: ['autodocs'],
+  args: {
+    tool: baseTool,
+    open: true,
+    onClose: () => undefined,
+  },
+};
+
+export default meta;
+type Story = StoryObj<typeof meta>;
+
+export const Installed: Story = {
+  args: {
+    tool: {
+      ...baseTool,
+      status: { status: 'available', toolName: 'tmux' },
+    },
+    open: true,
+  },
+};
+
+export const MissingAutoInstall: Story = {
+  args: {
+    tool: {
+      ...baseTool,
+      status: { status: 'missing', toolName: 'tmux' },
+      autoInstall: true,
+    },
+    open: true,
+  },
+};
+
+export const MissingManual: Story = {
+  args: {
+    tool: {
+      ...baseTool,
+      status: { status: 'missing', toolName: 'tmux' },
+      autoInstall: false,
+    },
+    open: true,
+  },
+};
+
+export const Error: Story = {
+  args: {
+    tool: {
+      ...baseTool,
+      status: {
+        status: 'error',
+        toolName: 'tmux',
+        errorMessage: 'Permission denied: /usr/bin/tmux',
+      },
+    },
+    open: true,
+  },
+};
+
+export const WithLogs: Story = {
+  args: {
+    tool: baseTool,
+    open: true,
+  },
+  parameters: {
+    mockData: {
+      useToolInstallStream: {
+        logs: [
+          'Reading package lists...',
+          'Building dependency tree...',
+          'Reading state information...',
+          'The following NEW packages will be installed:',
+          '  tmux',
+          'Setting up tmux (3.3a-3) ...',
+        ],
+        status: 'streaming',
+        result: null,
+      },
+    },
+  },
+};
+
+export const AfterInstall: Story = {
+  args: {
+    tool: {
+      ...baseTool,
+      status: { status: 'available', toolName: 'tmux' },
+    },
+    open: true,
+  },
+  parameters: {
+    mockData: {
+      useToolInstallStream: {
+        logs: [
+          'Reading package lists...',
+          'Building dependency tree...',
+          'Setting up tmux (3.3a-3) ...',
+          'Processing triggers...',
+        ],
+        status: 'done',
+        result: { status: 'available', toolName: 'tmux' },
+      },
+    },
+  },
+};