promptgraph-mcp 2.2.0 → 2.2.2

package/index.js

@@ -196,31 +196,31 @@ if (args[0] === 'marketplace') {
   const installedSet = new Set();
   try {
     const cfg = _lcMkt();
-    for (const s of cfg.sources) {
-      if (s.source.startsWith('github:')) installedSet.add(s.source.replace('github:', ''));
-      if (s.source === 'marketplace') installedSet.add('marketplace');
-    }
     const db = _getDbMkt();
-    for (const row of db.prepare('SELECT id FROM skills').all()) installedSet.add(row.id);
-    // Also add bundle IDs by matching repo names
+
+    // Collect installed skill IDs from DB
+    const dbSkillIds = new Set(db.prepare('SELECT id FROM skills').all().map(r => r.id));
+
+    // Build set of cloned repo names from config sources (exact: github:owner-repo)
+    const githubSources = new Set(
+      cfg.sources.filter(s => s.source.startsWith('github:')).map(s => s.source.replace('github:', '').toLowerCase())
+    );
+
     for (const b of (Array.isArray(bundles) ? bundles : [])) {
       if (b.repo_url) {
-        const repoName = b.repo_url.split('/').join('-');
-        if ([...installedSet].some(s => s.toLowerCase().includes(b.id.split('-').pop()))) {
-          installedSet.add(b.id);
-        }
-      }
-    }
-    // match github sources to bundle ids
-    for (const s of cfg.sources) {
-      if (!s.source.startsWith('github:')) continue;
-      const srcName = s.source.replace('github:', '').toLowerCase();
-      for (const b of (Array.isArray(bundles) ? bundles : [])) {
-        if (srcName.toLowerCase().includes(b.id.toLowerCase()) || b.id.toLowerCase().includes(srcName.split('-')[0])) {
+        // repo_url = "owner/repo" → cloned as "owner-repo" in github: source
+        const clonedName = b.repo_url.replace('/', '-').toLowerCase();
+        if (githubSources.has(clonedName)) installedSet.add(b.id);
+      } else if (Array.isArray(b.skills)) {
+        // skill-list bundle: installed if ALL skills are in DB
+        if (b.skills.length > 0 && b.skills.every(sid => dbSkillIds.has(sid))) {
           installedSet.add(b.id);
         }
       }
     }
+
+    // Individual skills
+    for (const id of dbSkillIds) installedSet.add(id);
   } catch {}
 
   const { runTUI } = await import('./tui.js');
@@ -246,12 +246,40 @@ if (args[0] === 'marketplace') {
 
 if (args[0] === 'validate') {
   const { validateSkill } = await import('./validator.js');
+  const { isSkillFile } = await import('./parser.js');
   const file = args[1];
   if (!file) { error('Usage: ' + bin + ' validate <skill.md>'); process.exit(1); }
+
+  const raw = fs.existsSync(file) ? fs.readFileSync(file, 'utf8') : null;
+
+  // Show indexing score breakdown
+  if (raw) {
+    const { skillScore: _score } = await import('./parser.js').catch(() => ({}));
+    const willIndex = isSkillFile(file, raw);
+    const scoreLabel = willIndex ? chalk.green('✓ will be indexed') : chalk.red('✗ will be skipped by indexer');
+    console.log(chalk.bold('\n  Indexing check: ') + scoreLabel);
+
+    // Show which signals were detected
+    const lines = raw.split('\n').filter(l => l.trim());
+    const signals = [];
+    try { const { data } = (await import('gray-matter')).default(raw); if (data.name) signals.push(chalk.green('+4 frontmatter name:')); } catch {}
+    if (/^#{1,3}\s+(steps?|usage|instructions?|how\s+to|when\s+to\s+use|workflow)/im.test(raw)) signals.push(chalk.green('+2 instructional headers (## Steps / ## Usage)'));
+    if (lines.filter(l => /^#{1,3}\s/.test(l)).some(h => /\b(run|use|fix|debug|check|create|deploy|scan|audit)\b/i.test(h))) signals.push(chalk.green('+2 imperative verbs in headers'));
+    if (raw.includes('```')) signals.push(chalk.green('+1 code block'));
+    if (lines.some(l => /^\d+\.\s/.test(l))) signals.push(chalk.green('+1 numbered list'));
+    if (lines.some(l => /^[-*+]\s/.test(l))) signals.push(chalk.green('+1 bullet list'));
+    const firstH = lines.find(l => /^#{1,3}\s/.test(l))?.replace(/^#+\s*/, '') || '';
+    if (/^(overview|introduction|about|background|welcome)/i.test(firstH)) signals.push(chalk.red('-3 first header looks like docs ("' + firstH + '")'));
+    if (signals.length) {
+      signals.forEach(s => console.log('    ' + s));
+    }
+    console.log();
+  }
+
   const result = validateSkill(file);
   result.warnings.forEach(w => console.log(chalk.yellow('⚠') + '  ' + chalk.gray(w)));
   if (result.ok) {
-    success('Skill is valid');
+    success('Skill is valid — ready to publish');
     process.exit(0);
   } else {
     error('Validation failed:');

package/package.json

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

package/tui.js

@@ -143,9 +143,7 @@ function render(state, installedSet = new Set()) {
     }
 
     // item row
-    const isInstalled = item.type === 'bundle'
-      ? installedSet.has(item.id) || installedSet.has(item.id.toLowerCase())
-      : installedSet.has(item.id) || installedSet.has(item.code);
+    const isInstalled = installedSet.has(item.id) || (item.code && installedSet.has(item.code));
     const arrow   = selected ? cyan('▶') : ' ';
     const type    = item.type === 'bundle' ? blue('⊞') : dim('·');
     const badge   = isInstalled ? green('✓') : ' ';
@@ -171,9 +169,7 @@ function render(state, installedSet = new Set()) {
   write(dim('─'.repeat(cols)) + CLEAR_EOL + '\n');
   const sel = items[cursor];
   if (sel && !searching) {
-    const isInst = sel.type === 'bundle'
-      ? installedSet.has(sel.id) || installedSet.has(sel.id.toLowerCase())
-      : installedSet.has(sel.id) || installedSet.has(sel.code);
+    const isInst = installedSet.has(sel.id) || (sel.code && installedSet.has(sel.code));
     const installCmd = sel.type === 'bundle' ? `bundle install ${sel.id}` : `install ${sel.code || sel.id}`;
     const instLabel = isInst ? green(' ✓ installed') + dim('  ') : dim(' Enter') + chalk.white(' install') + dim('  ');
     write(instLabel + dim('Tab') + ' switch  ' + dim('/') + ' search  ' + dim('q') + ' quit' + CLEAR_EOL + '\n');