@tablation/crew 0.59.0 → 0.60.0

package/README.md

@@ -255,6 +255,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, --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

@@ -5211,7 +5211,8 @@ var VALUE_FLAGS = /* @__PURE__ */ new Set([
   "tail",
   "ticket",
   "workspace",
-  "workspace-id"
+  "workspace-id",
+  "write"
 ]);
 function positionalArgs(argv2) {
   const out = [];
@@ -12659,77 +12660,12 @@ async function showLogEntry(route, entryId, opts = {}) {
   };
 }
 
-// src/sample-ticket.ts
-import { basename as basename7, dirname as dirname12, join as join20 } from "path";
-var SAMPLE_REPO = "crew-sample";
-var SAMPLE_REMOTE = `local/${SAMPLE_REPO}`;
-var SAMPLE_MARKER = "sample: true";
-var SAMPLE_CREW_YAML = `version: 1
-platform: unix
-branch:
-  base: main
-hooks:
-  test: "true"
-  build: "true"
-release:
-  mode: integrate
-  versioning: none
-  changelog: false
-`;
-var SAMPLE_README = `# crew-sample
-
-A throwaway repository created by \`crew sample-ticket\`. It exists so the crew
-has something safe to work on: its test and build hooks always pass, it has no
-remote, and \`crew sample-ticket --remove\` deletes it.
-`;
-function sampleDir(existing, reposBase, workspace) {
-  if (existing.length > 0) return join20(dirname12(existing[0]), SAMPLE_REPO);
-  return join20(reposBase, workspace, SAMPLE_REPO);
-}
-var SAMPLE_TITLE = "Sample: add a documented `greet` function with a test";
-function sampleDescription() {
-  return [
-    SAMPLE_MARKER,
-    "",
-    "> This is a sample ticket filed by `crew sample-ticket` on a throwaway repository. Nothing here is real work.",
-    "",
-    "## What",
-    "",
-    "Add `greet.js` exporting `greet(name)`, which returns `Hello, <name>!` (and `Hello, world!` for no name),",
-    "with a JSDoc comment, and `greet.test.js` covering both cases using `node:test`.",
-    "",
-    "## Acceptance",
-    "",
-    '- `greet("Ada")` returns `Hello, Ada!`; `greet()` returns `Hello, world!`.',
-    "- The test file runs with `node --test` and passes.",
-    "- Add a line to the README saying the function exists.",
-    "",
-    "## Notes",
-    "",
-    "The repo has no remote: leave the branch local. The hooks in `.crew.yaml` always pass, so QA is checking the",
-    "diff against the acceptance list above."
-  ].join("\n");
-}
-function isSampleTicket(description) {
-  return (description ?? "").trimStart().startsWith(SAMPLE_MARKER);
-}
-function planSampleTicket(i) {
-  return {
-    title: SAMPLE_TITLE,
-    description: sampleDescription(),
-    report_type: "feature",
-    severity: "s4",
-    reporter_name: "crew sample-ticket",
-    status: i.approvedStatus,
-    project_id: i.projectId,
-    repo_id: i.repoId,
-    ...i.filedBy ? { filed_by_id: i.filedBy } : {}
-  };
-}
+// src/repos-hooks.ts
+import { isMap as isMap2, parseDocument as parseDocument2, YAMLMap } from "yaml";
 
 // src/repos-cmd.ts
 import { homedir as homedir7 } from "os";
-import { basename as basename8, join as join21, resolve as resolve3 } from "path";
+import { basename as basename7, join as join20, resolve as resolve3 } from "path";
 import { isMap, isSeq, parseDocument, stringify as stringify2 } from "yaml";
 var ReposError = class extends Error {
 };
@@ -12801,7 +12737,7 @@ function configuredRepos(text, route, base) {
     });
   }
   const dir = m.get("dir");
-  return dir ? [{ name: basename8(String(dir)), dir: expandDir(String(dir), base) }] : [];
+  return dir ? [{ name: basename7(String(dir)), dir: expandDir(String(dir), base) }] : [];
 }
 function planRepoAdd(i) {
   const doc = parseDocument(i.text);
@@ -12813,7 +12749,7 @@ function planRepoAdd(i) {
     throw new ReposError(`${target} is already configured on route ${i.route}`);
   }
   const matched = matchRepoName(i.origin, i.resolved);
-  const name = matched ?? i.name ?? normalizeRemote(i.origin)?.split("/").pop() ?? basename8(target);
+  const name = matched ?? i.name ?? normalizeRemote(i.origin)?.split("/").pop() ?? basename7(target);
   let warning;
   if (!matched) {
     const known = Object.values(i.resolved?.repoNames ?? {});
@@ -12927,13 +12863,218 @@ function planRepoClone(i) {
   }
   const ship = doc.get("ship", true);
   const basePath = String(m.get("reposBasePath") ?? (isMap(ship) ? ship.get("reposBasePath") : void 0) ?? "~/Crew");
-  const target = i.dir ? expandDir(i.dir, i.base) : join21(expandDir(basePath, i.base), i.route.split("/")[0], name);
+  const target = i.dir ? expandDir(i.dir, i.base) : join20(expandDir(basePath, i.base), i.route.split("/")[0], name);
   const add = planRepoAdd({ text: i.text, base: i.base, route: i.route, path: target, origin: remote, resolved: i.resolved, name });
   return { ...add, remote, target };
 }
 
+// src/repos-hooks.ts
+function packageManager(f) {
+  if (f.exists("pnpm-lock.yaml")) return "pnpm";
+  if (f.exists("yarn.lock")) return "yarn";
+  if (f.exists("bun.lock") || f.exists("bun.lockb")) return "bun";
+  return "npm";
+}
+function fromPackageJson(f) {
+  const text = f.read("package.json");
+  if (text === void 0) return void 0;
+  let pkg;
+  try {
+    pkg = JSON.parse(text);
+  } catch {
+    return void 0;
+  }
+  if (!pkg || typeof pkg !== "object") return void 0;
+  const pm = packageManager(f);
+  const scripts = pkg.scripts && typeof pkg.scripts === "object" ? pkg.scripts : {};
+  const out = {};
+  if (typeof scripts.test === "string") out.test = `${pm} test`;
+  if (typeof scripts.build === "string") out.build = pm === "yarn" ? "yarn build" : `${pm} run build`;
+  if (typeof pkg.version === "string") {
+    out.version = `node -p "require('./package.json').version"`;
+    out.bump = `npm version "$CREW_BUMP" --no-git-tag-version | sed 's/^v//'`;
+  }
+  return out;
+}
+function fromPackageSwift(f) {
+  return f.exists("Package.swift") ? { test: "swift test", build: "swift build" } : void 0;
+}
+function fromPyproject(f) {
+  return f.exists("pyproject.toml") ? { test: "pytest" } : void 0;
+}
+function fromMakefile(f) {
+  const text = f.read("Makefile") ?? f.read("makefile") ?? f.read("GNUmakefile");
+  if (text === void 0) return void 0;
+  const out = {};
+  for (const target of ["test", "build"]) {
+    if (new RegExp(`^${target}\\s*:`, "m").test(text)) out[target] = `make ${target}`;
+  }
+  return out;
+}
+var DETECTORS = [
+  { source: "package.json", detect: fromPackageJson },
+  { source: "Package.swift", detect: fromPackageSwift },
+  { source: "pyproject.toml", detect: fromPyproject },
+  { source: "Makefile", detect: fromMakefile }
+];
+function suggestHooks(f) {
+  const found = DETECTORS.flatMap((d) => {
+    const hooks = d.detect(f);
+    return hooks && Object.keys(hooks).length > 0 ? [{ source: d.source, hooks }] : [];
+  });
+  if (found.length === 0) {
+    return { source: null, hooks: {}, notes: ["no package.json scripts, Package.swift, pyproject.toml or Makefile test/build targets found \u2014 nothing to propose"] };
+  }
+  const [first, ...others] = found;
+  const notes = others.length > 0 ? [`also found ${others.map((o) => o.source).join(", ")}; proposing from ${first.source} only`] : [];
+  return { source: first.source, hooks: first.hooks, notes };
+}
+function parseHooksArgs(args, routes) {
+  const rest = [...args];
+  let route;
+  if (rest.length > 1 && routes.includes(rest[0])) route = rest.shift();
+  if (!route) {
+    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] [--write route|repo] [--json]");
+  return { route, name: rest[0] };
+}
+function repoDirOf(text, base, route, name) {
+  const repos = configuredRepos(text, route, base);
+  const hit = repos.find((r) => r.name === name);
+  if (!hit) {
+    throw new ReposError(`route ${route} has no repo named "${name}" (have: ${repos.map((r) => r.name).join(", ") || "none"})`);
+  }
+  return hit.dir;
+}
+function planRouteHooks(text, route, name, hooks) {
+  const doc = parseDocument2(text);
+  if (doc.errors.length > 0) throw new ReposError(`config is not valid YAML \u2014 ${doc.errors[0].message}`);
+  const m = routeMapOf(doc, route);
+  const repos = m.get("repos", true);
+  let target = m;
+  if (isMap2(repos)) {
+    const entry = repos.get(name, true);
+    if (isMap2(entry)) target = entry;
+    else {
+      const created = new YAMLMap();
+      created.set("dir", String(repos.get(name)));
+      repos.set(name, created);
+      target = created;
+    }
+  }
+  const current = target.get("hooks", true);
+  let hooksMap;
+  if (isMap2(current)) hooksMap = current;
+  else {
+    hooksMap = new YAMLMap();
+    target.set("hooks", hooksMap);
+  }
+  const written = [];
+  const kept = [];
+  for (const [k, v] of Object.entries(hooks)) {
+    if (v === void 0) continue;
+    if (hooksMap.has(k)) kept.push(k);
+    else {
+      hooksMap.set(k, v);
+      written.push(k);
+    }
+  }
+  return { text: String(doc), written, kept };
+}
+function starterRepoConfig(hooks) {
+  const lines = [
+    "# Starter .crew.yaml proposed by `crew repos hooks --write repo`.",
+    "# Review the hooks, then commit this file; it is the durable home for them",
+    "# (docs/REPO_SPEC.md). A route-level copy in crew.yaml is shadowed by it.",
+    `version: ${SPEC_VERSION}`,
+    "platform: any",
+    "",
+    "hooks:"
+  ];
+  const entries = Object.entries(hooks).filter(([, v]) => v !== void 0);
+  if (entries.length === 0) lines.push('  # test: "..."', '  # build: "..."');
+  for (const [k, v] of entries) lines.push(`  ${k}: ${JSON.stringify(v)}`);
+  lines.push("", "release:", "  # integrate: merging to the base branch is the release; nothing is deployed.", "  mode: integrate", "");
+  const text = lines.join("\n");
+  parseRepoConfig(text, ".crew.yaml");
+  return text;
+}
+function existingRepoConfig(f) {
+  return REPO_CONFIG_FILENAMES.find((n) => f.exists(n));
+}
+
+// src/sample-ticket.ts
+import { basename as basename8, dirname as dirname13, join as join21 } from "path";
+var SAMPLE_REPO = "crew-sample";
+var SAMPLE_REMOTE = `local/${SAMPLE_REPO}`;
+var SAMPLE_MARKER = "sample: true";
+var SAMPLE_CREW_YAML = `version: 1
+platform: unix
+branch:
+  base: main
+hooks:
+  test: "true"
+  build: "true"
+release:
+  mode: integrate
+  versioning: none
+  changelog: false
+`;
+var SAMPLE_README = `# crew-sample
+
+A throwaway repository created by \`crew sample-ticket\`. It exists so the crew
+has something safe to work on: its test and build hooks always pass, it has no
+remote, and \`crew sample-ticket --remove\` deletes it.
+`;
+function sampleDir(existing, reposBase, workspace) {
+  if (existing.length > 0) return join21(dirname13(existing[0]), SAMPLE_REPO);
+  return join21(reposBase, workspace, SAMPLE_REPO);
+}
+var SAMPLE_TITLE = "Sample: add a documented `greet` function with a test";
+function sampleDescription() {
+  return [
+    SAMPLE_MARKER,
+    "",
+    "> This is a sample ticket filed by `crew sample-ticket` on a throwaway repository. Nothing here is real work.",
+    "",
+    "## What",
+    "",
+    "Add `greet.js` exporting `greet(name)`, which returns `Hello, <name>!` (and `Hello, world!` for no name),",
+    "with a JSDoc comment, and `greet.test.js` covering both cases using `node:test`.",
+    "",
+    "## Acceptance",
+    "",
+    '- `greet("Ada")` returns `Hello, Ada!`; `greet()` returns `Hello, world!`.',
+    "- The test file runs with `node --test` and passes.",
+    "- Add a line to the README saying the function exists.",
+    "",
+    "## Notes",
+    "",
+    "The repo has no remote: leave the branch local. The hooks in `.crew.yaml` always pass, so QA is checking the",
+    "diff against the acceptance list above."
+  ].join("\n");
+}
+function isSampleTicket(description) {
+  return (description ?? "").trimStart().startsWith(SAMPLE_MARKER);
+}
+function planSampleTicket(i) {
+  return {
+    title: SAMPLE_TITLE,
+    description: sampleDescription(),
+    report_type: "feature",
+    severity: "s4",
+    reporter_name: "crew sample-ticket",
+    status: i.approvedStatus,
+    project_id: i.projectId,
+    repo_id: i.repoId,
+    ...i.filedBy ? { filed_by_id: i.filedBy } : {}
+  };
+}
+
 // src/doctor-fix.ts
-import { parseDocument as parseDocument2 } from "yaml";
+import { parseDocument as parseDocument3 } from "yaml";
 var sound = (r) => r.hasRepos && r.problems.length === 0;
 function planFix(f) {
   const actions = [];
@@ -12960,7 +13101,7 @@ function routesInScope(routes, named) {
   return named ? routes.filter((r) => r.route === named) : routes;
 }
 function planEnable(text, route) {
-  const doc = parseDocument2(text);
+  const doc = parseDocument3(text);
   routeMapOf(doc, route).set("enabled", true);
   return String(doc);
 }
@@ -14703,6 +14844,7 @@ function usage() {
   crew skills sync [route]      push only crew's skill files (e.g. grill-me) into the workspace Agent Skills table
   crew repos add [ROUTE] [NAME] [PATH]  attach a local checkout to a route in crew.yaml (--dry-run previews)
   crew repos clone [ROUTE] NAME  clone a Repos row's remote into the route directory and attach it (--dir PATH, --dry-run, --json)
+  crew repos hooks [ROUTE] NAME suggest test/build hooks for a checkout (--write route|repo, --json)
   crew repos list ROUTE         the checkouts a route has, with the tracker's Repos row each matches
   crew logbook list [route]     recent Agent Log entries, filterable by --role/--ticket
   crew logbook show [route] ID [--prompt]   one entry; --prompt reconstructs and verifies its prompt
@@ -16382,11 +16524,67 @@ ${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>\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>\ncrew repos hooks [workspace/project] <name> [--suggest] [--write route|repo] [--json]\n");
         process.exit(2);
       };
-      if (sub !== "add" && sub !== "list" && sub !== "clone") usage2();
+      if (sub !== "add" && sub !== "list" && sub !== "clone" && sub !== "hooks") usage2();
       const configText = readFileSync22(cfg.configFile, "utf8");
+      if (sub === "hooks") {
+        try {
+          const a = parseHooksArgs(positional.slice(2), routeNamesOf(configText));
+          const dir = repoDirOf(configText, dirname16(cfg.configFile), a.route, a.name);
+          if (!existsSync29(dir)) throw new ReposError(`${dir} does not exist on disk`);
+          const files = { exists: (r) => existsSync29(resolvePath(dir, r)), read: (r) => {
+            try {
+              return readFileSync22(resolvePath(dir, r), "utf8");
+            } catch {
+              return void 0;
+            }
+          } };
+          const suggestion = suggestHooks(files);
+          const write = value("write");
+          if (write !== void 0 && write !== "route" && write !== "repo") throw new ReposError(`--write takes "route" or "repo" (got "${write}")`);
+          let written;
+          if (write && suggestion.source === null) throw new ReposError(`nothing to write: ${suggestion.notes[0]}`);
+          if (write === "route") {
+            const plan = planRouteHooks(configText, a.route, a.name, suggestion.hooks);
+            writeFileSync16(cfg.configFile, plan.text);
+            written = { target: "route", file: cfg.configFile, hooks: plan.written, kept: plan.kept };
+          } else if (write === "repo") {
+            const have = existingRepoConfig(files);
+            if (have) throw new ReposError(`${resolvePath(dir, have)} already exists; not overwriting it`);
+            const file = resolvePath(dir, ".crew.yaml");
+            writeFileSync16(file, starterRepoConfig(suggestion.hooks), { flag: "wx" });
+            written = { target: "repo", file, hooks: Object.keys(suggestion.hooks), kept: [] };
+          }
+          if (flag("json")) {
+            process.stdout.write(`${JSON.stringify({ route: a.route, repo: a.name, dir, ...suggestion, written: written ?? null })}
+`);
+          } else {
+            if (suggestion.source) process.stdout.write(`${a.name}: proposed from ${suggestion.source}
+`);
+            for (const [k, v] of Object.entries(suggestion.hooks)) process.stdout.write(`  ${k}: ${v}
+`);
+            for (const n of suggestion.notes) process.stdout.write(`note: ${n}
+`);
+            if (written) {
+              process.stdout.write(`wrote ${written.hooks.join(", ") || "nothing new"} to ${written.file}
+`);
+              if (written.kept.length) process.stdout.write(`kept existing route hooks: ${written.kept.join(", ")}
+`);
+              if (written.target === "repo") process.stdout.write("review it and commit it to the repository\n");
+            }
+          }
+        } catch (e) {
+          if (e instanceof ReposError) {
+            process.stderr.write(`crew repos: ${e.message}
+`);
+            process.exit(2);
+          }
+          throw e;
+        }
+        break;
+      }
       let routeName = positional[2];
       let addPath = ".";
       let addName;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tablation/crew",
-  "version": "0.59.0",
+  "version": "0.60.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",