@yaag/cli 0.2.0 → 0.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yaag/cli",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "publishConfig": {
     "access": "public"
   },
@@ -21,8 +21,8 @@
   },
   "dependencies": {
     "@earendil-works/pi-tui": "^0.84.0",
-    "@yaag/runtime": "0.2.0",
-    "@yaag/tui": "0.2.0",
+    "@yaag/runtime": "0.2.1",
+    "@yaag/tui": "0.2.1",
     "typebox": "1.3.7"
   }
 }

package/src/cli-tree-host.ts

@@ -1,9 +1,9 @@
 /**
- * The read-only half of `RunTreeViewHost` for the standalone CLI (spec §4).
+ * The read-only half of `RunTreeViewHost` for the standalone CLI (ADR-0008).
  *
  * The Host Session gets the same shape from `packages/extension`; here the pi
  * services are replaced by pi-tui primitives, so `apps/yaag` needs no pi. The
- * caller adds `stop`, `detach`, and `done`: nothing in this module can prompt
+ * caller adds `stop` and `done`: nothing in this module can prompt
  * an Agent (ADR — a Peek observes, it never sends).
  */
 import { readFile, writeFile } from "node:fs/promises";
@@ -26,7 +26,7 @@ export interface CliTreeHostOptions {
 }
 
 /** The read-only capabilities the CLI Run view has. */
-export type ReadOnlyCliHost = Omit<RunTreeViewHost, "stop" | "detach" | "done">;
+export type ReadOnlyCliHost = Omit<RunTreeViewHost, "stop" | "done">;
 
 /** Builds the CLI's read-only host; it never writes to an Agent or to the Run. */
 export function createCliTreeHost(options: CliTreeHostOptions): ReadOnlyCliHost {

package/src/interactive-run.ts

@@ -1,8 +1,8 @@
 /**
- * The `yaag run` path that opens the alt-screen Run tree (spec §4).
+ * The `yaag run` path that opens the alt-screen Run tree (architecture §9).
  *
  * `orchestrateInteractiveRun` is process-free: every collaborator arrives as a
- * parameter, so each exit path — settle, detach, fatal render, signal — is
+ * parameter, so each exit path — settle, close, fatal render, signal — is
  * driven in tests without a TTY. `runInteractive` is the thin production edge
  * that binds it to the real terminal, the real signals, and the real streams.
  *
@@ -63,9 +63,10 @@ export interface InteractiveRunDeps {
  * exit code of the Run.
  *
  * The terminal is restored before this resolves or rejects, on every path: the
- * settle-then-`q` path, a confirmed stop, a detach, and a render throw. A
- * detach closes the alt-screen and keeps awaiting the Run on the plain lines,
- * so the process never exits with a Run still running. A fatal render error is
+ * settle-then-`esc` path, a confirmed stop, an `esc` over a live Run, and a
+ * render throw. Closing the view over a live Run closes the alt-screen and
+ * keeps awaiting the Run on the plain lines, so the process never exits with a
+ * Run still running. A fatal render error is
  * reported only after the terminal is restored and the Run is aborted and
  * reaped, so no Run is abandoned pending. A fatal accepted at any point while
  * the handlers are installed is rethrown, never converted into a normal return.
@@ -118,7 +119,7 @@ export async function orchestrateInteractiveRun(deps: InteractiveRunDeps): Promi
     }
     const settled = await running;
     app.settle(viewResult(settled));
-    // The settled view stays interactive until the reader presses `q` (spec §1).
+    // The settled view stays interactive until the reader presses `esc`.
     const exit = await Promise.race([app.exit, signal.wait]);
     let fatal = exit.kind === "fatal" ? exit : signal.taken();
     const closing = app.close();

package/src/tree-app.ts

@@ -1,6 +1,6 @@
 /**
  * The standalone CLI's interactive Run view: the same `@yaag/tui` tree the Host
- * Session opens, hosted in an alt-screen pi-tui app (spec §4).
+ * Session opens, hosted in an alt-screen pi-tui app (architecture §9).
  *
  * Every collaborator arrives as a parameter, so the app is driven in tests over
  * a recording terminal; `alt-screen.ts` owns the real `ProcessTerminal`.
@@ -28,7 +28,6 @@ export interface TreeAppDeps {
 /** How the app ended. A fatal exit still restored the terminal. */
 export type TreeAppExit =
   | { readonly kind: "dismissed" }
-  | { readonly kind: "detached" }
   | { readonly kind: "fatal"; readonly error: unknown };
 
 /** What `finish` writes after the alt-screen closes (architecture §9). */
@@ -43,7 +42,7 @@ export interface FinalOutput {
 export interface TreeApp {
   /** The projection the view reads; the caller renders the final frame from it. */
   readonly state: TreeState;
-  /** Resolves when the reader dismisses or detaches, or a render throws. */
+  /** Resolves when the reader closes the view, or a render throws. */
   readonly exit: Promise<TreeAppExit>;
   /** Folds one Lifecycle Event into the projection and redraws. */
   present(event: LifecycleEvent): void;
@@ -54,7 +53,7 @@ export interface TreeApp {
   /**
    * Drains input and leaves the alt-screen, writing no channel. Once.
    *
-   * A detach closes the alt-screen while the Run is still live, so this step is
+   * `esc` closes the alt-screen while the Run is still live, so this step is
    * separate from the final channel emission.
    */
   close(): Promise<void>;
@@ -98,13 +97,11 @@ export function startTreeApp(deps: TreeAppDeps): TreeApp {
     host: {
       ...host,
       stop: deps.stop,
-      // A CLI has no background surface, so a detach closes the alt-screen and
-      // the caller keeps awaiting the Run on the plain line presenter.
-      detach: () => resolveExit({ kind: "detached" }),
-      done: (kind) => resolveExit({ kind: kind === "detached" ? "detached" : "dismissed" }),
+      // A CLI has no background surface, so closing the view closes the
+      // alt-screen and the caller keeps awaiting the Run on the plain lines.
+      done: () => resolveExit({ kind: "dismissed" }),
     },
     ...(deps.label === undefined ? {} : { label: deps.label }),
-    surface: "foreground",
     ...(deps.now === undefined ? {} : { now: deps.now }),
   });