@justai/cuts 0.32.0 → 0.34.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@justai/cuts",
-  "version": "0.32.0",
+  "version": "0.34.0",
   "description": "A persona's named parts \u2014 the page shell that holds them, and the cuts themselves.",
   "license": "UNLICENSED",
   "repository": {
@@ -31,7 +31,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@justai/core": "^0.5.0",
+    "@justai/core": "^0.6.0",
     "@justai/ui": "^0.3.0",
     "@justai/atrium": "^0.1.9"
   }

package/src/FilePicker.astro

@@ -93,8 +93,18 @@ const { id, title = "Choose a file", src = "/files" } = Astro.props;
   // two kernels at once would otherwise have both arrive calling themselves "files", and the second
   // would silently take the first one's place.
   import atrium from "@justai/atrium";
+  import { PRESENTATION_PARAM, PRESENTATION_EMBEDDED } from "@justai/core";
 
-  interface Picked { path: string; name: string; mime: string; bytes: ArrayBuffer }
+  interface Picked {
+    path: string;
+    name: string;
+    mime: string;
+    bytes: ArrayBuffer;
+    /** The same file as a Blob — for URL.createObjectURL, an upload body, an <img> src. */
+    blob(): Blob;
+    /** The same file as a data: URL — for anything that must EMBED it (an SVG, a saved document). */
+    dataUrl(): Promise<string>;
+  }
   interface PickRequest { accept?: string; title?: string; timeout?: number }
 
   for (const root of document.querySelectorAll<HTMLElement>("[data-picker]")) {
@@ -138,7 +148,20 @@ const { id, title = "Choose a file", src = "/files" } = Astro.props;
 
     async function pick(req: PickRequest = {}): Promise<Picked | null> {
       const bus = await router();
-      if (!frame.getAttribute("src")) frame.setAttribute("src", root.dataset.src!);
+      if (!frame.getAttribute("src")) {
+        // DECLARE THE PRESENTATION. A page opened by a person and a page opened by another persona
+        // are two different situations, and the same controls do not belong in both — standalone
+        // there is nothing to cancel. The frame is the slower truth (a handshake takes a beat), so
+        // the declaration rides in the URL and is available in the first paint. It is presentation
+        // only: nothing that must not be forged is decided by it.
+        const src = root.dataset.src!;
+        frame.setAttribute("src", src + (src.includes("?") ? "&" : "?") + PRESENTATION_PARAM + "=" + PRESENTATION_EMBEDDED);
+      }
+      // ONE HEADING, and it is the sheet's. A per-call title retitles the sheet rather than being
+      // forwarded into the frame: the kernel renders a heading of its own for callers that have no
+      // sheet around them, and sending it one here printed "Upload logo" twice, one under the other.
+      const heading = sheet.querySelector<HTMLElement>(".sheet-title");
+      if (heading && req.title) heading.textContent = req.title;
       sheet.showModal();
 
       // TWO WAYS THIS ENDS, and only one of them is the kernel's.
@@ -155,14 +178,36 @@ const { id, title = "Choose a file", src = "/files" } = Astro.props;
         .proxy({
           service: "files",
           method: "pick",
-          args: { accept: req.accept, title: req.title },
+          args: { accept: req.accept },
           options: { timeout: req.timeout ?? 300000 },
         }))
         .catch(() => null);
 
       const result = await Promise.race([answered, dismissed]);
       if (sheet.open) sheet.close();
-      return (result as Picked) ?? null;
+      if (!result) return null;
+
+      // THE CUT BRINGS ITS OWN GROUND, and bytes are not ground. Every consumer of this needs the
+      // file in some other shape, and the obvious hand-written conversion is a trap:
+      //
+      //     btoa(String.fromCharCode(...new Uint8Array(bytes)))
+      //
+      // makes one argument per BYTE, so a 7.5 MB photo is a 7.5-million-argument call and the stack
+      // gives out — "Maximum call stack size exceeded", thrown far from anything that mentions size,
+      // and invisible to any test whose fixture image is small. qr wrote exactly that line and a
+      // person found it with a real photograph. It belongs here once, not in every persona.
+      const r = result as any;
+      const blob = () => new Blob([r.bytes], { type: r.mime });
+      return {
+        ...r,
+        blob,
+        dataUrl: () => new Promise<string>((resolve, reject) => {
+          const rd = new FileReader();
+          rd.onload = () => resolve(rd.result as string);
+          rd.onerror = () => reject(rd.error);
+          rd.readAsDataURL(blob());   // no ceiling, unlike the spread above
+        }),
+      } as Picked;
     }
 
     (host as any).pick = pick;