@songsid/agend 2.1.2-beta.2 → 2.1.2-beta.21

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (147) hide show
  1. package/dist/access-path.js +15 -6
  2. package/dist/access-path.js.map +1 -1
  3. package/dist/agent-endpoint.d.ts +2 -0
  4. package/dist/agent-endpoint.js +17 -1
  5. package/dist/agent-endpoint.js.map +1 -1
  6. package/dist/backend/antigravity.d.ts +18 -0
  7. package/dist/backend/antigravity.js +32 -3
  8. package/dist/backend/antigravity.js.map +1 -1
  9. package/dist/backend/claude-code.d.ts +37 -0
  10. package/dist/backend/claude-code.js +43 -1
  11. package/dist/backend/claude-code.js.map +1 -1
  12. package/dist/backend/codex.d.ts +1 -0
  13. package/dist/backend/codex.js +25 -0
  14. package/dist/backend/codex.js.map +1 -1
  15. package/dist/backend/grok.d.ts +23 -0
  16. package/dist/backend/grok.js +45 -0
  17. package/dist/backend/grok.js.map +1 -1
  18. package/dist/backend/kiro.d.ts +20 -0
  19. package/dist/backend/kiro.js +77 -2
  20. package/dist/backend/kiro.js.map +1 -1
  21. package/dist/backend/types.d.ts +34 -0
  22. package/dist/backend/types.js +1 -1
  23. package/dist/backend/types.js.map +1 -1
  24. package/dist/channel/adapters/discord.d.ts +15 -0
  25. package/dist/channel/adapters/discord.js +91 -2
  26. package/dist/channel/adapters/discord.js.map +1 -1
  27. package/dist/channel/adapters/telegram.d.ts +21 -0
  28. package/dist/channel/adapters/telegram.js +82 -0
  29. package/dist/channel/adapters/telegram.js.map +1 -1
  30. package/dist/channel/ipc-bridge.d.ts +17 -1
  31. package/dist/channel/ipc-bridge.js +53 -17
  32. package/dist/channel/ipc-bridge.js.map +1 -1
  33. package/dist/channel/ipc-timeouts.d.ts +46 -0
  34. package/dist/channel/ipc-timeouts.js +65 -0
  35. package/dist/channel/ipc-timeouts.js.map +1 -0
  36. package/dist/channel/mcp-server.js +30 -14
  37. package/dist/channel/mcp-server.js.map +1 -1
  38. package/dist/channel/mcp-tools.js +28 -1
  39. package/dist/channel/mcp-tools.js.map +1 -1
  40. package/dist/channel/reconnect-backoff.d.ts +17 -0
  41. package/dist/channel/reconnect-backoff.js +21 -0
  42. package/dist/channel/reconnect-backoff.js.map +1 -0
  43. package/dist/channel/types.d.ts +29 -0
  44. package/dist/classic-channel-manager.js +1 -5
  45. package/dist/classic-channel-manager.js.map +1 -1
  46. package/dist/cli.js +132 -52
  47. package/dist/cli.js.map +1 -1
  48. package/dist/completion.d.ts +27 -0
  49. package/dist/completion.js +121 -0
  50. package/dist/completion.js.map +1 -0
  51. package/dist/config-validator.js +9 -0
  52. package/dist/config-validator.js.map +1 -1
  53. package/dist/config.js +1 -0
  54. package/dist/config.js.map +1 -1
  55. package/dist/cost-guard.d.ts +3 -1
  56. package/dist/cost-guard.js +3 -1
  57. package/dist/cost-guard.js.map +1 -1
  58. package/dist/daemon.d.ts +219 -13
  59. package/dist/daemon.js +919 -303
  60. package/dist/daemon.js.map +1 -1
  61. package/dist/event-log.d.ts +31 -0
  62. package/dist/event-log.js +96 -0
  63. package/dist/event-log.js.map +1 -1
  64. package/dist/fleet-manager.d.ts +238 -2
  65. package/dist/fleet-manager.js +1008 -90
  66. package/dist/fleet-manager.js.map +1 -1
  67. package/dist/general-knowledge/skills/cross-instance-messaging/SKILL.md +22 -0
  68. package/dist/general-knowledge/skills/model-discovery/SKILL.md +14 -23
  69. package/dist/general-knowledge/skills/session-management/SKILL.md +15 -34
  70. package/dist/hang-detector.d.ts +19 -13
  71. package/dist/hang-detector.js +19 -49
  72. package/dist/hang-detector.js.map +1 -1
  73. package/dist/instance-lifecycle.d.ts +13 -1
  74. package/dist/instance-lifecycle.js +84 -3
  75. package/dist/instance-lifecycle.js.map +1 -1
  76. package/dist/instructions.d.ts +5 -0
  77. package/dist/instructions.js +9 -11
  78. package/dist/instructions.js.map +1 -1
  79. package/dist/locale.js +2 -1
  80. package/dist/locale.js.map +1 -1
  81. package/dist/logger.js +14 -0
  82. package/dist/logger.js.map +1 -1
  83. package/dist/mcp-liveness.d.ts +21 -0
  84. package/dist/mcp-liveness.js +27 -0
  85. package/dist/mcp-liveness.js.map +1 -0
  86. package/dist/outbound-handlers.d.ts +16 -0
  87. package/dist/outbound-handlers.js +121 -27
  88. package/dist/outbound-handlers.js.map +1 -1
  89. package/dist/outbound-schemas.d.ts +8 -3
  90. package/dist/outbound-schemas.js +13 -2
  91. package/dist/outbound-schemas.js.map +1 -1
  92. package/dist/pane-write-lock.d.ts +48 -0
  93. package/dist/pane-write-lock.js +73 -0
  94. package/dist/pane-write-lock.js.map +1 -0
  95. package/dist/process-memory.d.ts +31 -0
  96. package/dist/process-memory.js +79 -0
  97. package/dist/process-memory.js.map +1 -0
  98. package/dist/quickstart.js +17 -16
  99. package/dist/quickstart.js.map +1 -1
  100. package/dist/reply-dedup.d.ts +41 -0
  101. package/dist/reply-dedup.js +0 -0
  102. package/dist/reply-dedup.js.map +1 -0
  103. package/dist/scheduler/db.js +3 -0
  104. package/dist/scheduler/db.js.map +1 -1
  105. package/dist/sd-notify.d.ts +27 -0
  106. package/dist/sd-notify.js +33 -1
  107. package/dist/sd-notify.js.map +1 -1
  108. package/dist/secret-file.d.ts +33 -0
  109. package/dist/secret-file.js +36 -0
  110. package/dist/secret-file.js.map +1 -0
  111. package/dist/setup-wizard.js +9 -7
  112. package/dist/setup-wizard.js.map +1 -1
  113. package/dist/tmux-control.d.ts +58 -5
  114. package/dist/tmux-control.js +102 -14
  115. package/dist/tmux-control.js.map +1 -1
  116. package/dist/tmux-manager.d.ts +31 -1
  117. package/dist/tmux-manager.js +40 -11
  118. package/dist/tmux-manager.js.map +1 -1
  119. package/dist/topic-commands.d.ts +50 -10
  120. package/dist/topic-commands.js +242 -106
  121. package/dist/topic-commands.js.map +1 -1
  122. package/dist/tui-glyphs.d.ts +25 -0
  123. package/dist/tui-glyphs.js +26 -0
  124. package/dist/tui-glyphs.js.map +1 -0
  125. package/dist/types.d.ts +6 -0
  126. package/dist/ui/view.html +123 -2
  127. package/dist/usage/format-rich.d.ts +11 -0
  128. package/dist/usage/format-rich.js +108 -0
  129. package/dist/usage/format-rich.js.map +1 -0
  130. package/dist/usage/providers.d.ts +97 -0
  131. package/dist/usage/providers.js +834 -0
  132. package/dist/usage/providers.js.map +1 -0
  133. package/dist/usage/usage-api.d.ts +50 -0
  134. package/dist/usage/usage-api.js +111 -0
  135. package/dist/usage/usage-api.js.map +1 -0
  136. package/dist/view-api.d.ts +3 -0
  137. package/dist/view-api.js +11 -2
  138. package/dist/view-api.js.map +1 -1
  139. package/dist/web-api.js +5 -2
  140. package/dist/web-api.js.map +1 -1
  141. package/package.json +4 -1
  142. package/dist/channel/tool-tracker.d.ts +0 -13
  143. package/dist/channel/tool-tracker.js +0 -58
  144. package/dist/channel/tool-tracker.js.map +0 -1
  145. package/dist/daemon-entry.d.ts +0 -1
  146. package/dist/daemon-entry.js +0 -30
  147. package/dist/daemon-entry.js.map +0 -1
package/dist/daemon.js CHANGED
@@ -1,18 +1,23 @@
1
1
  import { join, dirname, basename, resolve } from "node:path";
2
- import { mkdirSync, writeFileSync, readFileSync, existsSync, unlinkSync, rmSync, appendFileSync, statSync, chmodSync, renameSync } from "node:fs";
2
+ import { mkdirSync, writeFileSync, readFileSync, existsSync, unlinkSync, rmSync, appendFileSync, statSync, renameSync } from "node:fs";
3
3
  import { fileURLToPath } from "node:url";
4
4
  import { createHash, randomBytes } from "node:crypto";
5
5
  import { EventEmitter } from "node:events";
6
+ import { rotateLogIfNeeded } from "./logger.js";
7
+ import { mcpServerState } from "./mcp-liveness.js";
6
8
  import { clearPausedMarker, writePausedMarker } from "./pause-marker.js";
7
9
  import { TmuxManager, resolveTmuxLogicalSize } from "./tmux-manager.js";
8
10
  import { TranscriptMonitor } from "./transcript-monitor.js";
9
11
  import { ContextGuardian } from "./context-guardian.js";
10
12
  import { IpcServer } from "./channel/ipc-bridge.js";
13
+ import { daemonBudgetMs } from "./channel/ipc-timeouts.js";
11
14
  import { MessageBus } from "./channel/message-bus.js";
12
15
  import { shellQuote } from "./backend/types.js";
13
16
  import { getTmuxSession } from "./config.js";
14
17
  import { routeToolCall } from "./channel/tool-router.js";
15
18
  import { HangDetector } from "./hang-detector.js";
19
+ import { writeSecretFile } from "./secret-file.js";
20
+ import { PaneWriteLock } from "./pane-write-lock.js";
16
21
  import { buildFleetInstructions } from "./instructions.js";
17
22
  const __filename = fileURLToPath(import.meta.url);
18
23
  const __dirname = dirname(__filename);
@@ -37,9 +42,54 @@ export function buildInstructionReloadNotice(binaryName, instanceName, instanceD
37
42
  export const DEFAULT_STUCK_TIMEOUT_MS = 10 * 60_000;
38
43
  export const DEFAULT_STATE_IDLE_DEBOUNCE_MS = 2_000;
39
44
  export const DEFAULT_STATE_SAFETY_SWEEP_MS = 60_000;
40
- /** @deprecated State detection is event-driven; this now aliases the safety sweep. */
41
- export const DEFAULT_STATE_POLL_INTERVAL_MS = DEFAULT_STATE_SAFETY_SWEEP_MS;
42
45
  const LAST_INBOUND_FILE = "last-inbound-at";
46
+ /** Minimum gap between "health check is failing" notifications for one instance. */
47
+ const HEALTH_ERROR_NOTIFY_INTERVAL_MS = 10 * 60_000;
48
+ /**
49
+ * Whether two working directories belong to the same project, so a fleet-scoped
50
+ * decision recorded in one reaches the other. Covers the worktree/checkout
51
+ * layout AgEnD fleets actually use — `AgEnD`, `AgEnD-dev1`, `AgEnD-dev2`,
52
+ * `AgEnD-reviewer`, `AgEnD-main` are one project; `DouPo_Server` is not.
53
+ * Nesting counts too (a subdirectory of a project belongs to it).
54
+ */
55
+ export function sameProjectFamily(a, b) {
56
+ const norm = (p) => p.replace(/\/+$/, "");
57
+ const x = norm(a), y = norm(b);
58
+ if (x === y)
59
+ return true;
60
+ if (x.startsWith(y + "/") || y.startsWith(x + "/"))
61
+ return true;
62
+ // Strip a trailing worktree/role suffix: "AgEnD-dev2" → "agend".
63
+ const family = (p) => {
64
+ const base = p.slice(p.lastIndexOf("/") + 1).toLowerCase();
65
+ return base.replace(/[-_](dev\d*|main|reviewer|tester|leader|sol|codex|worktree|wt\d*)$/i, "");
66
+ };
67
+ const fx = family(x), fy = family(y);
68
+ return fx.length > 0 && fx === fy;
69
+ }
70
+ /**
71
+ * Pick the decisions worth injecting into one instance's system prompt.
72
+ *
73
+ * `scope: "fleet"` previously bypassed the project check outright, so every
74
+ * instance carried every other project's playbook — measured at 8 of 14 here,
75
+ * resent on every API call. A fleet decision must now also be relevant: global
76
+ * (no project_root), the same project, or the same project family.
77
+ *
78
+ * `isDispatcher` (a general) opts out of that narrowing on purpose: it routes
79
+ * work across all projects, so cross-project rules ("X 文件操作由 Y 負責") are
80
+ * precisely what it must know. Narrowing a general would cause misrouting.
81
+ */
82
+ export function selectRelevantDecisions(all, workDir, isDispatcher = false) {
83
+ return all.filter(d => {
84
+ if (d.project_root === workDir)
85
+ return true; // own project, any scope
86
+ if (d.scope !== "fleet")
87
+ return false; // project-scoped, elsewhere
88
+ if (!d.project_root)
89
+ return true; // truly global
90
+ return isDispatcher || sameProjectFamily(d.project_root, workDir);
91
+ });
92
+ }
43
93
  /** Read the last real channel inbound timestamp persisted across daemon restarts. */
44
94
  export function readLastInboundAt(instanceDir, now = Date.now()) {
45
95
  try {
@@ -58,6 +108,40 @@ export function writeLastInboundAt(instanceDir, timestamp) {
58
108
  writeFileSync(temp, String(timestamp));
59
109
  renameSync(temp, target);
60
110
  }
111
+ /**
112
+ * Render the handoff/routing metadata that rides along with an inbound message
113
+ * into the block actually pasted into the agent's pane.
114
+ *
115
+ * These fields were all populated by the sender (outbound-handlers builds them
116
+ * into `ipcMeta`) and delivered over IPC, but never rendered — so the receiving
117
+ * agent could not see them, and several documented flows could not work:
118
+ *
119
+ * - `report_result` requires a `correlation_id` the agent had no way to know, so
120
+ * the "correlation_id not recognized" warning fired on essentially every call.
121
+ * - Delegation cancel buttons are retired by correlation id, so they never retired.
122
+ * - `react`, `edit_message` and `reply.reply_to` need `message_id`, which the tool
123
+ * descriptions tell the agent to take "from the inbound block".
124
+ * - `download_attachment` needs `attachment_file_id`.
125
+ * - `requires_reply` was invisible, so a delegated task looked like an FYI.
126
+ *
127
+ * Only non-empty fields are emitted, so a plain user message gains at most a
128
+ * message_id line.
129
+ */
130
+ export function renderHandoffMetadata(meta) {
131
+ const rows = [];
132
+ const add = (label, value) => {
133
+ if (value && value.trim())
134
+ rows.push(`${label}: ${value.trim()}`);
135
+ };
136
+ add("message_id", meta.message_id);
137
+ add("correlation_id", meta.correlation_id);
138
+ add("request_kind", meta.request_kind);
139
+ add("task_summary", meta.task_summary);
140
+ add("working_directory", meta.working_directory);
141
+ add("branch", meta.branch);
142
+ add("attachment_file_id", meta.attachment_file_id);
143
+ return rows.length ? `\n(${rows.join(" | ")})` : "";
144
+ }
61
145
  /** Headless inactivity timer used by the daemon and unit tests. */
62
146
  export class AutoPauseController {
63
147
  thresholdMs;
@@ -102,24 +186,47 @@ export class AutoPauseController {
102
186
  * Pane motion wins over a ready match because several backends keep their ready
103
187
  * marker in a persistent header/footer while generating. A stable ready pane is
104
188
  * idle; changing content is working; stable non-ready content eventually sticks.
189
+ *
190
+ * ## Why an optional busy pattern exists
191
+ *
192
+ * "Motion wins" degrades badly when a backend's ready marker is *always* on
193
+ * screen. `stuck` requires `!ready`, so for such a backend the state can only ever
194
+ * be idle or working — the stuck edge, and with it `handleStuckTransition` and the
195
+ * hang notification, is unreachable. A frozen CLI is then reported as idle, which
196
+ * also clears pending work: the message the user sent is quietly booked as done.
197
+ *
198
+ * claude-code was exactly this case (see ClaudeCodeBackend.getReadyPattern). A
199
+ * backend that can point at a marker meaning "generating right now" supplies
200
+ * `getBusyPattern()`, and that match vetoes ready regardless of what the ready
201
+ * pattern says.
105
202
  */
106
203
  export class PaneStateMachine {
107
204
  stuckTimeoutMs;
108
205
  readyPattern;
206
+ busyPattern;
109
207
  lastPaneHash = null;
110
208
  lastPaneChangeAt;
111
209
  lastObservedAt;
112
210
  stateChangedAt;
113
211
  currentState = "idle";
114
- constructor(readyPattern, stuckTimeoutMs = DEFAULT_STUCK_TIMEOUT_MS, now = Date.now()) {
212
+ constructor(readyPattern, stuckTimeoutMs = DEFAULT_STUCK_TIMEOUT_MS, now = Date.now(), busyPattern) {
115
213
  this.stuckTimeoutMs = stuckTimeoutMs;
116
214
  // Stateful g/y regexes mutate lastIndex and can alternate true/false across
117
215
  // polls. State detection must be deterministic for identical pane content.
118
216
  this.readyPattern = new RegExp(readyPattern.source, readyPattern.flags.replace(/[gy]/g, ""));
217
+ this.busyPattern = busyPattern
218
+ ? new RegExp(busyPattern.source, busyPattern.flags.replace(/[gy]/g, ""))
219
+ : null;
119
220
  this.lastPaneChangeAt = now;
120
221
  this.lastObservedAt = now;
121
222
  this.stateChangedAt = now;
122
223
  }
224
+ /** Ready, unless the backend can positively see that it is still generating. */
225
+ isReady(pane) {
226
+ if (this.busyPattern?.test(pane))
227
+ return false;
228
+ return this.readyPattern.test(pane);
229
+ }
123
230
  observe(pane, now = Date.now()) {
124
231
  const paneHash = createHash("sha256").update(pane).digest("hex");
125
232
  const firstObservation = this.lastPaneHash === null;
@@ -129,7 +236,7 @@ export class PaneStateMachine {
129
236
  this.lastPaneChangeAt = now;
130
237
  }
131
238
  this.lastObservedAt = now;
132
- const ready = this.readyPattern.test(pane);
239
+ const ready = this.isReady(pane);
133
240
  const nextState = firstObservation
134
241
  ? ready ? "idle" : "working"
135
242
  : paneChanged
@@ -193,6 +300,23 @@ export class PendingWorkTracker {
193
300
  const NORMAL_ENTER_SETTLE_MS = 500;
194
301
  const FIRST_ENTER_SETTLE_MS = 1_750;
195
302
  const FIRST_DELIVERY_WINDOW_MS = 5_000;
303
+ /** After busy native-queue paste+Enter, wait before checking the pane for silent loss. */
304
+ const NATIVE_QUEUE_PASTE_VERIFY_MS = 2_000;
305
+ /** Enter-confirmation poll: 10 × 200ms ≈ 2s of observed silence before giving up. */
306
+ const CONFIRM_BUSY_POLLS = 10;
307
+ const CONFIRM_BUSY_POLL_MS = 200;
308
+ /**
309
+ * Ceiling on the whole confirmation, including time re-earned after a control-mode
310
+ * reconnect. Without it a reconnect loop could restart the poll indefinitely and
311
+ * hold the pane write lock along with it.
312
+ */
313
+ const CONFIRM_BUSY_MAX_WAIT_MS = 10_000;
314
+ /**
315
+ * How long a delivery waits for an in-flight spawn. Comfortably past the default
316
+ * 25s startup timeout plus dialog dismissal; past it we fall back to the old
317
+ * behaviour rather than holding a message indefinitely.
318
+ */
319
+ const SPAWN_SETTLE_MAX_WAIT_MS = 60_000;
196
320
  /**
197
321
  * One-shot timing gate for the first paste after a CLI reaches its ready
198
322
  * prompt. Recent tmux versions expose a short interval where the prompt is
@@ -244,13 +368,13 @@ export class Daemon extends EventEmitter {
244
368
  topicMode;
245
369
  backend;
246
370
  controlClient;
371
+ runtimeIdentity;
247
372
  logger;
248
373
  tmuxSessionName;
249
374
  tmux = null;
250
375
  ipcServer = null;
251
376
  messageBus;
252
377
  transcriptMonitor = null;
253
- toolTracker = null;
254
378
  guardian = null;
255
379
  adapter = null;
256
380
  pendingIpcRequests = new Map();
@@ -261,6 +385,8 @@ export class Daemon extends EventEmitter {
261
385
  // Pending ack: react 🫡 on first transcript activity after receiving a message
262
386
  pendingAckMessage = null;
263
387
  // Tool status tracking for channel adapter
388
+ /** Last activity published to the fleet manager; null when nothing is running. */
389
+ currentActivity = null;
264
390
  toolStatusMessageId = null;
265
391
  toolStatusLines = [];
266
392
  toolStatusDebounce = null;
@@ -276,9 +402,19 @@ export class Daemon extends EventEmitter {
276
402
  lastSpawnAt = 0;
277
403
  crashTimestamps = [];
278
404
  healthCheckPaused = false;
405
+ lastHealthErrorNotifyAt = 0;
279
406
  /** CLI pane availability, independent from the daemon process and tri-state. */
280
407
  processStatus = "running";
281
408
  spawning = false;
409
+ /**
410
+ * Resolves when the in-flight spawn finishes; null when none is running.
411
+ *
412
+ * `spawning` alone can only be polled. Delivery needs to *wait*, and a boolean
413
+ * poll would either busy-loop or race the flag being cleared.
414
+ */
415
+ spawnSettled = null;
416
+ resolveSpawnSettled = null;
417
+ spawnDepth = 0;
282
418
  skipResume = false;
283
419
  backgroundSessionRecoveryAttempted = false;
284
420
  /** Whether the last spawn started a fresh session (not resumed). */
@@ -301,10 +437,18 @@ export class Daemon extends EventEmitter {
301
437
  instanceStateIdleDebounceMs = DEFAULT_STATE_IDLE_DEBOUNCE_MS;
302
438
  instanceStateStuckTimeoutMs = DEFAULT_STUCK_TIMEOUT_MS;
303
439
  instanceStateReadyPattern = null;
440
+ instanceStateBusyPattern = null;
304
441
  instanceStateMonitorActive = false;
305
442
  statePollInFlight = false;
306
443
  autoPauseController;
307
444
  pauseRequested = false;
445
+ /**
446
+ * Sticky "pause as soon as possible" (auth failure). Unlike pauseRequested —
447
+ * which the state monitor clears whenever the pane is not idle — this survives
448
+ * busy/stuck states, because an auth error fires mid-turn and a plain pause()
449
+ * would silently no-op exactly when we most need to stop feeding the CLI.
450
+ */
451
+ pausePending = false;
308
452
  pauseWakeState = "active";
309
453
  pauseWakeTransition = null;
310
454
  // Model failover: override model on next spawn when rate-limited
@@ -314,7 +458,13 @@ export class Daemon extends EventEmitter {
314
458
  recentEvents = [];
315
459
  recentToolActivity = [];
316
460
  snapshotConsumed = false;
461
+ /** Orders inbound channel messages against each other (queue-depth accounting
462
+ * and ⏳/👀/✅ reactions live here). It does NOT cover the other writers that
463
+ * reach the pane — {@link paneWriteLock} does. */
317
464
  pasteLock = Promise.resolve();
465
+ /** Mutual exclusion for *every* write into the pane, whichever subsystem it
466
+ * comes from. See PaneWriteLock for why interleaving is destructive. */
467
+ paneWriteLock = new PaneWriteLock();
318
468
  pendingInstructionsUpdate;
319
469
  pendingInstructionsNotice = false;
320
470
  // Whether the warmup steering-reload notice should be injected after spawn.
@@ -327,6 +477,10 @@ export class Daemon extends EventEmitter {
327
477
  firstDeliveryDelay = new FirstDeliveryDelay();
328
478
  // PTY error pattern monitoring
329
479
  errorMonitorTimer = null;
480
+ /** Same 5-min gate the error monitor uses, so a dead MCP server alerts once. */
481
+ static MCP_DEATH_COOLDOWN_MS = 5 * 60_000;
482
+ lastMcpDeathNotifiedAt = 0;
483
+ mcpDeathNotifiedForPid = null;
330
484
  /** Prevent in-flight monitor callbacks from re-arming after a pause. */
331
485
  runtimeMonitorsFrozen = false;
332
486
  errorWaitingForRecovery = false; // true = error detected, waiting for ready pattern
@@ -371,7 +525,7 @@ export class Daemon extends EventEmitter {
371
525
  this.errorRecoveryDeadlineAt = 0;
372
526
  this.activeErrorPatternKey = null;
373
527
  }
374
- constructor(name, config, instanceDir, topicMode = false, backend, controlClient, rootLogger) {
528
+ constructor(name, config, instanceDir, topicMode = false, backend, controlClient, rootLogger, runtimeIdentity) {
375
529
  super();
376
530
  this.name = name;
377
531
  this.config = config;
@@ -379,8 +533,14 @@ export class Daemon extends EventEmitter {
379
533
  this.topicMode = topicMode;
380
534
  this.backend = backend;
381
535
  this.controlClient = controlClient;
536
+ this.runtimeIdentity = runtimeIdentity;
382
537
  if (!rootLogger)
383
538
  throw new Error("Daemon requires a shared root logger");
539
+ this.runtimeIdentity ??= {
540
+ kind: "fleet-topic",
541
+ backend: config.backend ?? backend?.binaryName ?? "unknown",
542
+ model: config.model ?? "default",
543
+ };
384
544
  this.logger = rootLogger.child({ instance: name }, { level: config.log_level });
385
545
  this.tmuxSessionName = getTmuxSession();
386
546
  this.messageBus = new MessageBus();
@@ -447,9 +607,6 @@ export class Daemon extends EventEmitter {
447
607
  if ((type === "fleet_schedule_response" || type === "fleet_outbound_response" || type === "fleet_decision_response" || type === "fleet_task_response" || type === "fleet_display_name_response" || type === "fleet_description_response") && msg.fleetRequestId) {
448
608
  key = String(msg.fleetRequestId);
449
609
  }
450
- else if (type === "fleet_outbound_response" && msg.requestId != null) {
451
- key = `fleet_out_${msg.requestId}`;
452
- }
453
610
  if (key && this.pendingIpcRequests.has(key)) {
454
611
  const handler = this.pendingIpcRequests.get(key);
455
612
  this.pendingIpcRequests.delete(key);
@@ -600,7 +757,12 @@ export class Daemon extends EventEmitter {
600
757
  else {
601
758
  await new Promise(r => setTimeout(r, 5000));
602
759
  }
603
- await this.tmux?.pasteText(buildInstructionReloadNotice(this.backend?.binaryName ?? "unknown", this.name, this.instanceDir));
760
+ // This path only runs when pasteQueueDepth > 0 — i.e. exactly when a real
761
+ // delivery is already in flight or queued. Without the lock the notice and
762
+ // that delivery race into the same pane.
763
+ await this.paneWriteLock.run(async () => {
764
+ await this.tmux?.pasteText(buildInstructionReloadNotice(this.backend?.binaryName ?? "unknown", this.name, this.instanceDir), this.systemPasteOptions());
765
+ });
604
766
  // Record the value the agent has now been told about so the next
605
767
  // unchanged restart skips the reload.
606
768
  try {
@@ -612,8 +774,12 @@ export class Daemon extends EventEmitter {
612
774
  catch { /* non-fatal */ }
613
775
  })();
614
776
  if (!this.config.lightweight) {
615
- // 3. Pipe-pane for prompt detection
777
+ // 3. Pipe-pane for prompt detection. Rotate first so a ballooned log from a
778
+ // previous stuck splash (hundreds of MB of ANSI frames) is truncated before
779
+ // we attach — pipe-pane uses `cat >>` on the same inode, so copytruncate
780
+ // keeps the writer attached after size resets.
616
781
  const outputLog = join(this.instanceDir, "output.log");
782
+ rotateLogIfNeeded(outputLog);
617
783
  await this.tmux.pipeOutput(outputLog).catch(() => { });
618
784
  // 4. Transcript monitor
619
785
  this.transcriptMonitor = new TranscriptMonitor(this.instanceDir, this.logger);
@@ -629,27 +795,31 @@ export class Daemon extends EventEmitter {
629
795
  this.transcriptMonitor.on("tool_use", (name, input) => {
630
796
  this.logger.debug({ tool: name }, "Tool use");
631
797
  ackIfPending();
632
- this.hangDetector?.recordActivity();
633
798
  this.recordRecentEvent({ type: "tool_use", name, preview: this.summarizeTool(name, input) });
634
799
  this.recordRecentToolActivity(this.summarizeTool(name, input));
800
+ this.publishActivity(this.summarizeTool(name, input));
635
801
  });
636
802
  this.transcriptMonitor.on("tool_result", (name, _output) => {
637
- this.hangDetector?.recordActivity();
638
803
  this.recordRecentEvent({ type: "tool_result", name });
804
+ // The tool finished; whatever comes next has not started yet. Better to
805
+ // show only elapsed time than to leave a stale "Bash: npm test" on screen.
806
+ this.publishActivity(null);
639
807
  });
640
808
  this.transcriptMonitor.on("assistant_text", (text) => {
641
809
  this.logger.debug({ text: text.slice(0, 200) }, "Claude response");
642
810
  ackIfPending();
643
- this.hangDetector?.recordActivity();
644
811
  this.recordRecentEvent({ type: "assistant_text", preview: text.slice(0, 100) });
645
812
  });
646
813
  this.transcriptMonitor.startPolling();
647
- // HangDetector remains the fleet-manager notification event bridge. Its
648
- // legacy silence timer is intentionally not started: pane state transitions
649
- // below are now the sole source of hang events.
814
+ // HangDetector is the notification bridge to the fleet manager; pane state
815
+ // transitions below are the sole source of hang events (see hang-detector.ts).
816
+ // `hang_detector.enabled: false` opts out of the notification entirely.
817
+ // NOTE: `hang_detector.timeout_minutes` is NOT read here — the stuck timeout
818
+ // the pane monitor actually uses is resolved separately below. The two have
819
+ // long been separate; documented rather than silently ignored.
650
820
  const hangConfig = this.config.hang_detector;
651
821
  if (hangConfig?.enabled !== false) {
652
- this.hangDetector = new HangDetector(hangConfig?.timeout_minutes ?? 10);
822
+ this.hangDetector = new HangDetector();
653
823
  }
654
824
  // 8. Context guardian
655
825
  const statusFile = join(this.instanceDir, "statusline.json");
@@ -657,7 +827,6 @@ export class Daemon extends EventEmitter {
657
827
  this.guardian.startWatching();
658
828
  this.guardian.on("status_update", () => {
659
829
  this.saveSessionId();
660
- this.hangDetector?.recordStatuslineUpdate();
661
830
  });
662
831
  // Context rotation removed: all CLI backends have built-in auto-compact.
663
832
  // Crash recovery (health check + respawn with snapshot) is retained below.
@@ -677,6 +846,40 @@ export class Daemon extends EventEmitter {
677
846
  this.startInstanceStateMonitor();
678
847
  this.logger.info(`${this.name} ready`);
679
848
  }
849
+ /**
850
+ * Detect a CRASHED MCP server and report it once.
851
+ *
852
+ * The MCP server writes its pid to channel.mcp.pid at startup and unlinks it on
853
+ * a clean exit, so "file present + pid dead" specifically means it died without
854
+ * cleaning up (crash / OOM / SIGKILL). A missing file is either "not started
855
+ * yet" or "exited cleanly" — neither is an incident, and an orphan exit implies
856
+ * the CLI itself died, which the crash detector already covers.
857
+ *
858
+ * The daemon deliberately does NOT try to respawn it: MCP is a stdio protocol
859
+ * where the CLI spawns the server and owns its pipes, so a daemon-spawned
860
+ * process would have no client reading it. Only the CLI can restore its own
861
+ * tools — hence notify, and let the operator decide about restarting.
862
+ */
863
+ checkMcpServerAlive() {
864
+ if (this.isPaused)
865
+ return;
866
+ const status = mcpServerState(this.instanceDir);
867
+ if (status.state === "unknown")
868
+ return;
869
+ if (status.state === "alive") {
870
+ this.mcpDeathNotifiedForPid = null; // the CLI respawned it — re-arm
871
+ return;
872
+ }
873
+ // Dead: report once per pid, and at most once per cooldown window.
874
+ if (this.mcpDeathNotifiedForPid === status.pid)
875
+ return;
876
+ if (Date.now() - this.lastMcpDeathNotifiedAt < Daemon.MCP_DEATH_COOLDOWN_MS)
877
+ return;
878
+ this.mcpDeathNotifiedForPid = status.pid;
879
+ this.lastMcpDeathNotifiedAt = Date.now();
880
+ this.logger.error({ pid: status.pid }, "MCP server process is gone — instance has no agend tools");
881
+ this.emit("mcp_died", { name: this.name, pid: status.pid });
882
+ }
680
883
  startHealthCheck() {
681
884
  if (this.runtimeMonitorsFrozen || this.healthCheckTimer)
682
885
  return;
@@ -692,249 +895,285 @@ export class Daemon extends EventEmitter {
692
895
  this.healthCheckTimer = null;
693
896
  if (this.runtimeMonitorsFrozen)
694
897
  return;
695
- // Instance directory removed externally (e.g. `rm -rf ~/.agend/instances/<name>`).
696
- // Stop the loop permanently otherwise every tick triggers a respawn, whose
697
- // writeRotationSnapshot fails with ENOENT and gets caught as "Failed to respawn",
698
- // spamming errors every ~30s forever.
699
- if (!existsSync(this.instanceDir)) {
700
- this.logger.warn({ instanceDir: this.instanceDir }, "Instance directory missing — stopping health check");
701
- this.healthCheckPaused = true;
702
- this.healthCheckTimer = null;
703
- return;
704
- }
705
- if (!this.tmux || this.spawning || this.healthCheckPaused || Daemon.tmuxServerPaused) {
706
- scheduleNext();
707
- return;
708
- }
709
- // Human-readable backend label for logs (e.g. "claude", "kiro-cli")
710
- const cliLabel = this.backend?.binaryName ?? "CLI";
711
- let paneStatus = await this.tmux.getPaneStatus();
712
- // Auto-pause intentionally exits the pane process. A health tick that
713
- // began just before pause must not classify that exit as a crash.
714
- if (this.isPaused || this.pauseWakeState === "waking") {
715
- scheduleNext();
716
- return;
717
- }
718
- if (paneStatus?.alive) {
719
- scheduleNext();
720
- return;
721
- }
722
- // A null status is ambiguous: it can be a transient `tmux list-panes`
723
- // failure (e.g. tmux busy during a fleet-restart storm) rather than a
724
- // real exit. Re-confirm once after a short delay before treating it as
725
- // a crash. A non-null {alive:false} is a definite dead pane (real exit)
726
- // and needs no recheck.
727
- if (paneStatus === null) {
728
- await new Promise(r => setTimeout(r, 1500));
729
- paneStatus = await this.tmux.getPaneStatus();
898
+ // The whole tick is guarded: an unguarded throw in here (a tmux hiccup,
899
+ // ENOSPC on the crash-history write) became an unhandled rejection, which
900
+ // the CLI turned into stopAll() + exit(1) for the ENTIRE fleet, and it also
901
+ // ended this instance's health loop because scheduleNext() sat after the
902
+ // throwing code. Catch, log, and keep checking.
903
+ try {
904
+ // Instance directory removed externally (e.g. `rm -rf ~/.agend/instances/<name>`).
905
+ // Stop the loop permanently — otherwise every tick triggers a respawn, whose
906
+ // writeRotationSnapshot fails with ENOENT and gets caught as "Failed to respawn",
907
+ // spamming errors every ~30s forever.
908
+ if (!existsSync(this.instanceDir)) {
909
+ this.logger.warn({ instanceDir: this.instanceDir }, "Instance directory missing — stopping health check");
910
+ this.healthCheckPaused = true;
911
+ this.healthCheckTimer = null;
912
+ this.emitSupervisionEnded("instance directory was removed from disk", "Recreate the instance, or delete it from fleet.yaml.");
913
+ return;
914
+ }
915
+ if (!this.tmux || this.spawning || this.healthCheckPaused || Daemon.tmuxServerPaused) {
916
+ scheduleNext();
917
+ return;
918
+ }
919
+ // The CLI owns the MCP server process, so the daemon can only observe it.
920
+ this.checkMcpServerAlive();
921
+ // Human-readable backend label for logs (e.g. "claude", "kiro-cli")
922
+ const cliLabel = this.backend?.binaryName ?? "CLI";
923
+ let paneStatus = await this.tmux.getPaneStatus();
924
+ // Auto-pause intentionally exits the pane process. A health tick that
925
+ // began just before pause must not classify that exit as a crash.
926
+ if (this.isPaused || this.pauseWakeState === "waking") {
927
+ scheduleNext();
928
+ return;
929
+ }
730
930
  if (paneStatus?.alive) {
731
- this.logger.debug(`[health] ${cliLabel} pane reported gone then alive on recheck transient query failure, ignoring`);
931
+ // Instance output.log is fed by tmux pipe-pane and was previously never
932
+ // rotated (only fleet.log / daemon.log were). Cap growth every tick.
933
+ if (!this.config.lightweight) {
934
+ rotateLogIfNeeded(join(this.instanceDir, "output.log"));
935
+ }
732
936
  scheduleNext();
733
937
  return;
734
938
  }
735
- }
736
- // paneStatus === null → window gone entirely (e.g. tmux server crash)
737
- // paneStatus.alive === false pane dead, exit code available
738
- const exitCode = paneStatus?.exitCode;
739
- this.logger.debug({ exitCode }, `[health] pane exited with code: ${exitCode}`);
740
- // Normal exit (e.g. user Ctrl+C or /exit) — no crash, no respawn
741
- if (paneStatus && exitCode === 0) {
742
- this.setProcessStatus("stopped");
743
- this.logger.info("CLI exited normally (code 0) — pausing health check");
744
- await this.tmux.killWindow();
745
- this.healthCheckPaused = true;
746
- return;
747
- }
748
- this.setProcessStatus("crashed");
749
- // Distinguish tmux server crash from single window crash.
750
- // nullReason records *why* getPaneStatus returned null (for diagnosing
751
- // whether this was a real window loss or a transient query failure).
752
- let crashType = "window";
753
- let nullReason;
754
- if (!paneStatus) {
755
- const serverAlive = await TmuxManager.sessionExists(this.tmuxSessionName);
756
- if (!serverAlive) {
757
- crashType = "server";
758
- nullReason = "server_gone";
759
- this.logger.error(`tmux server died — all ${cliLabel} windows lost`);
760
- // Fleet-level circuit breaker: pause all instances on repeated tmux server crashes
761
- Daemon.tmuxServerCrashTimestamps.push(Date.now());
762
- const cutoff = Date.now() - 5 * 60_000;
763
- Daemon.tmuxServerCrashTimestamps = Daemon.tmuxServerCrashTimestamps.filter(t => t > cutoff);
764
- if (Daemon.tmuxServerCrashTimestamps.length >= 2 && !Daemon.tmuxServerPaused) {
765
- Daemon.tmuxServerPaused = true;
766
- this.logger.error("Fleet-level tmux server circuit breaker triggered — pausing all respawns for 30s");
767
- this.emit("tmux_server_crash", this.name);
768
- if (!Daemon.tmuxServerRecoveryTimer) {
769
- Daemon.tmuxServerRecoveryTimer = setTimeout(() => {
770
- Daemon.tmuxServerRecoveryTimer = null;
771
- Daemon.tmuxServerPaused = false;
772
- }, 30_000);
773
- }
939
+ // A null status is ambiguous: it can be a transient `tmux list-panes`
940
+ // failure (e.g. tmux busy during a fleet-restart storm) rather than a
941
+ // real exit. Re-confirm once after a short delay before treating it as
942
+ // a crash. A non-null {alive:false} is a definite dead pane (real exit)
943
+ // and needs no recheck.
944
+ if (paneStatus === null) {
945
+ await new Promise(r => setTimeout(r, 1500));
946
+ paneStatus = await this.tmux.getPaneStatus();
947
+ if (paneStatus?.alive) {
948
+ this.logger.debug(`[health] ${cliLabel} pane reported gone then alive on recheck — transient query failure, ignoring`);
774
949
  scheduleNext();
775
950
  return;
776
951
  }
777
- await new Promise(r => setTimeout(r, 2_000)); // let session stabilize
778
952
  }
779
- else {
780
- // null but server alive: window-level disappearance. Probe whether
781
- // the window truly no longer exists vs a transient query glitch.
782
- nullReason = "no_window";
783
- try {
784
- const windows = await TmuxManager.listWindows(this.tmuxSessionName);
785
- if (windows.some(w => w.name === this.name))
786
- nullReason = "window_present_query_glitch";
953
+ // paneStatus === null → window gone entirely (e.g. tmux server crash)
954
+ // paneStatus.alive === false pane dead, exit code available
955
+ const exitCode = paneStatus?.exitCode;
956
+ this.logger.debug({ exitCode }, `[health] pane exited with code: ${exitCode}`);
957
+ // Normal exit (e.g. user Ctrl+C or /exit) — no crash, no respawn
958
+ if (paneStatus && exitCode === 0) {
959
+ this.setProcessStatus("stopped");
960
+ this.logger.info("CLI exited normally (code 0) — pausing health check");
961
+ await this.tmux.killWindow();
962
+ this.healthCheckPaused = true;
963
+ this.emitSupervisionEnded("the CLI exited normally (code 0)", "Nothing crashed — start it again when you need it.");
964
+ return;
965
+ }
966
+ this.setProcessStatus("crashed");
967
+ // Distinguish tmux server crash from single window crash.
968
+ // nullReason records *why* getPaneStatus returned null (for diagnosing
969
+ // whether this was a real window loss or a transient query failure).
970
+ let crashType = "window";
971
+ let nullReason;
972
+ if (!paneStatus) {
973
+ const serverAlive = await TmuxManager.sessionExists(this.tmuxSessionName);
974
+ if (!serverAlive) {
975
+ crashType = "server";
976
+ nullReason = "server_gone";
977
+ this.logger.error(`tmux server died — all ${cliLabel} windows lost`);
978
+ // Fleet-level circuit breaker: pause all instances on repeated tmux server crashes
979
+ Daemon.tmuxServerCrashTimestamps.push(Date.now());
980
+ const cutoff = Date.now() - 5 * 60_000;
981
+ Daemon.tmuxServerCrashTimestamps = Daemon.tmuxServerCrashTimestamps.filter(t => t > cutoff);
982
+ if (Daemon.tmuxServerCrashTimestamps.length >= 2 && !Daemon.tmuxServerPaused) {
983
+ Daemon.tmuxServerPaused = true;
984
+ this.logger.error("Fleet-level tmux server circuit breaker triggered — pausing all respawns for 30s");
985
+ this.emit("tmux_server_crash", this.name);
986
+ if (!Daemon.tmuxServerRecoveryTimer) {
987
+ Daemon.tmuxServerRecoveryTimer = setTimeout(() => {
988
+ Daemon.tmuxServerRecoveryTimer = null;
989
+ Daemon.tmuxServerPaused = false;
990
+ }, 30_000);
991
+ }
992
+ scheduleNext();
993
+ return;
994
+ }
995
+ await new Promise(r => setTimeout(r, 2_000)); // let session stabilize
787
996
  }
788
- catch {
789
- nullReason = "query_error";
997
+ else {
998
+ // null but server alive: window-level disappearance. Probe whether
999
+ // the window truly no longer exists vs a transient query glitch.
1000
+ nullReason = "no_window";
1001
+ try {
1002
+ const windows = await TmuxManager.listWindows(this.tmuxSessionName);
1003
+ if (windows.some(w => w.name === this.name))
1004
+ nullReason = "window_present_query_glitch";
1005
+ }
1006
+ catch {
1007
+ nullReason = "query_error";
1008
+ }
1009
+ this.logger.warn({ exitCode, nullReason }, `${cliLabel} window not found (tmux server alive)`);
790
1010
  }
791
- this.logger.warn({ exitCode, nullReason }, `${cliLabel} window not found (tmux server alive)`);
792
1011
  }
793
- }
794
- else {
795
- this.logger.warn({ exitCode }, `${cliLabel} process exited`);
796
- }
797
- // Capture last output before killing. Best-effort even when the pane is
798
- // gone (paneStatus null) gives the crash record something to diagnose
799
- // from instead of an empty lastOutput.
800
- let lastOutput;
801
- try {
802
- const raw = await this.tmux.capturePaneWithHistory(50);
803
- // Strip ANSI escape codes for readability
804
- const cleaned = raw.replace(/\x1b\[[0-9;]*[a-zA-Z]/g, "");
805
- lastOutput = cleaned.trimEnd() || undefined;
806
- }
807
- catch { /* best effort pane may already be gone */ }
808
- // Kill the dead window (remain-on-exit keeps it around) before respawn
809
- if (paneStatus) {
810
- await this.tmux.killWindow();
811
- }
812
- // Detect claude-code background session conflict recover without counting as crash
813
- if (lastOutput && (lastOutput.includes("background agent") || lastOutput.includes("Session is currently running"))) {
814
- if (!this.backgroundSessionRecoveryAttempted) {
815
- this.backgroundSessionRecoveryAttempted = true;
816
- this.logger.warn("Detected lingering background agent session — starting fresh (no resume)");
817
- const sidFile = join(this.instanceDir, "session-id");
1012
+ else {
1013
+ this.logger.warn({ exitCode }, `${cliLabel} process exited`);
1014
+ }
1015
+ // Capture last output before killing. Best-effort even when the pane is
1016
+ // gone (paneStatus null) gives the crash record something to diagnose
1017
+ // from instead of an empty lastOutput.
1018
+ let lastOutput;
1019
+ try {
1020
+ const raw = await this.tmux.capturePaneWithHistory(50);
1021
+ // Strip ANSI escape codes for readability
1022
+ const cleaned = raw.replace(/\x1b\[[0-9;]*[a-zA-Z]/g, "");
1023
+ lastOutput = cleaned.trimEnd() || undefined;
1024
+ }
1025
+ catch { /* best effort — pane may already be gone */ }
1026
+ // Kill the dead window (remain-on-exit keeps it around) before respawn
1027
+ if (paneStatus) {
1028
+ await this.tmux.killWindow();
1029
+ }
1030
+ // Detect claude-code background session conflict — recover without counting as crash
1031
+ if (lastOutput && (lastOutput.includes("background agent") || lastOutput.includes("Session is currently running"))) {
1032
+ if (!this.backgroundSessionRecoveryAttempted) {
1033
+ this.backgroundSessionRecoveryAttempted = true;
1034
+ this.logger.warn("Detected lingering background agent session — starting fresh (no resume)");
1035
+ const sidFile = join(this.instanceDir, "session-id");
1036
+ try {
1037
+ unlinkSync(sidFile);
1038
+ }
1039
+ catch { }
1040
+ this.skipResume = true;
1041
+ await new Promise(r => setTimeout(r, 2_000));
1042
+ try {
1043
+ await this.spawnClaudeWindow();
1044
+ this.setProcessStatus("running");
1045
+ this.logger.info("Recovered from background session conflict");
1046
+ this.emit("crash_respawn", this.name);
1047
+ }
1048
+ catch (err) {
1049
+ this.logger.error({ err: err.message }, "Recovery from background session conflict failed");
1050
+ }
1051
+ return; // Don't count as crash
1052
+ }
1053
+ // Already attempted recovery — fall through to normal crash handling
1054
+ }
1055
+ // Detect a --continue/--resume failure (no conversation to resume). The
1056
+ // session-id file persists across the crash, so a blind respawn would add
1057
+ // --continue again and crash in the same way → loop. Clear the session id
1058
+ // and skip resume so the next spawn starts fresh. (skipResume also stops
1059
+ // saveSessionId below from resurrecting the id from statusline.json.)
1060
+ if (lastOutput && /no conversation found|no conversation to (continue|resume)|no previous (session|conversation)|--continue/i.test(lastOutput)) {
1061
+ this.logger.warn("Detected --continue/resume failure — clearing session-id; next spawn starts fresh");
818
1062
  try {
819
- unlinkSync(sidFile);
1063
+ unlinkSync(join(this.instanceDir, "session-id"));
820
1064
  }
821
- catch { }
1065
+ catch { /* may not exist */ }
822
1066
  this.skipResume = true;
823
- await new Promise(r => setTimeout(r, 2_000));
1067
+ }
1068
+ // Append to crash history
1069
+ this.appendCrashHistory({ exitCode, lastOutput, crashType, reason: nullReason });
1070
+ if (max_retries <= 0) {
1071
+ this.healthCheckPaused = true;
1072
+ this.logger.warn(`${cliLabel} window died — automatic restart is disabled`);
1073
+ this.emitSupervisionEnded("the CLI died and automatic restart is disabled (restart_policy.max_retries is 0)", "Start it manually, or raise max_retries in fleet.yaml.");
1074
+ return;
1075
+ }
1076
+ // Detect rapid crash: sliding window — 3+ crashes in 5 minutes
1077
+ this.crashTimestamps.push(Date.now());
1078
+ const crashWindowMs = 5 * 60_000;
1079
+ this.crashTimestamps = this.crashTimestamps.filter(t => t > Date.now() - crashWindowMs);
1080
+ if (this.crashTimestamps.length >= 3) {
1081
+ this.healthCheckPaused = true;
1082
+ this.logger.error({ crashesInWindow: this.crashTimestamps.length }, "3+ crashes in 5 minutes — pausing respawn");
1083
+ // P1: Persist crash state so next process restart skips resume
824
1084
  try {
825
- await this.spawnClaudeWindow();
826
- this.setProcessStatus("running");
827
- this.logger.info("Recovered from background session conflict");
828
- this.emit("crash_respawn", this.name);
829
- }
830
- catch (err) {
831
- this.logger.error({ err: err.message }, "Recovery from background session conflict failed");
1085
+ writeFileSync(join(this.instanceDir, "crash-state.json"), JSON.stringify({
1086
+ crashesInWindow: this.crashTimestamps.length,
1087
+ lastCrashAt: Date.now(),
1088
+ resumeDisabled: true,
1089
+ }));
832
1090
  }
833
- return; // Don't count as crash
1091
+ catch { /* best effort */ }
1092
+ this.emit("crash_loop", this.name);
1093
+ return; // don't schedule next — paused
834
1094
  }
835
- // Already attempted recovery fall through to normal crash handling
836
- }
837
- // Detect a --continue/--resume failure (no conversation to resume). The
838
- // session-id file persists across the crash, so a blind respawn would add
839
- // --continue again and crash in the same way → loop. Clear the session id
840
- // and skip resume so the next spawn starts fresh. (skipResume also stops
841
- // saveSessionId below from resurrecting the id from statusline.json.)
842
- if (lastOutput && /no conversation found|no conversation to (continue|resume)|no previous (session|conversation)|--continue/i.test(lastOutput)) {
843
- this.logger.warn("Detected --continue/resume failure — clearing session-id; next spawn starts fresh");
844
- try {
845
- unlinkSync(join(this.instanceDir, "session-id"));
1095
+ // Reset crash count if enough time has passed
1096
+ if (reset_after > 0 && Date.now() - this.lastCrashAt > reset_after) {
1097
+ this.crashCount = 0;
846
1098
  }
847
- catch { /* may not exist */ }
848
- this.skipResume = true;
849
- }
850
- // Append to crash history
851
- this.appendCrashHistory({ exitCode, lastOutput, crashType, reason: nullReason });
852
- if (max_retries <= 0) {
853
- this.healthCheckPaused = true;
854
- this.logger.warn(`${cliLabel} window died — automatic restart is disabled`);
855
- return;
856
- }
857
- // Detect rapid crash: sliding window — 3+ crashes in 5 minutes
858
- this.crashTimestamps.push(Date.now());
859
- const crashWindowMs = 5 * 60_000;
860
- this.crashTimestamps = this.crashTimestamps.filter(t => t > Date.now() - crashWindowMs);
861
- if (this.crashTimestamps.length >= 3) {
862
- this.healthCheckPaused = true;
863
- this.logger.error({ crashesInWindow: this.crashTimestamps.length }, "3+ crashes in 5 minutes — pausing respawn");
864
- // P1: Persist crash state so next process restart skips resume
865
- try {
866
- writeFileSync(join(this.instanceDir, "crash-state.json"), JSON.stringify({
867
- crashesInWindow: this.crashTimestamps.length,
868
- lastCrashAt: Date.now(),
869
- resumeDisabled: true,
870
- }));
1099
+ this.crashCount++;
1100
+ this.lastCrashAt = Date.now();
1101
+ if (this.crashCount > max_retries) {
1102
+ this.logger.error({ crashCount: this.crashCount, maxRetries: max_retries }, "Max crash retries exceeded — not respawning");
1103
+ this.healthCheckPaused = true;
1104
+ this.emitSupervisionEnded(`it crashed ${this.crashCount} times, exceeding restart_policy.max_retries (${max_retries})`, "Check the logs for the cause, then restart it.");
1105
+ return; // don't schedule next — given up
871
1106
  }
872
- catch { /* best effort */ }
873
- this.emit("crash_loop", this.name);
874
- return; // don't schedule next paused
875
- }
876
- // Reset crash count if enough time has passed
877
- if (reset_after > 0 && Date.now() - this.lastCrashAt > reset_after) {
878
- this.crashCount = 0;
879
- }
880
- this.crashCount++;
881
- this.lastCrashAt = Date.now();
882
- if (this.crashCount > max_retries) {
883
- this.logger.error({ crashCount: this.crashCount, maxRetries: max_retries }, "Max crash retries exceeded — not respawning");
884
- return; // don't schedule next — given up
885
- }
886
- // Calculate backoff delay
887
- const delay = backoff === "exponential"
888
- ? Math.min(1000 * Math.pow(2, this.crashCount - 1), 60_000)
889
- : 1000 * this.crashCount;
890
- this.logger.warn({ crashCount: this.crashCount, delay }, `${cliLabel} window died — respawning after backoff`);
891
- await new Promise(r => setTimeout(r, delay));
892
- try {
893
- this.saveSessionId();
894
- this.transcriptMonitor?.resetOffset();
895
- // Kill orphan MCP server from the crashed CLI session.
896
- // MCP server writes its PID to channel.mcp.pid on startup.
897
- try {
898
- const pidFile = join(this.instanceDir, "channel.mcp.pid");
899
- const pid = parseInt(readFileSync(pidFile, "utf-8").trim(), 10);
900
- process.kill(pid, "SIGTERM");
901
- this.logger.info({ pid }, "Killed orphan MCP server");
902
- }
903
- catch { /* no pid file or process already dead */ }
904
- // Kill any same-name windows before respawn to prevent orphans.
905
- // Wrapped in try-catch: if tmux server is dead, listWindows throws —
906
- // must not block spawnClaudeWindow (which calls ensureSession).
1107
+ // Calculate backoff delay
1108
+ const delay = backoff === "exponential"
1109
+ ? Math.min(1000 * Math.pow(2, this.crashCount - 1), 60_000)
1110
+ : 1000 * this.crashCount;
1111
+ this.logger.warn({ crashCount: this.crashCount, delay }, `${cliLabel} window died — respawning after backoff`);
1112
+ await new Promise(r => setTimeout(r, delay));
907
1113
  try {
908
- const windows = await TmuxManager.listWindows(this.tmuxSessionName);
909
- for (const w of windows) {
910
- if (w.name === this.name) {
911
- const tm = new TmuxManager(this.tmuxSessionName, w.id);
912
- await tm.killWindow();
913
- }
1114
+ this.saveSessionId();
1115
+ this.transcriptMonitor?.resetOffset();
1116
+ // Kill orphan MCP server from the crashed CLI session.
1117
+ // MCP server writes its PID to channel.mcp.pid on startup.
1118
+ try {
1119
+ const pidFile = join(this.instanceDir, "channel.mcp.pid");
1120
+ const pid = parseInt(readFileSync(pidFile, "utf-8").trim(), 10);
1121
+ process.kill(pid, "SIGTERM");
1122
+ this.logger.info({ pid }, "Killed orphan MCP server");
914
1123
  }
915
- }
916
- catch { /* tmux server may be dead ensureSession in trySpawn will recover */ }
917
- // Write snapshot before spawn consumed only if resume fails
918
- this.writeRotationSnapshot("crash");
919
- // Try --resume first; spawnClaudeWindow falls back to fresh session if resume fails
920
- const resumed = await this.spawnClaudeWindow();
921
- if (!resumed) {
922
- // Resume failed → fresh session → inject snapshot for context
923
- await this.injectSnapshotMessage();
924
- }
925
- else {
926
- // Clean up stale snapshot — resume restored full context
1124
+ catch { /* no pid file or process already dead */ }
1125
+ // Kill any same-name windows before respawn to prevent orphans.
1126
+ // Wrapped in try-catch: if tmux server is dead, listWindows throws —
1127
+ // must not block spawnClaudeWindow (which calls ensureSession).
927
1128
  try {
928
- unlinkSync(join(this.instanceDir, "rotation-state.json"));
1129
+ const windows = await TmuxManager.listWindows(this.tmuxSessionName);
1130
+ for (const w of windows) {
1131
+ if (w.name === this.name) {
1132
+ const tm = new TmuxManager(this.tmuxSessionName, w.id);
1133
+ await tm.killWindow();
1134
+ }
1135
+ }
929
1136
  }
930
- catch { /* may not exist */ }
1137
+ catch { /* tmux server may be dead — ensureSession in trySpawn will recover */ }
1138
+ // Write snapshot before spawn — consumed only if resume fails
1139
+ this.writeRotationSnapshot("crash");
1140
+ // Try --resume first; spawnClaudeWindow falls back to fresh session if resume fails
1141
+ const resumed = await this.spawnClaudeWindow();
1142
+ if (!resumed) {
1143
+ // Resume failed → fresh session → inject snapshot for context
1144
+ await this.injectSnapshotMessage();
1145
+ }
1146
+ else {
1147
+ // Clean up stale snapshot — resume restored full context
1148
+ try {
1149
+ unlinkSync(join(this.instanceDir, "rotation-state.json"));
1150
+ }
1151
+ catch { /* may not exist */ }
1152
+ }
1153
+ this.setProcessStatus("running");
1154
+ this.logger.info({ resumed }, `Respawned ${cliLabel} window after crash`);
1155
+ this.emit("crash_respawn", this.name);
1156
+ }
1157
+ catch (err) {
1158
+ this.logger.error({ err }, `Failed to respawn ${cliLabel} window`);
931
1159
  }
932
- this.setProcessStatus("running");
933
- this.logger.info({ resumed }, `Respawned ${cliLabel} window after crash`);
934
- this.emit("crash_respawn", this.name);
935
1160
  }
936
1161
  catch (err) {
937
- this.logger.error({ err }, `Failed to respawn ${cliLabel} window`);
1162
+ this.logger.error({ err }, "Health check tick failed — continuing");
1163
+ // Surface it to the operator, not just the log — a health check that
1164
+ // keeps throwing means this instance is no longer being supervised.
1165
+ // Throttled: the tick repeats every ~30s, so an unnotified persistent
1166
+ // fault would otherwise post twice a minute forever.
1167
+ const now = Date.now();
1168
+ if (now - this.lastHealthErrorNotifyAt > HEALTH_ERROR_NOTIFY_INTERVAL_MS) {
1169
+ this.lastHealthErrorNotifyAt = now;
1170
+ this.emit("health_check_error", {
1171
+ name: this.name,
1172
+ message: err instanceof Error ? err.message : String(err),
1173
+ });
1174
+ }
1175
+ scheduleNext();
1176
+ return;
938
1177
  }
939
1178
  scheduleNext();
940
1179
  }, healthCheckIntervalMs);
@@ -946,6 +1185,21 @@ export class Daemon extends EventEmitter {
946
1185
  * A dead remain-on-exit pane still contains the old ready prompt, so allowing
947
1186
  * the pane monitor to capture it would re-create a false idle state.
948
1187
  */
1188
+ /**
1189
+ * Announce that this instance is no longer being supervised.
1190
+ *
1191
+ * Four health-check exits set `healthCheckPaused = true` and returned without
1192
+ * telling anyone: a clean CLI exit, `max_retries <= 0`, crash retries exhausted,
1193
+ * and the instance directory being deleted. Only the crash-LOOP case emitted an
1194
+ * event, so a fleet could quietly contain a dead instance that still looked fine
1195
+ * on the dashboard — while messages routed to it queued or failed with a bare ❌.
1196
+ *
1197
+ * `crash_loop` already had this treatment; this is the same bridge for the other
1198
+ * four, carrying a human-readable cause and the operator's next step.
1199
+ */
1200
+ emitSupervisionEnded(reason, remedy) {
1201
+ this.emit("supervision_ended", { name: this.name, reason, remedy });
1202
+ }
949
1203
  setProcessStatus(status) {
950
1204
  if (this.processStatus === status)
951
1205
  return;
@@ -991,6 +1245,7 @@ export class Daemon extends EventEmitter {
991
1245
  if (!this.backend)
992
1246
  return; // lightweight mode has no backend
993
1247
  const readyPattern = this.backend.getReadyPattern();
1248
+ const busyPattern = this.backend.getBusyPattern?.() ?? null;
994
1249
  this.errorMonitorTimer = setInterval(async () => {
995
1250
  if (!this.tmux || this.spawning)
996
1251
  return;
@@ -1003,20 +1258,30 @@ export class Daemon extends EventEmitter {
1003
1258
  for (const dialog of dialogs) {
1004
1259
  if (!dialog.pattern.test(pane))
1005
1260
  continue;
1006
- this.logger.info(`Auto-dismissing runtime dialog: ${dialog.description}`);
1007
- const SPECIAL_KEYS = new Set(["Up", "Down", "Enter", "Escape", "Right", "Left"]);
1008
- for (const key of dialog.keys) {
1009
- if (SPECIAL_KEYS.has(key)) {
1010
- await this.tmux.sendSpecialKey(key);
1011
- }
1012
- else {
1013
- await this.tmux.pasteText(key);
1261
+ // These keys go straight into the pane. Sent while a message delivery is
1262
+ // mid-transaction, an `Escape` wipes the pasted text and an `Enter`
1263
+ // submits it half-composed — the user sees a message that vanished. Skip
1264
+ // (not queue) when the pane is busy: this poller runs every 5s, and the
1265
+ // dialog will still be on screen next tick.
1266
+ const dismissed = await this.paneWriteLock.tryRun(async () => {
1267
+ this.logger.info(`Auto-dismissing runtime dialog: ${dialog.description}`);
1268
+ const SPECIAL_KEYS = new Set(["Up", "Down", "Enter", "Escape", "Right", "Left"]);
1269
+ for (const key of dialog.keys) {
1270
+ if (SPECIAL_KEYS.has(key)) {
1271
+ await this.tmux.sendSpecialKey(key);
1272
+ }
1273
+ else {
1274
+ await this.tmux.pasteText(key, this.systemPasteOptions());
1275
+ }
1276
+ await new Promise(r => setTimeout(r, 200));
1014
1277
  }
1015
- await new Promise(r => setTimeout(r, 200));
1278
+ });
1279
+ if (!dismissed) {
1280
+ this.logger.debug({ dialog: dialog.description }, "Dialog dismissal deferred — pane write in flight");
1016
1281
  }
1017
- return; // Dialog dismissed, skip error checks this cycle
1282
+ return; // Dialog handled (or deliberately deferred): skip error checks this cycle
1018
1283
  }
1019
- this.evaluateErrorPatterns(pane, patterns, readyPattern);
1284
+ this.evaluateErrorPatterns(pane, patterns, readyPattern, Date.now(), busyPattern);
1020
1285
  }
1021
1286
  catch {
1022
1287
  // capturePane can fail if window is transitioning — ignore
@@ -1024,7 +1289,7 @@ export class Daemon extends EventEmitter {
1024
1289
  }, 5_000); // Check every 5 seconds (runtime dialogs need fast response)
1025
1290
  }
1026
1291
  /** Evaluate one pane snapshot. Kept synchronous so state-machine edges are unit-testable. */
1027
- evaluateErrorPatterns(pane, patterns, readyPattern, now = Date.now()) {
1292
+ evaluateErrorPatterns(pane, patterns, readyPattern, now = Date.now(), busyPattern = null) {
1028
1293
  // Count occurrences across the WHOLE pane (not just text after the last
1029
1294
  // ready prompt). Clone with `g` so stateful backend regexes cannot leak
1030
1295
  // lastIndex between monitor cycles.
@@ -1032,10 +1297,15 @@ export class Daemon extends EventEmitter {
1032
1297
  const flags = pattern.flags.includes("g") ? pattern.flags : pattern.flags + "g";
1033
1298
  return (pane.match(new RegExp(pattern.source, flags)) || []).length;
1034
1299
  };
1300
+ // Same veto as the state machine: a backend whose ready marker is permanently
1301
+ // on screen would otherwise "recover" on the very first tick after the error,
1302
+ // rebaselining the occurrence count while the error is still displayed — one
1303
+ // notification, a false recovery log, then silence.
1304
+ const looksReady = () => !busyPattern?.test(pane) && readyPattern.test(pane);
1035
1305
  // State: waiting for recovery. A missing/outdated ready pattern must not
1036
1306
  // suppress every future error forever, so the gate has a hard deadline.
1037
1307
  if (this.errorWaitingForRecovery) {
1038
- if (readyPattern.test(pane)) {
1308
+ if (looksReady()) {
1039
1309
  const downtime = Math.round((now - this.errorDetectedAt) / 1000);
1040
1310
  const active = patterns.find(ep => Daemon.errorPatternKey(ep) === this.activeErrorPatternKey);
1041
1311
  if (active) {
@@ -1134,6 +1404,12 @@ export class Daemon extends EventEmitter {
1134
1404
  * Interrupt the CLI's current generation (cancel button / `/cancel`).
1135
1405
  * Direct tmux key event (not a paste) so it registers as the interrupt key.
1136
1406
  * kiro-cli interrupts on Ctrl+C; the others (claude-code, codex, …) on Escape.
1407
+ *
1408
+ * Deliberately NOT taken through `paneWriteLock`. Cancel is the user's way out
1409
+ * of a pane that is busy or wedged — queueing it behind the very delivery chain
1410
+ * it exists to unblock would make the button useless exactly when it is needed.
1411
+ * The cost is accepted: an Escape landing between a paste and its Enter discards
1412
+ * that message, which is what the user asked for anyway.
1137
1413
  */
1138
1414
  async sendEscape() {
1139
1415
  const cancelKey = this.backend?.getCancelKey() ?? "Escape";
@@ -1171,7 +1447,6 @@ export class Daemon extends EventEmitter {
1171
1447
  this.toolStatusDebounce = null;
1172
1448
  }
1173
1449
  this.pendingIpcRequests.clear();
1174
- this.hangDetector?.stop();
1175
1450
  if (this.adapter)
1176
1451
  await this.adapter.stop();
1177
1452
  // Notify MCP servers of graceful shutdown (prevents reconnect attempts)
@@ -1198,7 +1473,12 @@ export class Daemon extends EventEmitter {
1198
1473
  if (!killed)
1199
1474
  this.logger.warn("CLI did not exit gracefully within 3s, force killing window");
1200
1475
  // Always kill window — remain-on-exit keeps dead panes around after CLI exits
1476
+ const stoppedWindowId = this.tmux.getWindowId();
1201
1477
  await this.tmux.killWindow();
1478
+ // The control client is shared across the fleet and outlives this daemon, so
1479
+ // a registration left behind here is re-resolved on every reconnect forever.
1480
+ if (stoppedWindowId)
1481
+ this.controlClient?.unregisterWindow(stoppedWindowId);
1202
1482
  const windowIdFile = join(this.instanceDir, "window-id");
1203
1483
  try {
1204
1484
  unlinkSync(windowIdFile);
@@ -1241,6 +1521,16 @@ export class Daemon extends EventEmitter {
1241
1521
  };
1242
1522
  }
1243
1523
  /** Gracefully stop the CLI while keeping its remain-on-exit tmux window. */
1524
+ /**
1525
+ * Mark the instance to pause as soon as its pane is idle. Used for auth
1526
+ * failures: pause() alone no-ops while the CLI is busy/stuck, which is the
1527
+ * usual state when the error surfaces. Cleared by a successful pause or wake.
1528
+ */
1529
+ requestPauseWhenIdle() {
1530
+ if (this.pauseWakeState === "paused")
1531
+ return;
1532
+ this.pausePending = true;
1533
+ }
1244
1534
  async pause() {
1245
1535
  if (this.pauseWakeState === "paused")
1246
1536
  return;
@@ -1310,6 +1600,9 @@ export class Daemon extends EventEmitter {
1310
1600
  }
1311
1601
  /** Respawn the CLI in the preserved window and block until its prompt is ready. */
1312
1602
  async wake(timeoutMs = 30_000) {
1603
+ // An explicit wake (e.g. after the user re-logs in) cancels a deferred
1604
+ // auth pause — otherwise the instance would pause again the moment it idles.
1605
+ this.pausePending = false;
1313
1606
  if (this.pauseWakeState === "active")
1314
1607
  return;
1315
1608
  if (this.pauseWakeState === "pausing")
@@ -1321,7 +1614,7 @@ export class Daemon extends EventEmitter {
1321
1614
  return;
1322
1615
  }
1323
1616
  this.pauseWakeState = "waking";
1324
- this.spawning = true;
1617
+ this.beginSpawn();
1325
1618
  const transition = this.autoPauseController.wakeOnDeliver(async () => {
1326
1619
  let timeout;
1327
1620
  try {
@@ -1362,7 +1655,7 @@ export class Daemon extends EventEmitter {
1362
1655
  throw err;
1363
1656
  }
1364
1657
  finally {
1365
- this.spawning = false;
1658
+ this.endSpawn();
1366
1659
  if (this.pauseWakeTransition === transition)
1367
1660
  this.pauseWakeTransition = null;
1368
1661
  }
@@ -1374,6 +1667,10 @@ export class Daemon extends EventEmitter {
1374
1667
  // observations between enqueue and paste must not clear a newer inbound.
1375
1668
  if (snapshot.state === "idle" && previous !== "idle") {
1376
1669
  this.pendingWork.recordIdle(snapshot.observedAt);
1670
+ // The turn is over. A transcript can end on a tool_use with no matching
1671
+ // tool_result (interrupted, crashed, cancelled), which would otherwise leave
1672
+ // the last tool pinned to the progress line for the rest of the session.
1673
+ this.publishActivity(null);
1377
1674
  }
1378
1675
  if (snapshot.state !== previous) {
1379
1676
  this.logger.info({
@@ -1389,6 +1686,12 @@ export class Daemon extends EventEmitter {
1389
1686
  }
1390
1687
  if (snapshot.state !== "idle")
1391
1688
  this.pauseRequested = false;
1689
+ // A pause deferred by an auth failure: retry the moment the pane settles.
1690
+ if (this.pausePending && snapshot.state === "idle" && this.pasteQueueDepth === 0) {
1691
+ this.pausePending = false;
1692
+ this.emit("auto_pause_requested", { name: this.name, idleSince: snapshot.stateChangedAt });
1693
+ return;
1694
+ }
1392
1695
  if (!this.pauseRequested && this.pasteQueueDepth === 0 && this.autoPauseController.observe(snapshot.state)) {
1393
1696
  this.pauseRequested = true;
1394
1697
  this.emit("auto_pause_requested", { name: this.name, idleSince: snapshot.stateChangedAt });
@@ -1458,6 +1761,13 @@ export class Daemon extends EventEmitter {
1458
1761
  this.instanceStateMachine.observe(pane, observedChangeAt);
1459
1762
  const snapshot = this.instanceStateMachine.observe(pane, Date.now());
1460
1763
  this.applyInstanceStateSnapshot(snapshot, pane);
1764
+ // Backends without a transcript feed can still say what they are doing, if
1765
+ // their TUI names it. Free-riding on a capture that already happened: no
1766
+ // extra tmux call. Cadence is the capture cadence (idle debounce + the 60s
1767
+ // safety sweep), which matches the progress ticker's own 60s interval.
1768
+ if (this.backend?.getPaneActivity) {
1769
+ this.publishActivity(snapshot.state === "idle" ? null : this.backend.getPaneActivity(pane));
1770
+ }
1461
1771
  if (snapshot.state === "idle") {
1462
1772
  this.clearInstanceStateStuckTimer();
1463
1773
  }
@@ -1512,7 +1822,8 @@ export class Daemon extends EventEmitter {
1512
1822
  ? rawConfig.idle_debounce_ms
1513
1823
  : DEFAULT_STATE_IDLE_DEBOUNCE_MS;
1514
1824
  this.instanceStateReadyPattern = this.backend.getReadyPattern();
1515
- this.instanceStateMachine = new PaneStateMachine(this.instanceStateReadyPattern, this.instanceStateStuckTimeoutMs);
1825
+ this.instanceStateBusyPattern = this.backend.getBusyPattern?.() ?? null;
1826
+ this.instanceStateMachine = new PaneStateMachine(this.instanceStateReadyPattern, this.instanceStateStuckTimeoutMs, Date.now(), this.instanceStateBusyPattern);
1516
1827
  this.instanceStateLastOutputAt = 0;
1517
1828
  this.instanceStateMonitorActive = true;
1518
1829
  if (this.controlClient) {
@@ -1553,6 +1864,12 @@ export class Daemon extends EventEmitter {
1553
1864
  paneTail: sanitizePaneTail(pane),
1554
1865
  readyPattern: readyPattern.toString(),
1555
1866
  readyMatched: deterministicReadyPattern.test(pane),
1867
+ // A pane that is stuck *while* the ready marker matches means the busy
1868
+ // pattern is what held it back — worth seeing when tuning either regex.
1869
+ busyPattern: this.instanceStateBusyPattern?.toString(),
1870
+ busyMatched: this.instanceStateBusyPattern
1871
+ ? new RegExp(this.instanceStateBusyPattern.source, this.instanceStateBusyPattern.flags.replace(/[gy]/g, "")).test(pane)
1872
+ : undefined,
1556
1873
  unchangedForMs: snapshot.unchangedForMs,
1557
1874
  pendingWork: this.pendingWork.hasPendingWork(),
1558
1875
  };
@@ -1595,6 +1912,40 @@ export class Daemon extends EventEmitter {
1595
1912
  return this.messageBus;
1596
1913
  }
1597
1914
  // ── Tool status tracking ──────────────────────────────────────
1915
+ /**
1916
+ * Tell the fleet manager what this instance is doing right now, for the live
1917
+ * progress line on the cancel button.
1918
+ *
1919
+ * Purely cosmetic — nothing decides anything from it, so it is fine that only
1920
+ * backends with a transcript feed report at all (claude-code today). For the
1921
+ * rest the progress line keeps showing just elapsed time.
1922
+ *
1923
+ * Repeats are dropped: the ticker only edits the channel message when the text
1924
+ * changes, and a stream of identical broadcasts would defeat that.
1925
+ */
1926
+ publishActivity(activity) {
1927
+ const next = activity && activity.trim() ? activity.trim() : null;
1928
+ if (next === this.currentActivity)
1929
+ return;
1930
+ this.currentActivity = next;
1931
+ this.ipcServer?.broadcast({
1932
+ type: "instance_activity",
1933
+ instanceName: this.name,
1934
+ activity: next,
1935
+ });
1936
+ }
1937
+ /**
1938
+ * Options for every system-initiated paste (startup notice, session snapshot,
1939
+ * runtime-dialog keys).
1940
+ *
1941
+ * One place, so the three call sites cannot drift apart on it. Backends with a
1942
+ * native input queue do not get the retry Enter: on those, a second bare Enter
1943
+ * is not the no-op the retry assumed but a queue mutation — the same reason
1944
+ * `deliverMessage` refuses to probe for busy with one.
1945
+ */
1946
+ systemPasteOptions() {
1947
+ return { retryEnter: this.backend?.supportsQueuedInput?.() !== true };
1948
+ }
1598
1949
  summarizeTool(name, input) {
1599
1950
  const inp = input;
1600
1951
  if (!inp)
@@ -1684,7 +2035,6 @@ export class Daemon extends EventEmitter {
1684
2035
  writeFileSync(join(this.instanceDir, "prev-instructions"), this.pendingInstructionsUpdate);
1685
2036
  this.pendingInstructionsUpdate = undefined;
1686
2037
  }
1687
- this.hangDetector?.recordInbound();
1688
2038
  this.pendingWork.recordInbound();
1689
2039
  // v3: record user messages for rotation snapshot
1690
2040
  this.recordRecentUserMessage(content, meta);
@@ -1707,12 +2057,26 @@ export class Daemon extends EventEmitter {
1707
2057
  // #77: show the sender's display name for readability, keeping the machine
1708
2058
  // instance name in parens so the recipient's send_to_instance target is valid.
1709
2059
  const fromLabel = meta.from_display ? `${meta.from_display} (${fromInstance})` : fromInstance;
1710
- formatted = `[from:${fromLabel}] ${content}\n(If you need to reply, use send_to_instance tool, NOT direct text. If there is nothing to add, you may stay silent.)`;
2060
+ formatted = `[from:${fromLabel}] ${content}`;
2061
+ formatted += renderHandoffMetadata(meta);
2062
+ // A delegated task that requires a reply must not read like a chatty FYI —
2063
+ // the "you may stay silent" line is for the latter only.
2064
+ formatted += meta.requires_reply === "true"
2065
+ ? "\n(A reply IS required: use report_result with the correlation_id above — or send_to_instance. Not direct text.)"
2066
+ : "\n(If you need to reply, use send_to_instance tool, NOT direct text. If there is nothing to add, you may stay silent.)";
1711
2067
  }
1712
2068
  else {
1713
2069
  const via = meta.source ? ` via ${meta.source}` : "";
1714
2070
  const idTag = meta.user_id ? `, id:${meta.user_id}` : "";
1715
- formatted = `[user:${user}${via}${idTag}] ${content}\n(Reply using the reply tool — do NOT respond with direct text)`;
2071
+ formatted = `[user:${user}${via}${idTag}] ${content}`;
2072
+ // Reactions queued since the last real message (#432). One leading line of
2073
+ // context, present only when something is pending — a reaction no longer
2074
+ // costs a turn of its own.
2075
+ if (meta.pending_reactions) {
2076
+ formatted = `[Recent reactions: ${meta.pending_reactions}]\n${formatted}`;
2077
+ }
2078
+ formatted += renderHandoffMetadata(meta);
2079
+ formatted += "\n(Reply using the reply tool — do NOT respond with direct text)";
1716
2080
  }
1717
2081
  if (meta.reply_to_text) {
1718
2082
  formatted += `\n(reply_to: "${meta.reply_to_text}")`;
@@ -1755,18 +2119,19 @@ export class Daemon extends EventEmitter {
1755
2119
  }
1756
2120
  /**
1757
2121
  * Deliver a single message and drive its status reactions:
1758
- * ⏳ message_queued — CLI busy; queued, waiting for idle
2122
+ * ⏳ message_queued — CLI busy; queued locally or by the backend
1759
2123
  * 👀 message_delivered — pasted + Enter sent; agent now has it
1760
- * ✅ message_confirmed — idle→busy transition observed; agent is processing
2124
+ * ✅ message_confirmed — processing observed, or native queue handoff succeeded
1761
2125
  * ❌ message_failed — tmux window gone, paste retries exhausted
1762
2126
  * Returns true once the message is in the CLI, false only on real delivery failure.
1763
2127
  *
1764
2128
  * Bug A (silent message loss): paste failures retry with backoff (window recovery)
1765
2129
  * and emit `message_failed` if all attempts fail.
1766
- * Busy handling (UX): we never force-paste into a busy CLI and never give up on a
1767
- * busy one — we show ⏳ and wait for idle indefinitely (a genuinely hung CLI is the
1768
- * hang detector's job; a user who can't wait presses Cancel → Escape → idle). The
1769
- * pasteLock is serial, so later messages naturally queue behind this wait.
2130
+ * Busy handling (UX): backends without a native input queue show and wait for
2131
+ * idle indefinitely (a genuinely hung CLI is the hang detector's job). Backends
2132
+ * that explicitly support queued input receive one complete paste+Enter transaction
2133
+ * immediately and own the application-level queue themselves. The pasteLock remains
2134
+ * serial in both cases so separate PTY writes can never overlap.
1770
2135
  */
1771
2136
  async deliverMessage(formatted, status) {
1772
2137
  // Sanitize unclosed code fences — they cause CLI to wait for closure on Enter
@@ -1775,14 +2140,50 @@ export class Daemon extends EventEmitter {
1775
2140
  // Odd number of fences = unclosed. Remove all code fences from the message.
1776
2141
  formatted = formatted.replace(/```/g, "");
1777
2142
  }
2143
+ // Before anything reads the window id: a spawn in progress is about to change it.
2144
+ await this.waitForSpawnToSettle();
1778
2145
  let windowId = this.getWindowId();
1779
- // If the CLI is busy, show ⏳ and wait for it to go idle — no timeout, no force.
2146
+ const supportsQueuedInput = this.backend?.supportsQueuedInput?.() === true;
2147
+ let handingOffToNativeQueue = false;
2148
+ // If the CLI is busy, either hand the complete submission to its native input
2149
+ // queue or wait for idle. Native queue support is an explicit backend capability:
2150
+ // normal Enter input has steering/interrupt semantics in several other CLIs.
1780
2151
  if (windowId && this.controlClient && !this.controlClient.isIdle(windowId)) {
1781
2152
  if (status)
1782
2153
  this.emit("message_queued", status);
1783
- this.logger.debug("CLI busy — queuing message until idle");
1784
- await this.controlClient.waitUntilIdle(windowId);
2154
+ if (supportsQueuedInput) {
2155
+ handingOffToNativeQueue = true;
2156
+ this.logger.debug("CLI busy — handing message to backend-native input queue");
2157
+ }
2158
+ else {
2159
+ this.logger.debug("CLI busy — queuing message until idle");
2160
+ const becameIdle = await this.controlClient.waitUntilIdle(windowId);
2161
+ if (!becameIdle) {
2162
+ // The pane never freed up. Report the failure instead of pasting into a
2163
+ // wedged CLI (where the text would sit unsubmitted and the next message
2164
+ // would land on top of it) — and instead of holding the queue silently.
2165
+ this.logger.error("Pane still busy after the idle wait — reporting delivery failure");
2166
+ if (status)
2167
+ this.emit("message_failed", status); // ❌
2168
+ return false;
2169
+ }
2170
+ }
1785
2171
  }
2172
+ // Everything above is *waiting*; everything below *writes*. Only the write is
2173
+ // held under the pane lock — holding it across the idle wait (up to 30 min)
2174
+ // would starve the runtime-dialog dismisser, which is often the very thing
2175
+ // that would let the pane go idle again.
2176
+ return this.paneWriteLock.run(() => this.writeMessageToPane(formatted, windowId, handingOffToNativeQueue, status));
2177
+ }
2178
+ /**
2179
+ * The write half of a delivery: paste → Enter → confirm, with retries.
2180
+ *
2181
+ * Always called under {@link paneWriteLock}. Split out from `deliverMessage`
2182
+ * precisely so the lock's scope is visible at the call site rather than being
2183
+ * an invariant maintained by comments.
2184
+ */
2185
+ async writeMessageToPane(formatted, initialWindowId, handingOffToNativeQueue, status) {
2186
+ let windowId = initialWindowId;
1786
2187
  // Bug A: paste with backoff. Transient failures are usually a stale window id
1787
2188
  // after a crash/respawn — recover by name and retry (max 3 attempts, 2s apart).
1788
2189
  const maxAttempts = 3;
@@ -1807,9 +2208,56 @@ export class Daemon extends EventEmitter {
1807
2208
  await this.tmux.sendSpecialKey("Enter");
1808
2209
  if (status)
1809
2210
  this.emit("message_delivered", status); // 👀
1810
- // Confirm the CLI accepted the message by transitioning idle→busy (new output
1811
- // after Enter). If still idle after ~2s the Enter was likely swallowed while
1812
- // the TUI was redrawing re-send Enter once and re-check.
2211
+ // Busy queue-capable CLIs (codex) may accept paste without an idle→busy
2212
+ // transition. Do NOT probe for busy a second bare Enter can mutate the
2213
+ // queue. Instead verify the paste actually landed in the pane (TUI redraw
2214
+ // can silently swallow it). Idle submissions keep the swallowed-Enter path.
2215
+ if (handingOffToNativeQueue) {
2216
+ await new Promise(r => setTimeout(r, NATIVE_QUEUE_PASTE_VERIFY_MS));
2217
+ if (await this.nativeQueuePasteVisible(formatted)) {
2218
+ if (status)
2219
+ this.emit("message_confirmed", status); // ✅ native queue accepted
2220
+ return true;
2221
+ }
2222
+ // Silent loss: fall back once to the normal idle-gated path.
2223
+ this.logger.warn("Native-queue paste not visible in pane — retrying via idle-gated delivery");
2224
+ if (windowId && this.controlClient) {
2225
+ await this.controlClient.waitUntilIdle(windowId);
2226
+ }
2227
+ const repasted = await this.tmux.pasteBuffer(formatted);
2228
+ if (!repasted) {
2229
+ this.logger.error("Idle-gated redelivery paste failed after native-queue silent loss");
2230
+ if (status)
2231
+ this.emit("message_failed", status); // ❌
2232
+ return false;
2233
+ }
2234
+ await new Promise(r => setTimeout(r, NORMAL_ENTER_SETTLE_MS));
2235
+ const retryAt = Date.now();
2236
+ await this.tmux.sendSpecialKey("Enter");
2237
+ if (windowId && this.controlClient) {
2238
+ let becameBusy = await this.confirmBusyAfterEnter(windowId, retryAt);
2239
+ if (!becameBusy) {
2240
+ this.logger.warn("No idle→busy after idle-gated redelivery — re-sending Enter once");
2241
+ const retry2At = Date.now();
2242
+ await this.tmux.sendSpecialKey("Enter");
2243
+ becameBusy = await this.confirmBusyAfterEnter(windowId, retry2At);
2244
+ }
2245
+ if (becameBusy) {
2246
+ if (status)
2247
+ this.emit("message_confirmed", status); // ✅
2248
+ return true;
2249
+ }
2250
+ }
2251
+ else if (await this.nativeQueuePasteVisible(formatted)) {
2252
+ if (status)
2253
+ this.emit("message_confirmed", status); // ✅
2254
+ return true;
2255
+ }
2256
+ this.logger.error("Idle-gated redelivery also failed after native-queue silent loss");
2257
+ if (status)
2258
+ this.emit("message_failed", status); // ❌
2259
+ return false;
2260
+ }
1813
2261
  if (windowId && this.controlClient) {
1814
2262
  let becameBusy = await this.confirmBusyAfterEnter(windowId, enterAt);
1815
2263
  if (!becameBusy) {
@@ -1818,8 +2266,20 @@ export class Daemon extends EventEmitter {
1818
2266
  await this.tmux.sendSpecialKey("Enter");
1819
2267
  becameBusy = await this.confirmBusyAfterEnter(windowId, retryAt);
1820
2268
  }
1821
- if (becameBusy && status)
1822
- this.emit("message_confirmed", status); // ✅
2269
+ if (becameBusy) {
2270
+ if (status)
2271
+ this.emit("message_confirmed", status); // ✅
2272
+ }
2273
+ else {
2274
+ // Both Enters were swallowed: the text is sitting UNSUBMITTED in the
2275
+ // CLI's input box. This used to return true, so the reaction stayed at 👀
2276
+ // forever and the next delivery pasted on top — submitting two messages
2277
+ // as one. Say so instead.
2278
+ this.logger.error("Message pasted but never submitted (no idle→busy after two Enters)");
2279
+ if (status)
2280
+ this.emit("message_failed", status); // ❌
2281
+ return false;
2282
+ }
1823
2283
  }
1824
2284
  else {
1825
2285
  // No control client to observe output: fall back to the legacy double-Enter.
@@ -1835,8 +2295,36 @@ export class Daemon extends EventEmitter {
1835
2295
  this.emit("message_failed", status); // ❌
1836
2296
  return false;
1837
2297
  }
2298
+ /**
2299
+ * True when a busy native-queue paste appears to have landed: Codex shows a
2300
+ * `↳` queue marker, or a distinctive slice of the pasted text is on screen.
2301
+ * Used only to detect silent paste loss — not as a general ready check.
2302
+ */
2303
+ async nativeQueuePasteVisible(formatted) {
2304
+ if (!this.tmux)
2305
+ return false;
2306
+ try {
2307
+ const pane = await this.tmux.capturePane();
2308
+ if (pane.includes("↳"))
2309
+ return true;
2310
+ for (const line of formatted.split(/\r?\n/)) {
2311
+ const t = line.trim();
2312
+ if (t.length >= 8 && pane.includes(t))
2313
+ return true;
2314
+ }
2315
+ const compact = formatted.replace(/\s+/g, " ").trim();
2316
+ if (compact.length >= 8 && pane.includes(compact.slice(0, Math.min(80, compact.length)))) {
2317
+ return true;
2318
+ }
2319
+ return false;
2320
+ }
2321
+ catch {
2322
+ return false;
2323
+ }
2324
+ }
1838
2325
  /** Re-resolve this instance's tmux window by name (stale id after crash/respawn). */
1839
2326
  async recoverWindow() {
2327
+ const previousWindowId = this.tmux?.getWindowId();
1840
2328
  try {
1841
2329
  const windows = await TmuxManager.listWindows(this.tmuxSessionName);
1842
2330
  const match = windows.find(w => w.name === this.name);
@@ -1844,6 +2332,12 @@ export class Daemon extends EventEmitter {
1844
2332
  return undefined;
1845
2333
  this.tmux = new TmuxManager(this.tmuxSessionName, match.id, resolveTmuxLogicalSize(this.config.terminal));
1846
2334
  writeFileSync(join(this.instanceDir, "window-id"), match.id);
2335
+ // The window we were talking to is gone; leaving it registered means the
2336
+ // control client re-resolves a dead id — one tmux subprocess — on every
2337
+ // reconnect, for the life of the fleet process.
2338
+ if (previousWindowId && previousWindowId !== match.id) {
2339
+ this.controlClient?.unregisterWindow(previousWindowId);
2340
+ }
1847
2341
  await this.controlClient?.registerWindow(match.id);
1848
2342
  this.bindInstanceStateOutputListener(match.id);
1849
2343
  this.logger.info({ windowId: match.id }, "Recovered window ID for message delivery");
@@ -1854,12 +2348,38 @@ export class Daemon extends EventEmitter {
1854
2348
  return undefined;
1855
2349
  }
1856
2350
  }
1857
- /** Poll up to ~2s (200ms × 10) for the pane to emit output after `since`. */
2351
+ /**
2352
+ * Poll up to ~2s (200ms × 10) for the pane to emit output after `since`.
2353
+ *
2354
+ * A `false` here is not "no output" — it is read as **"the message was pasted but
2355
+ * never submitted"**, which re-sends Enter and then reports ❌ to the user. So a
2356
+ * false negative costs a possible double submit and a failure notice for a
2357
+ * message that actually arrived.
2358
+ *
2359
+ * That is exactly what a control-mode reconnect used to produce. `connect()`
2360
+ * drops every output timestamp, so `hasOutputSince` answers `false` for a pane
2361
+ * that did react — the evidence was thrown away, not absent. Blind time is
2362
+ * therefore not counted against the budget: after a reset the poll restarts from
2363
+ * the moment observation resumed. A hard wall-clock cap keeps a reconnect loop
2364
+ * from extending this forever.
2365
+ */
1858
2366
  async confirmBusyAfterEnter(windowId, since) {
1859
- for (let i = 0; i < 10; i++) {
1860
- await new Promise(r => setTimeout(r, 200));
1861
- if (this.controlClient.hasOutputSince(windowId, since))
2367
+ const client = this.controlClient;
2368
+ const hardDeadline = Date.now() + CONFIRM_BUSY_MAX_WAIT_MS;
2369
+ let observedFrom = since;
2370
+ let polls = 0;
2371
+ while (polls < CONFIRM_BUSY_POLLS) {
2372
+ await new Promise(r => setTimeout(r, CONFIRM_BUSY_POLL_MS));
2373
+ if (client.hasOutputSince(windowId, observedFrom))
1862
2374
  return true;
2375
+ const resetAt = client.getObservationResetAt();
2376
+ if (resetAt > observedFrom && Date.now() < hardDeadline) {
2377
+ this.logger.debug({ windowId, resetAt }, "Control mode reconnected mid-confirmation — restarting the Enter check from when we could see again");
2378
+ observedFrom = resetAt;
2379
+ polls = 0;
2380
+ continue;
2381
+ }
2382
+ polls++;
1863
2383
  }
1864
2384
  return false;
1865
2385
  }
@@ -1930,8 +2450,8 @@ export class Daemon extends EventEmitter {
1930
2450
  });
1931
2451
  const timeout = setTimeout(() => {
1932
2452
  this.pendingIpcRequests.delete(fleetReqId);
1933
- respond(null, "Task operation timed out after 30s");
1934
- }, 30_000);
2453
+ respond(null, `Task operation timed out after ${daemonBudgetMs(tool) / 1000}s`);
2454
+ }, daemonBudgetMs(tool));
1935
2455
  this.pendingIpcRequests.set(fleetReqId, (respMsg) => {
1936
2456
  clearTimeout(timeout);
1937
2457
  respond(respMsg.result, respMsg.error);
@@ -1953,8 +2473,8 @@ export class Daemon extends EventEmitter {
1953
2473
  });
1954
2474
  const timeout = setTimeout(() => {
1955
2475
  this.pendingIpcRequests.delete(fleetReqId);
1956
- respond(null, "Decision operation timed out after 30s");
1957
- }, 30_000);
2476
+ respond(null, `Decision operation timed out after ${daemonBudgetMs(tool) / 1000}s`);
2477
+ }, daemonBudgetMs(tool));
1958
2478
  this.pendingIpcRequests.set(fleetReqId, (respMsg) => {
1959
2479
  clearTimeout(timeout);
1960
2480
  respond(respMsg.result, respMsg.error);
@@ -1980,8 +2500,8 @@ export class Daemon extends EventEmitter {
1980
2500
  // Wait for fleet_schedule_response via pending request map
1981
2501
  const timeout = setTimeout(() => {
1982
2502
  this.pendingIpcRequests.delete(fleetReqId);
1983
- respond(null, "Schedule operation timed out after 30s");
1984
- }, 30_000);
2503
+ respond(null, `Schedule operation timed out after ${daemonBudgetMs(tool) / 1000}s`);
2504
+ }, daemonBudgetMs(tool));
1985
2505
  this.pendingIpcRequests.set(fleetReqId, (respMsg) => {
1986
2506
  clearTimeout(timeout);
1987
2507
  respond(respMsg.result, respMsg.error);
@@ -2002,7 +2522,9 @@ export class Daemon extends EventEmitter {
2002
2522
  fleetRequestId: fleetReqId,
2003
2523
  senderSessionName,
2004
2524
  });
2005
- const crossTimeoutMs = (tool === "start_instance" || tool === "create_instance" || tool === "replace_instance") ? 60_000 : 30_000;
2525
+ // Shared table, so delete_instance and the deployment tools get the same
2526
+ // budget here as the MCP client expects (the ad-hoc list omitted them).
2527
+ const crossTimeoutMs = daemonBudgetMs(tool);
2006
2528
  const timeout = setTimeout(() => {
2007
2529
  this.pendingIpcRequests.delete(fleetReqId);
2008
2530
  respond(null, `Cross-instance operation timed out after ${crossTimeoutMs / 1000}s`);
@@ -2045,8 +2567,8 @@ export class Daemon extends EventEmitter {
2045
2567
  this.ipcServer?.broadcast({ type: "fleet_outbound", tool, args, fleetRequestId: fleetReqId });
2046
2568
  const timeout = setTimeout(() => {
2047
2569
  this.pendingIpcRequests.delete(outboundKey);
2048
- respond(null, "Fleet outbound timed out after 30s");
2049
- }, 30_000);
2570
+ respond(null, `Fleet outbound timed out after ${daemonBudgetMs(tool) / 1000}s`);
2571
+ }, daemonBudgetMs(tool));
2050
2572
  this.pendingIpcRequests.set(outboundKey, (respMsg) => {
2051
2573
  clearTimeout(timeout);
2052
2574
  respond(respMsg.result, respMsg.error);
@@ -2110,7 +2632,14 @@ export class Daemon extends EventEmitter {
2110
2632
  try {
2111
2633
  const all = JSON.parse(process.env.AGEND_DECISIONS);
2112
2634
  const workDir = this.config.working_directory;
2113
- decisions = all.filter(d => d.scope === "fleet" || d.project_root === workDir);
2635
+ // `scope: "fleet"` used to bypass the project check entirely, so a
2636
+ // single-project worker inherited every other project's playbook (8 of
2637
+ // 14 injected here were foreign) — resent on every API call. Now a
2638
+ // fleet decision must also be RELEVANT: global (no project_root), the
2639
+ // same project, or the same project family (worktrees/checkouts of one
2640
+ // repo). A general is exempt: it dispatches across all projects, so
2641
+ // cross-project routing rules are exactly what it needs.
2642
+ decisions = selectRelevantDecisions(all, workDir, this.config.general_topic === true);
2114
2643
  // Stable ordering so identical decision sets always build byte-identical
2115
2644
  // instructions — otherwise source ordering jitter flips the warmup hash.
2116
2645
  decisions.sort((a, b) => a.title.localeCompare(b.title));
@@ -2124,9 +2653,18 @@ export class Daemon extends EventEmitter {
2124
2653
  AGEND_SOCKET_PATH: sockPath,
2125
2654
  AGEND_INSTANCE_NAME: this.name,
2126
2655
  AGEND_WORKING_DIR: this.config.working_directory,
2656
+ AGEND_INSTANCE_KIND: this.runtimeIdentity?.kind ?? "fleet-topic",
2657
+ AGEND_BACKEND: this.runtimeIdentity?.backend ?? this.config.backend ?? this.backend?.binaryName ?? "unknown",
2658
+ AGEND_MODEL: this.runtimeIdentity?.model ?? this.config.model ?? "default",
2127
2659
  };
2128
- if (this.config.tool_set)
2129
- mcpEnv.AGEND_TOOL_SET = this.config.tool_set;
2660
+ // A general is a dispatcher, not a worker: default it to the `general` tool
2661
+ // profile instead of `full`. Every tool's schema is resent on every API call,
2662
+ // so full (44 tools) costs a general ~22k chars per turn — the single largest
2663
+ // slice of its prompt. Explicit config still wins.
2664
+ const defaultToolSet = this.config.general_topic ? "general" : undefined;
2665
+ const toolSet = this.config.tool_set ?? defaultToolSet;
2666
+ if (toolSet)
2667
+ mcpEnv.AGEND_TOOL_SET = toolSet;
2130
2668
  if (this.config.display_name)
2131
2669
  mcpEnv.AGEND_DISPLAY_NAME = this.config.display_name;
2132
2670
  if (this.config.description)
@@ -2159,6 +2697,7 @@ export class Daemon extends EventEmitter {
2159
2697
  instructions = buildFleetInstructions({
2160
2698
  instanceName: this.name,
2161
2699
  workingDirectory: this.config.working_directory,
2700
+ runtimeIdentity: this.runtimeIdentity,
2162
2701
  displayName: this.config.display_name,
2163
2702
  description: this.config.description,
2164
2703
  customPrompt: resolvedCustomPrompt,
@@ -2171,6 +2710,7 @@ export class Daemon extends EventEmitter {
2171
2710
  instructions = buildFleetInstructions({
2172
2711
  instanceName: this.name,
2173
2712
  workingDirectory: this.config.working_directory,
2713
+ runtimeIdentity: this.runtimeIdentity,
2174
2714
  displayName: this.config.display_name,
2175
2715
  description: this.config.description,
2176
2716
  customPrompt: resolvedCustomPrompt,
@@ -2217,7 +2757,9 @@ export class Daemon extends EventEmitter {
2217
2757
  // Small delay to let the CLI fully render its ready prompt
2218
2758
  await new Promise(r => setTimeout(r, 1_000));
2219
2759
  try {
2220
- await this.tmux.pasteText(`[system:session-snapshot]\n${snapshot}\n\nThis is a background context restore do NOT reply to or acknowledge this message. Simply resume normal operation when the next user or instance message arrives.`);
2760
+ // Messages can arrive during a restart and be queued on pasteLock before the
2761
+ // snapshot lands; both write to the pane, so both go through the same lock.
2762
+ await this.paneWriteLock.run(() => this.tmux.pasteText(`[system:session-snapshot]\n${snapshot}\n\nThis is a background context restore — do NOT reply to or acknowledge this message. Simply resume normal operation when the next user or instance message arrives.`, this.systemPasteOptions()));
2221
2763
  this.logger.info("Injected session snapshot as first message");
2222
2764
  this.emit("snapshot_injected", this.name);
2223
2765
  }
@@ -2226,9 +2768,69 @@ export class Daemon extends EventEmitter {
2226
2768
  this.emit("snapshot_failed", this.name);
2227
2769
  }
2228
2770
  }
2771
+ /**
2772
+ * Depth, not a flag: the wake path marks a spawn and then calls trySpawn, which
2773
+ * marks another. With a plain boolean the inner one's completion would clear the
2774
+ * latch while the outer spawn was still dismissing dialogs, releasing delivery
2775
+ * into exactly the window this exists to close.
2776
+ */
2777
+ beginSpawn() {
2778
+ this.spawnDepth++;
2779
+ this.spawning = true;
2780
+ if (!this.spawnSettled) {
2781
+ this.spawnSettled = new Promise(resolve => { this.resolveSpawnSettled = resolve; });
2782
+ }
2783
+ }
2784
+ endSpawn() {
2785
+ this.spawnDepth = Math.max(0, this.spawnDepth - 1);
2786
+ if (this.spawnDepth > 0)
2787
+ return; // an enclosing spawn is still running
2788
+ this.spawning = false;
2789
+ const resolve = this.resolveSpawnSettled;
2790
+ this.spawnSettled = null;
2791
+ this.resolveSpawnSettled = null;
2792
+ resolve?.();
2793
+ }
2794
+ /**
2795
+ * Hold until the CLI has finished starting up.
2796
+ *
2797
+ * Startup is not a quiet period: `dismissDialogsUntilReady` is clicking through
2798
+ * trust prompts and session pickers. A pane showing a modal dialog produces no
2799
+ * output, so `waitUntilIdle` reports it idle and a queued message gets pasted
2800
+ * *into the dialog* — where the text is discarded and the Enter picks a menu
2801
+ * item. The paste and the Enter both "succeed", so this loses the message
2802
+ * without even a ❌.
2803
+ *
2804
+ * The pane write lock (#414) does not cover this: it serialises each key
2805
+ * sequence, but it is released between one dialog and the next.
2806
+ *
2807
+ * Bounded, and called BEFORE the pane lock is taken — waiting on the spawn while
2808
+ * holding the lock the spawn itself needs would deadlock.
2809
+ */
2810
+ async waitForSpawnToSettle() {
2811
+ const settled = this.spawnSettled;
2812
+ if (!settled)
2813
+ return;
2814
+ this.logger.debug("Holding delivery until the CLI has finished starting up");
2815
+ let timer;
2816
+ const cap = new Promise(resolve => {
2817
+ timer = setTimeout(resolve, SPAWN_SETTLE_MAX_WAIT_MS);
2818
+ timer.unref?.();
2819
+ });
2820
+ try {
2821
+ await Promise.race([settled, cap]);
2822
+ }
2823
+ finally {
2824
+ if (timer)
2825
+ clearTimeout(timer);
2826
+ }
2827
+ if (this.spawning) {
2828
+ this.logger.warn("CLI still starting after the delivery hold — delivering anyway");
2829
+ }
2830
+ }
2229
2831
  /** Spawn a CLI window. Returns true if --resume was used successfully. */
2230
2832
  async spawnClaudeWindow() {
2231
- this.spawning = true;
2833
+ this.beginSpawn();
2232
2834
  let resumedSuccessfully = false;
2233
2835
  try {
2234
2836
  this.toolStatusLines = [];
@@ -2265,7 +2867,7 @@ export class Daemon extends EventEmitter {
2265
2867
  this.backgroundSessionRecoveryAttempted = false;
2266
2868
  }
2267
2869
  finally {
2268
- this.spawning = false;
2870
+ this.endSpawn();
2269
2871
  }
2270
2872
  return resumedSuccessfully;
2271
2873
  }
@@ -2336,11 +2938,13 @@ export class Daemon extends EventEmitter {
2336
2938
  // the global web token) from impersonating instances.
2337
2939
  const agentTokenPath = join(this.instanceDir, "agent.token");
2338
2940
  const agentToken = randomBytes(32).toString("hex");
2339
- writeFileSync(agentTokenPath, agentToken, { mode: 0o600 });
2340
- try {
2341
- chmodSync(agentTokenPath, 0o600);
2941
+ const tokenWrite = writeSecretFile(agentTokenPath, agentToken);
2942
+ if (!tokenWrite.ok) {
2943
+ // Do not fail the spawn over it — an instance that cannot start is worse
2944
+ // than one whose token is readable. But say so, loudly: the whole point of
2945
+ // this token is that other local processes cannot use it.
2946
+ this.logger.error({ path: agentTokenPath, mode: tokenWrite.mode?.toString(8), reason: tokenWrite.reason }, "Agent token file is not owner-only — other local users can impersonate this instance");
2342
2947
  }
2343
- catch { }
2344
2948
  // AGEND_HOME points the child's agent-cli at the same data dir the daemon
2345
2949
  // is using, so it can locate <instanceDir>/agent.token.
2346
2950
  const agendHome = join(this.instanceDir, "..", "..");
@@ -2358,7 +2962,12 @@ export class Daemon extends EventEmitter {
2358
2962
  windowId = this.tmux.getWindowId();
2359
2963
  }
2360
2964
  else {
2965
+ // A crash respawn makes a brand new window. The dead one stays registered
2966
+ // unless we say so — the respawn branch above already does.
2967
+ const retired = this.tmux.getWindowId();
2361
2968
  windowId = await this.tmux.createWindow(cmd, resolvedCwd, this.name);
2969
+ if (retired && retired !== windowId)
2970
+ this.controlClient?.unregisterWindow(retired);
2362
2971
  }
2363
2972
  writeFileSync(join(this.instanceDir, "window-id"), windowId);
2364
2973
  // Enable remain-on-exit to capture exit codes on crash
@@ -2366,7 +2975,9 @@ export class Daemon extends EventEmitter {
2366
2975
  this.logger.warn({ err }, "Failed to set remain-on-exit — exit codes will not be captured");
2367
2976
  });
2368
2977
  if (reuseWindow && !this.config.lightweight) {
2369
- await this.tmux.pipeOutput(join(this.instanceDir, "output.log")).catch(err => {
2978
+ const outputLog = join(this.instanceDir, "output.log");
2979
+ rotateLogIfNeeded(outputLog);
2980
+ await this.tmux.pipeOutput(outputLog).catch(err => {
2370
2981
  this.logger.warn({ err }, "Failed to restore pipe-pane after wake");
2371
2982
  });
2372
2983
  }
@@ -2422,15 +3033,20 @@ export class Daemon extends EventEmitter {
2422
3033
  for (const dialog of startupDialogs) {
2423
3034
  if (dialog.pattern.test(pane)) {
2424
3035
  this.logger.debug(`Dismissing startup dialog: ${dialog.description}`);
2425
- for (const key of dialog.keys) {
2426
- if (key === "Up" || key === "Down" || key === "Enter" || key === "Escape") {
2427
- await this.tmux.sendSpecialKey(key);
3036
+ // Restart is exactly when inbound messages pile up, and nothing gates
3037
+ // delivery on `spawning`. Take the pane lock for the key sequence so a
3038
+ // queued message cannot be pasted into a half-dismissed trust dialog.
3039
+ await this.paneWriteLock.run(async () => {
3040
+ for (const key of dialog.keys) {
3041
+ if (key === "Up" || key === "Down" || key === "Enter" || key === "Escape") {
3042
+ await this.tmux.sendSpecialKey(key);
3043
+ }
3044
+ else {
3045
+ await this.tmux.sendKeys(key);
3046
+ }
3047
+ await new Promise(r => setTimeout(r, 200));
2428
3048
  }
2429
- else {
2430
- await this.tmux.sendKeys(key);
2431
- }
2432
- await new Promise(r => setTimeout(r, 200));
2433
- }
3049
+ });
2434
3050
  // Wait for next screen to render
2435
3051
  if (this.controlClient) {
2436
3052
  const wid = readFileSync(join(this.instanceDir, "window-id"), "utf-8").trim();