@cat-factory/agents 0.68.3 → 0.69.0

package/dist/agents/kinds/pr-review-context.d.ts

@@ -0,0 +1,103 @@
+import type { GitHubChangedFile, GitHubReviewThread, RepoOp } from '@cat-factory/kernel';
+import type { ComposableFragment } from '../runtime/fragments.js';
+/** The injected context file (under `.cat-context/`) the diff preOp writes for the reviewer. */
+export declare const PR_DIFF_CONTEXT_FILE = "pr-diff.md";
+/** The injected context file listing the PR's already-posted review comments (for de-dup). */
+export declare const PR_EXISTING_COMMENTS_CONTEXT_FILE = "pr-existing-comments.md";
+/**
+ * Filename prefix for the per-standard context files the standards preOp writes. Re-exports the
+ * SHARED convention (`@cat-factory/agents` `fragments.ts`) so `standardsDeliveredAsFiles` — which
+ * decides whether to fold standards into the prompt as a fallback — recognises what this preOp
+ * writes without knowing anything pr-review-specific.
+ */
+export declare const PR_STANDARD_CONTEXT_PREFIX = "standard-";
+/** Resolve the reviewed PR's number from the block's task-type fields (prefer `prNumber`). */
+export declare function resolvePrNumber(fields: {
+    prNumber?: number;
+    prUrl?: string;
+} | undefined): number | null;
+/** One suggested review slice: a cohesive group of changed files, with its size. */
+export interface SuggestedSlice {
+    /** Short name — the grouping path, or `assorted` / a `(part N)` suffix when split. */
+    title: string;
+    paths: string[];
+    changedLines: number;
+}
+/**
+ * Group the changed files into candidate review slices, deterministically and with no model
+ * turns. Files group by their leading path segments (the project / module / top-level area),
+ * one-off groups collapse into a shared `assorted` slice, and any group over
+ * {@link SLICE_MAX_FILES} / {@link SLICE_MAX_CHANGED_LINES} is split into numbered parts.
+ *
+ * This is a SUGGESTION the reviewer may regroup — path adjacency is a good proxy for
+ * "belongs together" but it cannot see that a refactor and its call sites live apart. Its
+ * value is that the reviewer starts from a usable grouping instead of probing the file list
+ * to build one, and that the size caps stop a single oversized slice forming: cost scales with
+ * turns × context, so one slice big enough to need 100+ turns costs more than three that
+ * replace it.
+ */
+export declare function planSlices(files: GitHubChangedFile[]): SuggestedSlice[];
+/**
+ * Render the changed-file list, the change shape, a suggested slicing, and — only for a diff
+ * small enough to review in one pass — the patches, as the injected `.cat-context/pr-diff.md`.
+ * See the module header for why the patch budget is all-or-nothing.
+ */
+export declare function renderPrDiffContext(number: number, files: GitHubChangedFile[]): string;
+/**
+ * Render the PR's existing review threads as `.cat-context/pr-existing-comments.md`, so the
+ * reviewer de-dups against findings already raised (prior rounds / humans / other bots).
+ *
+ * Threads are GROUPED BY FILE under a path index. De-dup is a per-slice concern, so a slice
+ * reviewer should grep out the handful of threads on its own paths; reading the whole file
+ * into the parent's context makes every remaining turn of the review pay for all of them.
+ */
+export declare function renderExistingReviewComments(number: number, threads: GitHubReviewThread[]): string;
+/**
+ * The `.cat-context/` filename a resolved best-practice standard is written to. Non-portable
+ * characters are replaced with `-` so the name survives the harness's context-file sanitizer
+ * (it keeps only `[A-Za-z0-9._-]` and flattens any directory) unchanged — the name in the index
+ * is then the name on disk.
+ *
+ * When sanitizing ALTERS the id, two distinct ids can sanitize to the same slug (`org/team` and
+ * `org team` both → `org-team`); the harness drops the duplicate path, silently losing the second
+ * standard while the index still advertises it. So a short hash of the ORIGINAL id is appended
+ * whenever the slug differs from the id, making the filename unique per id. An already-safe id is
+ * left untouched (no suffix), so the common case stays readable.
+ */
+export declare function standardsContextFileName(fragmentId: string): string;
+/** Render one standard as its own context file, with the title the reviewer cites it by. */
+export declare function renderStandardContext(fragment: ComposableFragment): string;
+/** Render the index that tells the reviewer which standards exist and where each one is. */
+export declare function renderStandardsIndex(fragments: ComposableFragment[]): string;
+/** The index file listing every injected standard (the shared `context-files` convention). */
+export declare const PR_STANDARDS_INDEX_CONTEXT_FILE = "standards.md";
+/**
+ * PreOp for the `pr-reviewer` kind: hand the reviewer the PR's changed-file list, change shape and
+ * a suggested slicing as `.cat-context/pr-diff.md` (plus the patches when the whole diff is small
+ * enough to review in one pass). Pass-through — injecting nothing, so the prompt's git fallback
+ * runs — when the PR number can't be resolved, the bound client can't list changed files
+ * (unwired / a VCS provider without the capability), or the PR reports no changed files.
+ */
+export declare const prReviewerDiffPreOp: RepoOp;
+/**
+ * PreOp for the `pr-reviewer` kind: hand the reviewer the PR's EXISTING review threads up front as
+ * `.cat-context/pr-existing-comments.md`, grouped by file, so it de-dups against findings already
+ * raised instead of re-reporting them. Pass-through — injecting nothing — when the PR number can't
+ * be resolved, the bound client can't read review threads, or the PR has no review threads yet.
+ */
+export declare const prReviewerExistingCommentsPreOp: RepoOp;
+/**
+ * PreOp for the `pr-reviewer` kind: write the task's selected best-practice standards as one
+ * `.cat-context/standard-<id>.md` file each, plus an index.
+ *
+ * The reviewer's own prompt does NOT carry the standards (the kind declares
+ * `standardsDelivery: 'context-files'`, so the engine skips the fold). Folding them in charged
+ * the parent for every standard on every turn — 145 KB across 5 standards on the measured run,
+ * ~3.7M tokens over 96 turns — while the agents that actually review the code, the parallel slice
+ * subagents, never received them and worked from the parent's one-line paraphrase instead. As
+ * files, each standard is read once by the slices it applies to, from the real text.
+ *
+ * Pass-through when the run resolved no fragments (a review task with none selected).
+ */
+export declare const prReviewerStandardsPreOp: RepoOp;
+//# sourceMappingURL=pr-review-context.d.ts.map

package/dist/agents/kinds/pr-review-context.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"pr-review-context.d.ts","sourceRoot":"","sources":["../../../src/agents/kinds/pr-review-context.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,iBAAiB,EACjB,kBAAkB,EAClB,MAAM,EAGP,MAAM,qBAAqB,CAAA;AAC5B,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,yBAAyB,CAAA;AAmCjE,gGAAgG;AAChG,eAAO,MAAM,oBAAoB,eAAe,CAAA;AAEhD,8FAA8F;AAC9F,eAAO,MAAM,iCAAiC,4BAA4B,CAAA;AAE1E;;;;;GAKG;AACH,eAAO,MAAM,0BAA0B,cAAgC,CAAA;AAkCvE,8FAA8F;AAC9F,wBAAgB,eAAe,CAC7B,MAAM,EAAE;IAAE,QAAQ,CAAC,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,SAAS,GACxD,MAAM,GAAG,IAAI,CAef;AAMD,oFAAoF;AACpF,MAAM,WAAW,cAAc;IAC7B,sFAAsF;IACtF,KAAK,EAAE,MAAM,CAAA;IACb,KAAK,EAAE,MAAM,EAAE,CAAA;IACf,YAAY,EAAE,MAAM,CAAA;CACrB;AAcD;;;;;;;;;;;;GAYG;AACH,wBAAgB,UAAU,CAAC,KAAK,EAAE,iBAAiB,EAAE,GAAG,cAAc,EAAE,CAiDvE;AAqDD;;;;GAIG;AACH,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,iBAAiB,EAAE,GAAG,MAAM,CAyDtF;AAyBD;;;;;;;GAOG;AACH,wBAAgB,4BAA4B,CAC1C,MAAM,EAAE,MAAM,EACd,OAAO,EAAE,kBAAkB,EAAE,GAC5B,MAAM,CA2CR;AAgBD;;;;;;;;;;;GAWG;AACH,wBAAgB,wBAAwB,CAAC,UAAU,EAAE,MAAM,GAAG,MAAM,CAInE;AAED,4FAA4F;AAC5F,wBAAgB,qBAAqB,CAAC,QAAQ,EAAE,kBAAkB,GAAG,MAAM,CAS1E;AAED,4FAA4F;AAC5F,wBAAgB,oBAAoB,CAAC,SAAS,EAAE,kBAAkB,EAAE,GAAG,MAAM,CAgB5E;AAED,8FAA8F;AAC9F,eAAO,MAAM,+BAA+B,iBAA+B,CAAA;AAM3E;;;;;;GAMG;AACH,eAAO,MAAM,mBAAmB,EAAE,MAYjC,CAAA;AAED;;;;;GAKG;AACH,eAAO,MAAM,+BAA+B,EAAE,MAiB7C,CAAA;AAED;;;;;;;;;;;;GAYG;AACH,eAAO,MAAM,wBAAwB,EAAE,MActC,CAAA"}

package/dist/agents/kinds/pr-review-context.js

@@ -0,0 +1,456 @@
+import { STANDARDS_CONTEXT_FILE_PREFIX, STANDARDS_CONTEXT_INDEX_FILE, } from '../runtime/fragments.js';
+// ---------------------------------------------------------------------------
+// What the `pr-reviewer` is handed UP FRONT, as `.cat-context/*.md` files.
+//
+// The governing constraint is that an agentic loop re-sends its whole transcript on every
+// turn, so ANYTHING placed in the reviewer's context is paid for again on each of the
+// (hundreds of) turns that follow. A 10k-token file read early in a 400-turn review costs
+// millions of tokens by the end. Everything here is therefore sized by "how many turns will
+// carry this?", not by "does it fit?".
+//
+// Measured on a 437-turn review of a ~450-file PR, that produced three rules:
+//
+//  1. The reviewer slices from the changed-file LIST; its parallel slice subagents pull their
+//     own diffs from git. Pre-inlining a large diff pays for a map nobody reads — that run
+//     inlined 319 KB and the subagents referenced it once while running 141 `git diff`/`git
+//     show` calls. So patches are inlined ONLY when the WHOLE diff is small enough to review
+//     in one pass ({@link MAX_INLINE_DIFF_BYTES}); past that the file is a manifest and the
+//     diffs are read per slice. All-or-nothing, because a half-inlined file is the worst case:
+//     it is big AND the reviewer still has to probe it to find out what is missing.
+//  2. A manifest the reviewer has to reverse-engineer costs more than one it can read. That
+//     run spent 21 `grep`/`awk`/`sed`/`wc` probes working out the injected file's own shape,
+//     each probe's output then carried for the rest of the review. So the manifest states the
+//     per-directory rollup and a SUGGESTED SLICING outright ({@link planSlices}) — grouping
+//     changed files is mechanical, and doing it here costs zero model turns.
+//  3. Reference material is per-slice, so it must be readable per-slice. Existing review
+//     threads are grouped under a path index, and best-practice standards go to one file each,
+//     so a subagent greps or reads only its own — rather than the parent reading the whole lot
+//     into a context that then carries it for every remaining turn.
+// ---------------------------------------------------------------------------
+/** The injected context file (under `.cat-context/`) the diff preOp writes for the reviewer. */
+export const PR_DIFF_CONTEXT_FILE = 'pr-diff.md';
+/** The injected context file listing the PR's already-posted review comments (for de-dup). */
+export const PR_EXISTING_COMMENTS_CONTEXT_FILE = 'pr-existing-comments.md';
+/**
+ * Filename prefix for the per-standard context files the standards preOp writes. Re-exports the
+ * SHARED convention (`@cat-factory/agents` `fragments.ts`) so `standardsDeliveredAsFiles` — which
+ * decides whether to fold standards into the prompt as a fallback — recognises what this preOp
+ * writes without knowing anything pr-review-specific.
+ */
+export const PR_STANDARD_CONTEXT_PREFIX = STANDARDS_CONTEXT_FILE_PREFIX;
+/**
+ * Total inlined-patch budget. Under it, the WHOLE diff is inlined and a small PR is reviewable
+ * in one pass with no git turns at all. Over it, no patch is inlined: the reviewer slices from
+ * the manifest and each slice reads only its own files from the checkout — which is what the
+ * parallel slice subagents do regardless, so inlining more only pays for the same bytes twice.
+ */
+const MAX_INLINE_DIFF_BYTES = 64 * 1024;
+/** Per-file inline cap: one generated blob (a lockfile, a snapshot) never crowds out the rest. */
+const MAX_SINGLE_PATCH_BYTES = 32 * 1024;
+/**
+ * Byte budget for the injected existing-comments file. Review threads are short prose, so this is
+ * generous; past it, the remaining threads are summarised as a count so the file never dominates
+ * the context on a PR with a very long comment history.
+ */
+const MAX_EXISTING_COMMENTS_BYTES = 64 * 1024;
+/** Path depth a suggested slice groups at — `src/Foo.Bar/Baz/x.cs` groups under `src/Foo.Bar`. */
+const SLICE_GROUP_DEPTH = 2;
+/** A suggested slice is split once it exceeds either budget, so no single slice dwarfs the rest. */
+const SLICE_MAX_FILES = 20;
+const SLICE_MAX_CHANGED_LINES = 2_500;
+/** Groups smaller than this are merged into a shared "assorted" slice rather than standing alone. */
+const SLICE_MIN_FILES = 2;
+// ---------------------------------------------------------------------------
+// PR number resolution
+// ---------------------------------------------------------------------------
+/** Resolve the reviewed PR's number from the block's task-type fields (prefer `prNumber`). */
+export function resolvePrNumber(fields) {
+    if (!fields)
+        return null;
+    if (typeof fields.prNumber === 'number' &&
+        Number.isInteger(fields.prNumber) &&
+        fields.prNumber > 0)
+        return fields.prNumber;
+    const url = fields.prUrl?.trim();
+    if (!url)
+        return null;
+    // GitHub `/pull/<n>`, GitLab `/-/merge_requests/<n>`, or a trailing `#<n>` / `/<n>`.
+    const m = url.match(/(?:pull|pulls|merge_requests)\/(\d+)|[#/](\d+)\s*$/);
+    const raw = m?.[1] ?? m?.[2];
+    const n = raw ? Number(raw) : Number.NaN;
+    return Number.isInteger(n) && n > 0 ? n : null;
+}
+/** The directory a path groups under: its leading {@link SLICE_GROUP_DEPTH} segments. */
+function groupKey(path) {
+    const segments = path.split('/');
+    segments.pop(); // the filename
+    if (segments.length === 0)
+        return '(repo root)';
+    return segments.slice(0, SLICE_GROUP_DEPTH).join('/');
+}
+function changedLinesOf(file) {
+    return file.additions + file.deletions;
+}
+/**
+ * Group the changed files into candidate review slices, deterministically and with no model
+ * turns. Files group by their leading path segments (the project / module / top-level area),
+ * one-off groups collapse into a shared `assorted` slice, and any group over
+ * {@link SLICE_MAX_FILES} / {@link SLICE_MAX_CHANGED_LINES} is split into numbered parts.
+ *
+ * This is a SUGGESTION the reviewer may regroup — path adjacency is a good proxy for
+ * "belongs together" but it cannot see that a refactor and its call sites live apart. Its
+ * value is that the reviewer starts from a usable grouping instead of probing the file list
+ * to build one, and that the size caps stop a single oversized slice forming: cost scales with
+ * turns × context, so one slice big enough to need 100+ turns costs more than three that
+ * replace it.
+ */
+export function planSlices(files) {
+    const groups = new Map();
+    for (const file of files) {
+        const key = groupKey(file.path);
+        const bucket = groups.get(key);
+        if (bucket)
+            bucket.push(file);
+        else
+            groups.set(key, [file]);
+    }
+    // Collapse one-off groups together: a slice per single changed file is pure overhead.
+    const assorted = [];
+    const named = [];
+    for (const [key, bucket] of groups) {
+        if (bucket.length < SLICE_MIN_FILES)
+            assorted.push(...bucket);
+        else
+            named.push([key, bucket]);
+    }
+    if (assorted.length)
+        named.push(['assorted', assorted]);
+    const slices = [];
+    for (const [key, bucket] of named) {
+        // Largest-first within a group, so a split keeps the heavy files spread across parts
+        // rather than stacking them all into part 1.
+        const sorted = [...bucket].sort((a, b) => changedLinesOf(b) - changedLinesOf(a));
+        const parts = [[]];
+        let lines = 0;
+        for (const file of sorted) {
+            const current = parts[parts.length - 1];
+            const wouldExceed = current.length >= SLICE_MAX_FILES ||
+                (current.length > 0 && lines + changedLinesOf(file) > SLICE_MAX_CHANGED_LINES);
+            if (wouldExceed) {
+                parts.push([file]);
+                lines = changedLinesOf(file);
+            }
+            else {
+                current.push(file);
+                lines += changedLinesOf(file);
+            }
+        }
+        for (const [index, part] of parts.entries()) {
+            if (part.length === 0)
+                continue;
+            slices.push({
+                title: parts.length > 1 ? `${key} (part ${index + 1})` : key,
+                // Back to path order within the slice, so the list reads like the repo.
+                paths: part.map((f) => f.path).sort(),
+                changedLines: part.reduce((sum, f) => sum + changedLinesOf(f), 0),
+            });
+        }
+    }
+    return slices.sort((a, b) => b.changedLines - a.changedLines);
+}
+// ---------------------------------------------------------------------------
+// `.cat-context/pr-diff.md`
+// ---------------------------------------------------------------------------
+/** Per-directory rollup of the change, so the reviewer sees the shape without probing. */
+function renderDirectoryRollup(files) {
+    const rollup = new Map();
+    for (const file of files) {
+        const key = groupKey(file.path);
+        const entry = rollup.get(key) ?? { files: 0, lines: 0 };
+        entry.files += 1;
+        entry.lines += changedLinesOf(file);
+        rollup.set(key, entry);
+    }
+    const rows = [...rollup.entries()].sort((a, b) => b[1].lines - a[1].lines);
+    return [
+        `\n## Change shape (${rows.length} areas)\n`,
+        ...rows.map(([key, e]) => `- ${key} — ${e.files} file(s), ${e.lines} changed line(s)`),
+    ];
+}
+function renderSuggestedSlices(slices) {
+    return [
+        `\n## Suggested slicing (${slices.length} slices)\n`,
+        'Grouped mechanically by path and capped by size, so you can start from it instead of',
+        'deriving one. REGROUP where you know better — a refactor and its call sites often live in',
+        'different areas — but do not exceed these sizes: an oversized slice costs more than the two',
+        'that would replace it, because every extra turn re-sends the whole slice context.\n',
+        ...slices.map((s) => `- **${s.title}** (${s.paths.length} file(s), ${s.changedLines} changed line(s))\n` +
+            s.paths.map((p) => `  - ${p}`).join('\n')),
+    ];
+}
+/** The header for the manifest-only (large PR) shape: how to read a slice's diffs from git. */
+function largePrGuidance(files, bytes) {
+    return (`\n## Patches — NOT inlined (${files} files, ~${Math.ceil(bytes / 1024)} KiB of patch)\n\n` +
+        'This diff is too large to inline: it would sit in context for every turn of the review while ' +
+        'each slice needs only its own files. Read each slice’s diff when you review that slice:\n\n' +
+        '```sh\n' +
+        'git diff origin/<base>...origin/pr-head -- <path>   # the head diff for one file\n' +
+        'git show origin/pr-head:<path>                      # a file’s full body at the PR head\n' +
+        '```\n\n' +
+        "Read a RANGE, not a whole large file (`| sed -n '<from>,<to>p'`), and never re-read something " +
+        'you already have — both stay in context for the rest of the review.\n');
+}
+/**
+ * Render the changed-file list, the change shape, a suggested slicing, and — only for a diff
+ * small enough to review in one pass — the patches, as the injected `.cat-context/pr-diff.md`.
+ * See the module header for why the patch budget is all-or-nothing.
+ */
+export function renderPrDiffContext(number, files) {
+    const enc = new TextEncoder();
+    // Oversized single patches never inline, and never count toward the inline decision — one
+    // generated blob must not push an otherwise-small PR onto the manifest-only path.
+    const inlinable = files.filter((f) => f.patch != null && enc.encode(f.patch).length <= MAX_SINGLE_PATCH_BYTES);
+    const inlineBytes = inlinable.reduce((sum, f) => sum + enc.encode(f.patch ?? '').length, 0);
+    const inlinePatches = inlineBytes <= MAX_INLINE_DIFF_BYTES;
+    // The manifest-only header reports the WHOLE patch size (incl. over-cap blobs), not just the
+    // inlinable slice, so the "~N KiB of patch" figure matches the diff the reviewer is told to
+    // read from git rather than understating it by the size of the excluded lockfile/snapshot.
+    const totalPatchBytes = files.reduce((sum, f) => sum + enc.encode(f.patch ?? '').length, 0);
+    const header = `# Pull request #${number} — changed files and diff\n\n` +
+        'Prepared from the API so you can plan your review slices WITHOUT reconstructing the diff ' +
+        'yourself. You have the full base checkout AND (usually) the PR head fetched as ' +
+        '`origin/pr-head`.\n';
+    const list = [
+        `\n## Changed files (${files.length})\n`,
+        ...files.map((f) => {
+            const rename = f.previousPath ? ` (renamed from ${f.previousPath})` : '';
+            return `- ${f.status} ${f.path} (+${f.additions}/-${f.deletions})${rename}`;
+        }),
+    ];
+    const sections = [
+        header,
+        list.join('\n'),
+        renderDirectoryRollup(files).join('\n'),
+        renderSuggestedSlices(planSlices(files)).join('\n'),
+    ];
+    if (!inlinePatches) {
+        sections.push(largePrGuidance(files.length, totalPatchBytes));
+        return `${sections.join('\n')}\n`;
+    }
+    const patches = ['\n## Patches\n'];
+    for (const f of files) {
+        const heading = `\n### ${f.path} (${f.status}, +${f.additions}/-${f.deletions})\n`;
+        if (f.patch == null) {
+            patches.push(`${heading}(no patch — binary or too large; read the file from the checkout)\n`);
+        }
+        else if (enc.encode(f.patch).length > MAX_SINGLE_PATCH_BYTES) {
+            const kib = Math.ceil(enc.encode(f.patch).length / 1024);
+            patches.push(`${heading}(patch ~${kib} KiB — over the per-file inline budget; read it with ` +
+                '`git show origin/pr-head:<path>`)\n');
+        }
+        else {
+            patches.push(`${heading}\`\`\`diff\n${f.patch}\n\`\`\`\n`);
+        }
+    }
+    sections.push(patches.join(''));
+    return `${sections.join('\n')}\n`;
+}
+// ---------------------------------------------------------------------------
+// `.cat-context/pr-existing-comments.md`
+// ---------------------------------------------------------------------------
+/** A single review-thread comment's body, trimmed + collapsed to a one-liner excerpt for the list. */
+function commentExcerpt(body, max = 500) {
+    const flat = body.trim().replace(/\s+/g, ' ');
+    return flat.length > max ? `${flat.slice(0, max)}…` : flat;
+}
+function renderThread(thread) {
+    const anchor = thread.path
+        ? `${thread.path}${thread.line != null ? `:${thread.line}` : ''}`
+        : 'general';
+    const state = thread.isResolved ? 'RESOLVED' : 'UNRESOLVED';
+    const first = thread.comments[0];
+    const author = first?.author ? `@${first.author}` : 'unknown';
+    const excerpt = first ? commentExcerpt(first.body) : '(no comment body)';
+    const extra = thread.comments.length - 1;
+    const replies = extra > 0 ? ` (+${extra} repl${extra === 1 ? 'y' : 'ies'})` : '';
+    return `\n### ${anchor} — ${state}\n${author}${replies}: ${excerpt}\n`;
+}
+/**
+ * Render the PR's existing review threads as `.cat-context/pr-existing-comments.md`, so the
+ * reviewer de-dups against findings already raised (prior rounds / humans / other bots).
+ *
+ * Threads are GROUPED BY FILE under a path index. De-dup is a per-slice concern, so a slice
+ * reviewer should grep out the handful of threads on its own paths; reading the whole file
+ * into the parent's context makes every remaining turn of the review pay for all of them.
+ */
+export function renderExistingReviewComments(number, threads) {
+    const header = `# Pull request #${number} — existing review comments\n\n` +
+        'These findings have ALREADY been raised on this PR (earlier rounds, human reviewers, or other ' +
+        'bots). Do NOT re-report an issue an existing comment already covers. Skip UNRESOLVED threads ' +
+        '(already awaiting action); re-raise a RESOLVED thread only if the change shows its fix is wrong ' +
+        'or incomplete.\n\n' +
+        'Threads are grouped by file below. When reviewing ONE slice, read only that slice’s files — ' +
+        '`grep -n -A2 "^### <path>" .cat-context/pr-existing-comments.md` — rather than the whole file.\n';
+    // Insertion-ordered by first appearance, so the index reads in the order the API returned.
+    const byPath = new Map();
+    for (const thread of threads) {
+        const key = thread.path ?? '(general)';
+        const bucket = byPath.get(key);
+        if (bucket)
+            bucket.push(thread);
+        else
+            byPath.set(key, [thread]);
+    }
+    const index = [
+        `\n## Files with existing threads (${byPath.size})\n`,
+        ...[...byPath.entries()].map(([path, list]) => `- ${path} (${list.length} thread(s))`),
+    ];
+    const enc = new TextEncoder();
+    let bytes = 0;
+    let omitted = 0;
+    const sections = [`\n## Threads (${threads.length})\n`];
+    for (const [path, list] of byPath) {
+        const block = `\n## ${path}\n${list.map(renderThread).join('')}`;
+        const size = enc.encode(block).length;
+        if (bytes + size > MAX_EXISTING_COMMENTS_BYTES) {
+            omitted += list.length;
+            continue;
+        }
+        bytes += size;
+        sections.push(block);
+    }
+    const footer = omitted > 0
+        ? `\n_${omitted} more thread(s) omitted to stay within the injected-context budget._\n`
+        : '';
+    return `${header}${index.join('\n')}\n${sections.join('')}${footer}`;
+}
+// ---------------------------------------------------------------------------
+// `.cat-context/standard-<id>.md`
+// ---------------------------------------------------------------------------
+/** A short, stable base36 hash of a string — enough to disambiguate a sanitized-name collision. */
+function shortHash(value) {
+    let h = 2166136261; // FNV-1a
+    for (let i = 0; i < value.length; i++) {
+        h ^= value.charCodeAt(i);
+        h = Math.imul(h, 16777619);
+    }
+    return (h >>> 0).toString(36);
+}
+/**
+ * The `.cat-context/` filename a resolved best-practice standard is written to. Non-portable
+ * characters are replaced with `-` so the name survives the harness's context-file sanitizer
+ * (it keeps only `[A-Za-z0-9._-]` and flattens any directory) unchanged — the name in the index
+ * is then the name on disk.
+ *
+ * When sanitizing ALTERS the id, two distinct ids can sanitize to the same slug (`org/team` and
+ * `org team` both → `org-team`); the harness drops the duplicate path, silently losing the second
+ * standard while the index still advertises it. So a short hash of the ORIGINAL id is appended
+ * whenever the slug differs from the id, making the filename unique per id. An already-safe id is
+ * left untouched (no suffix), so the common case stays readable.
+ */
+export function standardsContextFileName(fragmentId) {
+    const slug = fragmentId.replace(/[^A-Za-z0-9._-]/g, '-').replace(/^[.-]+/, '') || 'unnamed';
+    const suffix = slug === fragmentId ? '' : `-${shortHash(fragmentId)}`;
+    return `${PR_STANDARD_CONTEXT_PREFIX}${slug}${suffix}.md`;
+}
+/** Render one standard as its own context file, with the title the reviewer cites it by. */
+export function renderStandardContext(fragment) {
+    const title = fragment.title?.trim() || fragment.id;
+    return (`# ${title}\n\n` +
+        `Best-practice standard \`${fragment.id}\`, selected for this review task. Cite it by its ` +
+        'title in `fragmentAdherence`.\n\n' +
+        '---\n\n' +
+        `${fragment.body.trim()}\n`);
+}
+/** Render the index that tells the reviewer which standards exist and where each one is. */
+export function renderStandardsIndex(fragments) {
+    return ('# Best-practice standards for this review\n\n' +
+        'These are the standards this review is judged against. Each is a SEPARATE file so a slice ' +
+        'reviewer reads only the ones its slice needs — do not read them all into one context.\n\n' +
+        'When you dispatch a slice reviewer, name the standards that apply to that slice and tell it ' +
+        'to READ those files itself. Do NOT paraphrase a standard into the subagent’s prompt: a ' +
+        'summary is not the standard, and `fragmentAdherence` ratings must come from the real text.\n\n' +
+        fragments
+            .map((f) => `- **${f.title?.trim() || f.id}** (\`${f.id}\`) — \`.cat-context/${standardsContextFileName(f.id)}\``)
+            .join('\n') +
+        '\n');
+}
+/** The index file listing every injected standard (the shared `context-files` convention). */
+export const PR_STANDARDS_INDEX_CONTEXT_FILE = STANDARDS_CONTEXT_INDEX_FILE;
+// ---------------------------------------------------------------------------
+// PreOps
+// ---------------------------------------------------------------------------
+/**
+ * PreOp for the `pr-reviewer` kind: hand the reviewer the PR's changed-file list, change shape and
+ * a suggested slicing as `.cat-context/pr-diff.md` (plus the patches when the whole diff is small
+ * enough to review in one pass). Pass-through — injecting nothing, so the prompt's git fallback
+ * runs — when the PR number can't be resolved, the bound client can't list changed files
+ * (unwired / a VCS provider without the capability), or the PR reports no changed files.
+ */
+export const prReviewerDiffPreOp = async (ctx) => {
+    const listChangedFiles = ctx.repo.listChangedFiles;
+    if (!listChangedFiles)
+        return;
+    const number = resolvePrNumber(ctx.context.block.taskTypeFields);
+    if (number == null)
+        return;
+    const files = await listChangedFiles(number);
+    if (!files.length)
+        return;
+    return {
+        contextFiles: [{ path: PR_DIFF_CONTEXT_FILE, content: renderPrDiffContext(number, files) }],
+    };
+};
+/**
+ * PreOp for the `pr-reviewer` kind: hand the reviewer the PR's EXISTING review threads up front as
+ * `.cat-context/pr-existing-comments.md`, grouped by file, so it de-dups against findings already
+ * raised instead of re-reporting them. Pass-through — injecting nothing — when the PR number can't
+ * be resolved, the bound client can't read review threads, or the PR has no review threads yet.
+ */
+export const prReviewerExistingCommentsPreOp = async (ctx) => {
+    const listReviewThreads = ctx.repo.listReviewThreads;
+    if (!listReviewThreads)
+        return;
+    const number = resolvePrNumber(ctx.context.block.taskTypeFields);
+    if (number == null)
+        return;
+    const threads = await listReviewThreads(number);
+    if (!threads.length)
+        return;
+    return {
+        contextFiles: [
+            {
+                path: PR_EXISTING_COMMENTS_CONTEXT_FILE,
+                content: renderExistingReviewComments(number, threads),
+            },
+        ],
+    };
+};
+/**
+ * PreOp for the `pr-reviewer` kind: write the task's selected best-practice standards as one
+ * `.cat-context/standard-<id>.md` file each, plus an index.
+ *
+ * The reviewer's own prompt does NOT carry the standards (the kind declares
+ * `standardsDelivery: 'context-files'`, so the engine skips the fold). Folding them in charged
+ * the parent for every standard on every turn — 145 KB across 5 standards on the measured run,
+ * ~3.7M tokens over 96 turns — while the agents that actually review the code, the parallel slice
+ * subagents, never received them and worked from the parent's one-line paraphrase instead. As
+ * files, each standard is read once by the slices it applies to, from the real text.
+ *
+ * Pass-through when the run resolved no fragments (a review task with none selected).
+ */
+export const prReviewerStandardsPreOp = async (ctx) => {
+    const fragments = ctx.context.block.resolvedFragments ?? [];
+    if (!fragments.length)
+        return;
+    return {
+        contextFiles: [
+            { path: PR_STANDARDS_INDEX_CONTEXT_FILE, content: renderStandardsIndex(fragments) },
+            ...fragments.map((fragment) => ({
+                path: standardsContextFileName(fragment.id),
+                content: renderStandardContext(fragment),
+            })),
+        ],
+    };
+};
+//# sourceMappingURL=pr-review-context.js.map

package/dist/agents/kinds/pr-review-context.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"pr-review-context.js","sourceRoot":"","sources":["../../../src/agents/kinds/pr-review-context.ts"],"names":[],"mappings":"AAQA,OAAO,EACL,6BAA6B,EAC7B,4BAA4B,GAC7B,MAAM,yBAAyB,CAAA;AAEhC,8EAA8E;AAC9E,2EAA2E;AAC3E,EAAE;AACF,0FAA0F;AAC1F,sFAAsF;AACtF,0FAA0F;AAC1F,4FAA4F;AAC5F,uCAAuC;AACvC,EAAE;AACF,8EAA8E;AAC9E,EAAE;AACF,8FAA8F;AAC9F,2FAA2F;AAC3F,4FAA4F;AAC5F,6FAA6F;AAC7F,4FAA4F;AAC5F,+FAA+F;AAC/F,oFAAoF;AACpF,4FAA4F;AAC5F,6FAA6F;AAC7F,8FAA8F;AAC9F,4FAA4F;AAC5F,6EAA6E;AAC7E,yFAAyF;AACzF,+FAA+F;AAC/F,+FAA+F;AAC/F,oEAAoE;AACpE,8EAA8E;AAE9E,gGAAgG;AAChG,MAAM,CAAC,MAAM,oBAAoB,GAAG,YAAY,CAAA;AAEhD,8FAA8F;AAC9F,MAAM,CAAC,MAAM,iCAAiC,GAAG,yBAAyB,CAAA;AAE1E;;;;;GAKG;AACH,MAAM,CAAC,MAAM,0BAA0B,GAAG,6BAA6B,CAAA;AAEvE;;;;;GAKG;AACH,MAAM,qBAAqB,GAAG,EAAE,GAAG,IAAI,CAAA;AAEvC,kGAAkG;AAClG,MAAM,sBAAsB,GAAG,EAAE,GAAG,IAAI,CAAA;AAExC;;;;GAIG;AACH,MAAM,2BAA2B,GAAG,EAAE,GAAG,IAAI,CAAA;AAE7C,kGAAkG;AAClG,MAAM,iBAAiB,GAAG,CAAC,CAAA;AAE3B,oGAAoG;AACpG,MAAM,eAAe,GAAG,EAAE,CAAA;AAC1B,MAAM,uBAAuB,GAAG,KAAK,CAAA;AAErC,qGAAqG;AACrG,MAAM,eAAe,GAAG,CAAC,CAAA;AAEzB,8EAA8E;AAC9E,uBAAuB;AACvB,8EAA8E;AAE9E,8FAA8F;AAC9F,MAAM,UAAU,eAAe,CAC7B,MAAyD;IAEzD,IAAI,CAAC,MAAM;QAAE,OAAO,IAAI,CAAA;IACxB,IACE,OAAO,MAAM,CAAC,QAAQ,KAAK,QAAQ;QACnC,MAAM,CAAC,SAAS,CAAC,MAAM,CAAC,QAAQ,CAAC;QACjC,MAAM,CAAC,QAAQ,GAAG,CAAC;QAEnB,OAAO,MAAM,CAAC,QAAQ,CAAA;IACxB,MAAM,GAAG,GAAG,MAAM,CAAC,KAAK,EAAE,IAAI,EAAE,CAAA;IAChC,IAAI,CAAC,GAAG;QAAE,OAAO,IAAI,CAAA;IACrB,qFAAqF;IACrF,MAAM,CAAC,GAAG,GAAG,CAAC,KAAK,CAAC,oDAAoD,CAAC,CAAA;IACzE,MAAM,GAAG,GAAG,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC,CAAC,CAAA;IAC5B,MAAM,CAAC,GAAG,GAAG,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAA;IACxC,OAAO,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAA;AAChD,CAAC;AAcD,yFAAyF;AACzF,SAAS,QAAQ,CAAC,IAAY;IAC5B,MAAM,QAAQ,GAAG,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAA;IAChC,QAAQ,CAAC,GAAG,EAAE,CAAA,CAAC,eAAe;IAC9B,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,aAAa,CAAA;IAC/C,OAAO,QAAQ,CAAC,KAAK,CAAC,CAAC,EAAE,iBAAiB,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAA;AACvD,CAAC;AAED,SAAS,cAAc,CAAC,IAAuB;IAC7C,OAAO,IAAI,CAAC,SAAS,GAAG,IAAI,CAAC,SAAS,CAAA;AACxC,CAAC;AAED;;;;;;;;;;;;GAYG;AACH,MAAM,UAAU,UAAU,CAAC,KAA0B;IACnD,MAAM,MAAM,GAAG,IAAI,GAAG,EAA+B,CAAA;IACrD,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,MAAM,GAAG,GAAG,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QAC/B,MAAM,MAAM,GAAG,MAAM,CAAC,GAAG,CAAC,GAAG,CAAC,CAAA;QAC9B,IAAI,MAAM;YAAE,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;;YACxB,MAAM,CAAC,GAAG,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC,CAAA;IAC9B,CAAC;IAED,sFAAsF;IACtF,MAAM,QAAQ,GAAwB,EAAE,CAAA;IACxC,MAAM,KAAK,GAAoC,EAAE,CAAA;IACjD,KAAK,MAAM,CAAC,GAAG,EAAE,MAAM,CAAC,IAAI,MAAM,EAAE,CAAC;QACnC,IAAI,MAAM,CAAC,MAAM,GAAG,eAAe;YAAE,QAAQ,CAAC,IAAI,CAAC,GAAG,MAAM,CAAC,CAAA;;YACxD,KAAK,CAAC,IAAI,CAAC,CAAC,GAAG,EAAE,MAAM,CAAC,CAAC,CAAA;IAChC,CAAC;IACD,IAAI,QAAQ,CAAC,MAAM;QAAE,KAAK,CAAC,IAAI,CAAC,CAAC,UAAU,EAAE,QAAQ,CAAC,CAAC,CAAA;IAEvD,MAAM,MAAM,GAAqB,EAAE,CAAA;IACnC,KAAK,MAAM,CAAC,GAAG,EAAE,MAAM,CAAC,IAAI,KAAK,EAAE,CAAC;QAClC,qFAAqF;QACrF,6CAA6C;QAC7C,MAAM,MAAM,GAAG,CAAC,GAAG,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,cAAc,CAAC,CAAC,CAAC,GAAG,cAAc,CAAC,CAAC,CAAC,CAAC,CAAA;QAChF,MAAM,KAAK,GAA0B,CAAC,EAAE,CAAC,CAAA;QACzC,IAAI,KAAK,GAAG,CAAC,CAAA;QACb,KAAK,MAAM,IAAI,IAAI,MAAM,EAAE,CAAC;YAC1B,MAAM,OAAO,GAAG,KAAK,CAAC,KAAK,CAAC,MAAM,GAAG,CAAC,CAAE,CAAA;YACxC,MAAM,WAAW,GACf,OAAO,CAAC,MAAM,IAAI,eAAe;gBACjC,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC,IAAI,KAAK,GAAG,cAAc,CAAC,IAAI,CAAC,GAAG,uBAAuB,CAAC,CAAA;YAChF,IAAI,WAAW,EAAE,CAAC;gBAChB,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,CAAC,CAAA;gBAClB,KAAK,GAAG,cAAc,CAAC,IAAI,CAAC,CAAA;YAC9B,CAAC;iBAAM,CAAC;gBACN,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;gBAClB,KAAK,IAAI,cAAc,CAAC,IAAI,CAAC,CAAA;YAC/B,CAAC;QACH,CAAC;QACD,KAAK,MAAM,CAAC,KAAK,EAAE,IAAI,CAAC,IAAI,KAAK,CAAC,OAAO,EAAE,EAAE,CAAC;YAC5C,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC;gBAAE,SAAQ;YAC/B,MAAM,CAAC,IAAI,CAAC;gBACV,KAAK,EAAE,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,GAAG,UAAU,KAAK,GAAG,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG;gBAC5D,wEAAwE;gBACxE,KAAK,EAAE,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE;gBACrC,YAAY,EAAE,IAAI,CAAC,MAAM,CAAC,CAAC,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,GAAG,GAAG,cAAc,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;aAClE,CAAC,CAAA;QACJ,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,YAAY,GAAG,CAAC,CAAC,YAAY,CAAC,CAAA;AAC/D,CAAC;AAED,8EAA8E;AAC9E,4BAA4B;AAC5B,8EAA8E;AAE9E,0FAA0F;AAC1F,SAAS,qBAAqB,CAAC,KAA0B;IACvD,MAAM,MAAM,GAAG,IAAI,GAAG,EAA4C,CAAA;IAClE,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,MAAM,GAAG,GAAG,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QAC/B,MAAM,KAAK,GAAG,MAAM,CAAC,GAAG,CAAC,GAAG,CAAC,IAAI,EAAE,KAAK,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAA;QACvD,KAAK,CAAC,KAAK,IAAI,CAAC,CAAA;QAChB,KAAK,CAAC,KAAK,IAAI,cAAc,CAAC,IAAI,CAAC,CAAA;QACnC,MAAM,CAAC,GAAG,CAAC,GAAG,EAAE,KAAK,CAAC,CAAA;IACxB,CAAC;IACD,MAAM,IAAI,GAAG,CAAC,GAAG,MAAM,CAAC,OAAO,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAA;IAC1E,OAAO;QACL,sBAAsB,IAAI,CAAC,MAAM,WAAW;QAC5C,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,GAAG,MAAM,CAAC,CAAC,KAAK,aAAa,CAAC,CAAC,KAAK,kBAAkB,CAAC;KACvF,CAAA;AACH,CAAC;AAED,SAAS,qBAAqB,CAAC,MAAwB;IACrD,OAAO;QACL,2BAA2B,MAAM,CAAC,MAAM,YAAY;QACpD,sFAAsF;QACtF,2FAA2F;QAC3F,6FAA6F;QAC7F,qFAAqF;QACrF,GAAG,MAAM,CAAC,GAAG,CACX,CAAC,CAAC,EAAE,EAAE,CACJ,OAAO,CAAC,CAAC,KAAK,OAAO,CAAC,CAAC,KAAK,CAAC,MAAM,aAAa,CAAC,CAAC,YAAY,qBAAqB;YACnF,CAAC,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAC5C;KACF,CAAA;AACH,CAAC;AAED,+FAA+F;AAC/F,SAAS,eAAe,CAAC,KAAa,EAAE,KAAa;IACnD,OAAO,CACL,+BAA+B,KAAK,YAAY,IAAI,CAAC,IAAI,CAAC,KAAK,GAAG,IAAI,CAAC,oBAAoB;QAC3F,+FAA+F;QAC/F,6FAA6F;QAC7F,SAAS;QACT,oFAAoF;QACpF,2FAA2F;QAC3F,SAAS;QACT,gGAAgG;QAChG,uEAAuE,CACxE,CAAA;AACH,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,mBAAmB,CAAC,MAAc,EAAE,KAA0B;IAC5E,MAAM,GAAG,GAAG,IAAI,WAAW,EAAE,CAAA;IAC7B,0FAA0F;IAC1F,kFAAkF;IAClF,MAAM,SAAS,GAAG,KAAK,CAAC,MAAM,CAC5B,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,IAAI,GAAG,CAAC,MAAM,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,MAAM,IAAI,sBAAsB,CAC/E,CAAA;IACD,MAAM,WAAW,GAAG,SAAS,CAAC,MAAM,CAAC,CAAC,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,GAAG,GAAG,GAAG,CAAC,MAAM,CAAC,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,CAAA;IAC3F,MAAM,aAAa,GAAG,WAAW,IAAI,qBAAqB,CAAA;IAC1D,6FAA6F;IAC7F,4FAA4F;IAC5F,2FAA2F;IAC3F,MAAM,eAAe,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,GAAG,GAAG,GAAG,CAAC,MAAM,CAAC,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,CAAA;IAE3F,MAAM,MAAM,GACV,mBAAmB,MAAM,+BAA+B;QACxD,2FAA2F;QAC3F,iFAAiF;QACjF,qBAAqB,CAAA;IAEvB,MAAM,IAAI,GAAG;QACX,uBAAuB,KAAK,CAAC,MAAM,KAAK;QACxC,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE;YACjB,MAAM,MAAM,GAAG,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,kBAAkB,CAAC,CAAC,YAAY,GAAG,CAAC,CAAC,CAAC,EAAE,CAAA;YACxE,OAAO,KAAK,CAAC,CAAC,MAAM,IAAI,CAAC,CAAC,IAAI,MAAM,CAAC,CAAC,SAAS,KAAK,CAAC,CAAC,SAAS,IAAI,MAAM,EAAE,CAAA;QAC7E,CAAC,CAAC;KACH,CAAA;IAED,MAAM,QAAQ,GAAG;QACf,MAAM;QACN,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC;QACf,qBAAqB,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC;QACvC,qBAAqB,CAAC,UAAU,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC;KACpD,CAAA;IAED,IAAI,CAAC,aAAa,EAAE,CAAC;QACnB,QAAQ,CAAC,IAAI,CAAC,eAAe,CAAC,KAAK,CAAC,MAAM,EAAE,eAAe,CAAC,CAAC,CAAA;QAC7D,OAAO,GAAG,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAA;IACnC,CAAC;IAED,MAAM,OAAO,GAAa,CAAC,gBAAgB,CAAC,CAAA;IAC5C,KAAK,MAAM,CAAC,IAAI,KAAK,EAAE,CAAC;QACtB,MAAM,OAAO,GAAG,SAAS,CAAC,CAAC,IAAI,KAAK,CAAC,CAAC,MAAM,MAAM,CAAC,CAAC,SAAS,KAAK,CAAC,CAAC,SAAS,KAAK,CAAA;QAClF,IAAI,CAAC,CAAC,KAAK,IAAI,IAAI,EAAE,CAAC;YACpB,OAAO,CAAC,IAAI,CAAC,GAAG,OAAO,qEAAqE,CAAC,CAAA;QAC/F,CAAC;aAAM,IAAI,GAAG,CAAC,MAAM,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,MAAM,GAAG,sBAAsB,EAAE,CAAC;YAC/D,MAAM,GAAG,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,MAAM,GAAG,IAAI,CAAC,CAAA;YACxD,OAAO,CAAC,IAAI,CACV,GAAG,OAAO,WAAW,GAAG,uDAAuD;gBAC7E,qCAAqC,CACxC,CAAA;QACH,CAAC;aAAM,CAAC;YACN,OAAO,CAAC,IAAI,CAAC,GAAG,OAAO,eAAe,CAAC,CAAC,KAAK,YAAY,CAAC,CAAA;QAC5D,CAAC;IACH,CAAC;IACD,QAAQ,CAAC,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC,CAAA;IAC/B,OAAO,GAAG,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAA;AACnC,CAAC;AAED,8EAA8E;AAC9E,yCAAyC;AACzC,8EAA8E;AAE9E,sGAAsG;AACtG,SAAS,cAAc,CAAC,IAAY,EAAE,GAAG,GAAG,GAAG;IAC7C,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,EAAE,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAA;IAC7C,OAAO,IAAI,CAAC,MAAM,GAAG,GAAG,CAAC,CAAC,CAAC,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAA;AAC5D,CAAC;AAED,SAAS,YAAY,CAAC,MAA0B;IAC9C,MAAM,MAAM,GAAG,MAAM,CAAC,IAAI;QACxB,CAAC,CAAC,GAAG,MAAM,CAAC,IAAI,GAAG,MAAM,CAAC,IAAI,IAAI,IAAI,CAAC,CAAC,CAAC,IAAI,MAAM,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,EAAE;QACjE,CAAC,CAAC,SAAS,CAAA;IACb,MAAM,KAAK,GAAG,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,YAAY,CAAA;IAC3D,MAAM,KAAK,GAAG,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAA;IAChC,MAAM,MAAM,GAAG,KAAK,EAAE,MAAM,CAAC,CAAC,CAAC,IAAI,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC,SAAS,CAAA;IAC7D,MAAM,OAAO,GAAG,KAAK,CAAC,CAAC,CAAC,cAAc,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,mBAAmB,CAAA;IACxE,MAAM,KAAK,GAAG,MAAM,CAAC,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAA;IACxC,MAAM,OAAO,GAAG,KAAK,GAAG,CAAC,CAAC,CAAC,CAAC,MAAM,KAAK,QAAQ,KAAK,KAAK,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,CAAC,EAAE,CAAA;IAChF,OAAO,SAAS,MAAM,MAAM,KAAK,KAAK,MAAM,GAAG,OAAO,KAAK,OAAO,IAAI,CAAA;AACxE,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,4BAA4B,CAC1C,MAAc,EACd,OAA6B;IAE7B,MAAM,MAAM,GACV,mBAAmB,MAAM,iCAAiC;QAC1D,gGAAgG;QAChG,+FAA+F;QAC/F,kGAAkG;QAClG,oBAAoB;QACpB,8FAA8F;QAC9F,kGAAkG,CAAA;IAEpG,2FAA2F;IAC3F,MAAM,MAAM,GAAG,IAAI,GAAG,EAAgC,CAAA;IACtD,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,IAAI,WAAW,CAAA;QACtC,MAAM,MAAM,GAAG,MAAM,CAAC,GAAG,CAAC,GAAG,CAAC,CAAA;QAC9B,IAAI,MAAM;YAAE,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;;YAC1B,MAAM,CAAC,GAAG,CAAC,GAAG,EAAE,CAAC,MAAM,CAAC,CAAC,CAAA;IAChC,CAAC;IAED,MAAM,KAAK,GAAG;QACZ,qCAAqC,MAAM,CAAC,IAAI,KAAK;QACrD,GAAG,CAAC,GAAG,MAAM,CAAC,OAAO,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,EAAE,IAAI,CAAC,EAAE,EAAE,CAAC,KAAK,IAAI,KAAK,IAAI,CAAC,MAAM,aAAa,CAAC;KACvF,CAAA;IAED,MAAM,GAAG,GAAG,IAAI,WAAW,EAAE,CAAA;IAC7B,IAAI,KAAK,GAAG,CAAC,CAAA;IACb,IAAI,OAAO,GAAG,CAAC,CAAA;IACf,MAAM,QAAQ,GAAa,CAAC,iBAAiB,OAAO,CAAC,MAAM,KAAK,CAAC,CAAA;IACjE,KAAK,MAAM,CAAC,IAAI,EAAE,IAAI,CAAC,IAAI,MAAM,EAAE,CAAC;QAClC,MAAM,KAAK,GAAG,QAAQ,IAAI,KAAK,IAAI,CAAC,GAAG,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,CAAA;QAChE,MAAM,IAAI,GAAG,GAAG,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,MAAM,CAAA;QACrC,IAAI,KAAK,GAAG,IAAI,GAAG,2BAA2B,EAAE,CAAC;YAC/C,OAAO,IAAI,IAAI,CAAC,MAAM,CAAA;YACtB,SAAQ;QACV,CAAC;QACD,KAAK,IAAI,IAAI,CAAA;QACb,QAAQ,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IACtB,CAAC;IACD,MAAM,MAAM,GACV,OAAO,GAAG,CAAC;QACT,CAAC,CAAC,MAAM,OAAO,wEAAwE;QACvF,CAAC,CAAC,EAAE,CAAA;IACR,OAAO,GAAG,MAAM,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,KAAK,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC,GAAG,MAAM,EAAE,CAAA;AACtE,CAAC;AAED,8EAA8E;AAC9E,kCAAkC;AAClC,8EAA8E;AAE9E,mGAAmG;AACnG,SAAS,SAAS,CAAC,KAAa;IAC9B,IAAI,CAAC,GAAG,UAAU,CAAA,CAAC,SAAS;IAC5B,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACtC,CAAC,IAAI,KAAK,CAAC,UAAU,CAAC,CAAC,CAAC,CAAA;QACxB,CAAC,GAAG,IAAI,CAAC,IAAI,CAAC,CAAC,EAAE,QAAQ,CAAC,CAAA;IAC5B,CAAC;IACD,OAAO,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAA;AAC/B,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,wBAAwB,CAAC,UAAkB;IACzD,MAAM,IAAI,GAAG,UAAU,CAAC,OAAO,CAAC,kBAAkB,EAAE,GAAG,CAAC,CAAC,OAAO,CAAC,QAAQ,EAAE,EAAE,CAAC,IAAI,SAAS,CAAA;IAC3F,MAAM,MAAM,GAAG,IAAI,KAAK,UAAU,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,SAAS,CAAC,UAAU,CAAC,EAAE,CAAA;IACrE,OAAO,GAAG,0BAA0B,GAAG,IAAI,GAAG,MAAM,KAAK,CAAA;AAC3D,CAAC;AAED,4FAA4F;AAC5F,MAAM,UAAU,qBAAqB,CAAC,QAA4B;IAChE,MAAM,KAAK,GAAG,QAAQ,CAAC,KAAK,EAAE,IAAI,EAAE,IAAI,QAAQ,CAAC,EAAE,CAAA;IACnD,OAAO,CACL,KAAK,KAAK,MAAM;QAChB,4BAA4B,QAAQ,CAAC,EAAE,oDAAoD;QAC3F,mCAAmC;QACnC,SAAS;QACT,GAAG,QAAQ,CAAC,IAAI,CAAC,IAAI,EAAE,IAAI,CAC5B,CAAA;AACH,CAAC;AAED,4FAA4F;AAC5F,MAAM,UAAU,oBAAoB,CAAC,SAA+B;IAClE,OAAO,CACL,+CAA+C;QAC/C,4FAA4F;QAC5F,2FAA2F;QAC3F,8FAA8F;QAC9F,yFAAyF;QACzF,gGAAgG;QAChG,SAAS;aACN,GAAG,CACF,CAAC,CAAC,EAAE,EAAE,CACJ,OAAO,CAAC,CAAC,KAAK,EAAE,IAAI,EAAE,IAAI,CAAC,CAAC,EAAE,SAAS,CAAC,CAAC,EAAE,wBAAwB,wBAAwB,CAAC,CAAC,CAAC,EAAE,CAAC,IAAI,CACxG;aACA,IAAI,CAAC,IAAI,CAAC;QACb,IAAI,CACL,CAAA;AACH,CAAC;AAED,8FAA8F;AAC9F,MAAM,CAAC,MAAM,+BAA+B,GAAG,4BAA4B,CAAA;AAE3E,8EAA8E;AAC9E,SAAS;AACT,8EAA8E;AAE9E;;;;;;GAMG;AACH,MAAM,CAAC,MAAM,mBAAmB,GAAW,KAAK,EAC9C,GAAkB,EACY,EAAE;IAChC,MAAM,gBAAgB,GAAG,GAAG,CAAC,IAAI,CAAC,gBAAgB,CAAA;IAClD,IAAI,CAAC,gBAAgB;QAAE,OAAM;IAC7B,MAAM,MAAM,GAAG,eAAe,CAAC,GAAG,CAAC,OAAO,CAAC,KAAK,CAAC,cAAc,CAAC,CAAA;IAChE,IAAI,MAAM,IAAI,IAAI;QAAE,OAAM;IAC1B,MAAM,KAAK,GAAG,MAAM,gBAAgB,CAAC,MAAM,CAAC,CAAA;IAC5C,IAAI,CAAC,KAAK,CAAC,MAAM;QAAE,OAAM;IACzB,OAAO;QACL,YAAY,EAAE,CAAC,EAAE,IAAI,EAAE,oBAAoB,EAAE,OAAO,EAAE,mBAAmB,CAAC,MAAM,EAAE,KAAK,CAAC,EAAE,CAAC;KAC5F,CAAA;AACH,CAAC,CAAA;AAED;;;;;GAKG;AACH,MAAM,CAAC,MAAM,+BAA+B,GAAW,KAAK,EAC1D,GAAkB,EACY,EAAE;IAChC,MAAM,iBAAiB,GAAG,GAAG,CAAC,IAAI,CAAC,iBAAiB,CAAA;IACpD,IAAI,CAAC,iBAAiB;QAAE,OAAM;IAC9B,MAAM,MAAM,GAAG,eAAe,CAAC,GAAG,CAAC,OAAO,CAAC,KAAK,CAAC,cAAc,CAAC,CAAA;IAChE,IAAI,MAAM,IAAI,IAAI;QAAE,OAAM;IAC1B,MAAM,OAAO,GAAG,MAAM,iBAAiB,CAAC,MAAM,CAAC,CAAA;IAC/C,IAAI,CAAC,OAAO,CAAC,MAAM;QAAE,OAAM;IAC3B,OAAO;QACL,YAAY,EAAE;YACZ;gBACE,IAAI,EAAE,iCAAiC;gBACvC,OAAO,EAAE,4BAA4B,CAAC,MAAM,EAAE,OAAO,CAAC;aACvD;SACF;KACF,CAAA;AACH,CAAC,CAAA;AAED;;;;;;;;;;;;GAYG;AACH,MAAM,CAAC,MAAM,wBAAwB,GAAW,KAAK,EACnD,GAAkB,EACY,EAAE;IAChC,MAAM,SAAS,GAAG,GAAG,CAAC,OAAO,CAAC,KAAK,CAAC,iBAAiB,IAAI,EAAE,CAAA;IAC3D,IAAI,CAAC,SAAS,CAAC,MAAM;QAAE,OAAM;IAC7B,OAAO;QACL,YAAY,EAAE;YACZ,EAAE,IAAI,EAAE,+BAA+B,EAAE,OAAO,EAAE,oBAAoB,CAAC,SAAS,CAAC,EAAE;YACnF,GAAG,SAAS,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE,EAAE,CAAC,CAAC;gBAC9B,IAAI,EAAE,wBAAwB,CAAC,QAAQ,CAAC,EAAE,CAAC;gBAC3C,OAAO,EAAE,qBAAqB,CAAC,QAAQ,CAAC;aACzC,CAAC,CAAC;SACJ;KACF,CAAA;AACH,CAAC,CAAA"}

package/dist/agents/kinds/pr-reviewer.d.ts

@@ -1,4 +1,3 @@
-import type { GitHubChangedFile, GitHubReviewThread, RepoOp } from '@cat-factory/kernel';
 import type { AgentKindDefinition, AgentKindRegistry } from './registry.js';
 export declare const PR_REVIEWER_KIND = "pr-reviewer";
 /**
@@ -34,44 +33,11 @@ export declare const prReview: import("./structured-output.js").StructuredOutput
 }>;
 export type PrReviewOutput = ReturnType<typeof prReview.parse>;
 export declare const PR_REVIEWER_SYSTEM_PROMPT: string;
-/** The injected context file (under `.cat-context/`) the diff preOp writes for the reviewer. */
-export declare const PR_DIFF_CONTEXT_FILE = "pr-diff.md";
-/** The injected context file listing the PR's already-posted review comments (for de-dup). */
-export declare const PR_EXISTING_COMMENTS_CONTEXT_FILE = "pr-existing-comments.md";
-/** Resolve the reviewed PR's number from the block's task-type fields (prefer `prNumber`). */
-export declare function resolvePrNumber(fields: {
-    prNumber?: number;
-    prUrl?: string;
-} | undefined): number | null;
-/** Render the changed-file list + budgeted patches as the injected `.cat-context/pr-diff.md`. */
-export declare function renderPrDiffContext(number: number, files: GitHubChangedFile[]): string;
-/**
- * PreOp for the `pr-reviewer` kind: hand the reviewer the PR's changed-file list + diff UP FRONT
- * as `.cat-context/pr-diff.md`, so the container agent skips the early `git fetch`/`git diff`/
- * scratch-file reconstruction turns that dominate a long review's token burn (each agentic turn
- * re-sends the whole growing transcript). Pass-through — injecting nothing, so the prompt's git
- * fallback runs — when the PR number can't be resolved, the bound client can't list changed files
- * (unwired / a VCS provider without the capability), or the PR reports no changed files.
- */
-export declare const prReviewerDiffPreOp: RepoOp;
-/**
- * Render the PR's existing review threads as the injected `.cat-context/pr-existing-comments.md`, so
- * the reviewer can de-dup against findings already raised (prior rounds / humans / other bots). Each
- * thread shows its anchor (path:line, or "general"), resolved state and the opening comment's text.
- */
-export declare function renderExistingReviewComments(number: number, threads: GitHubReviewThread[]): string;
-/**
- * PreOp for the `pr-reviewer` kind: hand the reviewer the PR's EXISTING review threads up front as
- * `.cat-context/pr-existing-comments.md`, so it de-dups against findings already raised (prior
- * review rounds, human reviewers, other bots) instead of re-reporting them. Pass-through — injecting
- * nothing — when the PR number can't be resolved, the bound client can't read review threads
- * (unwired / a VCS provider without the capability), or the PR has no review threads yet.
- */
-export declare const prReviewerExistingCommentsPreOp: RepoOp;
 export declare const PR_REVIEWER_AGENT_KINDS: AgentKindDefinition[];
 /**
  * Register the pr-reviewer kind on the given registry. Called by `defaultAgentKindRegistry()`;
  * idempotent (the registry replaces by kind).
  */
 export declare function registerPrReviewerAgent(registry: AgentKindRegistry): void;
+export { PR_DIFF_CONTEXT_FILE, PR_EXISTING_COMMENTS_CONTEXT_FILE, PR_STANDARDS_INDEX_CONTEXT_FILE, planSlices, prReviewerDiffPreOp, prReviewerExistingCommentsPreOp, prReviewerStandardsPreOp, renderExistingReviewComments, renderPrDiffContext, renderStandardContext, renderStandardsIndex, resolvePrNumber, standardsContextFileName, type SuggestedSlice, } from './pr-review-context.js';
 //# sourceMappingURL=pr-reviewer.d.ts.map