dsh-galatea 0.4.0 → 0.4.2

package/index.js

@@ -74,7 +74,7 @@ export const Config = z.object({
 });
 
 /** 需要宿主注入的服务。 */
-export const inject = ['systemPrompt', 'tools', 'agents'];
+export const inject = ['systemPrompt', 'tools', 'agents', 'timer'];
 
 /**
  * volatile 字段是**引用**，普通字段是**值**。
@@ -114,12 +114,52 @@ function liveConfig(raw) {
  */
 export function apply(ctx, config) {
   const raw = config ?? {};
-  const cfg = raw;
+  // volatile 字段在 apply 里拿到的是引用（`{ get() }`），不是值。
+  //
+  // 直接读 `raw.x` 会拿到对象 —— `Number(对象)` 是 NaN，`if (NaN > 0)` 不成立，
+  // 于是那个开关永远不会生效，而且**没有任何地方会报错**。
+  //
+  // 0.4.0 的自动心跳就是这么死的：同一个文件里六个字段走了 pick()，
+  // 第七个（autoBeatMinutes）忘了，于是它从来没有被打开过。
+  //
+  // 与其要求每个调用点都记得 pick()，不如在这里一次性解引用：
+  // 之后所有地方只看普通值，「忘了 pick」这种错误就不可能再发生。
+  const cfg = {};
+  for (const [k, v] of Object.entries(raw)) {
+    cfg[k] = v && typeof v.get === 'function' ? v.get() : v;
+  }
+
+  // 解引用之后再自检一次：还有对象就说明这个字段的形状跟以为的不一样。
+  // 不能说"大概没配"就把它当默认值用掉 —— 那正是 0.4.0 的死法。
+  for (const [k, v] of Object.entries(cfg)) {
+    if (v !== null && typeof v === 'object' && !Array.isArray(v)) {
+      console.warn(`[galatea] 设置项 ${k} 解引用后仍然是对象，这项配置可能没生效`);
+    }
+  }
+
+  // ── 能力自检 ────────────────────────────────────────────
+  // inject 里声明了、但 ctx 上其实没有的，一次报全。
+  // 为什么需要这一段：今天这一类的错已经犯到第六次，其中两次是 inject ——
+  // 声明漏了，代码走进 else 分支，只有一行 console.warn，
+  // 而真机上用户看不到 console.warn，于是表现为「配了完全不生效」。
+  // marker 先算出来：自检要用它记下「哪些服务没接上」。
+  // 用户已经知道去哪儿看 marker（README 写着），所以它是最便宜的可见出口。
+  const marker = cfg.markerPath || join(homedir(), '.dsh', 'galatea-loaded.txt');
+
+  const missing = inject.filter((name) => ctx == null || ctx[name] === undefined);
+  if (missing.length > 0) {
+    const line = `[galatea] 这些服务没接上：${missing.join(', ')} —— 对应的功能不会工作`;
+    console.warn(line);
+    try {
+      appendFileSync(marker, `  ${line}\n`);
+    } catch {
+      // 自检自己失败不该连累插件
+    }
+  }
 
   // ── 正控：这一段真的跑过吗 ────────────────────────────────
   // marker 不是日志。它只回答一个问题：apply() 到底有没有被执行。
   // 「进程起来了」「配置里写了」都不算证据。
-  const marker = cfg.markerPath || join(homedir(), '.dsh', 'galatea-loaded.txt');
   try {
     mkdirSync(dirname(marker), { recursive: true });
     appendFileSync(marker, `galatea loaded at ${new Date().toISOString()} pid=${process.pid}\n`);
@@ -198,11 +238,13 @@ export function apply(ctx, config) {
   // 想确认它在动，看心跳账本里有没有新记录就够了。
   //
   // 默认关（autoBeatMinutes = 0）：一个会自己转的东西，不该在装上的那一刻就开始转。
-  const everyMin = Number(raw.autoBeatMinutes ?? 0);
+  const everyMin = Number(cfg.autoBeatMinutes ?? 0);
   if (Number.isFinite(everyMin) && everyMin > 0) {
-    if (ctx && typeof ctx.setInterval === 'function') {
+    if (ctx && typeof ctx.interval === 'function') {
       const ms = Math.max(60_000, everyMin * 60_000);
-      ctx.setInterval(() => {
+      // 用 ctx.interval，不用已 deprecated 的 ctx.setInterval —— 它返回 dispose，
+      // 由当前 fiber 管着，宿主热重载时不会叠出第二个定时器。
+      ctx.interval(() => {
         try {
           tick({});
         } catch (err) {
@@ -212,7 +254,15 @@ export function apply(ctx, config) {
       }, ms);
       console.log(`[galatea] 自动心跳已开：每 ${everyMin} 分钟跑一次（只算不发）`);
     } else {
-      console.warn('[galatea] 宿主没有 setInterval，自动心跳没接上。');
+      // 用户明确要求了、却被静默拒绝 —— 这一次不能只在 console 里说。
+      // 真机上 console.warn 用户看不见，表现是「配了完全不生效，也没有任何提示」。
+      const line = '[galatea] 你配了 autoBeatMinutes，但宿主没有 ctx.interval —— 自动心跳不会跑。';
+      console.warn(line);
+      try {
+        appendFileSync(marker, `  ${line}\n`);
+      } catch {
+        // 记录失败不该连累插件
+      }
     }
   }
 }

package/package.json

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