@max-null/dsh-skill-mcp-center 0.5.2 → 0.5.3

package/dist/service.d.ts

@@ -74,6 +74,15 @@ export interface McpConfig {
     url?: string;
     headers?: Record<string, string>;
 }
+/**
+ * Scan one root for SKILL.md entries and parse their frontmatter.
+ * @param root - directory to scan; a missing root yields no skills.
+ * @param source - discovery source label recorded on every skill found.
+ * @param writable - whether the surface may rewrite these SKILL.md files.
+ * @param provider - provider label recorded on every skill found.
+ * @returns the skills this root contributes, in directory order.
+ */
+export declare function scanSkillRoot(root: string, source: string, writable?: boolean, provider?: string): Promise<SkillView[]>;
 export declare class SkillMcpService extends Service {
     static inject: string[];
     private readonly officialSkillDirs;

package/dist/service.js

@@ -8,7 +8,7 @@
  */
 import { Service } from '@deepseek-ai/cordis';
 import { existsSync } from 'node:fs';
-import { readdir, readFile, writeFile } from 'node:fs/promises';
+import { readdir, readFile, stat, writeFile } from 'node:fs/promises';
 import { homedir } from 'node:os';
 import { join, sep } from 'node:path';
 import { fileURLToPath } from 'node:url';
@@ -65,16 +65,45 @@ function ownNodeModules() {
     const at = here.lastIndexOf(marker);
     return at < 0 ? null : here.slice(0, at + marker.length - 1);
 }
-/** Absolute SKILL.md path for one directory/file entry, or null. */
-function skillPathFor(root, name, isDirectory) {
+/**
+ * Absolute SKILL.md path for one root entry, or null when the entry is neither
+ * a directory nor a Markdown file.
+ *
+ * `readdir`'s Dirent reports lstat semantics: a Windows directory junction —
+ * and equally a POSIX symlink to a directory — is `isDirectory() === false`
+ * with `isSymbolicLink() === true`, so the Dirent alone cannot tell a linked
+ * directory from a plain file. Only entries the Dirent leaves open reach the
+ * `stat` probe, which follows the link to its target type and so keeps the
+ * common true-directory case at zero extra syscalls. A broken link fails that
+ * probe (ENOENT) and yields null rather than a path nothing can read.
+ * @param root - directory being scanned.
+ * @param entry - one `readdir(root, { withFileTypes: true })` entry.
+ * @returns the SKILL.md path this entry names, or null when it names no skill.
+ */
+async function skillPathFor(root, entry) {
+    const full = join(root, entry.name);
+    let isDirectory = entry.isDirectory();
+    if (!isDirectory && !entry.isFile()) {
+        try {
+            isDirectory = (await stat(full)).isDirectory();
+        }
+        catch {
+            return null; // broken link: the target is gone
+        }
+    }
     if (isDirectory)
-        return join(root, name, 'SKILL.md');
-    if (name.endsWith('.md'))
-        return join(root, name);
-    return null;
+        return join(full, 'SKILL.md');
+    return entry.name.endsWith('.md') ? full : null;
 }
-/** Scan one root for SKILL.md entries and parse their frontmatter. */
-async function scanSkillRoot(root, source, writable = true, provider = 'filesystem') {
+/**
+ * Scan one root for SKILL.md entries and parse their frontmatter.
+ * @param root - directory to scan; a missing root yields no skills.
+ * @param source - discovery source label recorded on every skill found.
+ * @param writable - whether the surface may rewrite these SKILL.md files.
+ * @param provider - provider label recorded on every skill found.
+ * @returns the skills this root contributes, in directory order.
+ */
+export async function scanSkillRoot(root, source, writable = true, provider = 'filesystem') {
     let entries;
     try {
         entries = await readdir(root, { withFileTypes: true });
@@ -84,7 +113,7 @@ async function scanSkillRoot(root, source, writable = true, provider = 'filesyst
     }
     const skills = [];
     for (const entry of entries) {
-        const skillPath = skillPathFor(root, entry.name, entry.isDirectory());
+        const skillPath = await skillPathFor(root, entry);
         if (skillPath === null)
             continue;
         let text;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@max-null/dsh-skill-mcp-center",
-  "version": "0.5.2",
+  "version": "0.5.3",
   "description": "Skill & MCP management center for DeepSeek Harness — manage skills and MCP servers in Settings, live MCP status in the sidebar",
   "license": "MIT",
   "publishConfig": {

package/release-notes.md

@@ -1,5 +1,25 @@
 # Release Notes — @max-null/dsh-skill-mcp-center
 
+## 0.5.3 (2026-10-02)
+
+### 修复
+
+- **Windows 下技能列表整体为空**。`~/.dsh/skills` 下的条目是目录 junction 或符号
+  链接时，`scanSkillRoot()` 拿 `Dirent.isDirectory()` 当形态判据——而 Dirent 是
+  lstat 语义，junction 报 `isDirectory() === false` / `isSymbolicLink() === true`，
+  于是 `skillPathFor()` 返回 null，每个链接型技能在 `continue` 处被丢弃。技能本身
+  完好可读，只是扫不到，面板上表现为列表为空。
+
+  判据改为**分层**：Dirent 能明确作答时直接采信（真目录走 `isDirectory()`、真文件
+  走 `isFile()`），只有链接与文件系统报 `DT_UNKNOWN` 的形态才跟随一次 `stat`。真
+  目录因此仍是零额外系统调用。断链在 `stat` 处失败即跳过，同根其它技能不受影响。
+  这不是 Windows 专用分支——junction 与 POSIX 符号链接走同一条路。
+
+### 兼容性
+
+- 无接口变更：`listSkills()` 的出入参、端点集合与请求信封均未改，只是扫描判据放宽。
+  新增 5 条单测覆盖真目录 / `.md` 文件 / 链接指向目录 / 断链 / 非技能条目。
+
 ## 0.5.2 (2026-09-29)
 
 ### 变更