@clemrig/factory 0.8.0 → 0.9.0

package/README.md

@@ -2,7 +2,7 @@
 
 Take a repository's issues from spec to pull request with agents, watch them on a local Kanban, and run the repository's own workflows.
 
-An idea becomes an issue; the operator specifies it with an agent; one run then plans, implements, checks and reviews it and opens its pull request; the operator approves by merging. Each of these steps is a workflow: an explicit graph of **CODE** stages (code), **AGENT** stages (an agent run by [Claude Code](https://code.claude.com), [Codex](https://developers.openai.com/codex) or [Pi](https://www.npmjs.com/package/@earendil-works/pi-coding-agent)) and **GATE** stages that accept a result or send it back for correction, a bounded number of times. The factory's workflows come with this package and are the same in every repository; the repository's `.factory/` folder adapts them (checks, budget, agents) and holds its own workflows, committed with it and run without a build step.
+An idea becomes an issue; the operator specifies it with an agent; one run then plans, implements, checks and reviews it and opens its pull request; the operator approves by merging. Between features, a `tidy` run chooses a cleanup of recent code and takes it the same way to a pull request. Each of these steps is a workflow: an explicit graph of **CODE** stages (code), **AGENT** stages (an agent run by [Claude Code](https://code.claude.com), [Codex](https://developers.openai.com/codex) or [Pi](https://www.npmjs.com/package/@earendil-works/pi-coding-agent)) and **GATE** stages that accept a result or send it back for correction, a bounded number of times. The factory's workflows come with this package and are the same in every repository; the repository's `.factory/` folder adapts them (checks, budget, agents) and holds its own workflows, committed with it and run without a build step.
 
 ## Requirements
 
@@ -107,6 +107,20 @@ No text of the run opens a code block or a comment that would swallow the rest o
 
 When the review still asks for changes after its last round, the run stops; `factory resume <RUN_ID>` gives it one more round, and `factory resume <RUN_ID> --accept-review` delivers it as the review left it, on your decision: the pull request then opens with a warning and lists, under Needs your judgment, the unmet criteria and blocking findings left open. What comes from the run (the spec, the criteria, the checks, the review, the deviations, the risks) is put together by the factory, never rewritten by an agent; the implementing agent's own summary is not shown. In a public repository, anyone can see the screenshots: the describing agent shows only test or seed data. The screenshots are kept in the run's folder and uploaded once the pull request is open; the body names them by file only, never by a path of the machine. A failed upload leaves the pull request recorded, saying again that its screenshots are coming, and fails the run; `factory resume` only uploads them again.
 
+### The `tidy` run
+
+Working fast with agents leaves waste in the code. `factory run tidy` removes it a little at a time, between features: it chooses the cleanup of the code that comes first, writes it as an issue with its spec, then delivers it as `implement` delivers an issue. Run it as often as you like, every night for instance: it opens one pull request at most, and none while the previous one waits for you. Scheduling it is up to you.
+
+| Stage | What it does |
+|---|---|
+| `prepare` | Refuses to start for the same reasons as `implement`, the spec's aside: the main branch, changes in progress, `origin`, `gh`. Then the repository's checks must pass on the main branch. Then, while a cleanup's pull request (a branch `tidy-…`) is open, ends without calling any agent: one cleanup at a time. Otherwise it creates the run's own branch, `tidy-<run id>`, from the starting commit, as `implement` creates the issue's: no other run starts or resumes on it |
+| `survey` | Reads what the scout needs: recent code, that is the commits of the main branch since the last cleanup merged (the last commit that brought a `.factory/issues/tidy-*/plan.md` to it), or of the last 14 days before the first one, 200 at most, merges aside, with their files and how often each changed; the files of the open pull requests; the cleanups closed without being merged |
+| `scout`, `scouted` | The `scout` agent, with the `plan` profile, explores recent code first, then the rest of the repository, compares candidates and answers with the cleanup that comes first, or none. A cleanup changes nothing observable: it removes the waste that fast changes leave (duplicates, dead code, needless defenses, functions too long to follow, departures from the conventions, stale comments and documentation, weak tests), leaves the files of the open pull requests alone and proposes no cleanup like those the operator closed. It answers with the cleanup's `issue.md` and `spec.md`: the gate reads them as `issue-save` does and refuses a dependency, since a cleanup starts from the main branch alone, otherwise the answer goes back to the scout (2 times at most). When none is worth a pull request, the run goes back to the main branch, deletes its empty branch (`release`) and ends there |
+| `commit-spec` | Commits the cleanup's `issue.md` and `spec.md` on the run's branch (`docs: spec tidy-<run id>`) |
+| `plan` … `attach` | As in `implement`, from that spec; the pull request links the spec at its own commit |
+
+The pull request is your approval: merge it, or close it, and the scout does not propose that cleanup again. A defect the scout finds is named in the issue, never fixed by a cleanup. Until it is merged, a cleanup lives on its branch only, so the Kanban does not show it: follow its run under `/runs/`; once merged, it is an issue like the others, in Done. The run needs the repository on its main branch without changes, as `implement` does: run it from a clone kept for it rather than the one you work in. A cleanup is named after its run: its branch and its folder `.factory/issues/tidy-<run id>/` lead to the run under `/runs/`, and its title is that of its pull request and `issue.md`. A failed run stops on its branch and resumes as `implement` does.
+
 ### Kanban
 
 `factory ui` shows the repository's issues in six columns. No status is stored: each column follows from facts.
@@ -118,9 +132,9 @@ When the review still asks for changes after its last round, the run stops; `fac
 | Executing | the last `implement` run is running |
 | Blocked | the last `implement` run failed |
 | To Approve | the last `implement` run completed, and `plan.md` is not on the main branch |
-| Done | `plan.md` is on the main branch |
+| Done | `plan.md` is on the main branch, a merged cleanup's too |
 
-The files are read in the working tree, the main branch is the local branch `origin/HEAD` names, and runs are those of `.factory/runs/`. A card shows the issue's title (the `# ` line of `issue.md`; its slug without one), its slug, the newest pull request its `implement` runs recorded (a new run updates the same one) and, for each workflow shown on cards, its last run on the issue: its steps (done, current, failed or to come), its duration and its estimated cost. `implement` shows six steps: prepare, plan, implement, verify, review and deliver (the description and the screenshots' upload included). The title opens the newest of these runs in the runs dashboard. The most recently active issue comes first in its column; the page refreshes every 2 seconds. What may leave the board incomplete is listed above it: a `factory.ts` that does not load (`factory ui` loads it when it starts: restart it once fixed), an unreadable `issue.md` or run, or a main branch Git cannot read (`origin/HEAD` missing: `git remote set-head origin --auto` sets it).
+The files are read in the working tree, the main branch is the local branch `origin/HEAD` names, and runs are those of `.factory/runs/`. A card shows the issue's title (the `# ` line of `issue.md`; its slug without one), its slug, the dependencies of its spec not delivered yet (none once the issue is delivered), the newest pull request its `implement` runs recorded (a new run updates the same one) and, for each workflow shown on cards, its last run on the issue: its steps (done, current, failed or to come, each with the corrections that sent the run back into it), its duration and its estimated cost. `implement` shows six steps: prepare, plan, implement, verify, review and deliver (the description and the screenshots' upload included). The title opens the newest of these runs in the runs dashboard. The most recently active issue comes first in its column; the page refreshes every 2 seconds. What may leave the board incomplete is listed above it: a `factory.ts` that does not load (`factory ui` loads it when it starts: restart it once fixed), an unreadable `issue.md` or run, an undelivered issue's `spec.md` that cannot be read or is invalid (its dependencies are then not shown), or a main branch Git cannot read (`origin/HEAD` missing: `git remote set-head origin --auto` sets it).
 
 ## Adapt the factory
 
@@ -159,13 +173,14 @@ export default defineFactory({
 ```
 
 - `verify`: the repository's checks, at least one: `{ name, command, timeoutMs }`, the command an executable and its arguments, run without a shell in the repository. They run in order and stop at the first failure: on the main branch before the run's agents, then after each implementation. A check may add `when`, globs of the files whose change makes it run (`when: ["scripts/*deploy*", "caddy/**"]`): it is for a long validation, such as a rehearsal, that only some changes call for; it never runs on the main branch, and after an implementation only if a file changed since the plan matches it.
-- `maxCostUsd`: the budget of an `implement` run, in USD; `factory run implement --issue <slug> --max-cost-usd 40` changes it for one run. Codex announces no cost, so the agents of `implement` (`plan`, `implement`, `review`, `describe`) cannot run with Codex.
-- `profiles`: the agents of `plan`, `implement` and `review`, and optionally `describe`, which writes the pull request (the review's profile without it; see Profiles below). Each profile decides what its agent may do: here, `plan` only reads the repository, handing areas of a large spec to sub-agents (`Agent`); `implement` also edits it and runs commands, and `permissionMode: "dontAsk"` refuses any other tool instead of waiting for a permission prompt; `review` and `describe` read it and run commands, such as tests, reproductions or the application in a browser (they must not change the repository: the run puts back what they change). `Agent` in the review's tools lets it split the review of a large change between sub-agents. Without a way to start the application and a browser, `describe` takes no screenshot and says why.
+- `maxCostUsd`: the budget of an `implement` run, in USD; `factory run implement --issue <slug> --max-cost-usd 40` changes it for one run. Codex announces no cost, so the agents of `implement` and `tidy` (`plan`, `implement`, `review`, `describe`) cannot run with Codex.
+- `tidy` (optional): `{ maxCostUsd }`, the budget of a `tidy` run, that of `implement` when absent; `--max-cost-usd` changes it for one run.
+- `profiles`: the agents of `plan`, `implement` and `review`, and optionally `describe`, which writes the pull request (the review's profile without it; see Profiles below); `tidy` runs its scout with the `plan` profile. Each profile decides what its agent may do: here, `plan` only reads the repository, handing areas of a large spec to sub-agents (`Agent`); `implement` also edits it and runs commands, and `permissionMode: "dontAsk"` refuses any other tool instead of waiting for a permission prompt; `review` and `describe` read it and run commands, such as tests, reproductions or the application in a browser (they must not change the repository: the run puts back what they change). `Agent` in the review's tools lets it split the review of a large change between sub-agents. Without a way to start the application and a browser, `describe` takes no screenshot and says why.
 - `workflows` (optional): the repository's own workflows.
 
-`defineFactory` lists every problem at once, each agent's profile checked on its own even when other fields are wrong; it also refuses a workflow not made by `defineWorkflow`, or named like another, `issue-save` and `implement` included. Any other default export of `factory.ts` is refused.
+`defineFactory` lists every problem at once, each agent's profile checked on its own even when other fields are wrong; it also refuses a workflow not made by `defineWorkflow`, or named like another, `issue-save`, `implement` and `tidy` included. Any other default export of `factory.ts` is refused.
 
-The rest is the factory's, the same in every repository: the issue folders and their formats, `issue-save`, the `implement` run (its starting conditions, stages, correction maximums, diff limit, branch and pull request), its prompts, shipped with the package, the Kanban and its columns. None of these can be changed by a repository yet. The `issue` skill is copied once by `factory init`: a later factory version does not update it.
+The rest is the factory's, the same in every repository: the issue folders and their formats, `issue-save`, the `implement` and `tidy` runs (their starting conditions, stages, correction maximums, diff limit, branch and pull request), their agents' prompts, shipped with the package (`prompts/`), the Kanban and its columns. None of these can be changed by a repository yet. The `issue` skill is copied once by `factory init`: a later factory version does not update it.
 
 ## Write a workflow
 
@@ -295,7 +310,7 @@ The run executes in the foreground. The terminal shows each stage's start and en
 
 **Ctrl-C** stops the run's child processes and ends the run as `failed` with its cause before its next stage, or instead of completing; a second Ctrl-C kills the remaining child processes and quits at once (exit code 130, the run may then stay `running`); commands that Codex runs in their own process groups are stopped by Codex on the first Ctrl-C, not by the second.
 
-**Resume.** `factory resume RUN_ID` runs a failed run again in the foreground, under the same ID, at the stage where it stopped (with `--accept-review`, an `implement` run its review stopped by asking for changes is delivered as the review left it: see The pull request): its input, data, corrections, cost and agents' conversations are kept, the stage's journal starts with why the run had stopped (`Resumed after: …`), and its agent is told so and asked to continue rather than start over. The workflow is the one `factory.ts` defines now, its prompts are read again, and its runtimes are checked as for `factory run`. The budget is the one given with `--max-cost-usd`, else the run's own, else the workflow's; a run that already spent it does not resume without a higher one. The run's budget still counts its whole cost, but calls that announced no cost before the resumption, such as a call stopped by its timeout, no longer fail it; their cost counts only if the runtime includes it in the total it later announces for the resumed session, and the run's cost stays marked partial. The run's duration leaves out the time it spent stopped. Only a failed run resumes, and by one command at a time: a run left `running` after a second Ctrl-C does not, and a workflow may refuse to resume from the state of the repository, as `implement` does. A resumed stage starts over: what it did before it failed must be safe to do again. A run stopped by a correction maximum resumes at the stage that asked for it, which may then take one more correction: the checks may pass after a fix by hand, and a gate judging the same answer sends it back once more.
+**Resume.** `factory resume RUN_ID` runs a failed run again in the foreground, under the same ID, at the stage where it stopped (with `--accept-review`, an `implement` or `tidy` run its review stopped by asking for changes is delivered as the review left it: see The pull request): its input, data, corrections, cost and agents' conversations are kept, the stage's journal starts with why the run had stopped (`Resumed after: …`), and its agent is told so and asked to continue rather than start over. The workflow is the one `factory.ts` defines now, its prompts are read again, and its runtimes are checked as for `factory run`. The budget is the one given with `--max-cost-usd`, else the run's own, else the workflow's; a run that already spent it does not resume without a higher one. The run's budget still counts its whole cost, but calls that announced no cost before the resumption, such as a call stopped by its timeout, no longer fail it; their cost counts only if the runtime includes it in the total it later announces for the resumed session, and the run's cost stays marked partial. The run's duration leaves out the time it spent stopped. Only a failed run resumes, and by one command at a time: a run left `running` after a second Ctrl-C does not, and a workflow may refuse to resume from the state of the repository, as `implement` does. A resumed stage starts over: what it did before it failed must be safe to do again. A run stopped by a correction maximum resumes at the stage that asked for it, which may then take one more correction: the checks may pass after a fix by hand, and a gate judging the same answer sends it back once more.
 
 `factory status` lists the repository's runs or shows one; while `factory.ts` cannot load, `status` and `ui` say so and still show the runs with the default rendering. `factory ui` serves on `127.0.0.1` (port chosen by the system unless `--port` sets it, URL printed; Ctrl-C stops it) the Kanban on `/` and, under `/runs/`, a read-only dashboard of the repository's runs: the runs with workflow, status, current stage, duration, estimated cost and last update, and for a run its runtimes, its launches (the first one, then each resumption), each with a strip of its stages sized by duration, then its stages beside the selected one's detail (start, duration, outcome, the agent call's model, settings, tools and tokens as they come, a folded Output section with the agent's answer and the correction sent back or not, the journal and the tool calls), the correction counters and the result. It follows a running run as it goes. To watch on another machine, forward the same port through SSH: `ssh -L 4321:127.0.0.1:4321 machine`, run `factory ui --port 4321` there, and open `http://localhost:4321/`. `factory doctor` checks Node, loads the workflows, reads their prompts, lists each workflow's options and checks each runtime's executable, version and connection, without any provider request.
 
@@ -314,6 +329,7 @@ Tokens and costs are the figures each runtime announces: Claude Code announces a
 - Runtimes are not sandboxes: the implementing agent's tools can run any command its profile allows, bounded only by its prompt, the repository's agent instructions and the run's checks.
 - The rule that a public repository's screenshots show only test or seed data holds only through the describing agent's prompt. The factory does not check the Mermaid syntax: GitHub shows an error in place of an invalid diagram.
 - Screenshots show the new state only, never the previous one; no videos.
+- `tidy` has no scheduler of its own: cron, launchd or a CI runner starts it. Its scout reads at most 200 recent commits and the latest 100 open and 100 closed pull requests, with the first 100 files of each open one, as `gh` lists them, and a defect it finds is only named in the cleanup's issue.
 - Not in this version: changing the factory's prompts, correction maximums, pull request format or columns from a repository; columns added by a workflow; backlog priority; actions or drag-and-drop on the board; resuming a run left `running` after its process was killed, detached runs, logs `--follow` and stop; local settings or profile overrides; graph drawing in the dashboard; runs or issues of several repositories; `factory new`; a shared stage library; a test harness for workflows; CI, cron or Git hook triggers; notifications; Windows.
 
 ## License

package/dist/app/board.js

@@ -1,12 +1,38 @@
 import { boardOf, } from "../domain/board.js";
 import { issueTitle } from "../domain/issue.js";
+import { parseSpec } from "../domain/spec.js";
 import { branchRef, defaultBranch, filesIn } from "../system/git.js";
-import { artifactPath, issuesPath, listIssueFolders, } from "../system/issues.js";
+import { artifactPath, issuesPath, listIssueFolders, readArtifact, } from "../system/issues.js";
 import { listRuns } from "./runs.js";
 const message = (error) => error instanceof Error ? error.message : String(error);
-/** The workflows that take an issue (`--issue`) and declare their steps. */
+/** The workflows that take an issue (`--issue`) and declare their steps, with where their retry edges go back to. */
 function shownWorkflows(factory) {
-    return [...factory.workflows.values()].flatMap(({ name, input, steps }) => steps && input?.shape.issue ? [{ name, steps }] : []);
+    return [...factory.workflows.values()].flatMap(({ name, input, steps, nodes }) => {
+        if (!steps || !input?.shape.issue)
+            return [];
+        const corrections = {};
+        for (const [stage, { on }] of Object.entries(nodes))
+            for (const [outcome, edge] of Object.entries(on))
+                if (edge !== null && typeof edge === "object")
+                    corrections[`${stage}.${outcome}`] = edge.to;
+        return [{ name, steps, corrections }];
+    });
+}
+/** The issues `slug`'s spec depends on; a spec that cannot be read or is invalid is a problem, and none are shown. */
+async function dependenciesOf(dir, slug, problems) {
+    try {
+        const text = await readArtifact(dir, slug, "spec");
+        if (text === undefined)
+            return [];
+        const spec = parseSpec(text, slug);
+        if (spec.ok)
+            return spec.value.dependencies;
+        problems.push(`Issue ${slug}: its dependencies cannot be shown, spec.md is invalid (/issue ${slug} fixes it): ${spec.problems.join("; ")}`);
+    }
+    catch (error) {
+        problems.push(`Issue ${slug}: spec.md cannot be read: ${message(error)}`);
+    }
+    return [];
 }
 /**
  * The Kanban as the repository is now: its issues, the main branch and its
@@ -23,7 +49,7 @@ export async function readBoard(factory, loadError) {
     }
     catch (error) {
         // A remote added by hand has no origin/HEAD until git sets it.
-        problems.push(`Done issues cannot be told apart (the main branch is origin/HEAD; git remote set-head origin --auto sets it): ${message(error)}`);
+        problems.push(`Done issues and delivered dependencies cannot be told apart (the main branch is origin/HEAD; git remote set-head origin --auto sets it): ${message(error)}`);
     }
     const runs = [];
     for (const entry of await listRuns(factory))
@@ -31,17 +57,26 @@ export async function readBoard(factory, loadError) {
             runs.push(entry.state);
         else
             problems.push(`Run ${entry.id} cannot be read: ${entry.error}`);
-    const issues = (await listIssueFolders(factory.dir)).map((folder) => {
+    const issues = [];
+    for (const folder of await listIssueFolders(factory.dir)) {
         if (folder.unreadable !== undefined)
             problems.push(`Issue ${folder.slug}: issue.md cannot be read: ${folder.unreadable}`);
-        return {
+        const isDelivered = delivered.has(artifactPath(folder.slug, "plan"));
+        issues.push({
             slug: folder.slug,
             title: (folder.issue && issueTitle(folder.issue)) || folder.slug,
             hasSpec: folder.hasSpec,
-            delivered: delivered.has(artifactPath(folder.slug, "plan")),
+            // A delivered issue's spec may predate today's format: it waits for nothing.
+            dependencies: folder.hasSpec && !isDelivered
+                ? (await dependenciesOf(factory.dir, folder.slug, problems)).map((slug) => ({
+                    slug,
+                    delivered: delivered.has(artifactPath(slug, "plan")),
+                }))
+                : [],
+            delivered: isDelivered,
             changedAt: folder.changedAt,
-        };
-    });
+        });
+    }
     return boardOf(issues, runs, shownWorkflows(factory), problems);
 }
 //# sourceMappingURL=board.js.map

package/dist/app/board.js.map

@@ -1 +1 @@
-{"version":3,"file":"board.js","sourceRoot":"","sources":["../../src/app/board.ts"],"names":[],"mappings":"AACA,OAAO,EACL,OAAO,GAGR,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EAAE,UAAU,EAAE,MAAM,oBAAoB,CAAC;AAEhD,OAAO,EAAE,SAAS,EAAE,aAAa,EAAE,OAAO,EAAE,MAAM,kBAAkB,CAAC;AACrE,OAAO,EACL,YAAY,EACZ,UAAU,EACV,gBAAgB,GACjB,MAAM,qBAAqB,CAAC;AAC7B,OAAO,EAAE,QAAQ,EAAE,MAAM,WAAW,CAAC;AAErC,MAAM,OAAO,GAAG,CAAC,KAAc,EAAE,EAAE,CACjC,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;AAEzD,4EAA4E;AAC5E,SAAS,cAAc,CAAC,OAAgB;IACtC,OAAO,CAAC,GAAG,OAAO,CAAC,SAAS,CAAC,MAAM,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,EAAE,EAAE,CACxE,KAAK,IAAI,KAAK,EAAE,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,CACrD,CAAC;AACJ,CAAC;AAED;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,SAAS,CAC7B,OAAgB,EAChB,SAAkB;IAElB,MAAM,QAAQ,GAAG,SAAS;QACxB,CAAC,CAAC,CAAC,2DAA2D,SAAS,EAAE,CAAC;QAC1E,CAAC,CAAC,EAAE,CAAC;IACP,IAAI,SAAS,GAAG,IAAI,GAAG,EAAU,CAAC;IAClC,IAAI,CAAC;QACH,MAAM,IAAI,GAAG,MAAM,aAAa,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;QAC9C,SAAS,GAAG,IAAI,GAAG,CACjB,MAAM,OAAO,CAAC,OAAO,CAAC,GAAG,EAAE,SAAS,CAAC,IAAI,CAAC,EAAE,UAAU,CAAC,CACxD,CAAC;IACJ,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,+DAA+D;QAC/D,QAAQ,CAAC,IAAI,CACX,iHAAiH,OAAO,CAAC,KAAK,CAAC,EAAE,CAClI,CAAC;IACJ,CAAC;IACD,MAAM,IAAI,GAAe,EAAE,CAAC;IAC5B,KAAK,MAAM,KAAK,IAAI,MAAM,QAAQ,CAAC,OAAO,CAAC;QACzC,IAAI,OAAO,IAAI,KAAK;YAAE,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;;YACxC,QAAQ,CAAC,IAAI,CAAC,OAAO,KAAK,CAAC,EAAE,oBAAoB,KAAK,CAAC,KAAK,EAAE,CAAC,CAAC;IACvE,MAAM,MAAM,GAAG,CAAC,MAAM,gBAAgB,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,MAAM,EAAE,EAAE;QAClE,IAAI,MAAM,CAAC,UAAU,KAAK,SAAS;YACjC,QAAQ,CAAC,IAAI,CACX,SAAS,MAAM,CAAC,IAAI,8BAA8B,MAAM,CAAC,UAAU,EAAE,CACtE,CAAC;QACJ,OAAO;YACL,IAAI,EAAE,MAAM,CAAC,IAAI;YACjB,KAAK,EAAE,CAAC,MAAM,CAAC,KAAK,IAAI,UAAU,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,MAAM,CAAC,IAAI;YAChE,OAAO,EAAE,MAAM,CAAC,OAAO;YACvB,SAAS,EAAE,SAAS,CAAC,GAAG,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;YAC3D,SAAS,EAAE,MAAM,CAAC,SAAS;SAC5B,CAAC;IACJ,CAAC,CAAC,CAAC;IACH,OAAO,OAAO,CAAC,MAAM,EAAE,IAAI,EAAE,cAAc,CAAC,OAAO,CAAC,EAAE,QAAQ,CAAC,CAAC;AAClE,CAAC"}
+{"version":3,"file":"board.js","sourceRoot":"","sources":["../../src/app/board.ts"],"names":[],"mappings":"AACA,OAAO,EACL,OAAO,GAIR,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EAAE,UAAU,EAAE,MAAM,oBAAoB,CAAC;AAChD,OAAO,EAAE,SAAS,EAAE,MAAM,mBAAmB,CAAC;AAE9C,OAAO,EAAE,SAAS,EAAE,aAAa,EAAE,OAAO,EAAE,MAAM,kBAAkB,CAAC;AACrE,OAAO,EACL,YAAY,EACZ,UAAU,EACV,gBAAgB,EAChB,YAAY,GACb,MAAM,qBAAqB,CAAC;AAC7B,OAAO,EAAE,QAAQ,EAAE,MAAM,WAAW,CAAC;AAErC,MAAM,OAAO,GAAG,CAAC,KAAc,EAAE,EAAE,CACjC,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;AAEzD,qHAAqH;AACrH,SAAS,cAAc,CAAC,OAAgB;IACtC,OAAO,CAAC,GAAG,OAAO,CAAC,SAAS,CAAC,MAAM,EAAE,CAAC,CAAC,OAAO,CAC5C,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,KAAK,EAAE,EAAE,EAAE;QAChC,IAAI,CAAC,KAAK,IAAI,CAAC,KAAK,EAAE,KAAK,CAAC,KAAK;YAAE,OAAO,EAAE,CAAC;QAC7C,MAAM,WAAW,GAA2B,EAAE,CAAC;QAC/C,KAAK,MAAM,CAAC,KAAK,EAAE,EAAE,EAAE,EAAE,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC;YACjD,KAAK,MAAM,CAAC,OAAO,EAAE,IAAI,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,EAAE,CAAC;gBAC9C,IAAI,IAAI,KAAK,IAAI,IAAI,OAAO,IAAI,KAAK,QAAQ;oBAC3C,WAAW,CAAC,GAAG,KAAK,IAAI,OAAO,EAAE,CAAC,GAAG,IAAI,CAAC,EAAE,CAAC;QACnD,OAAO,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,WAAW,EAAE,CAAC,CAAC;IACxC,CAAC,CACF,CAAC;AACJ,CAAC;AAED,sHAAsH;AACtH,KAAK,UAAU,cAAc,CAC3B,GAAW,EACX,IAAY,EACZ,QAAkB;IAElB,IAAI,CAAC;QACH,MAAM,IAAI,GAAG,MAAM,YAAY,CAAC,GAAG,EAAE,IAAI,EAAE,MAAM,CAAC,CAAC;QACnD,IAAI,IAAI,KAAK,SAAS;YAAE,OAAO,EAAE,CAAC;QAClC,MAAM,IAAI,GAAG,SAAS,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC;QACnC,IAAI,IAAI,CAAC,EAAE;YAAE,OAAO,IAAI,CAAC,KAAK,CAAC,YAAY,CAAC;QAC5C,QAAQ,CAAC,IAAI,CACX,SAAS,IAAI,kEAAkE,IAAI,eAAe,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAC7H,CAAC;IACJ,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,QAAQ,CAAC,IAAI,CAAC,SAAS,IAAI,6BAA6B,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;IAC5E,CAAC;IACD,OAAO,EAAE,CAAC;AACZ,CAAC;AAED;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,SAAS,CAC7B,OAAgB,EAChB,SAAkB;IAElB,MAAM,QAAQ,GAAG,SAAS;QACxB,CAAC,CAAC,CAAC,2DAA2D,SAAS,EAAE,CAAC;QAC1E,CAAC,CAAC,EAAE,CAAC;IACP,IAAI,SAAS,GAAG,IAAI,GAAG,EAAU,CAAC;IAClC,IAAI,CAAC;QACH,MAAM,IAAI,GAAG,MAAM,aAAa,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;QAC9C,SAAS,GAAG,IAAI,GAAG,CACjB,MAAM,OAAO,CAAC,OAAO,CAAC,GAAG,EAAE,SAAS,CAAC,IAAI,CAAC,EAAE,UAAU,CAAC,CACxD,CAAC;IACJ,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,+DAA+D;QAC/D,QAAQ,CAAC,IAAI,CACX,4IAA4I,OAAO,CAAC,KAAK,CAAC,EAAE,CAC7J,CAAC;IACJ,CAAC;IACD,MAAM,IAAI,GAAe,EAAE,CAAC;IAC5B,KAAK,MAAM,KAAK,IAAI,MAAM,QAAQ,CAAC,OAAO,CAAC;QACzC,IAAI,OAAO,IAAI,KAAK;YAAE,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;;YACxC,QAAQ,CAAC,IAAI,CAAC,OAAO,KAAK,CAAC,EAAE,oBAAoB,KAAK,CAAC,KAAK,EAAE,CAAC,CAAC;IACvE,MAAM,MAAM,GAAiB,EAAE,CAAC;IAChC,KAAK,MAAM,MAAM,IAAI,MAAM,gBAAgB,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC;QACzD,IAAI,MAAM,CAAC,UAAU,KAAK,SAAS;YACjC,QAAQ,CAAC,IAAI,CACX,SAAS,MAAM,CAAC,IAAI,8BAA8B,MAAM,CAAC,UAAU,EAAE,CACtE,CAAC;QACJ,MAAM,WAAW,GAAG,SAAS,CAAC,GAAG,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC,CAAC;QACrE,MAAM,CAAC,IAAI,CAAC;YACV,IAAI,EAAE,MAAM,CAAC,IAAI;YACjB,KAAK,EAAE,CAAC,MAAM,CAAC,KAAK,IAAI,UAAU,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,MAAM,CAAC,IAAI;YAChE,OAAO,EAAE,MAAM,CAAC,OAAO;YACvB,6EAA6E;YAC7E,YAAY,EACV,MAAM,CAAC,OAAO,IAAI,CAAC,WAAW;gBAC5B,CAAC,CAAC,CAAC,MAAM,cAAc,CAAC,OAAO,CAAC,GAAG,EAAE,MAAM,CAAC,IAAI,EAAE,QAAQ,CAAC,CAAC,CAAC,GAAG,CAC5D,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;oBACT,IAAI;oBACJ,SAAS,EAAE,SAAS,CAAC,GAAG,CAAC,YAAY,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;iBACrD,CAAC,CACH;gBACH,CAAC,CAAC,EAAE;YACR,SAAS,EAAE,WAAW;YACtB,SAAS,EAAE,MAAM,CAAC,SAAS;SAC5B,CAAC,CAAC;IACL,CAAC;IACD,OAAO,OAAO,CAAC,MAAM,EAAE,IAAI,EAAE,cAAc,CAAC,OAAO,CAAC,EAAE,QAAQ,CAAC,CAAC;AAClE,CAAC"}

package/dist/app/define.d.ts

@@ -19,9 +19,9 @@ export interface FactoryWorkflows {
 /** The workflows of a factory returned by `defineFactory`; nothing for any other value. */
 export declare function workflowsOf(value: unknown): FactoryWorkflows | undefined;
 /**
- * A repository's factory: its workflows (`issue-save`, `implement`) adapted to
- * the repository's checks, budget and agents, then its own workflows. Every
- * problem is listed at once.
+ * A repository's factory: its workflows (`issue-save`, `implement`, `tidy`)
+ * adapted to the repository's checks, budgets and agents, then its own
+ * workflows. Every problem is listed at once.
  */
 export declare function defineFactory(definition: FactoryDefinition): DefinedFactory;
 export {};

package/dist/app/define.js

@@ -1,8 +1,10 @@
 import { isWorkflow, profileProblems } from "@clemrig/flow";
 import { definitionProblems, } from "../domain/definition.js";
 import { DELIVERY } from "../domain/delivery.js";
+import { TIDY } from "../domain/tidy.js";
 import { implement } from "./workflows/implement.js";
 import { issueSave } from "./workflows/issue-save.js";
+import { tidy } from "./workflows/tidy.js";
 const defined = new WeakMap();
 /** The workflows of a factory returned by `defineFactory`; nothing for any other value. */
 export function workflowsOf(value) {
@@ -11,9 +13,9 @@ export function workflowsOf(value) {
         : undefined;
 }
 /**
- * A repository's factory: its workflows (`issue-save`, `implement`) adapted to
- * the repository's checks, budget and agents, then its own workflows. Every
- * problem is listed at once.
+ * A repository's factory: its workflows (`issue-save`, `implement`, `tidy`)
+ * adapted to the repository's checks, budgets and agents, then its own
+ * workflows. Every problem is listed at once.
  */
 export function defineFactory(definition) {
     const problems = definitionProblems(definition);
@@ -21,7 +23,7 @@ export function defineFactory(definition) {
     // factory.ts is not type-checked: the definition may be anything.
     const given = typeof definition === "object" && definition !== null ? definition : {};
     const own = Array.isArray(given.workflows) ? given.workflows : [];
-    const names = new Set([issueSave.name, DELIVERY]);
+    const names = new Set([issueSave.name, DELIVERY, TIDY]);
     for (const [index, workflow] of own.entries())
         if (!isWorkflow(workflow))
             problems.push(`workflows.${index}: expected a defineWorkflow(…) result`);
@@ -46,13 +48,17 @@ export function defineFactory(definition) {
     // (a budget with an agent that announces no cost).
     if (given.profiles && !flagged("profiles"))
         try {
+            const settings = { verify: given.verify ?? [], profiles: given.profiles };
+            // A refused budget is already listed; the workflow does not repeat it.
+            const maxCostUsd = flagged("maxCostUsd") ? undefined : given.maxCostUsd;
             base = [
                 issueSave,
-                implement({
-                    verify: given.verify ?? [],
-                    profiles: given.profiles,
-                    // A refused budget is already listed; the workflow does not repeat it.
-                    maxCostUsd: flagged("maxCostUsd") ? undefined : given.maxCostUsd,
+                implement({ ...settings, maxCostUsd }),
+                tidy({
+                    ...settings,
+                    maxCostUsd: flagged("tidy")
+                        ? undefined
+                        : (given.tidy?.maxCostUsd ?? maxCostUsd),
                 }),
             ];
         }

package/dist/app/define.js.map

@@ -1 +1 @@
-{"version":3,"file":"define.js","sourceRoot":"","sources":["../../src/app/define.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,eAAe,EAAiB,MAAM,eAAe,CAAC;AAC3E,OAAO,EACL,kBAAkB,GAEnB,MAAM,yBAAyB,CAAC;AACjC,OAAO,EAAE,QAAQ,EAAE,MAAM,uBAAuB,CAAC;AACjD,OAAO,EAAE,SAAS,EAAE,MAAM,0BAA0B,CAAC;AACrD,OAAO,EAAE,SAAS,EAAE,MAAM,2BAA2B,CAAC;AAqBtD,MAAM,OAAO,GAAG,IAAI,OAAO,EAA4B,CAAC;AAExD,2FAA2F;AAC3F,MAAM,UAAU,WAAW,CAAC,KAAc;IACxC,OAAO,OAAO,KAAK,KAAK,QAAQ,IAAI,KAAK,KAAK,IAAI;QAChD,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC;QACpB,CAAC,CAAC,SAAS,CAAC;AAChB,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,aAAa,CAAC,UAA6B;IACzD,MAAM,QAAQ,GAAG,kBAAkB,CAAC,UAAU,CAAC,CAAC;IAChD,MAAM,OAAO,GAAG,CAAC,KAAa,EAAE,EAAE,CAChC,QAAQ,CAAC,IAAI,CACX,CAAC,OAAO,EAAE,EAAE,CACV,OAAO,CAAC,UAAU,CAAC,GAAG,KAAK,GAAG,CAAC,IAAI,OAAO,CAAC,UAAU,CAAC,GAAG,KAAK,GAAG,CAAC,CACrE,CAAC;IACJ,kEAAkE;IAClE,MAAM,KAAK,GACT,OAAO,UAAU,KAAK,QAAQ,IAAI,UAAU,KAAK,IAAI,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,EAAE,CAAC;IAC1E,MAAM,GAAG,GAAG,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC;IAClE,MAAM,KAAK,GAAG,IAAI,GAAG,CAAC,CAAC,SAAS,CAAC,IAAI,EAAE,QAAQ,CAAC,CAAC,CAAC;IAClD,KAAK,MAAM,CAAC,KAAK,EAAE,QAAQ,CAAC,IAAI,GAAG,CAAC,OAAO,EAAE;QAC3C,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC;YACvB,QAAQ,CAAC,IAAI,CAAC,aAAa,KAAK,uCAAuC,CAAC,CAAC;aACtE,IAAI,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAC,IAAI,CAAC;YAC/B,QAAQ,CAAC,IAAI,CACX,aAAa,KAAK,iCAAiC,QAAQ,CAAC,IAAI,EAAE,CACnE,CAAC;;YACC,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;IAChC,yEAAyE;IACzE,oCAAoC;IACpC,MAAM,QAAQ,GACZ,OAAO,KAAK,CAAC,QAAQ,KAAK,QAAQ,IAAI,KAAK,CAAC,QAAQ,KAAK,IAAI;QAC3D,CAAC,CAAC,KAAK,CAAC,QAAQ;QAChB,CAAC,CAAC,EAAE,CAAC;IACT,KAAK,MAAM,IAAI,IAAI,CAAC,MAAM,EAAE,WAAW,EAAE,QAAQ,EAAE,UAAU,CAAC,EAAE,CAAC;QAC/D,MAAM,OAAO,GAAG,QAAQ,CAAC,IAAI,CAAC,CAAC;QAC/B,yEAAyE;QACzE,IAAI,OAAO,OAAO,KAAK,QAAQ,IAAI,OAAO,KAAK,IAAI;YACjD,KAAK,MAAM,EAAE,IAAI,EAAE,OAAO,EAAE,IAAI,eAAe,CAAC,OAAO,CAAC;gBACtD,QAAQ,CAAC,IAAI,CAAC,YAAY,IAAI,IAAI,IAAI,KAAK,OAAO,EAAE,CAAC,CAAC;IAC5D,CAAC;IACD,IAAI,IAAI,GAAe,EAAE,CAAC;IAC1B,6EAA6E;IAC7E,mDAAmD;IACnD,IAAI,KAAK,CAAC,QAAQ,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC;QACxC,IAAI,CAAC;YACH,IAAI,GAAG;gBACL,SAAS;gBACT,SAAS,CAAC;oBACR,MAAM,EAAE,KAAK,CAAC,MAAM,IAAI,EAAE;oBAC1B,QAAQ,EAAE,KAAK,CAAC,QAAQ;oBACxB,uEAAuE;oBACvE,UAAU,EAAE,OAAO,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,KAAK,CAAC,UAAU;iBACjE,CAAC;aACH,CAAC;QACJ,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,QAAQ,CAAC,IAAI,CAAC,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC;QACxE,CAAC;IACH,IAAI,QAAQ,CAAC,MAAM;QACjB,MAAM,IAAI,KAAK,CACb,CAAC,kBAAkB,EAAE,GAAG,QAAQ,CAAC,GAAG,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,KAAK,OAAO,EAAE,CAAC,CAAC,CAAC,IAAI,CACrE,IAAI,CACL,CACF,CAAC;IACJ,MAAM,OAAO,GAAG,MAAM,CAAC,MAAM,CAAC,EAAE,CAAmB,CAAC;IACpD,OAAO,CAAC,GAAG,CAAC,OAAO,EAAE,EAAE,IAAI,EAAE,GAAG,EAAE,CAAC,CAAC;IACpC,OAAO,OAAO,CAAC;AACjB,CAAC"}
+{"version":3,"file":"define.js","sourceRoot":"","sources":["../../src/app/define.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,eAAe,EAAiB,MAAM,eAAe,CAAC;AAC3E,OAAO,EACL,kBAAkB,GAEnB,MAAM,yBAAyB,CAAC;AACjC,OAAO,EAAE,QAAQ,EAAE,MAAM,uBAAuB,CAAC;AACjD,OAAO,EAAE,IAAI,EAAE,MAAM,mBAAmB,CAAC;AACzC,OAAO,EAAE,SAAS,EAAE,MAAM,0BAA0B,CAAC;AACrD,OAAO,EAAE,SAAS,EAAE,MAAM,2BAA2B,CAAC;AACtD,OAAO,EAAE,IAAI,EAAE,MAAM,qBAAqB,CAAC;AAqB3C,MAAM,OAAO,GAAG,IAAI,OAAO,EAA4B,CAAC;AAExD,2FAA2F;AAC3F,MAAM,UAAU,WAAW,CAAC,KAAc;IACxC,OAAO,OAAO,KAAK,KAAK,QAAQ,IAAI,KAAK,KAAK,IAAI;QAChD,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC;QACpB,CAAC,CAAC,SAAS,CAAC;AAChB,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,aAAa,CAAC,UAA6B;IACzD,MAAM,QAAQ,GAAG,kBAAkB,CAAC,UAAU,CAAC,CAAC;IAChD,MAAM,OAAO,GAAG,CAAC,KAAa,EAAE,EAAE,CAChC,QAAQ,CAAC,IAAI,CACX,CAAC,OAAO,EAAE,EAAE,CACV,OAAO,CAAC,UAAU,CAAC,GAAG,KAAK,GAAG,CAAC,IAAI,OAAO,CAAC,UAAU,CAAC,GAAG,KAAK,GAAG,CAAC,CACrE,CAAC;IACJ,kEAAkE;IAClE,MAAM,KAAK,GACT,OAAO,UAAU,KAAK,QAAQ,IAAI,UAAU,KAAK,IAAI,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,EAAE,CAAC;IAC1E,MAAM,GAAG,GAAG,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC;IAClE,MAAM,KAAK,GAAG,IAAI,GAAG,CAAC,CAAC,SAAS,CAAC,IAAI,EAAE,QAAQ,EAAE,IAAI,CAAC,CAAC,CAAC;IACxD,KAAK,MAAM,CAAC,KAAK,EAAE,QAAQ,CAAC,IAAI,GAAG,CAAC,OAAO,EAAE;QAC3C,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC;YACvB,QAAQ,CAAC,IAAI,CAAC,aAAa,KAAK,uCAAuC,CAAC,CAAC;aACtE,IAAI,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAC,IAAI,CAAC;YAC/B,QAAQ,CAAC,IAAI,CACX,aAAa,KAAK,iCAAiC,QAAQ,CAAC,IAAI,EAAE,CACnE,CAAC;;YACC,KAAK,CAAC,GAAG,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;IAChC,yEAAyE;IACzE,oCAAoC;IACpC,MAAM,QAAQ,GACZ,OAAO,KAAK,CAAC,QAAQ,KAAK,QAAQ,IAAI,KAAK,CAAC,QAAQ,KAAK,IAAI;QAC3D,CAAC,CAAC,KAAK,CAAC,QAAQ;QAChB,CAAC,CAAC,EAAE,CAAC;IACT,KAAK,MAAM,IAAI,IAAI,CAAC,MAAM,EAAE,WAAW,EAAE,QAAQ,EAAE,UAAU,CAAC,EAAE,CAAC;QAC/D,MAAM,OAAO,GAAG,QAAQ,CAAC,IAAI,CAAC,CAAC;QAC/B,yEAAyE;QACzE,IAAI,OAAO,OAAO,KAAK,QAAQ,IAAI,OAAO,KAAK,IAAI;YACjD,KAAK,MAAM,EAAE,IAAI,EAAE,OAAO,EAAE,IAAI,eAAe,CAAC,OAAO,CAAC;gBACtD,QAAQ,CAAC,IAAI,CAAC,YAAY,IAAI,IAAI,IAAI,KAAK,OAAO,EAAE,CAAC,CAAC;IAC5D,CAAC;IACD,IAAI,IAAI,GAAe,EAAE,CAAC;IAC1B,6EAA6E;IAC7E,mDAAmD;IACnD,IAAI,KAAK,CAAC,QAAQ,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC;QACxC,IAAI,CAAC;YACH,MAAM,QAAQ,GAAG,EAAE,MAAM,EAAE,KAAK,CAAC,MAAM,IAAI,EAAE,EAAE,QAAQ,EAAE,KAAK,CAAC,QAAQ,EAAE,CAAC;YAC1E,uEAAuE;YACvE,MAAM,UAAU,GAAG,OAAO,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,KAAK,CAAC,UAAU,CAAC;YACxE,IAAI,GAAG;gBACL,SAAS;gBACT,SAAS,CAAC,EAAE,GAAG,QAAQ,EAAE,UAAU,EAAE,CAAC;gBACtC,IAAI,CAAC;oBACH,GAAG,QAAQ;oBACX,UAAU,EAAE,OAAO,CAAC,MAAM,CAAC;wBACzB,CAAC,CAAC,SAAS;wBACX,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,EAAE,UAAU,IAAI,UAAU,CAAC;iBAC3C,CAAC;aACH,CAAC;QACJ,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,QAAQ,CAAC,IAAI,CAAC,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC;QACxE,CAAC;IACH,IAAI,QAAQ,CAAC,MAAM;QACjB,MAAM,IAAI,KAAK,CACb,CAAC,kBAAkB,EAAE,GAAG,QAAQ,CAAC,GAAG,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,KAAK,OAAO,EAAE,CAAC,CAAC,CAAC,IAAI,CACrE,IAAI,CACL,CACF,CAAC;IACJ,MAAM,OAAO,GAAG,MAAM,CAAC,MAAM,CAAC,EAAE,CAAmB,CAAC;IACpD,OAAO,CAAC,GAAG,CAAC,OAAO,EAAE,EAAE,IAAI,EAAE,GAAG,EAAE,CAAC,CAAC;IACpC,OAAO,OAAO,CAAC;AACjB,CAAC"}

package/dist/app/run.d.ts

@@ -10,6 +10,6 @@ export declare function startRun(factory: Factory, name: string, options: Record
  * Resumes a failed run of the repository at the stage where it stopped, with
  * its workflow as `factory.ts` defines it now and its prompts read again;
  * `--max-cost-usd` sets its budget, and `--accept-review` delivers an
- * implement run its review stopped by asking for changes.
+ * implement or tidy run its review stopped by asking for changes.
  */
 export declare function resumeRun(factory: Factory, runId: string, options: Record<string, string | boolean>, hooks: RunHooks): Promise<StartedRun>;

package/dist/app/run.js

@@ -2,6 +2,7 @@ import { exceededEdge, resumeRun as resume, startRun as start, } from "@clemrig/
 import { loadPrompts, workflowOf } from "../system/factory.js";
 import { asOptionsError, inputFromOptions, parseBudget } from "./options.js";
 import { DELIVERY } from "../domain/delivery.js";
+import { TIDY } from "../domain/tidy.js";
 import { loadRun } from "./runs.js";
 /**
  * Starts a workflow of the factory from `factory run` options, with its prompts
@@ -32,7 +33,7 @@ export async function startRun(factory, name, options, hooks) {
  * Resumes a failed run of the repository at the stage where it stopped, with
  * its workflow as `factory.ts` defines it now and its prompts read again;
  * `--max-cost-usd` sets its budget, and `--accept-review` delivers an
- * implement run its review stopped by asking for changes.
+ * implement or tidy run its review stopped by asking for changes.
  */
 export async function resumeRun(factory, runId, options, hooks) {
     const { "max-cost-usd": budget, "accept-review": acceptReview, ...others } = options;
@@ -47,9 +48,9 @@ export async function resumeRun(factory, runId, options, hooks) {
     // judges it again, and accepts it.
     if (acceptReview !== undefined &&
         (acceptReview !== true ||
-            run.workflow !== DELIVERY ||
+            ![DELIVERY, TIDY].includes(run.workflow) ||
             exceededEdge(workflow, run) !== "reviewed.changes"))
-        throw new Error(`--accept-review only resumes a run of ${DELIVERY} stopped by its review asking for changes, and takes no value`);
+        throw new Error(`--accept-review only resumes a run of ${DELIVERY} or ${TIDY} stopped by its review asking for changes, and takes no value`);
     return resume({
         workflow,
         runId,

package/dist/app/run.js.map

@@ -1 +1 @@
-{"version":3,"file":"run.js","sourceRoot":"","sources":["../../src/app/run.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,YAAY,EAEZ,SAAS,IAAI,MAAM,EAEnB,QAAQ,IAAI,KAAK,GAClB,MAAM,eAAe,CAAC;AACvB,OAAO,EAAgB,WAAW,EAAE,UAAU,EAAE,MAAM,sBAAsB,CAAC;AAC7E,OAAO,EAAE,cAAc,EAAE,gBAAgB,EAAE,WAAW,EAAE,MAAM,cAAc,CAAC;AAC7E,OAAO,EAAE,QAAQ,EAAE,MAAM,uBAAuB,CAAC;AACjD,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAEpC;;;;GAIG;AACH,MAAM,CAAC,KAAK,UAAU,QAAQ,CAC5B,OAAgB,EAChB,IAAY,EACZ,OAAyC,EACzC,KAAe;IAEf,MAAM,QAAQ,GAAG,UAAU,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC;IAC3C,MAAM,EAAE,cAAc,EAAE,MAAM,EAAE,GAAG,KAAK,EAAE,GAAG,OAAO,CAAC;IACrD,MAAM,UAAU,GAAG,MAAM,KAAK,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,WAAW,CAAC,MAAM,CAAC,CAAC;IAC1E,MAAM,KAAK,GAAG,gBAAgB,CAAC,QAAQ,EAAE,KAAK,CAAC,CAAC;IAChD,IAAI,CAAC;QACH,OAAO,MAAM,KAAK,CAChB;YACE,QAAQ;YACR,KAAK;YACL,UAAU;YACV,OAAO,EAAE,GAAG,EAAE,CAAC,WAAW,CAAC,OAAO,EAAE,QAAQ,CAAC;YAC7C,GAAG,EAAE,OAAO,CAAC,GAAG;YAChB,SAAS,EAAE,OAAO,CAAC,UAAU;YAC7B,OAAO,EAAE,OAAO,CAAC,OAAO;SACzB,EACD,KAAK,CACN,CAAC;IACJ,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,MAAM,cAAc,CAAC,QAAQ,EAAE,KAAK,EAAE,KAAK,CAAC,CAAC;IAC/C,CAAC;AACH,CAAC;AAED;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,SAAS,CAC7B,OAAgB,EAChB,KAAa,EACb,OAAyC,EACzC,KAAe;IAEf,MAAM,EACJ,cAAc,EAAE,MAAM,EACtB,eAAe,EAAE,YAAY,EAC7B,GAAG,MAAM,EACV,GAAG,OAAO,CAAC;IACZ,MAAM,CAAC,OAAO,CAAC,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IACtC,IAAI,OAAO;QAAE,MAAM,IAAI,KAAK,CAAC,oBAAoB,OAAO,aAAa,CAAC,CAAC;IACvE,MAAM,UAAU,GAAG,MAAM,KAAK,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,WAAW,CAAC,MAAM,CAAC,CAAC;IAC1E,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC;IAC1C,MAAM,QAAQ,GAAG,UAAU,CAAC,OAAO,EAAE,GAAG,CAAC,QAAQ,CAAC,CAAC;IACnD,0EAA0E;IAC1E,0EAA0E;IAC1E,mCAAmC;IACnC,IACE,YAAY,KAAK,SAAS;QAC1B,CAAC,YAAY,KAAK,IAAI;YACpB,GAAG,CAAC,QAAQ,KAAK,QAAQ;YACzB,YAAY,CAAC,QAAQ,EAAE,GAAG,CAAC,KAAK,kBAAkB,CAAC;QAErD,MAAM,IAAI,KAAK,CACb,yCAAyC,QAAQ,+DAA+D,CACjH,CAAC;IACJ,OAAO,MAAM,CACX;QACE,QAAQ;QACR,KAAK;QACL,UAAU;QACV,GAAG,CAAC,YAAY,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,EAAE,cAAc,EAAE,IAAI,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAC3D,OAAO,EAAE,GAAG,EAAE,CAAC,WAAW,CAAC,OAAO,EAAE,QAAQ,CAAC;QAC7C,GAAG,EAAE,OAAO,CAAC,GAAG;QAChB,SAAS,EAAE,OAAO,CAAC,UAAU;QAC7B,OAAO,EAAE,OAAO,CAAC,OAAO;KACzB,EACD,KAAK,CACN,CAAC;AACJ,CAAC"}
+{"version":3,"file":"run.js","sourceRoot":"","sources":["../../src/app/run.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,YAAY,EAEZ,SAAS,IAAI,MAAM,EAEnB,QAAQ,IAAI,KAAK,GAClB,MAAM,eAAe,CAAC;AACvB,OAAO,EAAgB,WAAW,EAAE,UAAU,EAAE,MAAM,sBAAsB,CAAC;AAC7E,OAAO,EAAE,cAAc,EAAE,gBAAgB,EAAE,WAAW,EAAE,MAAM,cAAc,CAAC;AAC7E,OAAO,EAAE,QAAQ,EAAE,MAAM,uBAAuB,CAAC;AACjD,OAAO,EAAE,IAAI,EAAE,MAAM,mBAAmB,CAAC;AACzC,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAEpC;;;;GAIG;AACH,MAAM,CAAC,KAAK,UAAU,QAAQ,CAC5B,OAAgB,EAChB,IAAY,EACZ,OAAyC,EACzC,KAAe;IAEf,MAAM,QAAQ,GAAG,UAAU,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC;IAC3C,MAAM,EAAE,cAAc,EAAE,MAAM,EAAE,GAAG,KAAK,EAAE,GAAG,OAAO,CAAC;IACrD,MAAM,UAAU,GAAG,MAAM,KAAK,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,WAAW,CAAC,MAAM,CAAC,CAAC;IAC1E,MAAM,KAAK,GAAG,gBAAgB,CAAC,QAAQ,EAAE,KAAK,CAAC,CAAC;IAChD,IAAI,CAAC;QACH,OAAO,MAAM,KAAK,CAChB;YACE,QAAQ;YACR,KAAK;YACL,UAAU;YACV,OAAO,EAAE,GAAG,EAAE,CAAC,WAAW,CAAC,OAAO,EAAE,QAAQ,CAAC;YAC7C,GAAG,EAAE,OAAO,CAAC,GAAG;YAChB,SAAS,EAAE,OAAO,CAAC,UAAU;YAC7B,OAAO,EAAE,OAAO,CAAC,OAAO;SACzB,EACD,KAAK,CACN,CAAC;IACJ,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,MAAM,cAAc,CAAC,QAAQ,EAAE,KAAK,EAAE,KAAK,CAAC,CAAC;IAC/C,CAAC;AACH,CAAC;AAED;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,SAAS,CAC7B,OAAgB,EAChB,KAAa,EACb,OAAyC,EACzC,KAAe;IAEf,MAAM,EACJ,cAAc,EAAE,MAAM,EACtB,eAAe,EAAE,YAAY,EAC7B,GAAG,MAAM,EACV,GAAG,OAAO,CAAC;IACZ,MAAM,CAAC,OAAO,CAAC,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IACtC,IAAI,OAAO;QAAE,MAAM,IAAI,KAAK,CAAC,oBAAoB,OAAO,aAAa,CAAC,CAAC;IACvE,MAAM,UAAU,GAAG,MAAM,KAAK,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,WAAW,CAAC,MAAM,CAAC,CAAC;IAC1E,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC;IAC1C,MAAM,QAAQ,GAAG,UAAU,CAAC,OAAO,EAAE,GAAG,CAAC,QAAQ,CAAC,CAAC;IACnD,0EAA0E;IAC1E,0EAA0E;IAC1E,mCAAmC;IACnC,IACE,YAAY,KAAK,SAAS;QAC1B,CAAC,YAAY,KAAK,IAAI;YACpB,CAAC,CAAC,QAAQ,EAAE,IAAI,CAAC,CAAC,QAAQ,CAAC,GAAG,CAAC,QAAQ,CAAC;YACxC,YAAY,CAAC,QAAQ,EAAE,GAAG,CAAC,KAAK,kBAAkB,CAAC;QAErD,MAAM,IAAI,KAAK,CACb,yCAAyC,QAAQ,OAAO,IAAI,+DAA+D,CAC5H,CAAC;IACJ,OAAO,MAAM,CACX;QACE,QAAQ;QACR,KAAK;QACL,UAAU;QACV,GAAG,CAAC,YAAY,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,EAAE,cAAc,EAAE,IAAI,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAC3D,OAAO,EAAE,GAAG,EAAE,CAAC,WAAW,CAAC,OAAO,EAAE,QAAQ,CAAC;QAC7C,GAAG,EAAE,OAAO,CAAC,GAAG;QAChB,SAAS,EAAE,OAAO,CAAC,UAAU;QAC7B,OAAO,EAAE,OAAO,CAAC,OAAO;KACzB,EACD,KAAK,CACN,CAAC;AACJ,CAAC"}

package/dist/app/workflows/implement.d.ts

@@ -1,8 +1,3 @@
-import type { FactoryDefinition } from "../../domain/definition.js";
-/** What the workflow uses: the factory requires the budget, the workflow also runs without one. */
-type Settings = Pick<FactoryDefinition, "verify" | "profiles"> & {
-    maxCostUsd?: number;
-};
+import type { Settings } from "./stages/run.js";
 /** Plans, implements, checks and reviews an approved spec, then describes it in the pull request it opens. */
 export declare function implement(definition: Settings): import("@clemrig/flow").Workflow;
-export {};