dsh-maoying 0.1.20 → 0.1.21

package/presets/maoying/agent.cordis.v2.yml

@@ -188,6 +188,11 @@
                   toolName: outline
                   backgroundMode: continuable
                   agentPreset: maoying-outline
+              # 命名注意：toolName: write 与 tool-fs 提供的文件写工具 write 同名
+              # （'write' 一名双义）。宿主 registry 对同名工具的处理仓内无法核实，
+              # 故不改 toolName（行为风险）；restrict.mjs MAIN_AGENT_ALLOW 单条
+              # 'write' 即同时放行两义，改名前须先核实 registry 并与 v1/工具面
+              # 一起改（详见 agent.cordis.yml 的 delegation-write 行注释）。
               - id: delegation-write
                 name: "@deepseek-ai/dsh-tool-subagent"
                 config:

package/presets/maoying/agent.cordis.yml

@@ -110,6 +110,9 @@
   disabled: !!js process.platform !== 'win32'
 
 # ── filesystem ──────────────────────────────────────────────────────────────
+# 命名注意：tool-fs 的文件写工具名是 write，与下方 delegation 组 delegation-write
+# 以 toolName: write 注册的委派工具同名（'write' 一名双义——restrict.mjs 的
+# MAIN_AGENT_ALLOW 单条同时放行两义，详见 delegation-write 行注释）。
 
 - id: tool-fs
   name: '@deepseek-ai/dsh-tool-fs'
@@ -232,6 +235,12 @@
             - todo_write
 
     # 写手：章节正文
+    # 命名注意：toolName: write 与本文件上方 filesystem 区 tool-fs 提供的文件
+    # 写工具 write 同名——主编工具面里 'write' 一名双义（文件写 vs 本委派）。
+    # 宿主 registry 对同名工具的处理仓内无法核实，故不改 toolName（行为风险）；
+    # restrict.mjs 的 MAIN_AGENT_ALLOW 单条 'write' 即同时放行两义。改名前必须
+    # 先核实宿主 registry 对同名工具的处理，并连同 restrict.mjs 与 filesystem
+    # 区一起改。
     - id: delegation-write
       name: '@deepseek-ai/dsh-tool-subagent'
       config:

package/presets/maoying/restrict.mjs

@@ -17,8 +17,11 @@
  * web_fetch（考据委派 research）、skill 目录（registry deny 堵 ~1.3k 注入）、
  * 以及未列出的一切通用委派与工作流工具。
  *
- * write / edit 保留在白名单（见 MAIN_AGENT_ALLOW 尾段）：主编需要亲自做小修
- * 小补（改 phase、补 frontmatter、微调摘要），这些活不值得委派一次。
+ * write / edit 保留在白名单（见 MAIN_AGENT_ALLOW）：主编需要亲自做小修
+ * 小补（改 phase、补 frontmatter、微调摘要），这些活不值得委派一次。注意
+ * 'write' 一名双义——既是 tool-fs 的文件写工具，也是 delegation-write 以
+ * toolName: write 注册的委派工具（宿主 registry 对同名工具的处理仓内无法
+ * 核实，改名前必须先核实；详见 MAIN_AGENT_ALLOW 内注释）。
  *
  * Host 工具（mcp__*、插件工具）一律不硬编码——是否挂载是检测期事实，
  * 硬编码会让未装对应插件的宿主直接出错。本文件只留 dsh 核心工具。
@@ -36,6 +39,11 @@ export const inject = ['systemPrompt', 'tools']
 const MAIN_AGENT_ALLOW = new Set([
   // ── 角色委派 ─────────────────────────────────────────────────────
   'outline',
+  // 命名注意：'write' 一名双义——delegation-write 以 toolName: write 注册的
+  // 委派工具，与 tool-fs 提供的文件写工具（agent.cordis*.yml 的 filesystem 区）
+  // 同名。白名单按工具名匹配，这里单条 'write' 即同时放行两义。宿主 registry
+  // 对同名工具的处理仓内无法核实，故不改名；改名前必须先核实 registry 行为，
+  // 并连同两份 agent.cordis*.yml 的 toolName/filesystem 一起改。
   'write',
   'review',
   'lore',
@@ -61,7 +69,9 @@ const MAIN_AGENT_ALLOW = new Set([
   'job_list',
   'job_kill',
   // ── 主编亲笔（小修小补不过委派） ────────────────────────────────
-  'write',
+  // 这里只列 'edit'：'write'（文件写语义正属此区）已由上方「角色委派」区的
+  // 同名条目放行——同一个 Set 里写两遍会被静默去重，反而掩盖「'write' 一名
+  // 双义（文件写 vs 委派工具）」这一事实，故显式只写一处并加注释。
   'edit',
 ])
 
@@ -84,6 +94,9 @@ let warned = false
 function isDelegatedChild(agent) {
   const header = agent?.session?.header
   if (header === undefined) return false
+  // 括号必须显式：?? 的优先级低于比较运算符，裸写 delegationDepth ?? 0 > 0
+  // 会被解析成 delegationDepth ?? (0 > 0)——当前取值侥幸同义，显式括号防未来
+  // 改默认值/比较符时悄悄变义。
   return header.origin === 'subagent' || (header.delegationDepth ?? 0) > 0
 }
 

package/scripts/smoke-client.mjs

@@ -0,0 +1,128 @@
+#!/usr/bin/env node
+/**
+ * client.js 冒烟（npm run smoke，CI 同跑）：不启浏览器、零依赖，三步校验
+ * lib/client.js 的可加载性与导出面契约——
+ *   ① 语法可编译：整份源码 new Function 包裹，解析失败即红；
+ *   ② ModuleLoader 拦截：stub window / document 求值，截获 load({ id, factory })，
+ *      id 必须是 dsh-maoying（图行 id 约定，挂错名宿主首载即 import 报错）；
+ *   ③ factory 求值：stub react（五个 hook + createElement + Component——
+ *      class extends 在定义期就要 Component）与宿主 UI 原语；react-dom/client
+ *      缺席由 factory 内部 catch（Node 下拿不到属正常）。导出面必须 =
+ *      { inject: 含 "slots" 的数组, apply: 函数 }，且 apply 两次求值：
+ *      带 slots stub 时应把入口挂到 sidebar.footer.action 键控槽（id
+ *      maoying-creator），不带 slots 时护栏折返也不炸——两次都返回 disposer
+ *      函数且可安全调用。
+ * 任何一步失败 exit 1（console.error 中文说明）。
+ */
+import { readFileSync } from 'node:fs'
+import path from 'node:path'
+import { fileURLToPath } from 'node:url'
+
+const fail = (message) => {
+  console.error(`smoke: ${message}`)
+  process.exit(1)
+}
+
+const pluginRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..')
+const source = readFileSync(path.join(pluginRoot, 'lib', 'client.js'), 'utf8')
+
+// ① 语法可编译
+let compiled
+try {
+  compiled = new Function('window', 'document', source)
+} catch (err) {
+  fail(`lib/client.js 语法不可编译：${err.message}`)
+}
+
+// ② 求值并截获 ModuleLoader.load（querySelector 给真值走 mountCss 幂等跳过支，
+//    Node 下不触碰任何真实 DOM）
+let definition = null
+const windowStub = {
+  __ModuleLoader__: {
+    load(def) {
+      definition = def
+    },
+  },
+}
+const documentStub = { currentScript: null, querySelector: () => ({}) }
+try {
+  compiled(windowStub, documentStub)
+} catch (err) {
+  fail(`lib/client.js 顶层求值失败：${err.message}`)
+}
+if (!definition || typeof definition.factory !== 'function') {
+  fail(`未截获 __ModuleLoader__.load({ factory })，实得：${JSON.stringify(definition)?.slice(0, 120)}`)
+}
+if (definition.id !== 'dsh-maoying') {
+  fail(`插件 id 应为 dsh-maoying（图行 id 约定），实得「${definition.id}」`)
+}
+
+// ③ factory 求值：导出面 + apply 双路
+const reactStub = {
+  createElement: (...args) => ({ $$el: args }),
+  useState: (initial) => [initial, () => {}],
+  useEffect: () => {},
+  useCallback: (fn) => fn,
+  useRef: (initial) => ({ current: initial }),
+  useMemo: (fn) => fn(),
+  Component: class Component {},
+}
+const requireStub = (name) => {
+  if (name === 'react') return reactStub
+  // 宿主 UI 原语：factory 只做 Object.assign 拷贝，空对象即可（缺名图标渲染期
+  // 才会用到，本冒烟不渲染）。
+  if (name === '@deepseek-ai/dsh-client-ui-primitives') return {}
+  // react-dom/client 由 factory 内部 catch（返回 null），这里抛错即模拟缺席；
+  // 其余未知模块视为依赖面外溢，直接红。
+  throw new Error(`smoke require stub: 意外模块「${name}」`)
+}
+
+let exportsSurface = null
+try {
+  exportsSurface = definition.factory(requireStub)
+} catch (err) {
+  fail(`factory 求值失败：${err.message}`)
+}
+if (!exportsSurface || typeof exportsSurface !== 'object') fail('factory 未返回导出对象')
+if (!Array.isArray(exportsSurface.inject) || !exportsSurface.inject.includes('slots')) {
+  fail(`exports.inject 应为含 "slots" 的依赖数组，实得：${JSON.stringify(exportsSurface.inject)}`)
+}
+if (typeof exportsSurface.apply !== 'function') fail('exports.apply 应为函数')
+
+// apply ①：带 slots stub——入口应挂到 sidebar.footer.action 键控槽
+const injectedSlots = []
+const registeredMetas = []
+let registerReturns = 0
+const slotsStub = {
+  inject(slot, mount) {
+    injectedSlots.push(slot)
+    const component = mount() // 触发 register：只求值组件引用，不渲染
+    if (component === undefined) fail('slots.inject 的 mount 工厂未返回 register 结果')
+    return () => {}
+  },
+  register(meta) {
+    registeredMetas.push(meta)
+    registerReturns += 1
+    return { disposed: false }
+  },
+}
+const loggerWarns = []
+const disposerA = exportsSurface.apply({
+  logger: { warn(...args) { loggerWarns.push(args.map(String).join(' ')) } },
+  slots: slotsStub,
+})
+if (typeof disposerA !== 'function') fail('apply(ctx) 应返回 disposer 函数')
+if (injectedSlots.join(',') !== 'sidebar.footer.action') {
+  fail(`入口应注入 sidebar.footer.action 槽，实得：${injectedSlots.join(',') || '（无）'}`)
+}
+if (registerReturns !== 1 || registeredMetas[0]?.id !== 'maoying-creator') {
+  fail(`应注册 id 为 maoying-creator 的入口，实得：${JSON.stringify(registeredMetas)}`)
+}
+disposerA() // disposer 必须可安全调用（幂等清理不抛）
+
+// apply ②：无 slots（护栏未声明该服务）——折返不炸，照样给出 disposer
+const disposerB = exportsSurface.apply({ logger: { warn() {} } })
+if (typeof disposerB !== 'function') fail('apply 无 slots 时也应返回 disposer（护栏折返语义）')
+disposerB()
+
+console.log('smoke 通过：client.js 编译 ✓ ModuleLoader 截获 ✓ 导出面 { inject, apply } ✓ slots 注入与 disposer ✓')