pixelkiln 0.86.1 → 0.86.3

package/PROVIDERS.md

@@ -27,7 +27,7 @@ them.
 | PixelLab | Production; paid generation and account workflows live-tested | `PIXELLAB_API_KEY` | generations |
 | Retro Diffusion | Experimental; authenticated paid still generation, download, provenance, and recovery live-tested; advanced workflows pending | `RD_API_KEY` | USD |
 | ComfyUI | Experimental; local generation, review, recovery, and image-to-image revision smoke live-tested on Apple MPS; inpaint/outpaint remain mock-tested | none; optional `COMFYUI_BASE_URL` | free |
-| Scenario | Experimental; BFL Flux 2 Dev authentication, CU preflight, paid single/two-output generation, review, download, and durable recovery live-tested | `SCENARIO_SDK_API_KEY` and `SCENARIO_SDK_API_SECRET` | compute-units |
+| Scenario | Experimental; BFL Flux 2 Dev authentication, CU preflight, paid single/two-output generation, review, download, and durable recovery live-tested; reference-image edits, tool models, and multi-model chains run once each | `SCENARIO_SDK_API_KEY` and `SCENARIO_SDK_API_SECRET` | compute-units |
 | FakeProvider | Test-only deterministic lifecycle | none | free |
 
 Live tests now cover single-candidate RD Fast and RD Plus stills from cost quote
@@ -267,9 +267,11 @@ to ComfyUI. Provider-keyed budgets, independent orchestration, lock-authoritativ
 recovery, and explicit account-provider selection ship with it.
 
 The Scenario still-image adapter from
-[GitHub issue #52](https://github.com/gfargo/pixelkiln/issues/52) now has mocked
-edge coverage and a paid BFL Flux 2 Dev lifecycle smoke. It remains
-experimental until more model schemas and representative art briefs pass.
+[GitHub issue #52](https://github.com/gfargo/pixelkiln/issues/52) has mocked edge
+coverage, a paid BFL Flux 2 Dev lifecycle smoke, and a broader evaluation (ten
+more models, six tool models, reference-image edits, multi-model chains, and a real
+game's art) recorded in [Set up Scenario](./docs/SCENARIO.md#where-the-evidence-is).
+It remains experimental until more model schemas and representative art briefs pass.
 
 | Candidate | What it adds | Fit with PixelKiln | Main cost or risk | Priority |
 |---|---|---|---|---:|

package/README.md

@@ -366,7 +366,7 @@ verification. See [Library API](./docs/LIBRARY.md).
 | [Set up PixelLab](./docs/PIXELLAB.md) | Production-provider credentials, manifest, generators, and account workflows. |
 | [Set up Retro Diffusion](./docs/RETRO_DIFFUSION.md) | Experimental-provider credentials, styles, formats, cost checks, and limits. |
 | [Set up ComfyUI](./docs/COMFYUI.md) | Self-hosted stills, revisions, ordered frame sets, per-asset inputs, and quality limits. |
-| [Set up Scenario](./docs/SCENARIO.md) | Experimental hosted models, two-part credentials, CU preflight, review, and durable downloads. |
+| [Set up Scenario](./docs/SCENARIO.md) | Experimental hosted models, two-part credentials, CU preflight, reference-image edits, multi-model chains, measured findings, and durable downloads. |
 | [Versioned recipes](./docs/RECIPES.md) | Pinned workflow packs, model hashes, manifest templates, and quality contracts. |
 | [Controlled revisions](./docs/REVISIONS.md) | Image-to-image, inpaint, palette cleanup, animation, and interpolation from a parent; masks, fail-closed readiness, provenance, and ComfyUI bindings. |
 | [CLI reference](./docs/CLI.md) | Every command, flag, JSON mode, and exit contract. |

package/dist/cli.js

@@ -17650,6 +17650,12 @@ async function currentArt(entry, spec) {
   }
   return { path: file, contentType, width, height };
 }
+async function cachedFrame(url) {
+  if (!url.startsWith("file://")) return null;
+  const file = url.slice("file://".length);
+  const contentType = mediaTypeFromExtension(file) ?? detectMediaType(await readFile23(file).catch(() => Buffer.alloc(0))) ?? "application/octet-stream";
+  return { path: file, contentType };
+}
 async function prepareReview(provider, lock, opts = {}) {
   const log2 = opts.onProgress ?? (() => {
   });
@@ -17659,6 +17665,7 @@ async function prepareReview(provider, lock, opts = {}) {
   );
   const reviewAssets = /* @__PURE__ */ new Map();
   const prefix = opts.routePrefix ?? "";
+  const providerUrls = /* @__PURE__ */ new Map();
   const groups = [];
   for (const [key, entry] of Object.entries(lock.entries)) {
     if (entry.provider !== provider.id || selectedKeys && !selectedKeys.has(key)) continue;
@@ -17667,7 +17674,19 @@ async function prepareReview(provider, lock, opts = {}) {
       const spec = specByKey.get(key);
       const state = await provider.poll(entry.reviewObjectId, entry.generator, { spec, metadata: entry.providerMetadata?.[provider.id] });
       if (state.status !== "review" && state.status !== "review-set" || (state.status === "review" ? !state.candidateUrls.length : !state.frameUrls.length)) continue;
-      const frameUrls = state.status === "review" ? state.candidateUrls : state.frameUrls;
+      const urls = state.status === "review" ? state.candidateUrls : state.frameUrls;
+      const frameUrls = [];
+      const frameAssets = [];
+      for (const [index, url] of urls.entries()) {
+        const cached = await cachedFrame(url);
+        if (!cached) {
+          frameUrls.push(url);
+          continue;
+        }
+        const route = `${prefix}/review-frame/${encodeURIComponent(String(groups.length))}/${index}`;
+        frameAssets.push([route, cached]);
+        frameUrls.push(route);
+      }
       const sourceRoute = spec?.revision ? `${prefix}/revision-source/${encodeURIComponent(String(groups.length))}` : null;
       if (sourceRoute && spec?.revision) {
         reviewAssets.set(sourceRoute, {
@@ -17680,6 +17699,8 @@ async function prepareReview(provider, lock, opts = {}) {
       if (current && currentRoute) {
         reviewAssets.set(currentRoute, { path: current.path, contentType: current.contentType });
       }
+      for (const [route, asset] of frameAssets) reviewAssets.set(route, asset);
+      providerUrls.set(key, urls);
       groups.push({
         key,
         assetId: entry.assetId,
@@ -17762,7 +17783,7 @@ async function prepareReview(provider, lock, opts = {}) {
           status: "selected",
           objectId,
           candidateIndex: index,
-          sourceUrl: sourceUrl ?? group.frameUrls[index] ?? null,
+          sourceUrl: sourceUrl ?? providerUrls.get(key)?.[index] ?? null,
           sourceUrls: [],
           provider: provider.id,
           providerMetadata: metadata ? {