promptgraph-mcp 2.0.1 → 2.0.3

package/github-import.js

@@ -4,9 +4,24 @@ import os from 'os';
 import fs from 'fs';
 import { globSync } from 'glob';
 import { indexAll } from './indexer.js';
-import { loadConfig, saveConfig } from './config.js';
+import { loadConfig, saveConfig, SKILLS_STORE_DIR } from './config.js';
 
-const SKILLS_DIR = path.join(os.homedir(), '.claude', 'skills-store');
+// Directories likely to contain skills — checked in priority order
+const SKILL_DIRS = ['skills', 'commands', 'prompts', 'agents', 'skills-store', 'slash-commands', 'custom-commands', 'templates'];
+
+// Find the best subdirectory to index in the cloned repo.
+// Returns the subdir path if a known skills dir exists with 2+ .md files,
+// otherwise returns the repo root (full scan with isSkillFile filtering).
+function detectSkillsDir(repoRoot) {
+  for (const dir of SKILL_DIRS) {
+    const candidate = path.join(repoRoot, dir);
+    if (fs.existsSync(candidate)) {
+      const files = globSync(`${candidate}/**/*.md`);
+      if (files.length >= 2) return { dir: candidate, auto: true, label: dir };
+    }
+  }
+  return { dir: repoRoot, auto: false, label: '(root)' };
+}
 
 export async function importFromGitHub(repoUrl) {
   if (!repoUrl) {
@@ -16,7 +31,7 @@ export async function importFromGitHub(repoUrl) {
 
   const url = repoUrl.startsWith('http') ? repoUrl : `https://github.com/${repoUrl}`;
   const repoName = url.split('/').slice(-2).join('-').replace('.git', '');
-  const dest = path.join(SKILLS_DIR, 'github', repoName);
+  const dest = path.join(SKILLS_STORE_DIR, 'github', repoName);
 
   fs.mkdirSync(path.dirname(dest), { recursive: true });
 
@@ -30,23 +45,31 @@ export async function importFromGitHub(repoUrl) {
     if (cloneResult.status !== 0) throw new Error(`git clone failed for ${url}`);
   }
 
-  const mdFiles = globSync(`${dest}/**/*.md`);
-  console.log(`Found ${mdFiles.length} .md files`);
+  const { dir: skillsDir, auto, label } = detectSkillsDir(dest);
+  const mdFiles = globSync(`${skillsDir}/**/*.md`);
+
+  if (auto) {
+    console.log(`Auto-detected skills directory: ${label}/ (${mdFiles.length} .md files)`);
+  } else {
+    console.log(`No skills/ dir found — scanning root (${mdFiles.length} .md files, non-skill files will be filtered)`);
+  }
 
-  if (mdFiles.length < 2) {
-    console.warn('Warning: repo has fewer than 2 .md files — may be empty');
+  if (mdFiles.length < 1) {
+    console.warn('Warning: no .md files found');
   }
 
   const config = loadConfig();
-  // Per-repo source so two repos with the same skill name don't overwrite each other
   const repoSource = `github:${repoName}`;
-  if (!config.sources.find(s => s.dir === dest)) {
-    config.sources.push({ dir: dest, source: repoSource });
+  if (!config.sources.find(s => s.dir === skillsDir)) {
+    // Remove any old entry pointing at the full repo root if we now have a subdir
+    const oldIdx = config.sources.findIndex(s => s.source === repoSource);
+    if (oldIdx !== -1) config.sources.splice(oldIdx, 1);
+    config.sources.push({ dir: skillsDir, source: repoSource });
     saveConfig(config);
-    console.log(`Added ${repoSource} to config`);
+    console.log(`Indexing from: ${skillsDir}`);
   }
 
   console.log('\nReindexing...');
   await indexAll();
-  console.log(`Done! Imported ${mdFiles.length} files from ${repoName}`);
+  console.log(`Done! Imported from ${repoName}/${label}`);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "promptgraph-mcp",
-  "version": "2.0.1",
+  "version": "2.0.3",
   "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 = {}) {