@pygmalionjs/pygmalion 0.5.29 → 0.5.31

package/dist-lib/pygmalion.js

@@ -11985,34 +11985,36 @@ function iv(e, t) {
   const o = e.slice(r + n.length);
   return o.startsWith(" · ") ? o.slice(3) : t;
 }
-function sv(e, t, n = () => !0) {
-  const r = /* @__PURE__ */ new Map(), o = /* @__PURE__ */ new Map(), i = /* @__PURE__ */ new Map();
-  for (const d of e) {
-    const u = d.canvas ?? "default";
-    i.set(u, (i.get(u) ?? 0) + 1), n(d) && o.set(u, (o.get(u) ?? 0) + 1), d.section && !r.has(u) && r.set(u, d.section);
+function sv(e, t, n = () => !0, r = () => !0) {
+  const o = /* @__PURE__ */ new Map(), i = /* @__PURE__ */ new Map(), a = /* @__PURE__ */ new Map(), c = /* @__PURE__ */ new Set();
+  for (const p of e) {
+    const m = p.canvas ?? "default";
+    c.add(m), p.section && !o.has(m) && o.set(m, p.section), r(p) && (a.set(m, (a.get(m) ?? 0) + 1), n(p) && i.set(m, (i.get(m) ?? 0) + 1));
   }
-  const a = [], c = [], l = /* @__PURE__ */ new Map();
-  for (const d of t) {
-    const u = o.get(d) ?? 0, p = i.get(d) ?? 0, m = r.get(d);
-    if (!m || !d.includes(` · ${m}`)) {
-      a.push({ canvas: d, label: d, frames: u, declared: p });
+  const l = [], d = [], u = /* @__PURE__ */ new Map();
+  for (const p of t) {
+    const m = i.get(p) ?? 0, b = a.get(p) ?? 0;
+    if (b === 0 && c.has(p)) continue;
+    const x = o.get(p);
+    if (!x || !p.includes(` · ${x}`)) {
+      l.push({ canvas: p, label: p, frames: m, declared: b });
       continue;
     }
-    const b = { canvas: d, label: iv(d, m), frames: u, declared: p }, x = l.get(m);
-    if (x === void 0) {
-      l.set(m, c.length), c.push({ section: m, canvases: [b], frames: u, declared: p, single: !0 });
+    const w = { canvas: p, label: iv(p, x), frames: m, declared: b }, v = u.get(x);
+    if (v === void 0) {
+      u.set(x, d.length), d.push({ section: x, canvases: [w], frames: m, declared: b, single: !0 });
       continue;
     }
-    const w = c[x];
-    c[x] = {
-      section: m,
-      canvases: [...w.canvases, b],
-      frames: w.frames + u,
-      declared: w.declared + p,
+    const E = d[v];
+    d[v] = {
+      section: x,
+      canvases: [...E.canvases, w],
+      frames: E.frames + m,
+      declared: E.declared + b,
       single: !1
     };
   }
-  return { loose: a, groups: c };
+  return { loose: l, groups: d };
 }
 let wa = null;
 const Sa = /* @__PURE__ */ new Set();
@@ -13078,9 +13080,10 @@ const ys = Be(function({
     ii
   );
   const [o, i] = Ev("frames", 176), [a, c] = Q({}), l = sv(
-    f.pages.filter((I) => f.isInSelectedRuntime(I)),
+    f.pages,
     f.canvasNames,
-    (I) => f.isScreenStateShown(I)
+    (I) => f.isScreenStateShown(I),
+    (I) => f.isInSelectedRuntime(I)
   ), d = (I) => I.canvases.some((N) => N.canvas === f.activeCanvas), u = Xe(
     Mf,
     ho,

package/dist-lib/types/editor/canvasTree.d.ts

@@ -54,5 +54,5 @@ export type CanvasTreeVisibility = (page: never) => boolean;
  * canvas list already resolved — that order carries the host's declared
  * section ranking, which a regrouping here would quietly discard.
  */
-export declare function buildCanvasTree<TPage extends CanvasTreePage>(pages: readonly TPage[], canvasNames: readonly string[], isShown?: (page: TPage) => boolean): CanvasTree;
+export declare function buildCanvasTree<TPage extends CanvasTreePage>(pages: readonly TPage[], canvasNames: readonly string[], isShown?: (page: TPage) => boolean, exists?: (page: TPage) => boolean): CanvasTree;
 export {};

package/node/preview-artifact-plugin.mjs

@@ -1,5 +1,9 @@
 import path from 'node:path';
-import { validateRoutePreviewArtifactBundle } from './route-preview-artifact-v3.mjs';
+import {
+  mergeRoutePreviewArtifactV3,
+  selectRoutePreviewArtifactFrames,
+  validateRoutePreviewArtifactBundle,
+} from './route-preview-artifact-v3.mjs';
 import { createPreviewArtifactStore } from './preview-artifact-store.mjs';
 
 export const PYGMALION_PREVIEW_ARTIFACT_ENDPOINT =
@@ -247,6 +251,7 @@ export function pygmalionPreviewArtifactPlugin({
   acquireLease,
   requestRuntime,
   runtimePrepared = () => true,
+  artifactStoreMaxBytes,
 } = {}) {
   const resolvedRoot = path.resolve(root);
   const resolvedArtifact = path.resolve(resolvedRoot, artifactFile);
@@ -288,6 +293,9 @@ export function pygmalionPreviewArtifactPlugin({
     // With the source root, a published frame records the files it actually
     // rendered, so freshness stops depending on which revision captured it.
     sourceRoot: resolvedRoot,
+    ...(artifactStoreMaxBytes == null
+      ? {}
+      : { maxBytes: artifactStoreMaxBytes }),
   });
   const identityQueues = new Map();
   const inFlightRequests = new Map();
@@ -485,7 +493,7 @@ export function pygmalionPreviewArtifactPlugin({
         );
         const absent = new Set([...before.missing, ...before.stale]);
         const remaining = wanted.filter((frame) => absent.has(frame.id));
-        if (remaining.length === 0) return before.bundle;
+        if (remaining.length === 0) return before;
 
         const generated = await withCaptureLease(sourceRevision, () =>
           generateArtifact({
@@ -508,13 +516,17 @@ export function pygmalionPreviewArtifactPlugin({
           sourceRevision,
           recordRevision: false,
         });
-        return (
-          await artifactStore.readFrameSelection(
-            namespace,
-            sourceRevision,
-            wanted,
-          )
-        ).bundle;
+        // Retention is allowed to decline a newly generated object when older
+        // entries have earned higher read recency. That cache decision must not
+        // discard the result from the request that just paid to generate it.
+        const responseArtifact = mergeRoutePreviewArtifactV3(
+          before.bundle,
+          generated,
+        );
+        return selectRoutePreviewArtifactFrames(responseArtifact, wanted, {
+          includeStale: true,
+          sourceRevision,
+        });
       }, priority),
     );
   }
@@ -644,17 +656,17 @@ export function pygmalionPreviewArtifactPlugin({
             }
             const requested = new Set(absent);
             try {
-              await generateFrames(
+              const captured = await generateFrames(
                 namespace,
                 sourceRevision,
                 captureBaseUrl,
                 wanted.filter((frame) => requested.has(frame.id)),
                 priority,
               );
-              picked = await artifactStore.readFrameSelection(
-                namespace,
-                sourceRevision,
+              picked = selectRoutePreviewArtifactFrames(
+                mergeRoutePreviewArtifactV3(picked.bundle, captured.bundle),
                 wanted,
+                { includeStale: true, sourceRevision },
               );
             } catch (error) {
               sendGenerationFailure(response, error);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pygmalionjs/pygmalion",
-  "version": "0.5.29",
+  "version": "0.5.31",
   "description": "Code-backed DOM design sandbox and visual QA editor",
   "license": "UNLICENSED",
   "publishConfig": {