@geminilight/mindos 0.6.69 → 0.6.70

package/app/components/settings/McpPortSection.tsx

@@ -0,0 +1,207 @@
+'use client';
+
+import { useState, useEffect, useRef, useCallback } from 'react';
+import { Loader2, AlertTriangle, CheckCircle2, Settings } from 'lucide-react';
+import { apiFetch } from '@/lib/api';
+import { toast } from '@/lib/toast';
+
+/* ── Types ─────────────────────────────────────────────────────── */
+
+interface PortStatus {
+  checking: boolean;
+  available: boolean | null;
+  isSelf: boolean;
+  suggestion: number | null;
+  invalid?: boolean;
+}
+
+interface CheckPortResult {
+  available: boolean;
+  isSelf?: boolean;
+  suggestion?: number | null;
+}
+
+const EMPTY_STATUS: PortStatus = { checking: false, available: null, isSelf: false, suggestion: null };
+
+/* ── McpPortSection (compact, inline) ──────────────────────────── */
+
+export default function McpPortSection({ m }: { m: Record<string, any> }) {
+  const [origPort, setOrigPort] = useState<number>(0);
+  const [port, setPort] = useState<number>(0);
+  const [status, setStatus] = useState<PortStatus>(EMPTY_STATUS);
+  const [updating, setUpdating] = useState(false);
+  const timerRef = useRef<ReturnType<typeof setTimeout>>(undefined);
+  const pollRef = useRef<ReturnType<typeof setInterval>>(undefined);
+
+  useEffect(() => {
+    apiFetch<{ mcpPort?: number }>('/api/settings').then(d => {
+      const p = d.mcpPort || 8781;
+      setOrigPort(p);
+      setPort(p);
+    }).catch(() => {});
+  }, []);
+
+  useEffect(() => () => { clearTimeout(timerRef.current); clearInterval(pollRef.current); }, []);
+
+  const hasChanges = port !== origPort;
+  const portInvalid = port < 1024 || port > 65535;
+  const portUnavailable = status.checking || (status.available === false && !status.isSelf);
+
+  const checkPort = useCallback(async (p: number) => {
+    if (p < 1024 || p > 65535) {
+      setStatus({ ...EMPTY_STATUS, available: false, invalid: true });
+      return;
+    }
+    setStatus({ ...EMPTY_STATUS, checking: true });
+    try {
+      const res = await apiFetch<CheckPortResult>('/api/setup/check-port', {
+        method: 'POST',
+        headers: { 'Content-Type': 'application/json' },
+        body: JSON.stringify({ port: p }),
+      });
+      setStatus({
+        checking: false,
+        available: res.available,
+        isSelf: res.isSelf ?? false,
+        suggestion: res.suggestion ?? null,
+      });
+    } catch {
+      setStatus(EMPTY_STATUS);
+    }
+  }, []);
+
+  const handleChange = (e: React.ChangeEvent<HTMLInputElement>) => {
+    const v = parseInt(e.target.value, 10) || port;
+    setPort(v);
+    setStatus(EMPTY_STATUS);
+    clearTimeout(timerRef.current);
+    if (v >= 1024 && v <= 65535) {
+      timerRef.current = setTimeout(() => checkPort(v), 500);
+    }
+  };
+
+  const handleBlur = () => {
+    clearTimeout(timerRef.current);
+    if (port >= 1024 && port <= 65535) checkPort(port);
+  };
+
+  const handleUpdate = async () => {
+    if (!hasChanges || portInvalid || portUnavailable || updating) return;
+
+    setUpdating(true);
+    try {
+      const res = await apiFetch<CheckPortResult>('/api/setup/check-port', {
+        method: 'POST',
+        headers: { 'Content-Type': 'application/json' },
+        body: JSON.stringify({ port }),
+      });
+      if (!res.available && !res.isSelf) {
+        setStatus({ checking: false, available: false, isSelf: false, suggestion: res.suggestion ?? null });
+        setUpdating(false);
+        toast.error(m.portInUse(port));
+        return;
+      }
+
+      await apiFetch('/api/settings', {
+        method: 'POST',
+        headers: { 'Content-Type': 'application/json' },
+        body: JSON.stringify({ mcpPort: port }),
+      });
+
+      try {
+        await apiFetch('/api/mcp/restart', { method: 'POST' });
+      } catch {
+        setUpdating(false);
+        toast.error(m.portUpdateFailed);
+        return;
+      }
+
+      const deadline = Date.now() + 60_000;
+      pollRef.current = setInterval(async () => {
+        if (Date.now() > deadline) {
+          clearInterval(pollRef.current);
+          setUpdating(false);
+          toast.error(m.portRestartTimeout);
+          return;
+        }
+        try {
+          const s = await apiFetch<{ running: boolean; port: number }>('/api/mcp/status', { timeout: 3000 });
+          if (s.running) {
+            clearInterval(pollRef.current);
+            setUpdating(false);
+            setOrigPort(port);
+            toast.success(m.portUpdateSuccess);
+          }
+        } catch { /* keep polling */ }
+      }, 3000);
+    } catch {
+      setUpdating(false);
+      toast.error(m.portUpdateFailed);
+    }
+  };
+
+  if (origPort === 0) return null;
+
+  return (
+    <div className="rounded-lg border border-border overflow-hidden">
+      <div className="flex items-center gap-2 px-3 py-2 bg-muted/30">
+        <Settings size={11} className="text-muted-foreground" />
+        <span className="text-xs font-medium text-muted-foreground">{m.mcpPortLabel}</span>
+      </div>
+      <div className="px-3 py-2.5 space-y-2">
+        <div className="flex items-center gap-2">
+          <input
+            type="number" min={1024} max={65535} value={port}
+            onChange={handleChange}
+            onBlur={handleBlur}
+            className="flex-1 px-2.5 py-1.5 text-sm rounded-lg border border-border bg-muted/30 text-foreground focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring tabular-nums"
+          />
+          <button
+            type="button"
+            onClick={handleUpdate}
+            disabled={!hasChanges || portInvalid || portUnavailable || updating}
+            className="px-3 py-1.5 rounded-lg text-xs font-medium transition-colors shrink-0
+              bg-[var(--amber)] text-[var(--amber-foreground)]
+              hover:opacity-90
+              disabled:opacity-40 disabled:cursor-not-allowed
+              focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring"
+          >
+            {updating ? <Loader2 size={12} className="animate-spin" /> : (m.portUpdateBtn)}
+          </button>
+        </div>
+
+        {/* Status feedback */}
+        {status.checking && (
+          <p className="text-2xs flex items-center gap-1 text-muted-foreground">
+            <Loader2 size={10} className="animate-spin" /> {m.portChecking}
+          </p>
+        )}
+        {!status.checking && status.available === false && !status.invalid && (
+          <div className="flex items-center gap-2">
+            <p className="text-2xs flex items-center gap-1 text-[var(--amber)]">
+              <AlertTriangle size={10} /> {m.portInUse(port)}
+            </p>
+            {status.suggestion !== null && (
+              <button type="button"
+                onClick={() => { setPort(status.suggestion!); setStatus(EMPTY_STATUS); setTimeout(() => checkPort(status.suggestion!), 0); }}
+                className="text-2xs px-1.5 py-0.5 rounded border border-[var(--amber)] text-[var(--amber)] transition-colors hover:bg-[var(--amber-subtle)]"
+              >
+                {m.portSuggest(status.suggestion)}
+              </button>
+            )}
+          </div>
+        )}
+        {!status.checking && status.invalid && (
+          <p className="text-2xs flex items-center gap-1 text-destructive">
+            <AlertTriangle size={10} /> 1024 – 65535
+          </p>
+        )}
+        {!status.checking && status.available === true && (
+          <p className="text-2xs flex items-center gap-1 text-success">
+            <CheckCircle2 size={10} /> {status.isSelf ? m.portSelf : m.portAvailable}
+          </p>
+        )}
+      </div>
+    </div>
+  );
+}

package/app/components/settings/McpTab.tsx

@@ -11,7 +11,6 @@ import type { McpTabProps, McpStatus, AgentInfo, ConnectionMode } from './types'
 import AgentInstall from './McpAgentInstall';
 import SkillsSection from './McpSkillsSection';
 import McpExternalTools from './McpExternalTools';
-import ServerPortsCard from './ServerPortsCard';
 
 /* ── Main Connections Tab ────────────────────────────────────────── */
 
@@ -99,9 +98,6 @@ export function McpTab({ t }: McpTabProps) {
 
   return (
     <div className="space-y-6">
-      {/* 0. Server Ports */}
-      {m && <ServerPortsCard m={m} />}
-
       {/* 1. Connection Mode Toggle */}
       <ConnectionModeCard
         mcpEnabled={mcpEnabled}
@@ -114,7 +110,7 @@ export function McpTab({ t }: McpTabProps) {
       {/* 2. Auth Token */}
       <AuthTokenCard status={mcp.status} m={m} />
 
-      {/* 3. Connect Agents (CLI/MCP guides + detected agents) */}
+      {/* 3. Connect Agents (CLI/MCP guides + detected agents + MCP port) */}
       <ConnectCard
         mode={mode}
         onModeChange={setMode}

package/app/components/settings/TestButton.tsx

@@ -8,6 +8,7 @@ function errorMessage(t: AiTabProps['t'], code?: ErrorCode): string {
   switch (code) {
     case 'auth_error': return t.settings.ai.testKeyAuthError;
     case 'model_not_found': return t.settings.ai.testKeyModelNotFound;
+    case 'endpoint_error': return t.settings.ai.testKeyEndpointError;
     case 'rate_limited': return t.settings.ai.testKeyRateLimited;
     case 'network_error': return t.settings.ai.testKeyNetworkError;
     default: return t.settings.ai.testKeyUnknown;

package/app/components/settings/{ServerPortsCard.tsx → WebPortSection.tsx}

@@ -51,8 +51,8 @@ function PortField({
 
   return (
     <div className="space-y-1">
-      <label className="text-xs font-medium text-foreground">{label}</label>
-      <p className="text-2xs text-muted-foreground">{hint}</p>
+      {label && <label className="text-xs font-medium text-foreground">{label}</label>}
+      {hint && <p className="text-2xs text-muted-foreground">{hint}</p>}
       <input
         type="number" min={1024} max={65535} value={value}
         onChange={handleChange}
@@ -107,46 +107,33 @@ function RestartOverlay({ message, sub }: { message: string; sub?: string }) {
   );
 }
 
-/* ── ServerPortsCard ───────────────────────────────────────────── */
-
-export default function ServerPortsCard({ m }: { m: Record<string, any> }) {
-  // Loaded from /api/settings
-  const [origWebPort, setOrigWebPort] = useState<number>(0);
-  const [origMcpPort, setOrigMcpPort] = useState<number>(0);
-
-  const [webPort, setWebPort] = useState<number>(0);
-  const [mcpPort, setMcpPort] = useState<number>(0);
-
-  const [webStatus, setWebStatus] = useState<PortStatus>(EMPTY_STATUS);
-  const [mcpStatus, setMcpStatus] = useState<PortStatus>(EMPTY_STATUS);
+/* ── WebPortSection ────────────────────────────────────────────── */
 
+export default function WebPortSection({ m }: { m: Record<string, any> }) {
+  const [origPort, setOrigPort] = useState<number>(0);
+  const [port, setPort] = useState<number>(0);
+  const [status, setStatus] = useState<PortStatus>(EMPTY_STATUS);
   const [updating, setUpdating] = useState(false);
   const [overlayMsg, setOverlayMsg] = useState<string | null>(null);
   const [overlaySub, setOverlaySub] = useState<string | undefined>(undefined);
   const pollRef = useRef<ReturnType<typeof setInterval>>(undefined);
 
-  // Load current ports from API
   useEffect(() => {
-    apiFetch<{ port?: number; mcpPort?: number }>('/api/settings').then(d => {
-      const wp = d.port || 3456;
-      const mp = d.mcpPort || 8781;
-      setOrigWebPort(wp); setWebPort(wp);
-      setOrigMcpPort(mp); setMcpPort(mp);
+    apiFetch<{ port?: number }>('/api/settings').then(d => {
+      const p = d.port || 3456;
+      setOrigPort(p);
+      setPort(p);
     }).catch(() => {});
   }, []);
 
   useEffect(() => () => clearInterval(pollRef.current), []);
 
-  const hasChanges = webPort !== origWebPort || mcpPort !== origMcpPort;
-  const portConflict = webPort === mcpPort && webPort > 0;
-  const portInvalid = webPort < 1024 || webPort > 65535 || mcpPort < 1024 || mcpPort > 65535;
-  const webChanged = webPort !== origWebPort;
-  const mcpChanged = mcpPort !== origMcpPort;
+  const hasChanges = port !== origPort;
+  const portInvalid = port < 1024 || port > 65535;
+  const portUnavailable = status.checking || (status.available === false && !status.isSelf);
 
-  // Port check via existing API
-  const checkPort = useCallback(async (port: number, which: 'web' | 'mcp') => {
-    const setStatus = which === 'web' ? setWebStatus : setMcpStatus;
-    if (port < 1024 || port > 65535) {
+  const checkPort = useCallback(async (p: number) => {
+    if (p < 1024 || p > 65535) {
       setStatus({ ...EMPTY_STATUS, available: false, invalid: true });
       return;
     }
@@ -155,7 +142,7 @@ export default function ServerPortsCard({ m }: { m: Record<string, any> }) {
       const res = await apiFetch<CheckPortResult>('/api/setup/check-port', {
         method: 'POST',
         headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ port }),
+        body: JSON.stringify({ port: p }),
       });
       setStatus({
         checking: false,
@@ -168,131 +155,74 @@ export default function ServerPortsCard({ m }: { m: Record<string, any> }) {
     }
   }, []);
 
-  // Update handler
   const handleUpdate = async () => {
-    if (!hasChanges || portConflict || portInvalid || updating) return;
+    if (!hasChanges || portInvalid || portUnavailable || updating) return;
 
     setUpdating(true);
     try {
-      // 0. Final port availability check before committing
-      //    User may have waited a while since input — port state could have changed.
-      const portsToCheck: Array<{ port: number; which: 'web' | 'mcp' }> = [];
-      if (webChanged) portsToCheck.push({ port: webPort, which: 'web' });
-      if (mcpChanged) portsToCheck.push({ port: mcpPort, which: 'mcp' });
-
-      for (const { port, which } of portsToCheck) {
-        const res = await apiFetch<CheckPortResult>('/api/setup/check-port', {
-          method: 'POST',
-          headers: { 'Content-Type': 'application/json' },
-          body: JSON.stringify({ port }),
+      // Final availability check
+      const res = await apiFetch<CheckPortResult>('/api/setup/check-port', {
+        method: 'POST',
+        headers: { 'Content-Type': 'application/json' },
+        body: JSON.stringify({ port }),
+      });
+      if (!res.available && !res.isSelf) {
+        setStatus({
+          checking: false,
+          available: false,
+          isSelf: false,
+          suggestion: res.suggestion ?? null,
         });
-        if (!res.available && !res.isSelf) {
-          // Port became occupied since last check
-          const setStatus = which === 'web' ? setWebStatus : setMcpStatus;
-          setStatus({
-            checking: false,
-            available: false,
-            isSelf: false,
-            suggestion: res.suggestion ?? null,
-          });
-          setUpdating(false);
-          toast.error(m.portInUse(port));
-          return;
-        }
+        setUpdating(false);
+        toast.error(m.portInUse(port));
+        return;
       }
 
-      // 1. Save both ports to config
+      // Save port
       await apiFetch('/api/settings', {
         method: 'POST',
         headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({
-          port: webPort,
-          mcpPort,
-        }),
+        body: JSON.stringify({ port }),
       });
 
-      // 2. Determine restart strategy
-      if (webChanged) {
-        // Full restart needed — Web port changed
-        setOverlayMsg(m.portWebRestarting);
-        setOverlaySub(m.portRedirecting);
+      // Full restart — Web port changed
+      setOverlayMsg(m.portWebRestarting);
+      setOverlaySub(m.portRedirecting);
 
-        try {
-          await apiFetch('/api/restart', { method: 'POST' });
-        } catch {
-          // Expected: server dies before response completes
-        }
+      try {
+        await apiFetch('/api/restart', { method: 'POST' });
+      } catch {
+        // Expected: server dies before response completes
+      }
 
-        // Poll new port for health
-        const newOrigin = `${window.location.protocol}//${window.location.hostname}:${webPort}`;
-        const deadline = Date.now() + 30_000;
-        pollRef.current = setInterval(async () => {
-          if (Date.now() > deadline) {
-            clearInterval(pollRef.current);
-            setOverlayMsg(null);
-            setUpdating(false);
-            toast.error(m.portRestartTimeout);
-            return;
-          }
-          try {
-            const res = await fetch(`${newOrigin}/api/health`, { signal: AbortSignal.timeout(2000) });
-            if (res.ok) {
-              clearInterval(pollRef.current);
-              // Redirect to new port
-              window.location.href = newOrigin;
-            }
-          } catch {
-            // Server not up yet, keep polling
-          }
-        }, 1500);
-      } else if (mcpChanged) {
-        // MCP-only restart
-        setOverlayMsg(m.portMcpRestarting);
-        try {
-          await apiFetch('/api/mcp/restart', { method: 'POST' });
-        } catch {
+      // Poll new port for health
+      const newOrigin = `${window.location.protocol}//${window.location.hostname}:${port}`;
+      const deadline = Date.now() + 30_000;
+      pollRef.current = setInterval(async () => {
+        if (Date.now() > deadline) {
+          clearInterval(pollRef.current);
           setOverlayMsg(null);
           setUpdating(false);
-          toast.error(m.portUpdateFailed);
+          toast.error(m.portRestartTimeout);
           return;
         }
-
-        // Poll MCP status
-        const deadline = Date.now() + 60_000;
-        pollRef.current = setInterval(async () => {
-          if (Date.now() > deadline) {
+        try {
+          const r = await fetch(`${newOrigin}/api/health`, { signal: AbortSignal.timeout(2000) });
+          if (r.ok) {
             clearInterval(pollRef.current);
-            setOverlayMsg(null);
-            setUpdating(false);
-            toast.error(m.portRestartTimeout);
-            return;
-          }
-          try {
-            const s = await apiFetch<{ running: boolean; port: number }>('/api/mcp/status', { timeout: 3000 });
-            if (s.running) {
-              clearInterval(pollRef.current);
-              setOverlayMsg(null);
-              setUpdating(false);
-              setOrigMcpPort(mcpPort);
-              toast.success(m.portUpdateSuccess);
-            }
-          } catch {
-            // keep polling
+            window.location.href = newOrigin;
           }
-        }, 3000);
-      } else {
-        // No restart needed (shouldn't reach here, but guard)
-        setUpdating(false);
-        toast.success(m.portUpdateSuccess);
-      }
+        } catch {
+          // Server not up yet
+        }
+      }, 1500);
     } catch {
       setUpdating(false);
       toast.error(m.portUpdateFailed);
     }
   };
 
-  // Don't render until data loaded
-  if (origWebPort === 0) return null;
+  if (origPort === 0) return null;
 
   return (
     <>
@@ -302,34 +232,20 @@ export default function ServerPortsCard({ m }: { m: Record<string, any> }) {
             <Monitor size={14} className="text-muted-foreground" />
           </div>
           <div className="flex-1 min-w-0">
-            <h3 className="text-sm font-semibold text-foreground">{m.portsCardTitle}</h3>
-            <p className="text-2xs text-muted-foreground">{m.portsCardDesc}</p>
+            <h3 className="text-sm font-semibold text-foreground">{m.webPortLabel}</h3>
+            <p className="text-2xs text-muted-foreground">{m.webPortHint}</p>
           </div>
         </div>
-        <div className="px-4 pb-4 space-y-4">
+        <div className="px-4 pb-4 space-y-3">
           <PortField
-            label={m.webPortLabel} hint={m.webPortHint}
-            value={webPort} onChange={v => { setWebPort(v); setWebStatus(EMPTY_STATUS); }}
-            status={webStatus} onCheckPort={p => checkPort(p, 'web')} m={m}
+            label="" hint=""
+            value={port} onChange={v => { setPort(v); setStatus(EMPTY_STATUS); }}
+            status={status} onCheckPort={checkPort} m={m}
           />
-          <PortField
-            label={m.mcpPortLabel} hint={m.mcpPortHint}
-            value={mcpPort} onChange={v => { setMcpPort(v); setMcpStatus(EMPTY_STATUS); }}
-            status={mcpStatus} onCheckPort={p => checkPort(p, 'mcp')} m={m}
-          />
-
-          {/* Conflict warning */}
-          {portConflict && (
-            <p className="text-xs flex items-center gap-1.5 text-[var(--amber)]">
-              <AlertTriangle size={12} /> {m.portConflict}
-            </p>
-          )}
-
-          {/* Update button */}
           <button
             type="button"
             onClick={handleUpdate}
-            disabled={!hasChanges || portConflict || portInvalid || updating}
+            disabled={!hasChanges || portInvalid || portUnavailable || updating}
             className="w-full py-2 rounded-lg text-xs font-medium transition-colors
               bg-[var(--amber)] text-[var(--amber-foreground)]
               hover:opacity-90
@@ -341,7 +257,6 @@ export default function ServerPortsCard({ m }: { m: Record<string, any> }) {
         </div>
       </div>
 
-      {/* Full-screen restart overlay */}
       {overlayMsg && <RestartOverlay message={overlayMsg} sub={overlaySub} />}
     </>
   );

package/app/components/settings/types.ts

@@ -18,6 +18,18 @@ export interface AgentSettings {
 export interface SettingsData {
   ai: AiSettings;
   agent?: AgentSettings;
+  embedding?: {
+    enabled: boolean;
+    baseUrl: string;
+    apiKey: string;
+    model: string;
+  };
+  embeddingStatus?: {
+    enabled: boolean;
+    ready: boolean;
+    building: boolean;
+    docCount: number;
+  };
   mindRoot: string;
   webPassword?: string;
   authToken?: string;   // masked: first-xxxx-••••-last pattern

package/app/components/settings/useCustomProviderForm.ts

@@ -5,7 +5,7 @@ import { type ProviderId } from '@/lib/agent/providers';
 import { type Provider, generateProviderId } from '@/lib/custom-endpoints';
 
 export type TestState = 'idle' | 'testing' | 'ok' | 'error';
-export type ErrorCode = 'auth_error' | 'model_not_found' | 'rate_limited' | 'network_error' | 'unknown';
+export type ErrorCode = 'auth_error' | 'model_not_found' | 'endpoint_error' | 'rate_limited' | 'network_error' | 'unknown';
 
 export interface TestResult {
   state: TestState;
@@ -78,7 +78,7 @@ export function useCustomProviderForm({
         body: JSON.stringify(
           initial?.id
             ? { provider: initial.id, apiKey, model, baseUrl }
-            : { protocol, apiKey, model, baseUrl },
+            : { provider: protocol, apiKey, model, baseUrl, baseProviderId: protocol },
         ),
       });
       const json = await res.json();

package/app/data/skills/mindos/SKILL.md

@@ -75,16 +75,60 @@ npm install -g @geminilight/mindos
 
 ---
 
+## Retrieval strategy
+
+When retrieving knowledge, use **two paths in parallel**, then filter before deep-reading:
+
+### Path 1: Directory scan (by name/structure)
+
+Browse the KB tree and **look at file names and directory names**. Titles often reveal content without reading. If a user asks about "authentication", and you see `Decisions/auth-jwt-vs-session.md`, that's a strong candidate — read it directly, no search needed.
+
+- After bootstrap, scan the tree for paths whose names relate to the query topic.
+- Pay attention to directory semantics: `Decisions/`, `Projects/`, `Workflows/`, `Resources/` etc. each imply what kind of content lives there.
+- If the KB is small (<50 files), a quick tree scan may be faster and more reliable than search.
+
+### Path 2: Full-text search (by content)
+
+Use `search` for content that can't be guessed from file names alone.
+
+- Craft queries from the user's actual words. If the user says "那个很慢的接口", search for "慢 接口" or "性能 API".
+- One well-targeted search is better than 4 vague ones. Only add a second search if the first returned <3 results or if the topic has obvious alternate terms (e.g., Chinese + English).
+- **Do NOT** mechanically fire 2-4 searches every time. Think first, search precisely.
+
+### Filter: snippet triage before full read
+
+Search results include a **snippet** and a **BM25 score**. Use them to decide what to read:
+
+- **High score + snippet clearly on-topic** → read full file.
+- **Medium score + snippet partially relevant** → read full file only if no better candidates exist.
+- **Low score or snippet off-topic** → skip. Do not read every search result.
+- Aim to read **1-3 files** deeply, not 10 files superficially.
+
+### Combined example
+
+```
+User: "之前关于数据库选型的讨论"
+
+Step 1 (tree scan): See "Decisions/database-postgres-vs-mongo.md" → strong match by name.
+Step 2 (search):    search("数据库选型") → returns 5 results.
+Step 3 (triage):    Result #1 snippet mentions "PostgreSQL vs MongoDB 对比" (score 18.3) → read.
+                    Result #2 snippet mentions "数据库连接池配置" (score 4.1) → skip, off-topic.
+                    Result #3 snippet mentions "选型会议纪要" (score 12.7) → read.
+Step 4 (answer):    Cite from the 2-3 files actually read.
+```
+
+---
+
 ## NEVER do (hard-won pitfalls)
 
 - **NEVER write to the KB root** unless explicitly told. Root is for governance files only. New content goes under the most fitting subdirectory.
 - **NEVER assume directory names.** Infer from the actual bootstrap tree — the KB may use Chinese names or flat layout.
 - **NEVER use full-file overwrite for a small edit.** Use `mindos file edit-section` or `mindos file insert-heading` for targeted changes. Full rewrites destroy git diffs.
-- **NEVER search with a single keyword.** Fire 2-4 parallel searches (synonyms, abbreviations, Chinese/English variants).
 - **NEVER modify `INSTRUCTION.md` or `README.md` without confirmation.** Governance docs — treat as high-sensitivity.
 - **NEVER create a file without checking siblings.** Read 1-2 files in the target directory to learn local style.
 - **NEVER leave orphan references.** After rename/move, check backlinks and update every referring file.
 - **NEVER skip routing confirmation for multi-file writes.** The user's mental model may differ from yours.
+- **NEVER read every search result.** Use snippet + score to triage. Only deep-read files that are clearly relevant.
 
 ---
 

package/app/hooks/useAcpDetection.ts

@@ -119,7 +119,7 @@ export function useAcpDetection(): AcpDetectionState {
         if (!cancelled) setLoading(false);
       });
 
-    return () => { cancelled = true; };
+    return () => { cancelled = true; inflight.current = false; };
   }, [trigger]); // eslint-disable-line react-hooks/exhaustive-deps
 
   return { installedAgents, notInstalledAgents, loading, error, refresh };

package/app/hooks/useAcpRegistry.ts

@@ -79,7 +79,7 @@ export function useAcpRegistry(): AcpRegistryState {
         if (!cancelled) setLoading(false);
       });
 
-    return () => { cancelled = true; };
+    return () => { cancelled = true; inflight.current = false; };
   }, [trigger]); // eslint-disable-line react-hooks/exhaustive-deps
 
   return { agents, loading, error, retry };