@geminilight/mindos 0.6.51 → 0.6.53

package/app/app/api/ask/route.ts

@@ -26,7 +26,8 @@ import { AGENT_SYSTEM_PROMPT, ORGANIZE_SYSTEM_PROMPT, CHAT_SYSTEM_PROMPT } from
 import type { AskModeApi } from '@/lib/types';
 import { toAgentMessages } from '@/lib/agent/to-agent-messages';
 import { logAgentOp } from '@/lib/agent/log';
-import { readSettings } from '@/lib/settings';
+import { readSettings, readBaseUrlCompat, writeBaseUrlCompat } from '@/lib/settings';
+import { en as i18nEn, zh as i18nZh } from '@/lib/i18n';
 import { MindOSError, apiError, ErrorCodes } from '@/lib/errors';
 import { metrics } from '@/lib/metrics';
 import { assertNotProtected } from '@/lib/core';
@@ -400,6 +401,134 @@ function toPiCustomToolDefinitions(tools: AgentTool<any>[]): ToolDefinition<any,
   }));
 }
 
+// ---------------------------------------------------------------------------
+// Non-streaming fallback for proxies that don't support stream + tools
+// ---------------------------------------------------------------------------
+
+/**
+ * Mini agent loop using non-streaming OpenAI-compatible API.
+ * Used when a proxy silently breaks stream+tools by returning plain text.
+ * Emits SSE events identical to the streaming path so the frontend is unaffected.
+ */
+async function runNonStreamingFallback(opts: {
+  baseUrl: string;
+  apiKey: string;
+  model: string;
+  systemPrompt: string;
+  historyMessages: { role: string; content: unknown }[];
+  userContent: string;
+  tools: AgentTool<any>[];
+  send: (event: MindOSSSEvent) => void;
+  signal: AbortSignal;
+  maxSteps: number;
+}): Promise<void> {
+  const { baseUrl, apiKey, model, systemPrompt, historyMessages, userContent, tools, send, signal, maxSteps } = opts;
+
+  const openaiTools = tools.map(t => ({
+    type: 'function' as const,
+    function: {
+      name: t.name,
+      description: t.description ?? '',
+      parameters: (t as any).parameters ?? { type: 'object', properties: {} },
+    },
+  }));
+
+  const messages: { role: string; content: unknown; tool_calls?: unknown; tool_call_id?: string }[] = [
+    { role: 'system', content: systemPrompt },
+    ...historyMessages,
+    { role: 'user', content: userContent },
+  ];
+
+  const toolMap = new Map(tools.map(t => [t.name, t]));
+  const endpoint = `${baseUrl.replace(/\/$/, '')}/chat/completions`;
+  let step = 0;
+
+  while (step < maxSteps) {
+    if (signal.aborted) throw new Error('Request aborted');
+    step++;
+
+    const resp = await fetch(endpoint, {
+      method: 'POST',
+      headers: {
+        'Content-Type': 'application/json',
+        'Authorization': `Bearer ${apiKey}`,
+      },
+      body: JSON.stringify({
+        model,
+        messages,
+        tools: openaiTools.length > 0 ? openaiTools : undefined,
+        tool_choice: openaiTools.length > 0 ? 'auto' : undefined,
+        stream: false,
+      }),
+      signal,
+    });
+
+    if (!resp.ok) {
+      const errText = await resp.text().catch(() => '');
+      throw new Error(`Non-streaming API error ${resp.status}: ${errText.slice(0, 200)}`);
+    }
+
+    const data = await resp.json() as any;
+    const choice = data?.choices?.[0];
+    if (!choice) throw new Error('Empty response from API');
+
+    const msg = choice.message;
+    const finishReason: string = choice.finish_reason ?? 'stop';
+
+    // Emit text content in chunks to simulate streaming appearance
+    if (msg.content) {
+      const text: string = typeof msg.content === 'string' ? msg.content : '';
+      if (text) {
+        const chunkSize = 40;
+        for (let i = 0; i < text.length; i += chunkSize) {
+          send({ type: 'text_delta', delta: text.slice(i, i + chunkSize) });
+          await new Promise(r => setTimeout(r, 8));
+        }
+      }
+    }
+
+    // No tool calls or naturally stopped → done
+    if (finishReason === 'stop' || !msg.tool_calls?.length) break;
+
+    // Execute each tool call
+    const toolResultMessages: { role: string; tool_call_id: string; content: string }[] = [];
+    for (const tc of msg.tool_calls) {
+      const toolName = tc.function?.name ?? '';
+      const toolCallId = tc.id ?? `call_${Date.now()}`;
+      let parsedArgs: Record<string, unknown> = {};
+      try { parsedArgs = JSON.parse(tc.function?.arguments ?? '{}'); } catch { /* ignore */ }
+
+      const tool = toolMap.get(toolName);
+      send({ type: 'tool_start', toolCallId, toolName, args: parsedArgs });
+
+      let resultText = '';
+      let isError = false;
+      if (tool) {
+        try {
+          const result = await tool.execute(toolCallId, parsedArgs, signal);
+          resultText = result.content
+            .filter((c: any) => c.type === 'text')
+            .map((c: any) => c.text)
+            .join('\n');
+        } catch (err) {
+          resultText = err instanceof Error ? err.message : String(err);
+          isError = true;
+        }
+      } else {
+        resultText = `Tool "${toolName}" not found`;
+        isError = true;
+      }
+
+      send({ type: 'tool_end', toolCallId, output: resultText, isError });
+      toolResultMessages.push({ role: 'tool', tool_call_id: toolCallId, content: resultText });
+    }
+
+    // Append assistant turn + tool results for next iteration
+    messages.push({ role: 'assistant', content: msg.content ?? null, tool_calls: msg.tool_calls });
+    messages.push(...toolResultMessages);
+  }
+}
+
 // ---------------------------------------------------------------------------
 // POST /api/ask
 // ---------------------------------------------------------------------------
@@ -433,6 +562,10 @@ export async function POST(req: NextRequest) {
   // Read agent config from settings
   const serverSettings = readSettings();
   const agentConfig = serverSettings.agent ?? {};
+
+  // Detect locale from Accept-Language header for i18n status messages
+  const acceptLang = req.headers.get('accept-language') ?? '';
+  const t = acceptLang.startsWith('zh') ? i18nZh.ask : i18nEn.ask;
   const defaultMaxSteps = askMode === 'chat' ? 8 : (agentConfig.maxSteps ?? 20);
   const stepLimit = Number.isFinite(body.maxSteps)
     ? Math.min(30, Math.max(1, Number(body.maxSteps)))
@@ -689,7 +822,7 @@ export async function POST(req: NextRequest) {
     const provOverride = body.providerOverride && isProviderId(body.providerOverride)
       ? body.providerOverride as ProviderId
       : undefined;
-    const { model, modelName, apiKey, provider } = getModelConfig({
+    const { model, modelName, apiKey, provider, baseUrl } = getModelConfig({
       provider: provOverride,
       hasImages: hasImages(messages),
     });
@@ -788,6 +921,7 @@ export async function POST(req: NextRequest) {
 
         let hasContent = false;
         let lastModelError = '';
+        const effectiveBaseUrlKey = baseUrl || 'default';
 
         session.subscribe((event: AgentEvent) => {
           if (isTextDeltaEvent(event)) {
@@ -993,6 +1127,36 @@ export async function POST(req: NextRequest) {
             safeClose();
           } else {
             // Route to MindOS agent (existing logic)
+
+            // ── Proxy compatibility check ──
+            // If this baseUrl is known to reject stream+tools, skip session.prompt() entirely
+            // and go straight to the non-streaming fallback path.
+            const compatCache = readBaseUrlCompat();
+            if (compatCache[effectiveBaseUrlKey] === 'non-streaming' && baseUrl && provider === 'openai') {
+              send({ type: 'status', message: t.proxyCompatMode });
+              try {
+                await runNonStreamingFallback({
+                  baseUrl,
+                  apiKey,
+                  model: modelName,
+                  systemPrompt,
+                  historyMessages: llmHistoryMessages,
+                  userContent: typeof lastUserContent === 'string' ? lastUserContent : JSON.stringify(lastUserContent),
+                  tools: requestTools,
+                  send,
+                  signal: req.signal,
+                  maxSteps: stepLimit,
+                });
+                metrics.recordRequest(Date.now() - requestStartTime);
+                send({ type: 'done' });
+              } catch (fallbackErr) {
+                metrics.recordRequest(Date.now() - requestStartTime);
+                send({ type: 'error', message: t.proxyCompatFailed(fallbackErr instanceof Error ? fallbackErr.message : String(fallbackErr)) });
+              }
+              safeClose();
+              return;
+            }
+
             // Retry with exponential backoff for transient failures (timeout, rate limit, 5xx).
             // Only retry if no content has been streamed yet — once the user sees partial
             // output, retrying would produce duplicate/garbled content.
@@ -1022,7 +1186,33 @@ export async function POST(req: NextRequest) {
 
             metrics.recordRequest(Date.now() - requestStartTime);
             if (!hasContent && lastModelError) {
-              send({ type: 'error', message: lastModelError });
+              // No content received — check if this looks like a proxy stream+tools incompatibility.
+              // Only attempt fallback for OpenAI-compatible endpoints with a custom baseUrl.
+              if (baseUrl && provider === 'openai') {
+                send({ type: 'status', message: t.proxyCompatDetecting });
+                try {
+                  await runNonStreamingFallback({
+                    baseUrl,
+                    apiKey,
+                    model: modelName,
+                    systemPrompt,
+                    historyMessages: llmHistoryMessages,
+                    userContent: typeof lastUserContent === 'string' ? lastUserContent : JSON.stringify(lastUserContent),
+                    tools: requestTools,
+                    send,
+                    signal: req.signal,
+                    maxSteps: stepLimit,
+                  });
+                  // Success → cache this endpoint as non-streaming so future requests skip the probe
+                  writeBaseUrlCompat(effectiveBaseUrlKey, 'non-streaming');
+                  console.log(`[ask] Proxy compat detected: ${effectiveBaseUrlKey} → non-streaming (cached)`);
+                  send({ type: 'done' });
+                } catch (fallbackErr) {
+                  send({ type: 'error', message: t.proxyCompatAlsoFailed(fallbackErr instanceof Error ? fallbackErr.message : String(fallbackErr)) });
+                }
+              } else {
+                send({ type: 'error', message: lastModelError });
+              }
             } else {
               send({ type: 'done' });
             }

package/app/app/api/restart/route.ts

@@ -3,31 +3,36 @@ import { NextResponse } from 'next/server';
 import { spawn } from 'node:child_process';
 import { resolve } from 'node:path';
 
+/**
+ * Strip ALL MINDOS_ and MIND_ prefixed env vars so the restart child
+ * process re-derives paths from its own installation root.
+ * Preserves old port values via MINDOS_OLD_ for cleanup.
+ */
+function cleanEnvForRestart(): { env: NodeJS.ProcessEnv; oldWebPort?: string; oldMcpPort?: string } {
+  const cleaned = { ...process.env };
+  const oldWebPort = cleaned.MINDOS_WEB_PORT;
+  const oldMcpPort = cleaned.MINDOS_MCP_PORT;
+  for (const key of Object.keys(cleaned)) {
+    if (key.startsWith('MINDOS_') || key.startsWith('MIND_')) {
+      delete cleaned[key];
+    }
+  }
+  delete cleaned.AUTH_TOKEN;
+  delete cleaned.WEB_PASSWORD;
+  delete cleaned.NODE_OPTIONS;
+  // Pass old ports so restart command can clean up stale listeners
+  if (oldWebPort) cleaned.MINDOS_OLD_WEB_PORT = oldWebPort;
+  if (oldMcpPort) cleaned.MINDOS_OLD_MCP_PORT = oldMcpPort;
+  return { env: cleaned, oldWebPort, oldMcpPort };
+}
+
 export async function POST() {
   try {
+    // Resolve CLI path BEFORE cleaning env (we still need current vars)
     const cliPath = process.env.MINDOS_CLI_PATH || resolve(process.env.MINDOS_PROJECT_ROOT || process.cwd(), '..', 'bin', 'cli.js');
     const nodeBin = process.env.MINDOS_NODE_BIN || process.execPath;
-    // Use 'restart' (stop all → wait for ports free → start) instead of bare
-    // 'start' which would fail assertPortFree because the current process and
-    // its MCP child are still holding the ports.
-    //
-    // IMPORTANT: Strip MINDOS_* env vars so the child's loadConfig() reads
-    // the *updated* config file instead of inheriting stale values from this
-    // process.  Without this, changing ports in the GUI has no effect on the
-    // restarted server — it would start on the old ports.
-    //
-    // Pass the current (old) ports via MINDOS_OLD_* so the restart command
-    // can clean up processes still listening on the previous ports.
-    const childEnv = { ...process.env };
-    const oldWebPort = childEnv.MINDOS_WEB_PORT;
-    const oldMcpPort = childEnv.MINDOS_MCP_PORT;
-    delete childEnv.MINDOS_WEB_PORT;
-    delete childEnv.MINDOS_MCP_PORT;
-    delete childEnv.MIND_ROOT;
-    delete childEnv.AUTH_TOKEN;
-    delete childEnv.WEB_PASSWORD;
-    if (oldWebPort) childEnv.MINDOS_OLD_WEB_PORT = oldWebPort;
-    if (oldMcpPort) childEnv.MINDOS_OLD_MCP_PORT = oldMcpPort;
+
+    const { env: childEnv } = cleanEnvForRestart();
     const child = spawn(nodeBin, [cliPath, 'restart'], {
       detached: true,
       stdio: 'ignore',

package/app/app/api/update/route.ts

@@ -3,25 +3,39 @@ import { NextResponse } from 'next/server';
 import { spawn } from 'node:child_process';
 import { resolve } from 'node:path';
 
+/**
+ * Strip ALL MINDOS_ and MIND_ prefixed env vars so the update child
+ * process re-derives paths from its own installation root after npm install.
+ * This prevents the "fake update" bug where the new process inherits
+ * stale MINDOS_PROJECT_ROOT / MINDOS_CLI_PATH pointing to old code.
+ */
+function cleanEnvForUpdate(): NodeJS.ProcessEnv {
+  const cleaned = { ...process.env };
+  for (const key of Object.keys(cleaned)) {
+    if (key.startsWith('MINDOS_') || key.startsWith('MIND_')) {
+      delete cleaned[key];
+    }
+  }
+  delete cleaned.AUTH_TOKEN;
+  delete cleaned.WEB_PASSWORD;
+  delete cleaned.NODE_OPTIONS;
+  return cleaned;
+}
+
 /**
  * POST /api/update — trigger `mindos update` as a detached child process.
  *
- * Similar to /api/restart: spawns the CLI command and returns immediately.
- * The update process will npm install, remove build stamp, and restart
- * the server. The current process will be killed during restart.
+ * Spawns the CLI command and returns immediately. The update process will
+ * npm install, remove build stamp, and restart the server.
+ * The current process will be killed during restart.
  */
 export async function POST() {
   try {
+    // Resolve CLI path BEFORE cleaning env (we still need current vars)
     const cliPath = process.env.MINDOS_CLI_PATH || resolve(process.env.MINDOS_PROJECT_ROOT || process.cwd(), '..', 'bin', 'cli.js');
     const nodeBin = process.env.MINDOS_NODE_BIN || process.execPath;
 
-    // Strip MINDOS_* env vars so the child reads fresh config
-    const childEnv = { ...process.env };
-    delete childEnv.MINDOS_WEB_PORT;
-    delete childEnv.MINDOS_MCP_PORT;
-    delete childEnv.MIND_ROOT;
-    delete childEnv.AUTH_TOKEN;
-    delete childEnv.WEB_PASSWORD;
+    const childEnv = cleanEnvForUpdate();
 
     const child = spawn(nodeBin, [cliPath, 'update'], {
       detached: true,

package/app/app/globals.css

@@ -81,7 +81,7 @@ body {
   --amber-dim: rgba(200, 135, 58, 0.18);
   --amber-subtle: rgba(200, 135, 30, 0.08);
   --amber-foreground: #ffffff;
-  --success: #7aad80;
+  --success: #5a8d60;
   --error: #c85050;
   --sidebar: #ede9e1;
   --sidebar-foreground: #1c1a17;

package/app/app/loading.tsx

@@ -0,0 +1,80 @@
+/**
+ * Home page skeleton — shown while server components load.
+ * Matches the HomeContent layout: hero → spaces grid → recent files.
+ */
+export default function Loading() {
+  return (
+    <div className="content-width px-4 md:px-6 py-10 md:py-14 animate-pulse" aria-busy="true" aria-label="Loading">
+      {/* Hero skeleton */}
+      <div className="mb-10">
+        <div className="flex items-center gap-3 mb-2">
+          <div className="w-1 h-7 rounded-full bg-muted" />
+          <div className="h-6 w-28 bg-muted rounded" />
+        </div>
+        <div className="h-4 w-64 bg-muted rounded ml-4 mb-6" />
+
+        {/* Command bar skeleton */}
+        <div className="w-full max-w-xl flex flex-col sm:flex-row items-stretch sm:items-center gap-2 ml-4">
+          <div className="flex-1 h-12 rounded-xl bg-muted" />
+          <div className="h-12 w-16 rounded-xl bg-muted shrink-0" />
+        </div>
+
+        {/* Quick actions skeleton */}
+        <div className="flex items-center gap-3 mt-4 ml-4">
+          <div className="h-9 w-28 rounded-lg bg-muted" />
+          <div className="h-9 w-40 rounded-lg bg-muted" />
+        </div>
+      </div>
+
+      {/* Spaces grid skeleton */}
+      <div className="mb-10">
+        <div className="flex items-center gap-2 mb-4">
+          <div className="w-4 h-4 rounded bg-muted" />
+          <div className="h-4 w-16 bg-muted rounded" />
+        </div>
+        <div className="grid grid-cols-2 sm:grid-cols-3 gap-2">
+          {Array.from({ length: 6 }).map((_, i) => (
+            <div key={i} className="flex items-start gap-3 px-3.5 py-3 rounded-xl border border-border">
+              <div className="w-5 h-5 rounded bg-muted shrink-0 mt-0.5" />
+              <div className="flex-1 space-y-1.5">
+                <div className="h-4 w-20 bg-muted rounded" />
+                <div className="h-3 w-full bg-muted rounded" />
+                <div className="h-3 w-12 bg-muted rounded" />
+              </div>
+            </div>
+          ))}
+        </div>
+      </div>
+
+      {/* Recent files skeleton */}
+      <div className="mb-10">
+        <div className="flex items-center gap-2 mb-4">
+          <div className="w-4 h-4 rounded bg-muted" />
+          <div className="h-4 w-24 bg-muted rounded" />
+        </div>
+        <div className="flex flex-col gap-3">
+          {Array.from({ length: 3 }).map((_, i) => (
+            <div key={i}>
+              <div className="flex items-center gap-2 px-1 py-1.5">
+                <div className="w-4 h-4 rounded bg-muted" />
+                <div className="h-3 w-24 bg-muted rounded" />
+              </div>
+              <div className="flex flex-col gap-0.5 ml-2 border-l border-border/40 pl-3">
+                {Array.from({ length: 2 }).map((_, j) => (
+                  <div key={j} className="flex items-center gap-3 px-3 py-2">
+                    <div className="w-3 h-3 rounded bg-muted shrink-0" />
+                    <div className="flex-1 space-y-1">
+                      <div className="h-4 w-40 bg-muted rounded" />
+                      <div className="h-3 w-24 bg-muted rounded" />
+                    </div>
+                    <div className="h-3 w-12 bg-muted rounded shrink-0" />
+                  </div>
+                ))}
+              </div>
+            </div>
+          ))}
+        </div>
+      </div>
+    </div>
+  );
+}

package/app/app/view/[...path]/loading.tsx

@@ -1,14 +1,37 @@
+/**
+ * File viewer skeleton — shown while file content loads.
+ * Matches the prose layout with heading + paragraph blocks.
+ */
 export default function Loading() {
   return (
-    <div className="flex items-center justify-center min-h-[50vh]">
-      <div className="flex flex-col items-center gap-3">
-        <div
-          className="w-6 h-6 border-2 rounded-full animate-spin"
-          style={{ borderColor: 'var(--border)', borderTopColor: 'var(--amber)' }}
-        />
-        <span className="text-xs text-muted-foreground font-display">
-          Loading...
-        </span>
+    <div className="content-width px-4 md:px-6 py-10 md:py-14 animate-pulse" aria-busy="true" aria-label="Loading">
+      {/* Title skeleton */}
+      <div className="h-8 w-64 bg-muted rounded mb-3" />
+      <div className="h-px w-full bg-border mb-8" />
+
+      {/* Paragraph skeletons */}
+      <div className="space-y-6">
+        {Array.from({ length: 4 }).map((_, i) => (
+          <div key={i} className="space-y-2">
+            {i % 2 === 0 && <div className="h-5 w-40 bg-muted rounded mb-3" />}
+            <div className="h-3.5 w-full bg-muted rounded" />
+            <div className="h-3.5 w-full bg-muted rounded" />
+            <div className="h-3.5 w-3/4 bg-muted rounded" />
+          </div>
+        ))}
+
+        {/* Code block skeleton */}
+        <div className="rounded-lg border border-border p-4 space-y-2">
+          <div className="h-3 w-48 bg-muted rounded" />
+          <div className="h-3 w-64 bg-muted rounded" />
+          <div className="h-3 w-36 bg-muted rounded" />
+        </div>
+
+        {/* More paragraphs */}
+        <div className="space-y-2">
+          <div className="h-3.5 w-full bg-muted rounded" />
+          <div className="h-3.5 w-5/6 bg-muted rounded" />
+        </div>
       </div>
     </div>
   );

package/app/components/ActivityBar.tsx

@@ -48,7 +48,7 @@ function RailButton({ icon, label, shortcut, active = false, expanded, onClick,
       onClick={onClick}
       aria-pressed={active}
       aria-label={label}
-      title={expanded ? undefined : (shortcut ? `${label} (${shortcut})` : label)}
+      title={shortcut ? `${label} (${shortcut})` : label}
       data-walkthrough={walkthroughId}
       className={`
         relative flex items-center ${expanded ? 'justify-start px-3 w-full' : 'justify-center w-10'} h-10 rounded-md transition-colors

package/app/components/AskFab.tsx

@@ -22,16 +22,16 @@ export default function AskFab({ onToggle, askPanelOpen }: AskFabProps) {
         fixed z-40 bottom-5 right-5
         items-center justify-center
         gap-0 hover:gap-2
-        p-[11px] rounded-xl
+        p-3 rounded-xl
         text-white font-medium text-[13px]
-        shadow-md shadow-amber-900/15
+        shadow-sm shadow-amber-900/10
         transition-all duration-200 ease-out
-        hover:shadow-lg hover:shadow-amber-800/25
+        hover:shadow-md hover:shadow-amber-900/15
         active:scale-95 cursor-pointer overflow-hidden font-display
         ${askPanelOpen ? 'opacity-0 pointer-events-none translate-y-2' : 'opacity-100 translate-y-0'}
       `}
       style={{
-        background: 'linear-gradient(135deg, var(--amber), color-mix(in srgb, var(--amber) 80%, white))',
+        background: 'var(--amber)',
       }}
       title={label}
       aria-label={label}

package/app/components/ExportModal.tsx

@@ -104,13 +104,16 @@ export default function ExportModal({ open, onClose, filePath, isDirectory, file
     <div className="fixed inset-0 z-50 flex items-center justify-center bg-black/40 backdrop-blur-sm" onClick={state === 'exporting' ? undefined : handleClose}>
       <div
         className="bg-card border border-border rounded-xl shadow-xl max-w-md w-full mx-4 animate-in fade-in-0 zoom-in-95 duration-200"
+        role="dialog"
+        aria-modal="true"
+        aria-labelledby="export-modal-title"
         onClick={e => e.stopPropagation()}
       >
         {/* Header */}
         <div className="flex items-center justify-between px-5 py-4 border-b border-border">
           <div className="flex items-center gap-2">
             <Download size={15} className="text-[var(--amber)]" />
-            <h3 className="text-sm font-semibold font-display">
+            <h3 id="export-modal-title" className="text-sm font-semibold font-display">
               {isDirectory ? (t.export?.exportSpace ?? 'Export Space') : (t.export?.exportFile ?? 'Export File')}
             </h3>
           </div>

package/app/components/FileTree.tsx

@@ -4,6 +4,7 @@ import { useState, useCallback, useRef, useTransition, useEffect, useSyncExterna
 import { useRouter, usePathname } from 'next/navigation';
 import { FileNode, SYSTEM_FILES, UNDELETABLE_FILES } from '@/lib/types';
 import { encodePath } from '@/lib/utils';
+import { ICON_SIZES } from '@/lib/config/icon-scale';
 import {
   ChevronDown, FileText, Table, Folder, FolderOpen, Plus, Loader2,
   Trash2, Pencil, Layers, ScrollText, FolderInput, Copy, MoreHorizontal, Star, Inbox,
@@ -61,8 +62,8 @@ interface FileTreeProps {
 
 function getIcon(node: FileNode) {
   if (node.type === 'directory') return null;
-  if (node.extension === '.csv') return <Table size={14} className="text-success shrink-0" />;
-  return <FileText size={14} className="text-muted-foreground shrink-0" />;
+  if (node.extension === '.csv') return <Table size={ICON_SIZES.md} className="text-success shrink-0" />;
+  return <FileText size={ICON_SIZES.md} className="text-muted-foreground shrink-0" />;
 }
 
 function getCurrentFilePath(pathname: string): string {
@@ -428,9 +429,10 @@ function DirectoryNode({ node, depth, currentPath, onNavigate, maxOpenDepth, onI
           className="shrink-0 p-1 rounded hover:bg-muted text-muted-foreground transition-colors"
           style={{ marginLeft: `${depth * 12 + 4}px` }}
           aria-label={open ? `Collapse ${node.name}` : `Expand ${node.name}`}
+          aria-expanded={open}
         >
           <span className="block transition-transform duration-150" style={{ transform: open ? 'rotate(0deg)' : 'rotate(-90deg)' }}>
-            <ChevronDown size={13} />
+            <ChevronDown size={ICON_SIZES.xs} />
           </span>
         </button>
         <button
@@ -446,7 +448,7 @@ function DirectoryNode({ node, depth, currentPath, onNavigate, maxOpenDepth, onI
           className={`
             flex-1 flex items-center gap-1.5 px-1 py-1 rounded text-left min-w-0 pr-16
             text-sm transition-colors duration-100
-            hover:bg-muted
+            hover:bg-muted cursor-text
             ${isActive ? 'text-foreground' : 'text-muted-foreground hover:text-foreground'}
           `}
         >
@@ -459,6 +461,9 @@ function DirectoryNode({ node, depth, currentPath, onNavigate, maxOpenDepth, onI
               : <Folder size={14} className="text-yellow-400 shrink-0" />
           }
           <span className="truncate leading-5" suppressHydrationWarning>{node.name}</span>
+          <span className="ml-auto text-xs text-muted-foreground/30 group-hover/dir:text-muted-foreground/60 transition-colors shrink-0 hidden group-hover/dir:inline-flex items-center gap-1">
+            <Pencil size={10} />
+          </span>
           {isSpace && !open && (
             <span className="ml-auto text-xs text-muted-foreground shrink-0 tabular-nums pr-1">{contentCount}</span>
           )}
@@ -659,7 +664,7 @@ function FileNodeItem({ node, depth, currentPath, onNavigate }: {
         data-filepath={node.path}
         className={`
           w-full flex items-center gap-1.5 px-2 py-1 rounded text-left
-          text-sm transition-colors duration-100 cursor-pointer pr-16
+          text-sm transition-colors duration-100 cursor-text pr-16
           ${isActive
             ? 'bg-accent text-foreground'
             : 'hover:bg-muted text-muted-foreground hover:text-foreground'
@@ -669,6 +674,9 @@ function FileNodeItem({ node, depth, currentPath, onNavigate }: {
       >
         {getIcon(node)}
         <span className="truncate leading-5" suppressHydrationWarning>{node.name}</span>
+        <span className="ml-auto text-xs text-muted-foreground/30 group-hover/file:text-muted-foreground/60 transition-colors shrink-0 hidden group-hover/file:inline-flex items-center gap-1">
+          <Pencil size={10} />
+        </span>
         {pinned && <Star size={10} className="shrink-0 fill-[var(--amber)] text-[var(--amber)] opacity-60" />}
       </button>
       <div className="absolute right-1 top-1/2 -translate-y-1/2 hidden group-hover/file:flex items-center gap-0.5">