@vintasoftware/pr-review-canvas 0.2.0 → 0.3.0

package/README.md

@@ -138,9 +138,9 @@ Customize generation and AI Chat prompts with the `prompts` map in your project'
 
 ```yaml
 prompts:
-  generation-format.md: review-prompts/generation-format.md
-  generation-surfacing.md: review-prompts/generation-surfacing.md
-  chat-seed.md: review-prompts/chat-seed.md
+    generation-format.md: review-prompts/generation-format.md
+    generation-surfacing.md: review-prompts/generation-surfacing.md
+    chat-seed.md: review-prompts/chat-seed.md
 ```
 
 Copy the installed templates to start editing (for an npm global install):
@@ -168,11 +168,17 @@ In a clone of this tool, use pnpm for the shared lockfile and development checks
 ```bash
 corepack pnpm --version
 corepack pnpm install --frozen-lockfile
+corepack pnpm hooks:install
 corepack pnpm exec playwright install --with-deps chromium
 corepack pnpm verify
 corepack pnpm start --repo /path/to/your-project
 ```
 
+The pre-commit hook runs `pnpm precommit`: lint, formatting, strict type checks, and tests.
+Any failure blocks the commit. Run `pnpm hooks:install`
+once per clone to enable it. Use `pnpm lint:fix` and `pnpm format` to apply automatic fixes.
+CI runs the same checks through `pnpm verify`, with coverage executing the unit tests once.
+
 Run the full `pnpm verify` before pushing. Keep branch coverage at least 96% when adding or
 changing behavior, leaving a margin above CI's 95% minimum. Cover meaningful failure and boundary
 cases rather than lowering thresholds. Each CI job uploads `coverage-node-<version>` with branch

package/docs/reference.md

@@ -15,14 +15,14 @@ For setup and the basic review workflow, see the [README](../README.md).
 
 ### Repository and runtime options
 
-| Option | Applies to | Default and behavior |
-|---|---|---|
-| `--repo <dir>` | All commands | Uses the current directory when omitted; resolves the repository root from there |
-| `--data-dir <dir>` | All except `install-skill` | Overrides `PR_REVIEW_DATA_DIR`, then the default `<main checkout>/.pr-review` |
-| `--port <n>` | `serve` | Overrides `PR_REVIEW_PORT`, then `3010`; accepts 1–65535 |
-| `--agent claude\|codex` | `serve` | Overrides the saved chat agent for this run |
-| `--model <id>` | `serve` | Overrides the saved chat model for this run |
-| `--fixture-canvas <review.json>` | `serve` | Development preview: uses the supplied canvas for every requested PR, with its head replaced by the live PR head |
+| Option                           | Applies to                 | Default and behavior                                                                                             |
+| -------------------------------- | -------------------------- | ---------------------------------------------------------------------------------------------------------------- |
+| `--repo <dir>`                   | All commands               | Uses the current directory when omitted; resolves the repository root from there                                 |
+| `--data-dir <dir>`               | All except `install-skill` | Overrides `PR_REVIEW_DATA_DIR`, then the default `<main checkout>/.pr-review`                                    |
+| `--port <n>`                     | `serve`                    | Overrides `PR_REVIEW_PORT`, then `3010`; accepts 1–65535                                                         |
+| `--agent claude\|codex`          | `serve`                    | Overrides the saved chat agent for this run                                                                      |
+| `--model <id>`                   | `serve`                    | Overrides the saved chat model for this run                                                                      |
+| `--fixture-canvas <review.json>` | `serve`                    | Development preview: uses the supplied canvas for every requested PR, with its head replaced by the live PR head |
 
 Repository operations require an `origin` remote on **github.com**. GitHub Enterprise Server
 hosts are not supported. Fetching a PR does not check out its branch.
@@ -129,13 +129,13 @@ JSON error. `serve` stays running and writes its startup message to stderr.
 Command failures use `{ "error": { "code", "message", "hint" } }`, with `hint` optional.
 Validation failures from `validate` use its report format instead.
 
-| Exit code | Meaning |
-|---|---|
-| `0` | Success |
-| `1` | Error, including a failed `doctor` check |
-| `2` | Command usage error, such as an unknown command or missing required flag |
-| `4` | GitHub CLI missing or unauthenticated |
-| `5` | Validation failed in `validate` or `publish` |
+| Exit code | Meaning                                                                  |
+| --------- | ------------------------------------------------------------------------ |
+| `0`       | Success                                                                  |
+| `1`       | Error, including a failed `doctor` check                                 |
+| `2`       | Command usage error, such as an unknown command or missing required flag |
+| `4`       | GitHub CLI missing or unauthenticated                                    |
+| `5`       | Validation failed in `validate` or `publish`                             |
 
 `doctor` reports failed checks with exit `1`, including authentication failures.
 
@@ -148,20 +148,20 @@ Lists you supply replace their defaults.
 Path patterns match repository-relative paths. `**` crosses directories; `*` and `?` match
 within one path segment.
 
-| Key | Default | Details |
-|---|---|---|
-| `version` | `1` | The only supported configuration version |
-| `rulebook` | Unset | Path to a Markdown file of project code standards, resolved from the repository root; these standards take precedence over bundled standards |
-| `layers` | `[]` | Optional review guidance; each entry has `id`, `title`, `description`, and optional `paths` patterns. The agent may combine, split, or reorder groups. When omitted or empty, it chooses semantic sections from the change |
-| `highRisk` | `[]` | Entries with a `pattern` glob and `label`; matching changes receive risk labels and cannot go in the Other layer |
-| `generation.mode` | `strict` | See [generation modes](#generation-modes) |
-| `generation.maxRepairRounds` | `3` | Failed validation rounds allowed by the generation skill |
-| `generation.inlineDiffMaxLines` | `1500` | Maximum diff length to include directly in the generation prompt |
-| `generation.smallPrHunks` | `10` | At or below this hunk count, the prompt asks for one layer unless concerns differ |
-| `generation.caps` | See below | Overrides individual text limits |
-| `tests.patterns` | `['**/*.test.*', '**/*.spec.*', '**/__tests__/**']` | Paths treated as tests for review ordering and labels |
-| `chat.enabled` | `true` | Set to `false` to disable AI Chat |
-| `prompts` | Bundled templates | See [prompt templates](#prompt-templates) for supported keys and behavior |
+| Key                             | Default                                             | Details                                                                                                                                                                                                                    |
+| ------------------------------- | --------------------------------------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
+| `version`                       | `1`                                                 | The only supported configuration version                                                                                                                                                                                   |
+| `rulebook`                      | Unset                                               | Path to a Markdown file of project code standards, resolved from the repository root; these standards take precedence over bundled standards                                                                               |
+| `layers`                        | `[]`                                                | Optional review guidance; each entry has `id`, `title`, `description`, and optional `paths` patterns. The agent may combine, split, or reorder groups. When omitted or empty, it chooses semantic sections from the change |
+| `highRisk`                      | `[]`                                                | Entries with a `pattern` glob and `label`; matching changes receive risk labels and cannot go in the Other layer                                                                                                           |
+| `generation.mode`               | `strict`                                            | See [generation modes](#generation-modes)                                                                                                                                                                                  |
+| `generation.maxRepairRounds`    | `3`                                                 | Failed validation rounds allowed by the generation skill                                                                                                                                                                   |
+| `generation.inlineDiffMaxLines` | `1500`                                              | Maximum diff length to include directly in the generation prompt                                                                                                                                                           |
+| `generation.smallPrHunks`       | `10`                                                | At or below this hunk count, the prompt asks for one layer unless concerns differ                                                                                                                                          |
+| `generation.caps`               | See below                                           | Overrides individual text limits                                                                                                                                                                                           |
+| `tests.patterns`                | `['**/*.test.*', '**/*.spec.*', '**/__tests__/**']` | Paths treated as tests for review ordering and labels                                                                                                                                                                      |
+| `chat.enabled`                  | `true`                                              | Set to `false` to disable AI Chat                                                                                                                                                                                          |
+| `prompts`                       | Bundled templates                                   | See [prompt templates](#prompt-templates) for supported keys and behavior                                                                                                                                                  |
 
 Generation's numeric options and text caps must be positive integers. An empty `layers` list
 provides no suggested groups; an empty `tests.patterns` list recognizes no files as tests.
@@ -182,14 +182,14 @@ affects canvas generation; chat answers the reviewer's selected question.
 
 The `prompts` map in `pr-review.config.yml` accepts these keys:
 
-| Key | Purpose |
-|---|---|
-| `generation-format.md` | Schema and output rules |
-| `generation-strict.md` | Instructions for strict mode |
-| `generation-surfacing.md` | Instructions for surfacing mode |
-| `quality-standards.md` | Bundled code standards |
-| `layering-guidance.md` | Guidance for grouping related changes |
-| `chat-seed.md` | Opening AI Chat instructions |
+| Key                       | Purpose                               |
+| ------------------------- | ------------------------------------- |
+| `generation-format.md`    | Schema and output rules               |
+| `generation-strict.md`    | Instructions for strict mode          |
+| `generation-surfacing.md` | Instructions for surfacing mode       |
+| `quality-standards.md`    | Bundled code standards                |
+| `layering-guidance.md`    | Guidance for grouping related changes |
+| `chat-seed.md`            | Opening AI Chat instructions          |
 
 Each configured file replaces a whole template. Paths resolve from the project root,
 including when running from a subdirectory or using `--repo`. Absolute paths work for
@@ -214,18 +214,18 @@ when upgrading.
 Set any of these keys under `generation.caps`. Prose limits count visible characters, so Markdown
 link targets do not count. `diagram` counts raw Mermaid source characters.
 
-| Key | Default |
-|---|---|
-| `summary` | 1200 |
-| `layerTitle` | 60 |
-| `rationale` | 300 |
-| `decisions` | 600 |
-| `checkByHand` | 400 |
-| `annotation` | 240 |
-| `pointTitle` | 90 |
-| `pointBody` | 600 |
-| `testBehavior` | 120 |
-| `diagram` | 1500 |
+| Key            | Default |
+| -------------- | ------- |
+| `summary`      | 1200    |
+| `layerTitle`   | 60      |
+| `rationale`    | 300     |
+| `decisions`    | 600     |
+| `checkByHand`  | 400     |
+| `annotation`   | 240     |
+| `pointTitle`   | 90      |
+| `pointBody`    | 600     |
+| `testBehavior` | 120     |
+| `diagram`      | 1500    |
 
 The canvas has a separate limit of 12 attention points, including entries generated from missing
 tests. Increasing text caps does not increase that limit.
@@ -236,7 +236,7 @@ Custom patterns replace the JavaScript/TypeScript defaults. For example, a Pytho
 
 ```yaml
 tests:
-  patterns: ['**/test_*.py', '**/tests/**']
+    patterns: ['**/test_*.py', '**/tests/**']
 ```
 
 Test recognition controls ordering, but matching a test to its source file currently follows
@@ -247,15 +247,15 @@ placed in Other while its source is in a regular layer.
 
 The data directory's `settings.yml` accepts these keys and values:
 
-| Key | Default | Accepted values |
-|---|---|---|
-| `version` | `1` | `1` |
-| `skin` | `terminal` | `terminal`, `github` |
-| `theme` | `auto` | `auto`, `light`, `dark` |
-| `agent` | `claude` | `claude`, `codex` |
-| `model` | `null` | A model ID, or `null` for the agent's default |
-| `chatTimeoutSec` | `600` | Integer seconds, 30–3600 |
-| `maxTurns` | `null` | Integer 1–100, or `null` for the agent's default |
+| Key              | Default    | Accepted values                                  |
+| ---------------- | ---------- | ------------------------------------------------ |
+| `version`        | `1`        | `1`                                              |
+| `skin`           | `terminal` | `terminal`, `github`                             |
+| `theme`          | `auto`     | `auto`, `light`, `dark`                          |
+| `agent`          | `claude`   | `claude`, `codex`                                |
+| `model`          | `null`     | A model ID, or `null` for the agent's default    |
+| `chatTimeoutSec` | `600`      | Integer seconds, 30–3600                         |
+| `maxTurns`       | `null`     | Integer 1–100, or `null` for the agent's default |
 
 Invalid settings fall back to defaults. URL parameters `?skin=github&theme=light` can override
 appearance for one page load without saving it.
@@ -274,13 +274,13 @@ Deleting the directory loses saved preferences, canvases, progress, and chat his
 
 Append a fragment to `/review/<pr-number>`:
 
-| Target | Fragment example |
-|---|---|
-| Layer | `#layer:data-access` |
-| File | `#file:src/store.ts` |
-| Hunk | `#hunk:src/store.ts#2` |
-| New-side line range | `#line:src/store.ts:40-52` |
-| Old-side line | `#line:src/store.ts:40:old` |
+| Target              | Fragment example            |
+| ------------------- | --------------------------- |
+| Layer               | `#layer:data-access`        |
+| File                | `#file:src/store.ts`        |
+| Hunk                | `#hunk:src/store.ts#2`      |
+| New-side line range | `#line:src/store.ts:40-52`  |
+| Old-side line       | `#line:src/store.ts:40:old` |
 
 Use the layer key and file path from the canvas. Links open the target file when needed; if a
 line is unavailable, navigation uses the nearest visible row. Recipients need their own running
@@ -344,41 +344,41 @@ sandbox for the agent. Its access also depends on the agent's own permissions. D
 
 ## Troubleshooting
 
-| Symptom or code | Next step |
-|---|---|
-| `NOT_A_REPO` | Run inside a Git clone or pass `--repo <dir>` |
-| `NO_ORIGIN` | Check that `origin` points to a repository on github.com |
-| `GH_MISSING` / `GH_UNAUTHENTICATED` | Install [GitHub CLI](https://cli.github.com), run `gh auth login`, and check authentication in the same environment that runs the server |
-| `GITHUB_API_ERROR` | Read the underlying error for permissions, rate limits, connectivity, or GitHub service problems |
-| `PR_NOT_FOUND` | Check the PR number, repository, and your access |
-| `CANVAS_NOT_FOUND` | Generate or import a canvas for the requested commit |
-| `CANVAS_INVALID` | Read the format errors; re-export or regenerate the canvas |
-| `CANVAS_REPO_MISMATCH` | Check which clone is open; use `import --force` only when importing from the other repository is intentional |
-| `CANVAS_TOO_LARGE` | The archive exceeds the 20 MiB import limit |
-| `CANVAS_STALE` | The PR head moved; prepare again for the current commit |
-| `MODEL_INVALID` | Fix the reported problems in `model.json`, validate, then publish again |
-| `SKILL_DIR_EXISTS` | The destination contains a customized directory; preserve it elsewhere before replacing it with `--force` |
-| `CHAT_BUSY` | Wait for the running reply or press **stop** |
-| `AGENT_AUTH_REQUIRED` | Sign in through the selected agent's CLI, then retry |
-| `AGENT_MISSING` or missing chat pane | Check `chat.enabled` and confirm the server can find `acpx` and the selected agent; run `pr-review doctor --all-checks` |
-| `AGENT_INCOMPLETE` | Retry the message or increase the chat timeout |
-| `COMMENT_FORBIDDEN` | Check the GitHub account's repository access and token permissions |
-| `COMMENT_LINE_NOT_IN_DIFF` | Choose a line shown in the current diff |
-| `SIGNOFF_INCOMPLETE` | Mark every layer except Other reviewed for this head |
-| `FORBIDDEN_HOST` / `CROSS_ORIGIN` | Open the local server using `localhost` or `127.0.0.1` and submit actions from that page |
+| Symptom or code                      | Next step                                                                                                                                |
+| ------------------------------------ | ---------------------------------------------------------------------------------------------------------------------------------------- |
+| `NOT_A_REPO`                         | Run inside a Git clone or pass `--repo <dir>`                                                                                            |
+| `NO_ORIGIN`                          | Check that `origin` points to a repository on github.com                                                                                 |
+| `GH_MISSING` / `GH_UNAUTHENTICATED`  | Install [GitHub CLI](https://cli.github.com), run `gh auth login`, and check authentication in the same environment that runs the server |
+| `GITHUB_API_ERROR`                   | Read the underlying error for permissions, rate limits, connectivity, or GitHub service problems                                         |
+| `PR_NOT_FOUND`                       | Check the PR number, repository, and your access                                                                                         |
+| `CANVAS_NOT_FOUND`                   | Generate or import a canvas for the requested commit                                                                                     |
+| `CANVAS_INVALID`                     | Read the format errors; re-export or regenerate the canvas                                                                               |
+| `CANVAS_REPO_MISMATCH`               | Check which clone is open; use `import --force` only when importing from the other repository is intentional                             |
+| `CANVAS_TOO_LARGE`                   | The archive exceeds the 20 MiB import limit                                                                                              |
+| `CANVAS_STALE`                       | The PR head moved; prepare again for the current commit                                                                                  |
+| `MODEL_INVALID`                      | Fix the reported problems in `model.json`, validate, then publish again                                                                  |
+| `SKILL_DIR_EXISTS`                   | The destination contains a customized directory; preserve it elsewhere before replacing it with `--force`                                |
+| `CHAT_BUSY`                          | Wait for the running reply or press **stop**                                                                                             |
+| `AGENT_AUTH_REQUIRED`                | Sign in through the selected agent's CLI, then retry                                                                                     |
+| `AGENT_MISSING` or missing chat pane | Check `chat.enabled` and confirm the server can find `acpx` and the selected agent; run `pr-review doctor --all-checks`                  |
+| `AGENT_INCOMPLETE`                   | Retry the message or increase the chat timeout                                                                                           |
+| `COMMENT_FORBIDDEN`                  | Check the GitHub account's repository access and token permissions                                                                       |
+| `COMMENT_LINE_NOT_IN_DIFF`           | Choose a line shown in the current diff                                                                                                  |
+| `SIGNOFF_INCOMPLETE`                 | Mark every layer except Other reviewed for this head                                                                                     |
+| `FORBIDDEN_HOST` / `CROSS_ORIGIN`    | Open the local server using `localhost` or `127.0.0.1` and submit actions from that page                                                 |
 
 ### Validation diagnostics
 
 Validation reports name the field, file, hunk, or line to fix. Common groups are:
 
-| Codes | What to check |
-|---|---|
-| `SCHEMA`, `TEXT_TOO_LONG` | Required fields, types, and text limits |
-| `HUNK_UNASSIGNED`, `HUNK_DUPLICATE`, `HUNK_UNKNOWN` | Each known hunk belongs to exactly one layer |
-| `PATH_UNKNOWN`, `TEST_PATH_UNKNOWN` | Referenced files exist in the relevant diff or PR head |
-| `LAYER_EMPTY`, `LAYER_KEY_DUPLICATE` | Layers contain hunks and have unique keys |
-| `OTHER_DUPLICATE`, `OTHER_NOT_LAST`, `RISK_IN_OTHER` | At most one Other layer, last, without risk-tagged changes |
-| `TEST_NOT_LAST`, `TEST_IN_OTHER` | Tests follow the code they cover and use the appropriate layer |
-| `ANNOTATION_OUTSIDE_HUNK`, `POINT_OUTSIDE_DIFF`, `FOLD_INVALID` | Locations and fold ranges fit the assigned diff |
-| `TOO_MANY_POINTS` | Count explicit points and missing-test entries together |
-| `LINK_UNRESOLVED`, `DIAGRAM_NODE_UNKNOWN`, `DIAGRAM_LIMIT` | Link targets, diagram node IDs, and diagram counts |
+| Codes                                                           | What to check                                                  |
+| --------------------------------------------------------------- | -------------------------------------------------------------- |
+| `SCHEMA`, `TEXT_TOO_LONG`                                       | Required fields, types, and text limits                        |
+| `HUNK_UNASSIGNED`, `HUNK_DUPLICATE`, `HUNK_UNKNOWN`             | Each known hunk belongs to exactly one layer                   |
+| `PATH_UNKNOWN`, `TEST_PATH_UNKNOWN`                             | Referenced files exist in the relevant diff or PR head         |
+| `LAYER_EMPTY`, `LAYER_KEY_DUPLICATE`                            | Layers contain hunks and have unique keys                      |
+| `OTHER_DUPLICATE`, `OTHER_NOT_LAST`, `RISK_IN_OTHER`            | At most one Other layer, last, without risk-tagged changes     |
+| `TEST_NOT_LAST`, `TEST_IN_OTHER`                                | Tests follow the code they cover and use the appropriate layer |
+| `ANNOTATION_OUTSIDE_HUNK`, `POINT_OUTSIDE_DIFF`, `FOLD_INVALID` | Locations and fold ranges fit the assigned diff                |
+| `TOO_MANY_POINTS`                                               | Count explicit points and missing-test entries together        |
+| `LINK_UNRESOLVED`, `DIAGRAM_NODE_UNKNOWN`, `DIAGRAM_LIMIT`      | Link targets, diagram node IDs, and diagram counts             |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vintasoftware/pr-review-canvas",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "Localhost PR review canvas: layered diffs, attention points, and GitHub comments for one PR",
   "type": "module",
   "bin": {
@@ -10,13 +10,20 @@
     "dev": "tsx watch --clear-screen=false --ignore '**/.pr-review/**' src/cli.ts serve",
     "start": "tsx src/cli.ts serve",
     "test": "vitest run",
+    "hooks:install": "git config core.hooksPath .githooks",
     "test:watch": "vitest",
     "test:browser": "playwright test",
     "coverage": "vitest run --coverage",
     "typecheck": "tsc --noEmit",
     "test:package": "node scripts/test-package.mjs",
-    "verify": "pnpm typecheck && pnpm coverage && pnpm test:browser --workers=2 && pnpm test:package",
-    "prepublishOnly": "pnpm verify"
+    "verify": "pnpm check && pnpm coverage && pnpm test:browser --workers=2 && pnpm test:package",
+    "prepublishOnly": "pnpm verify",
+    "lint": "oxlint --deny-warnings",
+    "lint:fix": "oxlint --fix --deny-warnings",
+    "format": "oxfmt --write",
+    "format:check": "oxfmt --check",
+    "check": "pnpm lint && pnpm format:check && pnpm typecheck",
+    "precommit": "pnpm check && pnpm test"
   },
   "dependencies": {
     "@highlightjs/cdn-assets": "^11.12.0",
@@ -36,6 +43,8 @@
     "@types/node": "^24.13.3",
     "@vitest/coverage-v8": "^4.1.10",
     "happy-dom": "20.11.2",
+    "oxfmt": "^0.68.0",
+    "oxlint": "^1.83.0",
     "typescript": "^5.9.3",
     "vite": "~8.0.16",
     "vitest": "^4.1.10"

package/pr-review.config.example.yml

@@ -30,10 +30,10 @@ highRisk:
     label: auth
 
 generation:
-  mode: strict          # strict: quality review; surfacing: walkthrough that audits as it reads
-  maxRepairRounds: 3      # publish retries the skill may make after a failed validation
+  mode: strict # strict: quality review; surfacing: walkthrough that audits as it reads
+  maxRepairRounds: 3 # publish retries the skill may make after a failed validation
   inlineDiffMaxLines: 1500 # diffs longer than this are read per file instead of inlined in the prompt
-  smallPrHunks: 10         # at most this many hunks: one layer unless concerns differ, fewer annotations
+  smallPrHunks: 10 # at most this many hunks: one layer unless concerns differ, fewer annotations
   # caps:                  # override the text caps (characters)
   #   summary: 1800
   #   rationale: 300
@@ -46,4 +46,4 @@ tests:
   patterns: ['**/*.test.*', '**/*.spec.*', '**/__tests__/**']
 
 chat:
-  enabled: true            # false removes the chat pane and its routes
+  enabled: true # false removes the chat pane and its routes

package/skills/pr-review-canvas/SKILL.md

@@ -39,7 +39,14 @@ pr-review prepare --base <ref> --head <ref> [--force]
 Progress goes to stderr. The last stdout line is JSON:
 
 ```json
-{ "canvasDir": "...", "headSha": "...", "mergeBaseSha": "...", "promptPath": "...", "contextPath": "...", "status": "prepared" }
+{
+    "canvasDir": "...",
+    "headSha": "...",
+    "mergeBaseSha": "...",
+    "promptPath": "...",
+    "contextPath": "...",
+    "status": "prepared"
+}
 ```
 
 - `status: "exists"` means a canvas already exists for this head. Stop and tell the user:

package/src/acpx/acpx.ts

@@ -231,7 +231,11 @@ export function createAgentRunner(opts: CreateAgentRunnerOptions = {}): AgentRun
         // The cancel call gets its own timeout, and SIGKILL when it elapses: a cancel that hangs
         // must neither hold the kill back nor stay behind as a process of its own.
         agentArgs =>
-          execQuiet(bin, agentArgs, { cwd: options.cwd, timeoutSec: CANCEL_TIMEOUT_SEC, killSignal: 'SIGKILL' }),
+          execQuiet(bin, agentArgs, {
+            cwd: options.cwd,
+            timeoutSec: CANCEL_TIMEOUT_SEC,
+            killSignal: 'SIGKILL',
+          }),
         slackMs,
         cancelGraceMs
       )
@@ -268,7 +272,12 @@ export function createAgentRunner(opts: CreateAgentRunnerOptions = {}): AgentRun
         }
       }
       if (!ended) {
-        return { ok: false, text: '', code: 'AGENT_INCOMPLETE', message: 'the agent stopped before finishing' }
+        return {
+          ok: false,
+          text: '',
+          code: 'AGENT_INCOMPLETE',
+          message: 'the agent stopped before finishing',
+        }
       }
       return text === ''
         ? { ok: false, text: '', code: 'AGENT_INCOMPLETE', message: 'the agent replied nothing' }
@@ -516,11 +525,19 @@ function startRun(
     const exit = code ?? 1
     const mapped = exitCodeToAgentCode(exit)
     if (mapped !== null) {
-      finish({ type: 'error', code: mapped, message: stderr.trim() === '' ? exitCodeMessage(exit) : stderr.trim() })
+      finish({
+        type: 'error',
+        code: mapped,
+        message: stderr.trim() === '' ? exitCodeMessage(exit) : stderr.trim(),
+      })
       return
     }
     // acpx 0.13.2 exits 0 when its own --timeout elapses, without a terminal event.
-    finish({ type: 'error', code: 'AGENT_INCOMPLETE', message: 'the agent stopped before finishing its answer' })
+    finish({
+      type: 'error',
+      code: 'AGENT_INCOMPLETE',
+      message: 'the agent stopped before finishing its answer',
+    })
   })
 
   child.stdin?.on('error', () => undefined)

package/src/acpx/events.ts

@@ -64,7 +64,11 @@ function contentText(value: unknown): string {
  */
 export function mapAcpxMessage(message: unknown): AgentEvent | null {
   if (!isRecord(message)) {
-    return { type: 'error', code: 'AGENT_PROTOCOL_INVALID', message: 'the agent wrote a line that is not a message' }
+    return {
+      type: 'error',
+      code: 'AGENT_PROTOCOL_INVALID',
+      message: 'the agent wrote a line that is not a message',
+    }
   }
   const error = read(message, 'error')
   if (isRecord(error)) {

package/src/acpx/ndjson.ts

@@ -50,7 +50,9 @@ export function createNdjsonSplitter(lineMax: number = NDJSON_LINE_MAX): NdjsonS
         const raw = buffer.slice(0, at)
         buffer = buffer.slice(at + 1)
         if (raw.length > lineMax) {
-          throw new NdjsonError(`the agent wrote a line of ${raw.length} characters, over the ${lineMax} limit`)
+          throw new NdjsonError(
+            `the agent wrote a line of ${raw.length} characters, over the ${lineMax} limit`
+          )
         }
         out.push(...take(raw))
         at = buffer.indexOf('\n')

package/src/acpx/preflight.ts

@@ -16,7 +16,11 @@ export interface PreflightProbe {
  * Is acpx on PATH? The answer barely changes while the server runs, so it is asked once and
  * reused; the page shows a banner and hides the chat when it says no.
  */
-export function createPreflightProbe(runner: AgentRunner, now: () => Date, ttlMs = PREFLIGHT_TTL_MS): PreflightProbe {
+export function createPreflightProbe(
+  runner: AgentRunner,
+  now: () => Date,
+  ttlMs = PREFLIGHT_TTL_MS
+): PreflightProbe {
   let cached: { at: number; value: AcpxPreflight } | null = null
   let inFlight: Promise<AcpxPreflight> | null = null
   return {

package/src/canvas/export.ts

@@ -37,7 +37,12 @@ export async function buildCanvasZipFor(
   const number = prNumber ?? stored.prNumber
   const manifest: CanvasManifest = number === undefined ? stored : { ...stored, prNumber: number }
   const zip: CanvasZip = {
-    name: buildCanvasZipName({ repo: manifest.repo, headSha, prNumber: number, generatedAt: manifest.generatedAt }),
+    name: buildCanvasZipName({
+      repo: manifest.repo,
+      headSha,
+      prNumber: number,
+      generatedAt: manifest.generatedAt,
+    }),
     bytes: buildCanvasZip(manifest, artifact),
     headSha,
   }
@@ -64,7 +69,11 @@ async function isDirectory(target: string): Promise<boolean> {
 }
 
 /** `--out` names a file when it ends in `.zip` or is not an existing directory. */
-export async function resolveOutPath(out: string | undefined, defaultDir: string, name: string): Promise<string> {
+export async function resolveOutPath(
+  out: string | undefined,
+  defaultDir: string,
+  name: string
+): Promise<string> {
   if (out === undefined) {
     return path.join(defaultDir, name)
   }

package/src/canvas/zip.ts

@@ -66,7 +66,10 @@ function decode(entry: Uint8Array | undefined, name: string, issues: string[]):
  */
 export function readCanvasZip(bytes: Uint8Array): CanvasZipContents {
   if (bytes.length > CANVAS_ZIP_MAX_BYTES) {
-    throw new CanvasZipError('CANVAS_TOO_LARGE', `the canvas zip is larger than ${CANVAS_ZIP_MAX_BYTES} bytes`)
+    throw new CanvasZipError(
+      'CANVAS_TOO_LARGE',
+      `the canvas zip is larger than ${CANVAS_ZIP_MAX_BYTES} bytes`
+    )
   }
   if (!hasZipMagic(bytes)) {
     throw new CanvasZipError('CANVAS_INVALID', 'this file is not a zip', [

package/src/chat/chat-manager.ts

@@ -10,7 +10,13 @@ import type { SettingsStore } from '../store/settings-store.js'
 import type { StateStore } from '../store/state-store.js'
 import { type ContextSources, renderChatContext } from './context.js'
 import { renderSeed, type SeedPaths } from './seed.js'
-import { NEW_THREAD_TITLE, nextThreadIndex, type TranscriptStore, threadName, threadTitle } from './threads.js'
+import {
+  NEW_THREAD_TITLE,
+  nextThreadIndex,
+  type TranscriptStore,
+  threadName,
+  threadTitle,
+} from './threads.js'
 
 export class ChatBusyError extends Error {
   constructor() {
@@ -188,7 +194,11 @@ export function createChatManager(deps: ChatManagerDeps): ChatManager {
     }
   }
 
-  async function* runTurn(target: ChatTarget, input: ChatSendInput, slot: RunningTurn): AsyncIterable<ChatEvent> {
+  async function* runTurn(
+    target: ChatTarget,
+    input: ChatSendInput,
+    slot: RunningTurn
+  ): AsyncIterable<ChatEvent> {
     const settings = await effectiveSettings()
     const thread = await resolveThread(target.prNumber, settings.agent, input.thread)
     const seeded = thread.seededHeadSha !== target.headSha
@@ -350,7 +360,10 @@ export function createChatManager(deps: ChatManagerDeps): ChatManager {
       if (thread === undefined) {
         return null
       }
-      await deps.state.update(prNumber, current => ({ ...current, chat: { ...current.chat, activeThread: name } }))
+      await deps.state.update(prNumber, current => ({
+        ...current,
+        chat: { ...current.chat, activeThread: name },
+      }))
       return thread
     },
     send,

package/src/chat/context.ts

@@ -106,7 +106,10 @@ export async function renderChatContext(context: ChatContext, sources: ContextSo
  * An attention point: what it says, plus the lines it sits on. The agent needs the text to know
  * what the reader is reacting to; the lines alone leave it guessing. One context, one heading.
  */
-async function renderPoint(context: Extract<ChatContext, { kind: 'point' }>, sources: ContextSources): Promise<string> {
+async function renderPoint(
+  context: Extract<ChatContext, { kind: 'point' }>,
+  sources: ContextSources
+): Promise<string> {
   const point = sources.artifact.points.find(p => p.fingerprint === context.fingerprint)
   if (point === undefined) {
     throw new ChatContextError('this canvas has no such attention point')
@@ -140,7 +143,10 @@ async function renderLines(
 }
 
 /** The quoted lines and the hunk around them, with no heading of their own. */
-async function linesBody(context: Extract<ChatContext, { kind: 'lines' }>, sources: ContextSources): Promise<string> {
+async function linesBody(
+  context: Extract<ChatContext, { kind: 'lines' }>,
+  sources: ContextSources
+): Promise<string> {
   if (context.end < context.start) {
     throw new ChatContextError('the last line of a selection comes after the first')
   }
@@ -155,6 +161,7 @@ async function linesBody(context: Extract<ChatContext, { kind: 'lines' }>, sourc
     lines === null || lines.length === 0
       ? '_these lines are not available locally_'
       : fence(lines.map((text, i) => `${context.start + i}: ${text}`).join('\n'))
-  const hunkBlock = hunk === undefined ? '' : `\n\nThe hunk around them is \`${hunk.id}\` (\`${hunk.header}\`).`
+  const hunkBlock =
+    hunk === undefined ? '' : `\n\nThe hunk around them is \`${hunk.id}\` (\`${hunk.header}\`).`
   return `${numbered}${hunkBlock}`
 }

package/src/chat/seed.ts

@@ -42,7 +42,9 @@ export function pointsMarkdown(artifact: ReviewArtifact): string {
   if (artifact.points.length === 0) {
     return '_none_'
   }
-  return artifact.points.map(p => `- ${p.level} · ${p.kind} · ${p.title} (\`${p.path}:${p.line}\`)`).join('\n')
+  return artifact.points
+    .map(p => `- ${p.level} · ${p.kind} · ${p.title} (\`${p.path}:${p.line}\`)`)
+    .join('\n')
 }
 
 export function testsMarkdown(artifact: ReviewArtifact): string {

package/src/cli.ts

@@ -26,7 +26,16 @@ import { startServer } from './server/node-server.js'
 import { readJson } from './store/atomic-json.js'
 import { ensureDataDir } from './store/data-dir.js'
 
-const SUBCOMMANDS = ['serve', 'prepare', 'validate', 'publish', 'export', 'import', 'install-skill', 'doctor'] as const
+const SUBCOMMANDS = [
+  'serve',
+  'prepare',
+  'validate',
+  'publish',
+  'export',
+  'import',
+  'install-skill',
+  'doctor',
+] as const
 
 const USAGE = `usage: pr-review <command> [flags]
 
@@ -78,7 +87,8 @@ async function buildContext(
   )
   const projectConfig = await loadProjectConfig(config.repoRoot)
   await ensureDataDir(config.dataDir)
-  const fixtureArtifact = config.fixtureCanvasPath === null ? null : await loadFixture(config.fixtureCanvasPath)
+  const fixtureArtifact =
+    config.fixtureCanvasPath === null ? null : await loadFixture(config.fixtureCanvasPath)
   return createAppContext({ config, projectConfig, fixtureArtifact })
 }