@justai/cuts 0.32.0 → 0.33.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@justai/cuts",
-  "version": "0.32.0",
+  "version": "0.33.0",
   "description": "A persona's named parts \u2014 the page shell that holds them, and the cuts themselves.",
   "license": "UNLICENSED",
   "repository": {

package/src/FilePicker.astro

@@ -94,7 +94,16 @@ const { id, title = "Choose a file", src = "/files" } = Astro.props;
   // would silently take the first one's place.
   import atrium from "@justai/atrium";
 
-  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]")) {
@@ -139,6 +148,11 @@ 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!);
+      // 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 +169,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;