skissue 0.1.17 → 0.1.19

package/dist/entry.js

@@ -9,7 +9,7 @@ import { resolve as resolve4 } from "node:path";
 // src/commands/init.ts
 import * as p2 from "@clack/prompts";
 import chalk2 from "chalk";
-import { existsSync as existsSync4 } from "node:fs";
+import { existsSync as existsSync4, statSync as statSync3 } from "node:fs";
 import { mkdir as mkdir4 } from "node:fs/promises";
 import { relative, resolve as resolve3 } from "node:path";
 
@@ -350,6 +350,13 @@ import chalk from "chalk";
 import { existsSync as existsSync3, statSync as statSync2 } from "node:fs";
 import { mkdir as mkdir3, writeFile as writeFile2 } from "node:fs/promises";
 import { join as join3, resolve as resolve2 } from "node:path";
+
+// src/commands/prompt-validators.ts
+function requiredTrimmed(v) {
+  return String(v ?? "").trim() ? void 0 : "Required";
+}
+
+// src/commands/init-registry.ts
 function validateSkillId(raw) {
   const id = raw.trim();
   if (!id) return "Skill id is required";
@@ -357,6 +364,18 @@ function validateSkillId(raw) {
   if (id === "." || id === ".." || id.includes("..")) return "Invalid skill id";
   return void 0;
 }
+function validateSkillIdPrompt(v) {
+  return validateSkillId(String(v ?? ""));
+}
+function registryDirectoryExists(root) {
+  const regDir = join3(root, "registry");
+  if (!existsSync3(regDir)) return false;
+  try {
+    return statSync2(regDir).isDirectory();
+  } catch {
+    return false;
+  }
+}
 function registryLayoutExists(root) {
   const jsonPath = join3(root, "registry.json");
   const regDir = join3(root, "registry");
@@ -440,7 +459,7 @@ async function promptMinimalRegistryScaffold(root) {
   const idRaw = await p.text({
     message: "Sample skill id (folder name under registry/)",
     initialValue: "sample-skill",
-    validate: (v) => validateSkillId(String(v ?? ""))
+    validate: validateSkillIdPrompt
   });
   if (p.isCancel(idRaw)) {
     return null;
@@ -494,6 +513,7 @@ async function runInitRegistry(cwd) {
   if (p.isCancel(where)) {
     p.cancel("Aborted.");
     process.exit(0);
+    return;
   }
   let root;
   if (where === "here") {
@@ -502,11 +522,12 @@ async function runInitRegistry(cwd) {
     const raw = await p.text({
       message: "Path to registry root (absolute or relative to current directory)",
       placeholder: "../my-skill-registry",
-      validate: (v) => v?.trim() ? void 0 : "Required"
+      validate: requiredTrimmed
     });
     if (p.isCancel(raw)) {
       p.cancel("Aborted.");
       process.exit(0);
+      return;
     }
     root = resolve2(cwd, String(raw).trim());
   }
@@ -514,6 +535,7 @@ async function runInitRegistry(cwd) {
   if (!prompted) {
     p.cancel("Aborted. No files were changed.");
     process.exit(0);
+    return;
   }
   const { skillId, runGitInit, hadGit } = prompted;
   try {
@@ -561,6 +583,59 @@ Transport: ${mode}
 Branch: ${r.branch}
 skillsRoot: ${cfg.skillsRoot}`;
 }
+async function runLocalBootstrapFlow(cwd, resolvedRoot) {
+  const atConsumerRoot = resolvedRoot === resolve3(cwd);
+  if (atConsumerRoot && !registryLayoutExists(resolvedRoot)) {
+    const okRoot = await p2.confirm({
+      message: "This adds registry.json and registry/ at your project root. Continue?",
+      initialValue: false
+    });
+    if (p2.isCancel(okRoot) || !okRoot) {
+      p2.cancel("Aborted.");
+      return { ok: false, exitCode: 0 };
+    }
+  }
+  const prompted = await promptMinimalRegistryScaffold(resolvedRoot);
+  if (!prompted) {
+    p2.cancel("Aborted. No files were changed.");
+    return { ok: false, exitCode: 0 };
+  }
+  let result;
+  try {
+    result = await scaffoldMinimalRegistry({
+      root: resolvedRoot,
+      skillId: prompted.skillId,
+      runGitInit: prompted.runGitInit
+    });
+  } catch (e) {
+    p2.cancel(e instanceof Error ? e.message : String(e));
+    return { ok: false, exitCode: 1 };
+  }
+  if (!prompted.hadGit && !result.gitInitRan) {
+    p2.note(
+      "This folder is not a git repository. Run `git init` before using it as a local skissue registry.",
+      chalk2.yellow("Heads up")
+    );
+  }
+  const branchDefault = await gitBranchShowCurrent(resolvedRoot) ?? "main";
+  const branch = await p2.text({
+    message: "Branch label (stored in config; installs use git HEAD from that directory)",
+    initialValue: branchDefault,
+    validate: requiredTrimmed
+  });
+  if (p2.isCancel(branch)) {
+    p2.cancel("Aborted.");
+    return { ok: false, exitCode: 0 };
+  }
+  const cfg = ConfigSchema.parse({
+    registry: {
+      path: localRegistryPathForConfig(cwd, resolvedRoot),
+      branch: String(branch).trim()
+    },
+    skillsRoot: ".agents/skills"
+  });
+  return { ok: true, cfg, bootstrappedSkillId: result.skillId };
+}
 async function runInit(cwd) {
   p2.intro(chalk2.bold("skissue init"));
   const existingPath = configPath(cwd);
@@ -626,100 +701,93 @@ async function runInit(cwd) {
         message: "Path to registry repo root (relative to this project or absolute). Must contain registry/ and be a git repository.",
         placeholder: ".",
         initialValue: ".",
-        validate: (v) => v?.trim() ? void 0 : "Required"
+        validate: requiredTrimmed
       });
       if (p2.isCancel(regPath)) {
         p2.cancel("Aborted.");
         process.exit(0);
       }
       const resolvedExisting = resolve3(cwd, String(regPath).trim());
-      const branchDefault = await gitBranchShowCurrent(resolvedExisting) ?? "main";
-      const branch = await p2.text({
-        message: "Branch label (stored in config; installs use git HEAD from that directory)",
-        initialValue: branchDefault,
-        validate: (v) => v?.trim() ? void 0 : "Required"
-      });
-      if (p2.isCancel(branch)) {
-        p2.cancel("Aborted.");
-        process.exit(0);
+      if (!existsSync4(resolvedExisting) || !statSync3(resolvedExisting).isDirectory()) {
+        p2.cancel(`Path does not exist or is not a directory: ${resolvedExisting}`);
+        process.exitCode = 1;
+        return;
+      }
+      if (!registryDirectoryExists(resolvedExisting)) {
+        const pathLabel = String(regPath).trim() || ".";
+        const bootstrapHere = await p2.confirm({
+          message: `${chalk2.yellow("No registry/ directory")} at ${chalk2.cyan(pathLabel)}. Bootstrap a minimal sample registry (registry.json + registry/<skill>) here?`,
+          initialValue: true
+        });
+        if (p2.isCancel(bootstrapHere)) {
+          p2.cancel("Aborted.");
+          process.exit(0);
+        }
+        if (!bootstrapHere) {
+          p2.cancel(
+            "Aborted. Create a registry/ folder in that directory, or run skissue init and choose Bootstrap a minimal sample registry."
+          );
+          process.exit(0);
+        }
+        const outcome = await runLocalBootstrapFlow(cwd, resolvedExisting);
+        if (!outcome.ok) {
+          if (outcome.exitCode === 1) {
+            process.exitCode = 1;
+          } else {
+            process.exit(0);
+          }
+          return;
+        }
+        cfg = outcome.cfg;
+        bootstrappedSkillId = outcome.bootstrappedSkillId;
+      } else {
+        const branchDefault = await gitBranchShowCurrent(resolvedExisting) ?? "main";
+        const branch = await p2.text({
+          message: "Branch label (stored in config; installs use git HEAD from that directory)",
+          initialValue: branchDefault,
+          validate: requiredTrimmed
+        });
+        if (p2.isCancel(branch)) {
+          p2.cancel("Aborted.");
+          process.exit(0);
+        }
+        cfg = ConfigSchema.parse({
+          registry: {
+            path: String(regPath).trim(),
+            branch: String(branch).trim()
+          },
+          skillsRoot: ".agents/skills"
+        });
       }
-      cfg = ConfigSchema.parse({
-        registry: {
-          path: String(regPath).trim(),
-          branch: String(branch).trim()
-        },
-        skillsRoot: ".agents/skills"
-      });
     } else {
       const pathRaw = await p2.text({
         message: "Path for the new registry root (relative to this project or absolute)",
         placeholder: "./skill-registry",
         initialValue: "./skill-registry",
-        validate: (v) => v?.trim() ? void 0 : "Required"
+        validate: requiredTrimmed
       });
       if (p2.isCancel(pathRaw)) {
         p2.cancel("Aborted.");
         process.exit(0);
       }
       const resolvedRoot = resolve3(cwd, String(pathRaw).trim());
-      const atConsumerRoot = resolvedRoot === resolve3(cwd);
-      if (atConsumerRoot && !registryLayoutExists(resolvedRoot)) {
-        const okRoot = await p2.confirm({
-          message: "This adds registry.json and registry/ at your project root. Continue?",
-          initialValue: false
-        });
-        if (p2.isCancel(okRoot) || !okRoot) {
-          p2.cancel("Aborted.");
+      const outcome = await runLocalBootstrapFlow(cwd, resolvedRoot);
+      if (!outcome.ok) {
+        if (outcome.exitCode === 1) {
+          process.exitCode = 1;
+        } else {
           process.exit(0);
         }
-      }
-      const prompted = await promptMinimalRegistryScaffold(resolvedRoot);
-      if (!prompted) {
-        p2.cancel("Aborted. No files were changed.");
-        process.exit(0);
-      }
-      let result;
-      try {
-        result = await scaffoldMinimalRegistry({
-          root: resolvedRoot,
-          skillId: prompted.skillId,
-          runGitInit: prompted.runGitInit
-        });
-      } catch (e) {
-        p2.cancel(e instanceof Error ? e.message : String(e));
-        process.exitCode = 1;
         return;
       }
-      bootstrappedSkillId = result.skillId;
-      if (!prompted.hadGit && !result.gitInitRan) {
-        p2.note(
-          "This folder is not a git repository. Run `git init` before using it as a local skissue registry.",
-          chalk2.yellow("Heads up")
-        );
-      }
-      const branchDefault = await gitBranchShowCurrent(resolvedRoot) ?? "main";
-      const branch = await p2.text({
-        message: "Branch label (stored in config; installs use git HEAD from that directory)",
-        initialValue: branchDefault,
-        validate: (v) => v?.trim() ? void 0 : "Required"
-      });
-      if (p2.isCancel(branch)) {
-        p2.cancel("Aborted.");
-        process.exit(0);
-      }
-      cfg = ConfigSchema.parse({
-        registry: {
-          path: localRegistryPathForConfig(cwd, resolvedRoot),
-          branch: String(branch).trim()
-        },
-        skillsRoot: ".agents/skills"
-      });
+      cfg = outcome.cfg;
+      bootstrappedSkillId = outcome.bootstrappedSkillId;
     }
   } else {
     const owner = await p2.text({
       message: "GitHub registry owner (org or user)",
       placeholder: "acme",
-      validate: (v) => v?.trim() ? void 0 : "Required"
+      validate: requiredTrimmed
     });
     if (p2.isCancel(owner)) {
       p2.cancel("Aborted.");
@@ -728,7 +796,7 @@ async function runInit(cwd) {
     const repo = await p2.text({
       message: "Registry repository name",
       placeholder: "skill-registry",
-      validate: (v) => v?.trim() ? void 0 : "Required"
+      validate: requiredTrimmed
     });
     if (p2.isCancel(repo)) {
       p2.cancel("Aborted.");
@@ -737,7 +805,7 @@ async function runInit(cwd) {
     const branch = await p2.text({
       message: "Branch to track",
       initialValue: "main",
-      validate: (v) => v?.trim() ? void 0 : "Required"
+      validate: requiredTrimmed
     });
     if (p2.isCancel(branch)) {
       p2.cancel("Aborted.");
@@ -755,7 +823,7 @@ async function runInit(cwd) {
   const skillsRoot = await p2.text({
     message: "Install skills under (relative to project root)",
     initialValue: cfg.skillsRoot,
-    validate: (v) => v?.trim() ? void 0 : "Required"
+    validate: requiredTrimmed
   });
   if (p2.isCancel(skillsRoot)) {
     p2.cancel("Aborted.");
@@ -1054,7 +1122,7 @@ async function runUpdate(cwd, skillId) {
 // src/commands/manage.ts
 import * as p3 from "@clack/prompts";
 import chalk9 from "chalk";
-import readline from "node:readline";
+import { emitKeypressEvents } from "node:readline";
 
 // src/commands/banner.ts
 import chalk8 from "chalk";
@@ -1156,7 +1224,7 @@ async function promptMainMenuSelect(availableCount, installedCount) {
   };
   const { stdin } = process;
   if (stdin.isTTY) {
-    readline.emitKeypressEvents(stdin);
+    emitKeypressEvents(stdin);
     stdin.prependListener("keypress", onCtrlC);
   }
   try {
@@ -1461,8 +1529,9 @@ async function runDoctor(cwd) {
     process.exitCode = 1;
     return;
   }
-  if (!isLocalRegistry(config)) {
-    console.log(chalk10.dim(`\u2022 ${authHint(config)}`));
+  const hint = authHint(config);
+  if (hint) {
+    console.log(chalk10.dim(`\u2022 ${hint}`));
   }
   try {
     const { path: repoPath, head } = await ensureRegistryCheckout(cwd, config);
@@ -1545,7 +1614,7 @@ program.command("update").description("Re-fetch and overwrite installed skill(s)
     process.exitCode = 1;
   }
 });
-program.command("doctor").description("Check Node, config, and sync registry checkout").option("-C, --cwd <path>", "Project root", process.cwd()).action(async (opts) => {
+program.command("doctor").description("Check Node, config, and sync registry checkout").option("-C, --cwd <path>", "Project root").action(async (opts) => {
   const cwd = resolve4(opts.cwd ?? process.cwd());
   try {
     await runDoctor(cwd);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skissue",
-  "version": "0.1.17",
+  "version": "0.1.19",
   "license": "MIT",
   "repository": {
     "type": "git",

package/scripts/ensure-local-bin.mjs

@@ -10,23 +10,29 @@ import { chmodSync, existsSync, mkdirSync, writeFileSync } from "node:fs";
 import { dirname, join, sep } from "node:path";
 import { fileURLToPath } from "node:url";
 
-const scriptPath = fileURLToPath(import.meta.url);
-if (scriptPath.split(sep).includes("node_modules")) {
-  process.exit(0);
-}
+/**
+ * @param {string} scriptPath Absolute path to this file (or a test fixture path with the same layout).
+ */
+export function runEnsureLocalBin(scriptPath) {
+  if (scriptPath.split(sep).includes("node_modules")) {
+    return;
+  }
 
-const root = dirname(dirname(scriptPath));
-const dist = join(root, "dist", "entry.js");
-const binDir = join(root, "node_modules", ".bin");
-const out = join(binDir, "skissue");
+  const root = dirname(dirname(scriptPath));
+  const dist = join(root, "dist", "entry.js");
+  const binDir = join(root, "node_modules", ".bin");
+  const out = join(binDir, "skissue");
 
-if (!existsSync(dist)) {
-  process.exit(0);
-}
+  if (!existsSync(dist)) {
+    return;
+  }
 
-mkdirSync(binDir, { recursive: true });
-const content = `#!/usr/bin/env sh
+  mkdirSync(binDir, { recursive: true });
+  const content = `#!/usr/bin/env sh
 exec node "${dist}" "$@"
 `;
-writeFileSync(out, content, { mode: 0o755 });
-chmodSync(out, 0o755);
+  writeFileSync(out, content, { mode: 0o755 });
+  chmodSync(out, 0o755);
+}
+
+runEnsureLocalBin(fileURLToPath(import.meta.url));