promptgraph-mcp 2.9.45 → 2.9.47

package/commands/bundle.js

@@ -107,14 +107,22 @@ export default async function handler(args, bin) {
       );
       process.exit(1);
     }
-    console.log(chalk.green(`found: ${detected.label}/`));
+    const { validMdCount, hasScripts } = detected;
+    console.log(chalk.green(`found: ${detected.label}/`) + chalk.gray(` (${validMdCount} valid skills${hasScripts ? ', has scripts 🔧' : ''})`));
+
+    if (validMdCount === 0) {
+      error(`Cannot publish: ${repo}/${detected.label} has no skills that pass validation.\n  All .md files were filtered (README/changelog/docs/etc).`);
+      process.exit(1);
+    }
+
     const name = repo.split('/')[1].replace(/-/g, ' ').replace(/\b\w/g, c => c.toUpperCase());
     const id = repo.replace('/', '-').toLowerCase();
     const bundle = {
       id, name, repo_url: repo, author: repo.split('/')[0],
       description: `Skills from ${repo}`,
       tags: ['community'],
-      stars: 0
+      stars: 0,
+      ...(hasScripts && { has_tools: true }),
     };
     const json = JSON.stringify(bundle, null, 2);
 

package/commands/update.js

@@ -35,17 +35,31 @@ export default async function handler(args, bin) {
 
   info(`Current: ${chalk.gray('v' + currentVersion)}  →  Latest: ${chalk.white.bold('v' + latest)}`);
 
-  // Kill other node processes that may lock native .node files (e.g. pg reindex still running)
-  // Exclude current PID to avoid killing ourselves
+  // Kill other promptgraph node processes that may lock native .node files
   if (process.platform === 'win32') {
-    spawnSync('wmic', ['process', 'where', `name='node.exe' and ProcessId!=${process.pid}`, 'delete'], { stdio: 'ignore', shell: true });
+    // taskkill is reliable on Windows 10/11 (wmic deprecated since Win11 22H2)
+    spawnSync('taskkill', ['/F', '/FI', `PID ne ${process.pid}`, '/FI', 'IMAGENAME eq node.exe'], { stdio: 'ignore', shell: true });
   } else {
     spawnSync('pkill', ['-f', 'promptgraph-mcp'], { stdio: 'ignore' });
   }
 
+  // Brief pause so OS releases file locks before npm touches them
+  await new Promise(r => setTimeout(r, 1500));
+
   const updateSpin = (await import('../cli.js')).spinner(`Installing promptgraph-mcp@latest (v${latest})...`);
   updateSpin.start();
-  const result = spawnSync('npm', ['install', '-g', 'promptgraph-mcp@latest'], { encoding: 'utf8', stdio: 'pipe', shell: true });
+
+  let result = spawnSync('npm', ['install', '-g', 'promptgraph-mcp@latest'], { encoding: 'utf8', stdio: 'pipe', shell: true });
+
+  // EBUSY: MCP server may have been re-spawned by the editor — retry once with --force
+  if (result.status !== 0 && (result.stderr || '').includes('EBUSY')) {
+    updateSpin.stop();
+    info('File busy (MCP server still running). Close Claude Code / your editor and press Enter to retry, or Ctrl+C to cancel.');
+    await new Promise(r => process.stdin.once('data', r));
+    updateSpin.start();
+    result = spawnSync('npm', ['install', '-g', 'promptgraph-mcp@latest', '--force'], { encoding: 'utf8', stdio: 'pipe', shell: true });
+  }
+
   updateSpin.stop();
 
   if (result.status !== 0) {

package/github-import.js

@@ -197,9 +197,22 @@ export async function validateRepoSkills(ownerRepo) {
   return { ok: errors.length === 0, errors, warnings };
 }
 
+const SCRIPT_EXTS_API = new Set(['.py', '.sh', '.bash', '.js', '.ts', '.rb']);
+
+function countValidMd(fileEntries) {
+  return fileEntries.filter(e =>
+    e.type === 'file' && e.name.endsWith('.md') &&
+    !SKIP_RE.test(e.name.replace(/\.md$/i, '').toLowerCase())
+  ).length;
+}
+
+function hasScriptFiles(fileEntries) {
+  return fileEntries.some(e => e.type === 'file' && SCRIPT_EXTS_API.has(path.extname(e.name).toLowerCase()));
+}
+
 // Ask GitHub API which subdir to use (without cloning anything). Exported for validation.
+// Returns { subdir, label, validMdCount, hasScripts } or null (repo not found / no skills).
 export
-// Returns { subdir, label } or null (use root).
 async function detectSkillsDirFromAPI(ownerRepo) {
   try {
     const json = await httpsGet(`https://api.github.com/repos/${ownerRepo}/contents`);
@@ -208,7 +221,14 @@ async function detectSkillsDirFromAPI(ownerRepo) {
     // 1. Known skill dir names (priority order)
     const dirMap = new Map(entries.filter(e => e.type === 'dir').map(e => [e.name.toLowerCase(), e.name]));
     for (const d of SKILL_DIRS) {
-      if (dirMap.has(d)) return { subdir: dirMap.get(d), label: d };
+      if (dirMap.has(d)) {
+        try {
+          const sub = JSON.parse(await httpsGet(`https://api.github.com/repos/${ownerRepo}/contents/${dirMap.get(d)}`));
+          const validMdCount = countValidMd(sub);
+          if (validMdCount === 0) continue; // dir exists but no valid skills — try next
+          return { subdir: dirMap.get(d), label: d, validMdCount, hasScripts: hasScriptFiles(sub) };
+        } catch {}
+      }
     }
 
     // 1.5 Nested skills dirs (e.g. .claude/skills, .claude-plugin/commands)
@@ -217,30 +237,29 @@ async function detectSkillsDirFromAPI(ownerRepo) {
         for (const d of SKILL_DIRS) {
           const nested = `${prefix}/${d}`;
           try {
-            const sub = await httpsGet(`https://api.github.com/repos/${ownerRepo}/contents/${nested}`);
-            const subEntries = JSON.parse(sub);
-            if (subEntries.length > 0) return { subdir: nested, label: nested };
+            const sub = JSON.parse(await httpsGet(`https://api.github.com/repos/${ownerRepo}/contents/${nested}`));
+            const validMdCount = countValidMd(sub);
+            if (validMdCount > 0) return { subdir: nested, label: nested, validMdCount, hasScripts: hasScriptFiles(sub) };
           } catch {}
         }
       }
     }
 
-    // 2. Any subdir with 2+ .md files — pick the one with most .md files
+    // 2. Any subdir with valid .md files — pick the one with most
     const subdirCandidates = entries.filter(e => e.type === 'dir' && !SKIP_DIRS_API.has(e.name.toLowerCase()));
-    let best = null, bestCount = 0;
+    let best = null, bestCount = 0, bestScripts = false;
     for (const dir of subdirCandidates) {
       try {
-        const sub = await httpsGet(`https://api.github.com/repos/${ownerRepo}/contents/${dir.name}`);
-        const subEntries = JSON.parse(sub);
-        const mdCount = subEntries.filter(e => e.type === 'file' && e.name.endsWith('.md')).length;
-        if (mdCount >= 1 && mdCount > bestCount) { best = dir.name; bestCount = mdCount; }
+        const sub = JSON.parse(await httpsGet(`https://api.github.com/repos/${ownerRepo}/contents/${dir.name}`));
+        const mdCount = countValidMd(sub);
+        if (mdCount > bestCount) { best = dir.name; bestCount = mdCount; bestScripts = hasScriptFiles(sub); }
       } catch {}
     }
-    if (best) return { subdir: best, label: best };
+    if (best) return { subdir: best, label: best, validMdCount: bestCount, hasScripts: bestScripts };
 
-    // 3. Root-level .md files (no subdirectory, skills live at repo root)
-    const rootMd = entries.filter(e => e.type === 'file' && e.name.endsWith('.md') && !SKIP_RE.test(e.name.replace(/\.md$/i, '').toLowerCase()));
-    if (rootMd.length >= 1) return { subdir: null, label: 'root' };
+    // 3. Root-level valid .md files
+    const validMdCount = countValidMd(entries);
+    if (validMdCount >= 1) return { subdir: null, label: 'root', validMdCount, hasScripts: hasScriptFiles(entries) };
 
   } catch {}
   return null; // repo not found or inaccessible

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "promptgraph-mcp",
-  "version": "2.9.45",
+  "version": "2.9.47",
   "files": [
     "*.js",
     "commands/",