dsh-skill-hub 0.3.19 → 0.3.21

package/CONTRIBUTING.md

@@ -39,7 +39,7 @@ src/client/locales/     dictionaries by view (common/skills/market/sources/detai
 ```bash
 npm install
 npm run typecheck   # tsc --noEmit
-npm test            # vitest (258 tests across 14 suites)
+npm test            # vitest (285 tests across 14 suites)
 npm run build       # tsc declarations + tsdown bundles (lib/index.js + lib/client.js)
 ```
 

package/README.md

@@ -108,7 +108,7 @@ Host uses only `ctx.skills.snapshot/get`, `ctx.webServer.register`, `ctx.systemP
 
 ```bash
 npm run typecheck  # tsc --noEmit
-npm test           # 258 tests, 14 suites
+npm test           # 285 tests, 14 suites
 npm run build      # tsc + tsdown → lib/index.js + lib/client.js
 ```
 
@@ -117,8 +117,9 @@ npm run build      # tsc + tsdown → lib/index.js + lib/client.js
 ## Troubleshooting
 
 - `duplicate loader entry id: skill-hub` — remove the duplicate install (keep one `dsh plugin add` method).
-- Skill missing — check the diagnostics section (frontmatter / name mismatch / short description).
-- Empty source group — its skills were deleted, or their disabled records were lost (sidecar restored/hand-edited). Startup reconciles `.disabled` files on disk; source groups with no visible member are no longer rendered.
+- Skill missing — check the diagnostics section (frontmatter / name mismatch / short description / two discovery forms claiming the same name: a `<name>/SKILL.md` bundle and a `<name>.md` file can coexist on disk, and the registry keeps only one of them).
+- Empty source group — its skills were deleted, or their disabled records were lost (sidecar restored/hand-edited). Startup reconciles `.disabled` files on disk **and prunes records whose file is gone** (a record left behind by a hand-deleted directory used to be un-actionable: enable, create and delete all failed). Source groups with no visible member are no longer rendered.
+- A skill cannot be created / stays "disabled" — the disabled record pointed at a file that no longer exists. At startup (and on the next enable attempt) the record is repointed at the surviving `.disabled` file, or dropped so the name is free again.
 - Dots missing in `/` menu — dsh internals changed; catalog still works.
 - Settings card missing — open the plugin in the Plugins manager (sidebar → 插件 → dsh-skill-hub) and expand the 「技能中枢」 card at the top of the page (collapsed by default). On dsh older than `0.1.7-alpha.1` this plugin does not load at all (the settings API it targets does not exist there).
 

package/README.zh.md

@@ -108,7 +108,7 @@ GitHub 仓库 ──扫描/导入──▶ ~/.dsh/skills
 
 ```bash
 npm run typecheck  # tsc --noEmit
-npm test           # 258 tests, 14 suites
+npm test           # 285 tests, 14 suites
 npm run build      # tsc + tsdown → lib/index.js + lib/client.js
 ```
 
@@ -117,8 +117,9 @@ npm run build      # tsc + tsdown → lib/index.js + lib/client.js
 ## 故障排查
 
 - `duplicate loader entry id: skill-hub` — 删掉重复安装（只留一种 `dsh plugin add`）。
-- 技能不出现 — 看诊断区（缺 frontmatter / 名称不一致 / 描述过短）。
-- 来源组空壳 — 组内技能已被删除、或禁用记录丢失（状态文件被恢复/手改）。启动时自动对账磁盘上的 `.disabled` 文件补记录；无可见成员的来源组不再渲染。
+- 技能不出现 — 看诊断区（缺 frontmatter / 名称不一致 / 描述过短 / 两种发现形态抢同一个名字：`<name>/SKILL.md` 目录包与 `<name>.md` 扁平文件可以并存，注册表只会留一个）。
+- 来源组空壳 — 组内技能已被删除、或禁用记录丢失（状态文件被恢复/手改）。启动时自动对账磁盘上的 `.disabled` 文件补记录，**并清掉文件已不存在的死记录**（手工删掉目录后留下的记录以前谁也动不了：启用、新建、删除三条路全失败）；无可见成员的来源组不再渲染。
+- 技能建不出来 / 一直显示「已禁用」— 禁用记录指向的文件已经不在了。启动时（以及下一次点「启用」时）会把记录改指到现存的 `.disabled` 文件，找不到就直接删掉记录、把名字释放出来。
 - `/` 菜单圆点消失 — dsh 内部触发源变更，目录功能不受影响。
 - 找不到配置卡片 — 在插件管理页打开本插件（侧边栏「插件」→ dsh-skill-hub），展开页面顶部的「技能中枢」卡片（默认折叠）。dsh 低于 `0.1.7-alpha.1` 时本插件无法加载（它所用的 settings API 在旧版不存在）。
 

package/lib/client.js

@@ -61,6 +61,10 @@ window.__ModuleLoader__.load({
 		* `ctx.settings.update(HUB_ENTRY_ID, …)`.
 		*/
 		const HUB_ENTRY_ID = "skill-hub";
+		/** HEX color validation shared by host routes and the settings card. */
+		const HEX_COLOR_RE = /^#[0-9a-f]{6}$/i;
+		/** GitHub token validation shared by host routes and the settings card. */
+		const GITHUB_TOKEN_RE = /^[A-Za-z0-9_\-]{8,255}$/;
 		//#endregion
 		//#region src/protocol/sources.ts
 		/** 项目级技能来源（它们有 workspace 归属，不属于「个人」组）。 */
@@ -161,7 +165,7 @@ window.__ModuleLoader__.load({
 			market() {
 				return this.get(SKILL_HUB_API.market);
 			}
-			/** Add a market source repo; resolves with the fresh list. */
+			/** Add a market source repo; resolves with the fresh list plus the resolved slug. */
 			addMarketSource(repo) {
 				return this.post(SKILL_HUB_API.marketSource, { repo });
 			}
@@ -1542,7 +1546,7 @@ window.__ModuleLoader__.load({
 			});
 		}
 		function ColorField(props) {
-			const value = /^#[0-9a-f]{6}$/i.test(props.text) ? props.text : props.defaultColor;
+			const value = HEX_COLOR_RE.test(props.text) ? props.text : props.defaultColor;
 			return /* @__PURE__ */ (0, react_jsx_runtime.jsxs)("div", {
 				className: settings_card_module_css_default.field,
 				children: [
@@ -2360,9 +2364,9 @@ window.__ModuleLoader__.load({
 		function colorField(field) {
 			return {
 				field,
-				format: (value) => typeof value === "string" && /^#[0-9a-f]{6}$/i.test(value) ? value : "",
+				format: (value) => typeof value === "string" && HEX_COLOR_RE.test(value) ? value : "",
 				parse: (text) => {
-					if (/^#[0-9a-f]{6}$/i.test(text)) return {
+					if (HEX_COLOR_RE.test(text)) return {
 						kind: "set",
 						value: text
 					};
@@ -2381,7 +2385,7 @@ window.__ModuleLoader__.load({
 				parse: (text) => {
 					const trimmed = text.trim();
 					if (trimmed === "") return void 0;
-					if (!/^[A-Za-z0-9_-]{8,255}$/.test(trimmed)) return void 0;
+					if (!GITHUB_TOKEN_RE.test(trimmed)) return void 0;
 					return {
 						kind: "set",
 						value: trimmed
@@ -6424,7 +6428,7 @@ window.__ModuleLoader__.load({
 					setNewSourceName("");
 					setRepoResult(null);
 					setRepoSelected(/* @__PURE__ */ new Set());
-					const slug = result.repos[result.repos.length - 1].repo;
+					const slug = result.repo;
 					setScanningRepo(slug);
 					setRepoDiscoverState({ status: "scanning" });
 					try {