promptgraph-mcp 2.6.6 → 2.6.8

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); }
 
@@ -88,6 +91,10 @@ export default async function handler(args, bin) {
         const r = await installBundle(item.id);
         if (r?.error) throw new Error(r.error);
         installedSet.add(item.id);
+        // Update skill count on the bundle object to reflect real survivors
+        const { getCachedCount: getCount } = await import('../bundle-counts.js');
+        const cached = getCount(item.repo_url);
+        if (cached !== null) item.skillCount = cached;
       } else {
         const r = await installSkill(item.code || item.id);
         if (r?.error) throw new Error(r.error);

package/github-import.js

@@ -580,7 +580,7 @@ export async function importFromGitHub(repoUrl) {
 
   await classifierCleanup(dest);
 
-  // Update skill count cache with real survivor count
+  // Count survivors after all cleanup
   const realCount = globSync(`${dest}/**/*.md`).length;
   const cacheKey = url.replace(/\.git$/, '');
   const cachePath = path.join(PROMPTGRAPH_DIR, 'skill-counts.json');
@@ -591,6 +591,11 @@ export async function importFromGitHub(repoUrl) {
     fs.writeFileSync(cachePath, JSON.stringify(cache, null, 2));
   } catch {}
 
+  if (realCount < 1) {
+    fs.rmSync(dest, { recursive: true, force: true });
+    throw new Error(`No valid skills in repo — all files were filtered out`);
+  }
+
   const { dir: localDir, label: localLabel } = detectSkillsDirLocal(dest);
   // Prefer the known skillsSubdir (from API detection or sparse patterns) as the
   // canonical skills directory — it's more accurate than local heuristics for
@@ -609,8 +614,6 @@ export async function importFromGitHub(repoUrl) {
     console.log(`Full clone: scanning ${label} (${mdFiles.length} .md files)`);
   }
 
-  if (mdFiles.length < 1) console.warn('Warning: no .md files found');
-
   const config = loadConfig();
   const repoSource = `github:${repoName}`;
   if (!config.sources.find(s => s.dir === skillsDir)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "promptgraph-mcp",
-  "version": "2.6.6",
+  "version": "2.6.8",
   "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;