dsh-turn-recovery 0.2.2 → 0.2.3

package/CHANGELOG.md

@@ -7,6 +7,50 @@
 > `@nlflora/dsh-empty-turn-guard`）冲突，自 0.2.0 起更名为 `dsh-turn-recovery`。0.1.x 及更早的条目保留旧名原文，
 > 其提到的 `plugin:empty-turn-guard`、`empty-turn-guard.log` 等标识在 0.2.0 起分别改为 `plugin:turn-recovery`、`turn-recovery.log`。
 
+## [0.2.3] - 2026-11-05
+
+**三方复核（架构 / 安全 / 验收）后的加固版**。0.2.2 的判定链没有 P0 缺陷，但复核在
+持久化文件与部署侧查出实质问题：预算文件可被本地输入拖垮或投毒、跨重启的补写 id 记忆
+不落盘、段落注册一次冲突就让插件永久停摆、inbox 去重依赖未公开结构却没有探测。
+本版逐项修掉，并补齐 README 里对「桌面端 vs 网页端」的错误期待。
+
+### 修复
+
+- **预算文件同步整读会阻塞宿主**（安全复核实测 64 MB ≈ 544 ms）：`hydrateBudget` 改为先
+  `stat`，超过 1 MiB 只读尾部并丢掉截断的首行，日志留一行
+  `persisted state read: file is …B, tail …B only`。
+- **预算文件可被未来时间戳投毒**：一条 `at > now` 的记录会让 `now - t < HOUR_MS` 恒成立、
+  永久留在预算里，该会话从此被判成「本小时预算用尽」而静默不补写。解析与修剪统一加
+  `+5min` 时钟容忍，超出即丢弃。
+- **预算记录格式改为 JSON 行**：旧格式 `id at` 在会话 id 含空格时会被截断成错账；
+  新格式兼容读旧文件（两种都认）。
+- **压缩预算文件会覆盖别的进程刚追加的行**：改为「重读磁盘 → 合并 → 临时文件 → `rename`
+  原子替换」，并把写入统一成 append-only。
+- **跨重启的补写 id 不落盘**：新增 `<budgetFile 同目录>/guard-ids.log`（JSON 行，
+  24 小时 TTL、每会话 8 条）。宿主重启后认领「重启前投递」的那条消息时仍记为
+  `LATE, from guardIds`，补写回合再空不会连环补第二笔。
+- **段落注册一次失败就永久停摆**：改为最多重试 3 次（150 ms 间隔），只有注册成功才
+  `armed`；重试耗尽仍 fail-closed（`guard NOT armed`），期间所有判定链不工作
+  （`session/event` 与 `deliver` 都先看 `armed`）。
+- **`agent.inbox.nextTurn` 去重依赖未公开结构**：加能力探测，缺失时记一行
+  `inbox dedupe UNAVAILABLE …` 并降级（认领 id 匹配仍在），不再静默失效。
+- **`skip scope` 日志上限**：200 → 2000，且触顶时明确记一行，不再无声无息。
+
+### 变更
+
+- `nudge DRY-RUN` 日志补 `budgetUsed=n/max`，并注明干跑不落预算 —— 旧文案容易让人以为
+  `dryRun` 能验证限流。
+- README 新增「桌面端与网页端」章节：内嵌 Web UI 与桌面端同进程同实例；独立
+  `dsh web` 在 `profiles/web` 不存在时走出厂模板，插件静默不加载。
+- 回归用例 49 → 58：预算尾部读取、未来时间戳、JSON 落盘、跨重启认领、inbox 探测降级、
+  段落注册重试成功，以及 `enabled:false` / `injectInstruction:false` 三条开关语义。
+
+### 未变
+
+- 补写消息仍是 `role: 'user'` + `source.kind = 'plugin:turn-recovery'`（宿主不允许插件改 role）。
+- 真实宿主上的端到端投递仍未观测到（生产日志 `nudge DELIVERED = 0`）。本版只加固了
+  与「能不能安全地投递」有关的边角，闭合这一项仍需要在真机上跑一次 `dryRun` + 一次真实投递。
+
 ## [0.2.2] - 2026-11-04
 
 **代码审查修复 + 可验证性补强**。针对 0.2.1 的复审（宿主 `0.2.2-alpha.3` 源码取证、

package/README.md

@@ -31,7 +31,7 @@
 // <DSH_HOME>/profiles/<你的 profile>/package.json
 {
   "dependencies": {
-    "dsh-turn-recovery": "^0.2.1"
+    "dsh-turn-recovery": "^0.2.3"
   },
   "dsh": {
     "profile": {
@@ -65,6 +65,7 @@
     sessionScope: root       # root = 只处理根会话（按 SessionHeader 判定）；all = 连子代理会话一起
     foregroundKinds: []      # 追加「会取消待发补写」的来源 kind（默认 user / agent-message）
     budgetFile: ''           # 空 = <DSH_HOME>/state/turn-recovery/nudges.log（跨重启限流）
+                             # 同目录还会写 guard-ids.log（跨重启认出晚到认领的补写 id）
     logFile: ''              # 空 = <DSH_HOME>/logs/turn-recovery.log
     dryRun: false            # true = 判定/限流/日志全部照走，但不真的投递（验证用）
 ```
@@ -73,6 +74,18 @@
 
 宿主**不校验**插件配置（本包不导出 `Config` schema，原因见「实现约束」），所以键名写错不会被谁拦下来。插件改为在加载期自查：未知键、类型不符、`sessionScope` 取值不对等，逐条写进日志（`config issue: "minNudgesPerHour" is unknown -> ignored`），插件照常启用。改完配置先看这几行，比猜「为什么没生效」划算。
 
+## 桌面端与网页端
+
+两者不是两条独立的加载路径，别按「装一份、两边都生效」理解：
+
+- **桌面应用里内嵌的 Web UI（浏览器打开的那个端口）与桌面端是同一个宿主进程、同一个 profile、同一个插件实例**。插件对它是生效的，不需要额外注册。
+- **独立运行 `dsh web`（等价于 `--profile web`）** 时，如果 `<DSH_HOME>/profiles/web` 不存在，宿主会用出厂模板初始化该 profile（只含 `dsh-base` + `dsh-web-app`）。**本插件不在其中，网页端会静默无保护** —— 既不加载、也不报错，连一条 `skip scope` 都不会有。
+- `headless` profile 同理（模板只含 base + headless）。
+
+需要独立 web/headless 宿主时，显式注册三处（与 desktop profile 一样）：`profiles/<name>/package.json` 的 `dependencies` 加上本包、`dsh.profile.bundles` 加上 `dsh-turn-recovery`、`profiles/<name>/cordis.patch.yml` 加 `- id: turn-recovery` 的 config；装完用 `dsh --profile web --dump-config | grep dsh-turn-recovery` 确认条目真的在。
+
+> 2026-11-05 本机实测：`<DSH_HOME>/profiles/` 只有 `desktop` 与 `headless`，没有 `web`。当时浏览器里那个端口由桌面宿主进程提供服务（端口 owner 与日志里 `plugin activated` 的 pid 相同）。
+
 ## 日志
 
 默认写到 `<DSH_HOME>/logs/turn-recovery.log`（未设 `DSH_HOME` 时是 `~/.dsh/logs/…`；主路径不可写时兜底到系统临时目录；超过 5 MB 自动轮转为 `.1`）：
@@ -116,7 +129,10 @@
 - **用户点停止的窗口**：宿主 `cancel()` 只清 inbox + abort，不 append 任何 session 事件。插件订阅了宿主 dispatch 的 `agent/inbox/discarded` 来撤掉自己那条补写，但该事件是否向插件广播取决于宿主的 dispatch 链路；收不到时，「用户已点停止、插件仍在 `delayMs` 后补写」这个窗口依然存在（默认 900ms，可通过 `delayMs` 调小）。真正发出去的那一回合仍受宿主 abort 语义约束。
 - **`sessionScope: 'root'` 的判定依据是宿主的 `Session.header`**：`origin === 'subagent'` 或 `delegationDepth > 0` 才算子代理会话，`orchestrator-*` 这类顶层会话会被正常处理（0.2.0 用 id 前缀判断，把它们全部判成子代理，实测日志里有 19 次静默跳过）。宿主没有 header 的场合（老宿主、测试桩）退回 id 形态：`session-*` / `orchestrator-*` 算根，纯 UUID / `social-*` 不算。被过滤的会话每个 id 只记一条 `skip scope` 日志，不会静默消失。
 - **可见字符是「按回合累计」而不是「末条消息」**：一个回合里先写了结论、后来又被工具调用打断的，累计口径下不算空（这是刻意的：用户此前已经看到了文字）。反过来，`aborted` / `interrupted` 等非 `completed` 回合一律不补写，即使它确实没留下正文 —— 那些回合多半是用户自己喊停的，再补一轮会变成「已取消还继续跑」。
-- **崩溃遗留**：补写已入队但宿主在认领前崩溃时，该消息可能在下次启动被重放。插件只认 id 匹配的认领（`NOT ours`），因此不会被这种消息骗成「补写回合」，但它仍可能作为一条插件消息出现在转录里。为覆盖「120s 认领超时后才被认领」的情形，最近 8 条投递 id 会留在内存里（`LATE, from guardIds`），进程重启后这份记忆就没了。
+- **崩溃遗留**：补写已入队但宿主在认领前崩溃时，该消息可能在下次启动被重放。插件只认 id 匹配的认领（`NOT ours`），因此不会被这种消息骗成「补写回合」，但它仍可能作为一条插件消息出现在转录里。为覆盖「120s 认领超时后才被认领」与「重启后才被认领」两种情形，最近 8 条投递 id 会同时留在内存与 `<budgetFile 同目录>/guard-ids.log` 里（`LATE, from guardIds`），重启后仍能认出它们，不会把补写回合再当成真空回合补第二笔。
+- **预算文件是不可信输入**：解析时会丢弃「晚于当前时钟 +5 分钟」的时间戳（否则 `now - t` 为负、该会话被永久判成预算耗尽），并且整份文件只在 1 MiB 以内整读、超过就只读尾部（避免配置写错指向几百 MB 的文件时同步阻塞宿主）。写入用 JSON 行（会话 id 含空格也不会被截断），压缩是「重读磁盘 + 临时文件原子替换」，不会覆盖另一个进程刚追加的行。
+- **多进程（桌面宿主 + 独立的 web/headless 宿主）**：各自一套内存态，`agent.inbox.nextTurn` 只对本进程可见。预算文件与补写 id 文件是共享的，因此限流大致成立，但「同一会话同时被两个宿主打开」仍可能各补一笔。同一会话请只用一个宿主。
+- **`dryRun` 不写预算文件**：判定链、日志、限流检查照走，但不投递、不记账，所以它只能证明「判定会发生」，不能用来验证限流是否生效（日志里的 `budgetUsed=n/max` 是当时的已用额度）。
 - **多宿主共用同一日志文件**：两个进程同时写同一路径时轮转可能失败，此时回退 `truncate(0)` 并记一条 `log rotate fell back to truncate`。
 - **生产环境仍未观测到真实补写**：截至 2026-11-04，本机生产日志里 `nudge DELIVERED` 仍为 0（空回合全部是 `aborted` / `blocked`），所有结论来自单测与 mock 复现。要闭合这一项，请在真机上临时设置 `dryRun: true`，等一次 `completed` 空回合，确认日志出现 `nudge DRY-RUN: would deliver`（判定链在生产成立），再关掉 `dryRun` 等真实投递 —— 这比直接开补写更安全：干跑不会多跑一轮模型。
 

package/lib/index.js

@@ -91,6 +91,23 @@ const HOUR_MS = 3600000;
  * 避免一个丢失的补写永久占住 nudgeMessageId。
  */
 const NUDGE_CLAIM_TIMEOUT_MS = 120000;
+/**
+ * 持久化时间戳的时钟偏移容忍（毫秒）。
+ * 预算文件与补写 id 文件都是本地可写文件，里面的时间戳不能当可信输入：
+ * 一条明显晚于当前时钟的记录会让 `now - t` 变成负数，`< HOUR_MS` 恒成立，
+ * 于是该会话被永久判成本小时预算耗尽、守卫静默失效。晚于 now + 容忍窗口的
+ * 记录一律丢弃（不报错、不中断，只丢那一条）。
+ */
+const BUDGET_FUTURE_SKEW_MS = 5 * 60 * 1000;
+/** 已投递补写 id 的持久化保留时长：覆盖宿主重启/插件热重载后的晚到认领。 */
+const GUARD_ID_TTL_MS = 24 * 60 * 60 * 1000;
+/** 每个会话保留的补写 id 上限（与内存态 guardIds 一致）。 */
+const GUARD_IDS_MAX = 8;
+/** prompt 段落注册失败后的重试次数与间隔（热重载时旧段落可能还没 dispose）。 */
+const SECTION_RETRY_MAX = 3;
+const SECTION_RETRY_DELAY_MS = 150;
+/** 每个会话只记一条 skip scope 日志；上限给足，避免长跑后新会话不再留痕。 */
+const SCOPE_SEEN_MAX = 2000;
 /** 单个日志文件上限，超过就轮转成 .1（appendFileSync 没有上限，长期跑会无限长）。 */
 const LOG_MAX_BYTES = 5 * 1024 * 1024;
 const LOG_ROTATE_CHECK_MS = 60000;
@@ -371,29 +388,134 @@ export function apply(ctx, config) {
     ? cfg.budgetFile
     : path.join(dshHome, 'state', 'turn-recovery', 'nudges.log');
   const BUDGET_MAX_BYTES = 1024 * 1024;
+  /** 已投递补写 id 的持久化文件（跨重启识别晚到认领用），与预算文件同目录。 */
+  const guardPath = path.join(path.dirname(budgetPath), 'guard-ids.log');
   let budgetHydrated = false;
   /** session.id -> 最近一小时内的投递时刻（只在本进程内累计）。 */
   const budgetHistory = new Map();
+  /** session.id -> 已投递补写 id（跨重启），只用于认出晚到认领，不参与判定。 */
+  const guardIdHistory = new Map();
+  let guardHydrated = false;
+
+  /**
+   * 只读文件尾部 maxBytes。
+   * 旧版直接 readFileSync 整个 budgetFile：配置写错或文件被替换成几百 MB 时，
+   * 这一行会同步卡住宿主的事件循环（实测 64 MB ≈ 544 ms，线性放大）。
+   * 现在先 stat，超上限只读尾部；截断点落在行中间时丢掉首个不完整行。
+   */
+  const readTail = (file, maxBytes) => {
+    const st = fs.statSync(file);
+    if (!st.isFile()) return '';
+    if (st.size <= maxBytes) return fs.readFileSync(file, 'utf8');
+    const fd = fs.openSync(file, 'r');
+    try {
+      const buf = Buffer.allocUnsafe(maxBytes);
+      const read = fs.readSync(fd, buf, 0, maxBytes, st.size - maxBytes);
+      let text = buf.subarray(0, read).toString('utf8');
+      const nl = text.indexOf('\n');
+      if (nl >= 0) text = text.slice(nl + 1);
+      log('persisted state read: file is ' + st.size + 'B, tail ' + maxBytes + 'B only ('
+        + file + ')');
+      return text;
+    } finally { fs.closeSync(fd); }
+  };
+
+  /**
+   * 解析一行预算记录。新格式是 JSON（id 里有空格也不会被截断），
+   * 0.2.2 及更早写的是 "id at" 纯文本，两者都认。
+   * 时间戳晚于当前时钟 + 容忍窗口的丢弃。
+   */
+  const parseBudgetLine = (line, now, cutoff) => {
+    const text = line.trim();
+    if (text === '') return null;
+    let id = '';
+    let at = NaN;
+    if (text.charCodeAt(0) === 123 /* '{' */) {
+      let rec;
+      try { rec = JSON.parse(text); } catch { return null; }
+      if (!rec || typeof rec.id !== 'string') return null;
+      id = rec.id;
+      at = Number(rec.at);
+    } else {
+      const sp = text.indexOf(' ');
+      if (sp <= 0) return null;
+      id = text.slice(0, sp);
+      at = Number(text.slice(sp + 1).trim());
+    }
+    if (id === '' || !Number.isFinite(at)) return null;
+    if (at > now + BUDGET_FUTURE_SKEW_MS) return null;
+    if (at < cutoff) return null;
+    return { id, at };
+  };
 
   const hydrateBudget = () => {
     if (budgetHydrated) return;
     budgetHydrated = true;
     try {
-      const rawText = fs.readFileSync(budgetPath, 'utf8');
-      const cutoff = Date.now() - HOUR_MS;
+      const rawText = readTail(budgetPath, BUDGET_MAX_BYTES);
+      const now = Date.now();
+      const cutoff = now - HOUR_MS;
       for (const line of rawText.split('\n')) {
-        const sp = line.indexOf(' ');
-        if (sp <= 0) continue;
-        const id = line.slice(0, sp);
-        const at = Number(line.slice(sp + 1).trim());
-        if (id === '' || !Number.isFinite(at) || at < cutoff) continue;
-        const list = budgetHistory.get(id);
-        if (list) list.push(at);
-        else budgetHistory.set(id, [at]);
+        const rec = parseBudgetLine(line, now, cutoff);
+        if (!rec) continue;
+        const list = budgetHistory.get(rec.id);
+        if (list) list.push(rec.at);
+        else budgetHistory.set(rec.id, [rec.at]);
       }
     } catch { /* 文件不存在或读不了：当作没有历史 */ }
   };
 
+  /**
+   * 已投递补写 id 的持久化。
+   * 宿主重启/插件热重载后内存态 guardIds 是空的，于是「重启前投递、重启后认领」
+   * 的那条消息会被判成 NOT ours，紧接着那个补写回合结束时空正文依然为空，
+   * 插件就把它当成真空回合再补一笔（跨重启的连环补写）。
+   * 这里只解决「认得出晚到认领」，不参与是否补写的判定。
+   */
+  const hydrateGuardIds = () => {
+    if (guardHydrated) return;
+    guardHydrated = true;
+    try {
+      const rawText = readTail(guardPath, BUDGET_MAX_BYTES);
+      const now = Date.now();
+      const floor = now - GUARD_ID_TTL_MS;
+      for (const line of rawText.split('\n')) {
+        const text = line.trim();
+        if (text === '') continue;
+        let rec;
+        try { rec = JSON.parse(text); } catch { continue; }
+        if (!rec || typeof rec.id !== 'string' || typeof rec.sid !== 'string') continue;
+        const at = Number(rec.at);
+        if (!Number.isFinite(at) || at < floor || at > now + BUDGET_FUTURE_SKEW_MS) continue;
+        const list = guardIdHistory.get(rec.sid);
+        if (!list) guardIdHistory.set(rec.sid, [rec.id]);
+        else if (list.indexOf(rec.id) < 0) list.push(rec.id);
+      }
+    } catch { /* 无文件：当作没有历史 */ }
+  };
+
+  /** 该会话跨重启保留的补写 id（最多 GUARD_IDS_MAX 条）。 */
+  const persistedGuardIds = (session) => {
+    hydrateGuardIds();
+    const id = session && typeof session.id === 'string' ? session.id : '';
+    const list = id !== '' ? guardIdHistory.get(id) : undefined;
+    if (!list || list.length === 0) return [];
+    return list.slice(-GUARD_IDS_MAX);
+  };
+
+  const recordGuardId = (session, id, at) => {
+    const sid = session && typeof session.id === 'string' ? session.id : '';
+    if (sid === '' || typeof id !== 'string' || id === '') return;
+    try {
+      let list = guardIdHistory.get(sid);
+      if (!list) { list = []; guardIdHistory.set(sid, list); }
+      if (list.indexOf(id) < 0) list.push(id);
+      while (list.length > GUARD_IDS_MAX) list.shift();
+      fs.mkdirSync(path.dirname(guardPath), { recursive: true });
+      fs.appendFileSync(guardPath, JSON.stringify({ sid, id, at }) + '\n');
+    } catch { /* 只服务于去重，写失败不影响补写本身 */ }
+  };
+
   /** 该会话最近一小时的投递时刻（合并持久化记录与本次进程内的记录）。 */
   const historyOf = (session) => {
     hydrateBudget();
@@ -409,35 +531,57 @@ export function apply(ctx, config) {
    * 长时间运行（每会话每小时若干条）会只增不减（P2-2）。
    */
   let recordTicks = 0;
+
+  const pruneBudgetHistory = (now) => {
+    const floor = now - HOUR_MS;
+    for (const [sid, times] of budgetHistory) {
+      const kept = times.filter((t) => t >= floor && t <= now + BUDGET_FUTURE_SKEW_MS);
+      if (kept.length === 0) budgetHistory.delete(sid);
+      else if (kept.length !== times.length) budgetHistory.set(sid, kept);
+    }
+  };
+
+  /**
+   * 预算文件压缩：先重读磁盘，再原子替换。
+   * 旧版直接用内存 map 覆盖写整份文件 —— 多进程（桌面宿主 + 独立 web/headless）
+   * 同时运行时，会把对方刚追加的行整段抹掉，预算就少算、于是多补。
+   */
+  const compactBudget = (now) => {
+    try {
+      const rawText = readTail(budgetPath, BUDGET_MAX_BYTES);
+      const cutoff = now - HOUR_MS;
+      for (const line of rawText.split('\n')) {
+        const rec = parseBudgetLine(line, now, cutoff);
+        if (!rec) continue;
+        const list = budgetHistory.get(rec.id);
+        if (!list) budgetHistory.set(rec.id, [rec.at]);
+        else if (list.indexOf(rec.at) < 0) list.push(rec.at);
+      }
+    } catch { /* 读不到就只用内存态 */ }
+    pruneBudgetHistory(now);
+    const lines = [];
+    for (const [sid, times] of budgetHistory) {
+      for (const t of times) lines.push(JSON.stringify({ id: sid, at: t }));
+    }
+    const tmp = budgetPath + '.tmp-' + process.pid;
+    fs.writeFileSync(tmp, lines.length > 0 ? lines.join('\n') + '\n' : '');
+    fs.renameSync(tmp, budgetPath);
+  };
+
   const recordNudge = (session, at) => {
     const id = session && typeof session.id === 'string' ? session.id : '';
     if (id === '') return;
     try {
       if (!budgetHistory.has(id)) budgetHistory.set(id, []);
       budgetHistory.get(id).push(at);
-      const cutoff = Date.now() - HOUR_MS;
       recordTicks += 1;
-      let rewrite = false;
-      try { rewrite = fs.statSync(budgetPath).size > BUDGET_MAX_BYTES; } catch { /* 首次写入 */ }
-      // 每 256 次投递做一次常规修剪：不写文件，只清内存里的过期条目和空会话。
-      // 投递受 maxNudgesPerHour 限制，256 次意味着至少几十个会话满额，频率够低。
-      if (rewrite || recordTicks % 256 === 0) {
-        for (const [sid, times] of budgetHistory) {
-          const kept = times.filter((t) => t >= cutoff);
-          if (kept.length === 0) budgetHistory.delete(sid);
-          else if (kept.length !== times.length) budgetHistory.set(sid, kept);
-        }
-      }
       fs.mkdirSync(path.dirname(budgetPath), { recursive: true });
-      if (rewrite) {
-        const lines = [];
-        for (const [sid, times] of budgetHistory) {
-          for (const t of times) lines.push(sid + ' ' + t);
-        }
-        fs.writeFileSync(budgetPath, lines.join('\n') + (lines.length > 0 ? '\n' : ''));
-      } else {
-        fs.appendFileSync(budgetPath, id + ' ' + at + '\n');
-      }
+      // 只追加、不改写：改写是 read-modify-write，多进程下会覆盖别人刚写的行。
+      fs.appendFileSync(budgetPath, JSON.stringify({ id, at }) + '\n');
+      let size = 0;
+      try { size = fs.statSync(budgetPath).size; } catch { /* 首次写入 */ }
+      if (size > BUDGET_MAX_BYTES) compactBudget(Date.now());
+      else if (recordTicks % 256 === 0) pruneBudgetHistory(Date.now());
     } catch { /* 限流退化到内存态，不影响补写本身 */ }
   };
 
@@ -473,6 +617,7 @@ export function apply(ctx, config) {
    * 「被 scope 静默过滤」在日志里长得一模一样，排障全靠猜。
    */
   const scopeSeen = new Set();
+  let scopeSeenCapped = false;
 
   /** header 判定根会话；返回 null 表示无法判定（没有 header）。 */
   const rootByHeader = (session) => {
@@ -489,7 +634,12 @@ export function apply(ctx, config) {
   const logScopeSkip = (session, why) => {
     const id = session && typeof session.id === 'string' ? session.id : '';
     if (id === '' || scopeSeen.has(id)) return;
-    if (scopeSeen.size < 200) scopeSeen.add(id);
+    if (scopeSeen.size < SCOPE_SEEN_MAX) {
+      scopeSeen.add(id);
+    } else if (!scopeSeenCapped) {
+      scopeSeenCapped = true;
+      log('skip scope log capped at ' + SCOPE_SEEN_MAX + ' sessions; further skips are silent');
+    }
     log('skip scope: non-root session ' + id.slice(0, 12) + ' (sessionScope=root, ' + why + ')');
   };
 
@@ -517,39 +667,46 @@ export function apply(ctx, config) {
   }
 
   // ---- A) system prompt 段落 ----
-  if (cfg.injectInstruction) {
-    try {
-      const text = cfg.instructionText.trim() !== '' ? cfg.instructionText : DEFAULT_INSTRUCTION;
-      const dispose = ctx.systemPrompt.section({
-        name: SECTION_NAME,
-        order: SECTION_ORDER,
-        text,
-        interpolate: false,
-      });
-      if (typeof dispose === 'function') {
-        try {
-          ctx.effect(() => () => { try { dispose(); } catch { /* ignore */ } });
-        } catch { /* effect 不可用时忽略 */ }
-      }
-      log('prompt section registered: order=' + SECTION_ORDER + ' chars=' + text.length);
-    } catch (err) {
-      // fail-closed：A 段装不上时不再启用补写，也不再写「guard armed」。
-      // 宿主对同名 section 重复注册是直接 throw（dsh-system-prompt 的 section
-      // 注册会拒绝重名），最常见的触发原因是同一插件被两个 bundle 各装了一次。
-      // 此时若照常 arm：两个实例各发一条补写；日志却写着「一切正常」，
-      // 而「222 字符的收尾指令其实没进 prompt」这个事实在日志里看不出来。
-      log('prompt section registration FAILED: ' + errText(err));
-      log('guard NOT armed (fail-closed: prompt section registration failed)');
-      return;
+  /**
+   * 注册收尾指令段落。宿主对同名 section 重复注册会直接 throw，
+   * 最常见的触发是「同一插件被两个 bundle 各装一次」，其次才是热重载时
+   * 旧段落还没 dispose。所以这里只负责「试一次」，重试与 fail-closed 在下面 arm 处。
+   */
+  const registerPromptSection = () => {
+    const text = cfg.instructionText.trim() !== '' ? cfg.instructionText : DEFAULT_INSTRUCTION;
+    const dispose = ctx.systemPrompt.section({
+      name: SECTION_NAME,
+      order: SECTION_ORDER,
+      text,
+      interpolate: false,
+    });
+    if (typeof dispose === 'function') {
+      try {
+        ctx.effect(() => () => { try { dispose(); } catch { /* ignore */ } });
+      } catch { /* effect 不可用时忽略 */ }
     }
-  }
+    log('prompt section registered: order=' + SECTION_ORDER + ' chars=' + text.length);
+  };
 
   if (!cfg.enabled) {
-    log('guard disabled by config; only instruction part active');
+    if (cfg.injectInstruction) {
+      try {
+        registerPromptSection();
+      } catch (err) {
+        log('prompt section registration FAILED: ' + errText(err));
+        log('guard NOT armed (fail-closed: prompt section registration failed)');
+        return;
+      }
+    }
+    log(cfg.injectInstruction
+      ? 'guard disabled by config; only instruction part active'
+      : 'guard disabled by config and injectInstruction=false; nothing active');
     return;
   }
 
   // ---- C) 空回合保护 ----
+  /** 段落注册成功才置 true；置位前所有判定链都不工作（fail-closed）。 */
+  let armed = false;
   const agents = new Map(); // session -> agent
   const agentsById = new Map(); // session.id -> agent（session 对象身份不稳定时兜底）
   const state = new WeakMap(); // session -> per-session guard state
@@ -595,7 +752,7 @@ export function apply(ctx, config) {
          * 那笔「超时」的消息；靠这个列表把晚到的认领仍算作补写回合，
          * 否则第二个空回合会被当成真空回合再补一笔。
          */
-        guardIds: [],
+        guardIds: persistedGuardIds(session),
         /**
          * 定时器是为哪个回合排的；回合变了就该丢弃。
          * 与 turn/end 的清零无关：turn/end 先清 s.turn 之外的字段不该顺手把它清空，
@@ -788,16 +945,52 @@ export function apply(ctx, config) {
    * 各装一次时它都是空的，于是会对同一个空回合各投一笔，模型连着被唤醒两次。
    * 队列本身是宿主维护的真相（agent.inbox.nextTurn），以它为准去重。
    */
-  const pendingNudgeInInbox = (agent) => {
+  /**
+   * 宿主「下一回合队列」的能力探测。
+   * `agent.inbox.nextTurn` 是宿主当前实现里的公开字段，但插件不该假设它永远在：
+   * 宿主把它改私有/改名/换实现时，旧写法会安静地恒返回 false —— 去重失效、
+   * 重复补写，日志里却没有任何线索。这里探测一次并留下明确的一行。
+   */
+  let inboxProbe = 'unknown';
+
+  const nextTurnQueue = (agent) => {
+    if (!agent) return null;
     try {
-      const inbox = agent && agent.inbox;
+      const inbox = agent.inbox;
       const queue = inbox && Array.isArray(inbox.nextTurn) ? inbox.nextTurn : null;
-      if (!queue) return false;
-      return queue.some((m) => m && m.source && m.source.kind === SOURCE_KIND);
-    } catch { return false; }
+      if (!queue) {
+        if (inboxProbe !== 'unavailable') {
+          inboxProbe = 'unavailable';
+          log('inbox dedupe UNAVAILABLE: host no longer exposes agent.inbox.nextTurn'
+            + ' -> cross-instance dedupe degraded (claim-id matching still applies)');
+        }
+        return null;
+      }
+      if (inboxProbe !== 'available') {
+        inboxProbe = 'available';
+        log('inbox dedupe available (agent.inbox.nextTurn)');
+      }
+      return queue;
+    } catch (err) {
+      if (inboxProbe !== 'unavailable') {
+        inboxProbe = 'unavailable';
+        log('inbox dedupe UNAVAILABLE: probe threw ' + errText(err));
+      }
+      return null;
+    }
+  };
+
+  const pendingNudgeInInbox = (agent) => {
+    const queue = nextTurnQueue(agent);
+    if (!queue) return false;
+    return queue.some((m) => m && m.source && m.source.kind === SOURCE_KIND);
   };
 
   const deliver = (session, text) => {
+    if (!armed) {
+      log('nudge SKIPPED [' + tag(session) + ']: guard not armed (prompt section missing)');
+      return null;
+    }
     const agent = agentOf(session);
     if (!agent) {
       log('nudge SKIPPED [' + tag(session) + ']: no live agent bound to session');
@@ -830,8 +1023,14 @@ export function apply(ctx, config) {
     return null;
   };
 
+  /**
+   * 修剪本会话最近一小时的投递时刻。
+   * 未来时间戳（> now + 容忍窗口）必须一起丢掉：`now - t` 对它是负数，
+   * `< HOUR_MS` 恒成立，只按上半句过滤就会把它永久留在预算里。
+   */
   const pruneBudget = (s, now) => {
-    s.nudgeTimes = s.nudgeTimes.filter((t) => now - t < HOUR_MS);
+    s.nudgeTimes = s.nudgeTimes.filter(
+      (t) => now - t < HOUR_MS && t <= now + BUDGET_FUTURE_SKEW_MS);
     return s.nudgeTimes.length;
   };
 
@@ -894,8 +1093,11 @@ export function apply(ctx, config) {
           if (cfg.dryRun) {
             // 干跑：判定链、限流、日志全部照走，只是不把消息交给宿主。
             // 这样可以在真实宿主里验证「会判空 → 会想补」而不产生任何模型回合。
-            log('nudge DRY-RUN: would deliver turn=' + turn + ' delay=' + delay + 'ms ['
-              + tag(session) + ']');
+            // 但 dryRun 只做预算检查、不记账也不落盘，所以它验证不了「限流真的生效」；
+            // 日志里带上 budgetUsed，至少能读出当前窗口已经用掉多少。
+            log('nudge DRY-RUN: would deliver turn=' + turn + ' delay=' + delay
+              + 'ms budgetUsed=' + s.nudgeTimes.length + '/' + cfg.maxNudgesPerHour
+              + ' (no delivery, no budget persist) [' + tag(session) + ']');
             return;
           }
           const message = deliver(session, NUDGE_TEXT);
@@ -911,6 +1113,8 @@ export function apply(ctx, config) {
             }
             // 跨重启限流：把投递时刻落盘，重启后的新进程回读最近一小时。
             recordNudge(session, s.nudgeDeliveredAt);
+            // 跨重启认领：把这笔 id 落盘，重启后宿主认领它时仍能认出是补写回合。
+            recordGuardId(session, message.id, s.nudgeDeliveredAt);
             log('nudge awaiting claim id=' + String(message.id).slice(0, 8) + ' [' + tag(session) + ']');
           } else {
             clearNudgeInFlight(s, null);
@@ -934,6 +1138,9 @@ export function apply(ctx, config) {
   try {
     ctx.on('session/event', (session, event) => {
       try {
+        // arm 之前（段落还没注册成功）不判定、不补写：宁可少补，也不能在
+        // 「收尾指令没进 prompt」的状态下往会话里塞自动消息。
+        if (!armed) return;
         if (!session || !event || typeof event.type !== 'string') return;
         if (!sessionInScope(session)) return;
         const data = event.data || {};
@@ -1087,8 +1294,40 @@ export function apply(ctx, config) {
     });
   } catch { /* effect 不可用时忽略 */ }
 
-  log('guard armed (instruction=' + (cfg.injectInstruction ? 'registered' : 'disabled')
-    + ', uptime=' + (Date.now() - startedAt) + 'ms)');
+  // ---- A) 段落注册 + arm ----
+  // 注册放在最后：热重载时旧段落可能还没 dispose，注册会抛；这里重试几次，
+  // 只有注册成功才 arm —— 「段落没进 prompt 就不补写」的 fail-closed 语义不变，
+  // 但不再因为一次瞬时冲突让整个插件永久停摆。
+  const armGuard = () => {
+    armed = true;
+    log('guard armed (instruction=' + (cfg.injectInstruction ? 'registered' : 'disabled')
+      + ', uptime=' + (Date.now() - startedAt) + 'ms)');
+  };
+  const tryArm = (attempt) => {
+    if (!cfg.injectInstruction) {
+      armGuard();
+      return;
+    }
+    try {
+      registerPromptSection();
+      armGuard();
+    } catch (err) {
+      log('prompt section registration FAILED (attempt ' + (attempt + 1) + '/'
+        + (SECTION_RETRY_MAX + 1) + '): ' + errText(err));
+      if (attempt < SECTION_RETRY_MAX) {
+        const retry = setTimeout(() => {
+          timers.delete(retry);
+          tryArm(attempt + 1);
+        }, SECTION_RETRY_DELAY_MS);
+        if (retry && typeof retry.unref === 'function') retry.unref();
+        timers.add(retry);
+        return;
+      }
+      log('guard NOT armed (fail-closed: prompt section registration failed after '
+        + (SECTION_RETRY_MAX + 1) + ' attempts)');
+    }
+  };
+  tryArm(0);
 }
 
 export default { name, inject, apply };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-turn-recovery",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "DSH plugin: detect assistant turns that end with no visible text (zero-width placeholder) and re-prompt the model to write the answer; also injects a visible-closing-answer system prompt rule.",
   "license": "MIT",
   "author": "HERO476",