@makerx/yoke 0.1.0 → 0.3.0

package/README.md

@@ -4,7 +4,6 @@ Claude Code skills for MakerX AI-assisted delivery, and the CLI they call.
 
 ## Principles
 
-- **User-invoked.** Skills run only when you type their slash command.
 - **Overridable.** A skill of your own with the same name wins.
 - **Lean.** Few skills, short prompts; the CLI enforces the rules.
 

package/dist/cli.js

@@ -7,7 +7,7 @@ import { Command } from "commander";
 // package.json
 var package_default = {
   name: "@makerx/yoke",
-  version: "0.1.0",
+  version: "0.3.0",
   description: "Lean, user-invoked Claude Code skills for MakerX AI-assisted delivery, plus the CLI they call.",
   license: "MIT",
   author: "MakerX",
@@ -58,6 +58,76 @@ var package_default = {
   }
 };
 
+// src/features/pr/guide.ts
+var raiseGuide = `
+Compose a short PR in the format below, get the user's approval in the terminal, push, then create it as a draft with \`yoke pr raise\`, which rejects a title or body that breaks the rules.
+
+Preconditions
+
+- Working tree clean, every change committed. If not, stop and say what is uncommitted.
+- Branch is not the default branch.
+- \`gh pr view\` reports no PR for the branch. If one exists, do not create another: propose edits to the existing title and body and apply them with \`gh pr edit\` after the same approval step.
+
+Body
+
+Use exactly these sections, in this order, with these headings. Never add a section and never rename one: no \`## Summary\`, no \`## Changes\`, no \`## Test plan\`.
+
+Closes #<issue number>
+
+## What
+
+<2\u20133 sentences>
+
+## Why
+
+<1\u20132 sentences>
+
+Then, only if earned: \`## How\` (1\u20132 sentences).
+
+Closes line. If the session or branch name gives an issue number, make it the first line, written \`Closes #123\` for a same-repo issue or \`Closes owner/repo#123\` for a cross-repo one. One \`Closes\` per issue. Nothing else goes above \`## What\`.
+
+One section, one question.
+
+  What  what is observably different for a user or caller. Never function, file or component names, or how it is built.
+  Why   the problem that made the change worth doing. Never mechanism.
+  How   a non-obvious decision the diff cannot explain: a trade-off, a workaround, a rejected obvious approach. Never a walkthrough of the diff or a restatement of What.
+
+Litmus for mechanism: a sentence with "by \u2026ing", "so that it can", "because the X uses", or the name of an internal component, function or file belongs in How, and probably does not belong at all.
+
+Brevity budget. Hard caps enforced by \`yoke pr raise\`: What 300 characters, Why 200, How 250, and 600 across all sections. Ceilings, not targets. Prefer deleting a sentence to shortening it.
+
+No per-file lists. Never list changes file by file; the diff already shows that.
+
+Register. Write like a commit message. Short declarative sentences, present tense, active voice. Cut "this change", "in order to", "it is worth noting", hedges and adverbs.
+
+  Wordy: This change introduces a confirmation step so that users are able to cancel before their results are reset.
+  Terse: Retake asks for confirmation before resetting results.
+
+Prose, not bullets. Bullets only for 3 or more genuinely parallel items, and never in What.
+
+Backticks around symbols, commands and flags.
+
+The title and body never mention Claude, AI assistance or attribution. No \`Co-Authored-By\` or "Generated with" anywhere.
+
+Title
+
+One line, at most 70 characters, conventional-commit prefix as the repo uses (\`fix(scope): \u2026\`, \`feat(scope): \u2026\`, \`chore: \u2026\`). Says what changed for the user, not what files moved.
+
+Approval
+
+Mandatory in every permission mode, including auto-accept and bypass. Write the full title and body verbatim in your visible reply. Then call \`AskUserQuestion\` with an approve option whose \`preview\` holds the complete title and body, plus a revise option. Approval covers the push as well as the create. On revise, redraft and ask again.
+
+Create
+
+git push -u origin HEAD
+yoke pr raise --title "<title>" --body "$(cat <<'EOF'
+<body>
+EOF
+)"
+
+Always a draft; the user marks it ready for review themselves. If \`yoke pr raise\` rejects the description, it names each rule that failed: fix those, show the revised description, and get approval again before rerunning. On success it prints the PR URL; report it.
+`;
+
 // src/features/pr/raise.ts
 import { execFileSync } from "child_process";
 
@@ -189,102 +259,17 @@ import { join } from "path";
 // src/skill.ts
 var ownerMarker = 'owner: "@makerx/yoke"';
 function renderSkill({ name, description, body }) {
-  return [
-    "---",
-    `name: ${name}`,
-    `description: ${description}`,
-    "disable-model-invocation: true",
-    "metadata:",
-    `  ${ownerMarker}`,
-    "---",
-    "",
-    body.trim(),
-    ""
-  ].join("\n");
+  return ["---", `name: ${name}`, `description: ${description}`, "metadata:", `  ${ownerMarker}`, "---", "", body.trim(), ""].join("\n");
 }
 
 // src/features/pr/skill.ts
 var prSkill = {
   name: "pr",
-  description: "Raise a PR with a concise What/Why/(How) description",
+  description: "Raise a PR with a concise What/Why/(How) description. Use when asked to raise, open or create a pull request, or to rewrite a PR description.",
   body: `
-# Raise a PR
-
-Compose a short PR in the format below, get the user's approval in the terminal, push, then create it as a draft with \`yoke pr raise\`, which rejects a title or body that breaks the rules.
-
-## Preconditions
-
-- \`yoke\` is on PATH. If not, stop and tell the user to run \`npm i -g @makerx/yoke\`.
-- Working tree clean, every change committed. If not, stop and say what is uncommitted.
-- Branch is not the default branch.
-- \`gh pr view\` reports no PR for the branch. If one exists, do not create another: propose edits to the existing title and body and apply them with \`gh pr edit\` after the same approval step.
-
-## Body
-
-Use exactly these sections, in this order, with these headings. Never add a section and never rename one: no \`## Summary\`, no \`## Changes\`, no \`## Test plan\`.
-
-\`\`\`
-Closes #<issue number>
-
-## What
-
-<2\u20133 sentences>
-
-## Why
-
-<1\u20132 sentences>
-\`\`\`
-
-Then, only if earned: \`## How\` (1\u20132 sentences).
-
-**Closes line.** If the session or branch name gives an issue number, make it the first line, written \`Closes #123\` for a same-repo issue or \`Closes owner/repo#123\` for a cross-repo one. One \`Closes\` per issue. Nothing else goes above \`## What\`.
-
-**One section, one question.**
-
-| Section | Answers | Never contains |
-|---|---|---|
-| What | What is observably different for a user or caller | Function, file or component names; how it is built |
-| Why | The problem that made the change worth doing | Mechanism |
-| How | A non-obvious decision the diff cannot explain: a trade-off, a workaround, a rejected obvious approach | A walkthrough of the diff; a restatement of What |
+Run \`yoke pr raise --help\` and follow it exactly. Do not use \`gh pr create\`.
 
-Litmus for mechanism: a sentence with "by \u2026ing", "so that it can", "because the X uses", or the name of an internal component, function or file belongs in How, and probably does not belong at all.
-
-**Brevity budget.** Hard caps enforced by \`yoke pr raise\`: What 300 characters, Why 200, How 250, and 600 across all sections. Ceilings, not targets. Prefer deleting a sentence to shortening it.
-
-**No per-file lists.** Never list changes file by file; the diff already shows that.
-
-**Register.** Write like a commit message. Short declarative sentences, present tense, active voice. Cut "this change", "in order to", "it is worth noting", hedges and adverbs.
-
-\`\`\`
-Wordy: This change introduces a confirmation step so that users are able to cancel before their results are reset.
-Terse: Retake asks for confirmation before resetting results.
-\`\`\`
-
-Prose, not bullets. Bullets only for 3 or more genuinely parallel items, and never in What.
-
-Backticks around symbols, commands and flags.
-
-The title and body never mention Claude, AI assistance or attribution. No \`Co-Authored-By\` or "Generated with" anywhere.
-
-## Title
-
-One line, at most 70 characters, conventional-commit prefix as the repo uses (\`fix(scope): \u2026\`, \`feat(scope): \u2026\`, \`chore: \u2026\`). Says what changed for the user, not what files moved.
-
-## Approval
-
-Mandatory in every permission mode, including auto-accept and bypass. Write the full title and body verbatim in your visible reply. Then call \`AskUserQuestion\` with an approve option whose \`preview\` holds the complete title and body, plus a revise option. Approval covers the push as well as the create. On revise, redraft and ask again.
-
-## Create
-
-\`\`\`bash
-git push -u origin HEAD
-yoke pr raise --title "<title>" --body "$(cat <<'EOF'
-<body>
-EOF
-)"
-\`\`\`
-
-Always a draft; the user marks it ready for review themselves. If \`yoke pr raise\` rejects the description, it names each rule that failed: fix those, show the revised description, and get approval again before rerunning. On success it prints the PR URL; report it.
+If \`yoke\` is not on PATH, stop and tell the user to run \`npm i -g @makerx/yoke\`.
 `
 };
 
@@ -314,7 +299,7 @@ function syncSkills(destination, skills = bundledSkills) {
 
 // src/cli.ts
 var { version } = package_default;
-var program = new Command("yoke").description("Lean, user-invoked Claude Code skills and the CLI they call.").version(version);
+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").action((options) => {
   const user = options.user ?? false;
   const display = (path) => user ? path : relative(process.cwd(), path);
@@ -323,7 +308,7 @@ program.command("sync").description("Write yoke's skills into this repo's .claud
   for (const dir of skipped) console.log(`skipped ${display(dir)} (your own skill; delete it to take yoke's)`);
 });
 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").action((options) => {
+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 {
     console.log(raisePr(options));
   } catch (error) {

package/package.json

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