@oddessentials/agent-guild 1.18.0 → 1.20.0

package/src/manager/session-manager.mjs

@@ -64,9 +64,16 @@ export class SessionManager extends EventEmitter {
    * @param {string|null} [opts.shimDir]  folder with the agent-guild-report launchers, put first on PATH
    * @param {import('./self-update.mjs').SelfUpdate|null} [opts.selfUpdate]  the manager's own upgrade
    * @param {import('./github.mjs').GitHub|null} [opts.github]
+   * @param {ReturnType<import('./card-names.mjs').createCardNames>|null} [opts.cardNames]  names given to cards, by tool session id
    */
-  constructor({ registry, baseEnv, getApiUrl, sessionDefaults = {}, shimDir = null, selfUpdate = null, github = null, sessionHooks = null, store = null }) {
+  constructor({ registry, baseEnv, getApiUrl, sessionDefaults = {}, shimDir = null, selfUpdate = null, github = null, sessionHooks = null, store = null, cardNames = null }) {
     super();
+    this.cardNames = cardNames;
+    /** Each card's latest choice of name for the tool session it holds, in the order the choices were made. */
+    this.nameChoices = new WeakMap();
+    this.nameOrder = 0;
+    /** Tool session → the order of the newest choice written to the store for it. */
+    this.namesWritten = new Map();
     this.registry = registry;
     this.baseEnv = baseEnv;
     this.getApiUrl = getApiUrl;
@@ -144,11 +151,12 @@ export class SessionManager extends EventEmitter {
     if (!current) throw httpError(404, `unknown provider "${provider.id}"`, 'unknown_provider');
     const signIn = this.registry.account(current, account);
     this.prepareAccount(provider, signIn, { hooksSupplied: hooks.args.length > 0 });
-    const sessionName = cleanName(name)
-      || (current.accounts.length > 1 ? `${provider.tool} · ${signIn.label}` : null)
-      || (runShell && this.registry.shellsFor(provider).shells.length > 1 ? `${provider.tool} · ${runShell.label}` : null);
+    const defaultName = (current.accounts.length > 1 ? `${provider.tool} · ${signIn.label}` : null)
+      || (runShell && this.registry.shellsFor(provider).shells.length > 1 ? `${provider.tool} · ${runShell.label}` : null)
+      || provider.tool;
+    const given = cleanName(name) || (resumeId ? this.cardNames?.get(provider.id, signIn.id, resumeId) : null) || null;
     const extraEnv = hooks.env ? { ...runShell?.env, ...hooks.env } : runShell?.env;
-    const options = { provider, cwd: workDir, cols, rows, name: sessionName, resume: resumeId, account: signIn, reporting: hooks.reporting, extraEnv };
+    const options = { provider, cwd: workDir, cols, rows, name: given ?? defaultName, defaultName, named: given !== null, resume: resumeId, account: signIn, reporting: hooks.reporting, extraEnv };
     if (runShell?.multiplexer) return this._withMultiplexerStart(provider.id, runShell.id, () => this._startMultiplexer(options, runShell, args || []));
     const spawnSpec = this.registry.spawnSpec(provider, args || [], resumeId, hooks.args, runShell);
     const session = this._spawn({ ...options, spawnSpec });
@@ -209,7 +217,7 @@ export class SessionManager extends EventEmitter {
     }
     this._track(session, { spawnSpec, mux, shell, card: {
       id, reportToken, provider: provider.id, account: account.id, shell: shell.id,
-      muxName, name: session.name, cwd: session.cwd, createdAt: session.createdAt,
+      muxName, name: session.name, defaultName: session.defaultName, named: session.named, cwd: session.cwd, createdAt: session.createdAt,
       // Herdr's request arguments belong to its client and must survive rebinding.
       // Tmux's request arguments belong to the already-running inner session.
       ...(shell.id === 'herdr' ? { args: [...args] } : {}),
@@ -247,13 +255,44 @@ export class SessionManager extends EventEmitter {
     this._saveCards();
     session.on('exit', () => this._checkMultiplexer(session, tracked));
     session.on('changed', () => {
-      if (card.name === session.name && (card.context?.compactions ?? 0) === session.context.compactions && (card.context?.resets ?? 0) === session.context.resets) return;
+      if (card.name === session.name && card.named === session.named && (card.context?.compactions ?? 0) === session.context.compactions && (card.context?.resets ?? 0) === session.context.resets) return;
       card.name = session.name;
+      card.named = session.named;
       card.context = { ...session.context };
       this._saveCards();
     });
   }
 
+  /**
+   * The store holds what a card last chose for the tool session it holds: its name, or none for its default. A
+   * card's choice is written once, when it is made: a rename, a new tool session, or the card's start. One whose
+   * write failed is written again on a later change, unless a newer choice for that tool session was written since,
+   * so a card shown again never puts back a name another card replaced. A card nobody named writes nothing.
+   */
+  _keepCardName(session) {
+    // The page's identity for the tool session a card holds; a resumed card knows it before the tool reports.
+    const toolSessionId = session.toolSessionId ?? session.resume;
+    if (!this.cardNames || session.task !== null || !toolSessionId || !session.nameChosen) return;
+    const account = session.account?.id ?? 'default';
+    const key = `${session.provider.id}\0${account}\0${toolSessionId}`;
+    const name = session.named ? session.name : null;
+    let choice = this.nameChoices.get(session);
+    if (choice?.key !== key || choice.name !== name) {
+      choice = { key, name, order: ++this.nameOrder, written: false };
+      this.nameChoices.set(session, choice);
+    }
+    if (choice.written) return;
+    if ((this.namesWritten.get(key) ?? 0) > choice.order) {
+      choice.written = true;
+      return;
+    }
+    // set() and delete() say whether the store now holds the choice; get() cannot tell an unreadable store from one without it.
+    const saved = name ? this.cardNames.set(session.provider.id, account, toolSessionId, name) : this.cardNames.delete(session.provider.id, account, toolSessionId);
+    if (!saved) return;
+    choice.written = true;
+    this.namesWritten.set(key, choice.order);
+  }
+
   _saveCards() {
     // A stopping manager leaves the file as it is, for the next one to bring the cards back.
     if (!this.store || this.restoring || this.closing) return;
@@ -309,12 +348,13 @@ export class SessionManager extends EventEmitter {
     const spawnSpec = this.registry.spawnSpec(provider, shell.id === 'herdr' ? card.args ?? [] : [], null, [], mux.named);
     const session = this._spawn({
       provider, spawnSpec: null, cwd: typeof card.cwd === 'string' ? card.cwd : os.homedir(), name: card.name, account,
+      defaultName: typeof card.defaultName === 'string' ? card.defaultName : undefined, named: card.named === true,
       id: card.id, reportToken: card.reportToken, dropEnv: mux.dropEnv, extraEnv: shell.env,
       createdAt: typeof card.createdAt === 'string' && !Number.isNaN(Date.parse(card.createdAt)) ? card.createdAt : undefined,
       context: card.context,
       multiplexer: { label: shell.label, attach: shell.multiplexer.attach.replaceAll('{name}', card.muxName), reattachable: true },
     });
-    this._track(session, { spawnSpec, mux, shell, card: { ...card, name: session.name, cwd: session.cwd, createdAt: session.createdAt } });
+    this._track(session, { spawnSpec, mux, shell, card: { ...card, name: session.name, defaultName: session.defaultName, named: session.named, cwd: session.cwd, createdAt: session.createdAt } });
   }
 
   /** Attach a stopped tmux or herdr session's card to its multiplexer session again, keeping its id and report token. */
@@ -711,7 +751,7 @@ export class SessionManager extends EventEmitter {
   }
 
   _spawn({
-    provider, description = this.registry.describe(provider), spawnSpec, cwd, cols, rows, name, resume = null, task = null, installKind = null, installPath = null, account = null,
+    provider, description = this.registry.describe(provider), spawnSpec, cwd, cols, rows, name, defaultName, named = false, resume = null, task = null, installKind = null, installPath = null, account = null,
     extraEnv = null, dropEnv = null, clone = null, reporting = null, multiplexer = null,
     id = newId(), reportToken = crypto.randomBytes(16).toString('hex'), createdAt, context = null, multiplexerInstall = null,
   }) {
@@ -731,6 +771,8 @@ export class SessionManager extends EventEmitter {
         cols: clampDimension(cols, 120, 2, 1000),
         rows: clampDimension(rows, 32, 1, 500),
         name,
+        defaultName,
+        named,
         reportToken,
         resume,
         task,
@@ -745,7 +787,9 @@ export class SessionManager extends EventEmitter {
       throw httpError(500, `could not start ${provider.tool}: ${err.message}`, 'spawn_failed');
     }
 
+    this._keepCardName(session);
     session.on('changed', () => {
+      this._keepCardName(session);
       if (this.sessions.has(id)) this.emit('event', { type: 'session.updated', session: session.toJSON() });
     });
     session.on('warning', (msg) => console.warn(`[session ${id}] ${msg}`));

package/src/manager/session.mjs

@@ -6,8 +6,7 @@ import { EventEmitter } from 'node:events';
 import crypto from 'node:crypto';
 import headless from '@xterm/headless';
 import serializeAddon from '@xterm/addon-serialize';
-import { killWindowsTree } from './command-resolver.mjs';
-import { loadPty } from './pty.mjs';
+import { loadPty, closeConsole, releasePty } from './pty.mjs';
 import { sessionLabel } from '../../web/activity-favicon.js';
 
 const { Terminal } = headless;
@@ -68,6 +67,8 @@ export class Session extends EventEmitter {
    * @param {number} opts.cols
    * @param {number} opts.rows
    * @param {string} [opts.name]
+   * @param {string} [opts.defaultName]  the name a card has when nobody named it
+   * @param {boolean} [opts.named]  the name was given, so it is kept for the tool session
    * @param {string|null} [opts.resume]  id of the tool's own session being resumed
    * @param {string|null} [opts.task]    "install" for a package install, "upgrade" for the manager's own, "clone" for a GitHub clone, else null
    * @param {{id: string, label: string}|null} [opts.account]  the tool sign-in the session runs under
@@ -84,7 +85,11 @@ export class Session extends EventEmitter {
     super();
     this.id = opts.id;
     this.provider = opts.provider;
-    this.name = cleanName(opts.name) || opts.provider.tool;
+    this.defaultName = cleanName(opts.defaultName) || opts.provider.tool;
+    this.name = cleanName(opts.name) || this.defaultName;
+    this.named = opts.named === true && this.name !== this.defaultName;
+    /** Someone gave or changed this card's name, its default included. */
+    this.nameChosen = opts.named === true;
     this.resume = opts.resume ?? null;
     this.task = opts.task ?? null;
     this.account = opts.account ?? null;
@@ -98,6 +103,7 @@ export class Session extends EventEmitter {
     this.activityIdleMs = opts.activityIdleMs ?? 2500;
     this.doneAgentLingerMs = opts.doneAgentLingerMs ?? 15000;
     this.killGraceMs = opts.killGraceMs ?? 4000;
+    this._killed = false;
     this.reportingTimeoutMs = opts.reportingTimeoutMs ?? 30000;
     this.reporting = REPORTING_STATES.has(opts.reporting?.state) ? { state: opts.reporting.state, reason: opts.reporting.reason ?? null } : null;
     this._reportingTimer = null;
@@ -198,6 +204,7 @@ export class Session extends EventEmitter {
       useConpty: true,
     });
     this.pty = proc;
+    this._killed = false;
     proc.onData((data) => { if (this.pty === proc) this._onData(data); });
     proc.onExit(({ exitCode, signal }) => { if (this.pty === proc) this._onExit(exitCode, signal); });
   }
@@ -244,6 +251,7 @@ export class Session extends EventEmitter {
 
   _onExit(exitCode, signal) {
     clearTimeout(this._killTimer);
+    if (process.platform === 'win32') releasePty(this.pty);
     this._resolveExited();
     if (this.disposed) return;
     this.exitedAt ??= new Date().toISOString();
@@ -412,16 +420,18 @@ export class Session extends EventEmitter {
 
   /**
    * End the process. On macOS and Linux: hang-up first, force after a grace
-   * period. On Windows: end the whole process tree at once, except for a
-   * multiplexer's client, whose server is its child there and must outlive
-   * it. node-pty's own Windows kill first asks a helper process for the
-   * console's process list, and when that helper fails it waits a fixed
-   * five seconds.
+   * period. On Windows: close the session's console, which ends every process
+   * attached to it; a multiplexer's server and anything started detached
+   * outlive it.
    */
   kill({ graceMs = this.killGraceMs } = {}) {
     if (this.status !== 'running') return;
     if (process.platform === 'win32') {
-      this._killWindowsTree();
+      // At most once per pty: node-pty 1.2.0-beta.15 closes the pseudoconsole again on a second close before the
+      // exit watcher runs, which corrupts this process's heap (microsoft/node-pty#983).
+      if (this._killed) return;
+      this._killed = true;
+      closeConsole(this.pty);
       return;
     }
     const force = () => { try { this.pty.kill('SIGKILL'); } catch { /* gone */ } };
@@ -431,15 +441,6 @@ export class Session extends EventEmitter {
     this._killTimer.unref?.();
   }
 
-  _killWindowsTree() {
-    const fallback = () => {
-      if (this.status === 'running') { try { this.pty.kill(); } catch { /* gone */ } }
-    };
-    const pid = this.pty.pid;
-    if (!pid) return fallback();
-    killWindowsTree(pid, (err) => { if (err) fallback(); }, { tree: !this.multiplexer });
-  }
-
   /**
    * Detach everything. A still-running process gets the normal hang-up and
    * is force-killed after the grace period if it ignores it.
@@ -461,6 +462,8 @@ export class Session extends EventEmitter {
     const clean = cleanName(name);
     if (!clean) return;
     this.name = clean;
+    this.named = clean !== this.defaultName;
+    this.nameChosen = true;
     this._changed();
   }
 

package/src/mcp/server.mjs

@@ -84,6 +84,13 @@ const TOOLS = [
     inputSchema: { type: 'object', properties: { session_id: sessionId }, required: ['session_id'], additionalProperties: false },
     annotations: { readOnlyHint: false, destructiveHint: true, idempotentHint: true, openWorldHint: false },
   },
+  {
+    name: 'fireworks',
+    title: 'Fireworks over the Yard',
+    description: 'Launch fireworks over the Yard on every screen connected to Agent Guild. Use it to celebrate something the person cares about, such as tests, a build or a release that passed. Says how many pages saw them.',
+    inputSchema: { type: 'object', properties: {}, additionalProperties: false },
+    annotations: { readOnlyHint: false, destructiveHint: false, idempotentHint: false, openWorldHint: false },
+  },
 ];
 
 const INPUT_TOOL = {
@@ -309,6 +316,10 @@ export function createToolHandlers(client, { allowInput = false } = {}) {
       const { session } = await client.api(`/sessions/${checkId(id)}/stop`, { method: 'POST', session: id });
       return text({ stopped: true, session: summarize(session) });
     },
+    async fireworks() {
+      const { pages } = await client.api('/yard/fireworks', { method: 'POST' });
+      return text({ launched: true, pages, ...(pages ? {} : { message: 'No page is open to see them. The person can open Agent Guild and choose the Yard.' }) });
+    },
   };
   if (allowInput) {
     handlers.send_input = async ({ session_id: id, text: typed, submit = true }) => {

package/web/app.js

@@ -2293,8 +2293,8 @@ function buildProvider(provider) {
     existing.addEventListener('click', (event) => showHistory(provider, false, event.currentTarget));
     node.querySelector('.launch-flags').addEventListener('click', (event) => showFlags(provider, event.currentTarget));
     const memory = node.querySelector('.memory-link');
-    memory.hidden = !provider.available || (!provider.memorySource && !provider.historyDetails);
-    const historyLink = !provider.memorySource && provider.historyDetails;
+    const historyLink = !provider.memorySource && provider.historySource === 'antigravity';
+    memory.hidden = !provider.available || (!provider.memorySource && !historyLink);
     memory.textContent = historyLink ? 'History' : 'Memory';
     memory.title = historyLink ? `Read ${provider.tool}'s saved conversations` : `See what ${provider.tool} remembers about the working folder`;
     memory.addEventListener('click', (event) => historyLink ? showHistory(provider, true, event.currentTarget) : showMemory(provider, event.currentTarget));
@@ -3970,7 +3970,7 @@ async function startSession(provider, card, { resume, cwd, account = selectedAcc
       ({ session } = await api('POST', '/sessions', body));
     } catch (err) {
       if (err.code !== 'bad_cwd' || !cwd) throw err;
-      if (resume && provider.historyDetails) {
+      if (resume && provider.historySource === 'antigravity') {
         const chosen = (await ask(`${cwd} no longer exists. Choose a working folder to resume this conversation:`, { ok: 'Resume', value: working }))?.trim();
         if (!chosen) return;
         ({ session } = await api('POST', '/sessions', { ...body, cwd: chosen }));
@@ -4011,20 +4011,22 @@ function renderHistoryPreview(view) {
   $('history-preview-title').textContent = view.target?.title || 'Select a conversation';
   $('history-preview-refresh').disabled = !view.target || view.loading;
   $('history-preview-more').hidden = !view.nextCursor;
-  paintPending($('history-preview-more'), view.loading, 'Read more', 'Reading…');
+  paintPending($('history-preview-more'), view.loading, 'Read earlier', 'Reading…');
   const notes = [];
   if (view.error) notes.push(view.error);
   else if (view.loading) notes.push('Reading saved messages…');
-  else if (view.target && !view.messages.length) notes.push(view.nextCursor ? 'No visible messages in this part. Continue reading below.' : 'No complete visible messages in this part.');
+  else if (view.target && !view.messages.length) notes.push(view.nextCursor ? 'No visible messages in this part. Read earlier messages above.' : 'No visible messages in this conversation.');
   else if (!view.target) notes.push('Read a saved exchange, then resume when you want to continue it.');
-  if (view.omitted) notes.push('Some unreadable or oversized records were omitted.');
+  if (view.omitted) notes.push('Some unreadable records were omitted.');
+  if (view.truncated) notes.push('Earlier messages cannot be shown: a record before these is too large or unreadable.');
   if (view.incomplete) notes.push('The last record is incomplete. Refresh after the conversation finishes writing.');
   $('history-preview-note').textContent = notes.join(' ');
   const messages = $('history-messages');
-  const scroll = messages.scrollTop;
+  const fromBottom = messages.scrollHeight - messages.scrollTop;
+  const first = messages.childElementCount === 0;
   messages.replaceChildren(...view.messages.map((message) => el('article', 'history-message',
     el('h4', null, message.role === 'user' ? 'You' : 'Assistant'), el('p', null, message.text))));
-  messages.scrollTop = view.messages.length ? scroll : 0;
+  messages.scrollTop = first ? messages.scrollHeight : messages.scrollHeight - fromBottom;
   for (const node of $('history-list').querySelectorAll('.history-preview-open')) node.setAttribute('aria-current', String(node.dataset.id === view.target?.id));
 }
 
@@ -4090,10 +4092,8 @@ async function loadHistory() {
   renderHistory();
   try {
     const query = new URLSearchParams({ account: accountId, limit: HISTORY_LIMIT });
-    if (provider.historyDetails) {
-      if ($('history-here').checked && $('cwd').value.trim()) query.set('cwd', $('cwd').value.trim());
-      if ($('history-filter').value.trim()) query.set('q', $('history-filter').value.trim());
-    }
+    if ($('history-here').checked && $('cwd').value.trim()) query.set('cwd', $('cwd').value.trim());
+    if ($('history-filter').value.trim()) query.set('q', $('history-filter').value.trim());
     const { history } = await api('GET', `/providers/${providerId}/history?${query}`);
     if (request !== historyView.request) return;
     historyView.snapshot = history;
@@ -4107,27 +4107,18 @@ async function loadHistory() {
 }
 
 function historyFiltersChanged() {
-  if (historyProvider()?.historyDetails) {
+  if (historyProvider()?.historySource) {
     historyView.snapshot = null;
     historyPreview.clear();
     loadHistory();
   } else renderHistory();
 }
 
-function historyText(entry) {
-  return `${entry.title ?? ''}\n${entry.cwd ?? ''}\n${entry.id}`.toLowerCase();
-}
-
 function folderName(dir) {
   const parts = dir.replace(/[\\/]+$/, '').split(/[\\/]/);
   return parts.at(-1) || dir;
 }
 
-function sameFolder(a, b) {
-  const clean = (dir) => (dir || '').replace(/[\\/]+$/, '').replace(/\\/g, '/').toLowerCase();
-  return clean(a) === clean(b);
-}
-
 function historyEntry(id) {
   return historyView.snapshot?.sessions.find((entry) => entry.id === id) ?? null;
 }
@@ -4138,7 +4129,7 @@ async function resumeFromHistory(provider, id, cwd) {
     closeHistory({ focusOpener: false });
     openPanel(running.id);
   } else {
-    if (provider.historyDetails && !cwd) {
+    if (provider.historySource === 'antigravity' && !cwd) {
       const workspaces = historyEntry(id)?.workspaces ?? [];
       const reason = workspaces.length > 1 ? `This conversation has several workspaces:\n${workspaces.join('\n')}\n\n` : 'The saved working folder is unknown.\n\n';
       cwd = (await ask(`${reason}Choose a working folder to resume in:`, { ok: 'Resume', value: $('cwd').value.trim() }))?.trim();
@@ -4164,27 +4155,28 @@ function buildHistoryRow(id) {
 
 function updateHistoryRow(node, provider, entry) {
   const running = runningOn(provider.id, historyView.accountId, entry.id);
-  node.classList.toggle('untitled', !entry.title);
+  const label = entry.name ?? entry.title;
+  node.classList.toggle('untitled', !label);
   node.classList.toggle('running', Boolean(running));
   const title = node.querySelector('.history-title');
-  title.title = entry.title ?? '';
+  title.title = label ?? '';
   if (provider.historyDetails) {
     let preview = title.querySelector('button');
     if (!preview) {
       preview = button('', () => {
         const current = historyEntry(entry.id);
-        if (current) historyPreview.select({ providerId: provider.id, accountId: historyView.accountId, id: entry.id, title: current.title ?? 'Untitled session' });
+        if (current) historyPreview.select({ providerId: provider.id, accountId: historyView.accountId, id: entry.id, title: current.name ?? current.title ?? 'Untitled session' });
       }, 'history-preview-open');
       preview.dataset.id = entry.id;
       title.replaceChildren(preview);
     }
-    preview.textContent = entry.title ?? 'Untitled session';
+    preview.textContent = label ?? 'Untitled session';
     preview.setAttribute('aria-current', String(historyPreview.state.target?.id === entry.id));
-  } else title.textContent = entry.title ?? 'Untitled session';
+  } else title.textContent = label ?? 'Untitled session';
   const meta = node.querySelector('.history-meta');
   const when = entry.updatedAt ? `updated ${relativeTime(entry.updatedAt)}` : '';
-  const folder = entry.cwd ? folderName(entry.cwd) : provider.historyDetails ? (entry.workspaces?.length > 1 ? `${entry.workspaces.length} workspaces` : 'Folder unknown') : null;
-  meta.textContent = [folder, when, running && `open in Agent Guild as ${running.name}`].filter(Boolean).join(' · ');
+  const folder = entry.cwd ? folderName(entry.cwd) : provider.historySource === 'antigravity' ? (entry.workspaces?.length > 1 ? `${entry.workspaces.length} workspaces` : 'Folder unknown') : null;
+  meta.textContent = [entry.name && entry.title !== entry.name ? entry.title : null, folder, when, running && `open in Agent Guild as ${running.name}`].filter(Boolean).join(' · ');
   meta.title = [...(entry.workspaces ?? [entry.cwd]), entry.startedAt && `started ${new Date(entry.startedAt).toLocaleString()}`].filter(Boolean).join('\n');
   const action = node.querySelector('.history-resume');
   const key = sessionActionKey(provider.id, historyView.accountId, entry.id);
@@ -4193,7 +4185,7 @@ function updateHistoryRow(node, provider, entry) {
   action.title = running
     ? `This session is running in Agent Guild as "${running.name}". Open it instead of resuming it twice.`
     : `Resume this ${provider.tool} session${entry.cwd ? ` in ${entry.cwd}` : ''}`;
-  action.setAttribute('aria-label', `${action.textContent} ${entry.title ?? entry.id}`);
+  action.setAttribute('aria-label', `${action.textContent} ${label ?? entry.id}`);
 }
 
 /** Rows are kept and updated in place, so a refresh never drops keyboard focus from a row's buttons. */
@@ -4222,9 +4214,8 @@ function renderHistory() {
   const here = $('history-here');
   here.disabled = !working;
   here.parentElement.title = working ? `Only sessions started in ${working}` : 'Set a working folder above to filter by it';
-  const serverFiltered = provider.historyDetails && Boolean(filter || (here.checked && !here.disabled));
-  const all = snapshot?.sessions ?? [];
-  const shown = provider.historyDetails ? all : all.filter((entry) => (!filter || historyText(entry).includes(filter)) && (!here.checked || here.disabled || sameFolder(entry.cwd, working)));
+  const serverFiltered = Boolean(filter || (here.checked && !here.disabled));
+  const shown = snapshot?.sessions ?? [];
   const parts = [];
   if ((provider.accounts?.length ?? 0) > 1 && account) parts.push(`${account.label} account`);
   if (snapshot && !snapshot.error) {
@@ -4239,14 +4230,11 @@ function renderHistory() {
   if (!provider.historySource) note = `Agent Guild cannot list ${provider.tool}'s sessions. Enter the id of one to resume it.`;
   else if (historyView.loading && !snapshot) note = `Reading ${provider.tool}'s sessions…`;
   else if (snapshot?.error) note = `Sessions could not be read: ${snapshot.error}`;
-  else if (snapshot && all.length === 0 && !serverFiltered) note = `No ${provider.tool} sessions were found${account && account.id !== 'default' ? ` for the ${account.label} account` : ''}.`;
+  else if (snapshot && shown.length === 0 && !serverFiltered) note = `No ${provider.tool} sessions were found${account && account.id !== 'default' ? ` for the ${account.label} account` : ''}.`;
   else if (snapshot && shown.length === 0) note = 'No session matches the filter.';
   if (snapshot?.note) note = [note, snapshot.note].filter(Boolean).join(' ');
-  if (snapshot?.total > all.length) {
-    const limitNote = provider.historyDetails
-      ? `Showing ${all.length} of ${snapshot.total} sessions. Narrow the filter to find older conversations.`
-      : `Loaded the newest ${all.length} of ${snapshot.total} sessions. Filtering searches these loaded sessions.`;
-    note = [note, limitNote].filter(Boolean).join(' ');
+  if (snapshot?.total > shown.length) {
+    note = [note, `Showing ${shown.length} of ${snapshot.total} sessions. Narrow the filter to find older conversations.`].filter(Boolean).join(' ');
   }
   $('history-note').textContent = note;
   $('history-note').hidden = !note;
@@ -5305,6 +5293,7 @@ async function loadView(view) {
   const request = { key, stamp, value: before?.stamp === stamp ? before.value : null, error: null, loading: true };
   githubPick.data[view] = request;
   renderGitHubViews();
+  const watching = view === 'actions' ? new Set(runWatch.runs.values()) : null;
   let value = null;
   let error = null;
   try {
@@ -5319,7 +5308,7 @@ async function loadView(view) {
   renderGitHubViews();
   if (view === 'actions') scheduleRuns();
   if (view === 'actions' && value) {
-    runWatchSeen(repo, value.runs);
+    runWatchSeen(repo, value.runs, watching);
     deliverRunWatch();
   }
 }
@@ -5556,11 +5545,12 @@ function endWatch(key, outcome) {
   runWatch.outcomes.push({ repo: watch.repo.fullName, name: watch.name, title: watch.title, url: watch.url, ...outcome });
 }
 
-function runWatchSeen(repo, runs) {
+/** Only the watches there were when the request for `runs` began: a later one may be for a run newer than those listed. */
+function runWatchSeen(repo, runs, watching = new Set(runWatch.runs.values())) {
   const key = repoKey(repo);
   let changed = false;
   for (const [id, watch] of runWatch.runs) {
-    if (repoKey(watch.repo) !== key) continue;
+    if (repoKey(watch.repo) !== key || !watching.has(watch)) continue;
     const run = runs.find((r) => r.id === watch.id);
     if (run && run.status !== 'completed') continue;
     changed = true;
@@ -5635,12 +5625,13 @@ function pollRunWatch() {
   runWatch.polling = (async () => {
     if (!state.connected || !runWatch.runs.size) return;
     if (!state.github) await api('GET', '/github').then(({ github }) => setGitHub(github), () => {});
+    const watching = new Set(runWatch.runs.values());
     const repos = new Map();
-    for (const watch of runWatch.runs.values()) repos.set(repoKey(watch.repo), watch.repo);
+    for (const watch of watching) repos.set(repoKey(watch.repo), watch.repo);
     const accounts = usableGitHubAccounts();
     await Promise.all([...repos.values()].filter((repo) => accounts.some((a) => a.id === repo.accountId)).map(async (repo) => {
       try {
-        runWatchSeen(repo, (await api('GET', `${repoPath(repo)}/actions`)).runs);
+        runWatchSeen(repo, (await api('GET', `${repoPath(repo)}/actions`)).runs, watching);
       } catch (err) {
         if (err instanceof AuthError) showAuth(err.message);
       }
@@ -7750,6 +7741,8 @@ function connectEvents() {
     } else if (msg.type === 'yard-art.updated') {
       state.yardArt = msg.yardArt;
       notifyViews();
+    } else if (msg.type === 'yard.fireworks') {
+      yardUi?.celebrate();
     } else if (msg.type === 'news.updated') {
       loadNews();
     } else if (msg.type === 'github.updated') {

package/web/history-preview.js

@@ -2,10 +2,10 @@
 // can be exercised with explicitly released promises, without timers or a browser.
 export function createHistoryPreview({ read, changed, authError = () => {} }) {
   let request = 0;
-  const state = { target: null, messages: [], nextCursor: null, loading: false, error: null, incomplete: false, omitted: false };
+  const state = { target: null, messages: [], nextCursor: null, loading: false, error: null, incomplete: false, omitted: false, truncated: false };
   function clear(target = null) {
     request++;
-    Object.assign(state, { target, messages: [], nextCursor: null, loading: false, error: null, incomplete: false, omitted: false });
+    Object.assign(state, { target, messages: [], nextCursor: null, loading: false, error: null, incomplete: false, omitted: false, truncated: false });
     changed(state);
   }
   async function load(more = false) {
@@ -14,19 +14,19 @@ export function createHistoryPreview({ read, changed, authError = () => {} }) {
     const target = state.target;
     const cursor = more ? state.nextCursor : null;
     Object.assign(state, { loading: true, error: null });
-    if (!more) Object.assign(state, { messages: [], nextCursor: null, incomplete: false, omitted: false });
+    if (!more) Object.assign(state, { messages: [], nextCursor: null, incomplete: false, omitted: false, truncated: false });
     changed(state);
     try {
       const detail = await read(target, cursor);
       if (request !== ticket) return;
-      Object.assign(state, { messages: more ? [...state.messages, ...detail.messages] : detail.messages,
-        nextCursor: detail.nextCursor, incomplete: detail.incomplete, omitted: state.omitted || detail.omitted });
+      Object.assign(state, { messages: more ? [...detail.messages, ...state.messages] : detail.messages,
+        nextCursor: detail.nextCursor, incomplete: more ? state.incomplete : detail.incomplete, omitted: state.omitted || detail.omitted, truncated: detail.truncated === true });
     } catch (err) {
       if (request !== ticket) return;
       authError(err);
       state.error = err.message || 'The conversation could not be read.';
-      // A stale cursor cannot be retried; other errors can retry the same page.
-      if (err.code === 'history_changed') state.nextCursor = null;
+      // A changed conversation drops what was read; other errors can retry the same page.
+      if (err.code === 'history_changed') Object.assign(state, { messages: [], nextCursor: null, incomplete: false, omitted: false, truncated: false });
     } finally {
       if (request === ticket) { state.loading = false; changed(state); }
     }

package/web/index.html

@@ -650,8 +650,8 @@
           <button id="history-preview-refresh" class="btn" type="button">Refresh</button>
         </div>
         <p id="history-preview-note" class="models-note" role="status"></p>
+        <button id="history-preview-more" class="btn" type="button" hidden>Read earlier</button>
         <div id="history-messages" class="history-messages" tabindex="0" aria-label="Saved messages"></div>
-        <button id="history-preview-more" class="btn" type="button" hidden>Read more</button>
       </section>
     </div>
     <form id="history-form" class="history-form">