@geminilight/mindos 0.6.69 → 0.6.70

package/app/hooks/useFileImport.ts

@@ -57,8 +57,13 @@ async function extractPdfText(file: File): Promise<string> {
     const err = await res.json().catch(() => ({}));
     throw new Error((err as { error?: string }).error || 'PDF extraction failed');
   }
-  const data = await res.json() as { text?: string; extracted?: boolean };
-  if (!data.extracted || !data.text) throw new Error('No text extracted from PDF');
+  const data = await res.json() as { text?: string; extracted?: 'success' | 'empty' | 'error'; extractionError?: string };
+  if (data.extracted === 'error') {
+    throw new Error(data.extractionError || 'Failed to parse PDF');
+  }
+  if (data.extracted !== 'success' || !data.text) {
+    throw new Error('No text extracted from PDF');
+  }
   return data.text;
 }
 

package/app/hooks/useFileUpload.ts

@@ -22,10 +22,6 @@ function uint8ToBase64(bytes: Uint8Array): string {
   return btoa(binary);
 }
 
-/**
- * Extract text from a PDF file via the backend API.
- * Returns a LocalAttachment with `status` reflecting the outcome.
- */
 async function extractPdfToAttachment(file: File): Promise<LocalAttachment> {
   const name = file.name;
 
@@ -41,7 +37,8 @@ async function extractPdfToAttachment(file: File): Promise<LocalAttachment> {
 
     let payload: {
       text?: string;
-      extracted?: boolean;
+      extracted?: 'success' | 'empty' | 'error';
+      extractionError?: string;
       error?: string;
       truncated?: boolean;
       totalChars?: number;
@@ -57,27 +54,37 @@ async function extractPdfToAttachment(file: File): Promise<LocalAttachment> {
       throw new Error(payload.error || `PDF extraction failed (${res.status})`);
     }
 
-    const extracted = payload.extracted ? (payload.text || '') : '';
+    // Handle extraction error state
+    if (payload.extracted === 'error') {
+      return {
+        name,
+        content: `[PDF: ${name}] Failed to extract text from this PDF.`,
+        status: 'error',
+        error: payload.extractionError || 'PDF extraction failed (unable to parse PDF)',
+      };
+    }
 
-    if (!extracted) {
+    // Handle empty PDF (no extraction error, but no text)
+    const text = payload.extracted === 'success' ? (payload.text || '') : '';
+    if (!text) {
       return {
         name,
         content: `[PDF: ${name}] Could not extract readable text (possibly scanned/image PDF).`,
         status: 'error',
-        error: 'Scanned or image-only PDF — no extractable text',
+        error: 'No extractable text found — PDF may be scanned, image-only, or have no text content',
       };
     }
 
     const att: LocalAttachment = {
       name,
-      content: `[PDF TEXT EXTRACTED: ${name}]\n\n${extracted}`,
+      content: `[PDF TEXT EXTRACTED: ${name}]\n\n${text}`,
       status: 'success',
     };
 
     if (payload.truncated && payload.totalChars) {
       att.truncatedInfo = {
         totalChars: payload.totalChars,
-        includedChars: extracted.length,
+        includedChars: text.length,
         totalPages: payload.pagesParsed ?? 0,
       };
     }

package/app/lib/acp/agent-descriptors.ts

@@ -260,6 +260,24 @@ export function getDetectableAgents(): DetectableAgent[] {
   }));
 }
 
+/**
+ * Look up user override for an agent, checking canonical ID, alias → canonical,
+ * and reverse alias (canonical → any alias) so users can configure with any name.
+ */
+export function findUserOverride(
+  agentId: string,
+  overrides?: Record<string, AcpAgentOverride>,
+): AcpAgentOverride | undefined {
+  if (!overrides) return undefined;
+  if (overrides[agentId]) return overrides[agentId];
+  const canonical = resolveAlias(agentId);
+  if (canonical !== agentId && overrides[canonical]) return overrides[canonical];
+  for (const [alias, target] of Object.entries(AGENT_ALIASES)) {
+    if (target === agentId && overrides[alias]) return overrides[alias];
+  }
+  return undefined;
+}
+
 /** Parse and validate acpAgents config from raw settings JSON. */
 export function parseAcpAgentOverrides(raw: unknown): Record<string, AcpAgentOverride> | undefined {
   if (!raw || typeof raw !== 'object' || Array.isArray(raw)) return undefined;

package/app/lib/acp/index.ts

@@ -1,9 +1,9 @@
 export { fetchAcpRegistry, getAcpAgents, findAcpAgent, clearRegistryCache } from './registry';
-export { spawnAcpAgent, sendMessage, sendAndWait, onMessage, onNotification, onRequest, sendResponse, installAutoApproval, killAgent, killAllAgents, getProcess, getActiveProcesses } from './subprocess';
+export { spawnAndConnect, spawnAcpAgent, killAgent, killAllAgents, getProcess, getActiveProcesses } from './subprocess';
 export { createSession, createSessionFromEntry, loadSession, listSessions, prompt, promptStream, cancelPrompt, setMode, setConfigOption, closeSession, getSession, getActiveSessions, closeAllSessions } from './session';
 export { bridgeA2aToAcp, bridgeAcpResponseToA2a, bridgeAcpUpdatesToA2a } from './bridge';
 export { acpTools } from './acp-tools';
-export { AGENT_DESCRIPTORS, AGENT_ALIASES, resolveAlias, getDetectableAgents } from './agent-descriptors';
+export { AGENT_DESCRIPTORS, AGENT_ALIASES, resolveAlias, findUserOverride, getDetectableAgents } from './agent-descriptors';
 export { ACP_ERRORS } from './types';
 export type {
   AcpAgentCapabilities,
@@ -17,10 +17,6 @@ export type {
   AcpSessionState,
   AcpSession,
   AcpSessionInfo,
-  AcpJsonRpcRequest,
-  AcpJsonRpcResponse,
-  AcpJsonRpcError,
-  AcpPromptRequest,
   AcpPromptResponse,
   AcpUpdateType,
   AcpSessionUpdate,
@@ -38,5 +34,5 @@ export type {
   AcpRegistry,
   AcpTransportType,
 } from './types';
-export type { AcpProcess, AcpIncomingRequest, AcpNotification } from './subprocess';
+export type { AcpProcess, AcpClientCallbacks, AcpConnection } from './subprocess';
 export type { AcpAgentDescriptor, AcpAgentOverride, ResolvedAgentCommand, DetectableAgent } from './agent-descriptors';

package/app/lib/acp/registry.ts

@@ -16,7 +16,7 @@ import { AGENT_DESCRIPTORS, getDescriptorDisplayName, getDescriptorDescription,
 
 const REGISTRY_URL = 'https://cdn.agentclientprotocol.com/registry/v1/latest/registry.json';
 const CACHE_TTL_MS = 7 * 24 * 60 * 60 * 1000; // 7 days — registry updates very rarely
-const FETCH_TIMEOUT_MS = 10_000;
+const FETCH_TIMEOUT_MS = 3_000;
 
 /* ── Built-in Registry (from AGENT_DESCRIPTORS) ────────────────────────── */