@cueloop/client 0.1.0-alpha.67 → 0.1.0-alpha.68

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cueloop/client",
-  "version": "0.1.0-alpha.67",
+  "version": "0.1.0-alpha.68",
   "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.67",
-    "@cueloop/extension-api": "0.1.0-alpha.67",
-    "@cueloop/integration-obsidian": "0.1.0-alpha.67",
-    "@cueloop/schema": "0.1.0-alpha.67",
+    "@cueloop/daemon": "0.1.0-alpha.68",
+    "@cueloop/extension-api": "0.1.0-alpha.68",
+    "@cueloop/integration-obsidian": "0.1.0-alpha.68",
+    "@cueloop/schema": "0.1.0-alpha.68",
     "@opentui/core": "^0.5.8",
     "@opentui/keymap": "0.5.8",
     "@opentui/react": "^0.5.8",

package/src/ghostty-terminal.test.ts

@@ -21,10 +21,11 @@ describe("ghostty-terminal FFI", () => {
     term.write(new TextEncoder().encode("hi\x1b[1;32mGO\x1b[0m"));
     const cells = [0, 1, 2, 3].map((x) => term.readCell(x, 0));
 
-    // Assert - glyphs decode, and the green run is bold on ANSI palette index 2
+    // Assert - glyphs decode, the row reads as text, and the green run is bold on ANSI palette index 2
     expect(
       cells.map((c) => (c?.codepoint ? String.fromCodePoint(c.codepoint) : " ")).join(""),
     ).toBe("hiGO");
+    expect(term.rowText(0, 80)).toBe("hiGO");
     expect(cells[2]!.bold).toBe(true);
     expect(cells[2]!.fg).toEqual({ kind: "palette", index: 2 });
     term.free();

package/src/ghostty-terminal.ts

@@ -60,25 +60,31 @@ function nativeLibraryPath(): string | null {
   return existsSync(path) ? path : null;
 }
 
+/** One shared load of the VT library; null when no dylib ships or it failed to load. */
+let ghosttyFactory: GhosttyTerminalFactory | null | undefined;
+
 /**
  * Open the Ghostty VT library for this platform, or return null when the prebuilt
  * dylib is missing (the caller degrades to a herdr split instead of embedding).
+ * The library is opened once per process; later calls return the same factory.
  */
 export function loadGhosttyTerminals(): GhosttyTerminalFactory | null {
+  if (ghosttyFactory !== undefined) return ghosttyFactory;
   const path = nativeLibraryPath();
 
-  if (!path) return null;
+  if (!path) return (ghosttyFactory = null);
   try {
     const lib = dlopen(path, CVT_SYMBOLS);
 
-    return new GhosttyTerminalFactory(lib.symbols);
+    ghosttyFactory = new GhosttyTerminalFactory(lib.symbols);
   } catch (error) {
     // The file exists but failed to load (bad arch, missing symbol) - a real
     // fault, not the expected no-prebuilt case; surface it, then degrade.
     console.error(`cueloop: failed to load ${path}:`, error);
-
-    return null;
+    ghosttyFactory = null;
   }
+
+  return ghosttyFactory;
 }
 
 /** Creates `GhosttyTerminal`s that share one loaded copy of the VT library. */
@@ -146,6 +152,23 @@ export class GhosttyTerminal {
     };
   }
 
+  /**
+   * Row `y` as text: blank cells read as spaces, the trailing half of a wide
+   * glyph is skipped, and trailing spaces are trimmed.
+   */
+  rowText(y: number, cols: number): string {
+    let line = "";
+
+    for (let x = 0; x < cols; x++) {
+      const cell = this.readCell(x, y);
+
+      if (cell === null || cell.width === 2) continue;
+      line += cell.codepoint === 0 ? " " : String.fromCodePoint(cell.codepoint);
+    }
+
+    return line.trimEnd();
+  }
+
   /** The cursor's current viewport position and visibility. */
   readCursor(): GhosttyCursor {
     if (this.freed) return { x: 0, y: 0, visible: false };

package/src/test-support.test.ts

@@ -14,6 +14,7 @@ import {
   frameRow,
   isolateUserConfig,
   locateText,
+  locateTextInFrame,
   pressKey,
   waitForText,
   type FrameLocation,
@@ -250,3 +251,14 @@ describe("dragText", () => {
     expect(log.mouseEvents).toContainEqual({ type: "up", x: 9, y: 3 });
   });
 });
+
+describe("locateTextInFrame", () => {
+  test("finds the 0-based row and column of the first match, null when absent", () => {
+    // Arrange
+    const frame = "header\n  ● draft here\nfooter";
+
+    // Act + Assert
+    expect(locateTextInFrame(frame, "draft")).toEqual({ row: 1, column: 4 });
+    expect(locateTextInFrame(frame, "missing")).toBeNull();
+  });
+});

package/src/test-support.ts

@@ -12,6 +12,16 @@ import type { TestRendererSetup } from "@opentui/core/testing";
 /** Pass budget for waits that include daemon or subprocess round-trips. */
 export const WAIT_PASSES = { maxPasses: 400 };
 
+/**
+ * The CUELOOP_CONFIG path that isolates a test from the developer's real user
+ * config: a file inside the test home that does not exist unless the test
+ * writes it. Subprocess harnesses set this in the child env; in-process suites
+ * use `isolateUserConfig`.
+ */
+export function isolatedUserConfigPath(home: string, fileName = "no-config.toml"): string {
+  return join(home, fileName);
+}
+
 /**
  * Isolate the user config: point CUELOOP_CONFIG into the test home so
  * loadConfig never reads the developer's real ~/.config/cueloop (a persisted
@@ -20,10 +30,10 @@ export const WAIT_PASSES = { maxPasses: 400 };
  * defaults; pass a file name for suites that write and assert a config of
  * their own. Call in beforeEach; invoke the returned restore in afterEach.
  */
-export function isolateUserConfig(home: string, fileName = "no-config.toml"): () => void {
+export function isolateUserConfig(home: string, fileName?: string): () => void {
   const priorUserConfig = process.env.CUELOOP_CONFIG;
 
-  process.env.CUELOOP_CONFIG = join(home, fileName);
+  process.env.CUELOOP_CONFIG = isolatedUserConfigPath(home, fileName);
 
   return () => {
     if (priorUserConfig === undefined) delete process.env.CUELOOP_CONFIG;
@@ -114,17 +124,27 @@ export interface FrameLocation {
   column: number;
 }
 
-/** Locate the first visual line containing the needle: 0-based row and start column. */
-export function locateText(setup: TestRendererSetup, needle: string): FrameLocation {
-  const frame = setup.captureCharFrame();
-
+/** Locate the first line of a text frame containing the needle; null when absent. */
+export function locateTextInFrame(frame: string, needle: string): FrameLocation | null {
   for (const [row, line] of frame.split("\n").entries()) {
     const column = line.indexOf(needle);
 
     if (column !== -1) return { row, column };
   }
 
-  throw new Error(`locateText ${JSON.stringify(needle)} not found.\nframe:\n${frame}`);
+  return null;
+}
+
+/** Locate the first visual line containing the needle: 0-based row and start column. */
+export function locateText(setup: TestRendererSetup, needle: string): FrameLocation {
+  const frame = setup.captureCharFrame();
+  const location = locateTextInFrame(frame, needle);
+
+  if (location === null) {
+    throw new Error(`locateText ${JSON.stringify(needle)} not found.\nframe:\n${frame}`);
+  }
+
+  return location;
 }
 
 /** The 0-based visual row of the needle - for relative vertical-layout assertions. */