agentgui 1.0.1061 → 1.0.1062

package/.gm/prd.yml

@@ -3106,3 +3106,24 @@
 - id: docstudio-cue-session-group-eyebrow
   subject: Verify ConversationList's session-group headers (Today/Yesterday/etc) use the --tr-label token + --fg-3 tone consistently, matching docstudio's uppercase letter-spaced low-opacity eyebrow label idiom
   status: pending
+- id: gc-delete-undo-trash
+  subject: No undo-after-delete (soft-delete/trash) anywhere in the confined delete surface
+  status: pending
+- id: gc-search-hit-hash-anchor
+  subject: Search-hit event anchor (focusEventI/focusEventTs) not carried in the hash - reload/Back loses highlighted line
+  status: pending
+- id: gc-settings-section-scrollspy
+  subject: Settings section anchor is deep-link-in only - scrolling never updates state.settingsSection or the URL
+  status: pending
+- id: gc-context-budget-affordance
+  subject: No context-size/token-budget affordance - only turn count and dollar cost shown, never remaining context headroom
+  status: pending
+- id: gc-expanded-body-highlight
+  subject: Search-result highlighting inside expanded event bodies never highlights the matched query term
+  status: pending
+- id: gc-search-results-export
+  subject: No per-session or global export of History search results themselves
+  status: pending
+- id: gc-settings-storage-estimate
+  subject: Settings has no cache/data breakdown beyond agentgui's own localStorage keys - no navigator.storage.estimate()
+  status: pending

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentgui",
-  "version": "1.0.1061",
+  "version": "1.0.1062",
   "description": "Multi-agent ACP client with real-time communication",
   "type": "module",
   "main": "electron/main.js",

package/site/app/js/app.js

@@ -1012,6 +1012,15 @@ function fileMutationCopy(e) {
   if (e.status === 413) return 'Too large (50MB upload cap).';
   return e.message || 'The operation failed.';
 }
+// On a 409 name-collision, suggest 'name (2)' / 'name (3)' etc instead of
+// leaving the user to retype the identical blocked string from scratch -
+// mirrors the numbering convention most desktop file managers use.
+function suggestAlternateName(name) {
+  const m = /^(.*) \((\d+)\)(\.[^.]*)?$/.exec(name);
+  if (m) return m[1] + ' (' + (parseInt(m[2], 10) + 1) + ')' + (m[3] || '');
+  const dot = name.lastIndexOf('.');
+  return dot > 0 ? name.slice(0, dot) + ' (2)' + name.slice(dot) : name + ' (2)';
+}
 function openFileDialog(kind, file) {
   const trigger = typeof document !== 'undefined' ? document.activeElement : null;
   state.files.dialog = { kind, file: file || null, error: null, busy: false, _trigger: trigger || null };
@@ -1285,7 +1294,7 @@ function fileDialog() {
   // flow, so a sibling alert was invisible and outside the focus trap).
   if (d.kind === 'rename') {
     return PromptDialog({
-      title: 'Rename ' + d.file.name, value: d.file.name, placeholder: 'new name',
+      title: 'Rename ' + d.file.name, value: d.suggestedValue ?? d.file.name, placeholder: 'new name',
       error: d.error || null, busy: !!d.busy,
       confirmLabel: d.busy ? 'renaming…' : 'rename', cancelLabel: 'cancel',
       onCancel: closeFileDialog,
@@ -1295,7 +1304,17 @@ function fileDialog() {
         runFileMutation(() => B.renameEntry(state.backend, d.file.path, v), 'renamed to ' + v,
           (entries) => entries.map((e) => (e.path || e.name) === (d.file.path || d.file.name)
             ? { ...e, name: v, path: e.path ? e.path.slice(0, e.path.length - d.file.name.length) + v : v }
-            : e));
+            : e))
+          .then(() => {
+            // On a 409 name-collision, prefill the retry input with a
+            // suggested alternate instead of leaving the identical blocked
+            // string - matching the pattern uploadFiles' 409-retry rows
+            // already establish for the same class of conflict.
+            if (state.files.dialog === d && d.error && /already exists/i.test(d.error)) {
+              d.suggestedValue = suggestAlternateName(v);
+              render();
+            }
+          });
       },
     });
   }
@@ -1348,11 +1367,18 @@ function fileDialog() {
     }
     return PromptDialog({
       title: 'Move ' + n + ' selected ' + (n === 1 ? 'entry' : 'entries'),
-      value: state.files.path || '', placeholder: 'destination folder path',
+      value: d._draft ?? (state.files.path || ''), placeholder: 'destination folder path',
       error: d.error || null, busy: !!d.busy,
       confirmLabel: d.busy ? 'moving…' : 'move ' + n, cancelLabel: 'cancel',
+      // A second/third allowed root has no discoverable path other than
+      // typing it from memory - one-click chips for every accessible root,
+      // matching the same practicality upgrade the cwd editor got.
+      roots: (Array.isArray(state.files.roots) && state.files.roots.length > 1)
+        ? state.files.roots.map((r) => ({ path: r, label: truncate(projectLabel(r) || r, 14, 24) }))
+        : undefined,
       onCancel: closeFileDialog,
-      onInput: (v) => { d.error = null; d._validateDest(v); },
+      onInput: (v) => { d._draft = v; d.error = null; d._validateDest(v); },
+      onPickRoot: (v) => { d._draft = v; d.error = null; d._validateDest(v); render(); },
       onConfirm: (v) => {
         if (!v) { d.error = 'enter a destination folder'; render(); return; }
         if (v === state.files.path) { d.error = 'already in that folder - enter a different destination'; render(); return; }
@@ -1362,7 +1388,7 @@ function fileDialog() {
   }
   if (d.kind === 'mkdir') {
     return PromptDialog({
-      title: 'New folder', value: '', placeholder: 'folder name',
+      title: 'New folder', value: d.suggestedValue ?? '', placeholder: 'folder name',
       error: d.error || null, busy: !!d.busy,
       confirmLabel: d.busy ? 'creating…' : 'create', cancelLabel: 'cancel',
       onCancel: closeFileDialog,
@@ -1371,7 +1397,13 @@ function fileDialog() {
         runFileMutation(() => B.makeDir(state.backend, state.files.path, v), 'created ' + v,
           (entries) => entries.some((e) => e.name === v)
             ? entries
-            : [...entries, { name: v, path: (state.files.path ? state.files.path.replace(/\/$/, '') + '/' : '') + v, type: 'dir' }]);
+            : [...entries, { name: v, path: (state.files.path ? state.files.path.replace(/\/$/, '') + '/' : '') + v, type: 'dir' }])
+          .then(() => {
+            if (state.files.dialog === d && d.error && /already exists/i.test(d.error)) {
+              d.suggestedValue = suggestAlternateName(v);
+              render();
+            }
+          });
       },
     });
   }
@@ -2104,6 +2136,7 @@ function applyToolResult(parts, block) {
 const ERR_COPY = [
   [/^ws closed$/, 'Lost connection to the server.'],
   [/connection lost during stream/, 'Lost connection while the agent was responding - use retry.'],
+  [/connection dropped mid-turn/, 'Connection dropped mid-turn - the response above may be incomplete (events weren\'t replayed). Retry to try again.'],
   [/no sessionId from server/, 'The server could not start the agent - check it is installed.'],
   [/^sessions: \d+/, 'History is still indexing - try again in a moment.'],
 ];
@@ -2166,11 +2199,17 @@ function chatMain() {
     banners.push(Alert({ key: 'cwderr', kind: 'warn', title: 'Invalid working directory', children: state.cwdError }));
   }
   if (state.chat.externalUpdate) {
+    const hasDraft = !!(state.chat.draft && state.chat.draft.trim());
     banners.push(Alert({ key: 'xupd', kind: 'info', title: 'This chat was updated in another tab',
       children: [
-        h('span', { key: 'xutxt' }, 'Reload it to see the latest turns, or dismiss to keep this tab\'s view. '),
+        h('span', { key: 'xutxt' }, 'Reload it to see the latest turns, or dismiss to keep this tab\'s view. '
+          // "reload it" re-pulls the transcript from localStorage, which
+          // silently discarded whatever was typed but not yet sent in THIS
+          // tab - warn before that data-loss, not just after.
+          + (hasDraft ? 'This tab has an unsent draft that reloading will discard. ' : '')),
         Btn({ key: 'xureload', disabled: state.chat.busy, onClick: () => {
           if (state.chat.busy) return;
+          if (hasDraft && !window.confirm('Reloading will discard your unsent draft in this tab. Continue?')) return;
           state.chat.externalUpdate = false;
           state.chat.messages = []; state.chat.resumeSid = null; state.chat.totalCost = 0;
           restoreChat(); render();
@@ -2189,9 +2228,15 @@ function chatMain() {
         Btn({ key: 'ptruncdismiss', onClick: () => { state.chat.persistTruncated = false; render(); }, children: 'dismiss' })] }));
   }
   if (state.agentsError) {
+    // A failed agents.list call (server/network issue) is a distinct case
+    // from "the list loaded but nothing is installed" (installHint below,
+    // which only fires on a genuinely non-empty, all-unavailable list) -
+    // recommending npx-install commands here would be actively wrong, since
+    // we don't actually know whether anything is installed. Just clarify the
+    // failure is transient/server-side, not a "nothing is installed" state.
     banners.push(Alert({ key: 'agerr', kind: 'error', title: 'Could not load agents from the server',
       children: [
-        h('span', { key: 'agtxt', title: state.agentsError }, 'The agent list failed to load. '),
+        h('span', { key: 'agtxt', title: state.agentsError }, 'The agent list failed to load - this is a connection issue, not necessarily a sign nothing is installed. '),
         Btn({ key: 'agretry', onClick: () => loadAgents(), children: 'retry' })] }));
   }
   if (state.chat.loadingTranscript) {
@@ -2200,9 +2245,19 @@ function chatMain() {
   }
   if (state.chat.confirmingEdit) {
     const isRetry = state.chat.confirmingEdit.kind === 'retry';
+    // Name the actual count/cost about to be discarded instead of the vague
+    // "the later turns" - a user retrying/editing message 2 of 40 needs to
+    // know whether that's 1 turn or 38 before confirming a destructive undo.
+    const truncIdx = state.chat.confirmingEdit.idx;
+    const discarded = state.chat.messages.slice(truncIdx + 1);
+    const discardedTurns = discarded.filter((m) => m.role === 'user').length;
+    const discardedCost = discarded.reduce((s, m) => s + (typeof m.costUsd === 'number' ? m.costUsd : 0), 0);
+    const discardSummary = discardedTurns
+      ? plural(discardedTurns, 'turn') + (discardedCost > 0 ? ' ($' + discardedCost.toFixed(4) + ')' : '')
+      : null;
     banners.push(Alert({ key: 'confedit', kind: 'warn', title: isRetry ? 'Retry this turn?' : 'Edit this message?',
       children: [
-        h('span', { key: 'cetext' }, (isRetry ? 'Retrying' : 'Editing') + ' will remove the later turns - continue? '),
+        h('span', { key: 'cetext' }, (isRetry ? 'Retrying' : 'Editing') + ' will remove ' + (discardSummary ? discardSummary + ' after this point' : 'the later turns') + ' - continue? '),
         Btn({ key: 'ceno', onClick: cancelEditAndResend, children: 'cancel' }),
         Btn({ key: 'ceyes', danger: true, onClick: confirmEditAndResend, children: isRetry ? 'retry' : 'continue' })] }));
   }
@@ -2466,9 +2521,25 @@ function cwdBrowsePick(path) {
 }
 
 function offlineBanner() {
+  // A session-expired 401 is a distinct, more actionable case than a plain
+  // network-down offline state (the fix is "reload", not "wait for
+  // reconnect") - surfaced first since it takes priority as an explanation.
+  if (state.sessionExpired) {
+    return Alert({ key: 'sessionexpired', kind: 'error', title: 'Session expired',
+      children: [
+        h('span', { key: 'setxt' }, 'Your access token is no longer valid (it may have rotated or expired). Reload the page to sign in again. '),
+        Btn({ key: 'sereload', onClick: () => window.location.reload(), children: 'reload now' }),
+      ] });
+  }
   if (state.health.status === 'ok' || state.health.status === 'unknown') return null;
   return Alert({ key: 'offline', kind: 'error', title: 'Backend unreachable',
-    children: 'agentgui can\'t reach the server (' + (state.health.error || state.health.status) + '). Chat and history actions will fail until it reconnects.' });
+    children: [
+      h('span', { key: 'otxt' }, 'agentgui can\'t reach the server (' + (state.health.error || state.health.status) + '). Chat and history actions will fail until it reconnects. '),
+      // Previously the only recovery was waiting for the WS backoff timer or
+      // a manual page reload - a direct retry button re-probes /health
+      // immediately instead of making the user guess how long to wait.
+      Btn({ key: 'oretry', disabled: !!state.healthChecking, onClick: () => recheckHealth(), children: state.healthChecking ? 'checking…' : 'retry now' }),
+    ] });
 }
 
 // (The working-directory bar now lives in the AgentChat kit; agentgui wires its
@@ -2972,10 +3043,10 @@ function eventMatchesFilter(e, f) {
 
 // Scroll to + flash the first error event, widening the render window (and
 // clearing the type filter) so the row is actually rendered.
-function jumpToFirstError() {
-  const idx = state.events.findIndex(e => e.isError);
+function jumpToEvent(idx) {
   if (idx < 0) return;
   state.eventFilter = 'all';
+  state._errorNavIdx = idx;
   const fromEnd = state.events.length - idx;
   if (fromEnd > state.eventsLimit) state.eventsLimit = Math.ceil(fromEnd / 300) * 300;
   render();
@@ -2987,6 +3058,21 @@ function jumpToFirstError() {
     if (row) { row.scrollIntoView({ block: 'center' }); row.classList.add('event-flash'); setTimeout(() => row.classList.remove('event-flash'), 2000); }
   });
 }
+function jumpToFirstError() {
+  const idx = state.events.findIndex(e => e.isError);
+  jumpToEvent(idx);
+}
+// Persistent next/prev navigation between error events - jumpToFirstError
+// alone only reaches the FIRST error once; a session with multiple errors had
+// no way to step through them without manually scanning the event list.
+function jumpToNextError(dir) {
+  const errIdxs = state.events.reduce((acc, e, i) => { if (e.isError) acc.push(i); return acc; }, []);
+  if (!errIdxs.length) return;
+  const cur = state._errorNavIdx;
+  let pos = errIdxs.indexOf(cur);
+  pos = pos < 0 ? (dir > 0 ? 0 : errIdxs.length - 1) : (pos + dir + errIdxs.length) % errIdxs.length;
+  jumpToEvent(errIdxs[pos]);
+}
 
 function historyMain() {
   if (!state.selectedSid) {
@@ -3012,8 +3098,14 @@ function historyMain() {
   }
 
   const sess = (Array.isArray(state.sessions) ? state.sessions : []).find(s => s.sid === state.selectedSid);
+  // sess.model is the raw ccsniff-sourced field (41st-run fix); ccsniff only
+  // reads Claude Code's own JSONL so the agent is always constant. The
+  // History detail header previously never surfaced either, reading
+  // identity-thin next to the same session's Running-panel/Live-dashboard
+  // rows which both show an agent+model badge.
+  const agentModelBit = sess?.model ? ((agentById('claude-code')?.name || 'Claude Code') + ' · ' + sess.model) : null;
   const lede = sess
-    ? (projectLabel(sess.project) || pathBasename(sess.cwd) || 'unknown location') + ' · ' + plural(sess.events || 0, 'event') + ' · ' + plural(sess.userTurns || 0, 'turn') + ' · ' + fmtRelTime(sess.last)
+    ? (projectLabel(sess.project) || pathBasename(sess.cwd) || 'unknown location') + (agentModelBit ? ' · ' + agentModelBit : '') + ' · ' + plural(sess.events || 0, 'event') + ' · ' + plural(sess.userTurns || 0, 'turn') + ' · ' + fmtRelTime(sess.last)
     : UNTITLED_CONVERSATION;
 
   const head = PageHeader({
@@ -3031,6 +3123,11 @@ function historyMain() {
       onClick: () => downloadBlob(JSON.stringify(state.events, null, 2), (projectLabel(sess?.project) || 'session') + '-' + state.selectedSid + '.json', 'application/json'),
       children: 'export' }),
     hasErrors ? Btn({ key: 'jumperr', onClick: jumpToFirstError, children: 'jump to first error' }) : null,
+    // Persistent next/prev stepping between errors - jump-to-first alone only
+    // ever reaches the FIRST one; a session with several errors had no way to
+    // step through the rest without manually scrolling/scanning.
+    hasErrors ? Btn({ key: 'errprev', title: 'previous error', 'aria-label': 'previous error', onClick: () => jumpToNextError(-1), children: 'prev error' }) : null,
+    hasErrors ? Btn({ key: 'errnext', title: 'next error', 'aria-label': 'next error', onClick: () => jumpToNextError(1), children: 'next error' }) : null,
   ].filter(Boolean));
 
   if (state.events.length === 0) {
@@ -4047,6 +4144,13 @@ function registerWsStatusOnce() {
       if (state.health.ws) { delete state.health.ws; render(); }
     }
   });
+  // backend.js already detects a mid-session 401 (a token that stopped being
+  // valid - e.g. PASSWORD rotated, cookie expired) and exports this hook, but
+  // nothing ever subscribed to it: every subsequent fetch just failed
+  // silently with no on-screen indication of WHY. Surface it as a persistent
+  // banner instructing a reload (the only real recovery - the token lives in
+  // window.__WS_TOKEN, injected server-side at page load).
+  B.onSessionExpired?.(() => { state.sessionExpired = true; render(); });
 }
 
 hydratePrefs();

package/site/app/vendor/anentrypoint-design/247420.css

@@ -2686,6 +2686,17 @@
 @media (hover: none), (pointer: coarse) {
   .ds-247420 .ds-modal-input { font-size: 16px; min-height: 44px; }
 }
+/* PromptDialog's optional roots-chip row (destination-path prompts with more
+   than one allowed root) - same chip idiom the cwd editor's roots row uses. */
+.ds-247420 .ds-prompt-roots { display: flex; align-items: center; gap: 6px; flex-wrap: wrap; margin-top: 8px; }
+.ds-247420 .ds-prompt-root-chip {
+  background: var(--bg-2); border: var(--bw-hair) solid var(--rule); color: var(--fg-2);
+  border-radius: var(--r-pill); padding: 3px 10px; font: inherit; font-size: var(--fs-tiny);
+  cursor: pointer;
+}
+.ds-247420 .ds-prompt-root-chip:hover { border-color: var(--accent); color: var(--fg); }
+.ds-247420 .ds-prompt-root-chip:focus-visible { outline: var(--focus-w) solid var(--focus-color); outline-offset: var(--focus-offset); }
+@media (pointer: coarse) { .ds-247420 .ds-prompt-root-chip { min-height: 44px; } }
 /* In-body modal error (role=alert): mutation failures (409/403) surface INSIDE
    the dialog, not behind the fixed overlay. */
 .ds-247420 .ds-modal-error {