@aragon-agent/cli 0.6.11 → 0.6.13

package/dist/ui/selection/selection-controller.js

@@ -27,9 +27,10 @@
  *    same `decorate` call. What you see is what you get by construction, rather
  *    than by a staleness check that can be wrong.
  */
+import stripAnsi from 'strip-ansi';
 import { createScreenMirror } from './screen-mirror.js';
 import { paintRow, selectionOpen } from './highlight.js';
-import { isEmpty, normalize, rowSpan, selectedText, } from './selection.js';
+import { isEmpty, normalize, rowSpan, selectedText, sliceColumns, } from './selection.js';
 /**
  * Motion coalescing window. The same 16 ms and the same reasoning as
  * `use-wheel-routing.ts`: this one is in the direct path of a hand gesture, so
@@ -67,10 +68,28 @@ export function createSelectionController(options) {
     let disposed = false;
     /** Guards `decorate` against being re-entered from inside a repaint. */
     let painting = false;
+    let notifiedHold = false;
+    let holdNotificationPending = false;
+    const notifyHold = () => {
+        if (disposed || notifiedHold === holdActive)
+            return;
+        notifiedHold = holdActive;
+        for (const listener of [...holdListeners])
+            listener(holdActive);
+    };
     const emitHold = (hold) => {
         holdActive = hold;
-        for (const listener of [...holdListeners])
-            listener(hold);
+        if (!painting) {
+            notifyHold();
+            return;
+        }
+        if (holdNotificationPending)
+            return;
+        holdNotificationPending = true;
+        queueMicrotask(() => {
+            holdNotificationPending = false;
+            notifyHold();
+        });
     };
     const clearFlushTimer = () => {
         if (!flushTimer)
@@ -97,13 +116,7 @@ export function createSelectionController(options) {
             holdWatchdog = null;
             if (!dragging)
                 return;
-            // Let go of `hold` and the drag; KEEP whatever was selected. The user may
-            // still want to copy it, and throwing it away would turn a terminal quirk
-            // into lost work.
-            dragging = false;
-            pendingFocus = null;
-            clearFlushTimer();
-            emitHold(false);
+            clear();
         }, holdMaxMs);
         holdWatchdog.unref?.();
     };
@@ -122,7 +135,7 @@ export function createSelectionController(options) {
         if (!painted && wanted)
             options.requestRedraw();
     };
-    const clear = () => {
+    const clearState = () => {
         // Nothing to undo: no timers, no repaint, no redraw. This is what keeps
         // `onInvalidate` — which fires on every resize and every foreign write —
         // free for a session that has never selected anything.
@@ -139,7 +152,12 @@ export function createSelectionController(options) {
         // exit for two states, not one.
         if (holdActive)
             emitHold(false);
-        paint(false);
+    };
+    const clear = () => {
+        const hadSelection = sel !== null || holdActive;
+        clearState();
+        if (hadSelection)
+            paint(false);
     };
     const cellOf = (event) => {
         // SGR reports are 1-based, frame line `i` is terminal row `i + 1`. A report
@@ -191,6 +209,7 @@ export function createSelectionController(options) {
         // key, resize, overlay, /mouse off).
         settled = true;
         paint(true);
+        options.requestRedraw();
     };
     const onMouse = (event) => {
         if (disposed)
@@ -200,8 +219,7 @@ export function createSelectionController(options) {
             // a selection — N1 already declines auto-scroll-while-dragging. The notch
             // then scrolls normally, because the wheel router is a separate subscriber
             // on the same channel.
-            if (dragging)
-                clear();
+            clear();
             return;
         }
         if (event.kind === 'press') {
@@ -213,6 +231,7 @@ export function createSelectionController(options) {
                 return;
             }
             const cell = cellOf(event);
+            settled = false;
             sel = { anchor: cell, focus: cell };
             pendingFocus = null;
             clearFlushTimer();
@@ -248,16 +267,35 @@ export function createSelectionController(options) {
         finishSelection();
     };
     const unsubscribe = options.source.subscribe(onMouse);
+    const selectedCellsChanged = (lines) => {
+        if (!settled || !sel)
+            return false;
+        const normal = normalize(sel);
+        for (let row = normal.start.row; row <= normal.end.row; row += 1) {
+            const span = rowSpan(normal, row);
+            if (!span)
+                continue;
+            const before = sliceColumns(stripAnsi(mirror.raw[row] ?? ''), span.from, span.to);
+            const after = sliceColumns(stripAnsi(lines[row] ?? ''), span.from, span.to);
+            if (before !== after)
+                return true;
+        }
+        return false;
+    };
     return {
         decorate(lines) {
-            mirror.set(lines);
-            if (painting || sel === null)
-                return lines;
-            const normal = normalize(sel);
-            if (isEmpty(normal))
+            if (painting || disposed)
                 return lines;
             painting = true;
             try {
+                if (selectedCellsChanged(lines))
+                    clearState();
+                mirror.set(lines);
+                if (sel === null)
+                    return lines;
+                const normal = normalize(sel);
+                if (isEmpty(normal))
+                    return lines;
                 const open = selectionOpen(options.theme(), options.caps);
                 const cols = Math.max(1, options.cols());
                 const out = lines.slice();
@@ -317,11 +355,11 @@ export function createSelectionController(options) {
         dispose() {
             if (disposed)
                 return;
-            disposed = true;
             // 4.2.3: a pending selection holds the viewport; teardown must let go
             // of it explicitly, exactly like `clear()` does.
             if (holdActive)
                 emitHold(false);
+            disposed = true;
             unsubscribe();
             clearFlushTimer();
             clearWatchdog();

package/dist/ui/transcript-text.js

@@ -73,6 +73,8 @@ function serviceTextGlyph(status, exitCode, glyphs) {
 }
 function renderEntry(entry, glyphs) {
     switch (entry.kind) {
+        case 'queued':
+            return block('Queued but never sent:', entry.text);
         case 'user':
             return block(glyphs.user, entry.text);
         case 'assistant': {
@@ -211,20 +213,26 @@ export function renderTranscriptText(entries, opts) {
         lines.push(`${g.ellipsis} ${dropped} earlier ${dropped === 1 ? 'entry' : 'entries'} dropped by ` +
             `transcriptRetain ${g.midDot} raise it or /save sooner`);
     }
-    if (entries.length > max && max > 1) {
-        const head = Math.ceil(max / 2);
-        const tail = max - head;
-        const omitted = entries.length - max;
-        for (const entry of entries.slice(0, head))
-            lines.push(...renderEntry(entry, g));
-        lines.push(`${g.ellipsis} ${omitted} entries omitted ${g.midDot} use /save before exiting`);
-        for (const entry of entries.slice(entries.length - tail)) {
+    const historyCount = entries.filter((entry) => entry.kind !== 'queued').length;
+    const omitted = max > 1 ? Math.max(0, historyCount - max) : 0;
+    const head = Math.ceil(max / 2);
+    const tailStart = historyCount - (max - head);
+    let historyIndex = 0;
+    let reported = false;
+    for (const entry of entries) {
+        if (entry.kind === 'queued') {
             lines.push(...renderEntry(entry, g));
+            continue;
         }
-    }
-    else {
-        for (const entry of entries)
-            lines.push(...renderEntry(entry, g));
+        const index = historyIndex++;
+        if (omitted > 0 && index >= head && index < tailStart) {
+            if (!reported) {
+                lines.push(`${g.ellipsis} ${omitted} entries omitted ${g.midDot} use /save before exiting`);
+                reported = true;
+            }
+            continue;
+        }
+        lines.push(...renderEntry(entry, g));
     }
     return `${lines.join('\n')}\n`;
 }

package/dist/ui/use-wheel-routing.js

@@ -106,7 +106,8 @@ export function useWheelRouting(options) {
         if (overlay) {
             // Modes A only. `model` / `confirm` / `question` own their own keys and
             // scrolling them would leave an inconsistent cursor (R-11).
-            const controlled = overlay === 'help' || overlay === 'settings' || overlay === 'plan';
+            const controlled = overlay === 'help' || overlay === 'settings'
+                || overlay === 'plan' || overlay === 'queue';
             if (!controlled)
                 return;
             // D-3: the row is not consulted here either. Behind an overlay the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aragon-agent/cli",
-  "version": "0.6.11",
+  "version": "0.6.13",
   "description": "AragonAgent CLI — a Claude-Code / Codex-style interactive terminal UI (TUI) on top of the @aragon-agent/core engine. Installable and runnable via npm / npx.",
   "license": "MIT",
   "author": "AragonAgent contributors",
@@ -56,7 +56,7 @@
     "prepublishOnly": "npm run build"
   },
   "dependencies": {
-    "@aragon-agent/core": "^0.2.23",
+    "@aragon-agent/core": "^0.2.25",
     "chalk": "^5.6.2",
     "cli-highlight": "^2.0.0",
     "commander": "^12.1.0",