pixelkiln 0.89.2 → 0.89.4

package/CONTRIBUTING.md

@@ -42,6 +42,12 @@ npm run website:dev
 The site reads canonical Markdown from `docs/` and the root policy files at
 build time. Do not create a second documentation copy under `website/`.
 
+The home page's "Latest updates from our providers" section reads
+`website/app/lib/provider-updates.ts`. When a provider ships a feature
+PixelKiln can use, add an entry at the top of that list. The newest entry
+shows its figures and code sample; earlier ones collapse to a short row. Show
+only output from a real call, with the bill the docs record.
+
 Website sprites are generated from `website/art/pixelkiln.manifest.json`.
 Treat those requests as paid work: run `plan` and `audit` first, use an explicit
 hard budget, and commit the manifest, lockfile, and reviewed outputs together.

package/dist/cli.js

@@ -6704,7 +6704,8 @@ var PixelLabClient = class {
     });
     const text = await res.text();
     if (!res.ok) {
-      throw new PixelLabError(`${init?.method ?? "GET"} ${path53} \u2192 ${res.status}`, res.status, text);
+      const hint = res.status === 500 && text.includes("Failed to start any animation jobs") ? '; a skeleton-v3 loop needs a character with a stored 3D skeleton (a v3 base), use mode: "template" on a standard one' : "";
+      throw new PixelLabError(`${init?.method ?? "GET"} ${path53} \u2192 ${res.status}${hint}`, res.status, text);
     }
     if (!text) return {};
     try {
@@ -9598,6 +9599,11 @@ var PixelLabProvider = class _PixelLabProvider {
       if (character.animation.mode !== "template" && character.animation.mode !== "skeleton-v3" && !spec.prompt.trim()) {
         throw new Error(`${spec.styleId}/${spec.assetId} needs a prompt describing the motion, or a template`);
       }
+      if (character.animation.mode === "skeleton-v3" && character.mode === "standard") {
+        throw new Error(
+          `${spec.styleId}/${spec.assetId}: skeleton-v3 poses a template onto a 3D skeleton, which a ${character.mode} character has none of; use a v3 base (style mode: "v3"), or mode: "template" for this loop`
+        );
+      }
     }
   }
   validateObjectPro(spec, styleImages = []) {
@@ -9904,7 +9910,10 @@ var PixelLabProvider = class _PixelLabProvider {
           stateName: character.state_name ?? null,
           directions: character.directions,
           size: character.size,
-          updatedAt: character.updated_at ?? null
+          updatedAt: character.updated_at ?? null,
+          // Kept: this metadata replaces the submit-time `character` entry,
+          // and the job id is the only way to read the bill again later.
+          ...backgroundJobId ? { backgroundJobId } : {}
         }
       },
       billed: await this.billedForJob(backgroundJobId)
@@ -10555,7 +10564,12 @@ var PixelLabProvider = class _PixelLabProvider {
     if (!backgroundJobId) return null;
     try {
       const job = await this.client.getBackgroundJob(backgroundJobId);
-      return billedFromUsage(job.usage);
+      const nested = job.last_response?.billing_usage;
+      for (const usage of [job.usage, nested]) {
+        const billed = billedFromUsage(usage);
+        if (billed && !(billed.unit === "usd" && billed.amount === 0)) return billed;
+      }
+      return null;
     } catch {
       return null;
     }
@@ -19443,8 +19457,18 @@ GROUPS.forEach((g, gi) => {
     preview.loading = 'lazy';
     const index = document.createElement('span');
     index.className = 'idx';
-    index.textContent = (g.frameLabels?.[i] || String(i + 1)) + ' \xB7 ' +
-      g.width + '\xD7' + g.height + ' \xB7 ' + scaleLabel;
+    // A frame's own size, not the manifest's: a 96px loop of a 64px
+    // character is 96\xD796. Known up front for a cached file, else once loaded.
+    const label = (w, h) => {
+      index.textContent = (g.frameLabels?.[i] || String(i + 1)) + ' \xB7 ' + w + '\xD7' + h + ' \xB7 ' + scaleLabel;
+    };
+    const known = g.frameSizes?.[i];
+    label(known ? known.width : g.width, known ? known.height : g.height);
+    if (!known) {
+      preview.addEventListener('load', () => {
+        if (preview.naturalWidth && preview.naturalHeight) label(preview.naturalWidth, preview.naturalHeight);
+      }, { once: true });
+    }
     c.append(preview, index);
     if (g.recommendedIndex != null && i === recommended) {
       const badge = document.createElement('span');
@@ -19625,12 +19649,15 @@ async function prepareReview(provider, lock, opts = {}) {
       const urls = state.status === "review" ? state.candidateUrls : state.frameUrls;
       const frameUrls = [];
       const frameAssets = [];
+      const frameSizes = [];
       for (const [index, url] of urls.entries()) {
         const cached = await cachedFrame(url);
         if (!cached) {
           frameUrls.push(url);
+          frameSizes.push(null);
           continue;
         }
+        frameSizes.push(cached.contentType === "image/png" ? pngSize(await readFile25(cached.path).catch(() => Buffer.alloc(0))) : null);
         const route = `${prefix}/review-frame/${encodeURIComponent(String(groups.length))}/${index}`;
         frameAssets.push([route, cached]);
         frameUrls.push(route);
@@ -19656,6 +19683,7 @@ async function prepareReview(provider, lock, opts = {}) {
         prompt: entry.prompt,
         reviewObjectId: entry.reviewObjectId,
         frameUrls,
+        ...frameSizes.some(Boolean) ? { frameSizes } : {},
         width: entry.width,
         height: entry.height,
         ...state.status === "review-set" ? {