dsh-channel-memory 0.1.0 → 0.1.2

package/README.md

@@ -44,31 +44,47 @@ DeepSeek Harness 插件：**手机通道专属工作区** + **跨会话互通记
 
 ## 安装
 
+> ⚠️ npm 上 `dsh-channel-memory` 目前只有 `0.0.0-stage` 占位版（`latest` 指向它），
+> 真实版本要等 staging approve 完成。**在此之前请用下面的 git 或本地目录方式安装**，
+> 不要直接 `pnpm add dsh-channel-memory`。
+
+在 DSH 的应用 profile 目录里执行（Windows 桌面形态是
+`%USERPROFILE%\.dsh\profiles\desktop`）：
+
 ```bash
 cd %USERPROFILE%\.dsh\profiles\desktop
-node <DSH>\resources\runtime\..\..\resources\app.asar\dsh\... # 见下
+pnpm add github:Yuncan050115/dsh-channel-memory
 ```
 
-推荐用 DSH 插件管理器，或直接：
+或从本地目录安装：
 
 ```bash
-pnpm add dsh-channel-memory
+pnpm add file:C:\path\to\dsh-channel-memory
 ```
 
-然后把 bundle 名加进 `package.json` 的 `dsh.profile.bundles`：
+然后把 bundle 名加进同目录 `package.json` 的 `dsh.profile.bundles`
+（DSH 插件管理器会自动做这一步，手工加也可以）：
 
 ```json
 "dsh": { "profile": { "bundles": [ ..., "dsh-channel-memory" ] } }
 ```
 
-重启 DSH 生效。校验（不启动 GUI，只做组合干跑）：
+重启 DSH 生效。不想重启也可以先校验组合结果——用 DSH 自带运行时做一次干跑
+（`--dump-config` 只组合配置、不启动 GUI）。`<APP>` 是 DSH 的安装目录，
+Windows 桌面形态默认是 `C:\Program Files\DeepSeek Harness` 或
+`D:\DeepSeek Harness`：
 
-```bash
+```bat
 set ELECTRON_RUN_AS_NODE=1
-"DeepSeek Harness.exe" --expose-internals <profile>\..\..\..\app.asar\dsh\node_modules\@deepseek-ai\dsh\lib\bin.js ^
+set DSH_HOME=%USERPROFILE%\.dsh
+"<APP>\DeepSeek Harness.exe" --expose-internals ^
+  "<APP>\resources\app.asar\dsh\node_modules\@deepseek-ai\dsh\lib\bin.js" ^
   --profile desktop --dump-config
 ```
 
+输出里应能看到 `# == dsh-channel-memory` 这一层；若出现 `skipped bundles` 或
+`did not activate`，说明 bundle 没挂上。
+
 ## 配置
 
 `cordis.patch.yml` 里 `channel-memory` 行的 `config` 全部可选，注释掉了默认值：
@@ -100,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,55 +1,55 @@
-{
-  "name": "dsh-channel-memory",
-  "version": "0.1.0",
-  "description": "微信通道专属工作区 + 跨会话互通记忆：把手机发起的 DSH 会话归到一个独立工作区，并让所有会话共享一份可检索的长期记忆。",
-  "license": "MIT",
-  "type": "module",
-  "main": "./lib/index.js",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/Yuncan050115/dsh-channel-memory.git"
-  },
-  "bugs": {
-    "url": "https://github.com/Yuncan050115/dsh-channel-memory/issues"
-  },
-  "homepage": "https://github.com/Yuncan050115/dsh-channel-memory#readme",
-  "exports": {
-    ".": "./lib/index.js",
-    "./package.json": "./package.json"
-  },
-  "files": [
-    "lib",
-    "cordis.patch.yml",
-    "README.md",
-    "PERMISSIONS.md",
-    "LICENSE",
-    "package.json"
-  ],
-  "dsh": {
-    "bundle": {
-      "patch": "./cordis.patch.yml"
-    }
-  },
-  "peerDependencies": {
-    "@deepseek-ai/cordis": "^4.0.1",
-    "@deepseek-ai/dsh-tools": ">=0.2.1-rc.5"
-  },
-  "peerDependenciesMeta": {
-    "@deepseek-ai/cordis": {
-      "optional": true
-    },
-    "@deepseek-ai/dsh-tools": {
-      "optional": true
-    }
-  },
-  "engines": {
-    "node": ">=22"
-  },
-  "keywords": [
-    "deepseek-harness",
-    "dsh-plugin",
-    "wechat",
-    "memory",
-    "workspace"
-  ]
-}
+{
+  "name": "dsh-channel-memory",
+  "version": "0.1.2",
+  "description": "微信通道专属工作区 + 跨会话互通记忆：把手机发起的 DSH 会话归到一个独立工作区，并让所有会话共享一份可检索的长期记忆。",
+  "license": "MIT",
+  "type": "module",
+  "main": "./lib/index.js",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/Yuncan050115/dsh-channel-memory.git"
+  },
+  "bugs": {
+    "url": "https://github.com/Yuncan050115/dsh-channel-memory/issues"
+  },
+  "homepage": "https://github.com/Yuncan050115/dsh-channel-memory#readme",
+  "exports": {
+    ".": "./lib/index.js",
+    "./package.json": "./package.json"
+  },
+  "files": [
+    "lib",
+    "cordis.patch.yml",
+    "README.md",
+    "PERMISSIONS.md",
+    "LICENSE",
+    "package.json"
+  ],
+  "dsh": {
+    "bundle": {
+      "patch": "./cordis.patch.yml"
+    }
+  },
+  "peerDependencies": {
+    "@deepseek-ai/cordis": "^4.0.1",
+    "@deepseek-ai/dsh-tools": ">=0.2.1-rc.5"
+  },
+  "peerDependenciesMeta": {
+    "@deepseek-ai/cordis": {
+      "optional": true
+    },
+    "@deepseek-ai/dsh-tools": {
+      "optional": true
+    }
+  },
+  "engines": {
+    "node": ">=22"
+  },
+  "keywords": [
+    "deepseek-harness",
+    "dsh-plugin",
+    "wechat",
+    "memory",
+    "workspace"
+  ]
+}