@yaag/extension 0.1.1 → 0.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yaag/extension",
-  "version": "0.1.1",
+  "version": "0.1.4",
   "publishConfig": {
     "access": "public"
   },
@@ -19,14 +19,14 @@
   },
   "scripts": {
     "typecheck": "tsc --noEmit",
-    "test": "for file in src/*.test.ts; do bun test \"$file\" || exit 1; done",
-    "test:e2e": "bun test e2e"
+    "test": "for file in src/*.test.ts; do bun test --timeout 30000 \"$file\" || exit 1; done",
+    "test:e2e": "bun test --timeout 30000 e2e"
   },
   "dependencies": {
     "@earendil-works/pi-tui": "^0.84.0",
-    "@yaag/runtime": "0.1.1",
-    "@yaag/tui": "0.1.1",
-    "@yaag/cli": "0.1.1"
+    "@yaag/runtime": "0.1.4",
+    "@yaag/tui": "0.1.4",
+    "@yaag/cli": "0.1.4"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "*",

package/src/cli-child.ts

@@ -38,6 +38,7 @@ export function startCliChild(options: CliChildOptions): CliChild {
   });
   const stderr = new StderrTail();
   const exited = observeExit(child);
+  releasePipesAfterExit(child);
   const target: ReapTarget = {
     exited,
     endStdin: () => endStdin(child.stdin),
@@ -91,7 +92,30 @@ async function drain(stream: Readable | null, onChunk: (chunk: string) => void):
 
 function observeExit(child: ReturnType<typeof spawn>): Promise<void> {
   return new Promise((resolve, reject) => {
-    child.on("close", () => resolve());
+    // "exit" (process death), not "close": "close" additionally waits for every
+    // stdio pipe to end, and a lost pipe EOF would park the Run forever.
+    child.on("exit", () => resolve());
     child.on("error", reject);
   });
 }
+
+/** How long an exited child's pipes may take to deliver buffered data and EOF. */
+const PIPE_EOF_GRACE_MS = 2_000;
+
+/**
+ * Failsafe for a lost pipe EOF: once the child process is dead, any of its
+ * pipes (stdout, stderr, the fd 3 Lifecycle Event stream) that has not ended
+ * within a bounded grace is destroyed, so the iterators that drain them finish
+ * instead of parking the Run's outcome — and the test process — forever.
+ */
+function releasePipesAfterExit(child: ReturnType<typeof spawn>): void {
+  child.once("exit", () => {
+    const timer = setTimeout(() => {
+      for (const stream of [child.stdout, child.stderr, child.stdio[3]]) {
+        if (isReadable(stream) && !stream.destroyed) stream.destroy();
+      }
+    }, PIPE_EOF_GRACE_MS);
+    // A housekeeping timer must never keep the host process alive.
+    timer.unref();
+  });
+}

package/src/resolve-cli.ts

@@ -5,5 +5,5 @@ import { createRequire } from "node:module";
  * in-monorepo (workspace symlink) and from a published tarball (ADR-0015).
  */
 export function resolveCliEntry(): string {
-  return createRequire(import.meta.url).resolve("yaag/src/cli.ts");
+  return createRequire(import.meta.url).resolve("@yaag/cli/src/cli.ts");
 }