skissue 0.1.17 → 0.1.18

package/dist/entry.js

@@ -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,9 @@ function validateSkillId(raw) {
   if (id === "." || id === ".." || id.includes("..")) return "Invalid skill id";
   return void 0;
 }
+function validateSkillIdPrompt(v) {
+  return validateSkillId(String(v ?? ""));
+}
 function registryLayoutExists(root) {
   const jsonPath = join3(root, "registry.json");
   const regDir = join3(root, "registry");
@@ -440,7 +450,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 +504,7 @@ async function runInitRegistry(cwd) {
   if (p.isCancel(where)) {
     p.cancel("Aborted.");
     process.exit(0);
+    return;
   }
   let root;
   if (where === "here") {
@@ -502,11 +513,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 +526,7 @@ async function runInitRegistry(cwd) {
   if (!prompted) {
     p.cancel("Aborted. No files were changed.");
     process.exit(0);
+    return;
   }
   const { skillId, runGitInit, hadGit } = prompted;
   try {
@@ -626,7 +639,7 @@ 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.");
@@ -637,7 +650,7 @@ async function runInit(cwd) {
       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"
+        validate: requiredTrimmed
       });
       if (p2.isCancel(branch)) {
         p2.cancel("Aborted.");
@@ -655,7 +668,7 @@ async function runInit(cwd) {
         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.");
@@ -701,7 +714,7 @@ async function runInit(cwd) {
       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"
+        validate: requiredTrimmed
       });
       if (p2.isCancel(branch)) {
         p2.cancel("Aborted.");
@@ -719,7 +732,7 @@ async function runInit(cwd) {
     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 +741,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 +750,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 +768,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 +1067,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 +1169,7 @@ async function promptMainMenuSelect(availableCount, installedCount) {
   };
   const { stdin } = process;
   if (stdin.isTTY) {
-    readline.emitKeypressEvents(stdin);
+    emitKeypressEvents(stdin);
     stdin.prependListener("keypress", onCtrlC);
   }
   try {
@@ -1461,8 +1474,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 +1559,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.18",
   "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));