@tablation/crew 0.65.0 → 0.67.0

package/README.md

@@ -263,7 +263,7 @@ crew inbox [--member NAME]    your tickets across every workspace
 crew connect WS[/PROJECT]     resolve a workspace/project's ids into the state tree
 crew repos add [ROUTE] [NAME] [PATH]  attach a local checkout to a route in crew.yaml, creating its Repos row if missing (--dry-run previews)
 crew repos list ROUTE         the checkouts a route has, and the tracker Repos row each matches
-crew repos hooks [ROUTE] NAME  suggest test/build hooks for a checkout (--write route|repo writes them, --recipes, --agent, --json)
+crew repos hooks [ROUTE] NAME  suggest test/build hooks for a checkout (--write route|repo writes them, --recipes, --hook KEY=CMD, --agent, --json)
 crew install [--replace]     write and load this platform's scheduler unit (macOS: clears a crew installed from another path)
 crew uninstall [--all]        unload and remove it (--all: every crew launchd unit, whichever path it came from)
 ```

package/dist/cli.js

@@ -1449,6 +1449,7 @@ var HOOK_NAMES = [
   "ports",
   "isolate",
   "handoff",
+  "ci_status",
   "version",
   "bump",
   "pr",
@@ -2792,7 +2793,8 @@ var HOOK_PURPOSE = [
   ["ports", "this worktree's port assignments; prints KEY=value overrides to export"],
   ["handoff", "leave the operator able to open what you built; put its output in your progress comment"],
   ["test", "run the suite"],
-  ["build", "build it"]
+  ["build", "build it"],
+  ["ci_status", "after you push and before you hand off: exit 0 means CI is green for your branch; on failure fix and push again, and say in your progress comment if it is still pending"]
 ];
 function hookTable(repo) {
   const declared = HOOK_PURPOSE.filter(([name]) => repo.hooks[name]);
@@ -5198,6 +5200,7 @@ var VALUE_FLAGS = /* @__PURE__ */ new Set([
   "dir",
   "extra-path",
   "harness",
+  "hook",
   "key",
   "level",
   "limit",
@@ -13281,11 +13284,11 @@ function detectRecipes(f) {
     {
       id: "github-actions-ci-status",
       title: "Check CI status before hand-off",
-      description: "Ask GitHub whether the latest run on the branch passed before a ticket is handed to QA. The repo spec has no hook for this yet, so it is reported but cannot be written.",
+      description: "Ask GitHub whether the latest run on the branch passed before a ticket is handed to QA (`hooks.ci_status`).",
       evidence,
-      hooks: {},
-      check: `gh run list --branch "$CREW_BRANCH" --limit 1 --json conclusion -q '.[0].conclusion == "success"'`,
-      writable: false,
+      // `gh` prints the conclusion and exits 0 either way, so the test is on the text.
+      hooks: { ci_status: `test "$(gh run list --branch "$CREW_BRANCH" --limit 1 --json conclusion -q '.[0].conclusion')" = success` },
+      writable: true,
       defaultOn: false
     }
   ];
@@ -13318,6 +13321,21 @@ function selectRecipes(s, ids) {
   }
   return { hooks, ...release ? { release } : {} };
 }
+var HOOK_KEYS = ["test", "build", "version", "bump", "released", "ci_status"];
+function parseHookOverrides(specs) {
+  const out = {};
+  for (const spec of specs) {
+    const eq2 = spec.indexOf("=");
+    const key = eq2 < 0 ? spec : spec.slice(0, eq2);
+    const val = eq2 < 0 ? "" : spec.slice(eq2 + 1);
+    if (!HOOK_KEYS.includes(key)) {
+      throw new ReposError(`--hook: unknown key "${key}" (valid keys: ${HOOK_KEYS.join(", ")})`);
+    }
+    if (val.trim() === "") throw new ReposError(`--hook ${key}= needs a command (expected key=value)`);
+    out[key] = val;
+  }
+  return out;
+}
 function parseHooksArgs(args, routes) {
   const rest = [...args];
   let route;
@@ -13326,7 +13344,7 @@ function parseHooksArgs(args, routes) {
     if (routes.length === 1) route = routes[0];
     else throw new ReposError(routes.length === 0 ? "the config has no routes \u2014 run `crew connect` first" : `several routes are configured; name one: ${routes.join(", ")}`);
   }
-  if (rest.length !== 1) throw new ReposError("expected: crew repos hooks [route] <name> [--suggest] [--agent] [--write route|repo] [--recipes ID,ID] [--json]");
+  if (rest.length !== 1) throw new ReposError("expected: crew repos hooks [route] <name> [--suggest] [--agent] [--write route|repo] [--recipes ID,ID] [--hook KEY=CMD] [--json]");
   return { route, name: rest[0] };
 }
 function repoDirOf(text, base, route, name) {
@@ -13410,7 +13428,7 @@ var AgentSuggestError = class extends Error {
     this.reason = reason;
   }
 };
-var HOOK_KEYS = ["test", "build", "version", "bump", "released"];
+var HOOK_KEYS2 = ["test", "build", "version", "bump", "released"];
 var READ_ONLY_TOOLS = ["Read", "Grep", "Glob"];
 var WRITING_TOOLS = ["Bash", "Edit", "Write", "NotebookEdit", "WebFetch", "WebSearch", "Task", "Agent"];
 var DEFAULT_TIMEOUT_MS = 18e4;
@@ -13492,7 +13510,7 @@ function parseAgentHooks(text) {
   }
   const raw = obj && typeof obj === "object" && obj.hooks && typeof obj.hooks === "object" ? obj.hooks : {};
   const hooks = {};
-  for (const k of HOOK_KEYS) if (typeof raw[k] === "string" && raw[k].trim()) hooks[k] = raw[k].trim();
+  for (const k of HOOK_KEYS2) if (typeof raw[k] === "string" && raw[k].trim()) hooks[k] = raw[k].trim();
   const notes = Array.isArray(obj.notes) ? obj.notes.filter((n) => typeof n === "string") : [];
   return { hooks, notes };
 }
@@ -13567,7 +13585,7 @@ ${run2.err}` });
     throw new AgentSuggestError(reason, reason === "not_installed" ? `${bin} is not installed` : reason === "not_signed_in" ? `${o.spec.name} is not signed in` : (parsed.text || run2.err).trim().slice(0, 300) || `exited ${run2.code}`);
   }
   const { hooks, notes } = parseAgentHooks(parsed.text);
-  const differsFromSniff = HOOK_KEYS.filter((k) => hooks[k] !== void 0 && hooks[k] !== o.sniff.hooks[k]);
+  const differsFromSniff = HOOK_KEYS2.filter((k) => hooks[k] !== void 0 && hooks[k] !== o.sniff.hooks[k]);
   return { source: "agent", hooks, filesRead: parsed.filesRead, differsFromSniff, notes, toolchain: o.spec.name, model };
 }
 
@@ -17160,7 +17178,7 @@ ${changed2.map((c) => `  - ${c}
     case "repos": {
       const sub = positional[1];
       const usage2 = () => {
-        process.stderr.write("crew repos add [workspace/project] [name] [path] [--name N] [--project P] [--dry-run] [--no-create]\ncrew repos clone [workspace/project] <name> [--dir PATH] [--dry-run] [--json]\ncrew repos list <workspace/project> [--json]\ncrew repos hooks [workspace/project] <name> [--suggest] [--agent] [--write route|repo] [--recipes ID,ID] [--json]\n");
+        process.stderr.write("crew repos add [workspace/project] [name] [path] [--name N] [--project P] [--dry-run] [--no-create]\ncrew repos clone [workspace/project] <name> [--dir PATH] [--dry-run] [--json]\ncrew repos list <workspace/project> [--json]\ncrew repos hooks [workspace/project] <name> [--suggest] [--agent] [--write route|repo] [--recipes ID,ID] [--hook KEY=CMD ...] [--json]\n");
         process.exit(2);
       };
       if (sub !== "add" && sub !== "list" && sub !== "clone" && sub !== "hooks") usage2();
@@ -17207,9 +17225,13 @@ ${changed2.map((c) => `  - ${c}
               sigintOwnedByCommand = false;
             }
           }
-          const picked = selectRecipes(suggestion, recipeIds);
+          const hookSpecs = argv.flatMap((x, i) => x === "--hook" && argv[i + 1] !== void 0 ? [argv[i + 1]] : x.startsWith("--hook=") ? [x.slice(7)] : []);
+          const overrides = parseHookOverrides(hookSpecs);
+          if (hookSpecs.length > 0 && !write) throw new ReposError("--hook only applies with --write route|repo");
+          const base2 = selectRecipes(suggestion, recipeIds);
+          const picked = { ...base2, hooks: { ...base2.hooks, ...overrides } };
           let written;
-          if (write && suggestion.source === null && recipeIds.length === 0) throw new ReposError(`nothing to write: ${suggestion.notes[0]}`);
+          if (write && suggestion.source === null && recipeIds.length === 0 && hookSpecs.length === 0) throw new ReposError(`nothing to write: ${suggestion.notes[0]}`);
           if (write === "route") {
             const plan = planRouteHooks(configText, a.route, a.name, picked.hooks);
             writeFileSync19(cfg.configFile, plan.text);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tablation/crew",
-  "version": "0.65.0",
+  "version": "0.67.0",
   "description": "A standing team of headless agents that picks work off a Tablation board, does it on a machine you control, and reports back.",
   "type": "module",
   "license": "MIT",