@geminilight/mindos 0.6.59 → 0.6.61

package/app/components/agents/CustomAgentModal.tsx

@@ -1,7 +1,7 @@
 'use client';
 
 import { useState, useEffect, useRef, useCallback, useMemo } from 'react';
-import { X, Loader2, ChevronRight, ChevronDown, CheckCircle2, Info, AlertCircle } from 'lucide-react';
+import { X, Loader2, ChevronRight, ChevronDown, CheckCircle2, Info, AlertCircle, Server, Zap } from 'lucide-react';
 import { useLocale } from '@/lib/stores/locale-store';
 import { useFocusTrap } from '@/lib/hooks/useFocusTrap';
 import { toast } from '@/lib/toast';
@@ -19,6 +19,8 @@ interface DetectResult {
   detectedSkillDir?: string;
   skillCount?: number;
   skillNames?: string[];
+  mcpServers?: string[];
+  mcpParseError?: string;
   suggestedName?: string;
 }
 
@@ -406,7 +408,7 @@ export default function CustomAgentModal({
           {phase === 'result' && (
             <>
               {/* Result Card */}
-              <div className={`rounded-lg p-4 space-y-2 ${
+              <div className={`rounded-lg p-4 space-y-3 ${
                 detectResult?.exists
                   ? 'border border-[var(--success)]/20 bg-[var(--success)]/5'
                   : 'border border-border bg-muted/30'
@@ -414,22 +416,74 @@ export default function CustomAgentModal({
                 <p className="text-xs font-medium text-muted-foreground uppercase tracking-wider">
                   {detectResult?.exists ? p.customAgentDetectedTitle : p.customAgentDefaultTitle}
                 </p>
-                <div className="grid grid-cols-[auto_1fr] gap-x-4 gap-y-1.5 text-sm">
-                  <span className="text-muted-foreground">{p.customAgentConfigLabel}</span>
-                  <span className="text-foreground font-mono text-xs truncate">{form.global}</span>
-
-                  <span className="text-muted-foreground">{p.customAgentFormatLabel}</span>
-                  <span className="text-foreground">{form.format.toUpperCase()} · {form.configKey}</span>
-
-                  <span className="text-muted-foreground">{p.customAgentTransportLabel}</span>
-                  <span className="text-foreground">{form.preferredTransport}</span>
-
-                  <span className="text-muted-foreground">{p.customAgentSkillsLabel}</span>
-                  <span className="text-foreground">
-                    {detectResult?.hasSkillsDir
-                      ? p.customAgentSkillsFound(detectResult.skillCount ?? 0)
-                      : p.customAgentSkillsNone ?? '—'}
-                  </span>
+                <div className="space-y-2.5">
+                  {/* Config Path */}
+                  <div>
+                    <p className="text-xs text-muted-foreground mb-1">{p.customAgentConfigLabel}</p>
+                    <span className="text-sm text-foreground font-mono truncate block">{form.global}</span>
+                    {detectResult?.mcpParseError && (
+                      <p className="text-xs text-[var(--error)] mt-1 flex items-center gap-1">
+                        <AlertCircle size={12} className="shrink-0" />
+                        {detectResult.mcpParseError}
+                      </p>
+                    )}
+                  </div>
+
+                  {/* Format & Key */}
+                  <div className="grid grid-cols-2 gap-2">
+                    <div>
+                      <p className="text-xs text-muted-foreground mb-1">{p.customAgentFormatLabel}</p>
+                      <span className="text-sm text-foreground">{form.format.toUpperCase()}</span>
+                    </div>
+                    <div>
+                      <p className="text-xs text-muted-foreground mb-1">Config Key</p>
+                      <span className="text-sm text-foreground font-mono">{form.configKey}</span>
+                    </div>
+                  </div>
+
+                  {/* Transport */}
+                  <div>
+                    <p className="text-xs text-muted-foreground mb-1">{p.customAgentTransportLabel}</p>
+                    <span className="text-sm text-foreground">{form.preferredTransport}</span>
+                  </div>
+
+                  {/* MCP Servers */}
+                  <div>
+                    <p className="text-xs text-muted-foreground mb-1 flex items-center gap-1">
+                      <Server size={11} className="shrink-0" aria-hidden="true" />
+                      MCP Servers
+                    </p>
+                    {detectResult?.mcpServers && detectResult.mcpServers.length > 0 ? (
+                      <div className="flex flex-wrap gap-1">
+                        {detectResult.mcpServers.map(name => (
+                          <span key={name} className="text-xs bg-muted/60 text-foreground px-2 py-0.5 rounded font-mono">
+                            {name}
+                          </span>
+                        ))}
+                      </div>
+                    ) : (
+                      <span className="text-sm text-muted-foreground">—</span>
+                    )}
+                  </div>
+
+                  {/* Skills */}
+                  <div>
+                    <p className="text-xs text-muted-foreground mb-1 flex items-center gap-1">
+                      <Zap size={11} className="shrink-0" aria-hidden="true" />
+                      {p.customAgentSkillsLabel}
+                    </p>
+                    {detectResult?.skillNames && detectResult.skillNames.length > 0 ? (
+                      <div className="flex flex-wrap gap-1">
+                        {detectResult.skillNames.map(name => (
+                          <span key={name} className="text-xs bg-[var(--amber-dim)] text-[var(--amber-text,var(--foreground))] px-2 py-0.5 rounded font-mono">
+                            {name}
+                          </span>
+                        ))}
+                      </div>
+                    ) : (
+                      <span className="text-sm text-muted-foreground">—</span>
+                    )}
+                  </div>
                 </div>
               </div>
 

package/app/components/ask/AskContent.tsx

@@ -403,6 +403,7 @@ export default function AskContent({ visible, currentFile, initialMessage, initi
     // Accept mindos file paths and image drops
     if (e.dataTransfer.types.includes('text/mindos-path') || e.dataTransfer.types.includes('Files')) {
       e.preventDefault();
+      e.stopPropagation();
       e.dataTransfer.dropEffect = 'copy';
       setIsDragOver(true);
     }
@@ -412,6 +413,7 @@ export default function AskContent({ visible, currentFile, initialMessage, initi
 
   const handleDrop = useCallback(async (e: React.DragEvent) => {
     e.preventDefault();
+    e.stopPropagation();
     setIsDragOver(false);
     const filePath = e.dataTransfer.getData('text/mindos-path');
     if (filePath) {
@@ -422,7 +424,21 @@ export default function AskContent({ visible, currentFile, initialMessage, initi
       }
       return;
     }
-    await imageUploadRef.current.handleDrop(e);
+    // Try image upload first, then fall back to general file upload
+    const files = e.dataTransfer.files;
+    if (files.length > 0) {
+      const hasImages = Array.from(files).some(f => f.type.startsWith('image/'));
+      if (hasImages) {
+        await imageUploadRef.current.handleDrop(e);
+      }
+      // Upload non-image files as attachments
+      const nonImageFiles = Array.from(files).filter(f => !f.type.startsWith('image/'));
+      if (nonImageFiles.length > 0) {
+        const dt = new DataTransfer();
+        nonImageFiles.forEach(f => dt.items.add(f));
+        await uploadRef.current.pickFiles(dt.files);
+      }
+    }
   }, []);
 
   const handlePaste = useCallback((e: React.ClipboardEvent) => {
@@ -571,12 +587,12 @@ export default function AskContent({ visible, currentFile, initialMessage, initi
                   <FileChip key={f} path={f} variant="kb" onRemove={() => setAttachedFiles(prev => prev.filter(x => x !== f))} />
                 ))}
                 {upload.localAttachments.map((f, idx) => (
-                  <FileChip key={`up-${f.name}-${idx}`} path={f.name} variant="upload" onRemove={() => upload.removeAttachment(idx)} />
+                  <FileChip key={`up-${f.name}-${idx}`} path={f.name} variant="upload" status={f.status} error={f.error} truncatedInfo={f.truncatedInfo} onRemove={() => upload.removeAttachment(idx)} />
                 ))}
                 {imageUpload.images.map((img, idx) => (
                   <FileChip
                     key={`img-${idx}`}
-                    path={`Image ${idx + 1}`}
+                    path={img.fileName || `Image ${idx + 1}`}
                     variant="image"
                     imageData={img.data}
                     imageMime={img.mimeType}

package/app/components/ask/FileChip.tsx

@@ -2,7 +2,7 @@
 
 import { useState, useRef, useEffect } from 'react';
 import { createPortal } from 'react-dom';
-import { X, FileText, Table, Paperclip, ImageIcon, Zap, Bot, Folder } from 'lucide-react';
+import { X, FileText, Table, Paperclip, ImageIcon, Zap, Bot, Folder, Loader2, CheckCircle2, AlertCircle, AlertTriangle } from 'lucide-react';
 
 interface FileChipProps {
   path: string;
@@ -12,6 +12,16 @@ interface FileChipProps {
   imageData?: string;
   /** MIME type for image preview */
   imageMime?: string;
+  /** Extraction status for uploaded files (e.g. PDF). */
+  status?: 'loading' | 'success' | 'error';
+  /** Human-readable error message shown on hover when status='error'. */
+  error?: string;
+  /** Present when the PDF text was truncated due to length. */
+  truncatedInfo?: {
+    totalChars: number;
+    includedChars: number;
+    totalPages: number;
+  };
 }
 
 const VARIANT_ICON = {
@@ -30,11 +40,13 @@ const VARIANT_STYLE = {
   agent: 'border-[var(--amber)]/25 bg-[var(--amber)]/10 text-foreground',
 } as const;
 
-export default function FileChip({ path, onRemove, variant = 'kb', imageData, imageMime }: FileChipProps) {
+export default function FileChip({ path, onRemove, variant = 'kb', imageData, imageMime, status, error, truncatedInfo }: FileChipProps) {
   const [showPreview, setShowPreview] = useState(false);
+  const [showTooltip, setShowTooltip] = useState(false);
   const [mounted, setMounted] = useState(false);
   const chipRef = useRef<HTMLSpanElement>(null);
   const [previewPos, setPreviewPos] = useState<{ top: number; left: number } | null>(null);
+  const [tooltipPos, setTooltipPos] = useState<{ top: number; left: number } | null>(null);
 
   useEffect(() => setMounted(true), []);
 
@@ -42,23 +54,63 @@ export default function FileChip({ path, onRemove, variant = 'kb', imageData, im
   const cleanPath = isDir ? path.slice(0, -1) : path;
   const name = cleanPath.split('/').pop() ?? cleanPath;
   const isCsv = variant === 'kb' && !isDir && name.endsWith('.csv');
-  const { icon: Icon, cls: iconClass } = isDir
+
+  // Determine the base icon from variant / file type
+  const { icon: BaseIcon, cls: baseIconCls } = isDir
     ? { icon: Folder, cls: 'text-yellow-400' }
     : isCsv
     ? { icon: Table, cls: 'text-success' }
     : VARIANT_ICON[variant];
-  const style = VARIANT_STYLE[variant];
 
-  // Position preview above the chip using getBoundingClientRect
+  // Determine status trailing indicator
+  const isTruncated = !!truncatedInfo;
+  let StatusIndicator: React.ComponentType<{ size: number; className?: string }> | null = null;
+  let statusIndicatorCls = '';
+  if (status === 'loading') {
+    StatusIndicator = Loader2;
+    statusIndicatorCls = 'text-muted-foreground animate-spin';
+  } else if (status === 'error') {
+    StatusIndicator = AlertCircle;
+    statusIndicatorCls = 'text-destructive';
+  } else if (isTruncated) {
+    StatusIndicator = AlertTriangle;
+    statusIndicatorCls = 'text-amber-500';
+  } else if (status === 'success') {
+    StatusIndicator = CheckCircle2;
+    statusIndicatorCls = 'text-emerald-500';
+  }
+
+  const style = status === 'error'
+    ? 'border-destructive/30 bg-destructive/5 text-foreground'
+    : isTruncated
+    ? 'border-amber-500/30 bg-amber-500/5 text-foreground'
+    : VARIANT_STYLE[variant];
+
+  // Build tooltip text
+  let tooltipText: string | null = null;
+  if (status === 'error' && error) {
+    tooltipText = error;
+  } else if (isTruncated) {
+    const pct = truncatedInfo.totalChars > 0
+      ? Math.round((truncatedInfo.includedChars / truncatedInfo.totalChars) * 100)
+      : 0;
+    tooltipText = `Document truncated — only ${pct}% included (${Math.round(truncatedInfo.includedChars / 1000)}K / ${Math.round(truncatedInfo.totalChars / 1000)}K chars, ${truncatedInfo.totalPages} pages total)`;
+  }
+
+  // Position image preview above the chip
   useEffect(() => {
     if (!showPreview || !chipRef.current) { setPreviewPos(null); return; }
     const rect = chipRef.current.getBoundingClientRect();
-    setPreviewPos({
-      top: rect.top - 8, // 8px gap above chip
-      left: rect.left,
-    });
+    setPreviewPos({ top: rect.top - 8, left: rect.left });
   }, [showPreview]);
 
+  // Position tooltip above the chip
+  useEffect(() => {
+    if (!showTooltip || !chipRef.current) { setTooltipPos(null); return; }
+    const rect = chipRef.current.getBoundingClientRect();
+    setTooltipPos({ top: rect.top - 8, left: rect.left });
+  }, [showTooltip]);
+
   const preview = showPreview && previewPos && imageData && imageMime ? (
     <div
       className="fixed z-50 p-1 rounded-lg border border-border bg-card shadow-lg pointer-events-none"
@@ -75,16 +127,44 @@ export default function FileChip({ path, onRemove, variant = 'kb', imageData, im
     </div>
   ) : null;
 
+  const tooltipBorderCls = status === 'error'
+    ? 'border-destructive/20 text-destructive'
+    : 'border-amber-500/20 text-amber-600 dark:text-amber-400';
+
+  const tooltip = showTooltip && tooltipPos && tooltipText ? (
+    <div
+      className={`fixed z-50 px-2.5 py-1.5 rounded-lg border bg-card text-xs shadow-lg pointer-events-none max-w-[280px] ${tooltipBorderCls}`}
+      style={{
+        left: tooltipPos.left,
+        bottom: window.innerHeight - tooltipPos.top,
+      }}
+    >
+      {tooltipText}
+    </div>
+  ) : null;
+
+  const handleMouseEnter = () => {
+    if (imageData) setShowPreview(true);
+    if (tooltipText) setShowTooltip(true);
+  };
+  const handleMouseLeave = () => {
+    setShowPreview(false);
+    setShowTooltip(false);
+  };
+
   return (
     <>
       <span
         ref={chipRef}
         className={`inline-flex items-center gap-1 px-2 py-0.5 rounded-md text-xs border max-w-[220px] ${style}`}
-        onMouseEnter={() => imageData && setShowPreview(true)}
-        onMouseLeave={() => setShowPreview(false)}
+        onMouseEnter={handleMouseEnter}
+        onMouseLeave={handleMouseLeave}
       >
-        <Icon size={11} className={`${iconClass} shrink-0`} />
+        <BaseIcon size={11} className={`${baseIconCls} shrink-0`} />
         <span className="truncate" title={isDir ? cleanPath : path}>{name}</span>
+        {StatusIndicator && (
+          <StatusIndicator size={10} className={`${statusIndicatorCls} shrink-0`} />
+        )}
         <button
           type="button"
           onClick={onRemove}
@@ -95,6 +175,7 @@ export default function FileChip({ path, onRemove, variant = 'kb', imageData, im
         </button>
       </span>
       {mounted && preview && createPortal(preview, document.body)}
+      {mounted && tooltip && createPortal(tooltip, document.body)}
     </>
   );
 }

package/app/components/changes/ChangesBanner.tsx

@@ -111,7 +111,7 @@ export default function ChangesBanner() {
             <History size={14} />
           </span>
           <div className="min-w-0 flex-1">
-            <p className="text-xs text-foreground font-display whitespace-nowrap">
+            <p className="text-xs text-foreground font-medium whitespace-nowrap">
               {t.changes.unreadBanner(unreadCount)}
             </p>
             <div className="mt-1 flex items-center gap-1.5">