dsh-galatea 0.4.6 → 0.4.8

package/README.md

@@ -13,7 +13,7 @@
 | 块 | 状态 |
 |---|---|
 | 能被 DSH 装起来 | ✅ 实测通过（dsh 0.2.2-rc.3） |
-| 发布到 npm | ✅ 0.4.6（GitHub Actions 自动发，不需要 token） |
+| 发布到 npm | ✅ 0.4.8（GitHub Actions 自动发，不需要 token） |
 | 记忆：读写 Markdown | ✅ 15 条测试 |
 | 记忆：注入索引 | ✅ **已核实内容真的进了上下文**（不是在会话日志里看到「接上了没告警」） |
 | 记忆：`remember` / `recall` | ✅ 23 条测试 |
@@ -187,7 +187,23 @@
 关着（0）也照常装定时器 —— 否则用户从 0 改成 5 的时候没人去装，又回到同一个坑。
 代价是每分钟空醒一次，读一个数字比大小。
 
-## 三条教训：粒度、上限、旧数据
+### 刚记下的事，不会马上被提起
+
+这是设计，不是缺陷 —— 但必须说清楚，否则用户会一直等一件不会发生的事。
+
+`hasHardMaterial` 这一版没有判据，所以「有素材」不等于「必须说」。
+在接上真正的硬素材（承诺到期、上次没说完的）之前，自动心跳只有两条路开口：
+
+- 掷色子中（`dice`，默认 0.15）
+- 连续 `quietRounds` 轮没开口，撞到下限保护（默认 8 轮）
+
+所以「我刚记了一件事，它应该会跟我提」——这一版不成立。
+素材只让它「有东西可说」，说不说还是门控的事。
+
+（同理，日志里那句「有素材，这次没掷中」和「记忆里一条都没有」是两件事。
+  前者什么都别做，后者去记一条。）
+
+## 四条教训：粒度、上限、旧数据、清理的对象
 
 ### 一 · 天粒度和秒粒度不能比较
 
@@ -226,6 +242,33 @@ lastSpokeAt         = 2026-11-15T09:45:59Z   ← 真的时刻
 它也不会一直错下去：下一次真的开口就归零。
 但它提醒一件事：**修「某个计数器的累加条件」时，旧值也是新逻辑的输入。**
 
+### 四 · 清理函数写对了，但清的不是自己
+
+`ctx.interval()` 内部是：
+
+```js
+interval(callback, delay) {
+  return this.ctx.effect(() => {
+    const timer = setInterval(callback, delay)
+    return () => clearInterval(timer)
+  })
+}
+```
+
+清理函数写得很对。问题在 `this.ctx` —— 那是 **TimerService 自己被注册时的 ctx**
+（`service.d.ts` 的构造函数把入参存成 `this.ctx`），不是调用方的。
+而 `Context` 本身没有 `ctx` 属性，所以通过 `mixin('timer', ...)` 混到 Context 上的
+这个方法，方法体里的 `this` 只能是 service。
+
+于是定时器挂在 timer 服务的 fiber 上，而那个 fiber 在根上、不销毁。
+**插件每重载一次，旧钟还在响一个。** 重载十次，十个钟同时写账本 ——
+而账本看起来只是在正常动，只是快了几倍，没有任何地方会报错。
+
+修法：自己的 `ctx.effect` 套一层，卸载时走自己的清理。
+
+这一条和上一条是同一类：**两样东西看起来配套，其实不是一对。**
+日期和时刻是「粒度不配对」，这里是「所有权不配对」。
+
 ## 一次教训：模块边界
 
 0.3.0 / 0.3.5 / 0.3.6 连着三次，坏的地方都在**两个文件交接处**：
@@ -262,7 +305,7 @@ $DSH_HOME/galatea/heartbeat-state.json                      # 心跳状态账本
 ## 本地开发
 
 ```bash
-npm test          # 107 条，不依赖 DSH，纯 node
+npm test          # 110 条，不依赖 DSH，纯 node
 ```
 
 装了之后验证它真的加载了，只看一个东西：
@@ -283,7 +326,7 @@ dsh plugin add dsh-galatea
 或者在 profile 的 `package.json` 里手写：
 
 ```json
-"dependencies": { "dsh-galatea": "0.4.6" }
+"dependencies": { "dsh-galatea": "0.4.8" }
 ```
 
 **版本号钉死，不要写 `^`。** 同一个版本号指向两个不同的包，以后就说不清你装的是哪一份。

package/index.js

@@ -252,9 +252,27 @@ export function apply(ctx, config) {
   const TICK_MS = 60_000;
   if (ctx && typeof ctx.interval === 'function') {
     let lastRunAt = 0;
-      // 用 ctx.interval，不用已 deprecated 的 ctx.setInterval —— 它返回 dispose，
-      // 由当前 fiber 管着，宿主热重载时不会叠出第二个定时器。
-      ctx.interval(() => {
+
+    // ⚠️ 必须包在**自己的** ctx.effect 里。
+    //
+    // 这里原来写的是「ctx.interval 返回 dispose，由当前 fiber 管着，
+    // 宿主热重载时不会叠出第二个定时器」。**那是假的，我从没验过就写上去了。**
+    //
+    // 翻 cordis-plugin-timer 才看见：
+    //     interval(callback, delay) { return this.ctx.effect(() => { ... }) }
+    // 而这个 this.ctx 是 **TimerService 自己被注册时的 ctx**
+    // （service.d.ts 构造函数里把入参存成 this.ctx），
+    // 又因为 Context 本身没有 `ctx` 属性（context.d.ts 只有
+    // events/logger/reflect/registry），所以通过 `mixin('timer', ...)`
+    // 混到 Context 上的这个方法，方法体里的 this 只能是 service。
+    //
+    // ⟹ 定时器挂在 timer 服务的 fiber 上，那个 fiber 在根上、不销毁。
+    // ⟹ 插件每重载一次，旧钟还在响一个。重载十次，十个钟同时写账本 ——
+    //    而账本看起来只是在正常动，只是快了几倍，没有任何地方会报错。
+    //
+    // 用 ctx.effect 套住：这个 ctx 是**我们自己**的 fiber，卸载时会走清理。
+    ctx.effect(() => {
+      const stop = ctx.interval(() => {
         try {
           // 现读 —— volatile 字段唯一的正确用法。
           // 取不到（没配过）就回落到 0，也就是「关着」。
@@ -294,7 +312,10 @@ export function apply(ctx, config) {
           console.warn('[galatea] 自动心跳失败:', err && err.message);
         }
       }, TICK_MS);
-      console.log('[galatea] 自动心跳的钟已装：每 60 秒醒一次，按设置里的 autoBeatMinutes 决定跑不跑（0 = 不跑）');
+      return () => stop();
+    }, 'galatea:auto-beat');
+
+    console.log('[galatea] 自动心跳的钟已装：每 60 秒醒一次，按设置里的 autoBeatMinutes 决定跑不跑（0 = 不跑）');
 
     // 一个很自然但会误会的组合：自动心跳间隔 < 最小间隔。
     // 它**不是错误** —— minGap 会把节奏压回每 minGap 分钟一次。

package/lib/beat-tool.js

@@ -143,7 +143,20 @@ export function makeBeatTool(deps) {
       let committed = null;
       let saveError = null;
       try {
-        committed = commitDecision(state, { speak: r.speak, reason: r.reason, detail: r.detail }, new Date());
+        // 手动档是「看一眼」，不是「跳一次」。
+        //
+        // 它照常记账（decisions 会多一条，quietStreak 也照实算），
+        // 但**不推进 lastBeatAt** —— 否则用户每查一次状态，自动心跳就往后推一个
+        // minGap（默认 20 分钟）。连查三次，一小时内不会真正判定，
+        // 而用户完全不知道，只会觉得「它怎么不动了」。
+        //
+        // 这一条是外面的人真机测出来的：18:37 手动 beat，19:08 才跨过 minGap。
+        committed = commitDecision(
+          state,
+          { speak: r.speak, reason: r.reason, detail: r.detail },
+          new Date(),
+          { manual: true },
+        );
         saveState(committed, stateFile());
       } catch (err) {
         // 失败必须让人看见。

package/lib/beat.js

@@ -47,7 +47,7 @@ export function planBeat(input) {
   const material = pickMaterial(input.entries ?? [], state.lastSpokeAt ?? null);
   if (!material.entry) {
     return no(
-      material.reason === 'noEntries' ? 'noMaterial' : 'nothingNew',
+      material.reason === 'noEntries' ? 'noEntries' : 'nothingNew',
       material.reason === 'noEntries'
         ? '记忆里一条可用的都没有 —— 没有出处的话不说'
         : '写了的东西都说过了 —— 没有新的就不开口',

package/lib/heartbeat-runner.js

@@ -103,7 +103,10 @@ export function commitDecision(state, decision, now = new Date(), opts = {}) {
   //
   // 默认 true：老调用点（不带这个字段的）行为不变，不会因为升级而卡住。
   if (decision.touchBeat !== false) {
-    next.lastBeatAt = at;
+    // 手动档（galatea_beat）也算一次判定 —— 它确实问了一遍「要不要说」，
+    // 所以 quietStreak 照涨。但它不该推进 lastBeatAt：那是**自动节奏**的基准，
+    // 手动查一次状态就把后台往后推 20 分钟，用户完全看不出来。
+    if (opts.manual !== true) next.lastBeatAt = at;
     next.quietStreak = decision.speak ? 0 : (state.quietStreak ?? 0) + 1;
   }
   if (decision.speak) next.lastSpokeAt = at;

package/lib/heartbeat.js

@@ -186,10 +186,13 @@ export function decide(input) {
   if (r < cfg.dice) {
     return { speak: true, reason: 'dice', detail: `色子 ${r.toFixed(3)} < ${cfg.dice}`, touchBeat: true };
   }
+  // 名字要准：这次「不说」不是因为没素材 —— 素材层早就给出结论了。
+  // 是「有东西可说，但这次没掷中」。叫 noMaterial 会让 journal 翻成
+  // 「记忆里没有可用的东西」，用户第一反应是去查库是不是没存上。
   return {
     speak: false,
-    reason: 'noMaterial',
-    detail: `没硬素材，色子 ${r.toFixed(3)} ≥ ${cfg.dice}`,
+    reason: 'notChosen',
+    detail: `有素材，但这次色子 ${r.toFixed(3)} ≥ ${cfg.dice}`,
     touchBeat: true,
   };
 }
@@ -203,4 +206,4 @@ function toMs(v) {
 }
 
 /** 所有可能的 reason —— 用来保证「没有第七种情况」。 */
-export const REASONS = ['quiet', 'tooSoon', 'cooldown', 'hard', 'floor', 'dice', 'noMaterial'];
+export const REASONS = ['quiet', 'tooSoon', 'cooldown', 'hard', 'floor', 'dice', 'notChosen'];

package/lib/journal.js

@@ -23,9 +23,16 @@ export const REASON_TEXT = {
   hard: '有具体的事要说',
   floor: '太久没开口，这次强制说一句',
   dice: '随机决定要说',
-  noMaterial: '记忆里没有可用的东西',
+  // 「没说」的三种必须分得开 —— 用户看到之后该做什么，三种完全不同：
+  //   noEntries  → 去记一条
+  //   notChosen  → 什么都不用做，这次没掷中
+  //   nothingNew → 去记一条新的
+  noEntries: '记忆里一条都没有',
+  notChosen: '有素材，这次没掷中',
   nothingNew: '记下的都说过了',
   noHuman: '没有正在进行的对话',
+  // 旧账本里躺着的名字。0.4.7 之前，上面 noEntries 和 notChosen 都被记成 noMaterial。
+  noMaterial: '（旧版）没有可用的素材',
 };
 
 /** HH:MM，本地时区。认不出的时间原样标出来，不替它猜一个像样的值。 */

package/package.json

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