Address /export-with-traces review: scrub, scope, and UX

Make tool-chain export safer and clearer: conversation-scoped listRuns,
resilient JSON serialization, event-load failure signals, distinct empty/
success copy, busy-state locale strings, and wiring tests for both browsers.

Author: esokullu

src/chrome/src/agent/agent.js

@@ -6320,24 +6320,58 @@ Rules: no prose intro, no conclusion, no "this screenshot shows...", no layout d
     await this._hydrate(tabId);
     const conversationId = this.conversationIds.get(tabId);
     if (!conversationId) return { ok: true, markdown: null, turnCount: 0, reason: 'no-conversation' };
+    // Cap matching runs for this conversation only (not a global newest-N).
+    const RUN_LIMIT = 500;
     let runs;
+    let truncated = false;
     try {
-      const all = await trace.listRuns();
-      runs = all
+      const matched = await trace.listRuns({ limit: RUN_LIMIT, conversationId });
+      truncated = matched.length >= RUN_LIMIT;
+      runs = matched
         .filter((r) => r && r.conversationId === conversationId)
         .sort((a, b) => (a.startedAt || 0) - (b.startedAt || 0));
     } catch (e) {
       return { ok: false, error: String((e && e.message) || e) };
     }
     if (runs.length === 0) return { ok: true, markdown: null, turnCount: 0, reason: 'no-traces' };
     const withEvents = [];
+    let failedEventLoads = 0;
     for (const run of runs) {
       let events = [];
-      try { events = await trace.getRunEvents(run.runId); } catch (e) { events = []; }
+      try {
+        events = await trace.getRunEvents(run.runId);
+      } catch (e) {
+        events = [];
+        failedEventLoads += 1;
+      }
       withEvents.push({ run, events });
     }
-    const { markdown, turnCount, toolCount } = tracesToMarkdown(withEvents);
-    return { ok: true, markdown, turnCount, toolCount };
+    if (failedEventLoads === runs.length) {
+      return { ok: false, error: 'Could not read any trace events for this conversation.' };
+    }
+    const notes = [];
+    if (failedEventLoads > 0) {
+      notes.push(`${failedEventLoads} of ${runs.length} turn(s) could not load their event log.`);
+    }
+    if (truncated) {
+      notes.push(`Export limited to the ${RUN_LIMIT} most recent traced turns for this conversation.`);
+    }
+    let markdown;
+    let turnCount;
+    let toolCount;
+    try {
+      ({ markdown, turnCount, toolCount } = tracesToMarkdown(withEvents, { notes }));
+    } catch (e) {
+      return { ok: false, error: String((e && e.message) || e) };
+    }
+    return {
+      ok: true,
+      markdown,
+      turnCount,
+      toolCount,
+      partial: failedEventLoads > 0,
+      truncated,
+    };
   }
 
   _isAgentInjectedUserContent(content) {

src/chrome/src/agent/trace-export.js

@@ -47,9 +47,23 @@ function fencedBlock(content) {
   return `${fence}${info}\n${body}\n${fence}`;
 }
 
+// IndexedDB can retain values that JSON.stringify rejects (circular / bigint /
+// sparse). Never throw mid-export — fall back to a readable marker.
+function safeJsonStringify(value) {
+  try {
+    return JSON.stringify(value);
+  } catch {
+    try {
+      return String(value);
+    } catch {
+      return '(unserializable)';
+    }
+  }
+}
+
 function stringifyArgs(args) {
   if (args == null) return '';
-  const s = typeof args === 'string' ? args : JSON.stringify(args);
+  const s = typeof args === 'string' ? args : safeJsonStringify(args);
   return truncate(oneLine(s), ARGS_LIMIT);
 }
 
@@ -64,11 +78,14 @@ function renderResult(result) {
     };
   }
   const failed = typeof result === 'object' ? (result.success === false || !!result.error) : false;
-  const s = typeof result === 'string' ? result : JSON.stringify(result);
+  const s = typeof result === 'string' ? result : safeJsonStringify(result);
   return { text: truncate(oneLine(s), RESULT_LIMIT), failed };
 }
 
-export function tracesToMarkdown(runsWithEvents, { title = 'WebBrain Conversation — tool chain' } = {}) {
+export function tracesToMarkdown(runsWithEvents, {
+  title = 'WebBrain Conversation — tool chain',
+  notes = [],
+} = {}) {
   const runs = Array.isArray(runsWithEvents) ? runsWithEvents : [];
   let md = `# ${title}\n\n`;
   let turnCount = 0;
@@ -109,6 +126,10 @@ export function tracesToMarkdown(runsWithEvents, { title = 'WebBrain Conversatio
     md += '\n';
   }
 
+  for (const note of Array.isArray(notes) ? notes : []) {
+    const line = oneLine(note);
+    if (line) md += `_Note: ${line}_\n`;
+  }
   md += `${FOOTER}\n`;
   return { markdown: md, turnCount, toolCount };
 }

src/chrome/src/trace/recorder.js

@@ -301,19 +301,24 @@ export async function endRun(runId, { status = 'done', finalContent = null } = {
 
 // ----- Reader API (used by traces.html) --------------------------------------
 
-export async function listRuns({ limit = 500 } = {}) {
+export async function listRuns({ limit = 500, conversationId = null } = {}) {
   const db = await openDB();
   const idx = tx(db, ['runs'], 'readonly').objectStore('runs').index('startedAt');
   const out = [];
-  await new Promise((resolve) => {
+  // When conversationId is set, only matching runs count toward `limit`, so a
+  // chat's tool-chain export is not starved by unrelated newer runs.
+  await new Promise((resolve, reject) => {
     const req = idx.openCursor(null, 'prev');
     req.onsuccess = () => {
       const c = req.result;
       if (!c || out.length >= limit) return resolve();
-      out.push(c.value);
+      const row = c.value;
+      if (!conversationId || row?.conversationId === conversationId) {
+        out.push(row);
+      }
       c.continue();
     };
-    req.onerror = () => resolve();
+    req.onerror = () => reject(req.error || new Error('listRuns failed'));
   });
   return out;
 }
@@ -327,15 +332,15 @@ export async function getRunEvents(runId) {
   const db = await openDB();
   const idx = tx(db, ['events'], 'readonly').objectStore('events').index('runId');
   const out = [];
-  await new Promise((resolve) => {
+  await new Promise((resolve, reject) => {
     const req = idx.openCursor(IDBKeyRange.only(runId));
     req.onsuccess = () => {
       const c = req.result;
       if (!c) return resolve();
       out.push(c.value);
       c.continue();
     };
-    req.onerror = () => resolve();
+    req.onerror = () => reject(req.error || new Error('getRunEvents failed'));
   });
   out.sort((a, b) => a.seq - b.seq);
   return out;

src/chrome/src/ui/locales/ar.js

@@ -551,7 +551,7 @@ export default {
   'sp.plan.cancelled': 'تم إلغاء الخطة.',
   'sp.plan.expired': 'لم تعد هذه الخطة قيد المراجعة — تم إلغاء التشغيل.',
   'sp.plan.awaiting_review': 'وافق على الخطة أعلاه أو ألغها قبل إرسال رسالة أخرى.',
-  'sp.slash.busy_only_oob': 'تُضاف الرسائل إلى قائمة الانتظار بينما يكون WebBrain مشغولًا. يمكن فقط لـ /help و /check-progress و /show-scratchpad و /show-memory و /list-schedules و /dangerously-skip-permissions و /screenshot و /export و /verbose العمل فورًا كأوامر slash.',
+  'sp.slash.busy_only_oob': 'تُضاف الرسائل إلى قائمة الانتظار بينما يكون WebBrain مشغولًا. يمكن فقط لـ /help و /check-progress و /show-scratchpad و /show-memory و /list-schedules و /dangerously-skip-permissions و /screenshot و /export و /export-with-traces و /verbose العمل فورًا كأوامر slash.',
   'tool.go_back': 'العودة للخلف',
   'tool.go_forward': 'التقدم للأمام',
   'st.display.search.placeholder': 'البحث في الإعدادات العامة',
@@ -707,4 +707,8 @@ export default {
   "sp.slash.export_traces": "تصدير سلسلة الأدوات (التتبعات)",
   "sp.export_traces.none": "لا توجد تتبعات لهذه المحادثة. فعّل «تسجيل التتبعات» في الإعدادات ثم أعد التشغيل.",
   "sp.export_traces.error": "تعذّر تصدير التتبعات.",
+  "sp.export_traces.done": "تم تصدير سلسلة الأدوات.",
+  "sp.export_traces.no_conversation": "لا يوجد ما يُصدَّر بعد — ابدأ محادثة أولًا.",
+  "sp.export_traces.partial": "تم تصدير سلسلة الأدوات، لكن تعذّر قراءة بعض أحداث الجولات.",
+  "sp.export_traces.truncated": "تم تصدير سلسلة الأدوات. قد تُفقد الجولات الأقدم إذا كان لهذه المحادثة كثير من التتبعات.",
 };

src/chrome/src/ui/locales/en.js

@@ -250,7 +250,7 @@ export default {
   'sp.permissions.disabled_html': '⚠️ <strong>Permission prompts are OFF.</strong> WebBrain will click, type, navigate, run JavaScript, upload, download, record, and schedule future work without asking first. Re-enable this from Settings → Permissions → Ask before consequential actions.',
 
   'sp.help_html': '<strong>Slash Commands</strong><br><code>/help</code> — Show this list<br><code>/schedule</code> — Create a scheduled task<br><code>/list-schedules</code> — Show scheduled tasks<br><code>/check-progress</code> — Show current progress ledger<br><code>/show-scratchpad</code> — Show current scratchpad<br><code>/edit-scratchpad &lt;text&gt;</code> — Append text to the current scratchpad<br><code>/clear-scratchpad</code> — Clear the current scratchpad<br><code>/remember &lt;text&gt;</code> — Save a user preference to memory<br><code>/show-memory</code> — Show saved user memory<br><code>/forget-memory &lt;id&gt;</code> — Forget a saved memory<br><code>/allow-api</code> — Allow API mutations for this conversation<br><code>/dangerously-skip-permissions</code> — Disable permission prompts globally<br><code>/compact</code> — Compact this conversation context<br><code>/verbose</code> — Toggle verbose/compact tool display<br><code>/reset</code> — Clear conversation<br><code>/screenshot</code> — Capture current tab<br><code>/full-page-screenshot</code> — Capture the full page (Chrome only)<br><code>/record</code> — Start recording the current tab<br><code>/record --transcribe</code> — Record and save a Whisper transcript after stop<br><code>/record-full-screen</code> — Record a screen or window (Chrome only)<br><code>/record-full-screen --transcribe</code> — Record screen/window and save a transcript<br><code>/export</code> — Download conversation as Markdown<br><code>/export-with-traces</code> — Export the tool chain (traces)<br><code>/profile</code> — Toggle profile auto-fill<br><code>/vision</code> — Toggle vision mode on active provider<br><code>/ask</code> — Switch to Ask mode<br><code>/act</code> — Switch to Act mode<br><code>/dev</code> — Switch to Dev mode<br><br><strong>Keyboard Shortcuts</strong><br><code>Ctrl/Cmd+/</code> — Focus the input<br><code>Ctrl/Cmd+Shift+A</code> — Switch to Ask mode<br><code>Ctrl/Cmd+Shift+X</code> — Switch to Act mode<br><code>Ctrl/Cmd+Shift+D</code> — Switch to Dev mode<br><code>Escape</code> — Stop the active run<br><code>Escape</code> twice — Stop an active recording',
-  'sp.slash.busy_only_oob': 'Messages are queued while WebBrain is busy. Only /help, /check-progress, /show-scratchpad, /show-memory, /list-schedules, /dangerously-skip-permissions, /screenshot, /export, and /verbose can run immediately as slash commands.',
+  'sp.slash.busy_only_oob': 'Messages are queued while WebBrain is busy. Only /help, /check-progress, /show-scratchpad, /show-memory, /list-schedules, /dangerously-skip-permissions, /screenshot, /export, /export-with-traces, and /verbose can run immediately as slash commands.',
   'sp.compact.nothing_to_compact': 'Nothing to compact yet — there is not enough older context.',
   'sp.compact.busy': 'Cannot compact while a run is in progress — wait for it to finish.',
   'sp.compact.failed': 'Context compaction failed: {error}',
@@ -710,4 +710,8 @@ export default {
   "sp.slash.export_traces": "Export the tool chain (traces)",
   "sp.export_traces.none": "No traces for this conversation. Enable Record traces in Settings, then run again.",
   "sp.export_traces.error": "Couldn't export traces.",
+  "sp.export_traces.done": "Tool chain exported.",
+  "sp.export_traces.no_conversation": "Nothing to export yet — start a conversation first.",
+  "sp.export_traces.partial": "Tool chain exported, but some turn events could not be read.",
+  "sp.export_traces.truncated": "Tool chain exported. Older turns may be missing if this conversation has many traced runs.",
 };

src/chrome/src/ui/locales/es.js

@@ -551,7 +551,7 @@ export default {
   'sp.plan.cancelled': 'Plan cancelado.',
   'sp.plan.expired': 'Este plan ya no está en espera de revisión — la ejecución fue cancelada.',
   'sp.plan.awaiting_review': 'Aprueba o cancela el plan anterior antes de enviar otro mensaje.',
-  'sp.slash.busy_only_oob': 'Los mensajes se ponen en cola mientras WebBrain está ocupado. Solo /help, /check-progress, /show-scratchpad, /show-memory, /list-schedules, /dangerously-skip-permissions, /screenshot, /export y /verbose pueden ejecutarse de inmediato como comandos slash.',
+  'sp.slash.busy_only_oob': 'Los mensajes se ponen en cola mientras WebBrain está ocupado. Solo /help, /check-progress, /show-scratchpad, /show-memory, /list-schedules, /dangerously-skip-permissions, /screenshot, /export, /export-with-traces y /verbose pueden ejecutarse de inmediato como comandos slash.',
   'tool.go_back': 'Volviendo atrás',
   'tool.go_forward': 'Avanzando',
   'st.display.search.placeholder': 'Buscar en ajustes generales',
@@ -707,4 +707,8 @@ export default {
   "sp.slash.export_traces": "Exportar la cadena de herramientas (trazas)",
   "sp.export_traces.none": "No hay trazas para esta conversación. Activa «Registrar trazas» en Ajustes y vuelve a ejecutar.",
   "sp.export_traces.error": "No se pudieron exportar las trazas.",
+  "sp.export_traces.done": "Cadena de herramientas exportada.",
+  "sp.export_traces.no_conversation": "Aún no hay nada que exportar: inicia una conversación primero.",
+  "sp.export_traces.partial": "Cadena de herramientas exportada, pero no se pudieron leer algunos eventos de turno.",
+  "sp.export_traces.truncated": "Cadena de herramientas exportada. Puede faltar turnos antiguos si esta conversación tiene muchas trazas.",
 };

src/chrome/src/ui/locales/fr.js

@@ -551,7 +551,7 @@ export default {
   'sp.plan.cancelled': 'Plan annulé.',
   'sp.plan.expired': 'Ce plan n\'est plus en attente de révision — l\'exécution a été annulée.',
   'sp.plan.awaiting_review': 'Approuvez ou annulez le plan ci-dessus avant d\'envoyer un autre message.',
-  'sp.slash.busy_only_oob': 'Les messages sont mis en file d\'attente pendant que WebBrain est occupé. Seuls /help, /check-progress, /show-scratchpad, /show-memory, /list-schedules, /dangerously-skip-permissions, /screenshot, /export et /verbose peuvent s\'exécuter immédiatement comme commandes slash.',
+  'sp.slash.busy_only_oob': 'Les messages sont mis en file d\'attente pendant que WebBrain est occupé. Seuls /help, /check-progress, /show-scratchpad, /show-memory, /list-schedules, /dangerously-skip-permissions, /screenshot, /export, /export-with-traces et /verbose peuvent s\'exécuter immédiatement comme commandes slash.',
   'tool.go_back': 'Revenir en arrière',
   'tool.go_forward': 'Aller en avant',
   'st.display.search.placeholder': 'Rechercher dans les paramètres généraux',
@@ -707,4 +707,8 @@ export default {
   "sp.slash.export_traces": "Exporter la chaîne d'outils (traces)",
   "sp.export_traces.none": "Aucune trace pour cette conversation. Activez « Enregistrer les traces » dans les paramètres, puis relancez.",
   "sp.export_traces.error": "Impossible d'exporter les traces.",
+  "sp.export_traces.done": "Chaîne d'outils exportée.",
+  "sp.export_traces.no_conversation": "Rien à exporter pour l'instant — commencez une conversation d'abord.",
+  "sp.export_traces.partial": "Chaîne d'outils exportée, mais certains événements de tour n'ont pas pu être lus.",
+  "sp.export_traces.truncated": "Chaîne d'outils exportée. Des tours plus anciens peuvent manquer si cette conversation a beaucoup de traces.",
 };