@webappwiz/cli 0.0.9 → 0.0.11

package/index.js

@@ -2,24 +2,22 @@
 // @bun
 import {
   add,
+  add1 as add2,
   ls,
+  ls1 as ls2,
+  newRule,
+  review,
   update,
   update1 as update2,
+  update2 as update3,
   version
-} from "./index-g81gg9gz.js";
-import {
-  JUDGE_RULES
-} from "./index-htwb54c6.js";
-import {
-  JudgeCommands
-} from "./index-pyjg1rtk.js";
+} from "./index-d8p57dk5.js";
 
 // index.ts
 import { NodeFs, NodeGlob } from "webappwiz/system";
-import { SystemClock } from "webappwiz/time";
 
 // webappwiz.ts
-import { AGENTS } from "@webappwiz/rules";
+import { DEFAULT_CHUNK } from "@webappwiz/rules";
 import { cli } from "webappwiz/cmd";
 import { t } from "webappwiz/t";
 var webappwiz = cli("webappwiz");
@@ -29,49 +27,47 @@ webappwiz.command("update").description("pin every webappwiz dependency in a tre
 }).option("version", t.string(), {
   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, { 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: .)"
-}).option("agent", t.optional(t.enum(Object.keys(AGENTS))), {
-  description: "model to check 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 prompts and run no agent at all"
-}).option("chunk", t.number(), {
-  default: 25,
-  description: "files per review"
-}).option("since", t.optional(t.string()), {
-  description: "only check files added or changed since this git ref"
-}).option("concurrency-override", t.optional(t.number()), {
-  description: "agent calls in flight at once, over the config's concurrency"
-}).option("ci", t.boolean(), {
-  default: false,
-  description: "line-by-line output with no live progress block"
-}).action((opts, deps) => judge(deps).judge(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));
-var skillsGroup = webappwiz.group("skills").description("manage webappwiz agent skills in .agents/skills");
-skillsGroup.command("ls").description("list the skills there are, and what the project has of them").arg("dir", t.string(), {
+}).action((opts, { log, fs }) => update3({ ...opts, log, fs }));
+var rules = webappwiz.group("rules").description("keep rules in .wiz/rules, and divide a review of them up");
+rules.command("ls").description("list the rules there are, and what the project has of them").arg("dir", t.string(), {
   default: ".",
   description: "project to inspect (default: .)"
 }).action((opts, { log, fs }) => ls({ ...opts, log, fs }));
-skillsGroup.command("add").description("add a skill to a project").arg("skill", t.string(), { description: "skill name" }).arg("dir", t.string(), {
+rules.command("new").description("scaffold a rule to fill in, under .wiz/rules/<name>").arg("name", t.string(), { description: "rule id, kebab case" }).arg("dir", t.string(), {
+  default: ".",
+  description: "project to add it to (default: .)"
+}).action((opts, { log, fs }) => newRule({ ...opts, log, fs }));
+rules.command("add").description("copy a shipped rule into the project").arg("rule", t.string(), { description: "rule id, as `rules ls` lists it" }).arg("dir", t.string(), {
   default: ".",
   description: "project to add it to (default: .)"
 }).action((opts, { log, fs }) => add({ ...opts, log, fs }));
-skillsGroup.command("update").description("refresh the skills a project already has").arg("dir", t.string(), {
+rules.command("update").description("refresh the shipped rules the project has copies of").arg("dir", t.string(), {
   default: ".",
   description: "project to refresh (default: .)"
 }).action((opts, { log, fs }) => update({ ...opts, log, fs }));
+rules.command("review").description("print one block of review work per rule the change touches").arg("dir", t.string(), {
+  default: ".",
+  description: "project to review (default: .)"
+}).option("since", t.string(), {
+  default: "HEAD",
+  description: "git ref the change is measured from (default: HEAD)"
+}).option("chunk", t.number(), {
+  default: DEFAULT_CHUNK,
+  description: "files per block, at most"
+}).action((opts, { log, fs, ps, glob }) => review({ ...opts, log, fs, ps, glob }));
+var skills = webappwiz.group("skills").description("manage webappwiz agent skills in .agents/skills");
+skills.command("ls").description("list the skills there are, and what the project has of them").arg("dir", t.string(), {
+  default: ".",
+  description: "project to inspect (default: .)"
+}).action((opts, { log, fs }) => ls2({ ...opts, log, fs }));
+skills.command("add").description("add a skill to a project").arg("skill", t.string(), { description: "skill name" }).arg("dir", t.string(), {
+  default: ".",
+  description: "project to add it to (default: .)"
+}).action((opts, { log, fs }) => add2({ ...opts, log, fs }));
+skills.command("update").description("refresh the skills a project already has").arg("dir", t.string(), {
+  default: ".",
+  description: "project to refresh (default: .)"
+}).action((opts, { log, fs }) => update2({ ...opts, log, fs }));
 
 // index.ts
-await webappwiz.run({
-  fs: new NodeFs,
-  clock: new SystemClock,
-  glob: new NodeGlob
-});
+await webappwiz.run({ fs: new NodeFs, glob: new NodeGlob });

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@webappwiz/cli",
-	"version": "0.0.9",
-	"description": "The webappwiz CLI: judge code against rules and manage agent skills",
+	"version": "0.0.11",
+	"description": "The webappwiz CLI: keep rules and agent skills in a project, and divide a review up",
 	"license": "MIT",
 	"author": "Jared Johnson",
 	"repository": {
@@ -17,8 +17,8 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@webappwiz/rules": "^0.0.9",
-		"webappwiz": "^0.0.9"
+		"@webappwiz/rules": "^0.0.11",
+		"webappwiz": "^0.0.11"
 	},
 	"peerDependencies": {
 		"typescript": "^7"
@@ -33,14 +33,6 @@
 		"./webappwiz": {
 			"types": "./webappwiz.d.ts",
 			"default": "./webappwiz.js"
-		},
-		"./rules": {
-			"types": "./rules.d.ts",
-			"default": "./rules.js"
-		},
-		"./judge": {
-			"types": "./judge.d.ts",
-			"default": "./judge.js"
 		}
 	},
 	"bin": {

package/rules/add.d.ts

@@ -0,0 +1,7 @@
+import { type RulesProjectOptions } from "./rule-set.js";
+export interface AddOptions extends RulesProjectOptions {
+    /** The rule to install, as `rules ls` names it. */
+    rule: string;
+}
+/** Copies a shipped rule into the project, where it runs and can be edited. */
+export declare function add(opts: AddOptions): Promise<void>;

package/rules/ls.d.ts

@@ -0,0 +1,7 @@
+import { type RulesProjectOptions } from "./rule-set.js";
+/**
+ * Every rule there is: the project's own, validated, and the shipped ones it
+ * could add, one row each. A rule the project holds a copy of shows the
+ * version it came from beside the one that ships, so a stale copy is visible.
+ */
+export declare function ls(opts: RulesProjectOptions): Promise<void>;

package/rules/new.d.ts

@@ -0,0 +1,16 @@
+import { type Logger } from "webappwiz/log";
+import { type Fs } from "webappwiz/system";
+export interface NewOptions {
+    /** The rule's id, kebab case, and the directory it gets. */
+    name: string;
+    /** The project root: the directory holding `.wiz/rules`. */
+    dir: string;
+    log?: Logger;
+    fs?: Fs;
+}
+/**
+ * Scaffolds a rule to fill in: `.wiz/rules/<name>/RULE.md`, with every field
+ * a review needs and the headings a rule has to have. Refuses to overwrite a
+ * rule that is already there.
+ */
+export declare function newRule(opts: NewOptions): Promise<void>;

package/rules/review.d.ts

@@ -0,0 +1,21 @@
+import { type Logger } from "webappwiz/log";
+import type { Fs, Glob, Ps } from "webappwiz/system";
+export interface ReviewOptions {
+    /** The project root: where `.wiz/rules` is, and what paths are relative to. */
+    dir: string;
+    /** The git ref the change is measured from. */
+    since: string;
+    /** Files per block, at most. */
+    chunk: number;
+    log?: Logger;
+    fs?: Fs;
+    ps?: Ps;
+    glob?: Glob;
+}
+/**
+ * Divides a change into blocks of work, one rule over the changed files it
+ * matches, and prints them for a parent agent to hand to subagents. Nothing
+ * is spawned and no rule is quoted: each block names the rule's file, and the
+ * subagent reads it.
+ */
+export declare function review(opts: ReviewOptions): Promise<void>;

package/rules/rule-set.d.ts

@@ -0,0 +1,16 @@
+import type { Logger } from "webappwiz/log";
+import type { Fs } from "webappwiz/system";
+import type { Layout } from "../documents.js";
+/** Where a project keeps its rules: one directory per rule, holding `RULE.md`. */
+export declare const RULES: Layout;
+/** The project a rules command works on. */
+export interface RulesProjectOptions {
+    /** Its root: the directory holding `.wiz/rules`. */
+    dir: string;
+    log?: Logger;
+    fs?: Fs;
+    /** The rules on offer, id to `RULE.md`; the catalog by default. */
+    rules?: Record<string, string>;
+}
+/** The rules a command offers: what it was handed, else the catalog. */
+export declare const offered: (opts: RulesProjectOptions) => Record<string, string>;

package/rules/update.d.ts

@@ -0,0 +1,7 @@
+import { type RulesProjectOptions } from "./rule-set.js";
+/**
+ * Refreshes the shipped rules a project has copies of, and leaves the rules
+ * it wrote itself alone. Never adds one: a rule someone chose not to install
+ * should not arrive by way of an update.
+ */
+export declare function update(opts: RulesProjectOptions): Promise<void>;

package/skills/skill.d.ts

@@ -1,5 +1,6 @@
 import type { Logger } from "webappwiz/log";
 import type { Fs } from "webappwiz/system";
+import type { Layout } from "../documents.js";
 /** Skill name to the document a project installs under that name. */
 export type Skills = Record<string, string>;
 /**
@@ -12,7 +13,8 @@ export type Skills = Record<string, string>;
  * not happen to pick up.
  */
 export declare const bundled: Skills;
-export declare function versionOf(md: string): string | null;
+/** Where a project keeps its skills, the way every agent harness reads them. */
+export declare const SKILLS: Layout;
 /** The project a skills command works on. */
 export interface ProjectOptions {
     /** Its root: the directory holding `.agents/skills`. */
@@ -22,15 +24,3 @@ export interface ProjectOptions {
     /** The skills on offer; the ones this package ships by default. */
     skills?: Skills;
 }
-/**
- * Every skill on offer, name and document together, in the order they are
- * listed and installed in.
- */
-export declare function available(skills: Skills): Array<[string, string]>;
-/** What `copy` works through, once an action has resolved them. */
-export interface CopyOptions {
-    log: Logger;
-    fs: Fs;
-}
-/** Installs one skill into a project. */
-export declare function copy(name: string, doc: string, dir: string, opts: CopyOptions): Promise<void>;

package/update.d.ts

@@ -11,12 +11,14 @@ export interface UpdateOptions {
     fs?: Fs;
     /** The skills to refresh with; the ones this package ships by default. */
     skills?: Skills;
+    /** The rules to refresh with; the catalog by default. */
+    rules?: Record<string, string>;
 }
 /**
  * Pins every webappwiz dependency under `dir` to one version, so a project
  * never runs two of these packages built against different versions of each
  * other. They are released together, so there is only ever one right answer.
  * Installed skills are copies of files those packages ship, so they are
- * refreshed too.
+ * refreshed too, and so are the rules copied in from the catalog.
  */
 export declare function update(opts: UpdateOptions): Promise<void>;

package/webappwiz.d.ts

@@ -1,10 +1,8 @@
 import { type Deps } from "webappwiz/cmd";
 import type { Fs, Glob } from "webappwiz/system";
-import type { Clock } from "webappwiz/time";
 /** What webappwiz's commands are run with, on top of what any cli needs. */
 export interface CommandDeps extends Deps {
     fs: Fs;
-    clock: Clock;
     glob: Glob;
 }
 /**

package/webappwiz.js

@@ -1,19 +1,18 @@
 import {
   add,
+  add1 as add2,
   ls,
+  ls1 as ls2,
+  newRule,
+  review,
   update,
   update1 as update2,
+  update2 as update3,
   version
-} from "./index-g81gg9gz.js";
-import {
-  JUDGE_RULES
-} from "./index-htwb54c6.js";
-import {
-  JudgeCommands
-} from "./index-pyjg1rtk.js";
+} from "./index-d8p57dk5.js";
 
 // webappwiz.ts
-import { AGENTS } from "@webappwiz/rules";
+import { DEFAULT_CHUNK } from "@webappwiz/rules";
 import { cli } from "webappwiz/cmd";
 import { t } from "webappwiz/t";
 var webappwiz = cli("webappwiz");
@@ -23,45 +22,47 @@ webappwiz.command("update").description("pin every webappwiz dependency in a tre
 }).option("version", t.string(), {
   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, { 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: .)"
-}).option("agent", t.optional(t.enum(Object.keys(AGENTS))), {
-  description: "model to check 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 prompts and run no agent at all"
-}).option("chunk", t.number(), {
-  default: 25,
-  description: "files per review"
-}).option("since", t.optional(t.string()), {
-  description: "only check files added or changed since this git ref"
-}).option("concurrency-override", t.optional(t.number()), {
-  description: "agent calls in flight at once, over the config's concurrency"
-}).option("ci", t.boolean(), {
-  default: false,
-  description: "line-by-line output with no live progress block"
-}).action((opts, deps) => judge(deps).judge(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));
-var skillsGroup = webappwiz.group("skills").description("manage webappwiz agent skills in .agents/skills");
-skillsGroup.command("ls").description("list the skills there are, and what the project has of them").arg("dir", t.string(), {
+}).action((opts, { log, fs }) => update3({ ...opts, log, fs }));
+var rules = webappwiz.group("rules").description("keep rules in .wiz/rules, and divide a review of them up");
+rules.command("ls").description("list the rules there are, and what the project has of them").arg("dir", t.string(), {
   default: ".",
   description: "project to inspect (default: .)"
 }).action((opts, { log, fs }) => ls({ ...opts, log, fs }));
-skillsGroup.command("add").description("add a skill to a project").arg("skill", t.string(), { description: "skill name" }).arg("dir", t.string(), {
+rules.command("new").description("scaffold a rule to fill in, under .wiz/rules/<name>").arg("name", t.string(), { description: "rule id, kebab case" }).arg("dir", t.string(), {
+  default: ".",
+  description: "project to add it to (default: .)"
+}).action((opts, { log, fs }) => newRule({ ...opts, log, fs }));
+rules.command("add").description("copy a shipped rule into the project").arg("rule", t.string(), { description: "rule id, as `rules ls` lists it" }).arg("dir", t.string(), {
   default: ".",
   description: "project to add it to (default: .)"
 }).action((opts, { log, fs }) => add({ ...opts, log, fs }));
-skillsGroup.command("update").description("refresh the skills a project already has").arg("dir", t.string(), {
+rules.command("update").description("refresh the shipped rules the project has copies of").arg("dir", t.string(), {
   default: ".",
   description: "project to refresh (default: .)"
 }).action((opts, { log, fs }) => update({ ...opts, log, fs }));
+rules.command("review").description("print one block of review work per rule the change touches").arg("dir", t.string(), {
+  default: ".",
+  description: "project to review (default: .)"
+}).option("since", t.string(), {
+  default: "HEAD",
+  description: "git ref the change is measured from (default: HEAD)"
+}).option("chunk", t.number(), {
+  default: DEFAULT_CHUNK,
+  description: "files per block, at most"
+}).action((opts, { log, fs, ps, glob }) => review({ ...opts, log, fs, ps, glob }));
+var skills = webappwiz.group("skills").description("manage webappwiz agent skills in .agents/skills");
+skills.command("ls").description("list the skills there are, and what the project has of them").arg("dir", t.string(), {
+  default: ".",
+  description: "project to inspect (default: .)"
+}).action((opts, { log, fs }) => ls2({ ...opts, log, fs }));
+skills.command("add").description("add a skill to a project").arg("skill", t.string(), { description: "skill name" }).arg("dir", t.string(), {
+  default: ".",
+  description: "project to add it to (default: .)"
+}).action((opts, { log, fs }) => add2({ ...opts, log, fs }));
+skills.command("update").description("refresh the skills a project already has").arg("dir", t.string(), {
+  default: ".",
+  description: "project to refresh (default: .)"
+}).action((opts, { log, fs }) => update2({ ...opts, log, fs }));
 export {
   webappwiz
 };

package/changed.d.ts

@@ -1,30 +0,0 @@
-import { type Ps } from "webappwiz/system";
-/**
- * The files git says are new or changed in `dir` since `ref`, named the way a
- * rule's glob is: relative to `dir`, so a run over one package of a repo sees
- * its own paths rather than the repo's.
- *
- * Deletions are left out. A violation quotes the offending line from disk, so a
- * file that is gone has nothing to read and nothing to report.
- */
-export interface ChangedOptions {
-    /** What git is spawned through; the real process by default. */
-    ps?: Ps;
-}
-export declare function changed(dir: string, ref: string, opts?: ChangedOptions): Promise<Set<string>>;
-/**
- * The change itself, as a patch: everything in `dir` that differs from `ref`,
- * committed or not, and the paths of the files git has never been told about,
- * which no diff reaches.
- *
- * The new files are named rather than shown because a reader of this patch is
- * an agent with the working directory in front of it, and it can open them.
- */
-export interface DiffOptions {
-    /** What git is spawned through; the real process by default. */
-    ps?: Ps;
-}
-export declare function diff(dir: string, ref: string, opts?: DiffOptions): Promise<{
-    patch: string;
-    added: string[];
-}>;