dsh-galatea 0.4.7 → 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 条测试 |
@@ -203,7 +203,7 @@
 （同理，日志里那句「有素材，这次没掷中」和「记忆里一条都没有」是两件事。
   前者什么都别做，后者去记一条。）
 
-## 三条教训：粒度、上限、旧数据
+## 四条教训：粒度、上限、旧数据、清理的对象
 
 ### 一 · 天粒度和秒粒度不能比较
 
@@ -242,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 连着三次，坏的地方都在**两个文件交接处**：
@@ -278,7 +305,7 @@ $DSH_HOME/galatea/heartbeat-state.json                      # 心跳状态账本
 ## 本地开发
 
 ```bash
-npm test          # 109 条，不依赖 DSH，纯 node
+npm test          # 110 条，不依赖 DSH，纯 node
 ```
 
 装了之后验证它真的加载了，只看一个东西：
@@ -299,7 +326,7 @@ dsh plugin add dsh-galatea
 或者在 profile 的 `package.json` 里手写：
 
 ```json
-"dependencies": { "dsh-galatea": "0.4.7" }
+"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/package.json

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