@omdp/dsh-connector 0.3.0 → 0.3.2

package/index.js

@@ -41,7 +41,7 @@ const TRANSPORTS = new Set(['stdio', 'streamable-http'])
 // patch file itself (NUL, control chars). Tab/newline are already rejected by
 // the single-token rules; this catches the rest defensively.
 const CONTROL_CHARS = /[\x00-\x1f\x7f]/
-export const inject = ['webServer']
+export const inject = ['webServer', 'settings']
 
 // ── MCP 工具过滤（tool-filter, 0.3.0 新增）────────────────────────────
 // dsh-mcp-client 的 Config schema 是封闭的：未知键会拒收并导致下次启动失败，
@@ -924,31 +924,36 @@ async function readBody(req) {
 
 export function apply(ctx) {
   // ── 工具过滤三件套之 (0)：settings namespace 注册 + 快照缓存 ──
-  // settings 服务可选（无 provider 时 ctx.inject 回调不跑，过滤保持全放行）。
+  // settings 经声明式 inject 依赖保证就绪（见上方 export const inject），
+  // 这里同步拿服务；万一缺失不炸，过滤保持全放行（零回归）。
+  // 注：cordis 的 ctx.inject(deps, cb) 回调按 (ctx, config) 调用，不是服务展开，
+  // 不能拿回调参数当 service 用 —— 0.3.0 曾因此 register 从未落地。
+  const settings = ctx.get('settings')
   let toolFilters = {}
   const refreshFilters = () => { toolFilters = readToolFilters(ctx) }
-  ctx.inject(['settings'], (settingsCtx) => {
-    settingsCtx.settings.register(CONNECTOR_SETTINGS_NS, ToolFilterSchema, { base: { toolFilters: {} } })
-    refreshFilters()
-    settingsCtx.effect(() => settingsCtx.settings.watch(() => refreshFilters()), 'connector: watch tool filters')
-  })
+  if (settings) {
+    try {
+      const scope = settings.register(CONNECTOR_SETTINGS_NS, ToolFilterSchema, { base: { toolFilters: {} } })
+      refreshFilters()
+      ctx.effect(() => scope.watch(() => refreshFilters()), 'connector: watch tool filters')
+    } catch (error) {
+      console.error('[dsh-connector] register tool filter settings failed:', error?.message ?? error)
+    }
+  }
 
-  // ── 三件套之 (1)：prompt 层过滤 —— 被滤掉的工具不进模型 schema ──
-  // tools provider 在每次 assembly 时求值，读最新快照；无 tools/systemPrompt
-  // 服务时跳过（ctx.get 可选读，失败隔离：connector 照常提供设置页）。
+  // ── 三件套之 (1)：prompt 层过滤 —— assemble waterfall 真替换 ──
+  // 注意：systemPrompt.tools(provider) 是追加合并语义（各 provider 输出拼在一起），
+  // 只能加不能减，做减法必须监听 system-prompt/assemble，await next() 后替换 tools 数组。
+  // 成熟参照：hyqhyq3/dsh-mcp-manager 就是 ctx.on('system-prompt/assemble', …, { global: true })。
+  // { global: true } 让 scope filter 放行（见 dsh-scope scopeTarget 的 hook.global 短路）。
   try {
-    const systemPrompt = ctx.get('systemPrompt')
-    if (systemPrompt && typeof systemPrompt.tools === 'function') {
-      ctx.effect(() => systemPrompt.tools(() => {
-        let schemas = []
-        try {
-          const tools = ctx.get('tools')
-          schemas = tools ? tools.schemas() : []
-        } catch { return { schemas: [] } }
-        const kept = schemas.filter((s) => isToolAllowed(toolFilters, s && s.name))
-        return { schemas: kept, knownNames: schemas.map((s) => s && s.name).filter(Boolean) }
-      }), 'connector: mcp tool filter provider')
-    }
+    ctx.on('system-prompt/assemble', async (_assembly, _context, next) => {
+      const assembled = await next()
+      if (!assembled || !Array.isArray(assembled.tools)) return assembled
+      const kept = assembled.tools.filter((t) => isToolAllowed(toolFilters, t && t.name))
+      if (kept.length === assembled.tools.length) return assembled
+      return { ...assembled, tools: kept }
+    }, { global: true })
   } catch {}
   // ── 三件套之 (2)：执行期硬拦截 —— 补 provider 漏网（如 guard 注册时序）──
   try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@omdp/dsh-connector",
-  "version": "0.3.0",
+  "version": "0.3.2",
   "description": "Unified DeepSeek Harness connector: edit MCP servers (cordis.patch.yml), user skills (~/.dsh/skills) and browse the ModelScope MCP/Skills marketplaces from one Web UI settings page.",
   "type": "module",
   "main": "index.js",