dsh-skill-tag 0.1.6 → 0.1.7

package/lib/index.js

@@ -18,7 +18,7 @@ import { skillTagDomain } from "../src/storage/schema.js";
 import { createDshRepository, serializeConversation, deserializeConversation } from "../src/storage/repository.js";
 import { createServices } from "../src/service.js";
 import { groupIdOf } from "../src/domain/model.js";
-import { workspaceDeniedSkills, normalizeWorkspacePath, activeTagsOf } from "../src/domain/workspace.js";
+import { workspaceDeniedSkills, workspaceSkillView, normalizeWorkspacePath, activeTagsOf } from "../src/domain/workspace.js";
 import { candidatesFor } from "../src/domain/match.js";
 import { mountApi } from "../src/api.js";
 import { scanSkillRoots } from "../src/skill-scan.js";
@@ -336,6 +336,54 @@ export function runtimeDeniedSkills(ws, universe, candidates) {
 	});
 }
 
+/**
+ * 运行时真正下发给 provider 的拒绝集：新口径 ∪ 旧口径兜底的**严格补集**。
+ *
+ * 2026-10-27 修正的 bug：此前调用方把两段结果**无条件并集**——
+ *   ① workspaceDeniedSkills（懂 manualSkills / disabledSkills）
+ *   ② computeDeniedSkillIds（只看标签条件，linked − candidates）
+ * ② 不知道"手动添加"，于是把「打了标签、但没被标签规则命中、靠手动加进对话」
+ * 的技能重新判成 deny，推翻 ① 的放行结论。本机唯一受害者是 tagging-skills
+ * （它只挂来源标签，靠「管理标签」右列手动添加）。
+ *
+ * 兜底原本的前提是「universe 扫不到这个名字，所以新口径没机会判定它」。
+ * 那就把这个前提写成代码 —— 两道闸：
+ *   ① 新口径已放行的名字（标签命中 ∪ 手动添加 − 禁用）绝不再 deny；
+ *   ② universe 里存在的名字已由新口径给出判定，兜底不重复表态。
+ * 剩下的才是真正需要兜底的历史残留（已卸载但标签还在的名字）。
+ *
+ * @param {{config?:object, manualSkills?:string[], disabledSkills?:string[]}} effective 生效配置
+ * @param {object} p
+ * @param {string[]} p.universe       本趟可发现的全部技能名
+ * @param {string[]} p.candidates     标签规则命中的技能名
+ * @param {string[]} p.linkedSkillIds 有标签关联的全部技能名（兜底段的输入）
+ * @returns {Set<string>}
+ */
+export function computeRuntimeDeniedSkillIds(effective, { universe, candidates, linkedSkillIds }) {
+	const view = workspaceSkillView({
+		config: effective?.config,
+		manualSkills: effective?.manualSkills,
+		disabledSkills: effective?.disabledSkills,
+		universe,
+		candidates,
+	});
+	const denied = workspaceDeniedSkills({
+		config: effective?.config,
+		manualSkills: effective?.manualSkills,
+		disabledSkills: effective?.disabledSkills,
+		universe,
+		candidates,
+	});
+	const allowed = new Set(view.visible);
+	const inUniverse = new Set(universe);
+	for (const skillId of computeDeniedSkillIds(effective?.config, linkedSkillIds, () => candidates)) {
+		if (allowed.has(skillId)) continue;
+		if (inUniverse.has(skillId)) continue;
+		denied.add(skillId);
+	}
+	return denied;
+}
+
 /**
  * 技能发现：与「模型实际看到的技能目录」对齐的唯一入口。
  *
@@ -660,15 +708,13 @@ export async function apply(ctx, config) {
 					config: effective.config,
 					active: activeTagsOf(effective.config),
 				});
-				for (const skillId of workspaceDeniedSkills({ ...effective, universe, candidates })) denied.add(skillId);
-				// 旧口径兜底：已被打标签的技能同样要按条件过滤（universe 扫不到时）
-				for (const skillId of computeDeniedSkillIds(
-					effective.config,
-					links.map((l) => l.skillId),
-					() => candidates,
-				)) {
-					denied.add(skillId);
-				}
+				// 当前生效配置下"该放行"的名字（标签命中 ∪ 手动添加 − 禁用）。
+				// 兜底段必须尊重它，否则会把手动添加的技能重新判成 deny。
+				for (const skillId of computeRuntimeDeniedSkillIds(effective, {
+					universe,
+					candidates,
+					linkedSkillIds: links.map((l) => l.skillId),
+				})) denied.add(skillId);
 			}
 		}
 		const after = [...denied].join(",");

package/package.json

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