@cat-factory/agents 0.65.5 → 0.66.1

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

@@ -1,3 +1,4 @@
+import type { GitHubChangedFile, RepoOp } from '@cat-factory/kernel';
 import type { AgentKindDefinition, AgentKindRegistry } from './registry.js';
 export declare const PR_REVIEWER_KIND = "pr-reviewer";
 /**
@@ -26,6 +27,24 @@ 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";
+/** 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;
 export declare const PR_REVIEWER_AGENT_KINDS: AgentKindDefinition[];
 /**
  * Register the pr-reviewer kind on the given registry. Called by `defaultAgentKindRegistry()`;

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

@@ -1 +1 @@
-{"version":3,"file":"pr-reviewer.d.ts","sourceRoot":"","sources":["../../../src/agents/kinds/pr-reviewer.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,mBAAmB,EAAE,iBAAiB,EAAE,MAAM,eAAe,CAAA;AA4B3E,eAAO,MAAM,gBAAgB,gBAAgB,CAAA;AAE7C;;;;;GAKG;AACH,eAAO,MAAM,QAAQ;;;;;;;;;;;;;;;;;EAAoD,CAAA;AAEzE,MAAM,MAAM,cAAc,GAAG,UAAU,CAAC,OAAO,QAAQ,CAAC,KAAK,CAAC,CAAA;AAE9D,eAAO,MAAM,yBAAyB,QA0CjC,CAAA;AAEL,eAAO,MAAM,uBAAuB,EAAE,mBAAmB,EAuBxD,CAAA;AAED;;;GAGG;AACH,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,iBAAiB,GAAG,IAAI,CAEzE"}
+{"version":3,"file":"pr-reviewer.d.ts","sourceRoot":"","sources":["../../../src/agents/kinds/pr-reviewer.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,EAA+B,MAAM,qBAAqB,CAAA;AAEjG,OAAO,KAAK,EAAE,mBAAmB,EAAE,iBAAiB,EAAE,MAAM,eAAe,CAAA;AA4B3E,eAAO,MAAM,gBAAgB,gBAAgB,CAAA;AAE7C;;;;;GAKG;AACH,eAAO,MAAM,QAAQ;;;;;;;;;;;;;;;;;EAAoD,CAAA;AAEzE,MAAM,MAAM,cAAc,GAAG,UAAU,CAAC,OAAO,QAAQ,CAAC,KAAK,CAAC,CAAA;AAE9D,eAAO,MAAM,yBAAyB,QAgDjC,CAAA;AAML,gGAAgG;AAChG,eAAO,MAAM,oBAAoB,eAAe,CAAA;AAWhD,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;AAED,iGAAiG;AACjG,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,iBAAiB,EAAE,GAAG,MAAM,CAoCtF;AAED;;;;;;;GAOG;AACH,eAAO,MAAM,mBAAmB,EAAE,MAYjC,CAAA;AAED,eAAO,MAAM,uBAAuB,EAAE,mBAAmB,EAwBxD,CAAA;AAED;;;GAGG;AACH,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,iBAAiB,GAAG,IAAI,CAEzE"}

package/dist/agents/kinds/pr-reviewer.js

@@ -34,14 +34,20 @@ export const PR_REVIEWER_KIND = 'pr-reviewer';
  */
 export const prReview = defineStructuredOutput(prReviewAgentOutputSchema);
 export const PR_REVIEWER_SYSTEM_PROMPT = 'You are a meticulous senior code reviewer performing a DEEP review of an open pull request. ' +
-    'The task names the pull request to review — its number (e.g. #123) and URL. Fetch its head ' +
-    'into the checkout and diff it against the base branch:\n' +
+    'The task names the pull request to review — its number (e.g. #123) and URL. The PR’s ' +
+    'changed-file list and per-file diff have usually been prepared for you in ' +
+    '`.cat-context/pr-diff.md` — READ THAT FIRST and build your review plan from it, rather than ' +
+    'reconstructing the diff by hand. If that file is absent, or a slice needs a patch it does not ' +
+    'contain, fetch the head into the checkout and diff it against the base yourself:\n' +
     '  git fetch origin pull/<number>/head:pr-head\n' +
     '  git diff --name-status origin/<base>...pr-head   # <base> = the default branch unless told otherwise\n' +
+    'You always have the full base checkout: read unchanged neighbouring files (call sites, helpers, ' +
+    'tests) from it, and fetch the PR head for full file bodies, whenever a slice needs more than the ' +
+    'patch.\n' +
     'The PR may be large (hundreds of changed files), so review it in a way that stays within a ' +
     'bounded context rather than reading the whole diff at once:\n' +
-    '1. First list the changed files cheaply with the `--name-status` + `--stat` diffs above. ' +
-    'Do NOT read every patch yet.\n' +
+    '1. First read the changed-file list (from `.cat-context/pr-diff.md`, or the `--name-status` + ' +
+    '`--stat` diffs above). Do NOT read every patch yet.\n' +
     '2. Group the changed files into COHESIVE slices — files that are inherently linked and should ' +
     'be reviewed together (a refactor and its call sites and its tests; a schema change and its ' +
     'migration and its mapper). A slice is a unit you can review with full understanding on its own. ' +
@@ -75,10 +81,99 @@ export const PR_REVIEWER_SYSTEM_PROMPT = 'You are a meticulous senior code revie
     '    "suggestedFix": "a concrete suggested change, when applicable"\n' +
     '  }]\n' +
     '}';
+// ---------------------------------------------------------------------------
+// PreOp: hand the reviewer the PR diff up front.
+// ---------------------------------------------------------------------------
+/** The injected context file (under `.cat-context/`) the diff preOp writes for the reviewer. */
+export const PR_DIFF_CONTEXT_FILE = 'pr-diff.md';
+/**
+ * Byte budget for the injected PATCHES. The changed-file LIST (cheap, and the slicing signal) is
+ * always included in full; per-file patches are appended until this budget is reached, after
+ * which the agent reads the remaining files from its checkout per slice (the ADR's slicing
+ * model). ~256 KiB is a large diff handed over ONCE — far cheaper than reconstructing it across
+ * many transcript-re-sending turns — while never becoming the sole source (the full clone remains).
+ */
+const MAX_PR_DIFF_PATCH_BYTES = 256 * 1024;
+/** 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;
+}
+/** Render the changed-file list + budgeted patches as the injected `.cat-context/pr-diff.md`. */
+export function renderPrDiffContext(number, files) {
+    const header = `# Pull request #${number} — changed files and diff\n\n` +
+        'Prepared from the GitHub API so you can plan your review slices WITHOUT reconstructing the ' +
+        'diff yourself. You still have the full base checkout: read unchanged neighbouring files from ' +
+        `it, and \`git fetch origin pull/${number}/head\` for full head file bodies, when a slice ` +
+        'needs more than the patch below.\n\n';
+    const list = [`## Changed files (${files.length})\n`];
+    for (const f of files) {
+        const rename = f.previousPath ? ` (renamed from ${f.previousPath})` : '';
+        list.push(`- ${f.status} ${f.path} (+${f.additions}/-${f.deletions})${rename}`);
+    }
+    const enc = new TextEncoder();
+    let bytes = 0;
+    let omitted = 0;
+    const patches = [];
+    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`);
+            continue;
+        }
+        const block = `${heading}\`\`\`diff\n${f.patch}\n\`\`\`\n`;
+        const size = enc.encode(block).length;
+        if (bytes + size > MAX_PR_DIFF_PATCH_BYTES) {
+            omitted += 1;
+            continue;
+        }
+        bytes += size;
+        patches.push(block);
+    }
+    const footer = omitted > 0
+        ? `\n_${omitted} file patch(es) omitted to stay within the injected-diff budget — read those files from the checkout when their slice needs them._\n`
+        : '';
+    return `${header}${list.join('\n')}\n\n## Patches\n${patches.join('')}${footer}`;
+}
+/**
+ * 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 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) }],
+    };
+};
 export const PR_REVIEWER_AGENT_KINDS = [
     {
         kind: PR_REVIEWER_KIND,
         systemPrompt: PR_REVIEWER_SYSTEM_PROMPT,
+        preOps: [prReviewerDiffPreOp],
         // Read-only FULL clone of the repo's BASE (default) branch — a review task targets an
         // EXISTING external PR that the run never opened, so there is no work branch to clone; the
         // prompt fetches the PR head by number (`git fetch origin pull/<n>/head`) and diffs it against

package/dist/agents/kinds/pr-reviewer.js.map

@@ -1 +1 @@
-{"version":3,"file":"pr-reviewer.js","sourceRoot":"","sources":["../../../src/agents/kinds/pr-reviewer.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,yBAAyB,EAAE,MAAM,wBAAwB,CAAA;AAClE,OAAO,EAAE,sBAAsB,EAAE,MAAM,wBAAwB,CAAA;AAG/D,8EAA8E;AAC9E,kFAAkF;AAClF,iFAAiF;AACjF,kCAAkC;AAClC,EAAE;AACF,kFAAkF;AAClF,kFAAkF;AAClF,mFAAmF;AACnF,oFAAoF;AACpF,qFAAqF;AACrF,oFAAoF;AACpF,sFAAsF;AACtF,8DAA8D;AAC9D,EAAE;AACF,sFAAsF;AACtF,wFAAwF;AACxF,yFAAyF;AACzF,0FAA0F;AAC1F,iFAAiF;AACjF,2CAA2C;AAC3C,EAAE;AACF,wFAAwF;AACxF,mFAAmF;AACnF,sCAAsC;AACtC,8EAA8E;AAE9E,MAAM,CAAC,MAAM,gBAAgB,GAAG,aAAa,CAAA;AAE7C;;;;;GAKG;AACH,MAAM,CAAC,MAAM,QAAQ,GAAG,sBAAsB,CAAC,yBAAyB,CAAC,CAAA;AAIzE,MAAM,CAAC,MAAM,yBAAyB,GACpC,8FAA8F;IAC9F,6FAA6F;IAC7F,0DAA0D;IAC1D,iDAAiD;IACjD,0GAA0G;IAC1G,6FAA6F;IAC7F,+DAA+D;IAC/D,2FAA2F;IAC3F,gCAAgC;IAChC,gGAAgG;IAChG,6FAA6F;IAC7F,kGAAkG;IAClG,4FAA4F;IAC5F,gGAAgG;IAChG,8FAA8F;IAC9F,qDAAqD;IACrD,+FAA+F;IAC/F,8FAA8F;IAC9F,gGAAgG;IAChG,+BAA+B;IAC/B,8FAA8F;IAC9F,oBAAoB;IACpB,6FAA6F;IAC7F,gGAAgG;IAChG,+FAA+F;IAC/F,8FAA8F;IAC9F,uFAAuF;IACvF,kDAAkD;IAClD,KAAK;IACL,oDAAoD;IACpD,iHAAiH;IACjH,oBAAoB;IACpB,wCAAwC;IACxC,mBAAmB;IACnB,wBAAwB;IACxB,0DAA0D;IAC1D,oGAAoG;IACpG,kCAAkC;IAClC,+CAA+C;IAC/C,sEAAsE;IACtE,QAAQ;IACR,GAAG,CAAA;AAEL,MAAM,CAAC,MAAM,uBAAuB,GAA0B;IAC5D;QACE,IAAI,EAAE,gBAAgB;QACtB,YAAY,EAAE,yBAAyB;QACvC,sFAAsF;QACtF,2FAA2F;QAC3F,+FAA+F;QAC/F,qGAAqG;QACrG,KAAK,EAAE,EAAE,OAAO,EAAE,mBAAmB,EAAE,KAAK,EAAE,EAAE,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,EAAE;QAC9E,gBAAgB,EAAE,QAAQ;QAC1B,YAAY,EAAE;YACZ,KAAK,EAAE,aAAa;YACpB,IAAI,EAAE,0BAA0B;YAChC,KAAK,EAAE,SAAS;YAChB,WAAW,EACT,wFAAwF;gBACxF,yDAAyD;YAC3D,QAAQ,EAAE,QAAQ;YAClB,mFAAmF;YACnF,iFAAiF;YACjF,UAAU,EAAE,WAAW;SACxB;KACF;CACF,CAAA;AAED;;;GAGG;AACH,MAAM,UAAU,uBAAuB,CAAC,QAA2B;IACjE,QAAQ,CAAC,WAAW,CAAC,uBAAuB,CAAC,CAAA;AAC/C,CAAC"}
+{"version":3,"file":"pr-reviewer.js","sourceRoot":"","sources":["../../../src/agents/kinds/pr-reviewer.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,yBAAyB,EAAE,MAAM,wBAAwB,CAAA;AAElE,OAAO,EAAE,sBAAsB,EAAE,MAAM,wBAAwB,CAAA;AAG/D,8EAA8E;AAC9E,kFAAkF;AAClF,iFAAiF;AACjF,kCAAkC;AAClC,EAAE;AACF,kFAAkF;AAClF,kFAAkF;AAClF,mFAAmF;AACnF,oFAAoF;AACpF,qFAAqF;AACrF,oFAAoF;AACpF,sFAAsF;AACtF,8DAA8D;AAC9D,EAAE;AACF,sFAAsF;AACtF,wFAAwF;AACxF,yFAAyF;AACzF,0FAA0F;AAC1F,iFAAiF;AACjF,2CAA2C;AAC3C,EAAE;AACF,wFAAwF;AACxF,mFAAmF;AACnF,sCAAsC;AACtC,8EAA8E;AAE9E,MAAM,CAAC,MAAM,gBAAgB,GAAG,aAAa,CAAA;AAE7C;;;;;GAKG;AACH,MAAM,CAAC,MAAM,QAAQ,GAAG,sBAAsB,CAAC,yBAAyB,CAAC,CAAA;AAIzE,MAAM,CAAC,MAAM,yBAAyB,GACpC,8FAA8F;IAC9F,uFAAuF;IACvF,4EAA4E;IAC5E,8FAA8F;IAC9F,gGAAgG;IAChG,oFAAoF;IACpF,iDAAiD;IACjD,0GAA0G;IAC1G,kGAAkG;IAClG,mGAAmG;IACnG,UAAU;IACV,6FAA6F;IAC7F,+DAA+D;IAC/D,gGAAgG;IAChG,uDAAuD;IACvD,gGAAgG;IAChG,6FAA6F;IAC7F,kGAAkG;IAClG,4FAA4F;IAC5F,gGAAgG;IAChG,8FAA8F;IAC9F,qDAAqD;IACrD,+FAA+F;IAC/F,8FAA8F;IAC9F,gGAAgG;IAChG,+BAA+B;IAC/B,8FAA8F;IAC9F,oBAAoB;IACpB,6FAA6F;IAC7F,gGAAgG;IAChG,+FAA+F;IAC/F,8FAA8F;IAC9F,uFAAuF;IACvF,kDAAkD;IAClD,KAAK;IACL,oDAAoD;IACpD,iHAAiH;IACjH,oBAAoB;IACpB,wCAAwC;IACxC,mBAAmB;IACnB,wBAAwB;IACxB,0DAA0D;IAC1D,oGAAoG;IACpG,kCAAkC;IAClC,+CAA+C;IAC/C,sEAAsE;IACtE,QAAQ;IACR,GAAG,CAAA;AAEL,8EAA8E;AAC9E,iDAAiD;AACjD,8EAA8E;AAE9E,gGAAgG;AAChG,MAAM,CAAC,MAAM,oBAAoB,GAAG,YAAY,CAAA;AAEhD;;;;;;GAMG;AACH,MAAM,uBAAuB,GAAG,GAAG,GAAG,IAAI,CAAA;AAE1C,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;AAED,iGAAiG;AACjG,MAAM,UAAU,mBAAmB,CAAC,MAAc,EAAE,KAA0B;IAC5E,MAAM,MAAM,GACV,mBAAmB,MAAM,+BAA+B;QACxD,6FAA6F;QAC7F,+FAA+F;QAC/F,mCAAmC,MAAM,kDAAkD;QAC3F,sCAAsC,CAAA;IACxC,MAAM,IAAI,GAAG,CAAC,qBAAqB,KAAK,CAAC,MAAM,KAAK,CAAC,CAAA;IACrD,KAAK,MAAM,CAAC,IAAI,KAAK,EAAE,CAAC;QACtB,MAAM,MAAM,GAAG,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,kBAAkB,CAAC,CAAC,YAAY,GAAG,CAAC,CAAC,CAAC,EAAE,CAAA;QACxE,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,MAAM,IAAI,CAAC,CAAC,IAAI,MAAM,CAAC,CAAC,SAAS,KAAK,CAAC,CAAC,SAAS,IAAI,MAAM,EAAE,CAAC,CAAA;IACjF,CAAC;IACD,MAAM,GAAG,GAAG,IAAI,WAAW,EAAE,CAAA;IAC7B,IAAI,KAAK,GAAG,CAAC,CAAA;IACb,IAAI,OAAO,GAAG,CAAC,CAAA;IACf,MAAM,OAAO,GAAa,EAAE,CAAA;IAC5B,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;YAC7F,SAAQ;QACV,CAAC;QACD,MAAM,KAAK,GAAG,GAAG,OAAO,eAAe,CAAC,CAAC,KAAK,YAAY,CAAA;QAC1D,MAAM,IAAI,GAAG,GAAG,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,MAAM,CAAA;QACrC,IAAI,KAAK,GAAG,IAAI,GAAG,uBAAuB,EAAE,CAAC;YAC3C,OAAO,IAAI,CAAC,CAAA;YACZ,SAAQ;QACV,CAAC;QACD,KAAK,IAAI,IAAI,CAAA;QACb,OAAO,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IACrB,CAAC;IACD,MAAM,MAAM,GACV,OAAO,GAAG,CAAC;QACT,CAAC,CAAC,MAAM,OAAO,sIAAsI;QACrJ,CAAC,CAAC,EAAE,CAAA;IACR,OAAO,GAAG,MAAM,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,mBAAmB,OAAO,CAAC,IAAI,CAAC,EAAE,CAAC,GAAG,MAAM,EAAE,CAAA;AAClF,CAAC;AAED;;;;;;;GAOG;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,MAAM,CAAC,MAAM,uBAAuB,GAA0B;IAC5D;QACE,IAAI,EAAE,gBAAgB;QACtB,YAAY,EAAE,yBAAyB;QACvC,MAAM,EAAE,CAAC,mBAAmB,CAAC;QAC7B,sFAAsF;QACtF,2FAA2F;QAC3F,+FAA+F;QAC/F,qGAAqG;QACrG,KAAK,EAAE,EAAE,OAAO,EAAE,mBAAmB,EAAE,KAAK,EAAE,EAAE,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,EAAE;QAC9E,gBAAgB,EAAE,QAAQ;QAC1B,YAAY,EAAE;YACZ,KAAK,EAAE,aAAa;YACpB,IAAI,EAAE,0BAA0B;YAChC,KAAK,EAAE,SAAS;YAChB,WAAW,EACT,wFAAwF;gBACxF,yDAAyD;YAC3D,QAAQ,EAAE,QAAQ;YAClB,mFAAmF;YACnF,iFAAiF;YACjF,UAAU,EAAE,WAAW;SACxB;KACF;CACF,CAAA;AAED;;;GAGG;AACH,MAAM,UAAU,uBAAuB,CAAC,QAA2B;IACjE,QAAQ,CAAC,WAAW,CAAC,uBAAuB,CAAC,CAAA;AAC/C,CAAC"}

package/dist/repo-ops/run.d.ts

@@ -4,9 +4,11 @@ import type { RepoOp, RepoOpContext, RepoOpResult } from '@cat-factory/kernel';
  * op is deterministic backend work (read a baseline artifact, render + commit files);
  * a throw aborts the remaining ops and propagates so the engine fails the step.
  *
- * Returns the merged {@link RepoOpResult} the ops reported — currently just the last
- * `pullRequest` an op opened (a committing kind that delivers via PR, e.g. `spike`), so the
- * engine can record it on the block. Ops that report nothing leave it empty.
+ * Returns the merged {@link RepoOpResult} the ops reported: the last `pullRequest` an op opened
+ * (a committing kind that delivers via PR, e.g. `spike`) and the concatenation of any
+ * `contextFiles` preOps prepared for the agent (e.g. the `pr-reviewer` diff), so the engine can
+ * record the PR on the block and materialise the injected files. Ops that report nothing leave
+ * it empty.
  *
  * Lives in `@cat-factory/agents` (which owns the kind registry + the render lib the
  * post-ops use) so the orchestration engine can drive it without importing the server

package/dist/repo-ops/run.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"run.d.ts","sourceRoot":"","sources":["../../src/repo-ops/run.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,aAAa,EAAE,YAAY,EAAE,MAAM,qBAAqB,CAAA;AAE9E;;;;;;;;;;;;GAYG;AACH,wBAAsB,UAAU,CAC9B,GAAG,EAAE,SAAS,MAAM,EAAE,EACtB,GAAG,EAAE,aAAa,GACjB,OAAO,CAAC,YAAY,CAAC,CAOvB"}
+{"version":3,"file":"run.d.ts","sourceRoot":"","sources":["../../src/repo-ops/run.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,aAAa,EAAE,YAAY,EAAE,MAAM,qBAAqB,CAAA;AAE9E;;;;;;;;;;;;;;GAcG;AACH,wBAAsB,UAAU,CAC9B,GAAG,EAAE,SAAS,MAAM,EAAE,EACtB,GAAG,EAAE,aAAa,GACjB,OAAO,CAAC,YAAY,CAAC,CASvB"}

package/dist/repo-ops/run.js

@@ -3,9 +3,11 @@
  * op is deterministic backend work (read a baseline artifact, render + commit files);
  * a throw aborts the remaining ops and propagates so the engine fails the step.
  *
- * Returns the merged {@link RepoOpResult} the ops reported — currently just the last
- * `pullRequest` an op opened (a committing kind that delivers via PR, e.g. `spike`), so the
- * engine can record it on the block. Ops that report nothing leave it empty.
+ * Returns the merged {@link RepoOpResult} the ops reported: the last `pullRequest` an op opened
+ * (a committing kind that delivers via PR, e.g. `spike`) and the concatenation of any
+ * `contextFiles` preOps prepared for the agent (e.g. the `pr-reviewer` diff), so the engine can
+ * record the PR on the block and materialise the injected files. Ops that report nothing leave
+ * it empty.
  *
  * Lives in `@cat-factory/agents` (which owns the kind registry + the render lib the
  * post-ops use) so the orchestration engine can drive it without importing the server
@@ -17,6 +19,8 @@ export async function runRepoOps(ops, ctx) {
         const result = await op(ctx);
         if (result?.pullRequest)
             merged.pullRequest = result.pullRequest;
+        if (result?.contextFiles?.length)
+            merged.contextFiles = [...(merged.contextFiles ?? []), ...result.contextFiles];
     }
     return merged;
 }

package/dist/repo-ops/run.js.map

@@ -1 +1 @@
-{"version":3,"file":"run.js","sourceRoot":"","sources":["../../src/repo-ops/run.ts"],"names":[],"mappings":"AAEA;;;;;;;;;;;;GAYG;AACH,MAAM,CAAC,KAAK,UAAU,UAAU,CAC9B,GAAsB,EACtB,GAAkB;IAElB,MAAM,MAAM,GAAiB,EAAE,CAAA;IAC/B,KAAK,MAAM,EAAE,IAAI,GAAG,EAAE,CAAC;QACrB,MAAM,MAAM,GAAG,MAAM,EAAE,CAAC,GAAG,CAAC,CAAA;QAC5B,IAAI,MAAM,EAAE,WAAW;YAAE,MAAM,CAAC,WAAW,GAAG,MAAM,CAAC,WAAW,CAAA;IAClE,CAAC;IACD,OAAO,MAAM,CAAA;AACf,CAAC"}
+{"version":3,"file":"run.js","sourceRoot":"","sources":["../../src/repo-ops/run.ts"],"names":[],"mappings":"AAEA;;;;;;;;;;;;;;GAcG;AACH,MAAM,CAAC,KAAK,UAAU,UAAU,CAC9B,GAAsB,EACtB,GAAkB;IAElB,MAAM,MAAM,GAAiB,EAAE,CAAA;IAC/B,KAAK,MAAM,EAAE,IAAI,GAAG,EAAE,CAAC;QACrB,MAAM,MAAM,GAAG,MAAM,EAAE,CAAC,GAAG,CAAC,CAAA;QAC5B,IAAI,MAAM,EAAE,WAAW;YAAE,MAAM,CAAC,WAAW,GAAG,MAAM,CAAC,WAAW,CAAA;QAChE,IAAI,MAAM,EAAE,YAAY,EAAE,MAAM;YAC9B,MAAM,CAAC,YAAY,GAAG,CAAC,GAAG,CAAC,MAAM,CAAC,YAAY,IAAI,EAAE,CAAC,EAAE,GAAG,MAAM,CAAC,YAAY,CAAC,CAAA;IAClF,CAAC;IACD,OAAO,MAAM,CAAA;AACf,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cat-factory/agents",
-  "version": "0.65.5",
+  "version": "0.66.1",
   "description": "Agent catalog, routing, prompts and fragment library for the Agent Architecture Board.",
   "repository": {
     "type": "git",
@@ -32,13 +32,13 @@
     "handlebars": "^4.7.9",
     "valibot": "^1.4.2",
     "@cat-factory/contracts": "0.152.1",
-    "@cat-factory/kernel": "0.146.0",
+    "@cat-factory/kernel": "0.147.0",
     "@cat-factory/prompt-fragments": "0.13.44"
   },
   "devDependencies": {
     "typescript": "7.0.2",
     "vitest": "^4.1.10",
-    "@cat-factory/caching": "0.10.17"
+    "@cat-factory/caching": "0.10.18"
   },
   "scripts": {
     "build": "tsc -b tsconfig.build.json",