Remove EXECUTION_RESULT from Run Detail page (#196)

Signed-off-by: kerthcet <kerthcet@gmail.com>

Author: kerthcet

dashboard/src/pages/runs/[id].tsx

@@ -1,7 +1,6 @@
 import { useState } from 'react';
 import { useParams } from 'react-router-dom';
 import { useRun } from '../../hooks/use-runs';
-import { useArtifactContent } from '../../hooks/use-artifacts';
 import {
   Card,
   CardContent,
@@ -10,13 +9,10 @@ import {
   CardTitle,
 } from '../../components/ui/card';
 import { Badge } from '../../components/ui/badge';
-import { Button } from '../../components/ui/button';
 import { Skeleton } from '../../components/ui/skeleton';
 import { Tabs, TabsContent, TabsList, TabsTrigger } from '../../components/ui/tabs';
 import { TraceTimeline } from '../../components/traces/trace-timeline';
-import { ArtifactViewer } from '../../components/artifact-viewer';
 import { formatDistanceToNow } from 'date-fns';
-import { Eye } from 'lucide-react';
 import type { Status } from '../../types';
 
 const STATUS_VARIANTS: Record<Status, 'default' | 'secondary' | 'success' | 'warning' | 'destructive' | 'unknown' | 'info'> = {
@@ -33,7 +29,6 @@ export function RunDetailPage() {
 
   const { data: run, isLoading: runLoading, error: runError } = useRun(id!);
 
-  const [dialogOpen, setDialogOpen] = useState(false);
   const [activeTab, setActiveTab] = useState('overview');
 
   // Get metrics and traces from the nested run data
@@ -43,56 +38,6 @@ export function RunDetailPage() {
   const tracesLoading = runLoading;
   const tracesError = runError;
 
-  // Check if execution result exists in metadata
-  const executionResult = run?.meta?.execution_result as any;
-  const hasExecutionResult = executionResult?.path && executionResult?.file_name;
-
-  // Parse the path to extract the tag
-  let artifactTag = '';
-  if (hasExecutionResult) {
-    let tag = executionResult.path;
-
-    // If path contains ':', extract the part after the colon (the tag)
-    if (tag.includes(':')) {
-      tag = tag.split(':')[1];
-    }
-
-    // If path contains '/', it's a full path, extract just the tag part
-    if (tag.includes('/')) {
-      const parts = tag.split('/');
-      tag = parts[parts.length - 1];
-      if (tag.includes(':')) {
-        tag = tag.split(':')[1];
-      }
-    }
-
-    artifactTag = tag;
-  }
-
-  // Use the cached artifact content hook
-  // Only fetch when dialog is open to avoid unnecessary requests
-  // Repository name is just 'execution' - the backend prepends teamId
-  const {
-    data: artifactContent,
-    isLoading: loadingArtifact,
-    error: artifactError
-  } = useArtifactContent(
-    run?.teamId || '',
-    artifactTag,
-    'execution',
-    dialogOpen && hasExecutionResult // Only fetch when dialog is open
-  );
-
-  const handleViewArtifact = () => {
-    if (!hasExecutionResult || !run) return;
-    setDialogOpen(true);
-  };
-
-  // Show error if artifact fetch fails
-  if (artifactError && dialogOpen) {
-    console.error('Failed to load artifact:', artifactError);
-  }
-
   if (runLoading) {
     return (
       <div className="space-y-4">
@@ -149,21 +94,6 @@ export function RunDetailPage() {
         <CardContent className="p-4">
           <h3 className="text-base font-semibold mb-3">Overview</h3>
           <dl className="grid grid-cols-3 gap-3 text-sm">
-            {hasExecutionResult && (
-              <div>
-                <dt className="text-xs font-medium uppercase tracking-wide text-muted-foreground">Execution Result</dt>
-                <dd className="mt-1.5 text-foreground text-sm">
-                  <button
-                    onClick={handleViewArtifact}
-                    disabled={loadingArtifact}
-                    className="inline-flex items-center gap-1.5 text-blue-600 hover:text-blue-800 dark:text-blue-400 dark:hover:text-blue-300 hover:underline"
-                  >
-                    <Eye className="h-3.5 w-3.5" />
-                    {executionResult.file_name}
-                  </button>
-                </dd>
-              </div>
-            )}
             <div>
               <dt className="text-xs font-medium uppercase tracking-wide text-muted-foreground">Tokens</dt>
               <dd className="mt-1.5 text-foreground font-mono text-sm">
@@ -272,18 +202,6 @@ export function RunDetailPage() {
           )}
         </TabsContent>
       </Tabs>
-
-      {/* Artifact Content Viewer */}
-      <ArtifactViewer
-        open={dialogOpen}
-        onOpenChange={setDialogOpen}
-        artifactContent={artifactContent}
-        isLoading={loadingArtifact}
-        error={artifactError}
-        title="Artifact Content"
-        hideLineCount={true}
-        hideCloseButton={true}
-      />
     </div>
   );
 }