@shepai/cli 1.72.7 → 1.72.8-pr216.5a2ec32

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' },
+      },
+    },
+  },
+};

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

@@ -0,0 +1,313 @@
+'use client';
+
+import { useState, useRef, useEffect, useCallback } from 'react';
+import {
+  Copy,
+  Check,
+  ExternalLink,
+  Download,
+  Rocket,
+  Loader2,
+  Monitor,
+  Terminal,
+  GitBranch,
+  Package,
+} from 'lucide-react';
+import { Button } from '@/components/ui/button';
+import { BaseDrawer } from '@/components/common/base-drawer/base-drawer';
+import { useToolInstallStream } from '@/hooks/use-tool-install-stream';
+import type { ToolItem } from '@shepai/core/application/use-cases/tools/list-tools.use-case';
+
+const PLATFORM_LABELS: Record<string, string> = {
+  linux: 'Linux',
+  darwin: 'macOS',
+  win32: 'Windows',
+};
+
+export interface ToolDetailDrawerProps {
+  tool: ToolItem;
+  open: boolean;
+  onClose: () => void;
+  onRefresh?: () => Promise<void>;
+  autoStart?: boolean;
+}
+
+const TAG_CONFIG: Record<string, { label: string; icon: typeof Monitor }> = {
+  ide: { label: 'IDE', icon: Monitor },
+  'cli-agent': { label: 'CLI Agent', icon: Terminal },
+  vcs: { label: 'VCS', icon: GitBranch },
+  terminal: { label: 'Terminal', icon: Terminal },
+};
+
+const AUTO_SCROLL_THRESHOLD = 50;
+
+function StatusBadge({ status }: { status: string }) {
+  if (status === 'available') {
+    return (
+      <span className="inline-flex items-center gap-1 rounded-full bg-emerald-100 px-2 py-0.5 text-xs font-medium text-emerald-700 dark:bg-emerald-900/30 dark:text-emerald-400">
+        <Check className="h-3 w-3" />
+        Installed
+      </span>
+    );
+  }
+  if (status === 'error') {
+    return (
+      <span className="inline-flex items-center gap-1 rounded-full bg-red-100 px-2 py-0.5 text-xs font-medium text-red-700 dark:bg-red-900/30 dark:text-red-400">
+        Error
+      </span>
+    );
+  }
+  return (
+    <span className="inline-flex items-center gap-1 rounded-full bg-amber-100 px-2 py-0.5 text-xs font-medium text-amber-700 dark:bg-amber-900/30 dark:text-amber-400">
+      Missing
+    </span>
+  );
+}
+
+export function ToolDetailDrawer({
+  tool,
+  open,
+  onClose,
+  onRefresh,
+  autoStart,
+}: ToolDetailDrawerProps) {
+  const [copied, setCopied] = useState(false);
+  const [isLaunching, setIsLaunching] = useState(false);
+  const { logs, status, startInstall } = useToolInstallStream(tool.id);
+  const logRef = useRef<HTMLDivElement>(null);
+  const isAtBottomRef = useRef(true);
+  const autoStartedRef = useRef(false);
+
+  const isInstalled = tool.status.status === 'available';
+  const isError = tool.status.status === 'error';
+  const canLaunch = isInstalled && Boolean(tool.openDirectory);
+  const canInstall = !isInstalled && !isError && tool.autoInstall;
+
+  // Auto-start install when drawer opens with autoStart flag
+  useEffect(() => {
+    if (open && autoStart && canInstall && status === 'idle' && !autoStartedRef.current) {
+      autoStartedRef.current = true;
+      startInstall();
+    }
+    if (!open) {
+      autoStartedRef.current = false;
+    }
+  }, [open, autoStart, canInstall, status, startInstall]);
+
+  function handleCopy() {
+    if (!tool.installCommand) return;
+    void navigator.clipboard.writeText(tool.installCommand).then(() => {
+      setCopied(true);
+      setTimeout(() => setCopied(false), 2000);
+    });
+  }
+
+  async function handleLaunch() {
+    setIsLaunching(true);
+    try {
+      await fetch(`/api/tools/${tool.id}/launch`, { method: 'POST' });
+    } finally {
+      setIsLaunching(false);
+    }
+  }
+
+  const handleScroll = useCallback(() => {
+    const el = logRef.current;
+    if (!el) return;
+    isAtBottomRef.current =
+      el.scrollTop + el.clientHeight >= el.scrollHeight - AUTO_SCROLL_THRESHOLD;
+  }, []);
+
+  useEffect(() => {
+    if (isAtBottomRef.current && logRef.current) {
+      logRef.current.scrollTop = logRef.current.scrollHeight;
+    }
+  }, [logs, status]);
+
+  // Refresh tool status after install completes
+  useEffect(() => {
+    if (status === 'done' && onRefresh) {
+      void onRefresh();
+    }
+  }, [status, onRefresh]);
+
+  const header = (
+    <div className="flex flex-col gap-1.5 pr-6">
+      <div className="flex items-center gap-2">
+        {tool.iconUrl ? (
+          /* eslint-disable-next-line @next/next/no-img-element */
+          <img src={tool.iconUrl} alt="" width={24} height={24} className="shrink-0 dark:invert" />
+        ) : (
+          <Package className="text-muted-foreground h-6 w-6 shrink-0" />
+        )}
+        <h2 className="text-base font-bold">{tool.name}</h2>
+        <StatusBadge status={tool.status.status} />
+      </div>
+      <div className="flex items-center gap-2">
+        {tool.tags.map((tag) => {
+          const config = TAG_CONFIG[tag] ?? { label: tag, icon: Monitor };
+          const TagIcon = config.icon;
+          return (
+            <span
+              key={tag}
+              className="text-muted-foreground/70 inline-flex items-center gap-0.5 text-[10px]"
+            >
+              <TagIcon className="h-3 w-3" />
+              {config.label}
+            </span>
+          );
+        })}
+      </div>
+    </div>
+  );
+
+  return (
+    <BaseDrawer
+      open={open}
+      onClose={onClose}
+      size="md"
+      modal
+      title={tool.name}
+      header={header}
+      data-testid="tool-detail-drawer"
+    >
+      <div className="flex flex-1 flex-col gap-4 overflow-y-auto p-4">
+        {/* Description */}
+        <p className="text-muted-foreground text-sm">{tool.description}</p>
+
+        {/* Meta info row: author + platforms */}
+        {tool.author || tool.platforms ? (
+          <div className="flex flex-wrap items-center gap-x-4 gap-y-1 text-xs">
+            {tool.author ? (
+              <span className="text-muted-foreground">
+                by{' '}
+                {tool.website ? (
+                  <a
+                    href={tool.website}
+                    target="_blank"
+                    rel="noopener noreferrer"
+                    className="text-foreground hover:underline"
+                  >
+                    {tool.author}
+                  </a>
+                ) : (
+                  <span className="text-foreground">{tool.author}</span>
+                )}
+              </span>
+            ) : null}
+            {tool.platforms && tool.platforms.length > 0 ? (
+              <span className="text-muted-foreground">
+                {tool.platforms.map((p) => PLATFORM_LABELS[p] ?? p).join(' · ')}
+              </span>
+            ) : null}
+          </div>
+        ) : null}
+
+        {/* Install Command */}
+        {tool.installCommand ? (
+          <div className="flex flex-col gap-1.5">
+            <span className="text-muted-foreground text-xs font-semibold tracking-wide uppercase">
+              Install command
+            </span>
+            <div className="relative rounded-md bg-zinc-900 text-zinc-100">
+              <pre className="overflow-x-auto px-3 py-2.5 pr-16 font-mono text-xs leading-relaxed break-all whitespace-pre-wrap">
+                <code>{tool.installCommand}</code>
+              </pre>
+              <Button
+                variant="ghost"
+                size="sm"
+                className="absolute top-1.5 right-1.5 h-7 cursor-pointer px-2 text-zinc-400 hover:text-zinc-100"
+                onClick={handleCopy}
+                aria-label={copied ? 'Copied!' : 'Copy install command'}
+              >
+                {copied ? (
+                  <Check className="h-3.5 w-3.5 text-emerald-500" />
+                ) : (
+                  <Copy className="h-3.5 w-3.5" />
+                )}
+                <span className="ml-1 text-[10px]">{copied ? 'Copied!' : 'Copy'}</span>
+              </Button>
+            </div>
+          </div>
+        ) : null}
+
+        {/* Action Bar */}
+        <div className="flex items-center gap-2">
+          {canInstall ? (
+            <Button
+              size="sm"
+              onClick={startInstall}
+              disabled={status === 'streaming'}
+              aria-label={`Install ${tool.name}`}
+              className="cursor-pointer"
+            >
+              {status === 'streaming' ? (
+                <Loader2 className="mr-1.5 h-3.5 w-3.5 animate-spin" />
+              ) : (
+                <Download className="mr-1.5 h-3.5 w-3.5" />
+              )}
+              {status === 'streaming' ? 'Installing...' : 'Install'}
+            </Button>
+          ) : null}
+          {canLaunch ? (
+            <Button
+              size="sm"
+              variant="outline"
+              onClick={() => void handleLaunch()}
+              disabled={isLaunching}
+              aria-label={`Launch ${tool.name}`}
+              className="cursor-pointer"
+            >
+              {isLaunching ? (
+                <Loader2 className="mr-1.5 h-3.5 w-3.5 animate-spin" />
+              ) : (
+                <Rocket className="mr-1.5 h-3.5 w-3.5" />
+              )}
+              Launch
+            </Button>
+          ) : null}
+          {tool.documentationUrl ? (
+            <Button variant="outline" size="sm" className="cursor-pointer" asChild>
+              <a href={tool.documentationUrl} target="_blank" rel="noopener noreferrer">
+                Docs
+                <ExternalLink className="ml-1 h-3 w-3" />
+              </a>
+            </Button>
+          ) : null}
+        </div>
+
+        {/* Install Log */}
+        {(status === 'streaming' || status === 'done' || status === 'error') && logs.length > 0 ? (
+          <div className="flex flex-col gap-1.5">
+            <span className="text-muted-foreground text-xs font-semibold tracking-wide uppercase">
+              Install log
+            </span>
+            <div
+              ref={logRef}
+              onScroll={handleScroll}
+              className="max-h-60 overflow-auto rounded-md bg-zinc-900 p-3 font-mono text-xs leading-relaxed text-zinc-100"
+              data-testid="tool-install-log"
+            >
+              {logs.map((line, idx) => (
+                // eslint-disable-next-line react/no-array-index-key -- logs are append-only, never reorder
+                <div key={idx} className="break-all whitespace-pre-wrap">
+                  {line}
+                </div>
+              ))}
+              {status === 'done' ? (
+                <div className="mt-2 border-t border-zinc-700 pt-2 text-emerald-400">
+                  Installation complete
+                </div>
+              ) : null}
+              {status === 'error' ? (
+                <div className="mt-2 border-t border-zinc-700 pt-2 text-red-400">
+                  Installation failed
+                </div>
+              ) : null}
+            </div>
+          </div>
+        ) : null}
+      </div>
+    </BaseDrawer>
+  );
+}