@cat-factory/orchestration 0.323.2 → 0.324.0

package/dist/modules/review/reviewTaskImages.d.ts

@@ -0,0 +1,37 @@
+import type { Block, Logger, ModelRef, ResolveBinaryArtifactStore } from '@cat-factory/kernel';
+/** What an inline review call's prompt needs from the task's attached images. */
+export interface ReviewTaskImageDeps {
+    resolveBinaryArtifactStore?: ResolveBinaryArtifactStore;
+    runsInline?: (ref: ModelRef) => boolean;
+    logger?: Logger;
+}
+/**
+ * The request body of one inline review call: the prompt plus the images attached to the task
+ * under review, as a spread-ready `{ prompt }` or `{ messages }`.
+ *
+ * The reviewers judge a task's description, and an image its author attached is part of that
+ * description, so the call carries it as image input wherever the model takes one and STATES it
+ * otherwise. A task with no attached image (most of them) gets the plain prompt, byte-identical.
+ *
+ * No store failure fails a review. A LISTING that failed cannot say whether the task had images,
+ * so it adds nothing to the prompt; a BYTE read that failed after the listing named the images
+ * settles as `transfer_failed`, so the prompt states that they were withheld.
+ */
+export declare function reviewRequestWithTaskImages(deps: ReviewTaskImageDeps, workspaceId: string, block: Block, ref: ModelRef, prompt: string): Promise<{
+    messages: {
+        role: 'user';
+        content: ({
+            type: 'image';
+            image: Uint8Array<ArrayBufferLike>;
+            mediaType: string;
+        } | {
+            type: 'text';
+            text: string;
+        })[];
+    }[];
+    prompt?: undefined;
+} | {
+    messages?: undefined;
+    prompt: string;
+}>;
+//# sourceMappingURL=reviewTaskImages.d.ts.map

package/dist/modules/review/reviewTaskImages.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"reviewTaskImages.d.ts","sourceRoot":"","sources":["../../../src/modules/review/reviewTaskImages.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,KAAK,EAAE,MAAM,EAAE,QAAQ,EAAE,0BAA0B,EAAE,MAAM,qBAAqB,CAAA;AAU9F,iFAAiF;AACjF,MAAM,WAAW,mBAAmB;IAClC,0BAA0B,CAAC,EAAE,0BAA0B,CAAA;IACvD,UAAU,CAAC,EAAE,CAAC,GAAG,EAAE,QAAQ,KAAK,OAAO,CAAA;IACvC,MAAM,CAAC,EAAE,MAAM,CAAA;CAChB;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,2BAA2B,CAC/C,IAAI,EAAE,mBAAmB,EACzB,WAAW,EAAE,MAAM,EACnB,KAAK,EAAE,KAAK,EACZ,GAAG,EAAE,QAAQ,EACb,MAAM,EAAE,MAAM;;cA+BE,MAAM;;;;;;kBACM,MAAM;;;;;;;;GAKnC"}

package/dist/modules/review/reviewTaskImages.js

@@ -0,0 +1,48 @@
+import { noopLogger, runBestEffort } from '@cat-factory/kernel';
+import { imageParts, inlineImageStore, prepareInlineImages, renderTaskImages, } from '@cat-factory/agents';
+import { taskImageSet } from '../execution/run-task-images.js';
+/**
+ * The request body of one inline review call: the prompt plus the images attached to the task
+ * under review, as a spread-ready `{ prompt }` or `{ messages }`.
+ *
+ * The reviewers judge a task's description, and an image its author attached is part of that
+ * description, so the call carries it as image input wherever the model takes one and STATES it
+ * otherwise. A task with no attached image (most of them) gets the plain prompt, byte-identical.
+ *
+ * No store failure fails a review. A LISTING that failed cannot say whether the task had images,
+ * so it adds nothing to the prompt; a BYTE read that failed after the listing named the images
+ * settles as `transfer_failed`, so the prompt states that they were withheld.
+ */
+export async function reviewRequestWithTaskImages(deps, workspaceId, block, ref, prompt) {
+    const logger = deps.logger ?? noopLogger;
+    // Resolved once for the listing and the byte reads both.
+    const store = inlineImageStore(deps.resolveBinaryArtifactStore, workspaceId);
+    const set = store
+        ? await runBestEffort(logger, 'review: read task images', async () => {
+            const resolved = await store();
+            return resolved
+                ? taskImageSet(await resolved.listByBlock(workspaceId, block.id))
+                : undefined;
+        }, { workspaceId, blockId: block.id })
+        : undefined;
+    const prepared = await prepareInlineImages({
+        set,
+        ref,
+        textOnly: deps.runsInline?.(ref) ?? false,
+        store,
+        workspaceId,
+        logger,
+    });
+    const text = prompt + renderTaskImages(prepared.outcome?.set, prepared.outcome?.delivery);
+    return prepared.images.length
+        ? {
+            messages: [
+                {
+                    role: 'user',
+                    content: [{ type: 'text', text }, ...imageParts(prepared.images)],
+                },
+            ],
+        }
+        : { prompt: text };
+}
+//# sourceMappingURL=reviewTaskImages.js.map

package/dist/modules/review/reviewTaskImages.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"reviewTaskImages.js","sourceRoot":"","sources":["../../../src/modules/review/reviewTaskImages.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,UAAU,EAAE,aAAa,EAAE,MAAM,qBAAqB,CAAA;AAC/D,OAAO,EACL,UAAU,EACV,gBAAgB,EAChB,mBAAmB,EACnB,gBAAgB,GACjB,MAAM,qBAAqB,CAAA;AAC5B,OAAO,EAAE,YAAY,EAAE,MAAM,iCAAiC,CAAA;AAS9D;;;;;;;;;;;GAWG;AACH,MAAM,CAAC,KAAK,UAAU,2BAA2B,CAC/C,IAAyB,EACzB,WAAmB,EACnB,KAAY,EACZ,GAAa,EACb,MAAc;IAEd,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,IAAI,UAAU,CAAA;IACxC,yDAAyD;IACzD,MAAM,KAAK,GAAG,gBAAgB,CAAC,IAAI,CAAC,0BAA0B,EAAE,WAAW,CAAC,CAAA;IAC5E,MAAM,GAAG,GAAG,KAAK;QACf,CAAC,CAAC,MAAM,aAAa,CACjB,MAAM,EACN,0BAA0B,EAC1B,KAAK,IAAI,EAAE;YACT,MAAM,QAAQ,GAAG,MAAM,KAAK,EAAE,CAAA;YAC9B,OAAO,QAAQ;gBACb,CAAC,CAAC,YAAY,CAAC,MAAM,QAAQ,CAAC,WAAW,CAAC,WAAW,EAAE,KAAK,CAAC,EAAE,CAAC,CAAC;gBACjE,CAAC,CAAC,SAAS,CAAA;QACf,CAAC,EACD,EAAE,WAAW,EAAE,OAAO,EAAE,KAAK,CAAC,EAAE,EAAE,CACnC;QACH,CAAC,CAAC,SAAS,CAAA;IACb,MAAM,QAAQ,GAAG,MAAM,mBAAmB,CAAC;QACzC,GAAG;QACH,GAAG;QACH,QAAQ,EAAE,IAAI,CAAC,UAAU,EAAE,CAAC,GAAG,CAAC,IAAI,KAAK;QACzC,KAAK;QACL,WAAW;QACX,MAAM;KACP,CAAC,CAAA;IACF,MAAM,IAAI,GAAG,MAAM,GAAG,gBAAgB,CAAC,QAAQ,CAAC,OAAO,EAAE,GAAG,EAAE,QAAQ,CAAC,OAAO,EAAE,QAAQ,CAAC,CAAA;IACzF,OAAO,QAAQ,CAAC,MAAM,CAAC,MAAM;QAC3B,CAAC,CAAC;YACE,QAAQ,EAAE;gBACR;oBACE,IAAI,EAAE,MAAe;oBACrB,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAe,EAAE,IAAI,EAAE,EAAE,GAAG,UAAU,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;iBAC3E;aACF;SACF;QACH,CAAC,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAA;AACtB,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cat-factory/orchestration",
-  "version": "0.323.2",
+  "version": "0.324.0",
   "description": "Delivery-workflow engine for the Agent Architecture Board (execution, bootstrap, pipelines, board, boardScan, requirements, and composition root).",
   "repository": {
     "type": "git",
@@ -24,19 +24,19 @@
     "access": "public"
   },
   "dependencies": {
-    "@cat-factory/agents": "0.172.10",
-    "@cat-factory/caching": "0.20.113",
-    "@cat-factory/contracts": "0.376.0",
-    "@cat-factory/integrations": "0.178.2",
-    "@cat-factory/kernel": "0.363.2",
-    "@cat-factory/prompt-fragments": "1.1.75",
-    "@cat-factory/sandbox": "0.12.89",
-    "@cat-factory/spend": "0.23.24",
-    "@cat-factory/workspaces": "0.31.2",
+    "@cat-factory/agents": "0.173.0",
+    "@cat-factory/caching": "0.20.114",
+    "@cat-factory/contracts": "0.377.0",
+    "@cat-factory/integrations": "0.178.3",
+    "@cat-factory/kernel": "0.364.0",
+    "@cat-factory/prompt-fragments": "1.1.76",
+    "@cat-factory/sandbox": "0.12.90",
+    "@cat-factory/spend": "0.23.25",
+    "@cat-factory/workspaces": "0.31.3",
     "ai": "^7.0.142"
   },
   "devDependencies": {
-    "@cat-factory/sandbox-fixtures": "0.8.62",
+    "@cat-factory/sandbox-fixtures": "0.8.63",
     "typescript": "7.0.2",
     "vitest": "^5.0.5"
   },