promptgraph-mcp 2.3.6 → 2.3.8

package/github-import.js

@@ -33,7 +33,8 @@ function repoExists(ownerRepo) {
   });
 }
 
-// Ask GitHub API which subdir to use (without cloning anything).
+// Ask GitHub API which subdir to use (without cloning anything). Exported for validation.
+export
 // Returns { subdir, label } or null (use root).
 async function detectSkillsDirFromAPI(ownerRepo) {
   try {
@@ -211,19 +212,21 @@ export async function importFromGitHub(repoUrl) {
     const detected = await detectSkillsDirFromAPI(ownerRepo);
     skillsSubdir = detected?.subdir || null;
 
-    if (skillsSubdir) {
-      console.log(`found: ${detected.label}/`);
-      console.log(`Sparse-cloning ${url} (${skillsSubdir}/ only)...`);
-      cloneOk = sparseClone(url, dest, skillsSubdir);
-      if (!cloneOk) {
-        console.log('Sparse-checkout failed, falling back to full clone...');
-        fs.rmSync(dest, { recursive: true, force: true });
-        cloneOk = fullClone(url, dest);
-        skillsSubdir = null;
-      }
-    } else {
-      console.log(`no subdir found, cloning root...`);
-      cloneOk = fullClone(url, dest);
+    if (!skillsSubdir) {
+      throw new Error(
+        `No skill subdirectory found in ${ownerRepo}.\n` +
+        `Expected one of: ${SKILL_DIRS.join(', ')}\n` +
+        `Or any subfolder with .md files.\n` +
+        `Visit https://github.com/${ownerRepo} to check the repo structure.`
+      );
+    }
+
+    console.log(`found: ${detected.label}/`);
+    console.log(`Sparse-cloning ${url} (${skillsSubdir}/ only)...`);
+    cloneOk = sparseClone(url, dest, skillsSubdir);
+    if (!cloneOk) {
+      fs.rmSync(dest, { recursive: true, force: true });
+      throw new Error(`Sparse-checkout failed for ${url}`);
     }
 
     if (!cloneOk) throw new Error(`Clone failed for ${url}`);

package/index.js

@@ -449,6 +449,21 @@ if (args[0] === 'bundle') {
     const repoArg = doPush ? args[2] : args[2];
     if (!repoArg || !repoArg.includes('/')) { error('Usage: pg bundle add-repo <owner/repo> [--push]'); process.exit(1); }
     const repo = repoArg.replace('https://github.com/', '').replace('.git', '');
+
+    // Validate repo has a skill subdirectory before publishing
+    const { detectSkillsDirFromAPI: _detectDir } = await import('./github-import.js');
+    process.stdout.write(chalk.gray(`  Checking ${repo} for skill subdirectory... `));
+    const detected = await _detectDir(repo);
+    if (!detected) {
+      console.log(chalk.red('none found'));
+      error(
+        `Cannot publish: no skill subdirectory found in ${repo}\n` +
+        `  Expected: skills/, prompts/, commands/, agents/, or any folder with .md files\n` +
+        `  Visit: https://github.com/${repo}`
+      );
+      process.exit(1);
+    }
+    console.log(chalk.green(`found: ${detected.label}/`));
     const name = repo.split('/')[1].replace(/-/g, ' ').replace(/\b\w/g, c => c.toUpperCase());
     const id = repo.replace('/', '-').toLowerCase();
     const bundle = {

package/package.json

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

package/tui.js

@@ -178,7 +178,8 @@ function render(state, installedSet = new Set()) {
       ? green(' ✓ installed') + dim('  ') + dim('d') + chalk.red(' remove') + dim('  ')
       : dim(' Enter') + chalk.white(' install') + dim('  ');
     write(instLabel + dim('Tab') + ' switch  ' + dim('/') + ' search  ' + dim('q') + ' quit' + CLEAR_EOL + '\n');
-    write(dim(` → pg ${installCmd}`) + CLEAR_EOL + '\n');
+    const ghUrl = sel.repo_url ? chalk.hex('#3B82F6')(`  ↗ github.com/${sel.repo_url}`) : '';
+    write(dim(` → pg ${installCmd}`) + ghUrl + CLEAR_EOL + '\n');
   } else if (state.confirming) {
     write(chalk.red.bold(' Remove ') + chalk.white(state.confirming.name) + chalk.red('? ') +
       chalk.white.bold('[y]') + chalk.gray('es  ') + chalk.white.bold('[n]') + chalk.gray('o') + CLEAR_EOL + '\n');