dsh-skill-tag 0.1.5 → 0.1.7

package/README.md

@@ -57,6 +57,8 @@ office * plugin       # 只要 office 包带来的、且是插件相关
 <img src="https://raw.githubusercontent.com/aawef/dsh-skill-manager-tags/main/images/skill-to-tags.png" width="640" alt="skill 联系 tag：左列技能、右列标签勾选">
 
 左列 `技能 (105)` 每行标着它有几个标签；右边勾选**即时保存**，不用点确认。
+右列会把该技能**已有的标签排在列表最前面**（并保持它们原来的相对顺序），所以不用滚动
+去列表里找；顶部搜索框过滤之后，"已拥有优先"依然成立。
 这是配条件前最常开的一页：先看清楚"这个技能到底被哪些标签命中"。
 
 ### 图 5 · 工作区「管理标签」：左写条件，右就是下发名单
@@ -65,7 +67,8 @@ office * plugin       # 只要 office 包带来的、且是插件相关
 
 左列是标签条件，下面实时显示**解析结果**和状态「有条件筛选（严格子集）」；
 右列 `技能 (54)` 就是该工作区下发给模型的全部技能，每行可以「查看 tag」「禁用」，
-📁 直接打开技能所在文件夹。底部还能「+ 精细化添加技能」「查看被禁用的技能」。
+📁 直接打开技能所在文件夹。列表顶部的**搜索框**只过滤显示（标题会变成 `技能 (匹配/总数)`），
+不改动实际下发的技能集合。底部还能「+ 精细化添加技能」「查看被禁用的技能」。
 
 ### 图 6 · 对话「管理标签（对话）」：可以有自己的一份
 
@@ -120,8 +123,12 @@ dsh plugin --profile <your-profile> add github:aawef/dsh-skill-manager-tags
 
 ### 本地开发式安装
 
-如果你直接改源码，注意**源码目录和 profile 的 `node_modules` 里那份是两个物理文件**，
-改完必须手动复制并校验哈希：
+运行时加载的是 profile 里 `node_modules/<插件名>` 那一份，不是你的源码目录。两种接法：
+
+- **junction（推荐）**：让 `~/.dsh/profiles/web/my-plugins/<插件名>` 指向你的源码目录，
+  pnpm 会把它硬链接进 `node_modules` —— 三个路径是**同一个 inode、同一份文件**，
+  改完内容立刻生效，不需要复制；只有**新增/删除文件**才要在 profile 里重跑一次 `pnpm install`。
+- **物理拷贝**：如果当初是手动 `cp` 进去的，那就是两份独立文件，改完必须再 `cp` 一次并校验哈希：
 
 ```bash
 S=/path/to/your/dsh-skill-tag

package/lib/client.js

@@ -103,6 +103,20 @@ window.__ModuleLoader__.load({
 			return out;
 		}
 
+		/**
+		 * 「技能已有的标签排前面」：稳定分区，不改变各分区内部的原有顺序。
+		 * 只排显示顺序——勾选状态与写库内容都不受影响。
+		 * @param {Array<{tagId:string}>} tags 全部标签（顺序来自服务端）
+		 * @param {Iterable<string>} ownedIds 该技能已拥有的 tagId
+		 */
+		function ownedTagsFirst(tags, ownedIds) {
+			const owned = new Set(ownedIds ?? []);
+			const yes = [];
+			const no = [];
+			for (const t of tags ?? []) (owned.has(t?.tagId) ? yes : no).push(t);
+			return [...yes, ...no];
+		}
+
 		// ── 「打开技能所在文件夹」 ────────────────────────────────────
 		// 宿主没有给插件 reveal API，服务端由 src/reveal.js 拉起文件管理器。
 		// 这里只负责：名字 → 绝对路径（扫描结果自带 path）、发请求、缺路径时
@@ -515,6 +529,9 @@ window.__ModuleLoader__.load({
 			const ownedIds = selName ? (links.find((l) => l.skillId === selName)?.tagIds ?? []) : [];
 			const owned = new Set(ownedIds);
 			const tagMatch = (t) => !state.skillTagSearch.trim() || t.name.toLowerCase().includes(state.skillTagSearch.trim().toLowerCase());
+			// 已拥有的标签排在前面（不再是纯粹按服务端返回的字母序），便于确认"这个技能到底挂了什么"
+			const tagsOrdered = ownedTagsFirst(tags, ownedIds);
+			const tagsShown = tagsOrdered.filter(tagMatch);
 
 			/** 整组覆盖该技能的标签集合（勾选/取消都走这里，立即落库） */
 			const saveTags = async (nextIds) => {
@@ -576,9 +593,9 @@ window.__ModuleLoader__.load({
 							}),
 							react.createElement("div", { key: "rows", style: { maxHeight: "360px", overflowY: "auto" } }, [
 								tags.length === 0 && react.createElement("div", { key: "e", style: { color: "#888", fontSize: "12px", padding: "6px 2px" } }, "还没有任何标签，请先在主页面新建"),
-								tags.filter(tagMatch).length === 0 && tags.length > 0 &&
+								tagsShown.length === 0 && tags.length > 0 &&
 									react.createElement("div", { key: "e2", style: { color: "#888", fontSize: "12px", padding: "6px 2px" } }, "无匹配标签"),
-								...tags.filter(tagMatch).map((t) => react.createElement("label", { key: t.tagId, style: { ...rowStyle, cursor: "pointer" } }, [
+								...tagsShown.map((t) => react.createElement("label", { key: t.tagId, style: { ...rowStyle, cursor: "pointer" } }, [
 									react.createElement("input", {
 										key: "c", type: "checkbox", checked: owned.has(t.tagId), disabled: busy,
 										onChange: () => toggleTag(t.tagId),
@@ -1126,6 +1143,8 @@ window.__ModuleLoader__.load({
 			const [pickOpen, setPickOpen] = react.useState(false);
 			const [disabledOpen, setDisabledOpen] = react.useState(false);
 			const [tagOfSkill, setTagOfSkill] = react.useState(null);
+			/** 右列技能搜索词（对话版 / 工作区版共用；只影响列表显示，不影响下发结果） */
+			const [skillSearch, setSkillSearch] = react.useState("");
 
 			const load = react.useCallback(async () => {
 				try {
@@ -1190,6 +1209,10 @@ window.__ModuleLoader__.load({
 			/** 可添加的技能：排除已有（可见）与被禁用的 */
 			const addable = (ws?.universe ?? []).filter((n) => !visibleSet.has(n) && !disabledSet.has(n));
 
+			// 右列搜索：仅过滤显示，不影响「技能 (总)」这个语义数字
+			const skillQuery = skillSearch.trim().toLowerCase();
+			const shownSkills = (view.visible ?? []).filter((n) => !skillQuery || String(n).toLowerCase().includes(skillQuery));
+
 			// 技能名 → SKILL.md 路径（服务端把扫描结果的 path 一并带回来了）
 			const catalog = ws?.catalog ?? [];
 			const pathOf = (name) => catalog.find((s) => s.name === name)?.path ?? null;
@@ -1236,11 +1259,27 @@ window.__ModuleLoader__.load({
 
 					// ── 右列：技能 ──
 					react.createElement("div", { key: "right", style: colStyle }, [
-						react.createElement("div", { key: "h", style: { fontWeight: 600, marginBottom: "8px" } }, `技能 (${(view.visible ?? []).length})`),
+						react.createElement("div", { key: "h", style: { fontWeight: 600, marginBottom: "8px" } },
+							skillQuery
+								? `技能 (${shownSkills.length}/${(view.visible ?? []).length})`
+								: `技能 (${(view.visible ?? []).length})`),
+						// 搜索框：放列表顶部，只过滤显示（不改变实际下发的技能集合）
+						react.createElement("div", { key: "search", style: { display: "flex", gap: "6px", alignItems: "center", marginBottom: "6px" } }, [
+							react.createElement("input", {
+								key: "i", style: { ...input, flex: 1, minWidth: 0 }, type: "search", placeholder: "搜索技能…",
+								value: skillSearch, onChange: (e) => setSkillSearch(e.target.value),
+							}),
+							skillSearch.trim() !== "" && react.createElement("button", {
+								key: "c", style: { ...btn, padding: "1px 7px", fontSize: "11px" },
+								onClick: () => setSkillSearch(""),
+							}, "清除"),
+						]),
 						react.createElement("div", { key: "list", style: { flex: 1, overflowY: "auto", minHeight: "80px" } }, [
 							(view.visible ?? []).length === 0 && react.createElement("div", { key: "e", style: { color: "#888", fontSize: "12px" } },
 								"没有技能。可在下方「+ 精细化添加技能」手动加入，或清空标签条件回到全部技能。"),
-							...(view.visible ?? []).map((name) => react.createElement("div", { key: name, style: { ...listRow, flexWrap: "wrap" } }, [
+							(view.visible ?? []).length > 0 && shownSkills.length === 0 && react.createElement("div", { key: "nf", style: { color: "#888", fontSize: "12px" } },
+								`没有匹配「${skillSearch.trim()}」的技能。`),
+							...shownSkills.map((name) => react.createElement("div", { key: name, style: { ...listRow, flexWrap: "wrap" } }, [
 								react.createElement("span", { key: "n", style: { flex: 1 } }, name),
 								manualSet.has(name) && react.createElement("span", { key: "m", style: { color: "#3498db", fontSize: "11px" } }, "手动"),
 								react.createElement(RevealButton, { key: "rev", skillName: name, path: pathOf(name), label: "📁" }),
@@ -1266,7 +1305,8 @@ window.__ModuleLoader__.load({
 			};
 
 			return react.createElement("div", { style: overlayStyle, onMouseDown: (e) => { if (e.target === e.currentTarget) onClose(); } },
-				react.createElement("div", { style: modalBox, onClick: (e) => e.stopPropagation() }, [
+				// 固定高度：搜索让右列行数变少时窗口不能跟着缩（否则每敲一个字母窗口就抖一下）
+				react.createElement("div", { style: { ...modalBox, height: "82vh" }, onClick: (e) => e.stopPropagation() }, [
 					react.createElement("div", { key: "head", style: { display: "flex", alignItems: "center", gap: "10px", marginBottom: "10px" } }, [
 						react.createElement("strong", { key: "t", style: { fontSize: "14px" } }, scope.title),
 						react.createElement("span", { key: "w", style: { color: "#888", fontSize: "11px", ...monoStyle } }, scope.id),
@@ -1640,7 +1680,7 @@ window.__ModuleLoader__.load({
 			skillsLinkedTo, skillsNotLinkedTo, ZONE_LABEL,
 			workspaceIdOfRowKey, isWorkspaceMenu, manage, manageSet,
 			manageScope, registerComposerEntry,
-			skillPathOf, RevealButton,
+			skillPathOf, RevealButton, ownedTagsFirst,
 		};
 		return module.exports;
 	},

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