dsh-channel-memory 0.1.1 → 0.1.2

package/README.md

@@ -116,14 +116,33 @@ keeper → 桥的 `--cwd`），例如：
 
 ## 验证
 
-`plugin/contract_test.mjs` 用**宿主自己的 `defineTool`** 跑契约测试（不是 stub）：
+`test/contract_test.mjs` 用**宿主自己的 `defineTool`** 跑契约测试（不是 stub）：
 value-schema DSL 违规会当场抛。夹具强制宿主不变量（cwd 不符则 `attachSession` 抛），
 因此「外来会话被拒绝挂载」是被真实验证的，而不是被假 object 允许的。
 
+在插件目录里运行（`<APP>` 同上；Windows 桌面形态必须用 DSH 自带的 Electron 跑，
+因为宿主模块在 `app.asar` 里）：
+
+```bash
+set ELECTRON_RUN_AS_NODE=1
+"<APP>\DeepSeek Harness.exe" --expose-internals test\contract_test.mjs
+```
+
+可选覆盖（默认指向本机安装布局）：`DSH_TOOLS_ENTRY`、`DSH_PLUGIN_ENTRY`。
+
+实测输出：
+
 ```
-RESULT: PASS   （5 个工具注册、1 个提示词分节、外来会话未被挂载、post-apply 写入反映进索引）
+tools registered: 5
+prompt sections : channel-memory:index@3200(text=function)
+attached (channel session): ["sess-abc"]
+after foreign session     : ["sess-abc"] (must not contain sess-elsewhere)
+index reflects post-apply write: true
+RESULT: PASS
 ```
 
+测试会把 `DSH_HOME` 指到临时目录，不会碰你真实的 `~/.dsh`。
+
 权限面见 `PERMISSIONS.md`。
 
 ## License

package/lib/index.js

@@ -303,9 +303,25 @@ export function apply(ctx, config) {
     },
   ]
 
-  const diag = { toolsService: 'ok', defineTool: typeof defineTool, errors: [], registered: [] }
+  const diag = { toolsService: 'absent', defineTool: typeof defineTool, errors: [], registered: [] }
+
+  // Written before it is called: registration can land either synchronously (hosts
+  // that resolve `tools` eagerly) or from the inject callback, and both paths report
+  // through here. A single early snapshot would always show registered:0 on profiles
+  // where inject defers, which previously read as "plugin not installed".
+  const writeStatus = () => {
+    try {
+      writeJson(join(cfg.dataDir, 'status.json'), {
+        version: 1, at: new Date().toISOString(), registered: diag.registered.length, total: tools.length,
+        diag, workspace: state.workspaceId || null, memories: mem.items.length,
+      })
+    } catch { }
+  }
+
   const registerAll = (toolsSvc) => {
+    if (diag.registered.length) return                       // exactly once
     if (!toolsSvc || typeof toolsSvc.register !== 'function') { diag.toolsService = 'unavailable'; return }
+    diag.toolsService = 'ok'
     for (const t of tools) {
       try {
         toolsSvc.register(defineTool({ ...t, output: OUT() }))
@@ -316,29 +332,25 @@ export function apply(ctx, config) {
         ctx.logger?.warn?.(`channel-memory: tool ${msg}`)
       }
     }
+    writeStatus()
   }
 
-  // inject 声明保证了 ctx.tools 可用；注册同步完成后才落快照，
-  // 不会再出现 "toolsService absent / registered 0" 的时序假象。
-  registerAll(ctx.tools)
+  // `tools` is requested through inject rather than read off the root context:
+  // on some profiles (observed under `--profile acp`, the path phone turns take)
+  // touching ctx.tools without declaring it throws
+  // `cannot get property "tools" without inject`, which would abort apply() and
+  // leave the plugin mounted but tool-less.
+  ctx.inject(['tools'], (host) => registerAll(host.tools))
 
-  // Observability: without this, a silent registration failure is indistinguishable
-  // from "plugin not installed" when looking at the profile from the outside.
-  // Tool registration is synchronous above, but the workspace is created by an
-  // async effect, so a single snapshot here always shows workspace:null. Re-write
-  // after the effect has had time to land, otherwise the file contradicts
-  // channel_workspace_info and looks like a mount failure.
-  const writeStatus = () => {
-    try {
-      writeJson(join(cfg.dataDir, 'status.json'), {
-        version: 1, at: new Date().toISOString(), registered: diag.registered.length, total: tools.length,
-        diag, workspace: state.workspaceId || null, memories: mem.items.length,
-      })
-    } catch { }
+  // Hosts that already resolved the service call the callback synchronously; where
+  // they do not, keep the direct handle as a guarded fallback so registration can
+  // never silently no-op. registerAll ignores the second attempt.
+  if (!diag.registered.length) {
+    try { registerAll(ctx.tools) } catch (e) { diag.errors.push('direct: ' + String(e?.message ?? e).slice(0, 120)) }
   }
   writeStatus()
-  setTimeout(writeStatus, 2000)
+  setTimeout(writeStatus, 2000)      // after the workspace effect has had time to land
   setTimeout(writeStatus, 10000)
 
   ctx.logger?.info?.(`channel-memory ready: tools=${diag.registered.length}/${tools.length} memories=${mem.items.length} ws=${cfg.workspacePath}`)
-}
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-channel-memory",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "微信通道专属工作区 + 跨会话互通记忆：把手机发起的 DSH 会话归到一个独立工作区，并让所有会话共享一份可检索的长期记忆。",
   "license": "MIT",
   "type": "module",