promptgraph-mcp 2.6.5 → 2.6.7

package/commands/marketplace.js

@@ -31,8 +31,11 @@ export default async function handler(args, bin) {
   const { spinner: spin2 } = await import('../cli.js');
   const sp = spin2('Fetching marketplace...');
   sp.start();
-  const [skills, bundles] = await Promise.all([browseMarketplace(1000), browseBundles(1000)]);
-  sp.stop();
+  try {
+    var [skills, bundles] = await Promise.all([browseMarketplace(1000), browseBundles(1000)]);
+  } finally {
+    sp.stop();
+  }
 
   if (skills?.error) { error(skills.error); process.exit(1); }
 

package/github-import.js

@@ -582,14 +582,13 @@ export async function importFromGitHub(repoUrl) {
 
   // Update skill count cache with real survivor count
   const realCount = globSync(`${dest}/**/*.md`).length;
+  const cacheKey = url.replace(/\.git$/, '');
   const cachePath = path.join(PROMPTGRAPH_DIR, 'skill-counts.json');
   try {
-    const cache = JSON.parse(fs.readFileSync(cachePath, 'utf8'));
-    if (cache[url]) {
-      cache[url].count = realCount;
-      cache[url].ts = Date.now();
-      fs.writeFileSync(cachePath, JSON.stringify(cache, null, 2));
-    }
+    fs.mkdirSync(path.dirname(cachePath), { recursive: true });
+    const cache = JSON.parse(fs.readFileSync(cachePath, 'utf8') || '{}');
+    cache[cacheKey] = { count: realCount, ts: Date.now() };
+    fs.writeFileSync(cachePath, JSON.stringify(cache, null, 2));
   } catch {}
 
   const { dir: localDir, label: localLabel } = detectSkillsDirLocal(dest);

package/marketplace.js

@@ -262,6 +262,15 @@ async function countRepoSkills(repoUrl) {
   } catch { return null; }
 }
 
+// Count real .md files on disk for an installed bundle (always correct)
+function localSkillCount(repoUrl) {
+  const repoName = repoUrl.replace(/^https?:\/\/github\.com\//, '').replace(/\.git$/, '').replace('/', '-');
+  const dest = path.join(SKILLS_STORE_DIR, 'github', repoName);
+  if (!fs.existsSync(dest)) return null;
+  const files = globSync(`${dest}/**/*.md`);
+  return files.length;
+}
+
 export async function browseBundles(topK = 20) {
   try {
     const text = await fetchText(REGISTRY_URL);
@@ -273,20 +282,33 @@ export async function browseBundles(topK = 20) {
 
     await Promise.all(bundles.map(async b => {
       if (!b.repo_url) return;
+
+      // 1. If installed locally — count real files on disk (always correct)
+      const local = localSkillCount(b.repo_url);
+      if (local !== null) {
+        if (b.skillCount !== local) {
+          b.skillCount = local;
+          cache[b.repo_url] = { count: local, ts: now };
+          changed = true;
+        }
+        return;
+      }
+
+      // 2. Not installed — use cached API count if fresh
       const cached = cache[b.repo_url];
-      // Use cached count if fresh, else fetch from API
       if (cached && (now - cached.ts) < SKILL_COUNT_TTL) {
         b.skillCount = cached.count;
+        return;
+      }
+
+      // 3. Fetch from GitHub API
+      const count = await countRepoSkills(b.repo_url);
+      if (count !== null) {
+        b.skillCount = count;
+        cache[b.repo_url] = { count, ts: now };
+        changed = true;
       } else {
-        const count = await countRepoSkills(b.repo_url);
-        if (count !== null) {
-          b.skillCount = count;
-          cache[b.repo_url] = { count, ts: now };
-          changed = true;
-        } else {
-          // API failed — use stale cache if exists, else keep registry value as fallback
-          b.skillCount = cached?.count ?? b.skillCount ?? 0;
-        }
+        b.skillCount = cached?.count ?? b.skillCount ?? 0;
       }
     }));
 

package/package.json

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

package/tui.js

@@ -22,8 +22,10 @@ const green   = chalk.green;
 const red     = chalk.red;
 const blue    = chalk.blue;
 const white   = chalk.white;
+const magenta = chalk.magenta;
 
 const CAT_ICON = { Engineering:'🛠', 'AI Tools':'🤖', Coding:'💻', Creative:'🎨', Security:'🔒', Community:'🌐' };
+const SPINNER_FRAMES = ['⣾', '⣽', '⣻', '⢿', '⡿', '⣟', '⣯', '⣷']; // braille spinner
 
 // ── helpers ──────────────────────────────────────────────────────────────────
 
@@ -113,8 +115,11 @@ function render(state, installedSet = new Set()) {
     : dim(query ? query : 'type / to search, Tab to switch view');
   write(searchLabel + searchVal + CLEAR_EOL + '\n');
 
-  // Row 4: separator (with optional status inline)
-  if (status) {
+  // Row 4: separator (with optional status inline or spinner)
+  if (state.installing) {
+    const frame = SPINNER_FRAMES[Math.floor(Date.now() / 120) % SPINNER_FRAMES.length];
+    write(dim('─'.repeat(4)) + magenta(` ${frame} Installing… `) + CLEAR_EOL + '\n');
+  } else if (status) {
     const msg = status.ok ? green(' ✓ ' + status.msg) : red(' ✗ ' + status.msg);
     write(dim('─'.repeat(4)) + msg + CLEAR_EOL + '\n');
   } else {
@@ -388,11 +393,15 @@ export async function runTUI(allSkills, allBundles, installFn, installedSet = ne
       const sel = state.items[state.cursor];
       if (!sel || state.installing) return;
       state.installing = true;
-      setStatus(null, `Installing ${sel.id}…`);
+      // Live spinner — keeps rendering during long installs
+      const spinInterval = setInterval(() => render(state, installedSet), 120);
+      render(state, installedSet);
       try {
         await installFn(sel);
+        clearInterval(spinInterval);
         setStatus(true, `Installed ${sel.id}`);
       } catch (e) {
+        clearInterval(spinInterval);
         setStatus(false, e.message.slice(0, 60));
       } finally {
         state.installing = false;