@webappwiz/cli 0.0.8 → 0.0.9

package/README.md

@@ -9,49 +9,24 @@ bunx @webappwiz/cli skills add arbor   # install an agent skill
 bunx @webappwiz/cli skills update      # refresh the ones already installed
 bunx @webappwiz/cli rules ls           # every rule there is
 bunx @webappwiz/cli judge .            # check a directory against them
-bunx @webappwiz/cli signoff            # does this change need a person?
 ```
 
 ## rules
 
 Every rule webappwiz judges itself by is named in [`rules.ts`](./rules.ts), as
-`JUDGE_RULES` and `SIGNOFF_RULES`, off the classes
-[`@webappwiz/rules`](../rules/rules) ships. There is no config file and no
-preset: a rule is in one of those lists or it does not exist.
+`JUDGE_RULES`, off the classes [`@webappwiz/rules`](../rules/rules) ships.
+There is no config file and no preset: a rule is in that list or it does not
+exist.
 
 ```
-ID                  RULE                       SET      LEVEL    FILES
-no-em-dashes        No em dashes               judge    error    **/*.ts
-one-class-per-file  One class per file         judge    error    **/*.ts
-visual-work-tested  Visual work is tested      signoff
+ID                  RULE                 LEVEL    FILES
+no-em-dashes        No em dashes         error    **/*.ts
+one-class-per-file  One class per file   error    **/*.ts
 ```
 
 `rules show <id>` prints one in full: its glob, its level, and the document an
 agent is handed verbatim.
 
-The `SET` column is which of the two lists a rule is in. `judge` rules are what
-`judge` checks files against, file by file. `signoff` rules have no glob and no
-level because they are about a change rather than a file: `signoff` weighs them,
-and what they answer is whether it needs a person rather than where the code is
-wrong.
-
-## signoff
-
-Weighs a change against the signoff rules and exits 1 with a reason when one of
-them wants a person to look before it merges. One agent call over the whole
-diff, since that is what these rules are about.
-
-```bash
-bunx @webappwiz/cli signoff                    # everything since main
-bunx @webappwiz/cli signoff --since HEAD~3     # measured against another ref
-bunx @webappwiz/cli signoff --print            # the rules, to apply yourself
-```
-
-`--print` is the cheapest signoff there is: the agent about to merge reads the
-rules and weighs its own change, spawning nothing. A project points its agent
-instructions at that rather than at a list of rule ids, which goes stale the
-next time a rule is added.
-
 ## judge
 
 Runs the rules over a directory, one agent call per set of rules sharing a set

package/{index-rz4spnkb.js → index-g81gg9gz.js}

@@ -1,8 +1,8 @@
 import {
   table
-} from "./index-4xj59vmx.js";
+} from "./index-pyjg1rtk.js";
 // package.json
-var version = "0.0.8";
+var version = "0.0.9";
 
 // skills/add.ts
 import { ConsoleLogger } from "webappwiz/log";
@@ -15,7 +15,7 @@ import { dirname } from "node:path";
 var arbor_skill_default = `---
 name: arbor
 description: Use the @webappwiz/arbor CLI to land your work on trunk, or a base branch given as an argument, from an isolated git worktree without pull requests. Read this before making any code change in an arbor repository, since it decides where the work happens, and whenever you need to add, claim, merge, remove, list, show, locate, or escalate a task.
-version: 0.0.8
+version: 0.0.9
 ---
 
 # Using arbor
@@ -184,7 +184,7 @@ not squashing.
 var webappwiz_skill_default = `---
 name: webappwiz
 description: Check whether the webappwiz package already covers a piece of infrastructure before writing it by hand or adding a dependency for it. Read this before writing any of: time, clocks, durations or timers; logging; id generation; HTTP serving; CLI argument parsing; background tasks or queues; web workers; markdown parsing; typed event emitters; 2D geometry or spatial indexes; filesystem, env or process access; typed RPC over fetch; schema validation; AbortSignal plumbing; disposable resources; browser scroll, animation frames or visibility. Also use whenever the user says webappwiz.
-version: 0.0.8
+version: 0.0.9
 ---
 
 # Using webappwiz

package/{index-hz6bwqfy.js → index-htwb54c6.js}

@@ -17,9 +17,7 @@ import {
   ReactiveOverUseState,
   ResourcesAreDisposable,
   SimpleTestSetup,
-  TestsNotWeakened,
-  TestsOwnTheirState,
-  VisualWorkTested
+  TestsOwnTheirState
 } from "@webappwiz/rules/catalog";
 var JUDGE_RULES = defineRules({
   rules: [
@@ -42,9 +40,5 @@ var JUDGE_RULES = defineRules({
     new ResourcesAreDisposable
   ]
 });
-var SIGNOFF_RULES = [
-  new TestsNotWeakened,
-  new VisualWorkTested
-];
 
-export { JUDGE_RULES, SIGNOFF_RULES };
+export { JUDGE_RULES };

package/{index-r14q0cmx.js → index-pyjg1rtk.js}

@@ -1,16 +1,3 @@
-import {
-  changed,
-  compact,
-  count,
-  divider,
-  finished,
-  mode,
-  planned,
-  summary,
-  table,
-  tokens
-} from "./index-4xj59vmx.js";
-
 // judge.ts
 import {
   agentCommand,
@@ -18,17 +5,148 @@ import {
   Harness,
   prompt as reviewPrompt
 } from "@webappwiz/rules";
-import { ConsoleLogger, color as color2 } from "webappwiz/log";
+import { ConsoleLogger, color as color4 } from "webappwiz/log";
 import {
   NodeFs,
   NodeGlob,
-  NodePs
+  NodePs as NodePs2
 } from "webappwiz/system";
 import { SystemClock } from "webappwiz/time";
 
+// changed.ts
+import { NodePs } from "webappwiz/system";
+async function changed(dir, ref, opts = {}) {
+  const ps = opts.ps ?? new NodePs;
+  const files = new Set;
+  for (const argv of [
+    ["diff", "--name-only", "--diff-filter=d", "--relative", ref],
+    ["ls-files", "--others", "--exclude-standard"]
+  ]) {
+    for (const line of (await git(ps, dir, argv)).split(`
+`)) {
+      if (line !== "") {
+        files.add(line);
+      }
+    }
+  }
+  return files;
+}
+async function git(ps, dir, argv) {
+  const { exitCode, stdout, stderr } = await ps.spawnCapture([
+    "git",
+    "-C",
+    dir,
+    ...argv
+  ]);
+  if (exitCode !== 0) {
+    throw new Error(`git ${argv[0]} failed in ${dir}: ${stderr.trim() || `exit ${exitCode}`}`);
+  }
+  return stdout;
+}
+
+// mode.ts
+function mode({ print, agent, exec }) {
+  const named = [
+    print === true ? "--print" : undefined,
+    agent === undefined ? undefined : "--agent",
+    exec === undefined ? undefined : "--exec"
+  ].filter((flag) => flag !== undefined);
+  if (named.length > 1) {
+    throw new Error(`${named.join(" and ")} are different things to do with one run, so pass one`);
+  }
+  return print === true ? "print" : "run";
+}
+
 // progress.ts
-import { color } from "webappwiz/log";
+import { color as color3 } from "webappwiz/log";
 import { Duration, SystemTimer } from "webappwiz/time";
+
+// report.ts
+import { color as color2 } from "webappwiz/log";
+
+// table.ts
+import { color } from "webappwiz/log";
+var table = (rows) => {
+  const width = (cell) => color.strip(cell).length;
+  const widths = rows[0]?.map((_, i) => Math.max(...rows.map((row) => width(row[i] ?? ""))));
+  return rows.map((row) => row.map((cell, i) => cell.padEnd((widths?.[i] ?? 0) + cell.length - width(cell))).join("   ").trimEnd());
+};
+
+// report.ts
+var count = (total, word) => `${total} ${word}${total === 1 ? "" : "s"}`;
+var divider = (name) => {
+  if (name === undefined) {
+    return color2.dim("-".repeat(72));
+  }
+  const opening = `--- ${name} `;
+  return color2.dim(opening.padEnd(Math.max(72, opening.length + 3), "-"));
+};
+var compact = new Intl.NumberFormat("en", { notation: "compact" });
+var tokens = (bytes) => Math.ceil(bytes / 4);
+function planned({
+  files,
+  rules,
+  calls,
+  estimate,
+  concurrency,
+  agent
+}) {
+  const rows = [
+    [color2.dim("files"), String(files)],
+    [color2.dim("rules"), String(rules)],
+    [color2.dim("calls"), String(calls)],
+    ...concurrency === undefined ? [] : [[color2.dim("workers"), String(concurrency)]],
+    [color2.dim("reading"), `${compact.format(estimate)}+ tokens`]
+  ];
+  if (agent !== undefined) {
+    rows.push([color2.dim("agent"), agent]);
+  }
+  return ["", ...table(rows).map((line) => `  ${line}`), ""];
+}
+function finished({
+  rules,
+  files,
+  violations,
+  took,
+  tokens: tokens2,
+  worker,
+  workerTokens,
+  done,
+  total
+}) {
+  const heading = `${color2.gray(`[${done}/${total}]`)} ` + color2.gray(`(${count(rules.length, "rule")}, ${count(files, "file")})`);
+  const spent = tokens2 === undefined ? "" : `  ${compact.format(tokens2)} tokens` + (workerTokens === undefined ? "" : ` (w${worker + 1}: ${compact.format(workerTokens)})`);
+  const tail = `${color2.gray(`in ${took.human()}${spent}`)}`;
+  if (violations.length === 0) {
+    return [`${color2.green("✓")} ${heading}: clean ${tail}`];
+  }
+  return [
+    `${color2.red("✗")} ${heading}: ${count(violations.length, "problem")} ${tail}`,
+    ...violations.flatMap(finding)
+  ];
+}
+function finding(violation) {
+  const level = violation.level === "error" ? color2.red("error") : color2.yellow(violation.level);
+  const lines = [
+    `  ${color2.bold(`${violation.file}:${violation.line}`)}  ${level}  ${violation.message} ${color2.gray(`(${violation.id})`)}`
+  ];
+  if (violation.code !== "") {
+    lines.push(color2.gray(`  │ ${violation.code}`));
+  }
+  return lines;
+}
+function summary(violations, took, tokens2) {
+  const spent = tokens2 === undefined ? "" : `  ${compact.format(tokens2)} tokens total`;
+  const elapsed = color2.gray(`in ${took.human()}${spent}`);
+  if (violations.length === 0) {
+    return `${color2.green("✓ no violations")} ${elapsed}`;
+  }
+  const errors = violations.filter((violation) => violation.level === "error").length;
+  const line = `✖ ${count(violations.length, "problem")} (${count(errors, "error")}, ${count(violations.length - errors, "warning")})`;
+  return `${errors > 0 ? color2.red(line) : color2.yellow(line)} ${elapsed}`;
+}
+
+// progress.ts
 var terminal = () => ({
   tty: process.stdout.isTTY === true,
   write: (text) => process.stdout.write(text)
@@ -38,11 +156,11 @@ var FRAMES = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "
 function render(view, frame = 0) {
   const spin = view.files === 0 ? " " : FRAMES[frame % FRAMES.length] ?? " ";
   const filled = Math.round(BAR * view.done / Math.max(1, view.total));
-  const bar = color.green("█".repeat(filled)) + color.dim("░".repeat(BAR - filled));
+  const bar = color3.green("█".repeat(filled)) + color3.dim("░".repeat(BAR - filled));
   const judging = view.files === 0 ? "" : ` · judging ${count(view.files, "file")}`;
   const spent = view.tokens === undefined ? "" : ` · ${compact.format(view.tokens)} tokens`;
   const found = view.done === 0 ? "" : ` · ${view.problems === 0 ? "clean so far" : count(view.problems, "problem")}`;
-  return `${color.green(spin)} ${bar}  ${color.gray(`${view.done}/${view.total} calls${judging}${spent}${found}`)}`;
+  return `${color3.green(spin)} ${bar}  ${color3.gray(`${view.done}/${view.total} calls${judging}${spent}${found}`)}`;
 }
 
 class Progress {
@@ -104,12 +222,10 @@ class Progress {
 
 // judge.ts
 var estimated = (reviews) => tokens(reviews.reduce((bytes, review) => bytes + review.bytes, 0));
-var isFileRule = (rule) => ("files" in rule);
 var title = (rule) => /^#\s+(.+)$/m.exec(rule.document)?.[1]?.trim() ?? rule.id;
 
 class JudgeCommands {
   rules;
-  signoffRules;
   screen;
   log;
   fs;
@@ -118,38 +234,33 @@ class JudgeCommands {
   glob;
   constructor(rules, opts = {}) {
     this.rules = rules;
-    this.signoffRules = opts.signoffRules ?? [];
     this.screen = opts.screen ?? terminal();
     this.log = opts.log ?? new ConsoleLogger;
     this.fs = opts.fs ?? new NodeFs;
-    this.ps = opts.ps ?? new NodePs;
+    this.ps = opts.ps ?? new NodePs2;
     this.clock = opts.clock ?? new SystemClock;
     this.glob = opts.glob ?? new NodeGlob;
   }
   ls() {
-    const rows = [["id", "rule", "set", "level", "files"].map(color2.dim)];
+    const rows = [["id", "rule", "level", "files"].map(color4.dim)];
     for (const rule of this.rules.rules) {
-      rows.push([rule.id, title(rule), "judge", rule.level, rule.files]);
-    }
-    for (const rule of this.signoffRules) {
-      rows.push([rule.id, title(rule), "signoff", "", ""]);
+      rows.push([rule.id, title(rule), rule.level, rule.files]);
     }
     this.log.info(table(rows).join(`
 `));
   }
   show(opts) {
-    const all = [...this.rules.rules, ...this.signoffRules];
+    const all = this.rules.rules;
     const rule = all.find((candidate) => candidate.id === opts.id);
     if (!rule) {
       throw new Error(`no rule "${opts.id}". Known ids: ${all.map((candidate) => candidate.id).join(", ")}`);
     }
     const rows = [
-      [color2.dim("id"), rule.id],
-      [color2.dim("rule"), title(rule)]
+      [color4.dim("id"), rule.id],
+      [color4.dim("rule"), title(rule)],
+      [color4.dim("level"), rule.level],
+      [color4.dim("files"), rule.files]
     ];
-    if (isFileRule(rule)) {
-      rows.push([color2.dim("level"), rule.level], [color2.dim("files"), rule.files]);
-    }
     this.log.info(table(rows).join(`
 `));
     this.log.info("");
@@ -259,4 +370,4 @@ ${divider()}`);
   }
 }
 
-export { JudgeCommands };
+export { table, JudgeCommands };

package/index.js

@@ -6,18 +6,13 @@ import {
   update,
   update1 as update2,
   version
-} from "./index-rz4spnkb.js";
+} from "./index-g81gg9gz.js";
 import {
-  JUDGE_RULES,
-  SIGNOFF_RULES
-} from "./index-hz6bwqfy.js";
+  JUDGE_RULES
+} from "./index-htwb54c6.js";
 import {
   JudgeCommands
-} from "./index-r14q0cmx.js";
-import {
-  Signoff
-} from "./index-fh8js68z.js";
-import"./index-4xj59vmx.js";
+} from "./index-pyjg1rtk.js";
 
 // index.ts
 import { NodeFs, NodeGlob } from "webappwiz/system";
@@ -35,14 +30,7 @@ webappwiz.command("update").description("pin every webappwiz dependency in a tre
   default: version,
   description: "version to pin to"
 }).action((opts, { log, fs }) => update2({ ...opts, log, fs }));
-var judge = ({ log, fs, ps, clock, glob }) => new JudgeCommands(JUDGE_RULES, {
-  signoffRules: SIGNOFF_RULES,
-  log,
-  fs,
-  ps,
-  clock,
-  glob
-});
+var judge = ({ log, fs, ps, clock, glob }) => new JudgeCommands(JUDGE_RULES, { log, fs, ps, clock, glob });
 webappwiz.command("judge").description("check a directory against the config, one agent per glob").arg("dir", t.string(), {
   default: ".",
   description: "directory to judge (default: .)"
@@ -64,24 +52,6 @@ webappwiz.command("judge").description("check a directory against the config, on
   default: false,
   description: "line-by-line output with no live progress block"
 }).action((opts, deps) => judge(deps).judge(opts));
-webappwiz.command("signoff").description("weigh a change against the rules that ask for a person").arg("dir", t.string(), {
-  default: ".",
-  description: "directory whose change is weighed (default: .)"
-}).option("agent", t.optional(t.enum(Object.keys(AGENTS))), {
-  description: "model to weigh it with (default: the config's agent)"
-}).option("exec", t.optional(t.string()), {
-  description: "command the prompt is passed to, instead of --agent"
-}).option("print", t.boolean(), {
-  default: false,
-  description: "print the rules to apply yourself, and run no agent"
-}).option("since", t.string(), {
-  default: "main",
-  description: "the ref the change is measured against"
-}).action((opts, { log, ps, clock }) => new Signoff(SIGNOFF_RULES, JUDGE_RULES.agent, {
-  log,
-  ps,
-  clock
-}).run(opts));
 var rules = webappwiz.group("rules").description("list and print the rules, to run or to read yourself");
 rules.command("ls").description("list the rules").action((_opts, deps) => judge(deps).ls());
 rules.command("show").description("print one rule in full, by the id `rules ls` gives it").arg("id", t.string(), { description: "rule id" }).action((opts, deps) => judge(deps).show(opts));

package/judge.d.ts

@@ -1,4 +1,4 @@
-import { type Rule, type RuleSet } from "@webappwiz/rules";
+import { type RuleSet } from "@webappwiz/rules";
 import { type Logger } from "webappwiz/log";
 import { type Fs, type Glob, type Ps } from "webappwiz/system";
 import { type Clock } from "webappwiz/time";
@@ -25,8 +25,6 @@ export interface JudgeOptions {
 }
 /** What a `JudgeCommands` runs through, and what else it lists. */
 export interface JudgeCommandsOptions {
-    /** Rules only a reader applies, listed beside the ones a run checks. */
-    signoffRules?: Rule[];
     /** Where live progress draws; this process's terminal by default. */
     screen?: Screen;
     log?: Logger;
@@ -37,7 +35,6 @@ export interface JudgeCommandsOptions {
 }
 export declare class JudgeCommands {
     private rules;
-    private signoffRules;
     private screen;
     private log;
     private fs;
@@ -45,16 +42,11 @@ export declare class JudgeCommands {
     private clock;
     private glob;
     constructor(rules: RuleSet, opts?: JudgeCommandsOptions);
-    /**
-     * Lists every rule there is, one row each, ids first for citing. The rules a
-     * run checks and the ones only a reader applies are one list with a SET
-     * column, because "what rules are there" is one question.
-     */
+    /** Lists every rule there is, one row each, ids first for citing. */
     ls(): void;
     /**
      * Prints one rule in full: what it covers, and the document an agent is
-     * given, verbatim. Take the id from `rules ls` or from a finding. This is
-     * how a reader applies a rule nothing runs for them.
+     * given, verbatim. Take the id from `rules ls` or from a finding.
      */
     show(opts: ShowOptions): void;
     /**

package/judge.js

@@ -1,7 +1,6 @@
 import {
   JudgeCommands
-} from "./index-r14q0cmx.js";
-import"./index-4xj59vmx.js";
+} from "./index-pyjg1rtk.js";
 export {
   JudgeCommands
 };

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@webappwiz/cli",
-	"version": "0.0.8",
-	"description": "The webappwiz CLI: judge code against rules, sign off a diff, and manage agent skills",
+	"version": "0.0.9",
+	"description": "The webappwiz CLI: judge code against rules and manage agent skills",
 	"license": "MIT",
 	"author": "Jared Johnson",
 	"repository": {
@@ -17,8 +17,8 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@webappwiz/rules": "^0.0.8",
-		"webappwiz": "^0.0.8"
+		"@webappwiz/rules": "^0.0.9",
+		"webappwiz": "^0.0.9"
 	},
 	"peerDependencies": {
 		"typescript": "^7"
@@ -41,10 +41,6 @@
 		"./judge": {
 			"types": "./judge.d.ts",
 			"default": "./judge.js"
-		},
-		"./signoff": {
-			"types": "./signoff.d.ts",
-			"default": "./signoff.js"
 		}
 	},
 	"bin": {

package/rules.d.ts

@@ -1,4 +1,3 @@
-import { type Rule } from "@webappwiz/rules";
 /**
  * Every rule webappwiz judges itself by, named one by one. There is no preset
  * to spread and nothing runs implicitly: a rule is here or it does not run.
@@ -9,12 +8,3 @@ import { type Rule } from "@webappwiz/rules";
  * module for one of these to import.
  */
 export declare const JUDGE_RULES: import("@webappwiz/rules").RuleSet;
-/**
- * What an agent weighs before merging a change, rather than what it checks a
- * file against: whether the change needs a person to look at it.
- *
- * No command runs these. They are documents an agent reads and applies itself,
- * which is why they carry no glob and no check, and `wiz rules show` prints
- * them the same as any other.
- */
-export declare const SIGNOFF_RULES: Rule[];

package/rules.js

@@ -1,8 +1,6 @@
 import {
-  JUDGE_RULES,
-  SIGNOFF_RULES
-} from "./index-hz6bwqfy.js";
+  JUDGE_RULES
+} from "./index-htwb54c6.js";
 export {
-  SIGNOFF_RULES,
   JUDGE_RULES
 };

package/webappwiz.js

@@ -4,18 +4,13 @@ import {
   update,
   update1 as update2,
   version
-} from "./index-rz4spnkb.js";
+} from "./index-g81gg9gz.js";
 import {
-  JUDGE_RULES,
-  SIGNOFF_RULES
-} from "./index-hz6bwqfy.js";
+  JUDGE_RULES
+} from "./index-htwb54c6.js";
 import {
   JudgeCommands
-} from "./index-r14q0cmx.js";
-import {
-  Signoff
-} from "./index-fh8js68z.js";
-import"./index-4xj59vmx.js";
+} from "./index-pyjg1rtk.js";
 
 // webappwiz.ts
 import { AGENTS } from "@webappwiz/rules";
@@ -29,14 +24,7 @@ webappwiz.command("update").description("pin every webappwiz dependency in a tre
   default: version,
   description: "version to pin to"
 }).action((opts, { log, fs }) => update2({ ...opts, log, fs }));
-var judge = ({ log, fs, ps, clock, glob }) => new JudgeCommands(JUDGE_RULES, {
-  signoffRules: SIGNOFF_RULES,
-  log,
-  fs,
-  ps,
-  clock,
-  glob
-});
+var judge = ({ log, fs, ps, clock, glob }) => new JudgeCommands(JUDGE_RULES, { log, fs, ps, clock, glob });
 webappwiz.command("judge").description("check a directory against the config, one agent per glob").arg("dir", t.string(), {
   default: ".",
   description: "directory to judge (default: .)"
@@ -58,24 +46,6 @@ webappwiz.command("judge").description("check a directory against the config, on
   default: false,
   description: "line-by-line output with no live progress block"
 }).action((opts, deps) => judge(deps).judge(opts));
-webappwiz.command("signoff").description("weigh a change against the rules that ask for a person").arg("dir", t.string(), {
-  default: ".",
-  description: "directory whose change is weighed (default: .)"
-}).option("agent", t.optional(t.enum(Object.keys(AGENTS))), {
-  description: "model to weigh it with (default: the config's agent)"
-}).option("exec", t.optional(t.string()), {
-  description: "command the prompt is passed to, instead of --agent"
-}).option("print", t.boolean(), {
-  default: false,
-  description: "print the rules to apply yourself, and run no agent"
-}).option("since", t.string(), {
-  default: "main",
-  description: "the ref the change is measured against"
-}).action((opts, { log, ps, clock }) => new Signoff(SIGNOFF_RULES, JUDGE_RULES.agent, {
-  log,
-  ps,
-  clock
-}).run(opts));
 var rules = webappwiz.group("rules").description("list and print the rules, to run or to read yourself");
 rules.command("ls").description("list the rules").action((_opts, deps) => judge(deps).ls());
 rules.command("show").description("print one rule in full, by the id `rules ls` gives it").arg("id", t.string(), { description: "rule id" }).action((opts, deps) => judge(deps).show(opts));

package/index-4xj59vmx.js

@@ -1,135 +0,0 @@
-// changed.ts
-import { NodePs } from "webappwiz/system";
-async function changed(dir, ref, opts = {}) {
-  const ps = opts.ps ?? new NodePs;
-  const files = new Set;
-  for (const argv of [
-    ["diff", "--name-only", "--diff-filter=d", "--relative", ref],
-    ["ls-files", "--others", "--exclude-standard"]
-  ]) {
-    for (const line of (await git(ps, dir, argv)).split(`
-`)) {
-      if (line !== "") {
-        files.add(line);
-      }
-    }
-  }
-  return files;
-}
-async function diff(dir, ref, opts = {}) {
-  const ps = opts.ps ?? new NodePs;
-  const patch = await git(ps, dir, ["diff", "--relative", ref]);
-  const added = (await git(ps, dir, ["ls-files", "--others", "--exclude-standard"])).split(`
-`).filter((line) => line !== "");
-  return { patch: patch.trim(), added };
-}
-async function git(ps, dir, argv) {
-  const { exitCode, stdout, stderr } = await ps.spawnCapture([
-    "git",
-    "-C",
-    dir,
-    ...argv
-  ]);
-  if (exitCode !== 0) {
-    throw new Error(`git ${argv[0]} failed in ${dir}: ${stderr.trim() || `exit ${exitCode}`}`);
-  }
-  return stdout;
-}
-
-// mode.ts
-function mode({ print, agent, exec }) {
-  const named = [
-    print === true ? "--print" : undefined,
-    agent === undefined ? undefined : "--agent",
-    exec === undefined ? undefined : "--exec"
-  ].filter((flag) => flag !== undefined);
-  if (named.length > 1) {
-    throw new Error(`${named.join(" and ")} are different things to do with one run, so pass one`);
-  }
-  return print === true ? "print" : "run";
-}
-
-// table.ts
-import { color } from "webappwiz/log";
-var table = (rows) => {
-  const width = (cell) => color.strip(cell).length;
-  const widths = rows[0]?.map((_, i) => Math.max(...rows.map((row) => width(row[i] ?? ""))));
-  return rows.map((row) => row.map((cell, i) => cell.padEnd((widths?.[i] ?? 0) + cell.length - width(cell))).join("   ").trimEnd());
-};
-
-// report.ts
-import { color as color2 } from "webappwiz/log";
-var count = (total, word) => `${total} ${word}${total === 1 ? "" : "s"}`;
-var divider = (name) => {
-  if (name === undefined) {
-    return color2.dim("-".repeat(72));
-  }
-  const opening = `--- ${name} `;
-  return color2.dim(opening.padEnd(Math.max(72, opening.length + 3), "-"));
-};
-var compact = new Intl.NumberFormat("en", { notation: "compact" });
-var tokens = (bytes) => Math.ceil(bytes / 4);
-function planned({
-  files,
-  rules,
-  calls,
-  estimate,
-  concurrency,
-  agent
-}) {
-  const rows = [
-    [color2.dim("files"), String(files)],
-    [color2.dim("rules"), String(rules)],
-    [color2.dim("calls"), String(calls)],
-    ...concurrency === undefined ? [] : [[color2.dim("workers"), String(concurrency)]],
-    [color2.dim("reading"), `${compact.format(estimate)}+ tokens`]
-  ];
-  if (agent !== undefined) {
-    rows.push([color2.dim("agent"), agent]);
-  }
-  return ["", ...table(rows).map((line) => `  ${line}`), ""];
-}
-function finished({
-  rules,
-  files,
-  violations,
-  took,
-  tokens: tokens2,
-  worker,
-  workerTokens,
-  done,
-  total
-}) {
-  const heading = `${color2.gray(`[${done}/${total}]`)} ` + color2.gray(`(${count(rules.length, "rule")}, ${count(files, "file")})`);
-  const spent = tokens2 === undefined ? "" : `  ${compact.format(tokens2)} tokens` + (workerTokens === undefined ? "" : ` (w${worker + 1}: ${compact.format(workerTokens)})`);
-  const tail = `${color2.gray(`in ${took.human()}${spent}`)}`;
-  if (violations.length === 0) {
-    return [`${color2.green("✓")} ${heading}: clean ${tail}`];
-  }
-  return [
-    `${color2.red("✗")} ${heading}: ${count(violations.length, "problem")} ${tail}`,
-    ...violations.flatMap(finding)
-  ];
-}
-function finding(violation) {
-  const level = violation.level === "error" ? color2.red("error") : color2.yellow(violation.level);
-  const lines = [
-    `  ${color2.bold(`${violation.file}:${violation.line}`)}  ${level}  ${violation.message} ${color2.gray(`(${violation.id})`)}`
-  ];
-  if (violation.code !== "") {
-    lines.push(color2.gray(`  │ ${violation.code}`));
-  }
-  return lines;
-}
-function summary(violations, took, tokens2) {
-  const spent = tokens2 === undefined ? "" : `  ${compact.format(tokens2)} tokens total`;
-  const elapsed = color2.gray(`in ${took.human()}${spent}`);
-  if (violations.length === 0) {
-    return `${color2.green("✓ no violations")} ${elapsed}`;
-  }
-  const errors = violations.filter((violation) => violation.level === "error").length;
-  const line = `✖ ${count(violations.length, "problem")} (${count(errors, "error")}, ${count(violations.length - errors, "warning")})`;
-  return `${errors > 0 ? color2.red(line) : color2.yellow(line)} ${elapsed}`;
-}
-
-export { changed, diff, mode, table, count, divider, compact, tokens, planned, finished, summary };

package/index-fh8js68z.js

@@ -1,116 +0,0 @@
-import {
-  count,
-  diff,
-  divider,
-  mode,
-  tokens
-} from "./index-4xj59vmx.js";
-
-// signoff.ts
-import {
-  agentCommand,
-  Harness,
-  prompt as reviewPrompt
-} from "@webappwiz/rules";
-import { ConsoleLogger, color } from "webappwiz/log";
-import { NodePs } from "webappwiz/system";
-import { SystemClock } from "webappwiz/time";
-class Signoff {
-  rules;
-  defaultAgent;
-  log;
-  ps;
-  clock;
-  constructor(rules, defaultAgent, opts = {}) {
-    this.rules = rules;
-    this.defaultAgent = defaultAgent;
-    this.log = opts.log ?? new ConsoleLogger;
-    this.ps = opts.ps ?? new NodePs;
-    this.clock = opts.clock ?? new SystemClock;
-  }
-  async run(opts) {
-    if (mode(opts) === "print") {
-      this.print();
-      return;
-    }
-    const dir = opts.dir.replace(/\/+$/, "") || "/";
-    const { patch, added } = await diff(dir, opts.since, { ps: this.ps });
-    if (patch === "" && added.length === 0) {
-      this.log.info(`nothing has changed since ${opts.since}`);
-      return;
-    }
-    const agent = agentCommand(opts.exec === undefined ? { agent: opts.agent ?? this.defaultAgent } : opts);
-    const review = this.review(patch, added, opts.since);
-    this.log.info(`weighing ${opts.since}..working tree against ` + `${count(this.rules.length, "rule")}, reading ` + `${tokens(review.bytes ?? 0)}+ tokens with ${agent.label}`);
-    this.say(await this.judge(review, agent, dir));
-  }
-  async judge(review, agent, dir) {
-    const harness = new Harness({
-      log: this.log,
-      ps: this.ps,
-      clock: this.clock
-    });
-    harness.events.on("finished", ({ took, tokens: read }) => {
-      const spent = read === undefined ? "" : `  ${read} tokens`;
-      this.log.info(color.gray(`read in ${took.human()}${spent}`));
-    });
-    return await harness.run([review], agent, { cwd: dir });
-  }
-  say(findings) {
-    if (findings.length === 0) {
-      this.log.info(color.green("✓ nothing in this change needs a person"));
-      return;
-    }
-    for (const finding of findings) {
-      const where = finding.file === undefined ? "" : `${color.bold(finding.file + (finding.line === undefined ? "" : `:${finding.line}`))}  `;
-      this.log.info(`  ${where}${finding.message} ${color.gray(`(${finding.rule})`)}`);
-    }
-    throw new Error(`${count(findings.length, "reason")} to escalate rather than merge`);
-  }
-  review(patch, added, ref) {
-    const draft = {
-      rules: this.rules,
-      label: "signoff",
-      context: [
-        `The change, as \`git diff ${ref}\` prints it:`,
-        `\`\`\`diff
-${patch}
-\`\`\``,
-        ...added.length === 0 ? [] : [
-          "These files are new and are in no diff yet. Read each one from " + "your working directory:",
-          added.map((file) => `- ${file}`).join(`
-`)
-        ]
-      ].join(`
-
-`),
-      instructions: WEIGHING
-    };
-    return { ...draft, bytes: Buffer.byteLength(reviewPrompt(draft)) };
-  }
-  print() {
-    if (this.rules.length === 0) {
-      this.log.info("no signoff rules");
-      return;
-    }
-    this.log.info("Weigh your change against each rule below. Anything that needs " + "review goes to a person instead of trunk.");
-    for (const rule of this.rules) {
-      this.log.info(`
-${divider(rule.id)}
-`);
-      this.log.info(rule.document.trim());
-    }
-    this.log.info(`
-${divider()}`);
-  }
-}
-var WEIGHING = [
-  "These rules decide one thing: whether this change can merge on its own, " + "or needs a person to look at it first.",
-  "",
-  "Each rule's `Ships` section is what merges with nobody looking, and its " + "`Needs review` section is what does not. Report a violation only for " + "what the `Needs review` side covers, and only where the change " + "actually does it.",
-  "",
-  "These rules are about the change as a whole, so leave `file` and `line` " + "out unless one place in the change is the whole of the answer."
-].join(`
-`);
-
-export { Signoff };

package/signoff.d.ts

@@ -1,57 +0,0 @@
-import { type Rule } from "@webappwiz/rules";
-import { type Logger } from "webappwiz/log";
-import { type Ps } from "webappwiz/system";
-import { type Clock } from "webappwiz/time";
-export interface SignoffRunOptions {
-    /** The directory whose change is being weighed. */
-    dir: string;
-    agent?: string;
-    exec?: string;
-    /** Print the rules to the logger and spawn nothing, for the reader who is
-     * going to apply them itself. */
-    print?: boolean;
-    /** The ref the change is measured against: everything since it is the
-     * change. */
-    since: string;
-}
-/**
- * Weighs a change against the rules that decide whether it can merge on its
- * own or needs a person to look at it first.
- *
- * One agent call over the diff, rather than one per file like `judge`: these
- * rules are about the change as a whole, and a run that saw one file at a time
- * could not answer them. Exits 1 with a reason when the change needs a person,
- * so it reads the same to a merge gate as to whoever ran it.
- */
-/** What a `Signoff` runs through. */
-export interface SignoffOptions {
-    log?: Logger;
-    ps?: Ps;
-    clock?: Clock;
-}
-export declare class Signoff {
-    private rules;
-    private defaultAgent;
-    private log;
-    private ps;
-    private clock;
-    constructor(rules: Rule[], defaultAgent: string, opts?: SignoffOptions);
-    run(opts: SignoffRunOptions): Promise<void>;
-    /** The one call, and what it says about the change. */
-    private judge;
-    /**
-     * The verdict, and a reason for each rule that wants a person. Throws on any
-     * of them: an agent that ran this before merging should escalate rather than
-     * merge, and an exit code is what says so whoever is reading.
-     */
-    private say;
-    /** The change as the agent reads it, priced by the prompt: what a new file
-     * costs to open is the agent's business and unknowable from here. */
-    private review;
-    /**
-     * The rules in full, for the reader who is going to apply them. Nothing has
-     * to run these: an agent about to merge its own work can weigh it against
-     * them itself, and that is the cheapest signoff there is.
-     */
-    private print;
-}

package/signoff.js

@@ -1,7 +0,0 @@
-import {
-  Signoff
-} from "./index-fh8js68z.js";
-import"./index-4xj59vmx.js";
-export {
-  Signoff
-};