@makerx/yoke 0.14.0 → 0.16.0

package/README.md

@@ -30,6 +30,7 @@ yoke sync --user   # ~/.claude/skills
 
 ```bash
 yoke sync [--user] [--force]  # write yoke's skills
+yoke update                   # update yoke and re-sync its skills where they were synced
 yoke review [number]          # review a PR (used by /review)
 yoke review-ci                # add the CI review workflow (used by /review-ci)
 yoke review-ci-gate           # require the review / gate check on the default branch (used by /review-ci-gate)

package/dist/cli.js

@@ -1,13 +1,13 @@
 #!/usr/bin/env node
 
 // src/cli.ts
-import { relative } from "path";
+import { relative as relative2 } from "path";
 import { Command } from "commander";
 
 // package.json
 var package_default = {
   name: "@makerx/yoke",
-  version: "0.14.0",
+  version: "0.16.0",
   description: "Lean, user-invoked Claude Code skills for MakerX AI-assisted delivery, plus the CLI they call.",
   license: "MIT",
   author: "MakerX",
@@ -486,9 +486,9 @@ var ReviewError = class extends Error {
   }
 };
 var maxBuffer = 256 * 1024 * 1024;
-var run = (command, args, cwd, input) => new Promise((resolve, reject) => {
+var run = (command, args, cwd, input) => new Promise((resolve3, reject) => {
   const child = execFile(command, args, { cwd, encoding: "utf8", maxBuffer }, (error, stdout, stderr) => {
-    if (!error) return resolve(stdout);
+    if (!error) return resolve3(stdout);
     const detail = stderr.trim();
     reject(new ReviewError(`\`${command} ${args.join(" ")}\` failed${detail ? `: ${detail}` : ""}`));
   });
@@ -1904,7 +1904,7 @@ function exitError(harness, code, output) {
 function hasReview(path) {
   return existsSync3(path) && readFileSync4(path, "utf8").trim().length > 0;
 }
-var runHarness = (run2) => new Promise((resolve) => {
+var runHarness = (run2) => new Promise((resolve3) => {
   const shell = process.platform === "win32";
   const args = harnessArgs(run2);
   const child = spawn(run2.harness, shell ? args.map(windowsQuote) : args, {
@@ -1941,14 +1941,14 @@ var runHarness = (run2) => new Promise((resolve) => {
   child.stdin.on("error", () => {
   });
   child.stdin.end(run2.prompt);
-  child.on("error", (error) => resolve({ ok: false, error: spawnError(run2.harness, error) }));
+  child.on("error", (error) => resolve3({ ok: false, error: spawnError(run2.harness, error) }));
   child.on("close", (code) => {
     onLine(pending);
     if (code !== 0)
-      return resolve({ ok: false, error: exitError(run2.harness, code, [stderr || finalText, lastError].filter(Boolean).join("\n")) });
+      return resolve3({ ok: false, error: exitError(run2.harness, code, [stderr || finalText, lastError].filter(Boolean).join("\n")) });
     if (run2.harness === "claude") writeFileSync3(run2.outputPath, finalText);
-    if (!hasReview(run2.outputPath)) return resolve({ ok: false, error: `${run2.harness} returned an empty review` });
-    resolve({ ok: true, ...model ? { model } : {}, ...usage ? { usage } : {} });
+    if (!hasReview(run2.outputPath)) return resolve3({ ok: false, error: `${run2.harness} returned an empty review` });
+    resolve3({ ok: true, ...model ? { model } : {}, ...usage ? { usage } : {} });
   });
 });
 
@@ -2288,9 +2288,9 @@ var commandTimeout = 3e4;
 var probeTimeout = 6e4;
 var maxBodyChars2 = 300;
 function exec(harness, args) {
-  return new Promise((resolve) => {
+  return new Promise((resolve3) => {
     const options = { encoding: "utf8", timeout: commandTimeout, shell: process.platform === "win32" };
-    execFile2(harness, args, options, (error, stdout) => resolve({ ok: !error, stdout }));
+    execFile2(harness, args, options, (error, stdout) => resolve3({ ok: !error, stdout }));
   });
 }
 async function claudeSignedIn() {
@@ -2621,17 +2621,137 @@ function syncSkills(destination, userDestination, force = false, skills = bundle
   return result;
 }
 
+// src/features/update/install.ts
+import { execSync } from "child_process";
+import { dirname as dirname3, isAbsolute, relative, resolve } from "path";
+import { fileURLToPath } from "url";
+function installDir(moduleUrl = import.meta.url) {
+  return resolve(dirname3(fileURLToPath(moduleUrl)), "..");
+}
+function npmGlobalRoot() {
+  return execSync("npm root -g", { stdio: "pipe" }).toString().trim();
+}
+function gitToplevel(dir) {
+  return execSync("git rev-parse --show-toplevel", { cwd: dir, stdio: "pipe" }).toString().trim();
+}
+function isGitClone(dir, toplevel = gitToplevel) {
+  try {
+    return resolve(toplevel(dir)) === resolve(dir);
+  } catch {
+    return false;
+  }
+}
+function isInside(parent, child, platform = process.platform) {
+  const fold = (path) => platform === "win32" ? resolve(path).toLowerCase() : resolve(path);
+  const rel = relative(fold(parent), fold(child));
+  return rel !== "" && !rel.startsWith("..") && !isAbsolute(rel);
+}
+function isGlobalNpmInstall(dir, globalRoot = npmGlobalRoot, platform = process.platform) {
+  try {
+    return isInside(globalRoot(), dir, platform);
+  } catch {
+    return false;
+  }
+}
+function detectInstall(dir, globalRoot = npmGlobalRoot, toplevel = gitToplevel) {
+  if (isGitClone(dir, toplevel)) return "git";
+  return isGlobalNpmInstall(dir, globalRoot) ? "npm" : "unknown";
+}
+
+// src/features/update/targets.ts
+import { existsSync as existsSync9, readFileSync as readFileSync10 } from "fs";
+import { homedir as homedir4 } from "os";
+import { join as join10, resolve as resolve2 } from "path";
+function holdsYokeSkill(root, skills) {
+  return skills.some((skill) => {
+    if (skill.userLevel) return false;
+    const file = join10(root, skill.name, "SKILL.md");
+    return existsSync9(file) && readFileSync10(file, "utf8").includes(ownerMarker);
+  });
+}
+function syncTargets(cwd = process.cwd(), home = homedir4(), skills = bundledSkills) {
+  const userRoot = skillsRoot(true, cwd, home);
+  const repoRoot2 = skillsRoot(false, cwd, home);
+  const targets = [];
+  if (holdsYokeSkill(userRoot, skills)) targets.push("user");
+  if (resolve2(repoRoot2) !== resolve2(userRoot) && holdsYokeSkill(repoRoot2, skills)) targets.push("repo");
+  return targets;
+}
+
+// src/features/update/update.ts
+import { execSync as execSync2 } from "child_process";
+var packageName = "@makerx/yoke";
+var syncCommands = { user: "yoke sync --user", repo: "yoke sync" };
+var noTargetsMessage = "No yoke skills were synced here before, so none were re-synced. Run `yoke sync` for this repo's .claude/skills, or `yoke sync --user` for ~/.claude/skills.";
+function runInherited(command, cwd) {
+  execSync2(command, { cwd, stdio: "inherit" });
+}
+function installLatest(install, dir, run2, log) {
+  if (install === "npm") {
+    log("Updating the global npm install...");
+    run2(`npm i -g ${packageName}@latest`);
+    return;
+  }
+  log("Pulling the latest yoke...");
+  run2("git pull", dir);
+  log("Installing dependencies...");
+  run2("npm ci", dir);
+  log("Building...");
+  run2("npm run build", dir);
+}
+function updateYoke({ dir, install, targets, run: run2 = runInherited, log, error }) {
+  log(`yoke is installed at ${dir}`);
+  if (install === "unknown") {
+    error(
+      `Could not tell how yoke was installed at ${dir}. Expected a git clone or a global npm install; reinstall with \`npm i -g ${packageName}@latest\`.`
+    );
+    return 1;
+  }
+  try {
+    installLatest(install, dir, run2, log);
+  } catch {
+    error("Updating yoke failed, so no skills were re-synced.");
+    return 1;
+  }
+  const found = targets();
+  if (found.length === 0) {
+    log(noTargetsMessage);
+    return 0;
+  }
+  for (const target of found) {
+    const command = syncCommands[target];
+    log(`Running ${command}...`);
+    try {
+      run2(command);
+    } catch {
+      error(`${command} failed.`);
+      return 1;
+    }
+  }
+  return 0;
+}
+
 // src/cli.ts
 var { version } = package_default;
 var program = new Command("yoke").description("Lean Claude Code skills and the CLI they call.").version(version);
 program.command("sync").description("Write yoke's skills into this repo's .claude/skills, or ~/.claude/skills with --user").option("--user", "write to ~/.claude/skills instead, for every repo").option("--force", "overwrite same-named skills that yoke didn't write").action((options) => {
   const user = options.user ?? false;
   const cwd = process.cwd();
-  const display = (path) => path.startsWith(cwd) ? relative(cwd, path) : path;
+  const display = (path) => path.startsWith(cwd) ? relative2(cwd, path) : path;
   const { written, skipped } = syncSkills(skillsRoot(user), skillsRoot(true), options.force ?? false);
   for (const file of written) console.log(`wrote ${display(file)}`);
   for (const dir of skipped) console.log(`skipped ${display(dir)} (not yoke's skill; rerun with --force to overwrite it)`);
 });
+program.command("update").description("Update yoke to the latest version, then re-sync its skills wherever they were synced before").action(() => {
+  const dir = installDir();
+  process.exitCode = updateYoke({
+    dir,
+    install: detectInstall(dir),
+    targets: () => syncTargets(),
+    log: console.log,
+    error: console.error
+  });
+});
 var pr = program.command("pr").description("Pull request commands");
 pr.command("raise").description("Validate a PR title and body, then create the PR with gh").requiredOption("--title <title>", "single-line PR title").requiredOption("--body <body>", "PR body with ## What, ## Why and optionally ## How").option("--base <branch>", "base branch (defaults to the repo default)").option("--no-draft", "create a ready-for-review PR instead of a draft").addHelpText("after", raiseGuide).action((options) => {
   try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@makerx/yoke",
-  "version": "0.14.0",
+  "version": "0.16.0",
   "description": "Lean, user-invoked Claude Code skills for MakerX AI-assisted delivery, plus the CLI they call.",
   "license": "MIT",
   "author": "MakerX",