@cueloop/client 0.1.0-alpha.110 → 0.1.0-alpha.112

package/package.json

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

package/src/app/App.workbench.test.tsx

@@ -24,6 +24,7 @@ import {
   press,
   pressKey,
   typeText,
+  captureSettledFrame,
   waitForState,
   waitForText,
   waitForTextGone,
@@ -212,7 +213,7 @@ describe("the four-pane workbench", () => {
     await waitForState(setup, () => diffToggleColumn(setup) === -1);
     await setup.mockMouse.click(railToggleColumn(setup), HEADER_ROW);
 
-    expect(changedFileVisibleInRightTree(setup.captureCharFrame(), "store.ts")).toBe(false);
+    expect(changedFileVisibleInRightTree(await captureSettledFrame(setup), "store.ts")).toBe(false);
     setup.renderer.destroy();
   });
 

package/src/settings/config.test.ts

@@ -661,6 +661,23 @@ describe("the /lgtm default quick action", () => {
   });
 });
 
+test("herdr ACP defaults off and requires a boolean experimental TOML flag", () => {
+  const dir = mkdtempSync(join(tmpdir(), "cueloop-herdr-config-"));
+  const path = join(dir, "config.toml");
+
+  try {
+    expect(loadConfig({ userConfigPath: path }).experimental.herdrAcp).toBe(false);
+    writeFileSync(path, "[experimental]\nherdr_acp = true\n");
+    expect(loadConfig({ userConfigPath: path }).experimental.herdrAcp).toBe(true);
+    writeFileSync(path, "[experimental]\nherdr_acp = true\nthread_agent = true\n");
+    expect(loadConfig({ userConfigPath: path }).experimental.threadAgent).toBe(true);
+    writeFileSync(path, '[experimental]\nherdr_acp = "true"\n');
+    expect(loadConfig({ userConfigPath: path }).experimental.herdrAcp).toBe(false);
+  } finally {
+    rmSync(dir, { recursive: true, force: true });
+  }
+});
+
 test("Thread agents default off and require a boolean experimental TOML flag", () => {
   const dir = mkdtempSync(join(tmpdir(), "cueloop-agent-config-"));
   const path = join(dir, "config.toml");

package/src/settings/config.ts

@@ -142,6 +142,7 @@ export interface CueloopConfig {
 export interface ExperimentalConfig {
   prototypePixels: boolean;
   threadAgent: boolean;
+  herdrAcp: boolean;
 }
 
 /** Where a display name came from: typed by the user, or verified from a GitHub login. */
@@ -264,6 +265,7 @@ const IntegrationsSchema = v.object({ obsidian: v.optional(ObsidianSchema) });
 const ExperimentalSchema = v.object({
   prototype_pixels: v.fallback(v.optional(v.boolean()), undefined),
   thread_agent: v.fallback(v.optional(v.boolean()), undefined),
+  herdr_acp: v.fallback(v.optional(v.boolean()), undefined),
 });
 const ReviewSchema = v.object({
   skill: v.fallback(v.optional(v.string()), undefined),
@@ -452,7 +454,7 @@ export function loadConfig(
       fx: { ...THREAD_MODEL_DEFAULTS.fx },
     },
     integrations: { obsidian: { ...OBSIDIAN_DEFAULTS } },
-    experimental: { prototypePixels: false, threadAgent: false },
+    experimental: { prototypePixels: false, threadAgent: false, herdrAcp: false },
   };
   // Theme name and per-token overrides are separate concerns, composed once
   // after all layers: the last file to set [ui] theme wins, and every [theme]
@@ -700,6 +702,8 @@ function applyExperimental(
   if (raw.prototype_pixels !== undefined) config.prototypePixels = raw.prototype_pixels;
 
   if (raw.thread_agent !== undefined) config.threadAgent = raw.thread_agent;
+
+  if (raw.herdr_acp !== undefined) config.herdrAcp = raw.herdr_acp;
 }
 
 /** The preferred harness affects new Threads; existing Threads require a handoff. */

package/src/terminal/pty.ts

@@ -27,6 +27,7 @@ export interface PtyForkOptions {
   rows?: number;
   cwd?: string;
   env?: Record<string, string>;
+  readPollMaxMs?: number;
 }
 
 export interface ExitEvent {
@@ -127,6 +128,7 @@ class Pty implements IPty {
   private processId: number;
   private columns: number;
   private rowCount: number;
+  private readonly readPollMaxMs: number;
   private closing = false;
   private reading = false;
   private readonly decoder = new TextDecoder("utf-8");
@@ -139,6 +141,11 @@ class Pty implements IPty {
     args: string[],
     options: PtyForkOptions,
   ) {
+    this.readPollMaxMs = options.readPollMaxMs ?? READ_IDLE_MAX_MS;
+
+    if (!Number.isFinite(this.readPollMaxMs) || this.readPollMaxMs < 1)
+      throw new Error("PTY read poll limit must be at least one millisecond");
+
     this.columns = options.cols ?? DEFAULT_COLS;
     this.rowCount = options.rows ?? DEFAULT_ROWS;
     const cwd = options.cwd ?? process.cwd();
@@ -236,7 +243,7 @@ class Pty implements IPty {
         this.closing = true;
         this.exitEvent.fire({ exitCode });
       } else {
-        const delay = Math.min(READ_IDLE_MAX_MS, READ_IDLE_MS * 2 ** Math.min(idlePolls, 5));
+        const delay = Math.min(this.readPollMaxMs, READ_IDLE_MS * 2 ** Math.min(idlePolls, 5));
 
         idlePolls += 1;
         await new Promise((resolve) => setTimeout(resolve, delay));

package/src/testing/test-support.ts

@@ -308,7 +308,9 @@ async function waitForFramePredicate(
       );
     }
 
-    const frame = setup.captureCharFrame();
+    // Retry while a render still holds the buffer lease.
+    // eslint-disable-next-line no-await-in-loop
+    const frame = await captureSettledFrame(setup);
 
     polls++;
 
@@ -340,6 +342,27 @@ async function waitForFramePredicate(
   }
 }
 
+/**
+ * Capture a frame, retrying while OpenTUI still holds the buffer lease.
+ * A render in progress answers the capture with OutputBusy.
+ */
+export async function captureSettledFrame(
+  setup: { captureCharFrame(): string },
+  deadline = Date.now() + 1_000,
+): Promise<string> {
+  try {
+    return setup.captureCharFrame();
+  } catch (error) {
+    const busy = error instanceof Error && error.message.includes("OutputBusy");
+
+    if (!busy || Date.now() >= deadline) throw error;
+
+    await new Promise((resolve) => setTimeout(resolve, 15));
+
+    return captureSettledFrame(setup, deadline);
+  }
+}
+
 /** Wait until a state predicate holds (daemon round-trips included). */
 export async function waitForState(
   setup: TestRendererSetup,
@@ -373,7 +396,9 @@ export async function waitForState(
       );
     }
 
-    const frame = setup.captureCharFrame();
+    // Retry while a render still holds the buffer lease.
+    // eslint-disable-next-line no-await-in-loop
+    const frame = await captureSettledFrame(setup);
 
     polls++;
 

package/src/thread/thread-controller.ts

@@ -479,6 +479,9 @@ class Controller implements ReviewController {
 
         this.client = client;
         client.onEvent((event) => {
+          // Agent transcripts have their own observer; they do not change Thread records or the inbox.
+          if (event.event === "agent.updated") return;
+
           // another controller/observer changed state: re-fetch the active session's content
           const session = this.snapshot.session;