dsh-galatea 0.4.4 → 0.4.5

package/README.md

@@ -13,7 +13,7 @@
 | 块 | 状态 |
 |---|---|
 | 能被 DSH 装起来 | ✅ 实测通过（dsh 0.2.2-rc.3） |
-| 发布到 npm | ✅ 0.4.4（GitHub Actions 自动发，不需要 token） |
+| 发布到 npm | ✅ 0.4.5（GitHub Actions 自动发，不需要 token） |
 | 记忆：读写 Markdown | ✅ 15 条测试 |
 | 记忆：注入索引 | ✅ **已核实内容真的进了上下文**（不是在会话日志里看到「接上了没告警」） |
 | 记忆：`remember` / `recall` | ✅ 23 条测试 |
@@ -28,6 +28,7 @@
 | 心跳：听得见你说话 | ⚠️ 接了 `agent/pre-step`，**但那条事件有没有真的触发，没验过** |
 | 设置页可改的字段 | ⚠️ 声明被宿主接受，**表单没在 GUI 里看过** |
 | 心跳：定时器自己跑 | ✅ 实测：每 60 秒一次，账本持续累积，跨进程存活 |
+| 心跳：改配置立刻生效 | ✅ 定时器只负责「醒」，跑不跑每次现读配置 |
 | 心跳：自动开口（真的送出去） | ⬜ 未做 —— **这一版只算不发，但算得对了** |
 
 > ⚠️ **它现在还不会自己开口。** 定时器接上了，判定链路也完整了
@@ -73,13 +74,14 @@
 | `minGapMinutes` | 两次心跳最少隔多久 |
 | `dice` | 没有由头时的开口概率。0 = 只在有素材时说 |
 | `quietRounds` | 连续多少轮没开口就强制说一次。0 = 关掉这个保底 |
-| `autoBeatMinutes` | 每隔多少分钟自己跑一次。0 = 关（默认）。见下面那条 |
+| `autoBeatMinutes` | 每隔多少分钟自己跑一次。0 = 关（默认）。**改完立刻生效，不用重启** |
 
 四注意：
 
 - **`autoBeatMinutes` 小于 `minGapMinutes` 时，实际频率由后者决定。** 比如设 1 分钟、
   最小间隔 20 分钟，就是每 20 分钟才真正判定一次，中间那些跳只记一条 `tooSoon`。
   这不是没生效，是间隔在压着它 —— 装的时候会把这句话写进 marker 文件。
+  定时器本身固定 60 秒醒一次，跑不跑由这个数现读现定。
 - **「连续几轮没开口就强制一次」和「最少间隔」是耦合的。** 心跳间隔设得比保底所需的时间还短，保底永远不会触发。
 - **把 `quietRounds` 关掉，它可能安静到让你以为坏了。** 默认闭嘴需要一条下限，不然沉默和故障长得一样。
 - **安静时段优先于保底，但保底不会被清零。** 两者撞上时，安静时段赢：那段时间它不说话。可是「连续没开口」的次数照常往上加，所以走出安静时段后的第一次心跳就会走保底 —— 保底只是被推迟，没有被悄悄关掉。
@@ -162,6 +164,29 @@
 `tooSoon`，才把它揪出来 —— **单元测试全绿，因为测试都是按「一分钟一跳不合法」
 去构造的**，而没有人试过「一小时里每一分钟都跳一次」。
 
+## 又一次教训：第七个字段是快照
+
+0.4.4 修完基准刷新，同样的形状又冒出来一次，方向正好相反：
+
+`autoBeatMinutes` 是 volatile 字段 —— 用户改它时插件**不会重新 mount**，
+这正是 volatile 的意义。可定时器的周期是在 `apply()` 里读一次就钉死的。
+于是在设置页把 5 改成 30，间隔永远停在 5，**而且没有任何地方会报错**。
+
+同一个文件里，六个字段走 `liveConfig(raw)` 每次现读，第七个成了启动时的快照。
+
+0.4.0 是「忘了读这个字段」，这次是「在错的时机读了它」。
+两次的表现一模一样：配了完全没反应，而两次的代码看起来都是对的。
+
+现在定时器跟配置解耦：
+
+| | 谁负责 |
+|---|---|
+| 定时器周期 | 固定 60 秒，只负责「醒」 |
+| 跑不跑、隔多久 | 每次醒的时候现读 `autoBeatMinutes`，用 `isDue(上次跑的时刻, 现在, 配置)` 判 |
+
+关着（0）也照常装定时器 —— 否则用户从 0 改成 5 的时候没人去装，又回到同一个坑。
+代价是每分钟空醒一次，读一个数字比大小。
+
 ## 一次教训：模块边界
 
 0.3.0 / 0.3.5 / 0.3.6 连着三次，坏的地方都在**两个文件交接处**：
@@ -198,7 +223,7 @@ $DSH_HOME/galatea/heartbeat-state.json                      # 心跳状态账本
 ## 本地开发
 
 ```bash
-npm test          # 98 条，不依赖 DSH，纯 node
+npm test          # 101 条，不依赖 DSH，纯 node
 ```
 
 装了之后验证它真的加载了，只看一个东西：
@@ -219,7 +244,7 @@ dsh plugin add dsh-galatea
 或者在 profile 的 `package.json` 里手写：
 
 ```json
-"dependencies": { "dsh-galatea": "0.4.4" }
+"dependencies": { "dsh-galatea": "0.4.5" }
 ```
 
 **版本号钉死，不要写 `^`。** 同一个版本号指向两个不同的包，以后就说不清你装的是哪一份。

package/index.js

@@ -16,7 +16,7 @@ import z from '@deepseek-ai/schemastery';
 import { renderIndex, dataRoot, listEntries } from './lib/store.js';
 import { remember, recall } from './lib/tools.js';
 import { makeBeatTool, snapshotAgents } from './lib/beat-tool.js';
-import { commitDecision, loadState, noteUserMessage, saveState, stateFile } from './lib/heartbeat-runner.js';
+import { commitDecision, isDue, loadState, noteUserMessage, saveState, stateFile } from './lib/heartbeat-runner.js';
 import { planBeat } from './lib/beat.js';
 import { DEFAULTS } from './lib/heartbeat.js';
 
@@ -239,15 +239,37 @@ export function apply(ctx, config) {
   // 想确认它在动，看心跳账本里有没有新记录就够了。
   //
   // 默认关（autoBeatMinutes = 0）：一个会自己转的东西，不该在装上的那一刻就开始转。
-  const everyMin = Number(cfg.autoBeatMinutes ?? 0);
-  if (Number.isFinite(everyMin) && everyMin > 0) {
-    if (ctx && typeof ctx.interval === 'function') {
-      const ms = Math.max(60_000, everyMin * 60_000);
+  //
+  // ⚠ 间隔不能写进 ctx.interval 的周期里。
+  //   autoBeatMinutes 是 volatile 字段 —— 用户改它时插件**不会重新 mount**
+  //   （这正是 volatile 的意义：不重启）。而定时器是装一次就钉死的。
+  //   所以「每 N 分钟」一旦写进周期，改完设置页间隔永远不会变，
+  //   而且没有任何地方会报错 —— 同一个文件里六个字段是活的，第七个成了快照。
+  //   （0.4.0 是「忘了读这个字段」，这次是「在错的时机读了它」。）
+  //
+  //   改成固定一分钟醒一次，每次醒的时候**现读**配置，到点才真的跑。
+  //   代价是每分钟空醒一次（读一个数字比大小），换来的是改完立刻生效。
+  const TICK_MS = 60_000;
+  if (ctx && typeof ctx.interval === 'function') {
+    let lastRunAt = 0;
       // 用 ctx.interval，不用已 deprecated 的 ctx.setInterval —— 它返回 dispose，
       // 由当前 fiber 管着，宿主热重载时不会叠出第二个定时器。
       ctx.interval(() => {
         try {
-          const at = new Date();
+          // 现读 —— volatile 字段唯一的正确用法。
+          // 取不到（没配过）就回落到 0，也就是「关着」。
+          const everyMin = Number(
+            raw && raw.autoBeatMinutes && typeof raw.autoBeatMinutes.get === 'function'
+              ? raw.autoBeatMinutes.get()
+              : cfg.autoBeatMinutes,
+          );
+          const nowMs = Date.now();
+          // 到点才真的跑。这一句是「改配置立刻生效」的全部秘密：
+          // 定时器只负责醒，什么时候真的跑由这个现读的数决定。
+          if (!isDue(lastRunAt, nowMs, everyMin)) return;
+          lastRunAt = nowMs;
+
+          const at = new Date(nowMs);
           const state = loadState(stateFile());
           // 走和手动档**同一条**链路（planBeat）。
           //
@@ -271,32 +293,35 @@ export function apply(ctx, config) {
           // 定时器抛出去会打断整个宿主，所以在这里兜住，但要说出来
           console.warn('[galatea] 自动心跳失败:', err && err.message);
         }
-      }, ms);
-      console.log(`[galatea] 自动心跳已开：每 ${everyMin} 分钟跑一次（只算不发）`);
+      }, TICK_MS);
+      console.log('[galatea] 自动心跳的钟已装：每 60 秒醒一次，按设置里的 autoBeatMinutes 决定跑不跑（0 = 不跑）');
 
-      // 一个很自然但会误会的组合：自动心跳间隔 < 最小间隔。
-      // 它**不是错误** —— minGap 会把节奏压回 20 分钟一次。
-      // 但用户以为自己设的是 autoBeatMinutes，所以得把实际生效的那个数说出来。
-      const gapMin = Number(cfg.minGapMinutes ?? DEFAULTS.minGapMinutes);
-      if (Number.isFinite(gapMin) && gapMin > 0 && everyMin < gapMin) {
-        const note = `[galatea] 自动心跳设的是每 ${everyMin} 分钟一次，但最小间隔是 ${gapMin} 分钟 —— 实际每 ${gapMin} 分钟才真正判定一次（不是没生效，是被间隔压住了）。`;
-        console.warn(note);
-        try {
-          appendFileSync(marker, `  ${note}\n`);
-        } catch {
-          // 记录失败不该连累插件
-        }
-      }
-    } else {
-      // 用户明确要求了、却被静默拒绝 —— 这一次不能只在 console 里说。
-      // 真机上 console.warn 用户看不见，表现是「配了完全不生效，也没有任何提示」。
-      const line = '[galatea] 你配了 autoBeatMinutes，但宿主没有 ctx.interval —— 自动心跳不会跑。';
-      console.warn(line);
+    // 一个很自然但会误会的组合：自动心跳间隔 < 最小间隔。
+    // 它**不是错误** —— minGap 会把节奏压回每 minGap 分钟一次。
+    // 但用户以为自己设的是 autoBeatMinutes，所以得把实际生效的那个数说出来。
+    const gapMin = Number(cfg.minGapMinutes ?? DEFAULTS.minGapMinutes);
+    const bootEvery = Number(cfg.autoBeatMinutes ?? 0);
+    if (
+      Number.isFinite(bootEvery) && bootEvery > 0 &&
+      Number.isFinite(gapMin) && gapMin > 0 && bootEvery < gapMin
+    ) {
+      const note = `[galatea] 自动心跳设的是每 ${bootEvery} 分钟一次，但最小间隔是 ${gapMin} 分钟 —— 实际每 ${gapMin} 分钟才真正判定一次（不是没生效，是被间隔压住了）。`;
+      console.warn(note);
       try {
-        appendFileSync(marker, `  ${line}\n`);
+        appendFileSync(marker, `  ${note}\n`);
       } catch {
         // 记录失败不该连累插件
       }
     }
+  } else if (Number(cfg.autoBeatMinutes ?? 0) > 0) {
+    // 用户明确要求了、却被静默拒绝 —— 这一次不能只在 console 里说。
+    // 真机上 console.warn 用户看不见，表现是「配了完全不生效，也没有任何提示」。
+    const line = '[galatea] 你配了 autoBeatMinutes，但宿主没有 ctx.interval —— 自动心跳不会跑。';
+    console.warn(line);
+    try {
+      appendFileSync(marker, `  ${line}\n`);
+    } catch {
+      // 记录失败不该连累插件
+    }
   }
 }

package/lib/heartbeat-runner.js

@@ -100,6 +100,35 @@ export function commitDecision(state, decision, now = new Date(), opts = {}) {
   return next;
 }
 
+/**
+ * 自动心跳的节流：这一跳**该不该真的跑一次**。
+ *
+ * 为什么单独一个函数，而不是直接把间隔写进定时器周期：
+ *
+ *   `autoBeatMinutes` 是 volatile 字段 —— 用户改它时插件**不会重新 mount**
+ *   （这正是 volatile 的意义：不重启）。而定时器的周期是装配那一刻钉死的。
+ *
+ *   所以「每 N 分钟」一旦写进 interval 的周期，用户改完设置页，
+ *   间隔永远不会变，而且**没有任何地方会报错** ——
+ *   同一个文件里六个字段是活的，第七个成了快照，看起来完全正常。
+ *   （0.4.0 是「忘了读这个字段」，这次是「在错的时机读了它」。）
+ *
+ *   拆开之后：定时器固定一分钟醒一次，醒的时候现读配置、用这个函数判。
+ *
+ * @param {number} lastRunAtMs  上次真的跑过的时刻（毫秒；<=0 表示从没跑过）
+ * @param {number} nowMs        现在（毫秒）
+ * @param {number} everyMinutes 配置里的间隔。<=0 或不是数字 ⟹ 关着，永不跑
+ * @returns {boolean}
+ */
+export function isDue(lastRunAtMs, nowMs, everyMinutes) {
+  const every = Number(everyMinutes);
+  if (!Number.isFinite(every) || every <= 0) return false;
+  const last = Number(lastRunAtMs);
+  // 从没跑过 ⟹ 立刻就绪。这样重启之后不会平白多等一个完整周期。
+  if (!Number.isFinite(last) || last <= 0) return true;
+  return Number(nowMs) - last >= every * 60_000;
+}
+
 export function tick(opts = {}) {
   const now = opts.now instanceof Date ? opts.now : new Date(opts.now ?? Date.now());
   const file = opts.file ?? stateFile();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-galatea",
-  "version": "0.4.4",
+  "version": "0.4.5",
   "description": "Galatea — memory, heartbeat and continuity for DeepSeek Harness agents.",
   "repository": {
     "type": "git",