dsh-skill-tag 0.1.17 → 0.1.18

package/docs//346/212/200/350/203/275/345/217/221/347/216/260/344/270/216/345/256/277/344/270/273/345/257/271/351/275/220.md

@@ -181,6 +181,20 @@ tagging-skills 那 17 个长期漏网的原因（详见 `deny-shadow-覆盖率.m
 再发 —— 一个不合法候选会让**整个** `provider.list()` 抛错，宿主随即丢弃本插件的
 全部贡献（日志 `skill provider "skill-tag" skipped`），比漏一个更糟。
 
+**两段都要校验（2026-11-12，issue #6）**。上面这条当时只堵了「`denied` 里看不见
+的名字」那一段，**看得见的**那一段（照抄上游 summary）一直没校验，于是扫盘带进来的
+脏名字能原样发出去。触发链：`src/skill-scan.js` 把技能根下裸放的 `README-官方技能说明.md`
+当成技能（名字取文件名去 `.md`）→ 进 universe → `denied = universe − visible` 里也有它 →
+第一段照抄发射 → 宿主 `validateCandidate` 抛错 → **所有对话**每轮都失败，报
+`returned invalid skill name`。
+
+现在两层纵深防御：
+
+- `src/skill-scan.js`：根级裸放 `.md` **必须有 frontmatter** 才算技能，名字非 kebab-case
+  一律跳过 —— 与宿主 `discoverRoot` / `parseSkillFile` 同口径（宿主对同样的文件打的是
+  `skill file ... ignored: invalid skill name`）。
+- `lib/index.js`：第一段与第二段都过 `isSkillName()`，兜住任何别的来源漏进来的脏名字。
+
 ## 5. 约束（踩过的坑）
 
 - **cordis 服务必须声明式注入**：`export const inject` 里没有的服务，读 `ctx.xxx`

package/lib/index.js

@@ -101,8 +101,15 @@ export function createFilterProvider(sessionId, upstream, denied, ensureFresh, l
 			// 还是上游那份，影子只换掉 invocation 与 provider，右列能看出"它被屏蔽了"。
 			const shadows = [];
 			const emitted = new Set();
+			const invalid = new Set();
 			for (const summary of summaries) {
 				if (!summary || !denied.has(summary.name)) continue;
+				// 看得见 ≠ 合法（2026-11-12，issue #6）：扫盘会把技能根下裸放的
+				// `README-官方技能说明.md` 这类文件当成技能，名字取文件名去 `.md`。
+				// 旧实现只在第二段校验，这一段整条照抄 → 非法名字原样发出去 →
+				// 宿主 validateCandidate 抛错 → 整个 provider 作废 → **所有对话**
+				// 报 "returned invalid skill name" 且每轮都失败。两段都必须校验。
+				if (!isSkillName(summary.name)) { invalid.add(summary.name); continue; }
 				shadows.push({
 					...summary,
 					provider: PROVIDER_NAME,
@@ -123,10 +130,9 @@ export function createFilterProvider(sessionId, upstream, denied, ensureFresh, l
 			// 修法：屏蔽是**按名字**生效的，不需要知道那名字长什么样 —— 看不见的
 			// 名字直接造一条最小合法候选，就能在近层整条覆盖上游那份。
 			let backfilled = 0;
-			const invalid = [];
 			for (const name of denied) {
 				if (emitted.has(name)) continue;
-				if (!isSkillName(name)) { invalid.push(name); continue; }
+				if (!isSkillName(name)) { invalid.add(name); continue; }
 				shadows.push({
 					name,
 					description: `[blocked by skill-tag] ${name}`,
@@ -148,8 +154,8 @@ export function createFilterProvider(sessionId, upstream, denied, ensureFresh, l
 							+ ` shadowed=${emitted.size} backfilled=${backfilled}`
 							+ `${upstream.lastDegraded === true ? " hostSnapshot=skipped(nested)" : ""}`);
 					}
-					if (invalid.length > 0) {
-						logWarn(`[skill-tag] denied names skipped (not kebab-case): ${invalid.slice(0, 8).join(", ")}`);
+					if (invalid.size > 0) {
+						logWarn(`[skill-tag] names skipped (not kebab-case): ${[...invalid].slice(0, 8).join(", ")}`);
 					}
 				} catch { /* 观测不得影响主路径 */ }
 			}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-skill-tag",
-  "version": "0.1.17",
+  "version": "0.1.18",
   "description": "给 DSH 的技能（skill）打标签，用一条布尔表达式在「工作区 / 对话」两级选出要下发给模型的技能子集",
   "type": "module",
   "main": "lib/index.js",

package/src/skill-scan.js

@@ -70,6 +70,17 @@ function collectRoot(dir, depth, root, out) {
 	}
 }
 
+/**
+ * 合法技能名：kebab-case，与宿主 @deepseek-ai/dsh-skill 的 SKILL_NAME 同一正则。
+ *
+ * 为什么扫描器也要过这一关（2026-11-12，issue #6）：扫描结果会进 universe，
+ * 于是 denied = universe − visible 里也会出现它；只要这个名字非法，宿主
+ * validateCandidate 就会抛错、整个 provider 作废、**所有对话**都跑不下去。
+ * 宿主自己的文件系统 provider 早就这么做了（`skill file ... ignored: invalid
+ * skill name`），自扫描必须同口径，否则插件会把宿主本来就不认的名字当成技能。
+ */
+const SKILL_NAME_RE = /^[a-z0-9]+(?:-[a-z0-9]+)*$/;
+
 function pushSkill(out, mdPath, root, disabled, fallbackName) {
 	let fm = null;
 	try {
@@ -77,8 +88,15 @@ function pushSkill(out, mdPath, root, disabled, fallbackName) {
 	} catch {
 		return; // unreadable file — skip
 	}
+	// 裸放的根级 .md（flat skill）与宿主同口径：**必须有 frontmatter**。
+	// 没有 frontmatter 的 .md 只是一份普通文档（README、笔记…），不是技能；
+	// 旧实现会用文件名凭空造一个技能名，正是 issue #6 的来源。
+	const isFlat = fallbackName !== undefined;
+	if (isFlat && fm === null) return;
 	const name = fm?.name || fallbackName || path.basename(path.dirname(mdPath));
 	if (!name) return;
+	// 非法名字宁可丢掉：它既进不了宿主注册表，还会炸掉整批候选。
+	if (!SKILL_NAME_RE.test(name)) return;
 	out.push({
 		name,
 		description: fm?.description ?? "",