dsh-skill-tag 0.1.6 → 0.1.8

package/README.md

@@ -30,23 +30,24 @@ office * plugin       # 只要 office 包带来的、且是插件相关
 ## 界面预览
 
 > 全部是真实实例截图（105 个技能 / 48 个标签 / 一个开了筛选的工作区），没有 mock。
-> 图片源文件在 `images/`（不进 npm 包，README 里用 raw 绝对地址，GitHub 与 npm 都能显示）。
+> 图片源文件在 `images/`（不进 npm 包，README 里用 jsDelivr 绝对地址，GitHub 与 npm 都能显示；
+> 之所以不走 `raw.githubusercontent.com`，是因为部分网络到该域名的 IPv6 不通，图片会间歇性加载失败）。
 
 ### 图 1 · 入口：会话行「…」→「管理标签」
 
-<img src="https://raw.githubusercontent.com/aawef/dsh-skill-manager-tags/main/images/session-menu.png" width="240" alt="会话行菜单里的管理标签入口">
+<img src="https://cdn.jsdelivr.net/gh/aawef/dsh-skill-manager-tags@main/images/session-menu.png" width="240" alt="会话行菜单里的管理标签入口">
 
 对话级的配置就挂在会话行的「…」菜单最下面，不用去翻设置页。
 
 ### 图 2 · 入口：命令面板里的 `/skill-tag-manage`
 
-<img src="https://raw.githubusercontent.com/aawef/dsh-skill-manager-tags/main/images/command-palette.png" width="820" alt="命令面板中的管理对话标签命令">
+<img src="https://cdn.jsdelivr.net/gh/aawef/dsh-skill-manager-tags@main/images/command-palette.png" width="820" alt="命令面板中的管理对话标签命令">
 
 在输入框敲 `/` 也能直接开「管理对话标签」——同一个界面，省一次点菜单。
 
 ### 图 3 · 打标签：设置 →「Skill 标签」
 
-<img src="https://raw.githubusercontent.com/aawef/dsh-skill-manager-tags/main/images/tag-settings.png" width="620" alt="Skill 标签管理页：新建标签、按来源分组的标签列表">
+<img src="https://cdn.jsdelivr.net/gh/aawef/dsh-skill-manager-tags@main/images/tag-settings.png" width="620" alt="Skill 标签管理页：新建标签、按来源分组的标签列表">
 
 三个页签分工明确：**tag 联系 skill**（一个标签包含哪些技能）、**skill 联系 tag**
 （一个技能拥有哪些标签）、**编辑标签组**（写表达式存模板）。
@@ -54,7 +55,7 @@ office * plugin       # 只要 office 包带来的、且是插件相关
 
 ### 图 4 · 一个技能看它身上有哪些标签
 
-<img src="https://raw.githubusercontent.com/aawef/dsh-skill-manager-tags/main/images/skill-to-tags.png" width="640" alt="skill 联系 tag：左列技能、右列标签勾选">
+<img src="https://cdn.jsdelivr.net/gh/aawef/dsh-skill-manager-tags@main/images/skill-to-tags.png" width="640" alt="skill 联系 tag：左列技能、右列标签勾选">
 
 左列 `技能 (105)` 每行标着它有几个标签；右边勾选**即时保存**，不用点确认。
 右列会把该技能**已有的标签排在列表最前面**（并保持它们原来的相对顺序），所以不用滚动
@@ -63,7 +64,7 @@ office * plugin       # 只要 office 包带来的、且是插件相关
 
 ### 图 5 · 工作区「管理标签」：左写条件，右就是下发名单
 
-<img src="https://raw.githubusercontent.com/aawef/dsh-skill-manager-tags/main/images/workspace-filter.png" width="900" alt="工作区管理标签双列界面">
+<img src="https://cdn.jsdelivr.net/gh/aawef/dsh-skill-manager-tags@main/images/workspace-filter.png" width="900" alt="工作区管理标签双列界面">
 
 左列是标签条件，下面实时显示**解析结果**和状态「有条件筛选（严格子集）」；
 右列 `技能 (54)` 就是该工作区下发给模型的全部技能，每行可以「查看 tag」「禁用」，
@@ -72,7 +73,7 @@ office * plugin       # 只要 office 包带来的、且是插件相关
 
 ### 图 6 · 对话「管理标签（对话）」：可以有自己的一份
 
-<img src="https://raw.githubusercontent.com/aawef/dsh-skill-manager-tags/main/images/conversation-filter.png" width="900" alt="对话级管理标签界面与继承来源">
+<img src="https://cdn.jsdelivr.net/gh/aawef/dsh-skill-manager-tags@main/images/conversation-filter.png" width="900" alt="对话级管理标签界面与继承来源">
 
 同一个工作区下的某个对话想单独收窄，就在这里改：它会记下**继承来源**与时间
 （`改动不影响工作区`），左下角一键「继承工作区配置」覆盖回来。

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.8",
   "description": "给 DSH 的技能（skill）打标签，用一条布尔表达式在「工作区 / 对话」两级选出要下发给模型的技能子集",
   "type": "module",
   "main": "lib/index.js",