@whyshu/argent-erp 0.2.2 → 0.2.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@whyshu/argent-erp",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "type": "module",
   "description": "Argent ERP 会话视图（dsh conversation.view）+ 六禾园办公室助理 preset（只聊办公室、只读 ERP）+ 办公室聊一聊入口",
   "keywords": [

package/src/index.js

@@ -1,5 +1,5 @@
 /**
- * argent-erp · 宿主侧插件（v0.2.2）
+ * argent-erp · 宿主侧插件（v0.2.3）
  *
  * 职责只有两件：
  *  1. 注册本地只读端点 `/api/argent-erp/*`（带账号 token 去 whyshu.com，**token 不下发前端**）；

package/src/office-guard.js

@@ -36,28 +36,57 @@ export function apply(ctx) {
   try {
     let disposeMask = null
     let lastMask = ''
+    /**
+     * 重入闸（2026-11-08 修）：`ctx.tools.restrict()` 与它的 disposer **都会同步 emit
+     * `tools/change`**（见 dsh-scope `ScopedLayers.effect` 的两处 onChange）。
+     * 第一次收敛之后才出现的全局工具（典型：MCP 网关在启动后几秒才注册
+     * `mcp__argent__erp_*`）会让本函数在「dispose 旧 mask → restrict 新 mask」中途被
+     * 同步回调重入；此时 `lastMask` 还没更新 ⇒ 每层都判定"集合变了" ⇒ 无限同步递归
+     * （主线程 100% CPU + 堆 OOM，dsh 端口 bound 但 HTTP 不再响应）。
+     * 判据：工具名集合没变就只允许一个 reconcile 在跑；本函数自己触发的变化只记 dirty，
+     * 等这一轮收敛完再补一轮（保证真有新工具时不会漏收敛）。
+     */
+    let reconciling = false
+    let dirty = false
+
+    /** 收敛一轮：按当前全局工具名重算并换 mask。 */
+    const applyMask = () => {
+      // 全局视图（不传 scope）⇒ 拿到的是"未收敛"的全量工具，不受本作用域 mask 影响
+      const all = typeof ctx.tools?.definitions === 'function' ? ctx.tools.definitions() : []
+      const names = all.map((d) => (d && d.name) || '')
+      const allow = filterErpTools(names)
+      // 有 ERP 工具 ⇒ 只放行它们；一个都没有（没装 @whyshu/argent / MCP 未起）⇒ **全部禁掉**，
+      // 宁可这个会话无工具，也不让 Agent Team 等全局工具漏进来（边界优先）。
+      const mask = allow.length ? { allow } : (names.length ? { deny: names } : null)
+      if (!mask) return
+      const key = JSON.stringify(mask)
+      if (key === lastMask) return
+      // 先记账再动 mask：restrict/它的 disposer 同步 emit 时，重入会被闸挡住且只标 dirty；
+      // 即便真漏进来一次，key 也已是最新值，不会再次判定"集合变了"。
+      lastMask = key
+      try { disposeMask?.() } catch { /* noop */ }
+      disposeMask = ctx.tools.restrict(mask)
+      console.log(allow.length
+        ? `[argent-office] 工具边界：仅放行 ${allow.length} 个 ERP 只读工具`
+        : '[argent-office] 工具边界：未发现 ERP 工具 ⇒ 全部工具禁用（检查 @whyshu/argent 是否安装）')
+    }
 
     const reconcile = () => {
+      if (reconciling) { dirty = true; return }
+      reconciling = true
       try {
-        // 全局视图（不传 scope）⇒ 拿到的是"未收敛"的全量工具，不受本作用域 mask 影响
-        const all = typeof ctx.tools?.definitions === 'function' ? ctx.tools.definitions() : []
-        const names = all.map((d) => (d && d.name) || '')
-        const allow = filterErpTools(names)
-        // 有 ERP 工具 ⇒ 只放行它们；一个都没有（没装 @whyshu/argent / MCP 未起）⇒ **全部禁掉**，
-        // 宁可这个会话无工具，也不让 Agent Team 等全局工具漏进来（边界优先）。
-        const mask = allow.length ? { allow } : (names.length ? { deny: names } : null)
-        if (!mask) return
-        const key = JSON.stringify(mask)
-        if (key === lastMask) return
-        try { disposeMask?.() } catch { /* noop */ }
-        disposeMask = ctx.tools.restrict(mask)
-        lastMask = key
-        console.log(allow.length
-          ? `[argent-office] 工具边界：仅放行 ${allow.length} 个 ERP 只读工具`
-          : '[argent-office] 工具边界：未发现 ERP 工具 ⇒ 全部工具禁用（检查 @whyshu/argent 是否安装）')
+        // 最多补 20 轮：只有"收敛过程中又出现了新工具"才会再来一轮，正常情况 1 轮结束
+        for (let round = 0; round < 20; round++) {
+          dirty = false
+          applyMask()
+          if (!dirty) break
+        }
       } catch (e) {
-        // 收敛失败不阻塞会话：提示词仍然是第二道边界
+        // 收敛失败不阻塞会话：提示词仍然是第二道边界；清掉 key 让下一次工具变化时重试
+        lastMask = ''
         console.warn('[argent-office] 工具边界收敛失败（继续按提示词边界运行）:', (e && e.message) || e)
+      } finally {
+        reconciling = false
       }
     }