@geminilight/mindos 0.5.59 → 0.5.62

package/app/app/agents/[agentKey]/page.tsx

@@ -0,0 +1,17 @@
+import { redirect } from 'next/navigation';
+import { readSettings } from '@/lib/settings';
+import AgentDetailContent from '@/components/agents/AgentDetailContent';
+
+export const dynamic = 'force-dynamic';
+
+export default async function AgentDetailPage({
+  params,
+}: {
+  params: Promise<{ agentKey: string }>;
+}) {
+  const settings = readSettings();
+  if (settings.setupPending) redirect('/setup');
+
+  const { agentKey } = await params;
+  return <AgentDetailContent agentKey={decodeURIComponent(agentKey)} />;
+}

package/app/app/agents/page.tsx

@@ -0,0 +1,20 @@
+import { redirect } from 'next/navigation';
+import { readSettings } from '@/lib/settings';
+import AgentsContentPage from '@/components/agents/AgentsContentPage';
+import { parseAgentsTab } from '@/components/agents/agents-content-model';
+
+export const dynamic = 'force-dynamic';
+
+export default async function AgentsPage({
+  searchParams,
+}: {
+  searchParams: Promise<{ tab?: string }>;
+}) {
+  const settings = readSettings();
+  if (settings.setupPending) redirect('/setup');
+
+  const params = await searchParams;
+  const tab = parseAgentsTab(params.tab);
+
+  return <AgentsContentPage tab={tab} />;
+}

package/app/app/api/changes/route.ts

@@ -0,0 +1,57 @@
+import { NextRequest, NextResponse } from 'next/server';
+import {
+  listContentChanges,
+  getContentChangeSummary,
+  markContentChangesSeen,
+} from '@/lib/fs';
+
+export const dynamic = 'force-dynamic';
+
+function err(message: string, status = 400) {
+  return NextResponse.json({ error: message }, { status });
+}
+
+export async function GET(req: NextRequest) {
+  const op = req.nextUrl.searchParams.get('op') ?? 'summary';
+
+  try {
+    if (op === 'summary') {
+      const summary = getContentChangeSummary();
+      return NextResponse.json(summary);
+    }
+
+    if (op === 'list') {
+      const path = req.nextUrl.searchParams.get('path') ?? undefined;
+      const limitParam = req.nextUrl.searchParams.get('limit');
+      const limit = limitParam ? Number(limitParam) : 50;
+      if (!Number.isFinite(limit) || limit <= 0) return err('invalid limit');
+      return NextResponse.json({ events: listContentChanges({ path, limit }) });
+    }
+
+    return err(`unknown op: ${op}`);
+  } catch (error) {
+    return err((error as Error).message, 500);
+  }
+}
+
+export async function POST(req: NextRequest) {
+  let body: Record<string, unknown>;
+  try {
+    body = await req.json();
+  } catch {
+    return err('invalid JSON');
+  }
+
+  const op = body.op;
+  if (typeof op !== 'string') return err('missing op');
+
+  try {
+    if (op === 'mark_seen') {
+      markContentChangesSeen();
+      return NextResponse.json({ ok: true });
+    }
+    return err(`unknown op: ${op}`);
+  } catch (error) {
+    return err((error as Error).message, 500);
+  }
+}

package/app/app/api/file/route.ts

@@ -19,6 +19,7 @@ import {
   getMindRoot,
   invalidateCache,
   listMindSpaces,
+  appendContentChange,
 } from '@/lib/fs';
 import { createSpaceFilesystem } from '@/lib/core/create-space';
 
@@ -26,6 +27,22 @@ function err(msg: string, status = 400) {
   return NextResponse.json({ error: msg }, { status });
 }
 
+function safeRead(filePath: string): string {
+  try {
+    return getFileContent(filePath);
+  } catch {
+    return '';
+  }
+}
+
+function sourceFromRequest(req: NextRequest, body: Record<string, unknown>) {
+  const bodySource = body.source;
+  if (bodySource === 'agent' || bodySource === 'user' || bodySource === 'system') return bodySource;
+  const headerSource = req.headers.get('x-mindos-source');
+  if (headerSource === 'agent' || headerSource === 'user' || headerSource === 'system') return headerSource;
+  return 'user' as const;
+}
+
 // GET /api/file?path=foo.md&op=read_file|read_lines | GET ?op=list_spaces (no path)
 export async function GET(req: NextRequest) {
   const filePath = req.nextUrl.searchParams.get('path');
@@ -73,13 +90,32 @@ export async function POST(req: NextRequest) {
 
   try {
     let resp: NextResponse;
+    let changeEvent:
+      | {
+        op: string;
+        path: string;
+        summary: string;
+        before?: string;
+        after?: string;
+        beforePath?: string;
+        afterPath?: string;
+      }
+      | null = null;
 
     switch (op) {
 
       case 'save_file': {
         const { content } = params as { content: string };
         if (typeof content !== 'string') return err('missing content');
+        const before = safeRead(filePath);
         saveFileContent(filePath, content);
+        changeEvent = {
+          op,
+          path: filePath,
+          summary: 'Updated file content',
+          before,
+          after: content,
+        };
         resp = NextResponse.json({ ok: true });
         break;
       }
@@ -87,7 +123,15 @@ export async function POST(req: NextRequest) {
       case 'append_to_file': {
         const { content } = params as { content: string };
         if (typeof content !== 'string') return err('missing content');
+        const before = safeRead(filePath);
         appendToFile(filePath, content);
+        changeEvent = {
+          op,
+          path: filePath,
+          summary: 'Appended content to file',
+          before,
+          after: safeRead(filePath),
+        };
         resp = NextResponse.json({ ok: true });
         break;
       }
@@ -96,7 +140,15 @@ export async function POST(req: NextRequest) {
         const { after_index, lines } = params as { after_index: number; lines: string[] };
         if (typeof after_index !== 'number') return err('missing after_index');
         if (!Array.isArray(lines)) return err('lines must be array');
+        const before = safeRead(filePath);
         insertLines(filePath, after_index, lines);
+        changeEvent = {
+          op,
+          path: filePath,
+          summary: `Inserted ${lines.length} line(s)`,
+          before,
+          after: safeRead(filePath),
+        };
         resp = NextResponse.json({ ok: true });
         break;
       }
@@ -107,7 +159,15 @@ export async function POST(req: NextRequest) {
         if (!Array.isArray(lines)) return err('lines must be array');
         if (start < 0 || end < 0) return err('start/end must be >= 0');
         if (start > end) return err('start must be <= end');
+        const before = safeRead(filePath);
         updateLines(filePath, start, end, lines);
+        changeEvent = {
+          op,
+          path: filePath,
+          summary: `Updated lines ${start}-${end}`,
+          before,
+          after: safeRead(filePath),
+        };
         resp = NextResponse.json({ ok: true });
         break;
       }
@@ -116,7 +176,15 @@ export async function POST(req: NextRequest) {
         const { heading, content } = params as { heading: string; content: string };
         if (typeof heading !== 'string') return err('missing heading');
         if (typeof content !== 'string') return err('missing content');
+        const before = safeRead(filePath);
         insertAfterHeading(filePath, heading, content);
+        changeEvent = {
+          op,
+          path: filePath,
+          summary: `Inserted content after heading "${heading}"`,
+          before,
+          after: safeRead(filePath),
+        };
         resp = NextResponse.json({ ok: true });
         break;
       }
@@ -125,13 +193,29 @@ export async function POST(req: NextRequest) {
         const { heading, content } = params as { heading: string; content: string };
         if (typeof heading !== 'string') return err('missing heading');
         if (typeof content !== 'string') return err('missing content');
+        const before = safeRead(filePath);
         updateSection(filePath, heading, content);
+        changeEvent = {
+          op,
+          path: filePath,
+          summary: `Updated section "${heading}"`,
+          before,
+          after: safeRead(filePath),
+        };
         resp = NextResponse.json({ ok: true });
         break;
       }
 
       case 'delete_file': {
+        const before = safeRead(filePath);
         deleteFile(filePath);
+        changeEvent = {
+          op,
+          path: filePath,
+          summary: 'Deleted file',
+          before,
+          after: '',
+        };
         resp = NextResponse.json({ ok: true });
         break;
       }
@@ -139,14 +223,32 @@ export async function POST(req: NextRequest) {
       case 'rename_file': {
         const { new_name } = params as { new_name: string };
         if (typeof new_name !== 'string' || !new_name) return err('missing new_name');
+        const before = safeRead(filePath);
         const newPath = renameFile(filePath, new_name);
+        changeEvent = {
+          op,
+          path: newPath,
+          summary: `Renamed file to ${new_name}`,
+          before,
+          after: safeRead(newPath),
+          beforePath: filePath,
+          afterPath: newPath,
+        };
         resp = NextResponse.json({ ok: true, newPath });
         break;
       }
 
       case 'create_file': {
         const { content } = params as { content?: string };
-        createFile(filePath, typeof content === 'string' ? content : '');
+        const after = typeof content === 'string' ? content : '';
+        createFile(filePath, after);
+        changeEvent = {
+          op,
+          path: filePath,
+          summary: 'Created file',
+          before: '',
+          after,
+        };
         resp = NextResponse.json({ ok: true });
         break;
       }
@@ -154,7 +256,17 @@ export async function POST(req: NextRequest) {
       case 'move_file': {
         const { to_path } = params as { to_path: string };
         if (typeof to_path !== 'string' || !to_path) return err('missing to_path');
+        const before = safeRead(filePath);
         const result = moveFile(filePath, to_path);
+        changeEvent = {
+          op,
+          path: result.newPath,
+          summary: `Moved file to ${result.newPath}`,
+          before,
+          after: safeRead(result.newPath),
+          beforePath: filePath,
+          afterPath: result.newPath,
+        };
         resp = NextResponse.json({ ok: true, ...result });
         break;
       }
@@ -169,6 +281,13 @@ export async function POST(req: NextRequest) {
         try {
           const { path: spacePath } = createSpaceFilesystem(getMindRoot(), name, description, parent_path);
           invalidateCache();
+          changeEvent = {
+            op,
+            path: spacePath,
+            summary: 'Created space',
+            before: '',
+            after: description,
+          };
           resp = NextResponse.json({ ok: true, path: spacePath });
         } catch (e) {
           const msg = (e as Error).message;
@@ -186,6 +305,13 @@ export async function POST(req: NextRequest) {
         const { new_name } = params as { new_name: string };
         if (typeof new_name !== 'string' || !new_name.trim()) return err('missing new_name');
         const newPath = renameSpace(filePath, new_name.trim());
+        changeEvent = {
+          op,
+          path: newPath,
+          summary: `Renamed space to ${new_name.trim()}`,
+          beforePath: filePath,
+          afterPath: newPath,
+        };
         resp = NextResponse.json({ ok: true, newPath });
         break;
       }
@@ -193,7 +319,15 @@ export async function POST(req: NextRequest) {
       case 'append_csv': {
         const { row } = params as { row: string[] };
         if (!Array.isArray(row) || row.length === 0) return err('row must be non-empty array');
+        const before = safeRead(filePath);
         const result = appendCsvRow(filePath, row);
+        changeEvent = {
+          op,
+          path: filePath,
+          summary: `Appended CSV row (${row.length} cell${row.length === 1 ? '' : 's'})`,
+          before,
+          after: safeRead(filePath),
+        };
         resp = NextResponse.json({ ok: true, ...result });
         break;
       }
@@ -207,6 +341,17 @@ export async function POST(req: NextRequest) {
       try { revalidatePath('/', 'layout'); } catch { /* noop in test env */ }
     }
 
+    if (changeEvent) {
+      try {
+        appendContentChange({
+          ...changeEvent,
+          source: sourceFromRequest(req, body),
+        });
+      } catch (logError) {
+        console.warn('[file.route] failed to append content change log:', (logError as Error).message);
+      }
+    }
+
     return resp;
   } catch (e) {
     return err((e as Error).message, 500);

package/app/app/api/settings/test-key/route.ts

@@ -62,7 +62,49 @@ async function testOpenAI(apiKey: string, model: string, baseUrl: string): Promi
       signal: ctrl.signal,
     });
     const latency = Date.now() - start;
-    if (res.ok) return { ok: true, latency };
+    if (res.ok) {
+      // `/api/ask` always sends tool definitions, so key test should verify this
+      // compatibility as well (not just plain chat completion).
+      const toolRes = await fetch(url, {
+        method: 'POST',
+        headers: {
+          'Content-Type': 'application/json',
+          'Authorization': `Bearer ${apiKey}`,
+        },
+        body: JSON.stringify({
+          model,
+          max_tokens: 1,
+          messages: [
+            { role: 'system', content: 'You are a helpful assistant.' },
+            { role: 'user', content: 'hi' },
+          ],
+          tools: [{
+            type: 'function',
+            function: {
+              name: 'noop',
+              description: 'No-op function used for compatibility checks.',
+              parameters: {
+                type: 'object',
+                properties: {},
+                additionalProperties: false,
+              },
+            },
+          }],
+          tool_choice: 'none',
+        }),
+        signal: ctrl.signal,
+      });
+
+      if (toolRes.ok) return { ok: true, latency };
+
+      const toolBody = await toolRes.text();
+      const toolErr = classifyError(toolRes.status, toolBody);
+      return {
+        ok: false,
+        code: toolErr.code,
+        error: `Model endpoint passes basic test but is incompatible with agent tool calls: ${toolErr.error}`,
+      };
+    }
     const body = await res.text();
     return { ok: false, ...classifyError(res.status, body) };
   } catch (e: unknown) {

package/app/app/changes/page.tsx

@@ -0,0 +1,16 @@
+import { redirect } from 'next/navigation';
+import { readSettings } from '@/lib/settings';
+import ChangesContentPage from '@/components/changes/ChangesContentPage';
+
+export const dynamic = 'force-dynamic';
+
+export default async function ChangesPage({
+  searchParams,
+}: {
+  searchParams: Promise<{ path?: string }>;
+}) {
+  const settings = readSettings();
+  if (settings.setupPending) redirect('/setup');
+  const params = await searchParams;
+  return <ChangesContentPage initialPath={params.path ?? ''} />;
+}

package/app/components/ActivityBar.tsx

@@ -16,6 +16,7 @@ export const RAIL_WIDTH_EXPANDED = 180;
 interface ActivityBarProps {
   activePanel: PanelId | null;
   onPanelChange: (id: PanelId | null) => void;
+  onAgentsClick?: () => void;
   syncStatus: SyncStatus | null;
   expanded: boolean;
   onExpandedChange: (expanded: boolean) => void;
@@ -76,6 +77,7 @@ function RailButton({ icon, label, shortcut, active = false, expanded, onClick,
 export default function ActivityBar({
   activePanel,
   onPanelChange,
+  onAgentsClick,
   syncStatus,
   expanded,
   onExpandedChange,
@@ -160,7 +162,13 @@ export default function ActivityBar({
           <RailButton icon={<Radio size={18} />} label={t.sidebar.echo} active={activePanel === 'echo'} expanded={expanded} onClick={() => toggle('echo')} />
           <RailButton icon={<Search size={18} />} label={t.sidebar.searchTitle} shortcut="⌘K" active={activePanel === 'search'} expanded={expanded} onClick={() => toggle('search')} walkthroughId="search-button" />
           <RailButton icon={<Blocks size={18} />} label={t.sidebar.plugins} active={activePanel === 'plugins'} expanded={expanded} onClick={() => toggle('plugins')} />
-          <RailButton icon={<Bot size={18} />} label={t.sidebar.agents} active={activePanel === 'agents'} expanded={expanded} onClick={() => toggle('agents')} />
+          <RailButton
+            icon={<Bot size={18} />}
+            label={t.sidebar.agents}
+            active={activePanel === 'agents'}
+            expanded={expanded}
+            onClick={() => onAgentsClick ? debounced(onAgentsClick) : toggle('agents')}
+          />
           <RailButton icon={<Compass size={18} />} label={t.sidebar.discover} active={activePanel === 'discover'} expanded={expanded} onClick={() => toggle('discover')} />
         </div>
 

package/app/components/SidebarLayout.tsx

@@ -25,6 +25,7 @@ import SearchModal from './SearchModal';
 import AskModal from './AskModal';
 import SettingsModal from './SettingsModal';
 import KeyboardShortcuts from './KeyboardShortcuts';
+import ChangesBanner from './changes/ChangesBanner';
 import { MobileSyncDot, useSyncStatus } from './SyncStatusBar';
 import { FileNode } from '@/lib/types';
 import { useLocale } from '@/lib/LocaleContext';
@@ -81,6 +82,8 @@ export default function SidebarLayout({ fileTree, children }: SidebarLayoutProps
   const currentFile = pathname.startsWith('/view/')
     ? pathname.slice('/view/'.length).split('/').map(decodeURIComponent).join('/')
     : undefined;
+  const agentsContentActive = pathname?.startsWith('/agents');
+  const railActivePanel = lp.activePanel ?? (agentsContentActive ? 'agents' : null);
 
   // ── Event listeners ──
 
@@ -228,8 +231,12 @@ export default function SidebarLayout({ fileTree, children }: SidebarLayoutProps
 
       {/* ── Desktop: Activity Bar + Panel ── */}
       <ActivityBar
-        activePanel={lp.activePanel}
+        activePanel={railActivePanel}
         onPanelChange={lp.setActivePanel}
+        onAgentsClick={() => {
+          lp.setActivePanel((current) => (current === 'agents' ? null : 'agents'));
+          setAgentDetailKey(null);
+        }}
         syncStatus={syncStatus}
         expanded={lp.railExpanded}
         onExpandedChange={handleExpandedChange}
@@ -364,7 +371,10 @@ export default function SidebarLayout({ fileTree, children }: SidebarLayoutProps
       <AskModal open={mobileAskOpen} onClose={() => setMobileAskOpen(false)} currentFile={currentFile} />
 
       <main id="main-content" className="min-h-screen transition-all duration-200 pt-[52px] md:pt-0">
-        <div className="min-h-screen bg-background">{children}</div>
+        <div className="min-h-screen bg-background">
+          <ChangesBanner />
+          {children}
+        </div>
       </main>
 
       <style>{`

package/app/components/agents/AgentDetailContent.tsx

@@ -0,0 +1,117 @@
+'use client';
+
+import Link from 'next/link';
+import { useMemo } from 'react';
+import { ArrowLeft, Server, ShieldCheck, Activity, Compass } from 'lucide-react';
+import { useLocale } from '@/lib/LocaleContext';
+import { useMcpData } from '@/hooks/useMcpData';
+import { resolveAgentStatus } from './agents-content-model';
+
+export default function AgentDetailContent({ agentKey }: { agentKey: string }) {
+  const { t } = useLocale();
+  const a = t.agentsContent;
+  const mcp = useMcpData();
+
+  const agent = useMemo(() => mcp.agents.find((item) => item.key === agentKey), [mcp.agents, agentKey]);
+  const enabledSkills = useMemo(() => mcp.skills.filter((s) => s.enabled), [mcp.skills]);
+
+  if (!agent) {
+    return (
+      <div className="content-width px-4 md:px-6 py-8 md:py-10">
+        <Link href="/agents" className="inline-flex items-center gap-1.5 text-sm text-muted-foreground hover:text-foreground mb-4">
+          <ArrowLeft size={14} />
+          {a.backToOverview}
+        </Link>
+        <div className="rounded-lg border border-border bg-card p-6">
+          <p className="text-sm text-foreground">{a.detailNotFound}</p>
+        </div>
+      </div>
+    );
+  }
+
+  const status = resolveAgentStatus(agent);
+
+  return (
+    <div className="content-width px-4 md:px-6 py-8 md:py-10 space-y-4">
+      <div>
+        <Link href="/agents" className="inline-flex items-center gap-1.5 text-sm text-muted-foreground hover:text-foreground mb-3">
+          <ArrowLeft size={14} />
+          {a.backToOverview}
+        </Link>
+        <h1 className="text-2xl font-semibold tracking-tight font-display text-foreground">{agent.name}</h1>
+        <p className="mt-1 text-sm text-muted-foreground">{a.detailSubtitle}</p>
+      </div>
+
+      <section className="rounded-lg border border-border bg-card p-4">
+        <h2 className="text-sm font-medium text-foreground mb-2">{a.detail.identity}</h2>
+        <div className="grid grid-cols-1 md:grid-cols-3 gap-3 text-sm">
+          <DetailLine label={a.detail.agentKey} value={agent.key} />
+          <DetailLine label={a.detail.status} value={status} />
+          <DetailLine label={a.detail.transport} value={agent.transport ?? agent.preferredTransport} />
+        </div>
+      </section>
+
+      <section className="rounded-lg border border-border bg-card p-4">
+        <h2 className="text-sm font-medium text-foreground mb-2 flex items-center gap-1.5">
+          <Server size={14} className="text-muted-foreground" />
+          {a.detail.connection}
+        </h2>
+        <div className="grid grid-cols-1 md:grid-cols-3 gap-3 text-sm">
+          <DetailLine label={a.detail.endpoint} value={mcp.status?.endpoint ?? a.na} />
+          <DetailLine label={a.detail.port} value={String(mcp.status?.port ?? a.na)} />
+          <DetailLine label={a.detail.auth} value={mcp.status?.authConfigured ? a.detail.authConfigured : a.detail.authMissing} />
+        </div>
+      </section>
+
+      <section className="rounded-lg border border-border bg-card p-4">
+        <h2 className="text-sm font-medium text-foreground mb-2 flex items-center gap-1.5">
+          <ShieldCheck size={14} className="text-muted-foreground" />
+          {a.detail.capabilities}
+        </h2>
+        <ul className="text-sm text-muted-foreground space-y-1">
+          <li>{a.detail.projectScope}: {agent.hasProjectScope ? a.detail.yes : a.detail.no}</li>
+          <li>{a.detail.globalScope}: {agent.hasGlobalScope ? a.detail.yes : a.detail.no}</li>
+          <li>{a.detail.format}: {agent.format}</li>
+        </ul>
+      </section>
+
+      <section className="rounded-lg border border-border bg-card p-4">
+        <h2 className="text-sm font-medium text-foreground mb-2">{a.detail.skillAssignments}</h2>
+        {enabledSkills.length === 0 ? (
+          <p className="text-sm text-muted-foreground">{a.detail.noSkills}</p>
+        ) : (
+          <ul className="text-sm text-muted-foreground space-y-1">
+            {enabledSkills.map((skill) => (
+              <li key={skill.name}>- {skill.name}</li>
+            ))}
+          </ul>
+        )}
+      </section>
+
+      <section className="rounded-lg border border-border bg-card p-4">
+        <h2 className="text-sm font-medium text-foreground mb-2 flex items-center gap-1.5">
+          <Activity size={14} className="text-muted-foreground" />
+          {a.detail.recentActivity}
+        </h2>
+        <p className="text-sm text-muted-foreground">{a.detail.noActivity}</p>
+      </section>
+
+      <section className="rounded-lg border border-border bg-card p-4">
+        <h2 className="text-sm font-medium text-foreground mb-2 flex items-center gap-1.5">
+          <Compass size={14} className="text-muted-foreground" />
+          {a.detail.spaceReach}
+        </h2>
+        <p className="text-sm text-muted-foreground">{a.detail.noSpaceReach}</p>
+      </section>
+    </div>
+  );
+}
+
+function DetailLine({ label, value }: { label: string; value: string }) {
+  return (
+    <div className="rounded-md border border-border px-3 py-2">
+      <p className="text-2xs text-muted-foreground mb-1">{label}</p>
+      <p className="text-sm text-foreground truncate">{value}</p>
+    </div>
+  );
+}