promptgraph-mcp 2.2.5 → 2.2.7

package/index.js

@@ -208,35 +208,38 @@ if (args[0] === 'marketplace') {
 
   if (skills?.error) { error(skills.error); process.exit(1); }
 
-  // Build installed set: bundle IDs from config sources + skill IDs from DB
+  // Build installed set — source of truth is the FILESYSTEM, not DB/config
   const installedSet = new Set();
   try {
     const cfg = _lcMkt();
     const db = _getDbMkt();
-
-    // 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())
-    );
+    const { SKILLS_STORE_DIR } = await import('./config.js');
+    const githubDir = path.join(SKILLS_STORE_DIR, 'github');
 
     for (const b of (Array.isArray(bundles) ? bundles : [])) {
       if (b.repo_url) {
-        // 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);
+        // Check actual cloned directory exists and has files
+        const owner = b.repo_url.split('/')[0];
+        const repo  = b.repo_url.split('/')[1];
+        const clonedName = `${owner}-${repo}`;
+        const clonedDir  = path.join(githubDir, clonedName);
+        const dirExists  = fs.existsSync(clonedDir) &&
+          fs.readdirSync(clonedDir).length > 0; // not empty
+        if (dirExists) 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);
-        }
+        // skill-list bundle: check files exist on disk via DB path column
+        const allOnDisk = b.skills.every(sid => {
+          const row = db.prepare('SELECT path FROM skills WHERE id = ?').get(sid);
+          return row && fs.existsSync(row.path);
+        });
+        if (b.skills.length > 0 && allOnDisk) installedSet.add(b.id);
       }
     }
 
-    // Individual skills
-    for (const id of dbSkillIds) installedSet.add(id);
+    // Individual marketplace skills — check file exists on disk
+    for (const row of db.prepare('SELECT id, path FROM skills WHERE source = ?').all('marketplace')) {
+      if (fs.existsSync(row.path)) installedSet.add(row.id);
+    }
   } catch {}
 
   const { runTUI } = await import('./tui.js');

package/package.json

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

package/tui.js

@@ -153,9 +153,9 @@ function render(state, installedSet = new Set()) {
       ? item.skillCount
         ? blue((item.skillCount + ' sk').padEnd(8))
         : item.repo_url
-          ? blue('GitHub  ')
+          ? chalk.hex('#3B82F6')('↗ GitHub')
           : dim(((item.skills?.length || 0) + ' sk').padEnd(8))
-      : dim((item.code || '').padEnd(8));
+      : chalk.hex('#A78BFA')((item.code || '').padEnd(10));
     const desc = dim(truncate(item.description, Math.max(10, DESC_W)));
 
     write(bg + `  ${arrow} ${type} ${badge} ${nameCol}  ${extra}  ${desc}` + reset + CLEAR_EOL + '\n');
@@ -188,16 +188,37 @@ function render(state, installedSet = new Set()) {
 
 // ── clamp scroll ─────────────────────────────────────────────────────────────
 
+// Count how many screen rows items[start..end] occupy (including category headers)
+function countVisibleRows(items, start, end) {
+  let rows = 0;
+  let lastCat = start > 0 ? items[start - 1]?.category : null;
+  for (let i = start; i < end && i < items.length; i++) {
+    if (items[i].category !== lastCat) { rows++; lastCat = items[i].category; }
+    rows++;
+  }
+  return rows;
+}
+
 function clampScroll(state) {
   const { rows } = termSize();
   const HEADER_ROWS = 4, FOOTER_ROWS = 3;
   const LIST_ROWS = rows - HEADER_ROWS - FOOTER_ROWS;
   const { cursor, items } = state;
 
-  // ensure cursor visible — approximate (category headers add extra rows)
-  if (cursor < state.scroll) state.scroll = cursor;
-  if (cursor >= state.scroll + LIST_ROWS - 2) state.scroll = cursor - LIST_ROWS + 3;
   if (state.scroll < 0) state.scroll = 0;
+  if (state.scroll > cursor) state.scroll = cursor;
+
+  // Check if cursor is visible from current scroll
+  const visibleRows = countVisibleRows(items, state.scroll, cursor + 1);
+  if (visibleRows > LIST_ROWS - 1) {
+    // Cursor is below visible area — scroll forward until it fits
+    while (state.scroll < cursor) {
+      state.scroll++;
+      const v = countVisibleRows(items, state.scroll, cursor + 1);
+      if (v <= LIST_ROWS - 1) break;
+    }
+  }
+
   if (state.scroll >= items.length) state.scroll = Math.max(0, items.length - 1);
 }