mixdog 1.3.1 → 1.3.2

package/src/session-runtime/agent-home-delivery.mjs

@@ -17,10 +17,21 @@
  * re-drains every queue on start, and
  * registers the live surface so a thread open in a window shows the notice at
  * once; a cold thread picks it up from its stored record.
+ *
+ * A group-room member's task reports (enqueueTask `from` its member session)
+ * go to the room instead: each line lands in the member's own session (never
+ * waking it: the room host drives its turns), and a blocked or ended turn
+ * also lands in the room's log as that member's speech row
+ * (agent-room-delivery.mjs appendTaskReports), where the room's next exchange
+ * sees it. Both ride this same durable queue, keyed by those sessions.
  */
 import { randomUUID } from 'node:crypto';
-import { readFile } from 'node:fs/promises';
+import { readFileSync } from 'node:fs';
 import { join } from 'node:path';
+import { readAgentDefinitionKind } from '../runtime/shared/agent-kind.mjs';
+import { BOT_AGENT_KIND, WORKFLOW_AGENT_KIND, isContactAgentId } from '../runtime/shared/agent-route-config.mjs';
+import { botStoreActive, isStoreBot } from '../runtime/shared/agent-store.mjs';
+import { readMarkdownDocument } from '../runtime/shared/markdown-frontmatter.mjs';
 import {
   MIXY_AGENT_ID,
   agentHomeSessionId,
@@ -31,20 +42,23 @@ import {
   isRoomThreadSessionId,
   roomIdOfSession,
   roomMemberOfSession,
+  roomSessionId,
 } from '../runtime/agent/orchestrator/session/agent-room.mjs';
 import {
   TASK_LOOP_CAP,
   agentTaskMessage,
+  agentTaskRelayFailedMessage,
   agentTaskWakePrompt,
+  boundedReply,
   routineOmittedMessage,
   routineResultMessage,
   routineResultNotice,
   routineRunNotice,
   routineSummary,
 } from '../runtime/agent/orchestrator/session/routine-result.mjs';
-import { parseJsonText, updateJsonAtomic } from '../runtime/shared/atomic-file.mjs';
+import { readJsonState, updateJsonAtomic } from '../runtime/shared/atomic-file.mjs';
 import { ROUTINE_LIMITS } from '../runtime/shared/routine-limits.mjs';
-import { submissionIdsOf } from './agent-task-sessions.mjs';
+import { submissionIdsOf, submissionOutcome } from './agent-task-sessions.mjs';
 import {
   heldRequestMessage,
   heldRequestNotice,
@@ -71,6 +85,38 @@ function deliveredRow(item, messageId) {
   return { message, notice: routineResultNotice(message) };
 }
 
+/** Whether agent `id` can have a home chat, decided only from a definition
+ *  read successfully: 'bot' (a store bot, or one the kind rule calls a bot),
+ *  'none' (read, and a workflow agent: an explicit non-bot `kind`, a shipped
+ *  role, or an unmarked agents/ copy once bots have their own store), else
+ *  'unknown' — an unreadable file, an unparsable one, or an unmarked user
+ *  copy without bot evidence never proves the agent is no bot. */
+export function readHomeAgent({ dataDir, shippedRoot, id, read = (path) => readFileSync(path, 'utf8') }) {
+  if (!dataDir || !id) return 'unknown';
+  if (isStoreBot(dataDir, id)) return 'bot';
+  const readOrAbsent = (path) => {
+    try {
+      return { text: read(path) };
+    } catch (error) {
+      return error?.code === 'ENOENT' || error?.code === 'ENOTDIR' ? { absent: true } : { failed: true };
+    }
+  };
+  const user = readOrAbsent(join(dataDir, 'agents', id, 'AGENT.md'));
+  if (user.failed) return 'unknown';
+  const shipped = shippedRoot ? readOrAbsent(join(shippedRoot, 'agents', id, 'AGENT.md')) : { absent: true };
+  const own = user.text ?? (shipped.failed ? null : shipped.text);
+  if (typeof own !== 'string') return 'unknown';
+  const kind = String(readMarkdownDocument(own).frontmatter?.kind || '')
+    .trim()
+    .toLowerCase();
+  if (kind) {
+    if (isContactAgentId(id, kind)) return 'bot';
+    return kind === WORKFLOW_AGENT_KIND ? 'none' : 'unknown';
+  }
+  if (user.text === undefined || typeof shipped.text === 'string' || botStoreActive(dataDir)) return 'none';
+  return readAgentDefinitionKind({ dataDir, shippedRoot, id }) === BOT_AGENT_KIND ? 'bot' : 'unknown';
+}
+
 /** The agent a schedule belongs to; ownerless rows belong to Mixy. */
 export function scheduleOwnerAgentId(schedule) {
   const owner = typeof schedule?.ownerAgentId === 'string' ? schedule.ownerAgentId.trim() : '';
@@ -81,7 +127,8 @@ let liveSurface = null;
 /** `{ busy(sessionId), notice(sessionId, notice), wake(sessionId, prompt, { id }),
  *  submit(sessionId, prompt, options), abort(sessionId) }` from the session
  *  service; `wake` starts one hidden turn with submission `id` (resolves
- *  false when it was refused), `submit` queues a routine turn
+ *  false when it was refused, else `{ settled }`: a promise of the turn's end
+ *  status, 'done' when it finished), `submit` queues a routine turn
  *  (resolves whether it was accepted), `abort` stops the running turn. */
 export function setAgentHomeLiveSurface(surface) {
   liveSurface = surface || null;
@@ -100,6 +147,12 @@ export const ROUND_TTL_MS = 30 * 24 * 60 * 60 * 1000;
 // Per home: round-trip targets counted (least recently counted drop first),
 // reporting turns watched at once, and posted results owed a wake.
 export const QUEUE_CAPS = Object.freeze({ roundTargets: 100, watches: 200, wakeRunIds: 100 });
+// A task report's output kept for its room row (newlines kept).
+const ROOM_REPORT_TEXT_LIMIT = 4000;
+// Waits before the 2nd, 3rd and 4th try of a relay (wake) turn that did not finish.
+export const WAKE_RETRY_DELAYS_MS = Object.freeze([30_000, 120_000, 600_000]);
+// Refused wake submissions of one batch, a minute apart, before it is given up.
+export const WAKE_REFUSAL_LIMIT = 10;
 
 /** `homeId`'s wake obligations in `wakes`: posted task results not in a batch
  *  yet (`owed`), those past the cap collapsed into one marker (`overflow`:
@@ -127,6 +180,83 @@ function withWakeState(doc, homeId, { owed, batches, overflow }) {
   return { ...doc, wakes };
 }
 
+/** `own`'s per-target counts (`{ n, at }`) used within ROUND_TTL_MS of
+ *  `nowMs`, the `cap` most recently counted. */
+function freshRoundCounts(own, nowMs, cap) {
+  return Object.fromEntries(
+    Object.entries(own && typeof own === 'object' ? own : {})
+      .filter(([, count]) => count && typeof count === 'object' && nowMs - (Number(count.at) || 0) < ROUND_TTL_MS)
+      .sort(([, a], [, b]) => (Number(b.at) || 0) - (Number(a.at) || 0))
+      .slice(0, cap)
+  );
+}
+
+/** `doc` with `items` appended to `homeId`'s list, minus run ids it holds or
+ *  posted; past `limit` the oldest drop, counted under `omitted`. Null when
+ *  nothing changes. */
+function appendedItems(doc, homeId, items, limit, omittedCount) {
+  const list = doc.homes[homeId] || [];
+  const seen = new Set([...list.map((row) => row.runId), ...(doc.delivered[homeId] || [])]);
+  const fresh = items.filter((item) => !seen.has(item.runId) && seen.add(item.runId));
+  if (!fresh.length && !omittedCount) return null;
+  const next = [...list, ...fresh];
+  const drop = Math.max(0, next.length - limit);
+  const omitted = { ...doc.omitted };
+  const count = drop + omittedCount;
+  if (count) {
+    const previous = omitted[homeId];
+    omitted[homeId] = { id: previous?.id || `omit_${next[0].runId}`, count: (previous?.count || 0) + count };
+  }
+  return { ...doc, homes: { ...doc.homes, [homeId]: next.slice(drop) }, omitted };
+}
+
+/** `doc.wakes` with the run ids `wake` owed a wake by `homeId`, at most
+ *  `cap` per home: past it the oldest owed collapse into the overflow marker
+ *  (`overflowed` counts them). Stored batches never change. */
+function owedWakes(doc, homeId, wake, cap) {
+  const state = wakeState(doc.wakes, homeId);
+  const batched = state.batches.flatMap((batch) => batch.runIds);
+  const known = new Set([
+    ...state.owed,
+    ...batched,
+    ...[state.overflow?.oldest, state.overflow?.newest].filter(Boolean),
+  ]);
+  const added = wake.filter((runId) => !known.has(runId));
+  if (!added.length) return { wakes: doc.wakes, overflowed: 0 };
+  const owed = [...state.owed, ...added];
+  const room = Math.max(0, cap - batched.length);
+  const excess = owed.slice(0, Math.max(0, owed.length - room));
+  const overflow = excess.length
+    ? {
+        count: (state.overflow?.count || 0) + excess.length,
+        oldest: state.overflow?.oldest || excess[0],
+        newest: excess.at(-1),
+      }
+    : state.overflow;
+  const { wakes } = withWakeState(doc, homeId, { ...state, owed: owed.slice(excess.length), overflow });
+  return { wakes, overflowed: excess.length };
+}
+
+/** `doc` with the posted `runIds` of `homeId` dequeued into its posted
+ *  ledger, and the posted omission notice `omitted` retired. */
+function withDequeued(doc, homeId, runIds, omitted) {
+  const homes = { ...doc.homes };
+  const rest = (homes[homeId] || []).filter((row) => !runIds.includes(row.runId));
+  if (rest.length) homes[homeId] = rest;
+  else delete homes[homeId];
+  const delivered = { ...doc.delivered };
+  if (runIds.length) delivered[homeId] = [...(delivered[homeId] || []), ...runIds].slice(-DELIVERED_LEDGER);
+  const omittedNow = { ...doc.omitted };
+  const current = omittedNow[homeId];
+  if (omitted && current?.id === omitted.id) {
+    // More dropped while this pass posted: they get a notice of their own.
+    const left = current.count - omitted.count;
+    if (left > 0) omittedNow[homeId] = { id: `${current.id}.${omitted.count}`, count: left };
+    else delete omittedNow[homeId];
+  }
+  return { ...doc, homes, omitted: omittedNow, delivered };
+}
+
 /** The persisted queue at `filePath`: `{ homes: { [homeSessionId]: item[] },
  *  omitted: { [homeSessionId]: { id, count } }, delivered: { [homeSessionId]:
  *  runId[] } }`. A home keeps at most `limit` items: past it the oldest drop,
@@ -136,15 +266,7 @@ export function createRoutineResultQueue(
   { limit = ROUTINE_LIMITS.homeQueueLimit, now = Date.now, caps = QUEUE_CAPS } = {}
 ) {
   const section = (doc, key) => (doc && typeof doc[key] === 'object' && doc[key]) || {};
-  /** `own`'s per-target counts (`{ n, at }`) used within ROUND_TTL_MS, the
-   *  `caps.roundTargets` most recently counted. */
-  const freshRounds = (own) =>
-    Object.fromEntries(
-      Object.entries(own && typeof own === 'object' ? own : {})
-        .filter(([, count]) => count && typeof count === 'object' && now() - (Number(count.at) || 0) < ROUND_TTL_MS)
-        .sort(([, a], [, b]) => (Number(b.at) || 0) - (Number(a.at) || 0))
-        .slice(0, caps.roundTargets)
-    );
+  const freshRounds = (own) => freshRoundCounts(own, now(), caps.roundTargets);
   const tidyRounds = (rounds) =>
     Object.fromEntries(
       Object.entries(rounds || {})
@@ -162,39 +284,26 @@ export function createRoutineResultQueue(
     // Posted task results whose wake turn was not taken yet, per home.
     wakes: section(doc, 'wakes'),
   });
+  // A damaged queue file (empty, NUL-filled after a crash, malformed) is kept
+  // aside and its last good copy restored (readJsonState); it never blocks
+  // delivery. I/O errors still throw.
   async function read() {
-    try {
-      return parts(parseJsonText(await readFile(filePath, 'utf8')));
-    } catch (error) {
-      if (error?.code === 'ENOENT') return parts(null);
-      throw error;
-    }
+    return parts(await readJsonState(filePath));
   }
   /** Read-modify-write under the file lock (`mutator(doc)` returns the next
    *  doc, or nothing to keep the file); every write also retires stale and
    *  surplus round counts. */
   function update(mutator) {
-    return updateJsonAtomic(filePath, (raw) => {
-      const next = mutator(parts(raw));
-      return next ? { ...next, rounds: tidyRounds(next.rounds) } : undefined;
-    });
-  }
-  /** `doc` with `items` appended to `homeId`'s list, minus run ids it holds or posted. */
-  function appended(doc, homeId, items, omittedCount = 0) {
-    const list = doc.homes[homeId] || [];
-    const seen = new Set([...list.map((row) => row.runId), ...(doc.delivered[homeId] || [])]);
-    const fresh = items.filter((item) => !seen.has(item.runId) && seen.add(item.runId));
-    if (!fresh.length && !omittedCount) return null;
-    const next = [...list, ...fresh];
-    const drop = Math.max(0, next.length - limit);
-    const omitted = { ...doc.omitted };
-    const count = drop + omittedCount;
-    if (count) {
-      const previous = omitted[homeId];
-      omitted[homeId] = { id: previous?.id || `omit_${next[0].runId}`, count: (previous?.count || 0) + count };
-    }
-    return { ...doc, homes: { ...doc.homes, [homeId]: next.slice(drop) }, omitted };
+    return updateJsonAtomic(
+      filePath,
+      (raw) => {
+        const next = mutator(parts(raw));
+        return next ? { ...next, rounds: tidyRounds(next.rounds) } : undefined;
+      },
+      { backup: true }
+    );
   }
+  const appended = (doc, homeId, items, omittedCount = 0) => appendedItems(doc, homeId, items, limit, omittedCount);
   return {
     async pendingHomes() {
       const { homes, omitted, wakes } = await read();
@@ -237,6 +346,31 @@ export function createRoutineResultQueue(
         return withWakeState(doc, homeId, { ...state, batches });
       });
     },
+    /** Batch `batchId`'s relay turn did not finish: it is owed again under a
+     *  fresh wake id (the old one names the failed turn), as try `tries`, not
+     *  before `notBefore`. Returns the new batch; null when it is gone. */
+    async rewake(homeId, batchId, { tries, notBefore }) {
+      let next = null;
+      await update((doc) => {
+        next = null;
+        const state = wakeState(doc.wakes, homeId);
+        const index = state.batches.findIndex((batch) => batch.id === batchId);
+        if (index < 0) return undefined;
+        next = { ...state.batches[index], id: `awake_${randomUUID()}`, tries, notBefore };
+        const batches = state.batches.map((batch, i) => (i === index ? next : batch));
+        return withWakeState(doc, homeId, { ...state, batches });
+      });
+      return next;
+    },
+    /** Store `fields` on batch `batchId` (same wake id). */
+    async markBatch(homeId, batchId, fields) {
+      await update((doc) => {
+        const state = wakeState(doc.wakes, homeId);
+        if (!state.batches.some((batch) => batch.id === batchId)) return undefined;
+        const batches = state.batches.map((batch) => (batch.id === batchId ? { ...batch, ...fields } : batch));
+        return withWakeState(doc, homeId, { ...state, batches });
+      });
+    },
     /** Drop every wake obligation of `homeId` (its owner is gone). */
     async dropWakes(homeId) {
       await update((doc) =>
@@ -314,7 +448,9 @@ export function createRoutineResultQueue(
           (kept) => kept?.homeSessionId === watch.homeSessionId && kept.runId !== watch.runId
         );
         if (own.length >= caps.watches) {
-          throw new Error(`${watch.homeSessionId} already waits on ${own.length} session reports`);
+          throw Object.assign(new Error(`${watch.homeSessionId} already waits on ${own.length} session reports`), {
+            code: 'REPORT_WATCH_CAP',
+          });
         }
         return { ...doc, watches: { ...watches, [watch.runId]: watch } };
       });
@@ -363,45 +499,9 @@ export function createRoutineResultQueue(
     async remove(homeId, runIds, omitted = null, { wake = [] } = {}) {
       let overflowed = 0;
       await update((doc) => {
-        overflowed = 0;
-        const state = wakeState(doc.wakes, homeId);
-        const batched = state.batches.flatMap((batch) => batch.runIds);
-        const known = new Set([
-          ...state.owed,
-          ...batched,
-          ...[state.overflow?.oldest, state.overflow?.newest].filter(Boolean),
-        ]);
-        const added = wake.filter((runId) => !known.has(runId));
-        let wakes = doc.wakes;
-        if (added.length) {
-          const owed = [...state.owed, ...added];
-          const room = Math.max(0, caps.wakeRunIds - batched.length);
-          const excess = owed.slice(0, Math.max(0, owed.length - room));
-          overflowed = excess.length;
-          const overflow = excess.length
-            ? {
-                count: (state.overflow?.count || 0) + excess.length,
-                oldest: state.overflow?.oldest || excess[0],
-                newest: excess.at(-1),
-              }
-            : state.overflow;
-          ({ wakes } = withWakeState(doc, homeId, { ...state, owed: owed.slice(excess.length), overflow }));
-        }
-        const homes = { ...doc.homes };
-        const rest = (homes[homeId] || []).filter((row) => !runIds.includes(row.runId));
-        if (rest.length) homes[homeId] = rest;
-        else delete homes[homeId];
-        const delivered = { ...doc.delivered };
-        if (runIds.length) delivered[homeId] = [...(delivered[homeId] || []), ...runIds].slice(-DELIVERED_LEDGER);
-        const omittedNow = { ...doc.omitted };
-        const current = omittedNow[homeId];
-        if (omitted && current?.id === omitted.id) {
-          // More dropped while this pass posted: they get a notice of their own.
-          const left = current.count - omitted.count;
-          if (left > 0) omittedNow[homeId] = { id: `${current.id}.${omitted.count}`, count: left };
-          else delete omittedNow[homeId];
-        }
-        return { ...doc, homes, omitted: omittedNow, delivered, wakes };
+        let wakes;
+        ({ wakes, overflowed } = owedWakes(doc, homeId, wake, caps.wakeRunIds));
+        return { ...withDequeued(doc, homeId, runIds, omitted), wakes };
       });
       return { overflowed };
     },
@@ -410,6 +510,281 @@ export function createRoutineResultQueue(
 
 const messageIdOf = (runId) => `routine_${runId}`;
 
+/** Show `notice` live in `homeId` through the `live()` surface: its answer,
+ *  or false (logged) when the surface threw. */
+function noticeLive(live, log, homeId, notice) {
+  try {
+    return live()?.notice?.(homeId, notice);
+  } catch (error) {
+    log(`live notice in ${homeId} failed: ${error?.message || error}`);
+    return false;
+  }
+}
+
+/** Dequeue every queued item (and the omission) of `homeId` unposted, and
+ *  its wake obligations. */
+async function dropQueued(queue, homeId, items, omitted) {
+  await queue.remove(
+    homeId,
+    items.map((item) => item.runId),
+    omitted
+  );
+  await queue.dropWakes(homeId);
+}
+
+/** The queued item of `result` ({ runId, kind, ok, text, runSessionId }) of
+ *  a schedule or webhook `row` (enqueue). */
+function routineResultItem(row, result) {
+  return {
+    runId: String(result.runId),
+    kind: result.kind === 'webhook' ? 'webhook' : 'schedule',
+    name: row.name,
+    ok: result.ok === true,
+    text: routineSummary(result.text),
+    runSessionId: result.runSessionId || null,
+    ...(result.reason ? { reason: String(result.reason) } : {}),
+    ...(result.paused === true ? { paused: true } : {}),
+    at: Date.now(),
+  };
+}
+
+/** The run ids of `watches` a home at its watch `cap` gives up before
+ *  storing `watch`: its oldest whose target sessions are proven deleted. */
+function watchesToDrop(watches, watch, cap, sessionDeleted) {
+  const own = watches.filter((kept) => kept?.homeSessionId === watch.homeSessionId && kept.runId !== watch.runId);
+  const over = own.length + 1 - cap;
+  return over > 0
+    ? own
+        .filter((kept) => sessionDeleted(kept.taskSessionId))
+        .slice(0, over)
+        .map((kept) => kept.runId)
+    : [];
+}
+
+/** A bookkept home proven deleted: a bot's home whose definition is gone,
+ *  or a room thread whose room is. Never Mixy's home, never on a missing
+ *  thread record, never on an unknown answer. */
+function homeProvenDeleted(homeId, agentPresence, roomPresence) {
+  const agentId = agentIdOfHomeSession(homeId);
+  if (agentId) return agentId !== MIXY_AGENT_ID && agentPresence(agentId) === 'deleted';
+  const roomId = roomIdOfSession(homeId) || roomMemberOfSession(homeId)?.roomId || '';
+  return Boolean(roomId) && roomPresence(roomId) === 'deleted';
+}
+
+/** The queued item of one task report (enqueueTask). */
+function taskReportItem({ runId, taskSessionId, title, status, text = '', target, files = [], omitted = 0, hop = 0 }) {
+  // The model-facing reply keeps its lines up to REPLY_LIMIT; the row's
+  // one-line summary is derived from it (agentTaskMessage).
+  const reply = boundedReply(text);
+  return {
+    runId: String(runId),
+    kind: 'agent-task',
+    taskSessionId,
+    title,
+    status,
+    text: reply.text,
+    // The UI row's one line comes from the whole reply, not its bounded part.
+    summary: routineSummary(text),
+    ...(reply.chars > reply.text.length ? { textChars: reply.chars } : {}),
+    // Any session other than a task session the agent created.
+    ...(target === 'session' ? { target } : {}),
+    // The files the turn produced (handoff-attachments.mjs collectTurnArtifacts).
+    ...(Array.isArray(files) && files.length ? { files, ...(omitted > 0 ? { omitted } : {}) } : {}),
+    roundTrip: status !== 'started' && status !== 'blocked',
+    ...(hop > 0 ? { hop } : {}),
+    at: Date.now(),
+  };
+}
+
+/** The room-log item of task report `item` by member `agentId` (`text`: the
+ *  report's whole output, newlines kept). */
+function roomReportItem(item, agentId, text) {
+  return {
+    runId: item.runId,
+    kind: 'room-report',
+    speaker: agentId,
+    taskSessionId: item.taskSessionId,
+    title: item.title,
+    status: item.status,
+    text: String(text ?? '')
+      .trim()
+      .slice(0, ROOM_REPORT_TEXT_LIMIT),
+    ...(item.target ? { target: item.target } : {}),
+    ...(item.files ? { files: item.files } : {}),
+    at: item.at,
+  };
+}
+
+/** Queue a room member's task report: the member's own session (`from`) gets
+ *  the line without a wake; a blocked or ended turn (`toRoom`) is the
+ *  member's row in the room log too. The watch drops with the last write, so
+ *  a restart in between re-queues both (deduplicated). */
+async function queueRoomTaskReport(queue, { from, roomLogId, toRoom }, item, agentId, text, unwatch) {
+  await queue.add(from, { ...item, roundTrip: false }, { unwatch: toRoom ? '' : unwatch });
+  if (toRoom) {
+    await queue.add(roomLogId, roomReportItem(item, agentId, text), { unwatch });
+  }
+}
+
+/** Append to `session.messages` the omission notice and the queued `items`
+ *  not posted before. `fresh`: the rows newly posted; `toWake`: the run ids
+ *  owed a wake (every dequeued task result, posted now or before). */
+function postQueuedRows(session, items, omitted) {
+  const posted = new Set(session.messages.map((message) => message?.meta?.transcript?.messageId).filter(Boolean));
+  const fresh = [];
+  const omittedId = omitted ? messageIdOf(omitted.id) : '';
+  if (omitted && !posted.has(omittedId)) {
+    const message = routineOmittedMessage({ count: omitted.count, messageId: omittedId });
+    session.messages.push(message);
+    fresh.push({ message, notice: routineResultNotice(message) });
+  }
+  const toWake = [];
+  for (const item of items) {
+    const row = deliveredRow(item, messageIdOf(item.runId));
+    if (row.wake) toWake.push(item.runId);
+    if (posted.has(messageIdOf(item.runId))) continue;
+    session.messages.push(row.message);
+    fresh.push(row);
+  }
+  return { fresh, toWake };
+}
+
+/** The relay of posted task results: owed results wake the agent with one
+ *  hidden turn per batch. A batch is stored with its wake id BEFORE the wake
+ *  is submitted and dropped once its relay turn settled done, so a restart in
+ *  between re-checks that exact id; a relay turn that did not finish is owed
+ *  again under a fresh id (retryRelay). Results owed later form a new batch.
+ *  A replayed run id is never queued again. */
+function createHomeRelay({
+  queue,
+  saveSession,
+  live,
+  busy,
+  later,
+  drain,
+  log,
+  wakeRetryDelaysMs,
+  wakeRefusalLimit,
+  busyRetryMs,
+  missingRetryMs,
+}) {
+  // Wake batch id → 'running' while this process's relay turn runs, then
+  // 'done' or 'failed' once it settled (until the batch is retired).
+  const relays = new Map();
+
+  /** How batch `batchId`'s relay turn went: '' when no turn took it, else
+   *  'running', 'done' or 'failed'. A turn an earlier process took decides by
+   *  its recorded end; one with none was cut off by the restart. */
+  function relayOf(batchId, messages, taken) {
+    const local = relays.get(batchId);
+    if (local) return local;
+    if (!taken.has(batchId)) return '';
+    const outcome = submissionOutcome(messages, batchId);
+    return outcome && outcome !== 'open' && outcome.status === 'completed' ? 'done' : 'failed';
+  }
+
+  /** The relay turn of `batch` did not finish: owe it again after a backoff,
+   *  or, past the last try, leave a note in the thread and wake nobody. */
+  async function retryRelay(homeId, session, batch) {
+    const tries = (Number(batch.tries) || 0) + 1;
+    if (tries <= wakeRetryDelaysMs.length) {
+      log(`the relay turn in ${homeId} did not finish (try ${tries}); waking it again later`);
+      await queue.rewake(homeId, batch.id, { tries, notBefore: Date.now() + wakeRetryDelaysMs[tries - 1] });
+      return true;
+    }
+    return abandonRelay(homeId, session, batch, `did not finish after ${tries} tries`, tries);
+  }
+
+  /** Give up on `batch`'s relay: a note in the thread (once), nobody woken.
+   *  False while the thread is busy (try again once it is idle). */
+  async function abandonRelay(homeId, session, batch, why, tries) {
+    if (busy(homeId)) return false;
+    log(`the relay turn in ${homeId} ${why}; its results stay posted unrelayed`);
+    const messageId = messageIdOf(`relay_${batch.id}`);
+    if (!session.messages.some((message) => message?.meta?.transcript?.messageId === messageId)) {
+      const message = agentTaskRelayFailedMessage({
+        count: batch.runIds.length + (batch.overflow?.count || 0),
+        tries,
+        messageId,
+      });
+      session.messages.push(message);
+      session.updatedAt = Date.now();
+      await saveSession(session);
+      noticeLive(live, log, homeId, { ...routineResultNotice(message), messageId });
+    }
+    await queue.woke(homeId, batch.id);
+    return true;
+  }
+
+  /** Wake `homeId` (its idle `session`, just posted to) for its next owed batch. */
+  async function wakeOwed(homeId, session) {
+    const byId = new Map(session.messages.map((message) => [message?.meta?.transcript?.messageId, message]));
+    const taken = new Set(session.messages.flatMap((message) => submissionIdsOf(message)));
+    let batch = null;
+    let woken = [];
+    let more = 0;
+    for (;;) {
+      batch = (await queue.wakes(homeId)).batches[0] || (await queue.batchWake(homeId));
+      if (!batch) return;
+      const relay = relayOf(batch.id, session.messages, taken);
+      // Its settle drains again.
+      if (relay === 'running') return;
+      if (relay) {
+        relays.delete(batch.id);
+        if (relay === 'done') await queue.woke(homeId, batch.id);
+        else if (!(await retryRelay(homeId, session, batch))) return later(homeId, busyRetryMs);
+        continue;
+      }
+      const wait = (Number(batch.notBefore) || 0) - Date.now();
+      if (wait > 0) return later(homeId, wait);
+      woken = batch.runIds.map((runId) => byId.get(messageIdOf(runId))).filter(Boolean);
+      // Results past the wake cap ride this wake as a count of rows to read.
+      more = byId.has(messageIdOf(batch.overflow?.newest)) ? batch.overflow.count : 0;
+      if (woken.length || more) break;
+      // Its rows are gone with a thread reset.
+      await queue.woke(homeId, batch.id);
+    }
+    const surface = live();
+    if (typeof surface?.wake !== 'function') return later(homeId, busyRetryMs);
+    const batchId = batch.id;
+    let turn;
+    relays.set(batchId, 'running');
+    try {
+      // The turn descends from the bot-to-bot sends it reports: it carries their highest hop.
+      const hop = Math.max(0, ...woken.map((message) => Number(message?.meta?.routine?.hop) || 0));
+      turn = await surface.wake(homeId, agentTaskWakePrompt(woken, { more }), { id: batchId, hop });
+      if (turn === false) throw new Error('the wake turn was refused');
+    } catch (error) {
+      relays.delete(batchId);
+      // A wake refused (or failing) every time is retried a bounded number of times.
+      const refusals = (Number(batch.refusals) || 0) + 1;
+      if (refusals >= wakeRefusalLimit) {
+        if (await abandonRelay(homeId, session, batch, `could not start (${refusals} refusals)`, refusals)) return;
+        return later(homeId, busyRetryMs);
+      }
+      await queue.markBatch(homeId, batchId, { refusals });
+      log(`waking ${homeId} failed (${refusals}/${wakeRefusalLimit}), retrying: ${error?.message || error}`);
+      return later(homeId, missingRetryMs);
+    }
+    // `settled` resolves the relay turn's end status: only 'done' retires the batch.
+    if (typeof turn?.settled?.then === 'function') {
+      const settled = (status) => {
+        relays.set(batchId, status === 'done' ? 'done' : 'failed');
+        void drain(homeId);
+      };
+      turn.settled.then(settled, () => settled('failed'));
+      return;
+    }
+    // A surface that cannot report the turn's end: taking it is relaying it.
+    relays.delete(batchId);
+    await queue.woke(homeId, batchId);
+    // Another batch left by an earlier process wakes once this turn is done.
+    if (owesWake(await queue.wakes(homeId))) later(homeId, busyRetryMs);
+  }
+
+  return { wakeOwed };
+}
+
 export function createAgentHomeDelivery({
   loadSession,
   saveSession,
@@ -427,6 +802,19 @@ export function createAgentHomeDelivery({
   // True only on positive evidence that session `id` was deleted; an
   // unreadable or unknown record is not deleted.
   sessionDeleted = () => false,
+  // 'bot' | 'none' | 'unknown' per agent id: whether it can have a home chat
+  // at all. 'none' (proven: a workflow agent such as Reviewer) drops the rows
+  // queued for its missing home instead of waiting on it forever.
+  homeAgent = () => 'unknown',
+  // Waits before each new try of a relay turn that did not finish; past the
+  // last one the results stay posted with a note and nobody is woken.
+  wakeRetryDelaysMs = WAKE_RETRY_DELAYS_MS,
+  // Refused or failing wake submissions of one batch (missingRetryMs apart)
+  // before it is given up the same way.
+  wakeRefusalLimit = WAKE_REFUSAL_LIMIT,
+  // Post queued room task reports into a room log (agent-room-delivery.mjs).
+  postRoomReports = async (roomLogId, items) =>
+    (await import('./agent-room-delivery.mjs')).appendRoomTaskReports(roomLogId, items),
   log = (line) => process.stderr.write(`mixdog routines: ${line}\n`),
 }) {
   const busy = (sessionId) => {
@@ -441,6 +829,20 @@ export function createAgentHomeDelivery({
   const again = new Set(); // homes enqueued to while their pass ran
   const waiting = new Set(); // homes with a retry timer armed
   const shownLive = new Set(); // held-request message ids shown live in a busy thread
+  const waitLogged = new Set(); // missing homes whose wait was logged
+  const relay = createHomeRelay({
+    queue,
+    saveSession,
+    live,
+    busy,
+    later,
+    drain,
+    log,
+    wakeRetryDelaysMs,
+    wakeRefusalLimit,
+    busyRetryMs,
+    missingRetryMs,
+  });
 
   /** A held request waits on the user, possibly raised by the very turn that
    *  keeps its thread busy (a routine turn): show it live at once; the stored
@@ -451,11 +853,7 @@ export function createAgentHomeDelivery({
       const messageId = messageIdOf(item.runId);
       if (shownLive.has(messageId)) continue;
       shownLive.add(messageId);
-      try {
-        live()?.notice?.(homeId, { ...deliveredRow(item, messageId).notice, messageId });
-      } catch (error) {
-        log(`live notice in ${homeId} failed: ${error?.message || error}`);
-      }
+      noticeLive(live, log, homeId, { ...deliveredRow(item, messageId).notice, messageId });
     }
   }
 
@@ -468,8 +866,48 @@ export function createAgentHomeDelivery({
     }, ms);
   }
 
+  /** A room thread proven gone: its queued rows are dropped, never retried. */
+  async function dropGoneRoomThread(homeId, items, omitted) {
+    if (!isRoomThreadSessionId(homeId) || !(homeDeleted(homeId) || sessionDeleted(homeId))) return false;
+    log(`${items.length} task report(s) for the deleted ${homeId} are dropped`);
+    await dropQueued(queue, homeId, items, omitted);
+    return true;
+  }
+
+  /** A missing home of an agent that can never have one (no bot): its queued
+   *  rows are dropped with a log line, never retried. */
+  async function dropHomeless(homeId, items, omitted) {
+    const agentId = agentIdOfHomeSession(homeId);
+    if (!agentId || agentId === MIXY_AGENT_ID || homeAgent(agentId) !== 'none') return false;
+    log(`${items.length} result(s) for ${homeId} are dropped: ${agentId} is not a bot, so it has no chat`);
+    await dropQueued(queue, homeId, items, omitted);
+    return true;
+  }
+
+  /** Post every queued task report of the room log `roomLogId` as member rows. */
+  async function drainRoomLog(roomLogId) {
+    const items = await queue.items(roomLogId);
+    const omitted = await queue.omitted(roomLogId);
+    if (!items.length && !omitted) return;
+    const session = loadSession(roomLogId);
+    if (!session || session.closed === true) {
+      if (await dropGoneRoomThread(roomLogId, items, omitted)) return;
+      log(`${items.length} task report(s) wait for the room log ${roomLogId}`);
+      return later(roomLogId, missingRetryMs);
+    }
+    // A room log has no notice row: the dropped count is only logged.
+    if (omitted) log(`${omitted.count} older task report(s) for ${roomLogId} were dropped: its queue was full`);
+    if (items.length) await postRoomReports(roomLogId, items);
+    await queue.remove(
+      roomLogId,
+      items.map((item) => item.runId),
+      omitted
+    );
+  }
+
   /** Post every queued result of `homeId` if its thread is idle; else retry later. */
   async function drainOnce(homeId) {
+    if (roomIdOfSession(homeId)) return drainRoomLog(homeId);
     const items = await queue.items(homeId);
     const omitted = await queue.omitted(homeId);
     const owed = owesWake(await queue.wakes(homeId));
@@ -491,37 +929,23 @@ export function createAgentHomeDelivery({
     // From here to the push nothing awaits, so no turn can start in between.
     const session = loadSession(homeId);
     if (!session || session.closed === true) {
-      log(`${items.length} result(s) wait for the home thread ${homeId}`);
+      if (await dropGoneRoomThread(homeId, items, omitted)) return;
+      if (await dropHomeless(homeId, items, omitted)) return;
+      // Kept (a bot, or an agent whose kind cannot be read): logged once per process.
+      if (!waitLogged.has(homeId)) {
+        waitLogged.add(homeId);
+        log(`${items.length} result(s) wait for the home thread ${homeId}`);
+      }
       return later(homeId, missingRetryMs);
     }
     if (!Array.isArray(session.messages)) session.messages = [];
-    const posted = new Set(session.messages.map((message) => message?.meta?.transcript?.messageId).filter(Boolean));
-    const fresh = [];
-    const omittedId = omitted ? messageIdOf(omitted.id) : '';
-    if (omitted && !posted.has(omittedId)) {
-      const message = routineOmittedMessage({ count: omitted.count, messageId: omittedId });
-      session.messages.push(message);
-      fresh.push(message);
-    }
-    const toWake = [];
-    for (const item of items) {
-      const row = deliveredRow(item, messageIdOf(item.runId));
-      // Every dequeued task result is owed one wake, posted in this pass or before.
-      if (row.wake) toWake.push(item.runId);
-      if (posted.has(messageIdOf(item.runId))) continue;
-      session.messages.push(row.message);
-      fresh.push(row);
-    }
+    const { fresh, toWake } = postQueuedRows(session, items, omitted);
     for (const item of items) shownLive.delete(messageIdOf(item.runId));
     if (fresh.length) {
       session.updatedAt = Date.now();
       await saveSession(session);
       for (const { message, notice } of fresh) {
-        try {
-          live()?.notice?.(homeId, { ...notice, messageId: message.meta.transcript.messageId });
-        } catch (error) {
-          log(`live notice in ${homeId} failed: ${error?.message || error}`);
-        }
+        noticeLive(live, log, homeId, { ...notice, messageId: message.meta.transcript.messageId });
       }
     }
     // Dequeuing and owing the wake are one write: no restart between posting
@@ -533,39 +957,7 @@ export function createAgentHomeDelivery({
       { wake: toWake }
     );
     if (overflowed) log(`${homeId} is at its wake cap: ${overflowed} result(s) ride the next wake as a count`);
-    // Owed task results wake the agent with one hidden turn per batch. A batch
-    // is stored with its wake id BEFORE the wake is submitted and dropped once
-    // it is taken, so a restart in between re-checks that exact id; results
-    // owed later form a new batch. A replayed run id is never queued again.
-    const byId = new Map(session.messages.map((message) => [message?.meta?.transcript?.messageId, message]));
-    const taken = new Set(session.messages.flatMap((message) => submissionIdsOf(message)));
-    let batch = null;
-    let woken = [];
-    let more = 0;
-    for (;;) {
-      batch = (await queue.wakes(homeId)).batches[0] || (await queue.batchWake(homeId));
-      if (!batch) return;
-      const stale = taken.has(batch.id);
-      woken = stale ? [] : batch.runIds.map((runId) => byId.get(messageIdOf(runId))).filter(Boolean);
-      // Results past the wake cap ride this wake as a count of rows to read.
-      more = !stale && byId.has(messageIdOf(batch.overflow?.newest)) ? batch.overflow.count : 0;
-      if (woken.length || more) break;
-      // Taken before a restart, or its rows gone with a thread reset.
-      await queue.woke(homeId, batch.id);
-    }
-    const surface = live();
-    if (typeof surface?.wake !== 'function') return later(homeId, busyRetryMs);
-    try {
-      if ((await surface.wake(homeId, agentTaskWakePrompt(woken, { more }), { id: batch.id })) === false) {
-        throw new Error('the wake turn was refused');
-      }
-    } catch (error) {
-      log(`waking ${homeId} failed, retrying: ${error?.message || error}`);
-      return later(homeId, missingRetryMs);
-    }
-    await queue.woke(homeId, batch.id);
-    // Another batch left by an earlier process wakes once this turn is done.
-    if (owesWake(await queue.wakes(homeId))) later(homeId, busyRetryMs);
+    await relay.wakeOwed(homeId, session);
   }
 
   /** The single drainer of `homeId`; a call during a pass runs one more pass. */
@@ -598,17 +990,7 @@ export function createAgentHomeDelivery({
   async function enqueue(row, result) {
     const homeId = agentHomeSessionId(scheduleOwnerAgentId(row));
     if (!result?.runId) throw new Error('routine result: runId required');
-    await queue.add(homeId, {
-      runId: String(result.runId),
-      kind: result.kind === 'webhook' ? 'webhook' : 'schedule',
-      name: row.name,
-      ok: result.ok === true,
-      text: routineSummary(result.text),
-      runSessionId: result.runSessionId || null,
-      ...(result.reason ? { reason: String(result.reason) } : {}),
-      ...(result.paused === true ? { paused: true } : {}),
-      at: Date.now(),
-    });
+    await queue.add(homeId, routineResultItem(row, result));
     void drain(homeId);
     return homeId;
   }
@@ -631,30 +1013,22 @@ export function createAgentHomeDelivery({
   /** Persist one report of a coding task session `agentId` created (session
    *  create) for its home thread, then start draining it. `status` 'started'
    *  and 'blocked' only post a line; a turn's end also wakes the agent. */
-  async function enqueueTask(
-    agentId,
-    { runId, taskSessionId, title, status, text = '', target, watchId = '', files = [], omitted = 0 }
-  ) {
+  async function enqueueTask(agentId, report) {
+    const { runId, taskSessionId, status, text = '', watchId = '', from = '' } = report;
     const homeId = agentHomeSessionId(agentId) || agentHomeSessionId(MIXY_AGENT_ID);
     if (!runId || !taskSessionId) throw new Error('task report: runId and taskSessionId required');
-    await queue.add(
-      homeId,
-      {
-        runId: String(runId),
-        kind: 'agent-task',
-        taskSessionId,
-        title,
-        status,
-        text: routineSummary(text),
-        // Any session other than a task session the agent created.
-        ...(target === 'session' ? { target } : {}),
-        // The files the turn produced (handoff-attachments.mjs collectTurnArtifacts).
-        ...(Array.isArray(files) && files.length ? { files, ...(omitted > 0 ? { omitted } : {}) } : {}),
-        roundTrip: status !== 'started' && status !== 'blocked',
-        at: Date.now(),
-      },
-      { unwatch: watchId ? String(watchId) : '' }
-    );
+    const item = taskReportItem(report);
+    const unwatch = watchId ? String(watchId) : '';
+    const member = roomMemberOfSession(from);
+    if (member?.agentId === agentId) {
+      const roomLogId = roomSessionId(member.roomId);
+      const toRoom = status !== 'started';
+      await queueRoomTaskReport(queue, { from, roomLogId, toRoom }, item, agentId, text, unwatch);
+      void drain(from);
+      if (toRoom) void drain(roomLogId);
+      return toRoom ? roomLogId : from;
+    }
+    await queue.add(homeId, item, { unwatch });
     void drain(homeId);
     return homeId;
   }
@@ -667,23 +1041,10 @@ export function createAgentHomeDelivery({
     const homeId = agentHomeSessionId(scheduleOwnerAgentId(row));
     if (!run?.runId || busy(homeId)) return false;
     const notice = routineRunNotice({ ...run, name: row.name, messageId: messageIdOf(run.runId) });
-    try {
-      return live()?.notice?.(homeId, notice) === true;
-    } catch (error) {
-      log(`live notice in ${homeId} failed: ${error?.message || error}`);
-      return false;
-    }
+    return noticeLive(live, log, homeId, notice) === true;
   }
 
-  /** A bookkept home proven deleted: a bot's home whose definition is gone,
-   *  or a room thread whose room is. Never Mixy's home, never on a missing
-   *  thread record, never on an unknown answer. */
-  function homeDeleted(homeId) {
-    const agentId = agentIdOfHomeSession(homeId);
-    if (agentId) return agentId !== MIXY_AGENT_ID && agentPresence(agentId) === 'deleted';
-    const roomId = roomIdOfSession(homeId) || roomMemberOfSession(homeId)?.roomId || '';
-    return Boolean(roomId) && roomPresence(roomId) === 'deleted';
-  }
+  const homeDeleted = (homeId) => homeProvenDeleted(homeId, agentPresence, roomPresence);
 
   /** Retire the bookkeeping of deleted homes (and stale round counts). */
   async function prune() {
@@ -694,17 +1055,7 @@ export function createAgentHomeDelivery({
    *  whose target sessions are proven deleted, and refuses the watch (so the
    *  send claims no report) when that frees too few. */
   async function watchTask(watch) {
-    const own = (await queue.watches()).filter(
-      (kept) => kept?.homeSessionId === watch.homeSessionId && kept.runId !== watch.runId
-    );
-    const over = own.length + 1 - queue.watchCap;
-    const drop =
-      over > 0
-        ? own
-            .filter((kept) => sessionDeleted(kept.taskSessionId))
-            .slice(0, over)
-            .map((kept) => kept.runId)
-        : [];
+    const drop = watchesToDrop(await queue.watches(), watch, queue.watchCap, sessionDeleted);
     if (drop.length) {
       log(`${watch.homeSessionId} is at its watch cap: dropped ${drop.length} watch(es) of deleted sessions`);
     }
@@ -748,6 +1099,7 @@ async function processDelivery() {
       { createAgentEditorApi },
       { probePath, PROBE_ABSENT },
       { sessionPath },
+      { mixdogRoot },
     ] = await Promise.all([
       import('../runtime/agent/orchestrator/session/store.mjs'),
       import('../runtime/agent/orchestrator/session/manager/runtime-liveness.mjs'),
@@ -755,6 +1107,7 @@ async function processDelivery() {
       import('./workflow-agents-api/agent-editor.mjs'),
       import('../runtime/agent/orchestrator/session/store/fs-probe.mjs'),
       import('../runtime/agent/orchestrator/session/store/session-paths.mjs'),
+      import('../runtime/shared/plugin-paths.mjs'),
     ]);
     // The editor's per-id definition probe: 'deleted' only when proven gone.
     const { agentDefinitionPresence } = createAgentEditorApi({ cfgMod: { getPluginData } });
@@ -764,6 +1117,9 @@ async function processDelivery() {
       turnInFlight: isSessionTurnInFlight,
       queue: createRoutineResultQueue(join(getPluginData(), 'routine-results.json')),
       agentPresence: (agentId) => agentDefinitionPresence([agentId])[agentId],
+      // A home chat exists only for bots; only a definition read successfully
+      // can prove an agent is none (readHomeAgent).
+      homeAgent: (agentId) => readHomeAgent({ dataDir: getPluginData(), shippedRoot: mixdogRoot(), id: agentId }),
       // Deleting a session unlinks its record (session-delete.mjs) and leaves
       // no tombstone: only an absent file with no in-memory copy proves it.
       // An unreadable file probes 'unreadable' and stays.