@michengai/dsh-skills-manager 1.2.4 → 1.2.6

package/lib/core.js

@@ -900,6 +900,49 @@ function parseBoolValue(raw) {
   if (v === "false" || v === "no" || v === "off" || v === "0") return false;
   return void 0;
 }
+function linkedWriteError(path) {
+  return {
+    ok: false,
+    error: `\u76EE\u5F55\u94FE\u63A5\u4E0D\u80FD\u5220\u9664\u6216\u8986\u76D6: ${path}`,
+    code: "error.skill.linked",
+    params: { path }
+  };
+}
+async function readlinkOrNull(path) {
+  try {
+    return await fs.readlink(path);
+  } catch {
+    return null;
+  }
+}
+async function isDirectoryLink(path) {
+  const st = await lstatOrNull(path);
+  if (!st) return false;
+  if (st.isSymbolicLink()) {
+    try {
+      return (await fs.stat(path)).isDirectory();
+    } catch {
+      return false;
+    }
+  }
+  if (process.platform !== "win32" || !st.isDirectory()) return false;
+  return await readlinkOrNull(path) !== null;
+}
+async function linkedDirectoryBundle(directory, name) {
+  const bundlePath = entryPath(directory, name);
+  if (bundlePath === null || !await isDirectoryLink(bundlePath)) return null;
+  const docPath = join(bundlePath, "SKILL.md");
+  const docStat = await lstatOrNull(docPath);
+  if (!docStat?.isFile() || docStat.isSymbolicLink()) return null;
+  return {
+    kind: "bundle",
+    docPath,
+    entryPath: bundlePath,
+    realDocPath: await fs.realpath(docPath),
+    realEntryPath: await fs.realpath(bundlePath),
+    linked: true
+  };
+}
 async function resolveEntry(root, name) {
   const definition = rootDefinition(root);
   if (definition && (!definition.mutable || definition.key === "agents")) {
@@ -917,6 +960,8 @@ async function resolveEntry(root, name) {
     if (!rootStat || !rootStat.isDirectory() || rootStat.isSymbolicLink())
       return null;
     const rootReal = await resolvedPath(rootPath);
+    const linked = await linkedDirectoryBundle(directory, name);
+    if (linked) return linked;
     const bundleStat = await lstatOrNull(bundlePath);
     if (bundleStat && bundleStat.isDirectory() && !bundleStat.isSymbolicLink()) {
       const bundleDoc = join(bundlePath, "SKILL.md");
@@ -1032,7 +1077,18 @@ async function scanEntries(root, options = {}) {
   const rootReal = await resolvedPath(directory);
   for (const it of items) {
     try {
-      if (it.isSymbolicLink()) continue;
+      if (it.isSymbolicLink() || process.platform === "win32" && it.isDirectory()) {
+        const linked = await linkedDirectoryBundle(directory, it.name);
+        if (linked) {
+          const doc = options.metadataOnly ? { map: {}, hasFrontmatter: false } : parseSkillDoc(await fs.readFile(linked.realDocPath, "utf8"));
+          byName.set(it.name, {
+            ...entryOf(it.name, "bundle", linked.docPath, doc),
+            ...linked
+          });
+          continue;
+        }
+        if (it.isSymbolicLink()) continue;
+      }
       if (it.isDirectory() && entryPath(directory, it.name) !== null) {
         const docPath = join(directory, it.name, "SKILL.md");
         const st = await lstatOrNull(docPath);
@@ -1769,6 +1825,9 @@ async function deleteSkill(root, name, log, options = {}) {
       if ((await lstatOrNull(target.path))?.isSymbolicLink())
         return { ok: false, error: `\u6280\u80FD\u76EE\u5F55\u4E0D\u5B89\u5168\uFF0C\u62D2\u7EDD\u5199\u5165: ${target.path}`, code: "error.root.unsafe", params: { path: target.path } };
     }
+  } else {
+    const bundle = entryPath(definition.path, name);
+    if (bundle && await isDirectoryLink(bundle)) return linkedWriteError(bundle);
   }
   const id = options.repositoryTrash?.id || options.catalogTrash?.id || `${Date.now()}-${randomUUID()}`;
   if (!/^\d+-[a-f0-9-]{36}$/.test(id)) throw codedError("\u56DE\u6536\u7AD9\u6761\u76EE\u7F16\u53F7\u975E\u6CD5", "error.trash.invalid", { id });
@@ -2136,6 +2195,10 @@ async function copyToTemporary(source, target, isDir) {
   }
 }
 async function replaceWithCopy(source, dest, isDir, existing = []) {
+  for (const path of [dest, ...existing]) {
+    if (await isDirectoryLink(path))
+      throw codedError(`\u76EE\u5F55\u94FE\u63A5\u4E0D\u80FD\u5220\u9664\u6216\u8986\u76D6: ${path}`, "error.skill.linked", { path });
+  }
   const stage = await copyToTemporary(source, dest, isDir);
   const backups = [];
   try {
@@ -2279,6 +2342,16 @@ async function importSkill(source, log, options = {}) {
       });
       continue;
     }
+    const linkPath = join(targetRoot, c.kebab);
+    if (await isDirectoryLink(linkPath)) {
+      failed.push({
+        source: c.source,
+        error: `\u76EE\u5F55\u94FE\u63A5\u4E0D\u80FD\u5220\u9664\u6216\u8986\u76D6: ${linkPath}`,
+        code: "error.skill.linked",
+        params: { path: linkPath }
+      });
+      continue;
+    }
     const dest = c.isDir ? join(targetRoot, c.kebab) : join(targetRoot, `${c.kebab}.md`);
     const paths = [
       join(targetRoot, c.kebab),
@@ -2646,7 +2719,7 @@ async function skillDetail(key, name, options = {}) {
     frontmatter: summary.hasFrontmatter ? Object.fromEntries(Object.entries(doc.map)) : null,
     diagnostics: summary.diagnostics,
     loadable: summary.loadable,
-    sourceReadOnly: !root.mutable
+    sourceReadOnly: !root.mutable || entry.linked === true
   };
 }
 async function listProviderCandidates(options = {}) {
@@ -2673,7 +2746,7 @@ async function listProviderCandidates(options = {}) {
     const project = root.scope === "project";
     const policyOnly = root.mutable && root.key !== "agents";
     const overlayRank = group.length > 1 ? Math.min(...group.map((item) => Math.min(item.root.rank, item.entry.providerRank ?? item.root.rank))) - 1 : void 0;
-    const needsOverlay = !policyOnly || group.length > 1 || policy.override !== void 0 || !entry.invocationPolicyValid || policyResult.writable === false;
+    const needsOverlay = !policyOnly || group.length > 1 || policy.override !== void 0 || !entry.invocationPolicyValid || policyResult.writable === false || !policy.sourceEnabled;
     if (!needsOverlay) continue;
     candidates.push({
       name: canonicalSkillName({ root, entry, policy }),
@@ -2700,8 +2773,8 @@ async function listProviderCandidates(options = {}) {
       metadata: {
         dshSkillsManager: {
           root: root.key,
-          readOnly: !root.mutable,
-          sourceReadOnly: !root.mutable,
+          readOnly: !root.mutable || entry.linked === true,
+          sourceReadOnly: !root.mutable || entry.linked === true,
           policyOnly
         }
       }
@@ -2904,7 +2977,8 @@ async function state(options = {}) {
         effectiveUserInvocable: policy.userInvocable,
         diagnostics: e.diagnostics,
         path: e.docPath,
-        categoryId: policyResult.state.categoryMembers[root.key]?.[e.name] || null
+        categoryId: policyResult.state.categoryMembers[root.key]?.[e.name] || null,
+        linked: e.linked === true
       });
       all.push({
         root,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@michengai/dsh-skills-manager",
-  "version": "1.2.4",
+  "version": "1.2.6",
   "description": "DSH Skills Manager — 在 DeepSeek Harness 中统一加载并安全管理本机 Agent Skills · Load and safely manage local Agent Skills in DSH",
   "license": "Apache-2.0",
   "repository": {