promptgraph-mcp 2.6.8 → 2.7.0

package/github-import.js

@@ -436,43 +436,33 @@ async function classifierCleanup(dest) {
 
   const parsed = [];
   const fileMap = [];
-  let heuristicRemoved = 0;
 
   for (const fp of mdFiles) {
     try {
       const raw = fs.readFileSync(fp, 'utf8');
       if (!isSkillFile(fp, raw)) continue;
-      if (!seemsLikeSkill(raw)) {
-        try { fs.unlinkSync(fp); heuristicRemoved++; } catch {}
-        continue;
-      }
       parsed.push(parseSkillFile(fp, '', { raw }));
       fileMap.push(fp);
     } catch {}
   }
 
-  if (heuristicRemoved > 0) {
-    console.log(`Removed ${heuristicRemoved} files by content heuristic`);
-  }
-
   if (parsed.length === 0) {
-    if (heuristicRemoved > 0) removeEmptyDirs(dest);
+    removeEmptyDirs(dest);
     return;
   }
 
   const filtered = await filterWithClassifier(parsed);
   const keptPaths = new Set(filtered.map(s => s.path));
 
-  let classifierRemoved = 0;
+  let removed = 0;
   for (const fp of fileMap) {
     if (!keptPaths.has(fp)) {
-      try { fs.unlinkSync(fp); classifierRemoved++; } catch {}
+      try { fs.unlinkSync(fp); removed++; } catch {}
     }
   }
 
-  const totalRemoved = heuristicRemoved + classifierRemoved;
-  if (totalRemoved > 0) {
-    console.log(`Removed ${totalRemoved} non-skill files (heuristic: ${heuristicRemoved}, classifier: ${classifierRemoved})`);
+  if (removed > 0) {
+    console.log(`Removed ${removed} non-skill files (classifier)`);
     removeEmptyDirs(dest);
   }
 }

package/marketplace.js

@@ -13,6 +13,7 @@ import { isSkillFile } from './parser.js';
 
 const REGISTRY_URL = 'https://raw.githubusercontent.com/NeiP4n/promptgraph-registry/main/registry.json';
 const SKILL_COUNT_CACHE = path.join(PROMPTGRAPH_DIR, 'skill-counts.json');
+const DEAD_REPOS_FILE = path.join(PROMPTGRAPH_DIR, 'dead-repos.json');
 const SKILLS_DIR = path.join(SKILLS_STORE_DIR, 'marketplace');
 
 // Atomically write content to dest via tmp — cleans up on failure
@@ -143,6 +144,21 @@ function writeSkillCountCache(data) {
   } catch {}
 }
 
+function readDeadRepos() {
+  try { return JSON.parse(fs.readFileSync(DEAD_REPOS_FILE, 'utf8')); } catch { return []; }
+}
+
+function markDeadRepo(repoUrl) {
+  try {
+    const dead = readDeadRepos();
+    if (!dead.includes(repoUrl)) {
+      dead.push(repoUrl);
+      fs.mkdirSync(path.dirname(DEAD_REPOS_FILE), { recursive: true });
+      fs.writeFileSync(DEAD_REPOS_FILE, JSON.stringify(dead, null, 2));
+    }
+  } catch {}
+}
+
 async function fetchText(url) {
   const cacheFile = path.join(PROMPTGRAPH_DIR, 'registry-cache.json');
   const isRegistry = url === REGISTRY_URL;
@@ -275,7 +291,10 @@ export async function browseBundles(topK = 20) {
   try {
     const text = await fetchText(REGISTRY_URL);
     const registry = JSON.parse(text);
-    const bundles = (registry.bundles || []).filter(b => validateRegistryEntry(b).ok);
+    const deadRepos = new Set(readDeadRepos());
+    const bundles = (registry.bundles || []).filter(b =>
+      validateRegistryEntry(b).ok && !deadRepos.has(b.repo_url)
+    );
     const cache = readSkillCountCache();
     const now = Date.now();
     let changed = false;
@@ -346,7 +365,14 @@ export async function installBundle(bundleId) {
     if (!bundle) return { error: `No bundle matching "${bundleId}"` };
 
     if (bundle.repo_url) {
-      await importFromGitHub(bundle.repo_url);
+      try {
+        await importFromGitHub(bundle.repo_url);
+      } catch (e) {
+        if (e.message && e.message.includes('No valid skills')) {
+          markDeadRepo(bundle.repo_url);
+        }
+        throw e;
+      }
       return { success: true, bundle: bundle.name, type: 'repo_import', repo_url: bundle.repo_url };
     }
 

package/package.json

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

package/validator.js

@@ -50,22 +50,22 @@ export function validateSkill(filePath) {
     return { ok: false, errors, warnings };
   }
 
-  // name
+  // name — derive from filename if missing (handles plain .md repos)
   if (!data.name) {
-    errors.push('Missing required field: name');
+    warnings.push('Missing frontmatter "name" — derived from filename');
   } else if (typeof data.name !== 'string') {
     errors.push('Field "name" must be a string');
   } else if (!NAME_RE.test(data.name)) {
     errors.push(`Invalid name "${data.name}". Use lowercase, digits, hyphens (2-64 chars).`);
   }
 
-  // description
+  // description — derive from first paragraph if missing
   if (!data.description) {
-    errors.push('Missing required field: description');
+    warnings.push('Missing frontmatter "description" — derived from content');
   } else if (typeof data.description !== 'string') {
     errors.push('Field "description" must be a string');
   } else if (data.description.trim().length < MIN_DESCRIPTION_LENGTH) {
-    errors.push(`Description too short (min ${MIN_DESCRIPTION_LENGTH} chars).`);
+    warnings.push(`Description very short (${data.description.trim().length} chars).`);
   }
 
   // body must have real instruction content