mixdog 0.9.0 → 0.9.2

package/src/runtime/memory/tool-defs.mjs

@@ -52,7 +52,9 @@ export const TOOL_DEFS = [
         includeArchived: { type: 'boolean', description: 'Include archived entries.' },
         sessionId: { type: 'string', description: 'Scoped session id.' },
         session_id: { type: 'string', description: 'Alias for sessionId.' },
-        includeMembers: { type: 'boolean', description: 'Include chunk members.' },
+        includeMembers: { type: 'boolean', description: 'Include chunk members in output; does not widen the search pool.' },
+        includeRaw: { type: 'boolean', description: 'Include unchunked raw/episode rows.' },
+        sessionOnly: { type: 'boolean', description: 'Search this session only.' },
         projectScope: { type: 'string', description: 'Project pool selector: inferred from cwd, common, all, or slug.' },
         cwd: { type: 'string', description: 'Infer projectScope.' },
       },
@@ -74,8 +76,9 @@ export const TOOL_DEFS = [
         category: { anyOf: [{ type: 'string', enum: ['rule','constraint','decision','fact','goal','preference','task','issue'] }, { type: 'array', items: { type: 'string', enum: ['rule','constraint','decision','fact','goal','preference','task','issue'] }, minItems: 1 }], description: 'Category filter.' },
         limit: { type: 'number', default: 30, description: 'Max entries.' },
         offset: { type: 'number', default: 0, description: 'Skip entries.' },
-        includeMembers: { type: 'boolean', description: 'Include members.' },
-        includeRaw: { type: 'boolean', description: 'Include raw rows.' },
+        includeMembers: { type: 'boolean', description: 'Include chunk members in output; does not widen the search pool.' },
+        includeRaw: { type: 'boolean', description: 'Include unchunked raw/episode rows.' },
+        sessionOnly: { type: 'boolean', description: 'Search this session only.' },
         includeArchived: { type: 'boolean', description: 'Include archived.' },
         sessionId: { type: 'string', description: 'Scoped session id.' },
         session_id: { type: 'string', description: 'Alias for sessionId.' },

package/src/runtime/search/index.mjs

@@ -4,7 +4,6 @@ import { Server } from '@modelcontextprotocol/sdk/server/index.js'
 import { CallToolRequestSchema, ListToolsRequestSchema } from '@modelcontextprotocol/sdk/types.js'
 import { z } from 'zod'
 import fs from 'fs'
-import path from 'path'
 import {
   ensureDataDir,
   getRequestTimeoutMs,
@@ -32,7 +31,7 @@ import {
   loadUsageState,
   updateProviderState,
 } from './lib/state.mjs'
-import { assertPublicUrl, crawlSite, getScrapeCapabilities, pinnedFetch, scrapeUrls } from './lib/web-tools.mjs'
+import { getScrapeCapabilities, scrapeUrls } from './lib/web-tools.mjs'
 import { formatResponse } from './lib/formatter.mjs'
 ensureDataDir()
 
@@ -105,10 +104,6 @@ function formattedText(tool, payload) {
   }
 }
 
-function okText(text) {
-  return { content: [{ type: 'text', text }], isError: false }
-}
-
 function isInvalidSearchResult(result) {
   const title = String(result?.title || '').trim()
   return /\bpage not found\b|\b404\b.*\bnot found\b/i.test(title)
@@ -529,7 +524,7 @@ async function handleToolCall(name, rawArgs, options = {}) {
         throw e
       }
       try {
-        const result = await _fetchCore(urlArgs, { config, usageState, cacheState, timeoutMs, signal })
+        const result = await _fetchCore(urlArgs, { usageState, cacheState, timeoutMs, signal })
         flushCacheState()
         flushUsageState()
         return {

package/src/runtime/search/lib/config.mjs

@@ -44,10 +44,6 @@ export function writeJson(filePath, value) {
   writeJsonAtomicSync(filePath, value, { lock: true, fsyncDir: true })
 }
 
-export function saveConfig(config) {
-  return { ...DEFAULT_CONFIG, ...(config || {}) }
-}
-
 export function loadConfig() {
   ensureDataDir()
   if (Object.keys(readSection('search') || {}).length > 0) {

package/src/runtime/search/lib/state.mjs

@@ -88,10 +88,6 @@ export function loadUsageState() {
   return state
 }
 
-export function saveUsageState(state) {
-  scheduleUsageFlush(state)
-}
-
 export function updateProviderState(state, provider, patch) {
   let normalizedPatch = { ...patch }
   const remaining =
@@ -125,7 +121,7 @@ export function noteProviderSuccess(state, provider, extra = {}) {
   })
 }
 
-export const PROVIDER_ERROR_KIND = {
+const PROVIDER_ERROR_KIND = {
   AUTH: 'auth',
   QUOTA: 'quota',
   PAYMENT: 'payment',
@@ -151,16 +147,6 @@ export function classifyProviderError(error) {
   return PROVIDER_ERROR_KIND.UNKNOWN
 }
 
-/** Structured HTTP error for search backends (enables cooldown via classifyProviderError). */
-export function providerHttpError(provider, status, detail = '') {
-  const code = Number(status)
-  const snippet = detail ? `: ${String(detail).slice(0, 200)}` : ''
-  const err = new Error(`[search:${provider}] HTTP ${code}${snippet}`)
-  err.status = code
-  err.provider = provider
-  return err
-}
-
 const PROVIDER_DISABLE_TTL_MS = {
   auth: 24 * 3600 * 1000,
   quota: 24 * 3600 * 1000,

package/src/runtime/search/lib/web-tools.mjs

@@ -24,7 +24,6 @@ async function loadPuppeteer() {
   return _puppeteer
 }
 
-
 const PKG_VERSION = (() => { try { return JSON.parse(readFileSync(new URL('../package.json', import.meta.url), 'utf8')).version } catch { return '0.0.1' } })()
 import {
   noteProviderFailure,

package/src/runtime/shared/channel-notification-routing.mjs

@@ -0,0 +1,12 @@
+export function channelNotificationModelContent(params = {}) {
+  const meta = params?.meta && typeof params.meta === 'object' ? params.meta : {};
+  if (meta.silent_to_agent === true || meta.silent_to_agent === 'true') return '';
+  const instruction = typeof meta.instruction === 'string' ? meta.instruction.trim() : '';
+  const content = String(params?.content || '').trim();
+  return instruction || content;
+}
+
+export function shouldMirrorChannelNotificationToPending(meta = {}) {
+  const type = String(meta?.type || '').trim().toLowerCase();
+  return type === 'schedule';
+}

package/src/runtime/shared/channel-notification-routing.test.mjs

@@ -0,0 +1,45 @@
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import {
+  channelNotificationModelContent,
+  shouldMirrorChannelNotificationToPending,
+} from './channel-notification-routing.mjs';
+
+test('channel schedule notification uses instruction as model content', () => {
+  const content = channelNotificationModelContent({
+    content: ' ',
+    meta: {
+      type: 'schedule',
+      instruction: 'scheduled prompt body',
+    },
+  });
+  assert.equal(content, 'scheduled prompt body');
+});
+
+test('silent channel notification has no model content', () => {
+  const content = channelNotificationModelContent({
+    content: 'hidden',
+    meta: {
+      type: 'schedule',
+      silent_to_agent: true,
+    },
+  });
+  assert.equal(content, '');
+});
+
+test('non-schedule notification preserves instruction-first routing', () => {
+  const content = channelNotificationModelContent({
+    content: 'worker result body',
+    meta: {
+      type: 'webhook',
+      instruction: 'relay instruction',
+    },
+  });
+  assert.equal(content, 'relay instruction');
+});
+
+test('only schedule channel notifications are mirrored to pending fallback', () => {
+  assert.equal(shouldMirrorChannelNotificationToPending({ type: 'schedule' }), true);
+  assert.equal(shouldMirrorChannelNotificationToPending({ type: 'webhook' }), false);
+  assert.equal(shouldMirrorChannelNotificationToPending({}), false);
+});

package/src/runtime/shared/child-spawn-gate.mjs

@@ -137,9 +137,3 @@ export async function withGate(fn, signal = null) {
     release();
   }
 }
-
-export function gateStats() {
-  return { inflight: _inflight, queued: _queue.length, maxInflight: MAX_INFLIGHT };
-}
-
-export const CHILD_SPAWN_MAX_INFLIGHT = MAX_INFLIGHT;

package/src/runtime/shared/config.mjs

@@ -246,6 +246,7 @@ export function getCapabilities() {
 // the migration logic in seed.mjs.
 export const SECRET_ACCOUNTS = Object.freeze({
   discordToken: 'discord.token',
+  telegramToken: 'telegram.token',
   webhookAuth:  'webhook.authtoken',
   agentApiKey:  (provider) => `agent.${provider}.apiKey`,
   openaiUsageSessionKey: 'agent.openai.usageSessionKey',
@@ -293,6 +294,14 @@ export function getDiscordToken() {
   return _readSecret(SECRET_ACCOUNTS.discordToken)
 }
 
+/**
+ * Returns the Telegram bot token.
+ * Priority: MIXDOG_TELEGRAM_TOKEN → keychain('telegram.token') → null
+ */
+export function getTelegramToken() {
+  return _readSecret(SECRET_ACCOUNTS.telegramToken)
+}
+
 /**
  * Returns the ngrok/webhook authtoken.
  * Priority: MIXDOG_WEBHOOK_AUTHTOKEN → keychain('webhook.authtoken') → null

package/src/runtime/shared/llm/http-agent.mjs

@@ -19,7 +19,14 @@
  * so a shared long-lived pool is appropriate here.
  */
 
-import { Agent, getGlobalDispatcher, setGlobalDispatcher, request as undiciRequest } from 'undici'
+import { createRequire } from 'node:module'
+
+const require = createRequire(import.meta.url)
+let _undici = null
+function undici() {
+  if (!_undici) _undici = require('undici')
+  return _undici
+}
 
 let _agent = null
 let _globalInstalled = false
@@ -53,7 +60,7 @@ function proxyConfigured() {
   if (env.HTTP_PROXY || env.HTTPS_PROXY || env.http_proxy || env.https_proxy) return true
   if (env.NODE_USE_ENV_PROXY) return true
   try {
-    const g = getGlobalDispatcher?.()
+    const g = undici().getGlobalDispatcher?.()
     // Any non-default global dispatcher (constructor name other than the plain
     // `Agent` undici installs by default) is treated as custom — ProxyAgent,
     // EnvHttpProxyAgent, MockAgent, or a user subclass — and we step aside.
@@ -77,7 +84,7 @@ function proxyConfigured() {
 export function getLlmDispatcher() {
   if (proxyConfigured()) return undefined
   if (!_agent) {
-    _agent = new Agent({
+    _agent = new (undici().Agent)({
       keepAliveTimeout: envInt('MIXDOG_LLM_KEEPALIVE_MS', 60_000),
       keepAliveMaxTimeout: envInt('MIXDOG_LLM_KEEPALIVE_MAX_MS', 90_000),
       connections: envInt('MIXDOG_LLM_CONNECTIONS', 16),
@@ -87,7 +94,7 @@ export function getLlmDispatcher() {
   // a per-request dispatcher throws UND_ERR_INVALID_ARG. Install globally once
   // and omit the per-request dispatcher. See port-plan D7.
   if (!_globalInstalled) {
-    try { setGlobalDispatcher(_agent); _globalInstalled = true } catch { /* fall back */ }
+    try { undici().setGlobalDispatcher(_agent); _globalInstalled = true } catch { /* fall back */ }
   }
   return _globalInstalled ? undefined : _agent
 }
@@ -130,7 +137,7 @@ export function preconnect(origin) {
     // A throwaway HEAD lands a pooled socket without fetching a body. Any
     // failure (offline, DNS, 4xx/5xx) is irrelevant — the handshake is the
     // point, and the real request will surface genuine errors.
-    undiciRequest(key, {
+    undici().request(key, {
       method: 'HEAD',
       dispatcher: getLlmDispatcher(),
       signal: AbortSignal.timeout(10_000),

package/src/runtime/shared/schedules-store.mjs

@@ -2,32 +2,29 @@
  * Canonical reader for registered schedules.
  *
  * `<Mixdog data dir>/schedules/<name>/` is the single source of truth.
- * Each schedule directory contains `config.json` (metadata) and
- * `instructions.md` (prompt body). Both the setup UI (POST /schedules)
- * and the `schedule-add` skill write the same two files; every reader —
+ * Each schedule directory contains a single `SCHEDULE.md` — YAML-ish
+ * frontmatter (metadata: time/timezone/days/channel/model/enabled) plus a
+ * markdown body (the prompt). Both the setup UI (POST /schedules) and the
+ * `schedule-add` skill write the same one file; every reader —
  * setup-server (GET /schedules), channels/lib/config.mjs (loadConfig),
  * status/aggregator.mjs — must go through listSchedules() so a single
  * entry shape is presented everywhere.
  *
  * The legacy `mixdog-config.json` `channels.schedules.items` /
  * `channels.nonInteractive` / `channels.interactive` arrays are no longer
- * consulted. Migration: rename any legacy `prompt.md` to `instructions.md`
- * — no in-code fallback.
+ * consulted. Clean cut: the old `config.json` + `instructions.md` pair is
+ * no longer read — no in-code fallback.
  */
 
 import { readFileSync, readdirSync } from 'fs';
 import { join } from 'path';
 import { resolvePluginData } from './plugin-paths.mjs';
+import { readMarkdownDocument } from './markdown-frontmatter.mjs';
 
 function schedulesDir() {
   return join(resolvePluginData(), 'schedules');
 }
 
-function readJsonFile(path) {
-  try { return JSON.parse(readFileSync(path, 'utf8')); }
-  catch { return null; }
-}
-
 function readTextFile(path) {
   try { return readFileSync(path, 'utf8'); }
   catch { return ''; }
@@ -36,13 +33,13 @@ function readTextFile(path) {
 /**
  * List every registered schedule.
  *
- * Return shape per entry: `{ name, ...config, prompt }`.
+ * Return shape per entry: `{ name, ...frontmatter, prompt }`.
  * - `name` is the directory name (slug).
- * - Spread of `config.json` keys (time/days/type/channel/model/enabled
- *   when written by the setup UI; cron/timezone/mode/role when written
- *   by the schedule-add skill).
- * - `prompt` carries `instructions.md` content (empty string when the
- *   file is missing — caller decides whether that is a hard error).
+ * - Spread of `SCHEDULE.md` frontmatter keys (time/timezone/days/channel/
+ *   model/enabled). All frontmatter values arrive as strings; `enabled` is
+ *   cast to a boolean here so downstream `s.enabled !== false` filters work.
+ * - `prompt` carries the `SCHEDULE.md` body (empty string when the file is
+ *   missing — caller decides whether that is a hard error).
  *
  * Returns an empty array when the directory does not exist (fresh
  * install with no schedules registered yet).
@@ -62,9 +59,14 @@ export function listSchedules() {
   for (const ent of entries) {
     if (!ent.isDirectory()) continue;
     const name = ent.name;
-    const cfg = readJsonFile(join(dir, name, 'config.json')) || {};
-    const prompt = readTextFile(join(dir, name, 'instructions.md'));
-    out.push({ name, ...cfg, prompt });
+    const { frontmatter, body } = readMarkdownDocument(readTextFile(join(dir, name, 'SCHEDULE.md')));
+    const cfg = { ...frontmatter };
+    // Frontmatter is all-strings; cast enabled so `s.enabled !== false`
+    // filters (config.mjs, scheduler.mjs) treat `enabled: false` correctly.
+    if (Object.prototype.hasOwnProperty.call(cfg, 'enabled')) {
+      cfg.enabled = cfg.enabled !== 'false' && cfg.enabled !== false;
+    }
+    out.push({ name, ...cfg, prompt: body });
   }
   return out;
 }

package/src/runtime/shared/tool-surface.mjs

@@ -1,8 +1,15 @@
 import { displayModelName as sharedDisplayModelName } from '../../ui/model-display.mjs';
 
-const DEFAULT_SUMMARY_MAX = 160;
-// Semantic cap for collapsed agent/task card one-liners (spawn, send, response).
-export const AGENT_SURFACE_BRIEF_MAX = 120;
+// Default cap for tool-arg summaries (header parenthetical, channel tool lines).
+// Unified at 80 so every tool surface line — header arg summary and collapsed
+// detail alike — shares one width ceiling; per-call sites still clamp lower
+// (header 48, channel 50) where a tighter line is wanted.
+const DEFAULT_SUMMARY_MAX = 80;
+// Semantic cap for collapsed tool/agent/task card one-liners — the second row
+// under the ⎿ gutter (spawn, send, response, generic/shell result summaries).
+// Kept at 80 so every collapsed detail line is truncated to the same width
+// regardless of terminal columns; ctrl+o expand still shows the full body.
+export const AGENT_SURFACE_BRIEF_MAX = 80;
 const STATUS_SEPARATOR = ' · ';
 
 export function stripToolPrefix(name) {
@@ -205,7 +212,7 @@ function bridgeAgentModelSummary(args) {
   const modelId = firstText(args.model);
   const displayHint = firstText(args.modelDisplay, args.model_display, args.displayModel);
   return compactParts([
-    displayAgentName(firstText(args.agent, args.role, args.name, args.subagent_type)),
+    displayAgentName(firstText(args.agent, args.name, args.subagent_type)),
     displayModelName(modelId, provider, displayHint),
   ]);
 }
@@ -584,6 +591,22 @@ function countNonEmptyLines(text) {
     .filter((line) => line.trim()).length;
 }
 
+// Zero-result recognizer (audit HIGH): result text that SAYS "nothing found"
+// must summarize as an explicit zero, not be line-counted into "1 match".
+// Matches the shapes emitted by grep/glob/find/list/recall backends:
+//   "(no matches)", "no matches found", "(no results)", "No results",
+//   "(no fuzzy match for \"...\")", "0 matches", "(empty)", "(no entries)".
+function looksLikeZeroResultText(text) {
+  const trimmed = String(text ?? '').trim();
+  if (!trimmed) return false;
+  // Only trust short, single-line-ish payloads — a real listing that merely
+  // CONTAINS the words "no matches" somewhere must not be zeroed.
+  if (trimmed.length > 200 || trimmed.includes('\n')) return false;
+  return /^\(?\s*(?:no|0)\s+(?:fuzzy\s+)?(?:match(?:es)?|results?|files?|entries|candidates?|hits?)\b/i.test(trimmed)
+    || /^\(?\s*(?:empty|none)\s*\)?$/i.test(trimmed)
+    || /^no\s+\S+\s+(?:found|matched)\b/i.test(trimmed);
+}
+
 function splitPathAndDelta(value, explicitDelta = '') {
   let path = String(value ?? '').trim();
   let delta = String(explicitDelta ?? '').trim();
@@ -734,8 +757,8 @@ function summarizeGenericResult(text) {
       if (Array.isArray(parsed)) return `${parsed.length} ${pluralize(parsed.length, 'item')}`;
       if (parsed && typeof parsed === 'object') {
         const status = firstText(parsed.status, parsed.state, parsed.result, parsed.message);
-        if (status) return truncateSingleLine(titleStatus(status), 120);
-        if (parsed.cwd) return truncateSingleLine(parsed.cwd, 120);
+        if (status) return truncateSingleLine(titleStatus(status), AGENT_SURFACE_BRIEF_MAX);
+        if (parsed.cwd) return truncateSingleLine(parsed.cwd, AGENT_SURFACE_BRIEF_MAX);
         if (typeof parsed.ok === 'boolean') return parsed.ok ? 'Ok' : 'Failed';
         for (const key of ['items', 'results', 'resources', 'templates', 'providers', 'schedules', 'channels', 'tools']) {
           if (Array.isArray(parsed[key])) return `${parsed[key].length} ${pluralize(parsed[key].length, (key.slice(0, -1) || 'item').toLowerCase())}`;
@@ -751,7 +774,36 @@ function summarizeGenericResult(text) {
   if (/^(ok|done|success|saved|sent|updated|reloaded|connected|enabled|disabled|active|inactive)$/i.test(line)) {
     return titleStatus(line);
   }
-  return truncateSingleLine(line, 120);
+  return truncateSingleLine(line, AGENT_SURFACE_BRIEF_MAX);
+}
+
+// Error-cause extractor (audit HIGH): when a tool result is an error, surface
+// the actual cause line instead of a bare "Failed"/raw first line. Handles
+// JSON error envelopes ({error|message|cause|detail}) and plain-text bodies
+// (first line that carries error-ish signal, else first non-empty line).
+// Exported for ToolExecution's collapsed detail row.
+export function extractErrorCause(resultText) {
+  const text = String(resultText ?? '').trim();
+  if (!text) return '';
+  // JSON envelope: prefer explicit error-ish keys.
+  if (text.startsWith('{') || text.startsWith('[')) {
+    try {
+      const parsed = JSON.parse(text);
+      const obj = Array.isArray(parsed) ? parsed[0] : parsed;
+      if (obj && typeof obj === 'object') {
+        const cause = firstText(
+          typeof obj.error === 'string' ? obj.error : obj.error?.message,
+          obj.message, obj.cause, obj.detail, obj.reason, obj.status,
+        );
+        if (cause) return truncateSingleLine(String(cause), AGENT_SURFACE_BRIEF_MAX);
+      }
+    } catch { /* fall through to text scan */ }
+  }
+  const lines = text.split('\n').map((l) => l.trim()).filter(Boolean);
+  // Prefer the first line that looks like an error statement.
+  const errorish = lines.find((l) => /\b(error|failed|failure|denied|refused|timed?\s*out|timeout|not\s+found|missing|invalid|cannot|can't|exception|exit\s+(?:code\s+)?[1-9])\b/i.test(l));
+  const picked = errorish || lines[0] || '';
+  return truncateSingleLine(stripInlineMarkdown(picked), AGENT_SURFACE_BRIEF_MAX);
 }
 
 /**
@@ -760,7 +812,13 @@ function summarizeGenericResult(text) {
  * reliable can be derived, so the caller falls back to the raw result block.
  */
 export function summarizeToolResult(name, args, resultText, isError = false) {
-  if (isError) return null;
+  if (isError) {
+    // Audit HIGH: errors used to disable semantic summaries entirely, leaving
+    // the UI with a raw first line or a bare "Failed". Surface the extracted
+    // cause so the collapsed card answers "why" without ctrl+o.
+    const cause = extractErrorCause(resultText);
+    return cause || null;
+  }
   const text = String(resultText ?? '');
   const trimmed = text.trim();
   if (/^(?:undefined|null)$/i.test(trimmed)) return null;
@@ -800,18 +858,21 @@ export function summarizeToolResult(name, args, resultText, isError = false) {
     }
     case 'grep': {
       if (!trimmed || !looksLineOriented(text)) return null;
+      if (looksLikeZeroResultText(text)) return '0 matches';
       const n = countNonEmptyLines(text);
       if (n === 0) return null;
       return `${n} ${pluralize(n, 'match', 'matches')}`;
     }
     case 'glob': {
       if (!trimmed || !looksLineOriented(text)) return null;
+      if (looksLikeZeroResultText(text)) return '0 files';
       const n = countNonEmptyLines(text);
       if (n === 0) return null;
       return `${n} ${pluralize(n, 'file')}`;
     }
     case 'find': {
       if (!trimmed || !looksLineOriented(text)) return null;
+      if (looksLikeZeroResultText(text)) return '0 candidates';
       const n = countNonEmptyLines(text);
       if (n === 0) return null;
       return `${n} ${pluralize(n, 'candidate')}`;
@@ -819,6 +880,7 @@ export function summarizeToolResult(name, args, resultText, isError = false) {
     case 'list':
     case 'ls': {
       if (!trimmed || !looksLineOriented(text)) return null;
+      if (looksLikeZeroResultText(text)) return '0 entries';
       const n = countNonEmptyLines(text);
       if (n === 0) return null;
       return `${n} ${pluralize(n, 'entry', 'entries')}`;
@@ -840,15 +902,30 @@ export function summarizeToolResult(name, args, resultText, isError = false) {
         ]);
       }
       const firstLine = trimmed.split('\n').map((line) => line.trim()).find(Boolean) || trimmed;
-      return truncateSingleLine(firstLine, 120);
+      return truncateSingleLine(firstLine, AGENT_SURFACE_BRIEF_MAX);
     }
     case 'code_graph': {
       const match = /(\d+)\s+(references|definitions|symbols|callers|callees|results|matches)/i.exec(text);
       if (match) return `${match[1]} ${String(match[2]).toLowerCase()}`;
+      if (looksLikeZeroResultText(text)) return 'No results';
       return null;
     }
     case 'web_fetch':
     case 'fetch': {
+      // Audit HIGH: channel `fetch` (Discord message fetch — args carry
+      // channel/messageId/limit, never url/uri) was summarized as a WEB fetch,
+      // so its result missed both the status/size probes and fell to raw JSON.
+      // Route it to the generic JSON/text summarizer instead.
+      if (normalized === 'fetch') {
+        const a = parseToolArgs(args);
+        const isChannelFetch = !firstText(a.url, a.uri)
+          && Boolean(firstText(a.channel, a.channelId, a.chatId, a.messageId) || a.limit != null);
+        if (isChannelFetch) {
+          const n = countNonEmptyLines(text);
+          if (trimmed && looksLineOriented(text) && n > 0) return `${n} ${pluralize(n, 'message')}`;
+          return summarizeGenericResult(text);
+        }
+      }
       // Status: require a status-like context (HTTP NNN, "Status: NNN",
       // or "NNN OK"/"NNN Not Found") rather than any bare 3-digit number.
       const status = /(?:HTTP[\s/]*\d?\.?\d?\s*|status[:\s]+)([1-5]\d{2})\b/i.exec(text)
@@ -885,7 +962,15 @@ export function summarizeToolResult(name, args, resultText, isError = false) {
     }
     case 'recall':
     case 'search_memories':
-    case 'memory':
+    case 'memory': {
+      if (!trimmed || trimmed === '(no results)' || looksLikeZeroResultText(text)) return 'No Results';
+      let n = 0;
+      for (const line of text.split('\n')) {
+        if (/#\d+\s*$/.test(line)) n += 1;
+      }
+      if (n > 0) return `${n} ${pluralize(n, 'Memory', 'Memories')}`;
+      return summarizeGenericResult(text);
+    }
     case 'remember':
     case 'save_memory':
     case 'update_memory':
@@ -948,18 +1033,18 @@ export function summarizeToolResult(name, args, resultText, isError = false) {
       if (answerLine) return truncateSingleLine(stripInlineMarkdown(answerLine), AGENT_SURFACE_BRIEF_MAX);
       const task = /^agent task:\s*(\S+)/mi.exec(text);
       const status = /^status:\s*([^\s(]+)/mi.exec(text);
-      const role = /^role:\s*(.+)$/mi.exec(text);
+      const agent = /^agent:\s*(.+)$/mi.exec(text);
       const preset = /^preset:\s*(.+)$/mi.exec(text);
       const model = /^model:\s*(.+)$/mi.exec(text);
       const limits = /^limits:\s*(.+)$/mi.exec(text);
       const agentModel = compactParts([
-        displayAgentName(role ? role[1] : ''),
+        displayAgentName(agent ? agent[1] : ''),
         displayModelName(model ? model[1] : ''),
       ]);
       if (agentModel) return agentModel;
       const parts = [
         task ? task[1] : '',
-        role ? role[1] : '',
+        agent ? agent[1] : '',
         preset ? preset[1] : '',
         model ? model[1] : '',
         status ? titleStatus(status[1]) : '',