@yangdcm/dsh-expert-team 1.2.0 → 1.2.2

package/CHANGELOG.md

@@ -3,6 +3,29 @@
 本包遵循[语义化版本](https://semver.org/lang/zh-CN/)。dsh 宿主版本线的对应关系写在
 `package.json` 的 `engines.dsh` 与 `dsh.compatibility` 里，插件市场按它判断"这个插件跟你的宿主兼不兼容"。
 
+## 1.2.2
+
+**修掉"lead 工具面收窄静默失效"**（拿真机启动日志换来的）
+
+- 症状：每次创建 agent 都刷一行 `lead 工具面**未**收窄（nothing-to-deny）… 宿主里这些名字一个都不存在`，
+  而实际上模型可见的 `bash/write/edit/grep/glob` **存在** —— 结论不成立，收窄也没发生。
+- 根因：宿主 `tools.view(scope)` **不传 scope = 全局视图**；0.1.5 起模型可见工具由 preset 注册在
+  **agent 平面**，于是全局视图"非空但缺这几个名字"，纯函数据实报 `nothing-to-deny`。
+  宿主自己的 `restrict()` 用的就是 `scopeOf(this.ctx)`，我们却用了不传 scope 的 `view()`。
+- 修法：新增 `agentScopedToolNames()`（动态 import `@deepseek-ai/dsh-scope` 取 `scopeOf(agent.ctx)`，
+  再 `view(scope)`）；取不到 scope 时返回 `knownNames: undefined` ⇒ 如实报 **no-known-names**
+  （"我不知道有什么"），绝不再退化成"宿主里没有这些工具"这种不成立的结论。
+- 告警去重：同一 status 每进程只喊一次（十行同样的 warn 会把"响亮"变成噪声，
+  而噪声的代价是所有告警一起被降权）。两种零仍然分得清。
+## 1.2.1
+
+**卸载回收覆盖历史副本**（1.2.0 的补丁）
+
+- `/team uninstall` 此前只遍历登记清单，而 **1.2.0 之前铺下的 skill/preset 没有登记过**
+  （那时还没有清单）—— 从 1.1.x 升上来的用户跑它只会得到"没有需要回收的副本"，
+  而 `$DSH_HOME/skills/expert-team` 里的旧副本仍在（`gate:sync` 会一直报漂移）。
+  现在除清单外还会看两个众所周知的自举落点，判据仍是"有戳或身份对得上"，
+  用户自己写的同名内容照样不动。
 ## 1.2.0
 
 **安全加固（本机来源守卫）+ 自举安装收口 + 设置接进宿主命名空间**

package/lib/command.js

@@ -41,7 +41,7 @@ import { collectTokenUsage } from './metrics/token-usage.js';
 import { renderTokenSection } from './metrics/tokens.js';
 // A 线（token 成本治理 · 第 2 步）：**收窄 lead 自己的工具面**。判定全在
 // `lib/lead-toolface.js`（纯函数、零宿主依赖、可单测）；这里只负责在 agent **创建时刻**接线。
-import { planLeadToolFace, shouldNarrowLeadToolFace } from './lead-toolface.js';
+import { planLeadToolFace, shouldNarrowLeadToolFace, agentScopedToolNames } from './lead-toolface.js';
 // G 线（档位）：档位词表/裁剪表/suggestTier 的**唯一真源**。SKILL 的裁剪表、`/team status`
 // 的输出、浮层候选项全部从这里来 —— 两处各写一份就是又一次「一个事实多份拷贝」。
 import { normalizeTier, suggestTier, tierSummaryLine, tierDetailLines, tierChoices, TIER_SPEC, TIER_LABELS_ZH, DEFAULT_TIER, narrowedRoles } from './tier.js';
@@ -849,16 +849,32 @@ async function uninstallInstalled() {
   const rec = await readInstalled();
   const removed = [];
   const skipped = [];
-  for (const [kind, info] of Object.entries(rec)) {
-    const dir = info && info.path;
-    if (!dir) continue;
-    if (await pathExists(dir) && await isOwnedCopy(dir, kind)) {
+  const seen = new Set();
+  /**
+   * 回收一个候选目录（幂等；只删"本插件的副本"）。
+   * 为什么要 `seen`：清单里的路径与下面两个**历史默认路径**可能重合，重复 rm 会把
+   * "已回收"报两遍，也会让第二次那条 `pathExists` 失败被误记成"保留"。
+   */
+  const reclaim = async (kind, dir) => {
+    if (!dir || seen.has(dir)) return;
+    seen.add(dir);
+    if (!(await pathExists(dir))) return;
+    if (await isOwnedCopy(dir, kind)) {
       try { await rm(dir, { recursive: true, force: true }); removed.push(`${kind}：${dir}`); }
       catch (e) { skipped.push(`${kind}：${dir}（删除失败：${String(e && e.message ? e.message : e)}）`); }
-    } else if (await pathExists(dir)) {
+    } else {
       skipped.push(`${kind}：${dir}（不是本插件的副本 ⇒ 保留）`);
     }
-  }
+  };
+
+  for (const [kind, info] of Object.entries(rec)) await reclaim(kind, info && info.path);
+
+  // 1.2.0 之前的副本**没有登记过**（那时还没有清单）—— 而它们恰恰是这个命令最该处理的历史遗留。
+  // 所以除了清单，还要看两个众所周知的自举落点；判据仍是 `isOwnedCopy`（无戳时按身份判定），
+  // 用户自己写的同名 skill/preset 依旧不会被删。
+  await reclaim('skill', join(dshHome(), 'skills', 'expert-team'));
+  await reclaim('preset', join(dshHome(), '.agent-presets', 'expert-team'));
+
   try { await rm(installedManifestPath(), { force: true }); } catch { /* 清单删不掉就留着 */ }
   const lines = ['# /team uninstall', ''];
   lines.push(removed.length ? '已回收：' : '没有需要回收的副本。');
@@ -4393,7 +4409,19 @@ async function executeTeamCommand(ctx, invocation) {
 /** 并发写追踪器：**进程级单例**（跨工具调用累积；`_live` 暴露以便单测重置与断言）。 */
 const WRITE_TRACER = createWriteTracer();
 
-export const _live = { pushActivity, phaseAccountingViolations, loggedPhases, authorityViolations, WRITE_TRACER, createWriteTracer, formatConflict, summarizeTool, parseLogLine, roleOfSub, mapRoleToSub, membersFromState, buildRoleSubMap, resolveSubRoles, childSessionTiming, SUB_HEADER_CACHE, workflowEventIndex, workflowChildLabels, workflowChildMeta, workflowRuns, WF_EVENT_CACHE, rememberSessionRun, sessionRunFor, runOwnerSession, SESSION_RUNS, parseTeamCommand, deriveMemberEntries, schemaViolations, runHealth, RUN_STALL_MS, scaffoldFingerprint, SCAFFOLD_REQUIRED, strandedTasks, settleStranded, IN_FLIGHT_STATUSES, normalizeCoverage, SCHEMA_WARN_SEEN, pushActivityEvent, DEFAULT_LIMITS, LIMITS, resolveLimits, capacityViolations, DEFAULT_ROUND_LIMITS, ROUND_LIMITS, ROUND_LIMIT_ENV, resolveRoundLimits, ROUND_LIMIT_OF_KIND, roundOf, isQualityTask, normTitle, roundLimitViolations, reworkLoopWriteGuard, mutateTasks, readStandingRules, appendStandingRule, rulesRun, scopeOverlapWarnings, applyTaskStatus, waitRun, eventFamily, verdictFromToken, normalizeRoleName, truncateCodepoints, filterRunScopedSubs, runCreatedAtMs, runLogTail, liveFiles, LIVE_FILES_CACHE, DEFAULT_ROLES, resolveTierGate, TIER_GATE_ENV, snapshotRun, settingsPath, loadSettingsSync, currentSettings, limitsBaseFromSettings, roundLimitsBaseFromSettings, effectiveTierGate: () => TIER_GATE, ensureSkillInstalled, ensurePresetInstalled, uninstallInstalled, buildSkillRegistration, parseSkillMarkdown, PLUGIN_VERSION, INSTALL_STAMP, runtimeSkillRegistered: () => RUNTIME_SKILL_REGISTERED, installHostSettings, hostValues, hostScope, hostSettingsNote, updateHostSettings, pickFileOnly, pickHostExpressible, buildHostSchema, hostBase, hostSchemaPaths, reapplySettingsDerived, currentSettings, mergeSettings };
+/**
+ * lead 工具面告警去重：同一 status 每个进程只喊一次。
+ * 为什么：该告警按 **agent 创建**触发，一个会话里每建一个 agent 都会喊 —— 十行同样的 warn
+ * 会把"响亮"变成"噪声"，而噪声的代价是所有告警一起被降权（本仓反复记录过的失败模式）。
+ * 去重只影响**重复次数**，不影响"第一次一定喊"，也不合并不同 status（两种零仍然分得清）。
+ */
+const LEAD_TOOLFACE_WARNED = new Set();
+function warnLeadToolFaceOnce(status, detail) {
+  if (LEAD_TOOLFACE_WARNED.has(status)) return;
+  LEAD_TOOLFACE_WARNED.add(status);
+  console.warn(`[expert-team] lead 工具面**未**收窄（${status}）：${detail}`);
+}
+export const _live = { pushActivity, phaseAccountingViolations, loggedPhases, authorityViolations, WRITE_TRACER, createWriteTracer, formatConflict, summarizeTool, parseLogLine, roleOfSub, mapRoleToSub, membersFromState, buildRoleSubMap, resolveSubRoles, childSessionTiming, SUB_HEADER_CACHE, workflowEventIndex, workflowChildLabels, workflowChildMeta, workflowRuns, WF_EVENT_CACHE, rememberSessionRun, sessionRunFor, runOwnerSession, SESSION_RUNS, parseTeamCommand, deriveMemberEntries, schemaViolations, runHealth, RUN_STALL_MS, scaffoldFingerprint, SCAFFOLD_REQUIRED, strandedTasks, settleStranded, IN_FLIGHT_STATUSES, normalizeCoverage, SCHEMA_WARN_SEEN, pushActivityEvent, DEFAULT_LIMITS, LIMITS, resolveLimits, capacityViolations, DEFAULT_ROUND_LIMITS, ROUND_LIMITS, ROUND_LIMIT_ENV, resolveRoundLimits, ROUND_LIMIT_OF_KIND, roundOf, isQualityTask, normTitle, roundLimitViolations, reworkLoopWriteGuard, mutateTasks, readStandingRules, appendStandingRule, rulesRun, scopeOverlapWarnings, applyTaskStatus, waitRun, eventFamily, verdictFromToken, normalizeRoleName, truncateCodepoints, filterRunScopedSubs, runCreatedAtMs, runLogTail, liveFiles, LIVE_FILES_CACHE, DEFAULT_ROLES, resolveTierGate, TIER_GATE_ENV, snapshotRun, settingsPath, loadSettingsSync, currentSettings, limitsBaseFromSettings, roundLimitsBaseFromSettings, effectiveTierGate: () => TIER_GATE, ensureSkillInstalled, ensurePresetInstalled, uninstallInstalled, buildSkillRegistration, parseSkillMarkdown, PLUGIN_VERSION, INSTALL_STAMP, runtimeSkillRegistered: () => RUNTIME_SKILL_REGISTERED, agentScopedToolNames, warnLeadToolFaceOnce, installHostSettings, hostValues, hostScope, hostSettingsNote, updateHostSettings, pickFileOnly, pickHostExpressible, buildHostSchema, hostBase, hostSchemaPaths, reapplySettingsDerived, currentSettings, mergeSettings };
 
 export function apply(ctx, config) {
   // 留一份 config：设置在运行时改变（官方面板 / 浮层）时要**用同一份 config** 重算上限与档位门，
@@ -4573,7 +4601,7 @@ export function apply(ctx, config) {
       // 为什么是这个时刻：`agent/created` 由宿主 `announce()` 在**创建时刻**发出；宿主自己对
       //   子代理也是在组合（setup）阶段调 `childCtx.tools.restrict(...)`（`dsh-subagent:554`）。
       //   即：**照抄宿主自己的时序**，不另造。
-      ctx.on('agent/created', ({ agent }) => {
+      ctx.on('agent/created', async ({ agent }) => {
         try {
           const agents = ctx.get('agents');
           // ① preset 必须是专家团（`composedPreset` 读活动作用域链，未写会话头也答得出）
@@ -4583,12 +4611,16 @@ export function apply(ctx, config) {
           const isRoot = agents?.roots?.().includes(agent) === true;
           if (!shouldNarrowLeadToolFace({ presetId, isRoot }).apply) return;   // 无关 agent：静默返回，不是失败
 
-          const knownNames = ctx.get('tools')?.view?.()?.restrictableNames;
+          // ③ **必须取 agent 作用域**的名字清单：`view()` 不传 scope = 全局视图，而模型可见工具
+          //    在 0.1.5 起由 preset 注册在 agent 平面 ⇒ 全局视图非空但缺那几个名字，收窄会静默失效
+          //    且给出"宿主里没有这些工具"的错误结论（拿真机日志换来的教训，详见 lead-toolface.js）。
+          const { knownNames, reason } = await agentScopedToolNames({ agentCtx: agent.ctx });
           const plan = planLeadToolFace({ platform: process.platform, knownNames });
           if (!plan.effective) {
             // **两种零都要出声**：`no-known-names`（我不知道宿主有什么）与
             // `nothing-to-deny`（宿主确实没有这些）不是同一件事，不能长成同一个样子。
-            console.warn(`[expert-team] lead 工具面**未**收窄（${plan.status}）：${plan.notes.join(' ')}`);
+            // 但同一状态**只喊一次**：每个会话都刷会把 warn 变成噪声，而噪声会让所有告警一起被降权。
+            warnLeadToolFaceOnce(plan.status, `${plan.notes.join(' ')}（清单来源：${reason}）`);
             return;
           }
           // 成功不打印：每个会话都刷一行会把 warn 变成噪声（本仓纪律：warn 必须保持可见）。

package/lib/lead-toolface.js

@@ -177,3 +177,49 @@ export function shouldNarrowLeadToolFace({ presetId, isRoot } = {}) {
   if (isRoot !== true) return { apply: false, reason: 'is-subagent' };
   return { apply: true, reason: 'lead' };
 }
+
+/**
+ * 取**agent 作用域**下的"可限制工具名清单"。
+ *
+ * ── 为什么必须带 scope（这一条是拿真机日志换来的）────────────────────────────
+ * 宿主 `tools.view(scope)` 的文档写得很直白：**不传 scope = 全局视图**。而 0.1.5 起模型可见的
+ * `bash/write/edit/grep/glob` 由 **preset 注册在 agent 平面**，不再是全局工具 ——
+ * 于是 `view()`（全局）会返回一个**非空但缺这几个名字**的集合，`planLeadToolFace` 据此报
+ * `nothing-to-deny`，收窄**静默失效**，而日志还在说"宿主里这些名字一个都不存在"（不属实）。
+ * 宿主自己的 `restrict()` 用的就是 `scopeOf(this.ctx)` —— 本函数照抄同一条取 scope 的路。
+ *
+ * scope 是 `@deepseek-ai/dsh-scope` 里的**模块私有 Symbol**（`Symbol("dsh.scope")`，不是
+ * `Symbol.for`），因此只能经该包的 `scopeOf()` 取得 ⇒ **动态 import**：仓库里解析不到该包时
+ * 返回 `knownNames: undefined`，由调用方如实报 `no-known-names`（"我不知道有什么"），
+ * 绝不退化成"宿主里没有这些工具"那种**不成立**的结论。
+ *
+ * @param {object} input
+ * @param {object} [input.agentCtx] - `agent.ctx`（scope 就挂在它上面）
+ * @param {object} [input.tools] - 工具服务（默认取 `agentCtx.tools`）
+ * @param {Function} [input.loadScope] - **仅供测试**注入 `@deepseek-ai/dsh-scope` 的加载器
+ * @returns {Promise<{knownNames: Set<string>|undefined, reason: string}>}
+ */
+export async function agentScopedToolNames({ agentCtx, tools, loadScope } = {}) {
+  const loader = typeof loadScope === 'function' ? loadScope : () => import('@deepseek-ai/dsh-scope');
+  let scopeOf;
+  try {
+    const mod = await loader();
+    scopeOf = mod && mod.scopeOf;
+  } catch (e) {
+    return { knownNames: undefined, reason: 'scope-module-unavailable:' + String((e && e.message) || e) };
+  }
+  if (typeof scopeOf !== 'function') return { knownNames: undefined, reason: 'scope-of-missing' };
+  const service = tools || (agentCtx && agentCtx.tools);
+  if (!service || typeof service.view !== 'function') return { knownNames: undefined, reason: 'tools-service-unavailable' };
+  let scope;
+  try { scope = scopeOf(agentCtx); } catch { return { knownNames: undefined, reason: 'scope-undetectable' }; }
+  if (scope === undefined) return { knownNames: undefined, reason: 'not-a-scoped-ctx' };
+  try {
+    const view = service.view(scope);
+    const names = view && view.restrictableNames;
+    const size = names && typeof names.size === 'number' ? names.size : 0;
+    return { knownNames: names, reason: size > 0 ? 'ok' : 'empty-scoped-view' };
+  } catch (e) {
+    return { knownNames: undefined, reason: 'view-failed:' + String((e && e.message) || e) };
+  }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yangdcm/dsh-expert-team",
-  "version": "1.2.0",
+  "version": "1.2.2",
   "description": "dsh「专家团」bundle：一句自然语言自动组建/持久化一支 12 角色多智能体团队，共享工作区协议 + 阶段门控编排 + 结构化交接 + 质量门禁/自动调度，实现者直接改代码并产出持久工件；带 live 团队浮层（质量门禁/覆盖率/工件预览）。 · Role-based multi-agent expert team for DeepSeek Harness: one sentence in, a staged and gated team delivery out.",
   "type": "module",
   "license": "MIT",