promptgraph-mcp 2.0.1 → 2.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "promptgraph-mcp",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "main": "index.js",
   "type": "module",
   "bin": {

package/parser.js

@@ -5,12 +5,65 @@ import matter from 'gray-matter';
 // match /skill-name but not URLs (http://, https://, etc.)
 const SKILL_REF_RE = /(?<!https?:|ftp:)(?<![a-zA-Z0-9])\/([a-z0-9][a-z0-9-]{2,})/g;
 
-// files that are likely not skills
-const SKIP_FILENAMES = new Set(['readme', 'changelog', 'license', 'contributing', 'code-of-conduct', 'security', 'authors', 'credits']);
+// Filenames that are never skills — docs, meta, legal files
+const SKIP_FILENAMES = new Set([
+  'readme', 'changelog', 'license', 'contributing', 'code-of-conduct',
+  'security', 'authors', 'credits', 'install', 'installation', 'usage',
+  'engagements', 'contributing', 'contributors', 'maintainers',
+  'acknowledgements', 'faq', 'glossary', 'index', 'overview', 'summary',
+  'roadmap', 'todo', 'notes', 'template', 'example', 'sample', 'demo',
+  'getting-started', 'quickstart', 'guide', 'tutorial', 'walkthrough',
+  'architecture', 'design', 'spec', 'specification', 'requirements',
+  'privacy', 'terms', 'disclaimer', 'notice', 'copying', 'warranty',
+  'codeofconduct', 'pull_request_template', 'issue_template', 'funding',
+]);
+
+// Path segments that indicate the file is NOT a skill
+const SKIP_DIRS = new Set([
+  '.github', 'docs', 'doc', 'documentation', 'examples', 'example',
+  'tests', 'test', '__tests__', 'spec', 'fixtures', 'assets', 'images',
+  'img', 'screenshots', 'media', 'static', 'public', 'dist', 'build',
+  'node_modules', 'vendor', 'third_party',
+]);
 
 export function isSkillFile(filePath) {
-  const base = filePath.split(/[\\/]/).pop().replace(/\.md$/i, '').toLowerCase();
-  return !SKIP_FILENAMES.has(base);
+  const parts = filePath.replace(/\\/g, '/').split('/');
+  const base = parts[parts.length - 1].replace(/\.md$/i, '').toLowerCase();
+
+  // Skip by filename
+  if (SKIP_FILENAMES.has(base)) return false;
+
+  // Skip if any parent directory is in the skip list
+  for (const part of parts.slice(0, -1)) {
+    if (SKIP_DIRS.has(part.toLowerCase())) return false;
+  }
+
+  // Read and check content quality
+  try {
+    const raw = fs.readFileSync(filePath, 'utf8');
+
+    // Too short to be a real skill
+    if (raw.length < 150) return false;
+
+    // Has valid frontmatter with name → definitely a skill
+    try {
+      const { data } = matter(raw);
+      if (data.name && typeof data.name === 'string') return true;
+    } catch {}
+
+    // No frontmatter — check if it looks like instructions (not pure docs)
+    // Must have some imperative/instructional content, not just markdown prose
+    const lines = raw.split('\n').filter(l => l.trim());
+    const hasCodeBlock = raw.includes('```');
+    const hasBullets = lines.some(l => /^[-*]\s/.test(l));
+    const hasNumbered = lines.some(l => /^\d+\.\s/.test(l));
+    const hasHeaders = lines.filter(l => /^#{1,3}\s/.test(l)).length >= 2;
+
+    // Must look structured, not just a prose document
+    return (hasCodeBlock || hasBullets || hasNumbered) && hasHeaders;
+  } catch {
+    return false;
+  }
 }
 
 export function parseSkillFile(filePath, source, opts = {}) {