omniharness-cli 0.1.79 → 0.1.81

package/dist/cli.js

@@ -7,23 +7,6 @@ import { ownVersion, runUpdate } from './update.js';
 import { readActiveCombo } from './config/settings.js';
 import { OmniRouteClient } from './config/omniRoute.js';
 import { doctor, helpText, models } from './doctor.js';
-const ALT_ENTER = '\x1b[?1049h\x1b[H';
-const ALT_LEAVE = '\x1b[?1049l';
-/** Plain-text transcript flushed to the primary buffer on exit — the audit trail. */
-function dumpTranscript(messages) {
-    if (messages.length === 0)
-        return;
-    const label = {
-        user: 'you', assistant: 'assistant', thought: 'think', action: 'action',
-        tool: 'tool', command: 'command', error: 'error',
-    };
-    process.stdout.write('\n');
-    for (const message of messages) {
-        const who = message.model ?? label[message.role] ?? message.role;
-        process.stdout.write(`\n─ ${who} ─────\n${message.content.trim()}\n`);
-    }
-    process.stdout.write('\n');
-}
 // A crash anywhere below would otherwise surface as a raw Node stack trace, or
 // as an unhandled rejection that terminates the process without saying why.
 // A CLI should fail with a sentence.
@@ -81,23 +64,26 @@ else {
     void (async () => {
         const saved = await readActiveCombo();
         const engine = await createMastraEngine({ workspaceRoot: process.cwd(), model: saved });
-        // Render the structured UI in the alternate screen buffer so that, on exit,
-        // the primary buffer's scrollback is left holding the plain-text transcript —
-        // the raw audit trail, without maintaining a parallel log.
-        const alt = process.stdout.isTTY === true;
-        if (alt)
-            process.stdout.write(ALT_ENTER);
+        // Render in the primary buffer, not the alternate screen.
+        //
+        // The alternate screen has no scrollback — that is what it is for — so
+        // while the harness was running you could not scroll back to anything: not
+        // a file the agent read three steps ago, not the reasoning behind an edit.
+        // The transcript only appeared once you quit, dumped into the primary
+        // buffer on exit, which is the wrong time to want it.
+        //
+        // Staying in the primary buffer means the TUI's <Static> region writes
+        // settled turns straight into real scrollback, where the terminal scrolls
+        // them like any other output and they are still there afterwards. The exit
+        // dump goes with it: scrollback already holds the transcript, and printing
+        // it again would duplicate every line.
+        //
+        // The cost is that quitting no longer restores the pre-launch screen. For
+        // a tool whose output you are meant to read back, that is the right trade.
+        //
         // The app owns Ctrl+C so idle quits but an in-flight run is cancelled first.
         const { waitUntilExit } = render(_jsx(TerminalInterface, { engine: engine }), { exitOnCtrlC: false });
-        try {
-            await waitUntilExit();
-        }
-        finally {
-            if (alt) {
-                process.stdout.write(ALT_LEAVE);
-                dumpTranscript(engine.state.messages);
-            }
-        }
+        await waitUntilExit();
     })();
 }
 //# sourceMappingURL=cli.js.map

package/dist/ui/terminalInterface.js

@@ -80,6 +80,12 @@ const MODE_SEQ = ['plan', 'build', 'research', 'crazy'];
 const PERM_SEQ = ['ask', 'acceptEdits', 'bypass'];
 const PERM_LABEL = { ask: 'manual', acceptEdits: 'accept edits', bypass: 'bypass' };
 const PERM_COLOR = (p) => (p === 'bypass' ? PALETTE.error : p === 'acceptEdits' ? PALETTE.warn : PALETTE.muted);
+/**
+ * The widest the content is allowed to get, however wide the terminal is.
+ * Past roughly this the eye loses the start of a line on the way back from
+ * the end of it, and the chrome ends up further from what it describes.
+ */
+const MAX_MEASURE = 100;
 const clamp = (value, min, max) => Math.min(max, Math.max(min, value));
 const widthOf = (stdout) => Math.max(48, stdout.columns ?? 80);
 const clip = (text, width) => text.length <= width ? text : `${text.slice(0, Math.max(0, width - 1))}…`;
@@ -197,7 +203,7 @@ function Field({ label, children }) {
 export function Hero(props) {
     const { width, endpoint, model, mode, perm, workspace, sessions, skills, plugins, mcpTools } = props;
     const wide = twoColumn(width);
-    const outer = Math.min(width - 2, 84);
+    const outer = width;
     const column = wide ? Math.floor((outer - 3) / 2) : outer;
     const inner = Math.max(12, column - 4);
     // Values sit to the right of the label, so that is the room they actually get.
@@ -214,7 +220,7 @@ export function TerminalInterface({ engine }) {
     const { stdin } = useStdin();
     const [width, setWidth] = useState(() => widthOf(stdout));
     const [edit, setEdit] = useState({ value: '', cursor: 0 });
-    const inputWidth = Math.max(16, width - 12);
+    const inputWidth = Math.max(16, Math.min(width, MAX_MEASURE) - 12);
     // Settled transcript. Rendered once each into <Static> — the terminal's own
     // scrollback is the history; there is no in-app viewport to scroll.
     const [lines, setLines] = useState(() => engine.state.messages.map(lineFromMessage));
@@ -912,7 +918,13 @@ export function TerminalInterface({ engine }) {
     }, [busy]);
     const modeKey = kitty === true ? 'M' : 'E';
     const modeAccent = MODE_ACCENT[mode];
-    const contentWidth = Math.max(20, width - 6);
+    // On a wide terminal the UI used to stretch edge to edge, which puts the
+    // input box and the text you are reading at opposite ends of the screen.
+    // Hold the content to a readable measure and centre it instead; below that
+    // measure the gutter collapses and nothing changes.
+    const measure = Math.min(width, MAX_MEASURE);
+    const gutter = Math.max(2, Math.floor((width - measure) / 2));
+    const contentWidth = Math.max(20, measure - 6);
     const terminalRows = stdout.rows ?? 24;
     const metrics = engine.client.snapshotMetrics();
     const meter = contextMeter(metrics.compression.inputTokens, engine.state.activeModel, metrics.fallback.activeProvider);
@@ -933,7 +945,7 @@ export function TerminalInterface({ engine }) {
     const liveThinkView = liveThinkLines.slice(-Math.max(2, Math.floor(liveBudget / 2)));
     const liveAnswerView = liveAnswerLines.slice(-liveBudget);
     const doneAgents = agents.filter((lane) => lane.status === 'done').length;
-    return _jsxs(Box, { flexDirection: "column", width: width, paddingX: 2, children: [_jsx(Static, { items: lines, children: (line, index) => _jsx(TranscriptEntry, { line: line, width: contentWidth, fallbackModel: engine.state.activeModel }, index) }, staticKey), _jsxs(Box, { flexDirection: "column", children: [lines.length === 0 && !busy && _jsx(Hero, { width: width, endpoint: engine.client.endpoint ?? 'omniroute', model: engine.state.activeModel, mode: mode, perm: permMode, workspace: engine.state.workspace.root, sessions: recentSessions, skills: engine.skills.length, plugins: new Set(engine.skills.map((skill) => skill.source).filter((source) => source !== undefined)).size, mcpTools: engine.mcpTools.length }), liveThink !== '' && _jsxs(Box, { flexDirection: "column", marginTop: 1, children: [_jsx(Text, { bold: true, color: PALETTE.warn, children: "\u00B7 thinking" }), liveThinkView.map((segments, index) => _jsx(SegmentText, { segments: segments, role: "thinking" }, index))] }), liveAnswer !== '' && _jsxs(Box, { flexDirection: "column", marginTop: 1, children: [_jsx(Text, { bold: true, color: PALETTE.accent, children: engine.state.activeModel }), liveAnswerView.map((segments, index) => _jsx(SegmentText, { segments: segments, role: "assistant" }, index))] }), toolCards.slice(-5).map((card) => {
+    return _jsxs(Box, { flexDirection: "column", width: width, paddingLeft: gutter, paddingRight: gutter, children: [_jsx(Static, { items: lines, children: (line, index) => _jsx(TranscriptEntry, { line: line, width: contentWidth, fallbackModel: engine.state.activeModel }, index) }, staticKey), _jsxs(Box, { flexDirection: "column", children: [lines.length === 0 && !busy && _jsx(Hero, { width: contentWidth + 2, endpoint: engine.client.endpoint ?? 'omniroute', model: engine.state.activeModel, mode: mode, perm: permMode, workspace: engine.state.workspace.root, sessions: recentSessions, skills: engine.skills.length, plugins: new Set(engine.skills.map((skill) => skill.source).filter((source) => source !== undefined)).size, mcpTools: engine.mcpTools.length }), liveThink !== '' && _jsxs(Box, { flexDirection: "column", marginTop: 1, children: [_jsx(Text, { bold: true, color: PALETTE.warn, children: "\u00B7 thinking" }), liveThinkView.map((segments, index) => _jsx(SegmentText, { segments: segments, role: "thinking" }, index))] }), liveAnswer !== '' && _jsxs(Box, { flexDirection: "column", marginTop: 1, children: [_jsx(Text, { bold: true, color: PALETTE.accent, children: engine.state.activeModel }), liveAnswerView.map((segments, index) => _jsx(SegmentText, { segments: segments, role: "assistant" }, index))] }), toolCards.slice(-5).map((card) => {
                         const expanded = expandedTool === card.id;
                         const dot = card.status === 'running' ? _jsx(Text, { color: PALETTE.warn, children: ".." }) : card.status === 'error' ? _jsx(Text, { color: PALETTE.error, children: "FAIL" }) : _jsx(Text, { color: PALETTE.success, children: "ok" });
                         const head = card.name === 'run_command'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "omniharness-cli",
-  "version": "0.1.79",
+  "version": "0.1.81",
   "description": "OmniHarness — local-first agent orchestration harness for OmniRoute.",
   "license": "MIT",
   "type": "module",
@@ -22,7 +22,7 @@
     "build": "tsc -p tsconfig.json",
     "dev": "tsx src/cli.tsx",
     "start": "node dist/cli.js",
-    "typecheck": "tsc -p tsconfig.json --noEmit",
+    "typecheck": "tsc -p tsconfig.json --noEmit && tsc -p tsconfig.test.json --noEmit",
     "test": "node --test --import ./test/setup.mjs --import tsx test/**/*.test.ts"
   },
   "keywords": [