@songsid/agend 2.3.0-beta.9 → 2.3.0

package/dist/daemon.js

@@ -1,4 +1,4 @@
-import { AGENT_WHATS_NEW, agendVersion, planWhatsNew, readWhatsNewRecord, recordWhatsNew, whatsNewNotice } from "./agent-whats-new.js";
+import { agentWhatsNew, agendVersion, planWhatsNew, readWhatsNewRecord, recordWhatsNew, whatsNewNotice } from "./agent-whats-new.js";
 import { join, dirname, resolve } from "node:path";
 import { checkoutTarget, readCheckouts, releaseCandidate, reposRoot, verifyWorktree, writeCheckouts } from "./repo-worktree-guard.js";
 import { mkdirSync, writeFileSync, readFileSync, readdirSync, existsSync, unlinkSync, rmSync, appendFileSync, statSync, renameSync } from "node:fs";
@@ -10,6 +10,7 @@ import { createHash, randomBytes, randomUUID } from "node:crypto";
 import { EventEmitter } from "node:events";
 import { rotateLogIfNeededAsync } from "./logger.js";
 import { coredumpFilterLaunchPrefix } from "./coredump-filter.js";
+import { observeCliOomIdentity, confirmedCliOom, consecutiveSessionOoms } from "./session-oom.js";
 import { mcpServerState } from "./mcp-liveness.js";
 import { clearPausedMarker, writePausedMarker } from "./pause-marker.js";
 import { writeBackgroundWorkEndedMarker } from "./background-work-ended.js";
@@ -19,7 +20,9 @@ import { createTranscriptSource } from "./transcript-sources.js";
 import { TranscriptTurnLedger } from "./transcript-turns.js";
 import { credentialProfileStoreHome, resolveCredentialProfile } from "./backend/credential-profile.js";
 import { resolveToolSet } from "./tool-permissions.js";
+import { dashboardAccessProblem, dashboardFeaturesOff } from "./dashboard-permissions.js";
 import { getAgendHome } from "./paths.js";
+import { readCanvasWorkerEnabled, scopedFleetConfigEnvironment } from "./web-experimental.js";
 import { readAgentEffect, sendEffectInParts } from "./web-effect-upload.js";
 import { readAgentFile } from "./agent-file-read.js";
 import { ASSET_UPLOAD_MAX_BYTES, MOD_FILE, MOD_MAX_PARTS, MOD_UPLOAD_MAX_BYTES, receiveFiles, receivePreview, sendAssetInParts, writeModFile, writePreview } from "./web-asset-parts.js";
@@ -53,6 +56,7 @@ import { measureSyncWork, noteSyncWork } from "./sync-work-attribution.js";
 import { TranscriptDeltaReader } from "./transcript-delta-reader.js";
 import { buildResumeContinuation, clearInFlightTurnMarker, consumeInFlightTurnMarker, decideTurnResume, normalizeResumeBackend, readResumeFingerprint, RESUME_BLOCKING_DELIVERY_STATES, writeInFlightTurnMarker, } from "./turn-resume.js";
 import { codexLiveRowMatches } from "./backend/codex.js";
+import { codexPackageProblemMessage } from "./backend/codex-package.js";
 import { claudeConfigDirFor, claudeProfileConfigDir, claudeSwitchDialog, claudeSwitchTargetMatches } from "./backend/claude-code.js";
 import { InteractionObservation, INTERACTION_CONFIRM_MS, sameInteractionOwner } from "./interaction-observation.js";
 import { unrecognisedPromptShape } from "./unrecognised-prompt.js";
@@ -1157,6 +1161,18 @@ export class Daemon extends EventEmitter {
     // injected by FleetManager below.
     healthCheckTimer = null;
     crashCount = 0;
+    cliOomObservation = null;
+    consecutiveOoms = null;
+    oomSessionKey() {
+        // Claude's per-instance statusline owns its ID. Other current getters
+        // either return null or select by cwd (Muse/Grok): those are not proof
+        // of this pane's session and must not trigger this destructive decision.
+        if ((this.config.backend ?? "claude-code") !== "claude-code")
+            return null;
+        const session = this.backend?.getSessionId?.();
+        return typeof session === "string" && session.trim() && session.length <= 512
+            ? JSON.stringify([this.config.backend ?? "claude-code", session.trim()]) : null;
+    }
     lastCrashAt = 0;
     lastSpawnAt = 0;
     crashTimestamps = [];
@@ -1565,6 +1581,7 @@ export class Daemon extends EventEmitter {
     windowQueryFailureTicks = 0;
     mcpRestartPending = false;
     mcpRestartStaleTimer = null;
+    mcpRestartCooldownTimer = null;
     /** In-flight replacement grace; also the guard against three triggers stacking. */
     mcpRestartGraceTimer = null;
     /**
@@ -1601,6 +1618,20 @@ export class Daemon extends EventEmitter {
     get isErrorState() {
         return this.errorWaitingForRecovery || (this.healthCheckPaused && !this.isPaused) || this.stormWindow?.isActive() === true;
     }
+    /**
+     * #2327: which of isErrorState's conditions holds, so a warning names the one actually observed. A CLI no longer
+     * supervised outranks a tmux recovery, which outranks a detected error pattern — one match is enough for that last
+     * one, so it proves neither a pause nor a repetition.
+     */
+    get errorStateCause() {
+        if (this.healthCheckPaused && !this.isPaused)
+            return "unsupervised";
+        if (this.stormWindow?.isActive() === true)
+            return "tmux_recovering";
+        if (this.errorWaitingForRecovery)
+            return "error_detected";
+        return null;
+    }
     get isPaused() { return this.pauseWakeState !== "active"; }
     /** Current CLI process state for status surfaces and recovery commands. */
     getProcessStatus() {
@@ -2356,6 +2387,7 @@ export class Daemon extends EventEmitter {
                 ackIfPending();
                 this.recordRecentEvent({ type: "assistant_text", preview: text.slice(0, 100) });
             });
+            this.transcriptMonitor.on("codex_package_missing_host", (root) => this.noteCodexPackageMissingHost(root));
             this.transcriptMonitor.startPolling();
             // HangDetector is the notification bridge to the fleet manager; pane state
             // transitions below are the sole source of hang events (see hang-detector.ts).
@@ -2583,6 +2615,24 @@ export class Daemon extends EventEmitter {
             this.fireMcpRestartRequest("already_idle");
         }
     }
+    /** Keep a cooldown-suppressed revival alive without bypassing the idle gate. */
+    deferMcpRestart(delayMs) {
+        this.clearMcpRestartRequest();
+        if (this.config.mcp_auto_restart === false || this.authFailureUnresolved
+            || this.mcpRestartCancelledByRecovery("cooldown"))
+            return;
+        this.mcpRestartPending = true;
+        this.mcpRestartCooldownTimer = setTimeout(() => {
+            this.mcpRestartCooldownTimer = null;
+            if (this.mcpRestartCancelledByRecovery("cooldown_expired"))
+                return;
+            this.clearMcpRestartRequest();
+            if (this.config.mcp_auto_restart === false || this.authFailureUnresolved || this.mcpServerAlive().alive)
+                return;
+            this.armMcpRestartWhenIdle();
+        }, Math.max(0, delayMs));
+        this.mcpRestartCooldownTimer.unref?.();
+    }
     /**
      * Single chokepoint for all three revival triggers (already_idle, idle_edge,
      * stale_timeout) — and therefore the one place worth re-checking reality.
@@ -2600,6 +2650,8 @@ export class Daemon extends EventEmitter {
     fireMcpRestartRequest(trigger) {
         if (this.mcpRestartCancelledByRecovery(trigger))
             return;
+        if (this.mcpRestartCooldownTimer)
+            return;
         // Authoritative re-read — the CLI may have replaced the server since the
         // health tick that armed this.
         if (this.mcpServerAlive().alive) {
@@ -2794,6 +2846,10 @@ export class Daemon extends EventEmitter {
     }
     clearMcpRestartRequest() {
         this.mcpRestartPending = false;
+        if (this.mcpRestartCooldownTimer) {
+            clearTimeout(this.mcpRestartCooldownTimer);
+            this.mcpRestartCooldownTimer = null;
+        }
         if (this.mcpRestartStaleTimer) {
             clearTimeout(this.mcpRestartStaleTimer);
             this.mcpRestartStaleTimer = null;
@@ -2862,6 +2918,19 @@ export class Daemon extends EventEmitter {
                     if (discardStaleHealth())
                         return;
                     if (paneStatus?.alive) {
+                        // Capture only a known primary while it lives. No extra tmux call: PID comes
+                        // from this same status read. Fast/unobserved deaths remain ordinary crashes.
+                        const key = this.oomSessionKey();
+                        if (this.cliOomObservation?.key !== key)
+                            this.cliOomObservation = null;
+                        if (paneStatus.pid && key && !this.cliOomObservation) {
+                            const identity = await observeCliOomIdentity(paneStatus.pid, cliLabel);
+                            if (discardStaleHealth())
+                                return;
+                            if (identity && this.oomSessionKey() === key) {
+                                this.cliOomObservation = { identity, key, owner: healthOwner, tmux: healthTmux };
+                            }
+                        }
                         this.windowQueryFailureTicks = 0;
                         this.stormWindow?.noteWindowAlive(this.name);
                         // Instance output.log is fed by tmux pipe-pane and was previously never
@@ -3016,6 +3085,30 @@ export class Daemon extends EventEmitter {
                         return;
                     this.setProcessStatus("crashed");
                     this.logPaneDeath(cliLabel, exitCode, lastOutput);
+                    const observed = this.cliOomObservation;
+                    const key = this.oomSessionKey();
+                    const oom = paneStatus && exitCode === 137 && observed && observed.key === key
+                        && observed.tmux === healthTmux && sameInteractionOwner(observed.owner, healthOwner)
+                        ? await confirmedCliOom(observed.identity, Date.now()) : null;
+                    if (discardStaleHealth())
+                        return;
+                    this.cliOomObservation = null;
+                    const decisionKey = this.oomSessionKey();
+                    this.consecutiveOoms = consecutiveSessionOoms(this.consecutiveOoms, decisionKey, decisionKey === key && oom !== null);
+                    if (this.consecutiveOoms && this.consecutiveOoms.count >= 2) {
+                        // Stop supervision, not an idle pause: queued work must not auto-wake
+                        // the same session. An explicit restart is the person's retry choice.
+                        this.healthCheckPaused = true;
+                        try {
+                            this.appendCrashHistory({ exitCode, lastOutput, crashType, reason: "session_oom" });
+                        }
+                        catch (error) {
+                            this.logger.warn({ error }, "Could not append session OOM history");
+                        }
+                        this.logger.error({ pid: oom.pid, count: this.consecutiveOoms.count }, "Repeated confirmed CLI OOM — automatic resume stopped; session preserved");
+                        this.emitSupervisionEnded(t("inst.session_oom_stopped"), t("inst.session_oom_remedy"), exitCode, "session_oom");
+                        return;
+                    }
                     // Kill the dead window (remain-on-exit keeps it around) before respawn
                     if (paneStatus) {
                         await healthTmux.killWindow();
@@ -3126,14 +3219,14 @@ export class Daemon extends EventEmitter {
                         // (bounded) session lookup. They only stop LATER ticks; this continuation is already
                         // past that check and must not clear the process/window or respawn an instance that
                         // was just stopped or paused (#1160 review).
-                        if (this.runtimeMonitorsFrozen || this.healthCheckPaused)
+                        if (discardStaleHealth())
                             return;
                         // Reset is synchronous for file sources, but Kiro's baseline is
                         // read off-thread. Finish it before starting the replacement CLI,
                         // otherwise its first work could become part of that baseline.
                         if (!await this.resetTranscriptBeforeAdmission())
                             return;
-                        if (this.runtimeMonitorsFrozen || this.healthCheckPaused)
+                        if (discardStaleHealth())
                             return;
                         // Kill orphan MCP server from the crashed CLI session.
                         // MCP server writes its PID to channel.mcp.pid on startup.
@@ -3157,6 +3250,8 @@ export class Daemon extends EventEmitter {
                             }
                         }
                         catch { /* tmux server may be dead — ensureSession in trySpawn will recover */ }
+                        if (discardStaleHealth())
+                            return;
                         // Write snapshot before spawn — consumed only if resume fails
                         this.writeRotationSnapshot("crash");
                         // Try --resume first; spawnClaudeWindow falls back to fresh session if resume fails
@@ -3651,6 +3746,17 @@ export class Daemon extends EventEmitter {
             const key = Daemon.errorPatternKey(ep);
             const count = countMatches(ep.pattern);
             const seen = this.lastErrorCount.get(key) ?? 0;
+            // #2282: a resumed exhausted429 row is history, not even a new
+            // notification/failover incident. Wait for the first live composer to
+            // establish its baseline; later appended occurrences remain eligible.
+            if (ep.type === "rate_limit" && ep.action === "nudge_continue") {
+                const baseline = this.nudgeBaselines?.get(key);
+                if (baseline === undefined || count <= baseline) {
+                    if (baseline !== undefined)
+                        this.lastErrorCount.set(key, count);
+                    continue;
+                }
+            }
             // The generic Codex usage-limit line remains in scrollback after the
             // user selects Luna Reserve. What proves that is NOT a live composer —
             // codex paints one after a plain hit too, and the old live-composer guard
@@ -4677,7 +4783,9 @@ export class Daemon extends EventEmitter {
     static AUTH_RECONFIRM_ROWS = 15;
     /** Why the deferred pause was requested; what its marker records (Phase 2a). */
     pendingPauseReason = "auth";
-    async pause(reason = "operator") {
+    async pause(reason = "operator", admission) {
+        if (admission && !admission.current())
+            return;
         if (this.pauseWakeState === "paused")
             return;
         if (this.pauseWakeState === "pausing" || this.pauseWakeState === "waking") {
@@ -4685,6 +4793,8 @@ export class Daemon extends EventEmitter {
             if (this.getPauseWakeState() !== "active")
                 return;
         }
+        if (admission && !admission.current())
+            return;
         // `pauseAllowStuck` is consumed here, not merely read: it authorises exactly
         // one pause from a stuck pane (the auth-deferred one) and must never leak
         // into an ordinary idle-timeout pause of a busy instance.
@@ -4704,7 +4814,7 @@ export class Daemon extends EventEmitter {
         // this cycle while the backend reports background work; the next idle
         // tick retries once the task ends. A pane-capture failure proceeds (the
         // quit-menu handler below stays the backstop), never wedges unpaused.
-        if (this.backend?.backgroundWorkRunning && this.tmux) {
+        if ((admission || this.backend?.backgroundWorkRunning) && this.tmux) {
             const tmux = this.tmux;
             const windowId = tmux.getWindowId?.();
             const generation = this.spawnGeneration;
@@ -4716,7 +4826,8 @@ export class Daemon extends EventEmitter {
             // #1730 P2-2: the await yields — the pane read is only valid for the
             // world it was taken from. Revalidate owner, phase, state and queue
             // before acting on it; anything that moved defers this cycle.
-            if (this.tmux !== tmux
+            if ((admission && (!admission.current() || pane === null || !admission.accepts(pane)))
+                || this.tmux !== tmux
                 || this.tmux?.getWindowId?.() !== windowId
                 || this.spawnGeneration !== generation
                 || this.getPauseWakeState() !== "active"
@@ -4913,7 +5024,9 @@ export class Daemon extends EventEmitter {
                 this.logger.info({ notLogged: this.stateEdgesNotLogged }, "Instance execution state edges were not logged while it was flapping");
                 this.stateEdgesNotLogged = 0;
             }
-            this.logger.info(fields, "Instance execution state changed");
+            // #2482: idle ↔ working is every turn (40% of daemon.log): debug. Anything involving stuck stays INFO.
+            const routine = previous !== "stuck" && snapshot.state !== "stuck";
+            this.logger[routine ? "debug" : "info"](fields, "Instance execution state changed");
             return;
         }
         if (this.stateEdgesNotLogged === 0) {
@@ -6353,6 +6466,15 @@ export class Daemon extends EventEmitter {
             this.config.cross_instance_visibility = update.cross_instance_visibility;
         else if (update.cross_instance_visibility === null)
             delete this.config.cross_instance_visibility;
+        if (Number.isInteger(update.board_cards_at_once) && update.board_cards_at_once >= 1)
+            this.config.board_cards_at_once = update.board_cards_at_once;
+        else if (update.board_cards_at_once === null)
+            delete this.config.board_cards_at_once;
+        // The fleet reads it at each tool call; kept current here for the next spawn's tool menu.
+        if (update.dashboard && typeof update.dashboard === "object" && !dashboardAccessProblem(update.dashboard))
+            this.config.dashboard = update.dashboard;
+        else if (update.dashboard === null)
+            delete this.config.dashboard;
         if (["trace", "debug", "info", "warn", "error"].includes(String(update.log_level))) {
             const level = update.log_level;
             if (this.config.log_level !== level) {
@@ -7612,6 +7734,46 @@ export class Daemon extends EventEmitter {
         this.logger.info({ delayMs }, "Model-capacity nudge armed");
         return true;
     }
+    /**
+     * Consume an exhausted nudge only if its failed turn still owns an empty,
+     * idle pane. Unlike arming, this verifies all input/work gates before pause.
+     * The returned owner survives pause's own monitor freeze (which advances the
+     * launch fence), but never another spawn/launch, cancel, delivery or episode.
+     */
+    async prepareCapacityNudgePause() {
+        const nudge = this.capacityNudge, episode = this.lastErrorEpisode;
+        const tmux = this.tmux, attempt = this.launchAttempt, boot = this.bootId;
+        this.cancelCapacityNudge("retries exhausted");
+        if (!nudge || !episode || !tmux)
+            return null;
+        const window = tmux.getWindowId?.();
+        const current = () => this.tmux === tmux && tmux.getWindowId?.() === window
+            && boot === this.bootId && nudge.spawn === this.spawnGeneration && attempt === this.launchAttempt
+            && episode === this.lastErrorEpisode && this.isDeliveryEpochCurrent(nudge.epoch)
+            && !this.startupAborted && !this.spawning && !this.isPaused
+            && this.getProcessStatus() === "running" && this.instanceState === "idle" && this.pasteQueueDepth === 0;
+        const accepts = (pane) => pane === nudge.pane && this.paneAuthoritativelyIdle(pane)
+            && this.backend?.isPeriodicRedrawIdlePane?.(pane) === true
+            && codexLiveRowMatches(pane, nudge.pattern) && !this.backend?.backgroundWorkRunning?.(pane);
+        const revalidate = () => this.paneWriteLock.run(async () => {
+            if (!current())
+                return false;
+            try {
+                const pane = await tmux.capturePane();
+                return current() && accepts(pane);
+            }
+            catch {
+                return false;
+            }
+        });
+        // Initial admission must also respect the captured monitor fence. Later
+        // revalidation deliberately tolerates only pause's own fence advancement.
+        if (nudge.fence !== this.launchFenceEpoch || this.healthCheckPaused || this.runtimeMonitorsFrozen)
+            return null;
+        if (!await revalidate() || nudge.fence !== this.launchFenceEpoch || this.healthCheckPaused || this.runtimeMonitorsFrozen)
+            return null;
+        return { current, accepts, revalidate };
+    }
     /** Drop an armed nudge (the user stopped / paused the instance, or the lifecycle gave up on it). */
     cancelCapacityNudge(reason) {
         if (!this.capacityNudge)
@@ -7954,6 +8116,21 @@ export class Daemon extends EventEmitter {
         this.footerFallbackEpoch = null;
     }
     footerFallbackWarnedGeneration = null;
+    /** #2357: the launch (spawn generation and attempt) the missing code-mode host was last reported for. */
+    codexPackageNoticeEpoch = null;
+    /**
+     * #2357: codex recorded that a tool call could not start its code-mode host. Every tool call of a code-mode-only
+     * account fails this way (AgEnD's reply too) while the pane shows only `⚠ N warnings · f2 to view`. Told once per
+     * launch however often the model retries; the launch check (CodexBackend.buildCommand) may already have said so.
+     */
+    noteCodexPackageMissingHost(root) {
+        const epoch = `${this.spawnGeneration}:${this.launchAttempt}`;
+        if (this.codexPackageNoticeEpoch === epoch)
+            return;
+        this.codexPackageNoticeEpoch = epoch;
+        this.logger.warn({ root }, "Codex tool call failed: code-mode host missing");
+        this.emit("backend_launch_warning", { name: this.name, message: codexPackageProblemMessage(root, { file: "bin/codex-code-mode-host", reason: "missing" }) });
+    }
     /**
      * The input-row proof used before a paste (#931/#947), with one fallback
      * (#1031). Codex sometimes paints an idle composer without its Context
@@ -10414,7 +10591,11 @@ export class Daemon extends EventEmitter {
             }
         }
         // ── MCP server env (dual-track: still passes env vars for MCP instructions fallback) ──
+        const configEnvironment = scopedFleetConfigEnvironment(join(this.instanceDir, "..", ".."));
         const mcpEnv = {
+            // Empty values clear an unrelated reference inherited by the CLI or tmux server.
+            AGEND_FLEET_DATA_DIR: configEnvironment.AGEND_FLEET_DATA_DIR ?? "",
+            AGEND_FLEET_CONFIG_PATH: configEnvironment.AGEND_FLEET_CONFIG_PATH ?? "",
             AGEND_SOCKET_PATH: sockPath,
             AGEND_INSTANCE_NAME: this.name,
             AGEND_WORKING_DIR: this.config.working_directory,
@@ -10437,6 +10618,11 @@ export class Daemon extends EventEmitter {
             ? resolveToolSet(this.config, this.name, "general")
             : resolveToolSet(this.config, this.name);
         mcpEnv.AGEND_TOOL_SET = toolSet;
+        // The dashboard features the person switched off: out of the tool menu from this start (the fleet refuses them
+        // at once, whatever the menu still shows).
+        const dashboardOff = dashboardFeaturesOff(this.config.dashboard);
+        if (dashboardOff.length)
+            mcpEnv.AGEND_DASHBOARD_OFF = dashboardOff.join(",");
         if (this.config.display_name)
             mcpEnv.AGEND_DISPLAY_NAME = this.config.display_name;
         if (this.config.description)
@@ -10469,6 +10655,7 @@ export class Daemon extends EventEmitter {
             catch { /* fallback to empty */ }
             instructions = buildFleetInstructions({
                 instanceName: this.name,
+                taskBoard: !dashboardOff.includes("task_board"),
                 workingDirectory: this.config.working_directory,
                 runtimeIdentity: this.runtimeIdentity,
                 displayName: this.config.display_name,
@@ -10483,6 +10670,7 @@ export class Daemon extends EventEmitter {
         else {
             instructions = buildFleetInstructions({
                 instanceName: this.name,
+                taskBoard: !dashboardOff.includes("task_board"),
                 workingDirectory: this.config.working_directory,
                 runtimeIdentity: this.runtimeIdentity,
                 displayName: this.config.display_name,
@@ -10585,6 +10773,7 @@ export class Daemon extends EventEmitter {
         this.clearReplyGuardConfirm();
         this.sawBlockedInput = false;
         if (this.spawnDepth === 0) {
+            this.cliOomObservation = null;
             this.spawnGeneration++;
             this.interactionObservation.reset(this.interactionOwner());
             this.clearInteractionConfirmation();
@@ -10888,6 +11077,10 @@ export class Daemon extends EventEmitter {
                 // is reachable again (a fresh prompt is local and proves nothing).
                 this.backendOutage?.clear(this.backendKey());
             }
+            // A pre-existing fresh fallback is not another resume of the old session.
+            // Providers may not have rewritten their session reading yet.
+            if (!resumedSuccessfully)
+                this.consecutiveOoms = null;
             this.lastSpawnAt = Date.now();
             // #1217: whatever session this CLI writes from here on is this backend's.
             this.recordSessionOwner();
@@ -11404,16 +11597,41 @@ export class Daemon extends EventEmitter {
             this.pendingWhatsNew = null;
             return null;
         }
-        if (ids.length === pending.ids.length)
-            return pending;
-        const narrowed = { notice: whatsNewNotice(pending.version, AGENT_WHATS_NEW.filter(e => ids.includes(e.id))), ids, version: pending.version };
+        // Rebuild even when ids are unchanged: an experimental switch may have changed
+        // while this delivery waited. Never paste the old enabled capability text.
+        // #2163: rebuilt as planned — only the entries this instance's roles are told; the rest are recorded with them.
+        const roles = this.whatsNewRoles();
+        const shown = agentWhatsNew(readCanvasWorkerEnabled(join(this.instanceDir, "..", ".."))).filter(e => ids.includes(e.id) && (!e.audience || e.audience.some(r => roles.includes(r))));
+        if (!shown.length) {
+            this.pendingWhatsNew = null;
+            try {
+                recordWhatsNew(this.instanceDir, ids, pending.version);
+            }
+            catch (err) {
+                this.logger.warn({ err }, "what's-new record not written — the notice may repeat once");
+            }
+            return null;
+        }
+        const narrowed = { notice: whatsNewNotice(pending.version, shown), ids, version: pending.version };
         this.pendingWhatsNew = narrowed;
         return narrowed;
     }
+    /**
+     * #2163: who this instance is to the what's-new notices — a ClassicBot room's agent, General, or a fleet worker (with
+     * `coordinator` too when its tool set is coordinator or full, the ones that carry get_/set_web_settings).
+     */
+    whatsNewRoles() {
+        if (this.runtimeIdentity?.kind === "classic")
+            return ["classic"];
+        if (this.config.general_topic === true || this.name === "general")
+            return ["general"];
+        const set = resolveToolSet(this.config, this.name);
+        return set === "coordinator" || set === "full" ? ["worker", "coordinator"] : ["worker"];
+    }
     /** #1633: what this instance has not been told about its AgEnD version yet (see agent-whats-new.ts). */
     prepareWhatsNew(fresh, version = agendVersion()) {
         try {
-            const plan = planWhatsNew({ record: readWhatsNewRecord(this.instanceDir), fresh, version });
+            const plan = planWhatsNew({ record: readWhatsNewRecord(this.instanceDir), fresh, version, roles: this.whatsNewRoles(), canvasWorker: readCanvasWorkerEnabled(join(this.instanceDir, "..", "..")) });
             if (!plan) {
                 this.pendingWhatsNew = null;
                 return;
@@ -11505,7 +11723,8 @@ export class Daemon extends EventEmitter {
         // AGEND_HOME points the child's agent-cli at the same data dir the daemon
         // is using, so it can locate <instanceDir>/agent.token.
         const agendHome = join(this.instanceDir, "..", "..");
-        let envPrefix = `TERM=xterm-256color AGEND_INSTANCE_NAME=${shellQuote(this.name)} AGEND_HOME=${shellQuote(agendHome)}`;
+        const configEnvironment = scopedFleetConfigEnvironment(agendHome);
+        let envPrefix = `TERM=xterm-256color AGEND_INSTANCE_NAME=${shellQuote(this.name)} AGEND_HOME=${shellQuote(agendHome)} AGEND_FLEET_DATA_DIR=${shellQuote(configEnvironment.AGEND_FLEET_DATA_DIR ?? "")} AGEND_FLEET_CONFIG_PATH=${shellQuote(configEnvironment.AGEND_FLEET_CONFIG_PATH ?? "")}`;
         if (backendConfig.agentMode === "cli" && backendConfig.agentPort) {
             envPrefix += ` AGEND_PORT=${backendConfig.agentPort}`;
         }
@@ -12228,7 +12447,7 @@ export class Daemon extends EventEmitter {
     }
     // ── Repo Checkout ─────────────────────────────────────────
     async handleCheckoutRepo(args, respond) {
-        const { execFile: execFileCb } = await import("node:child_process");
+        const { execFile: execFileCb } = await import("./child-process.js");
         const { promisify } = await import("node:util");
         const execFileAsync = promisify(execFileCb);
         const rawSource = args.source;
@@ -12264,7 +12483,7 @@ export class Daemon extends EventEmitter {
             respond(null, `Not a git repository: ${source}`);
             return;
         }
-        mkdirSync(join(this.instanceDir, "repos"), { recursive: true });
+        mkdirSync(join(this.instanceDir, "repos"), { recursive: true, mode: 0o700 });
         // Only a fresh direct child of a real repos/ directory (never through a symlinked repos/, never an existing path).
         const target = checkoutTarget(this.instanceDir, source, branch);
         if (!target.ok) {
@@ -12306,7 +12525,7 @@ export class Daemon extends EventEmitter {
             respond(null, `Cannot release: ${root.reason}`);
             return;
         }
-        const { execFile: execFileCb } = await import("node:child_process");
+        const { execFile: execFileCb } = await import("./child-process.js");
         const { promisify } = await import("node:util");
         const execFileAsync = promisify(execFileCb);
         const git = async (gitArgs, cwd) => (await execFileAsync("git", gitArgs, { cwd })).stdout;