mdk-skills 2.4.19 → 2.4.20

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mdk-skills",
-  "version": "2.4.19",
+  "version": "2.4.20",
   "description": "mdk-engineer - 沉稳靠谱的前端开发助手 Claude Skills 配置包，一键注入 .claude/ 技能目录和 CLAUDE.md 人设配置",
   "author": "XiaoMa",
   "license": "MIT",

package/scripts/cli.js

@@ -2,7 +2,7 @@
 
 const fs = require("fs");
 const path = require("path");
-const { getPackageSkills, getUserSkills, getSkillsSource, backupDir, listSkillDirs } = require("./core");
+const { getPackageSkills, getUserSkills, getSkillsSource, backupDir, listSkillDirs, copyDirSync } = require("./core");
 
 // npx 运行时：process.cwd() 是用户项目目录
 // __dirname 是包内 scripts/ 目录
@@ -47,20 +47,6 @@ function writeSettings(settings) {
 
 // ---------- 安装勾选的技能 ----------
 
-// 手动递归拷贝目录，绕开 Windows 中文路径下 fs.cpSync({ recursive: true }) 死锁的 bug
-function copyDirSync(src, dest) {
-  fs.mkdirSync(dest, { recursive: true });
-  for (const item of fs.readdirSync(src)) {
-    const srcPath = path.join(src, item);
-    const destPath = path.join(dest, item);
-    if (fs.statSync(srcPath).isDirectory()) {
-      copyDirSync(srcPath, destPath);
-    } else {
-      fs.copyFileSync(srcPath, destPath);
-    }
-  }
-}
-
 function installSelectedSkills(selectedNames) {
   // 创建 .claude/ 和 skills/ 目录
   if (!fs.existsSync(skillsDest)) {

package/scripts/web-ui/server/context.js

@@ -41,23 +41,23 @@ function refreshSource() {
   pkgSkillsSource = skillsSource ? path.join(skillsSource, ".claude", "skills") : null;
 }
 
-// ---------- Settings ----------
+// ---------- Settings（异步）----------
 
-function readSettings() {
-  if (!fs.existsSync(settingsPath))
+async function readSettings() {
+  if (!(await pathExists(settingsPath)))
     return { skills: {}, always_apply_skills: [] };
   try {
-    return JSON.parse(fs.readFileSync(settingsPath, "utf-8"));
+    return JSON.parse(await fs.promises.readFile(settingsPath, "utf-8"));
   } catch {
     return { skills: {}, always_apply_skills: [] };
   }
 }
 
-function writeSettings(settings) {
-  if (!fs.existsSync(claudeDest)) {
-    fs.mkdirSync(claudeDest, { recursive: true });
+async function writeSettings(settings) {
+  if (!(await pathExists(claudeDest))) {
+    await fs.promises.mkdir(claudeDest, { recursive: true });
   }
-  fs.writeFileSync(
+  await fs.promises.writeFile(
     settingsPath,
     JSON.stringify(settings, null, 2) + "\n",
     "utf-8",
@@ -70,25 +70,25 @@ function getPullSourcePath() {
   return path.join(pkgSkillsSource, ".pull-source.json");
 }
 
-function readPullSource() {
+async function readPullSource() {
   if (!pkgSkillsSource) return {};
   const filePath = getPullSourcePath();
-  if (!fs.existsSync(filePath)) return {};
+  if (!(await pathExists(filePath))) return {};
   try {
-    return JSON.parse(fs.readFileSync(filePath, "utf-8"));
+    return JSON.parse(await fs.promises.readFile(filePath, "utf-8"));
   } catch {
     return {};
   }
 }
 
-function writePullSource(data) {
-  fs.writeFileSync(getPullSourcePath(), JSON.stringify(data, null, 2) + "\n", "utf-8");
+async function writePullSource(data) {
+  await fs.promises.writeFile(getPullSourcePath(), JSON.stringify(data, null, 2) + "\n", "utf-8");
 }
 
-function addPullSource(skillName, url) {
-  const data = readPullSource();
+async function addPullSource(skillName, url) {
+  const data = await readPullSource();
   data[skillName] = { url, pulledAt: new Date().toISOString() };
-  writePullSource(data);
+  await writePullSource(data);
 }
 
 // 检查技能源是否已设置
@@ -102,11 +102,11 @@ function requireSource(res, utils) {
 
 // ---------- profiles ----------
 
-function loadProfiles() {
+async function loadProfiles() {
   const profilesPath = path.join(skillsSource, ".claude", "profiles.json");
-  if (!fs.existsSync(profilesPath)) return null;
+  if (!(await pathExists(profilesPath))) return null;
   try {
-    return JSON.parse(fs.readFileSync(profilesPath, "utf-8")).profiles;
+    return JSON.parse(await fs.promises.readFile(profilesPath, "utf-8")).profiles;
   } catch {
     return null;
   }
@@ -120,9 +120,15 @@ const runningTasks = new Map();
 
 const pullCache = new Map();
 
-// ---------- 应用场景 / 安装 ----------
+// ---------- 路径检查工具 ----------
 
-function applyProfile(profile, { copyDirSync, safeRmSync, readSettings, writeSettings }) {
+async function pathExists(p) {
+  try { await fs.promises.access(p); return true; } catch { return false; }
+}
+
+// ---------- 应用场景 / 安装（异步）----------
+
+async function applyProfile(profile, { copyDir, safeRm }) {
   const pkgSkills = core.listSkillDirs(pkgSkillsSource);
   const failedDelete = [];
   const failedInstall = [];
@@ -136,22 +142,22 @@ function applyProfile(profile, { copyDirSync, safeRmSync, readSettings, writeSet
     selected = profile.skills.filter((s) => pkgSkills.includes(s));
   }
 
-  if (!fs.existsSync(skillsDest)) {
-    fs.mkdirSync(skillsDest, { recursive: true });
+  if (!(await pathExists(skillsDest))) {
+    await fs.promises.mkdir(skillsDest, { recursive: true });
   }
   for (const name of pkgSkills) {
     const src = path.join(pkgSkillsSource, name);
     const dest = path.join(skillsDest, name);
     if (selected.includes(name)) {
-      if (!fs.existsSync(dest)) {
-        try { copyDirSync(src, dest); } catch (err) {
+      if (!(await pathExists(dest))) {
+        try { await copyDir(src, dest); } catch (err) {
           console.error(`安装技能 "${name}" 失败:`, err.message);
           failedInstall.push(name);
         }
       }
     } else {
-      if (fs.existsSync(dest)) {
-        const r = safeRmSync(dest, name);
+      if (await pathExists(dest)) {
+        const r = await safeRm(dest, name);
         if (!r.removed) failedDelete.push(name);
       }
     }
@@ -159,7 +165,7 @@ function applyProfile(profile, { copyDirSync, safeRmSync, readSettings, writeSet
 
   const hasFailed = failedDelete.length > 0 || failedInstall.length > 0;
   if (!hasFailed) {
-    const settings = readSettings();
+    const settings = await readSettings();
     if (!settings.skills) settings.skills = {};
     for (const name of selected) {
       if (!settings.skills[name]) settings.skills[name] = {};
@@ -174,18 +180,18 @@ function applyProfile(profile, { copyDirSync, safeRmSync, readSettings, writeSet
       settings.always_apply_skills = profile.always_apply;
     }
     settings._active_profile = profile.id;
-    writeSettings(settings);
+    await writeSettings(settings);
 
     const profilesSource = path.join(skillsSource, ".claude", "profiles.json");
     const profilesDest = path.join(claudeDest, "profiles.json");
-    if (fs.existsSync(profilesSource)) {
-      fs.copyFileSync(profilesSource, profilesDest);
+    if (await pathExists(profilesSource)) {
+      await fs.promises.copyFile(profilesSource, profilesDest);
     }
 
     const mdSource = path.join(skillsSource, "CLAUDE.md");
     const mdDest = path.join(projectRoot, "CLAUDE.md");
-    if (fs.existsSync(mdSource) && !fs.existsSync(mdDest)) {
-      fs.copyFileSync(mdSource, mdDest);
+    if (await pathExists(mdSource) && !(await pathExists(mdDest))) {
+      await fs.promises.copyFile(mdSource, mdDest);
     }
   }
 
@@ -198,10 +204,10 @@ function applyProfile(profile, { copyDirSync, safeRmSync, readSettings, writeSet
   };
 }
 
-function installSelectedSkills(selectedNames, { copyDirSync, safeRmSync, readSettings, writeSettings }) {
-  if (!fs.existsSync(pkgSkillsSource)) return { installed: 0, locked: [] };
-  if (!fs.existsSync(skillsDest)) {
-    fs.mkdirSync(skillsDest, { recursive: true });
+async function installSelectedSkills(selectedNames, { copyDir, safeRm }) {
+  if (!(await pathExists(pkgSkillsSource))) return { installed: 0, locked: [] };
+  if (!(await pathExists(skillsDest))) {
+    await fs.promises.mkdir(skillsDest, { recursive: true });
   }
   const allPkgSkills = core.listSkillDirs(pkgSkillsSource);
   let installedCount = 0;
@@ -211,41 +217,39 @@ function installSelectedSkills(selectedNames, { copyDirSync, safeRmSync, readSet
     const src = path.join(pkgSkillsSource, name);
     const dest = path.join(skillsDest, name);
     if (selectedNames.includes(name)) {
-      if (!fs.existsSync(dest)) {
-        try { copyDirSync(src, dest); installedCount++; } catch (err) {
+      if (!(await pathExists(dest))) {
+        try { await copyDir(src, dest); installedCount++; } catch (err) {
           console.error(`安装技能 "${name}" 失败:`, err.message);
         }
       }
     } else {
-      if (fs.existsSync(dest)) {
-        const r = safeRmSync(dest, name);
+      if (await pathExists(dest)) {
+        const r = await safeRm(dest, name);
         if (!r.removed && r.reason === 'locked') locked.push(name);
       }
     }
   }
 
-  const settings = readSettings();
+  const settings = await readSettings();
   if (!settings.skills) settings.skills = {};
   for (const name of allPkgSkills) {
     const enabled = selectedNames.includes(name);
     if (!settings.skills[name]) settings.skills[name] = {};
     settings.skills[name].enabled = enabled;
   }
-  writeSettings(settings);
+  await writeSettings(settings);
 
   const mdSource = path.join(skillsSource, "CLAUDE.md");
   const mdDest = path.join(projectRoot, "CLAUDE.md");
-  if (fs.existsSync(mdSource) && !fs.existsSync(mdDest)) {
-    fs.copyFileSync(mdSource, mdDest);
+  if (await pathExists(mdSource) && !(await pathExists(mdDest))) {
+    await fs.promises.copyFile(mdSource, mdDest);
   }
 
   return { installed: installedCount, locked };
 }
 
-// ---------- 回调函数注册（用于动态依赖注入） ----------
+// ---------- 回调函数注册 ----------
 
-// 由于 applyProfile / installSelectedSkills 需要在编译时传入依赖
-// 而在拆分前它们直接引用模块级变量，因此我们在 context 模块级保留接口
 function buildContext(utils) {
   return {
     projectRoot,
@@ -263,8 +267,8 @@ function buildContext(utils) {
     addPullSource,
     loadProfiles,
     requireSource: (res) => requireSource(res, utils),
-    applyProfile: (profile) => applyProfile(profile, utils),
-    installSelectedSkills: (names) => installSelectedSkills(names, utils),
+    applyProfile: (profile) => applyProfile(profile, { copyDir: utils.copyDir, safeRm: utils.safeRm }),
+    installSelectedSkills: (names) => installSelectedSkills(names, { copyDir: utils.copyDir, safeRm: utils.safeRm }),
     runningTasks,
     pullCache,
   };

package/scripts/web-ui/server/routes/others.js

@@ -6,6 +6,10 @@ const { buildContext } = require("../context");
 
 const ctx = buildContext(utils);
 
+async function pathExists(p) {
+  try { await fs.promises.access(p); return true; } catch { return false; }
+}
+
 function register(router) {
 
   // ----------------------------------------------------------------
@@ -14,7 +18,10 @@ function register(router) {
   router.get("/api/readme", async (req, res) => {
     if (!ctx.skillsSource) return utils.sendJSON(res, { content: null, found: false });
     const p = path.join(ctx.skillsSource, "README.md");
-    const content = fs.existsSync(p) ? fs.readFileSync(p, "utf-8") : null;
+    let content = null;
+    if (await pathExists(p)) {
+      content = await fs.promises.readFile(p, "utf-8");
+    }
     return utils.sendJSON(res, { content, found: !!content });
   });
 
@@ -24,7 +31,10 @@ function register(router) {
   router.get("/api/skills-readme", async (req, res) => {
     if (!ctx.skillsSource) return utils.sendJSON(res, { content: null, found: false });
     const p = path.join(ctx.skillsSource, ".claude", "skills", "README.md");
-    const content = fs.existsSync(p) ? fs.readFileSync(p, "utf-8") : null;
+    let content = null;
+    if (await pathExists(p)) {
+      content = await fs.promises.readFile(p, "utf-8");
+    }
     return utils.sendJSON(res, { content, found: !!content });
   });
 
@@ -37,8 +47,8 @@ function register(router) {
       : [path.join(ctx.projectRoot, "CLAUDE.md")];
     let content = null;
     for (const p of paths) {
-      if (fs.existsSync(p)) {
-        content = fs.readFileSync(p, "utf-8");
+      if (await pathExists(p)) {
+        content = await fs.promises.readFile(p, "utf-8");
         break;
       }
     }
@@ -50,7 +60,7 @@ function register(router) {
   // ----------------------------------------------------------------
   router.post("/api/readme/create", async (req, res) => {
     const body = await utils.parseBody(req);
-    const settings = ctx.readSettings();
+    const settings = await ctx.readSettings();
     const sourcePath = settings._skill_source;
     if (!sourcePath) return utils.sendJSON(res, { error: "未绑定本地源" }, 400);
 
@@ -61,7 +71,7 @@ function register(router) {
     const skipped = [];
 
     const skillsDir = path.join(sourcePath, ".claude", "skills");
-    const skillNames = fs.existsSync(skillsDir)
+    const skillNames = await pathExists(skillsDir)
       ? core.listSkillDirs(skillsDir)
       : [];
     const skillsList = skillNames.length
@@ -71,7 +81,7 @@ function register(router) {
     // 根目录 README.md
     if (root) {
       const rootPath = path.join(sourcePath, "README.md");
-      if (!fs.existsSync(rootPath)) {
+      if (!(await pathExists(rootPath))) {
         const tmpl = `# ${name}
 
 > 维护团队：${team}
@@ -97,7 +107,7 @@ ${skillsList}
 
 <!-- 许可证信息 -->
 `;
-        fs.writeFileSync(rootPath, tmpl, "utf-8");
+        await fs.promises.writeFile(rootPath, tmpl, "utf-8");
         created.push("README.md");
       } else {
         skipped.push("README.md");
@@ -107,7 +117,7 @@ ${skillsList}
     // skills/README.md
     if (skills) {
       const skillsReadmePath = path.join(skillsDir, "README.md");
-      if (!fs.existsSync(skillsReadmePath)) {
+      if (!(await pathExists(skillsReadmePath))) {
         const tmpl = `# 技能说明
 
 ## 快速开始
@@ -132,7 +142,7 @@ ${skillsList}
 技能使用过程中的注意事项。
 -->
 `;
-        fs.writeFileSync(skillsReadmePath, tmpl, "utf-8");
+        await fs.promises.writeFile(skillsReadmePath, tmpl, "utf-8");
         created.push("skills/README.md");
       } else {
         skipped.push("skills/README.md");
@@ -154,10 +164,10 @@ ${skillsList}
     for (const skill of pkgSkills) {
       const skillDir = path.join(ctx.skillsDest, skill.name);
       const issues = [];
-      if (!fs.existsSync(skillDir)) {
+      if (!(await pathExists(skillDir))) {
         issues.push("未安装");
       } else {
-        if (!fs.existsSync(path.join(skillDir, "SKILL.md"))) {
+        if (!(await pathExists(path.join(skillDir, "SKILL.md")))) {
           issues.push("缺少 SKILL.md");
         } else if (!core.parseFrontmatter(skillDir)) {
           issues.push("SKILL.md frontmatter 格式异常");
@@ -194,13 +204,13 @@ ${skillsList}
 
     if (!dirPath || dirPath === "") {
       if (process.platform === "win32") {
-        entries = utils.getWindowsDrives();
+        entries = await utils.getWindowsDrives();
       } else {
         entries = [{ name: "/", isDir: true }];
       }
     } else {
       try {
-        const items = fs.readdirSync(dirPath, { withFileTypes: true });
+        const items = await fs.promises.readdir(dirPath, { withFileTypes: true });
         entries = items
           .filter((item) => item.isDirectory())
           .map((item) => ({ name: item.name, isDir: true }))
@@ -221,7 +231,7 @@ ${skillsList}
     if (!dirPath) return utils.sendJSON(res, { error: "缺少路径" }, 400);
 
     const skillsDir = path.join(dirPath, ".claude", "skills");
-    const valid = fs.existsSync(skillsDir);
+    const valid = await pathExists(skillsDir);
     let skillCount = 0;
     if (valid) {
       skillCount = core.listSkillDirs(skillsDir).length;

package/scripts/web-ui/server/routes/profiles.js

@@ -5,6 +5,10 @@ const { buildContext } = require("../context");
 
 const ctx = buildContext(utils);
 
+async function pathExists(p) {
+  try { await fs.promises.access(p); return true; } catch { return false; }
+}
+
 function register(router) {
 
   // ----------------------------------------------------------------
@@ -14,8 +18,8 @@ function register(router) {
     if (!ctx.skillsSource || !ctx.pkgSkillsSource) {
       return utils.sendJSON(res, { error: "未设置技能目录，请在设置中连接" }, 400);
     }
-    const profiles = ctx.loadProfiles();
-    const settings = ctx.readSettings();
+    const profiles = await ctx.loadProfiles();
+    const settings = await ctx.readSettings();
     return utils.sendJSON(res, {
       profiles: profiles || [],
       activeProfile: settings._active_profile || null,
@@ -33,11 +37,11 @@ function register(router) {
     const body = await utils.parseBody(req);
     const { profileId } = body;
     if (!profileId) return utils.sendJSON(res, { error: "缺少场景 ID" }, 400);
-    const profiles = ctx.loadProfiles();
+    const profiles = await ctx.loadProfiles();
     const profile = profiles?.find((p) => p.id === profileId);
     if (!profile)
       return utils.sendJSON(res, { error: "场景不存在: " + profileId }, 404);
-    const result = ctx.applyProfile(profile);
+    const result = await ctx.applyProfile(profile);
     return utils.sendJSON(res, { ok: true, ...result });
   });
 
@@ -52,9 +56,9 @@ function register(router) {
 
     const profilesPath = path.join(ctx.skillsSource, ".claude", "profiles.json");
     let data = { version: 1, profiles: [] };
-    if (fs.existsSync(profilesPath)) {
+    if (await pathExists(profilesPath)) {
       try {
-        data = JSON.parse(fs.readFileSync(profilesPath, "utf-8"));
+        data = JSON.parse(await fs.promises.readFile(profilesPath, "utf-8"));
       } catch {}
     }
 
@@ -82,11 +86,11 @@ function register(router) {
       });
     }
 
-    fs.writeFileSync(profilesPath, JSON.stringify(data, null, 2) + "\n", "utf-8");
+    await fs.promises.writeFile(profilesPath, JSON.stringify(data, null, 2) + "\n", "utf-8");
 
     // 同步到项目 .claude
     const projectProfilesPath = path.join(ctx.claudeDest, "profiles.json");
-    fs.writeFileSync(projectProfilesPath, JSON.stringify(data, null, 2) + "\n", "utf-8");
+    await fs.promises.writeFile(projectProfilesPath, JSON.stringify(data, null, 2) + "\n", "utf-8");
 
     return utils.sendJSON(res, { ok: true, profiles: data.profiles });
   });
@@ -106,19 +110,19 @@ function register(router) {
       return utils.sendJSON(res, { error: "不能删除内置场景" }, 403);
 
     const profilesPath = path.join(ctx.skillsSource, ".claude", "profiles.json");
-    if (!fs.existsSync(profilesPath))
+    if (!(await pathExists(profilesPath)))
       return utils.sendJSON(res, { error: "场景文件不存在" }, 404);
 
-    const data = JSON.parse(fs.readFileSync(profilesPath, "utf-8"));
+    const data = JSON.parse(await fs.promises.readFile(profilesPath, "utf-8"));
     const index = data.profiles.findIndex((p) => p.id === id);
     if (index === -1) return utils.sendJSON(res, { error: "场景不存在" }, 404);
 
     data.profiles.splice(index, 1);
-    fs.writeFileSync(profilesPath, JSON.stringify(data, null, 2) + "\n", "utf-8");
+    await fs.promises.writeFile(profilesPath, JSON.stringify(data, null, 2) + "\n", "utf-8");
 
     // 同步到项目 .claude
     const projectProfilesPath = path.join(ctx.claudeDest, "profiles.json");
-    fs.writeFileSync(projectProfilesPath, JSON.stringify(data, null, 2) + "\n", "utf-8");
+    await fs.promises.writeFile(projectProfilesPath, JSON.stringify(data, null, 2) + "\n", "utf-8");
 
     return utils.sendJSON(res, { ok: true });
   });