@geminilight/mindos 0.6.52 → 0.6.54

package/_standalone/components/settings/SettingsContent.tsx

@@ -66,7 +66,7 @@ export default function SettingsContent({ visible, initialTab, variant, onClose
 
     if (justOpened) {
       apiFetch<SettingsData>('/api/settings').then(d => { setData(d); dataLoaded.current = true; }).catch(() => setStatus('load-error'));
-      setFont(localStorage.getItem('prose-font') ?? 'lora');
+      setFont(localStorage.getItem('prose-font') === 'geist' ? 'inter' : localStorage.getItem('prose-font') ?? 'lora');
       setFontSize(localStorage.getItem('prose-font-size') ?? '15px');
       setContentWidth(localStorage.getItem('content-width') ?? '780px');
       const stored = localStorage.getItem('theme');
@@ -90,7 +90,7 @@ export default function SettingsContent({ visible, initialTab, variant, onClose
     const fontMap: Record<string, string> = {
       'lora': "'Lora', Georgia, serif",
       'ibm-plex-sans': "'IBM Plex Sans', sans-serif",
-      'geist': 'var(--font-geist-sans), sans-serif',
+      'inter': 'var(--font-inter), sans-serif',
       'ibm-plex-mono': "'IBM Plex Mono', monospace",
     };
     document.documentElement.style.setProperty('--prose-font-override', fontMap[font] ?? '');
@@ -134,13 +134,47 @@ export default function SettingsContent({ visible, initialTab, variant, onClose
     }
   }, []);
 
+  // Track unsaved data so we can flush on close/unmount
+  const pendingData = useRef<SettingsData | null>(null);
+
   useEffect(() => {
     if (!data || !dataLoaded.current) return;
+    pendingData.current = data;
     clearTimeout(saveTimer.current);
-    saveTimer.current = setTimeout(() => doSave(data), 800);
+    saveTimer.current = setTimeout(() => {
+      pendingData.current = null;
+      doSave(data);
+    }, 800);
     return () => clearTimeout(saveTimer.current);
   }, [data, doSave]);
 
+  // Flush unsaved changes when panel hides (panel variant)
+  useEffect(() => {
+    if (!visible && pendingData.current) {
+      const d = pendingData.current;
+      pendingData.current = null;
+      clearTimeout(saveTimer.current);
+      doSave(d);
+    }
+  }, [visible, doSave]);
+
+  // Flush unsaved changes on unmount (modal variant: SettingsModal returns null when !open)
+  useEffect(() => {
+    return () => {
+      if (pendingData.current) {
+        clearTimeout(saveTimer.current);
+        const d = pendingData.current;
+        pendingData.current = null;
+        apiFetch('/api/settings', {
+          method: 'POST',
+          headers: { 'Content-Type': 'application/json' },
+          body: JSON.stringify({ ai: d.ai, agent: d.agent, mindRoot: d.mindRoot, webPassword: d.webPassword, authToken: d.authToken }),
+        }).catch(() => {});
+      }
+    };
+  // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, []);
+
   const updateAi = useCallback((patch: Partial<AiSettings>) => {
     setData(d => d ? { ...d, ai: { ...d.ai, ...patch } } : d);
   }, []);
@@ -250,7 +284,7 @@ export default function SettingsContent({ visible, initialTab, variant, onClose
         <div className="flex items-center justify-between px-4 py-3 border-b border-border shrink-0">
           <div className="flex items-center gap-2 text-sm font-medium text-foreground">
             <Settings size={14} className="text-muted-foreground" />
-            <span className="text-xs font-medium text-muted-foreground uppercase tracking-wider font-display">Settings</span>
+            <span className="text-xs font-medium text-muted-foreground uppercase tracking-wider">Settings</span>
           </div>
           <div className="flex items-center gap-1.5">
             <div className="flex items-center gap-1.5 text-[10px]" role="status" aria-live="polite">
@@ -295,7 +329,7 @@ export default function SettingsContent({ visible, initialTab, variant, onClose
           </div>
           <div className="flex items-center gap-2 text-sm font-medium text-foreground">
             <Settings size={15} className="text-muted-foreground" />
-            <span className="font-display">{t.settings.title}</span>
+            <span>{t.settings.title}</span>
           </div>
           <div className="flex items-center gap-1.5">
             <div className="flex items-center gap-1.5 text-[10px]" role="status" aria-live="polite">
@@ -345,7 +379,7 @@ export default function SettingsContent({ visible, initialTab, variant, onClose
         <div className="w-[180px] shrink-0 border-r border-border flex flex-col">
           <div className="flex items-center gap-2 px-4 py-3 border-b border-border">
             <Settings size={15} className="text-muted-foreground" />
-            <span className="text-sm font-medium font-display text-foreground">{t.settings.title}</span>
+            <span className="text-sm font-medium text-foreground">{t.settings.title}</span>
           </div>
           <nav className="flex-1 overflow-y-auto py-1.5">
             {TABS.map(tabItem => (
@@ -383,7 +417,7 @@ export default function SettingsContent({ visible, initialTab, variant, onClose
         <div className="flex-1 flex flex-col min-w-0 min-h-0">
           {/* Right header: tab title + status + close */}
           <div className="flex items-center justify-between px-5 py-3 border-b border-border shrink-0">
-            <span className="text-sm font-medium text-foreground font-display">{activeTabLabel}</span>
+            <span className="text-sm font-medium text-foreground">{activeTabLabel}</span>
             <div className="flex items-center gap-1.5">
               <div className="flex items-center gap-1.5 text-[10px]" role="status" aria-live="polite">
                 {saving && <Loader2 size={12} className="animate-spin text-muted-foreground" />}

package/_standalone/components/settings/types.ts

@@ -51,12 +51,17 @@ export const FONT_SIZES = [
 export const FONTS = [
   { value: 'lora', label: 'Lora', category: 'Serif', style: { fontFamily: 'Lora, Georgia, serif' } },
   { value: 'ibm-plex-sans', label: 'IBM Plex Sans', category: 'Sans', style: { fontFamily: "'IBM Plex Sans', sans-serif" } },
-  { value: 'geist', label: 'Geist', category: 'Sans', style: { fontFamily: 'var(--font-geist-sans), sans-serif' } },
+  { value: 'inter', label: 'Inter', category: 'Sans', style: { fontFamily: 'var(--font-inter), sans-serif' } },
   { value: 'ibm-plex-mono', label: 'IBM Plex Mono', category: 'Mono', style: { fontFamily: "'IBM Plex Mono', monospace" } },
 ];
 
 /* ── MCP Types ────────────────────────────────────────────────── */
 
+export interface ConnectionMode {
+  cli: boolean;
+  mcp: boolean;
+}
+
 export interface McpStatus {
   running: boolean;
   transport: string;
@@ -67,6 +72,7 @@ export interface McpStatus {
   maskedToken?: string;
   authToken?: string;
   localIP?: string | null;
+  connectionMode?: ConnectionMode;
 }
 
 export interface AgentInfo {

package/_standalone/components/walkthrough/WalkthroughOverlay.tsx

@@ -2,7 +2,7 @@
 
 import { useState, useEffect, useCallback, useId } from 'react';
 import { useLocale } from '@/lib/stores/locale-store';
-import { useWalkthrough } from '@/lib/stores/walkthrough-store';
+import { useWalkthrough, useWalkthroughStore } from '@/lib/stores/walkthrough-store';
 import { walkthroughSteps } from './steps';
 import WalkthroughTooltip from './WalkthroughTooltip';
 
@@ -68,16 +68,31 @@ export default function WalkthroughOverlay() {
     return () => window.removeEventListener('keydown', handler, true);
   }, [skipFn]);
 
-  // If target element doesn't exist (e.g. Echo not enabled), auto-skip this step
+  // If target element doesn't exist (e.g. panel not enabled), auto-skip
+  // after a grace period. Use stable store ref to avoid re-triggering on
+  // every store change (which caused spurious skips during step transitions).
+  const currentStep = wt?.currentStep ?? 0;
   useEffect(() => {
     if (!step) return;
-    const el = document.querySelector(`[data-walkthrough="${step.anchor}"]`);
-    if (!el) {
-      // Target not in DOM — skip to next step after a brief delay
-      const timer = setTimeout(() => wt?.next(), 100);
-      return () => clearTimeout(timer);
-    }
-  }, [step, wt]);
+    // Wait 500ms for DOM to settle (sidebar animations, lazy rendering)
+    const timer = setTimeout(() => {
+      const el = document.querySelector(`[data-walkthrough="${step.anchor}"]`);
+      if (!el) {
+        useWalkthroughStore.getState().next();
+      }
+    }, 500);
+    return () => clearTimeout(timer);
+  // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [currentStep]); // Only re-run when step index changes, not on every store update
+
+  // Safety timeout: if overlay stays for 60s without user interaction, auto-dismiss.
+  // Prevents permanently stuck overlay from any unforeseen edge case.
+  useEffect(() => {
+    const timer = setTimeout(() => {
+      useWalkthroughStore.getState().skip();
+    }, 60_000);
+    return () => clearTimeout(timer);
+  }, [currentStep]); // Reset on each step change
 
   if (!wt || !step) return null;
 
@@ -138,7 +153,7 @@ export default function WalkthroughOverlay() {
         />
       </svg>
 
-      {/* Tooltip */}
+      {/* Tooltip — shown when target element found */}
       {targetRect && (
         <WalkthroughTooltip
           stepIndex={wt.currentStep}
@@ -146,6 +161,20 @@ export default function WalkthroughOverlay() {
           position={step.position}
         />
       )}
+
+      {/* Fallback dismiss button — shown when target element NOT found.
+          Without this, user sees dark overlay but no visible UI to dismiss it. */}
+      {!targetRect && (
+        <div className="fixed inset-0 z-[101] flex items-center justify-center pointer-events-none">
+          <button
+            onClick={wt.skip}
+            className="pointer-events-auto px-4 py-2 text-sm rounded-lg font-medium transition-all hover:opacity-90"
+            style={{ background: 'var(--amber)', color: 'var(--amber-foreground)' }}
+          >
+            {wt.currentStep === wt.totalSteps - 1 ? '✓' : '→'}
+          </button>
+        </div>
+      )}
     </>
   );
 }
@@ -195,7 +224,7 @@ function MobileWalkthroughSheet() {
         </div>
 
         <h3
-          className="text-base font-semibold font-display mb-1"
+          className="text-base font-semibold mb-1"
           style={{ color: 'var(--foreground)' }}
         >
           {stepData.title}

package/_standalone/components/walkthrough/WalkthroughTooltip.tsx

@@ -72,7 +72,7 @@ export default function WalkthroughTooltip({ stepIndex, rect, position }: Walkth
 
         {/* Content */}
         <h3
-          className="text-sm font-semibold font-display mb-1"
+          className="text-sm font-semibold mb-1"
           style={{ color: 'var(--foreground)' }}
         >
           {stepData.title}

package/_standalone/hooks/useAiOrganize.ts

@@ -2,6 +2,7 @@
 
 import { useState, useCallback, useRef } from 'react';
 import type { LocalAttachment, Message } from '@/lib/types';
+import type { OrganizeSource } from '@/lib/organize-history';
 
 // ---------------------------------------------------------------------------
 // Types
@@ -248,8 +249,11 @@ export function useAiOrganize() {
   const lastEventRef = useRef<number>(0);
   const snapshotsRef = useRef<FileSnapshots>(new Map());
   const [sourceFileNames, setSourceFileNames] = useState<string[]>([]);
+  const [source, setSource] = useState<OrganizeSource>('upload');
+  const startTimeRef = useRef<number>(0);
+  const [durationMs, setDurationMs] = useState<number>(0);
 
-  const start = useCallback(async (files: LocalAttachment[], prompt: string) => {
+  const start = useCallback(async (files: LocalAttachment[], prompt: string, organizeSource: OrganizeSource = 'upload') => {
     setPhase('organizing');
     setChanges([]);
     setCurrentTool(null);
@@ -258,6 +262,9 @@ export function useAiOrganize() {
     setError(null);
     setToolCallCount(0);
     setSourceFileNames(files.map(f => f.name));
+    setSource(organizeSource);
+    startTimeRef.current = Date.now();
+    setDurationMs(0);
     snapshotsRef.current = new Map();
     lastEventRef.current = Date.now();
 
@@ -318,6 +325,7 @@ export function useAiOrganize() {
       setSummary(result.summary);
       setToolCallCount(result.toolCallCount);
       setCurrentTool(null);
+      setDurationMs(Date.now() - startTimeRef.current);
       setPhase('done');
     } catch (err) {
       if ((err as Error).name === 'AbortError') {
@@ -426,6 +434,9 @@ export function useAiOrganize() {
     setError(null);
     setToolCallCount(0);
     setSourceFileNames([]);
+    setSource('upload');
+    setDurationMs(0);
+    startTimeRef.current = 0;
     snapshotsRef.current = new Map();
     lastEventRef.current = 0;
   }, []);
@@ -439,6 +450,8 @@ export function useAiOrganize() {
     error,
     toolCallCount,
     sourceFileNames,
+    source,
+    durationMs,
     snapshots: snapshotsRef,
     start,
     abort,

package/_standalone/lib/organize-history.ts

@@ -12,6 +12,8 @@ export interface OrganizeHistoryFile {
   undone?: boolean;
 }
 
+export type OrganizeSource = 'upload' | 'drag-drop' | 'inbox-organize' | 'import-modal' | 'plugin' | 'web-clipper';
+
 export interface OrganizeHistoryEntry {
   id: string;
   /** Unix ms */
@@ -20,6 +22,10 @@ export interface OrganizeHistoryEntry {
   sourceFiles: string[];
   files: OrganizeHistoryFile[];
   status: 'completed' | 'partial' | 'undone';
+  /** How the files were ingested */
+  source?: OrganizeSource;
+  /** Processing time in milliseconds */
+  durationMs?: number;
 }
 
 const STORAGE_KEY = 'mindos:organize-history';

package/_standalone/lib/stores/walkthrough-store.ts

@@ -21,7 +21,20 @@ export interface WalkthroughStoreState {
 
 /* ── Helpers ── */
 
+const LS_KEY = 'mindos_walkthrough_done';
+
+/**
+ * Persist walkthrough state to server AND localStorage.
+ * localStorage acts as a safety net: even if the server persist fails
+ * (e.g. during update restart, network blip), the completion state
+ * survives page reload and prevents the "stuck overlay" bug.
+ */
 function persistStep(step: number, dismissed: boolean) {
+  // Local safety net — instant, survives server outage
+  if (step >= walkthroughSteps.length || dismissed) {
+    try { localStorage.setItem(LS_KEY, '1'); } catch {}
+  }
+  // Server persist — fire-and-forget (localStorage is the safety net)
   fetch('/api/setup', {
     method: 'PATCH',
     headers: { 'Content-Type': 'application/json' },
@@ -31,6 +44,11 @@ function persistStep(step: number, dismissed: boolean) {
   }).catch((err) => { console.warn('[walkthrough-store] persist failed:', err); });
 }
 
+/** Check if walkthrough was completed/dismissed (fast, sync, local) */
+function isLocallyDone(): boolean {
+  try { return localStorage.getItem(LS_KEY) === '1'; } catch { return false; }
+}
+
 /* ── Store ── */
 
 export const useWalkthroughStore = create<WalkthroughStoreState>((set, get) => {
@@ -84,12 +102,26 @@ export const useWalkthroughStore = create<WalkthroughStoreState>((set, get) => {
       // Only auto-start on desktop
       if (window.innerWidth < 768) return () => {};
 
+      // Fast local check: if walkthrough was completed/dismissed, never reactivate.
+      // This prevents the "stuck overlay" bug where server persist failed during
+      // update restart but localStorage recorded the completion.
+      if (isLocallyDone()) return () => {};
+
       fetch('/api/setup')
         .then(r => r.json())
         .then(data => {
           const gs = data.guideState;
           if (!gs) return;
-          if (gs.walkthroughDismissed) return;
+          if (gs.walkthroughDismissed) {
+            try { localStorage.setItem(LS_KEY, '1'); } catch {} // sync local
+            return;
+          }
+
+          // If server says completed (step >= totalSteps), mark locally done
+          if (typeof gs.walkthroughStep === 'number' && gs.walkthroughStep >= totalSteps) {
+            try { localStorage.setItem(LS_KEY, '1'); } catch {};
+            return;
+          }
 
           if (gs.active && !gs.dismissed && gs.walkthroughStep === undefined) {
             if (isWelcome) {