@impeccable/reviews 0.1.1 → 0.1.2

package/dist/ui/look.js

@@ -230,9 +230,15 @@ function runPage(scene, f, inner, st) {
                     : t.status === 'skipped' ? '–'
                         : '○';
         const ink = t.status === 'in_progress' || t.status === 'failed' ? st.full : t.status === 'skipped' ? st.faint : st.soft;
-        // The task in progress carries the wait's timer, so a screen waiting on a host's build is never still.
-        const detail = t.status === 'skipped' || t.status === 'failed' ? t.status : i === running && scene.wait ? scene.wait : undefined;
-        return ink(`${mark} ${t.title}${detail ? ` · ${detail}` : ''}`);
+        const row = (detail) => `${mark} ${t.title}${detail ? ` · ${detail}` : ''}`;
+        if (t.status === 'skipped' || t.status === 'failed')
+            return ink(row(t.status));
+        if (i !== running || !scene.wait)
+            return ink(row());
+        // The task in progress says what the run waits for and the time so far, never how long it may wait, so a screen
+        // waiting on a host's build is never still; the time alone where the words would push it off the row.
+        const waiting = row(`waiting for ${scene.wait.host ? `${scene.wait.host}'s preview` : 'the preview'} · ${scene.wait.timer}`);
+        return ink(visibleWidth(waiting) <= inner ? waiting : row(scene.wait.timer));
     });
     const counted = scene.tasks.filter((t) => t.status !== 'skipped');
     const skipped = scene.tasks.length - counted.length;
@@ -277,8 +283,10 @@ function pageOf(f, inner) {
     switch (scene.kind) {
         case 'wait':
             return { heading: START.heading, text: said(START.text, inner, st), rule: divider(inner, st), action: [center(st.faint('One moment…'), inner)] };
-        case 'intro':
-            return { heading: START.heading, text: said(START.text, inner, st), rule: divider(inner, st), action: [center(st.full("Let's start with login"), inner)], ways: [['Continue', 'Enter']] };
+        case 'intro': {
+            const note = scene.note ? ['', ...block(hang(scene.note, Math.min(TEXT_WIDTH, inner), '! ').map(st.soft), inner)] : [];
+            return { heading: START.heading, text: said(START.text, inner, st), rule: divider(inner, st), action: [center(st.full("Let's start with login"), inner), ...note], ways: [['Continue', 'Enter']] };
+        }
         case 'signin': {
             // P still takes a pasted code on a remote shell; --help says so, the screen does not.
             const under = scene.note ?? (scene.copied ? 'Copied to clipboard!' : '');

package/dist/ui/run-state.js

@@ -24,7 +24,7 @@ export class RunState {
     tasks = [];
     previous;
     current;
-    /** A bounded wait under way: the current line carries its timer, counting up to the limit. */
+    /** A bounded wait under way: the current line carries its timer, counting up. */
     waiting;
     /** When something last changed on screen: a status, the task list, an answered question. */
     changedAt;
@@ -51,12 +51,12 @@ export class RunState {
         this.touch();
     }
     /**
-     * The wait's timer, "1:05 of 5:00", while one is under way: the task in progress carries it on the tasks screen.
-     * `live` false gives the wait's start, "0:00 of 5:00", the same on every tick: a drawing that changes only when
-     * a wait begins or ends, so the frames record the wait once rather than once a second.
+     * The wait under way, which the task in progress carries on the tasks screen: the host whose preview it is and the
+     * time so far, "1:05". `live` false gives the wait's start, "0:00", the same on every tick: a drawing that changes
+     * only when a wait begins or ends, so the frames record the wait once rather than once a second.
      */
     timer(live = true) {
-        return this.waiting ? `${timer(live ? this.now() - this.waiting.since : 0)} of ${timer(this.waiting.limitMs)}` : undefined;
+        return this.waiting && { host: this.waiting.host, timer: timer(live ? this.now() - this.waiting.since : 0) };
     }
     setTasks(tasks) {
         this.tasks = tasks.map((t) => ({ ...t }));
@@ -70,14 +70,14 @@ export class RunState {
      * The band's two lines as they read now: after a minute with nothing new the
      * last line moves up, dimmed, under "Still working…" and the time since.
      * `quiet` is false while the person is being asked. During a bounded wait
-     * the current line carries the timer, "1:05 of 5:00", so the screen never
+     * the current line carries the timer, "1:05", so the screen never
      * looks stalled; `live` false holds the timer at the wait's start, for a
      * drawing that changes only when the words do.
      */
     band(quiet = true, live = true) {
-        if (this.waiting && this.current) {
-            return { previous: this.previous, current: `${this.current} · ${this.timer(live)}` };
-        }
+        const wait = this.timer(live);
+        if (wait && this.current)
+            return { previous: this.previous, current: `${this.current} · ${wait.timer}` };
         const silence = this.now() - this.changedAt;
         if (quiet && silence >= SILENCE_MS)
             return { previous: this.current, current: `Still working… ${elapsed(silence)}` };

package/dist/ui/screens.js

@@ -1,13 +1,17 @@
 import { readFileSync } from 'node:fs';
 import { format } from 'node:util';
 import { CURSOR_MARKER, getCapabilities, Input, KeybindingsManager, matchesKey, ProcessTerminal, setKeybindings, TUI_KEYBINDINGS, TuiAltScreen } from '@earendil-works/pi-tui';
-import { readCheckout } from '../preflight.js';
 import { copyText } from '../login.js';
 import { workStarted } from '../stages.js';
 import { mountUi } from './bridge.js';
 import { createFrameWriter, frameState } from './frames.js';
 import { drawFrame, OUTCOME, PRODUCT, palette } from './look.js';
 import { RunState } from './run-state.js';
+/** What the command does before a screen ends it at once (Ctrl+C before a run, or a signal): report the stop (main.ts), briefly. */
+let beforeExit = async () => { };
+export function onScreenExit(fn) {
+    beforeExit = fn;
+}
 /** How often what moves on screen moves: the loader, the spinner. */
 const TICK_MS = 80;
 /** The run screen's first line, before the agent has said anything. */
@@ -44,7 +48,7 @@ class Wizard {
     onExit = () => this.stop();
     onSignal = (signal) => {
         this.stop();
-        process.kill(process.pid, signal);
+        void beforeExit('terminated').finally(() => process.kill(process.pid, signal));
     };
     constructor() {
         // The alternate screen's own scrolling and search keys stay off: the wizard draws every row, and a typed answer keeps Home and End.
@@ -228,7 +232,8 @@ class Wizard {
         const run = this.views.find((v) => v.kind === 'run');
         if (!run) {
             this.stop();
-            process.exit(130);
+            void beforeExit('ctrl_c').finally(() => process.exit(130));
+            return;
         }
         this.stopAsked = 0;
         for (const question of this.questions.splice(0))
@@ -330,7 +335,7 @@ class Wizard {
         if (!view)
             return { kind: 'wait' };
         if (view.kind === 'intro')
-            return { kind: 'intro' };
+            return { kind: 'intro', note: view.note };
         if (view.kind === 'signin')
             return { kind: 'signin', link: view.link, copied: view.copied, again: view.again, note: view.pasting ? undefined : view.note };
         if (view.kind === 'pull')
@@ -401,30 +406,24 @@ function host() {
     wizard ??= new Wizard();
     return wizard;
 }
-function repository() {
-    try {
-        const checkout = readCheckout(process.cwd());
-        return `${checkout.owner}/${checkout.repo}`;
-    }
-    catch {
-        return undefined;
-    }
-}
 /** Writes every screen into `dir` as it changes (--frames), for the eval. */
 export function recordFrames(dir) {
     frames = createFrameWriter(dir);
 }
-/** The first screen: what the command does, and the way on. Resolves true to go on, false to cancel. */
-export function showIntro() {
+/** The first screen: what the command does, a warning to read before anything runs (`note`), and the way on; without a terminal, a line naming `repository`, the owner/name being set up. Resolves true to go on, false to cancel. */
+export function showIntro(repository, note) {
     const screen = host();
     if (!screen) {
-        console.log(`${PRODUCT}: setting up ${repository() ?? 'this repository'}.`);
+        console.log(`${PRODUCT}: setting up ${repository}.`);
+        if (note)
+            console.log(`! ${note}`);
         console.log();
         return Promise.resolve(true);
     }
     return new Promise((resolve) => {
         const view = {
             kind: 'intro',
+            note,
             done: (ok) => {
                 screen.remove(view);
                 if (ok)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@impeccable/reviews",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "Set up a repository for Impeccable for PRs from your terminal.",
   "license": "SEE LICENSE IN LICENSE",
   "private": false,
@@ -24,6 +24,7 @@
     "eval": "npm run build && node dist/eval/main.js"
   },
   "dependencies": {
+    "@anthropic-ai/sandbox-runtime": "0.0.78",
     "@earendil-works/pi-coding-agent": "~0.87.1",
     "@earendil-works/pi-tui": "~0.87.1"
   },