@cueloop/client 0.1.0-alpha.48 → 0.1.0-alpha.49

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cueloop/client",
-  "version": "0.1.0-alpha.48",
+  "version": "0.1.0-alpha.49",
   "description": "The cueloop review UI: projection renderer, keyboard grammar, review rail",
   "homepage": "https://github.com/mmurakaru/cueloop#readme",
   "bugs": {
@@ -26,10 +26,10 @@
     "access": "public"
   },
   "dependencies": {
-    "@cueloop/daemon": "0.1.0-alpha.48",
-    "@cueloop/extension-api": "0.1.0-alpha.48",
-    "@cueloop/integration-obsidian": "0.1.0-alpha.48",
-    "@cueloop/schema": "0.1.0-alpha.48",
+    "@cueloop/daemon": "0.1.0-alpha.49",
+    "@cueloop/extension-api": "0.1.0-alpha.49",
+    "@cueloop/integration-obsidian": "0.1.0-alpha.49",
+    "@cueloop/schema": "0.1.0-alpha.49",
     "@opentui/core": "^0.5.2",
     "@opentui/keymap": "0.5.2",
     "@opentui/react": "^0.5.2",

package/src/kitty-image.test.ts

@@ -0,0 +1,20 @@
+import { describe, expect, test } from "bun:test";
+import { placeKittyImage, transmitKittyImage } from "./kitty-image";
+
+const REGION = { column: 2, row: 3, columns: 40, rows: 12 };
+
+describe("kitty image stacking", () => {
+  test("places the prototype below cells with painted backgrounds", () => {
+    // Arrange
+    const writes: string[] = [];
+
+    // Act
+    transmitKittyImage((chunk) => writes.push(chunk), new Uint8Array([1]), REGION, 811);
+    placeKittyImage((chunk) => writes.push(chunk), REGION, 811);
+
+    // Assert
+    expect(writes).toHaveLength(2);
+    expect(writes[0]).toContain("z=-1073741825");
+    expect(writes[1]).toContain("z=-1073741825");
+  });
+});

package/src/kitty-image.ts

@@ -19,6 +19,9 @@ export type OrderedWrite = (chunk: string) => void;
 
 const ESCAPE_STRING_TERMINATOR = "\x1b\\";
 const CHUNK_BYTES = 4096;
+// The protocol reserves values below INT32_MIN/2 for images that must sit
+// beneath cells with painted backgrounds, such as the marker popover.
+const IMAGE_Z_INDEX = -1_073_741_825;
 
 /** Save the cursor, run the placement at the region's corner, restore it. */
 function atRegion(region: CellRegion, body: string): string {
@@ -43,9 +46,7 @@ export function transmitKittyImage(
 ): void {
   if (region.columns < 1 || region.rows < 1) return;
   const base64 = Buffer.from(png).toString("base64");
-  // z<0 places the image beneath the text layer, so the marker popover and
-  // compose card (ordinary cells) still paint over it
-  const controlKeys = `a=T,f=100,i=${imageId},q=2,C=1,z=-1,c=${region.columns},r=${region.rows}`;
+  const controlKeys = `a=T,f=100,i=${imageId},q=2,C=1,z=${IMAGE_Z_INDEX},c=${region.columns},r=${region.rows}`;
   let sequence = "";
   for (let offset = 0; offset < base64.length; offset += CHUNK_BYTES) {
     const chunk = base64.slice(offset, offset + CHUNK_BYTES);
@@ -62,7 +63,7 @@ export function transmitKittyImage(
 export function placeKittyImage(write: OrderedWrite, region: CellRegion, imageId: number): void {
   if (region.columns < 1 || region.rows < 1) return;
   const command = graphicsCommand(
-    `a=p,i=${imageId},q=2,C=1,z=-1,c=${region.columns},r=${region.rows}`,
+    `a=p,i=${imageId},q=2,C=1,z=${IMAGE_Z_INDEX},c=${region.columns},r=${region.rows}`,
     "",
   );
   write(atRegion(region, command));