dsh-toolbox-ui 1.6.1 → 1.6.3

package/lib/skills-frontmatter.js

@@ -25,15 +25,17 @@ const LEGACY_INVOCATION_KEYS = new Map([
 ])
 
 /**
- * 拆出 frontmatter 与正文，逐字照抄宿主 parseFrontmatter（dsh-skill-filesystem:780-806）。
+ * frontmatter 的字节范围——**解析与改写共用的唯一一套边界判据**。
  *
- * 宿主在这里的宽容度比一般实现低：首行必须**恰好**是 `---`（去尾随 `\r` 后比较），
- * 结束行同理。任何偏差都返回 ok:false，而不是猜一个边界。
+ * 为什么抽出来：读与写必须认同一个块。解析认得的块和改写动的块一旦分叉，后果是**静默改坏
+ * 用户的技能文件**，而且只在手写的、形状不标准的文件上才会踩到——测试里最容易漏的那一类。
  *
  * @param {string} raw - 技能文件全文。
- * @returns {{ ok: true, data: object, body: string } | { ok: false, code: string, message: string }}
+ * @returns {{ ok: true, fieldsStart: number, closingStart: number, bodyStart: number, eol: string }
+ *   | { ok: false, code: string, message: string }} fieldsStart/closingStart 界定字段区，
+ *   bodyStart 是正文起点，eol 是这份文件自己的行尾风格（插入新行时照用，别把 CRLF 写成混行）。
  */
-export function parseSkillFrontmatter(raw) {
+function frontmatterSpan(raw) {
   if (typeof raw !== 'string') {
     return { ok: false, code: 'frontmatter-missing', message: '技能内容必须是文本' }
   }
@@ -45,15 +47,37 @@ export function parseSkillFrontmatter(raw) {
     return { ok: false, code: 'frontmatter-missing', message: '技能缺少 YAML frontmatter：首行必须是 ---' }
   }
 
-  const start = firstLineEnd + 1
-  const closing = findClosingFrontmatter(raw, start)
+  const fieldsStart = firstLineEnd + 1
+  const closing = findClosingFrontmatter(raw, fieldsStart)
   if (closing === undefined) {
     return { ok: false, code: 'frontmatter-missing', message: '技能缺少 YAML frontmatter：找不到结束的 --- 行' }
   }
+  return {
+    ok: true,
+    fieldsStart,
+    closingStart: closing.start,
+    bodyStart: closing.bodyStart,
+    eol: raw[firstLineEnd - 1] === '\r' ? '\r\n' : '\n',
+  }
+}
+
+/**
+ * 拆出 frontmatter 与正文，逐字照抄宿主 parseFrontmatter（dsh-skill-filesystem:780-806）。
+ *
+ * 宿主在这里的宽容度比一般实现低：首行必须**恰好**是 `---`（去尾随 `\r` 后比较），
+ * 结束行同理。任何偏差都返回 ok:false，而不是猜一个边界。
+ *
+ * @param {string} raw - 技能文件全文。
+ * @returns {{ ok: true, data: object, body: string, span: object }
+ *   | { ok: false, code: string, message: string }}
+ */
+export function parseSkillFrontmatter(raw) {
+  const span = frontmatterSpan(raw)
+  if (!span.ok) return span
 
   let parsed
   try {
-    parsed = parse(raw.slice(start, closing.start))
+    parsed = parse(raw.slice(span.fieldsStart, span.closingStart))
   } catch (error) {
     return {
       ok: false,
@@ -64,7 +88,8 @@ export function parseSkillFrontmatter(raw) {
   if (typeof parsed !== 'object' || parsed === null || Array.isArray(parsed)) {
     return { ok: false, code: 'frontmatter-not-mapping', message: 'YAML frontmatter 必须是一个键值映射' }
   }
-  return { ok: true, data: parsed, body: raw.slice(closing.bodyStart) }
+  // span 一并交出去：改写方要用**同一套**边界（见 frontmatterSpan 的说明），不该自己再算一遍。
+  return { ok: true, data: parsed, body: raw.slice(span.bodyStart), span }
 }
 
 /** 找结束的 `---` 行（照抄宿主 findClosingFrontmatter:794-806）。 */
@@ -115,6 +140,121 @@ function frontmatterBoolean(data, key) {
   throw new TypeError(`frontmatter 字段 "${key}" 必须是布尔值`)
 }
 
+// ── 改写一个顶层布尔字段 ────────────────────────────────────────────────────
+//
+// 面板的「模型自动调用」开关就落在这里（写 SKILL.md 的 disable-model-invocation）。三条不变量：
+//
+//   1. **只动该动的那一行**：字段顺序、缩进、正文、行尾风格全部逐字保留。所以刻意不走
+//      `parse → stringify`——那会把用户的文件整体重排（diff 噪声、丢注释），而技能文件是
+//      **用户手写的资产**，不是我们的数据。
+//   2. **语义先于文本**：先用宿主那套规则（frontmatterBoolean）算出当前值，已经是要的那个值
+//      就原样返回 changed:false。于是「关掉」= 删掉那一行（缺省即 false），开关不留痕迹。
+//   3. **读不准就抛错**：值跨行、不是布尔、YAML 本身读不动、有宿主拒绝的驼峰键、键写在解析器
+//      认得而逐行扫描认不出的形状里（引号键 / 流式映射）、值带 YAML 锚点——一律拒绝并说清原因；
+//      另外**打开**时若那一行带行尾注释也拒绝（**关掉**不受注释拦阻：整行删除的边界是确定的，
+//      见下方那条「注释随行而去」的用例）。猜一个改法等于改坏用户的文件；说「我做不到」才是诚实的行为。
+
+/** frontmatter 字段区里的每一行（含它自己的 `\r`，不含 `\n`）。 */
+function frontmatterLines(raw, span) {
+  const lines = []
+  let lineStart = span.fieldsStart
+  while (lineStart < span.closingStart) {
+    const newline = raw.indexOf('\n', lineStart)
+    const end = newline < 0 || newline > span.closingStart ? span.closingStart : newline
+    lines.push({ start: lineStart, end, text: raw.slice(lineStart, end) })
+    lineStart = end + 1
+  }
+  return lines
+}
+
+/** 这一行是不是**顶层**的 `key:`——缩进行属于别的字段或嵌套结构，不算。 */
+function isTopLevelKey(text, key) {
+  if (/^[ \t]/.test(text)) return false
+  const content = text.endsWith('\r') ? text.slice(0, -1) : text
+  const colon = content.indexOf(':')
+  return colon >= 0 && content.slice(0, colon).trim() === key
+}
+
+/**
+ * 把 frontmatter 里的一个顶层布尔字段改成目标值，其余字节原样保留。
+ *
+ * @param {string} raw - 技能文件全文。
+ * @param {string} key - 顶层字段名（本插件用 disable-model-invocation）。
+ * @param {boolean} value - 目标值。
+ * @returns {{ text: string, changed: boolean }} changed 为 false 时 text 与输入逐字节相同。
+ * @throws {Error} 形状读不准时抛错，绝不猜一个改法。
+ */
+export function setFrontmatterBoolean(raw, key, value) {
+  const parsed = parseSkillFrontmatter(raw)
+  if (!parsed.ok) throw new Error(`无法改写 frontmatter：${parsed.message}`)
+
+  // 宿主对驼峰式旧键是**抛错**（整条技能被跳过，:850-861）。此时补一个规范键只会让两个键并存、
+  // 宿主照样跳过——面板不能给出「我改好了」的假象，只能点名让用户自己改。
+  for (const [legacy, canonical] of LEGACY_INVOCATION_KEYS) {
+    if (canonical === key && Object.hasOwn(parsed.data, legacy)) {
+      throw new Error(`frontmatter 里有宿主不支持的驼峰键 "${legacy}"（整条技能会被跳过）：请先手工改成 "${canonical}"`)
+    }
+  }
+
+  let current
+  try {
+    current = frontmatterBoolean(parsed.data, key)
+  } catch (error) {
+    throw new Error(`无法改写 ${key}：${error instanceof Error ? error.message : String(error)}。宿主会因此跳过整条技能，请先手工修正这一行`)
+  }
+  const desired = value === true
+  if ((current === true) === desired) return { text: raw, changed: false }
+
+  const lines = frontmatterLines(raw, parsed.span)
+  const at = lines.findIndex((line) => isTopLevelKey(line.text, key))
+  if (at < 0) {
+    // ⚠️ 追加的前提是「键**真的**不在」。行扫描认不出的形状是存在的：引号键（`"key": true`）与
+    // 流式映射（`{key: true}`）解析器都认得、宿主也读得到（面板照它显示权限），
+    // 而 isTopLevelKey 匹配不上那一行。此时追加会写出**第二个**同名键 → YAML 重复键让整份
+    // frontmatter 报废（宿主随即跳过整条技能），而用户要的那个方向根本没生效。
+    // 所以这里的判据取**解析结果**，不取行扫描结果：两者分叉时只能拒绝。
+    if (Object.hasOwn(parsed.data, key)) {
+      throw new Error(`无法改写 ${key}：这个键写在解析器认得、逐行改写认不出的形状里（引号键或流式映射），请手工修改`)
+    }
+    // 键确实不在、又要求打开：追加到 frontmatter 块**末尾**。插在 name/description 之前会把后面
+    // 每个字段的 diff 都推一行，而这里没有任何理由去动别人的字段。
+    return {
+      text: raw.slice(0, parsed.span.closingStart) + `${key}: true` + parsed.span.eol + raw.slice(parsed.span.closingStart),
+      changed: true,
+    }
+  }
+
+  const line = lines[at]
+  const next = lines[at + 1]
+  // 续行守卫：下一行只要缩进，值就跨了行——删掉首行会留下孤儿缩进行，整份 frontmatter 随之报废。
+  if (next !== undefined && /^[ \t]/.test(next.text)) {
+    throw new Error(`无法改写 ${key}：这个字段的值跨了多行（下一行有缩进），逐行改写会破坏 YAML，请手工修改`)
+  }
+
+  const hasCr = line.text.endsWith('\r')
+  const content = hasCr ? line.text.slice(0, -1) : line.text
+  const colon = content.indexOf(':')
+  const rest = content.slice(colon + 1)
+  // 锚点守卫：布尔字面量里不可能出现 `&`，所以走到这里还见到它只可能是 YAML 锚点（`key: &flag true`）。
+  // 这一行正给别处的 `*flag` 供值——整行删掉或就地换值都会让那个引用悬空、整份 frontmatter 报废。
+  // 两个方向都拦，所以放在方向分支之前（判据同样取解析结果那一侧的事实，不猜文本）。
+  if (rest.includes('&')) {
+    throw new Error(`无法改写 ${key}：这个字段的值带 YAML 锚点（&），别处可能正引用它，请手工修改`)
+  }
+
+  if (desired) {
+    // 能走到这里只可能是「写的是 false 那一类」（no / off / 0 / false）：就地把值换成 true，
+    // 键的位置不动。值是块标量或带行尾注释时不动手——那两种改法都得猜。
+    if (rest.trim() === '' || rest.includes('#')) {
+      throw new Error(`无法改写 ${key}：这一行不是「键: 简单值」的形状，请手工修改`)
+    }
+    const replaced = content.slice(0, colon + 1) + ' true' + (hasCr ? '\r' : '')
+    return { text: raw.slice(0, line.start) + replaced + raw.slice(line.end), changed: true }
+  }
+  // 关掉：整行连同它自己的换行一起删掉，其余行与行尾风格不变。
+  return { text: raw.slice(0, line.start) + raw.slice(line.end + 1), changed: true }
+}
+
 /**
  * 诊断一条技能原文，产出可展示的逐条诊断。
  *

package/lib/skills-registry.js

@@ -9,6 +9,7 @@
 import { access, stat } from 'node:fs/promises'
 import { homedir } from 'node:os'
 import { dirname, join, resolve } from 'node:path'
+import { readSkillDocument, resolveSkillTarget, SkillNotFoundError } from './skills-file-ops.js'
 
 // 宿主的 source 标签（见 dsh-skill-filesystem 的 roots()）：project-* 属项目级，
 // 其余属全局级。这是唯一可靠的作用域依据——本机就没有 ~/.dsh/skills，技能全在
@@ -110,24 +111,62 @@ async function presetScope(ctx) {
   }
 }
 
+/** 「这个技能读不到」：定位不到，或文件系统层面读不了（已被删、无权限、目录不可达）。 */
+function isUnreadable(error) {
+  if (error instanceof SkillNotFoundError) return true
+  return typeof error?.code === 'string' && /^E[A-Z]+$/.test(error.code)
+}
+
+/**
+ * 一行的调用权限——**一律来自技能文件本身**，不取宿主的发现缓存。
+ *
+ * 为什么不能用宿主 summary 里的那份：summary 是**发现缓存**（同一个文件被改写后它继续报旧值，
+ * 改名场景已实测复现，见 list() 里那段「陈旧行」的说明）。而调用权限正是本面板要**写**的东西：
+ * 写盘后显示回退，用户看到的是「点了没反应」；更糟的是把缓存里的旧值当事实展示。
+ * 行与详情因此都从这个文件的字节里解析——它们与用户看到的正文同源。
+ *
+ * 读不到就是 `null`（不知道）：宿主缓存可能指向一个已经被删/搬走的技能，此时「能不能被模型调用」
+ * 没有答案，编一个布尔值就是让面板撒谎。
+ *
+ * @param {{ name: string, root?: string, scope: 'global'|'project' }} row
+ * @returns {Promise<{ modelInvocable: boolean|null, userInvocable: boolean|null }>}
+ */
+async function invocationOf(row) {
+  const unknown = { modelInvocable: null, userInvocable: null }
+  if (typeof row?.name !== 'string' || typeof row?.root !== 'string' || row.root === '') return unknown
+  try {
+    const target = await resolveSkillTarget({ name: row.name, root: row.root, scope: row.scope })
+    const { invocation } = await readSkillDocument(target)
+    return {
+      modelInvocable: invocation?.modelInvocable ?? null,
+      userInvocable: invocation?.userInvocable ?? null,
+    }
+  } catch (error) {
+    // 只吞「读不到」这一类；其余错误照原样抛——把 bug 说成「未知」等于把缺陷埋起来。
+    if (isUnreadable(error)) return unknown
+    throw error
+  }
+}
+
 /**
  * 把宿主的一条 SkillSummary 投影成面板行。
  * @param {object} summary - 宿主 SkillSummary。
  * @param {{ cwd?: string }} options - 该条目所属的查询作用域。
- * @returns {object} 面板行。
+ * @returns {Promise<object>} 面板行。
  */
-function toView(summary, { cwd }) {
-  const invocation = summary.invocation ?? {}
+async function toView(summary, { cwd }) {
   const located = locateSkill(summary)
+  const scope = scopeOfSource(summary.source)
+  // 调用权限**不读** summary.invocation：那一份来自宿主的发现缓存（见 invocationOf 的说明）。
+  const invocation = await invocationOf({ name: summary.name, root: located.root, scope })
   return {
     name: summary.name,
     description: summary.description ?? '',
     ...(summary.whenToUse !== undefined ? { whenToUse: summary.whenToUse } : {}),
-    modelInvocable: invocation.modelInvocable !== false,
-    userInvocable: invocation.userInvocable !== false,
+    ...invocation,
     source: summary.source ?? 'unknown',
     provider: summary.provider ?? 'unknown',
-    scope: scopeOfSource(summary.source),
+    scope,
     // 根目录与形态推不出来就不填（上层据此提示「无法定位」，而不是作用在错的目录上）。
     ...(located.root !== undefined ? { root: located.root, kind: located.kind } : {}),
     ...(cwd !== undefined ? { cwd } : {}),
@@ -155,7 +194,7 @@ async function queryScope(skills, options) {
     ...(signal === undefined ? {} : { signal }),
   }
   const summaries = await skills.list(request)
-  return summaries.map((summary) => toView(summary, { cwd }))
+  return await Promise.all(summaries.map((summary) => toView(summary, { cwd })))
 }
 
 /**
@@ -199,8 +238,8 @@ async function listSkillViews(ctx, options = {}) {
  *
  * 为什么删掉而不是留着：`ctx.skills.get(name)` 按 rank 返回**生效者**，与面板
  * 「(name, root) 精确操作」的模型（D8）冲突——点被遮蔽的那行会读到生效者的正文。
- * 现在的 detail 走 `resolveTargetReference` + `resolveSkillTarget` + `readSkillFile`，
- * 既精确到根，又能读出已停用技能的 `.disabled` 正文。
+ * 现在的 detail 走 `resolveTargetReference` + `resolveSkillTarget` + `readSkillDocument`，
+ * 既精确到根，又能读出已停用技能的 `.disabled` 正文（并顺带从同一份字节里给出调用权限）。
  */
 
 // ── 安装目标（skillTargets）───────────────────────────────────────────────
@@ -402,4 +441,4 @@ async function resolveTargetReference(ctx, reference) {
  */
 const skillRowKey = (row) => `${row?.source || 'unknown'}\u0000${row?.cwd || ''}\u0000${row?.name || ''}`
 
-export { listSkillViews, queryScope, resolveTargetReference, skillRowKey, skillTargetsView }
+export { invocationOf, listSkillViews, queryScope, resolveTargetReference, skillRowKey, skillTargetsView }

package/lib/typert.js

@@ -106,6 +106,7 @@ export const TYPERT = {
     mvs('list', [skillPayloadParam], 'SkillListResult'),
     mvs('detail', [skillPayloadParam], 'SkillDetailResult'),
     mvs('setEnabled', [skillPayloadParam], 'SkillMutationResult'),
+    mvs('setModelInvocable', [skillPayloadParam], 'SkillMutationResult'),
     // ⚠️ 不能叫 `remove`：客户端适配器（dsh-api-gateway 的 RemoteNamespaceService）自己就有原型
     // 方法 `remove(kind, method, token)`，`ctx.remote.$mount` 会把「与命名空间服务自身成员重名」
     // 的 endpoint 直接拒掉——报 `client api: method "skillsManager/remove" conflicts with its

package/lib/whats-new.js

@@ -184,4 +184,30 @@ export const WHATS_NEW = [
       },
     ],
   },
+  {
+    version: '1.6.2',
+    items: [
+      {
+        text: '**技能能一键改成「只在你手动点名时可用」**：详情页新增「模型自动调用」开关，直接改 `SKILL.md` 的 `disable-model-invocation`，不必再手工翻文件。关闭时它把那**一行**删掉（缺省即允许，所以不留痕迹）、只在需要时于 frontmatter 末尾追加一行——字段顺序与正文都不动；形状读不准（值跨行，或要在带行尾注释的那一行上打开它）时它会**拒绝**并说明原因，而不是猜一个改法',
+        where: '技能 → 选中一条技能 → 概览',
+      },
+      {
+        text: '**调用权限现在如实显示**：以前已停用的技能一律被显示成「模型不可调用 / 用户不可调用」——那是凭空的断言（它的权限只有那份 `.disabled` 文件能回答）。现在按技能文件里真正写着的内容显示，读不出来（frontmatter 非法、宿主会整条跳过它）就显示「未知」而不猜一个值',
+        where: '技能列表的徽章与详情里的属性',
+      },
+      {
+        text: '**修好「有新版本却从不提示」**：更新检查原先读仓库的 release feed，而那里唯一的 latest 被另一条线的 pre-release 规则占着，于是本线的提示恒为「已是最新」——1.6.2 发出来你也看不到它。现在问的是本包在 npm 上的最新版。⚠️ 这条修复装在**这一版**里，所以 1.6.1 及更早的用户请手动 `dsh plugin --profile web update dsh-toolbox-ui` 升一次；从这一版起提示恢复正常',
+        where: '面板顶部的更新提示条',
+      },
+    ],
+  },
+  {
+    version: '1.6.3',
+    items: [
+      {
+        text: '**这一版没有功能改动**：改的是门面与发布资产——README 重写（更短、只留两张关键截图）、换上新图标，仓库也从旧名改成了 `dsh-toolbox-ui`，与包名终于对上。旧地址 GitHub 会自动跳转，已有的安装、书签与 `github:` 链接都不受影响',
+        where: '仓库主页',
+      },
+    ],
+  },
 ]

package/package.json

@@ -1,14 +1,14 @@
 {
   "name": "dsh-toolbox-ui",
-  "version": "1.6.1",
+  "version": "1.6.3",
   "description": "Responsive MCP management UI and skills manager for DeepSeek Harness Web — floating panel, JSON import, profile-backed Host Remote management, and a skill market with a hard license gate.",
   "repository": {
     "type": "git",
-    "url": "git+https://github.com/Imzl-zl/dsh-mcp-manager-ui.git"
+    "url": "git+https://github.com/Imzl-zl/dsh-toolbox-ui.git"
   },
-  "homepage": "https://github.com/Imzl-zl/dsh-mcp-manager-ui#readme",
+  "homepage": "https://github.com/Imzl-zl/dsh-toolbox-ui#readme",
   "bugs": {
-    "url": "https://github.com/Imzl-zl/dsh-mcp-manager-ui/issues"
+    "url": "https://github.com/Imzl-zl/dsh-toolbox-ui/issues"
   },
   "type": "module",
   "main": "lib/index.js",