@brimveyn/aimux 1.15.1 → 1.16.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brimveyn/aimux",
-  "version": "1.15.1",
+  "version": "1.16.0",
   "description": "A terminal multiplexer for AI CLIs. Run Claude, Codex, OpenCode side-by-side with tabbed navigation, split panes, and persistent sessions.",
   "keywords": [
     "ai",

package/src/app-runtime/selection-clipboard-dedup.ts

@@ -0,0 +1,21 @@
+// opentui re-emits 'finishSelection' on re-renders, focus changes, and
+// viewport refreshes. Each of those would call copyToSystemClipboard with the
+// *same* text, blindly clobbering whatever the user just copied in another
+// app. We dedupe by content: aimux never writes the same selection twice in
+// a row to the system clipboard, so an external copy survives any phantom
+// re-emit of the previous in-aimux selection.
+//
+// The dedup state is reset when the renderer reports an empty selection
+// (deselect), so a user who deliberately reselects the same text after
+// clicking elsewhere still gets it copied.
+let lastWrittenText: string | null = null
+
+export function shouldWriteSelectionToClipboard(text: string): boolean {
+  if (text === lastWrittenText) return false
+  lastWrittenText = text
+  return true
+}
+
+export function resetSelectionClipboardDedup(): void {
+  lastWrittenText = null
+}

package/src/app-runtime/selection-scroll.ts

@@ -126,11 +126,15 @@ export function applyViewportObservation(
   next: ViewportObservation | null
 ): ViewportObservation | null {
   if (!next) {
-    if (prior !== null) resetSelectionShiftState(renderer)
+    if (prior !== null) {
+      renderer.clearSelection()
+      resetSelectionShiftState(renderer)
+    }
     return null
   }
 
   if (!prior || prior.tabId !== next.tabId) {
+    if (prior && prior.tabId !== next.tabId) renderer.clearSelection()
     resetSelectionShiftState(renderer)
     return next
   }

package/src/app-runtime/use-mouse-handlers.ts

@@ -21,6 +21,7 @@ import {
 } from './click-selection-resolver'
 import { recordMultiClickClipboardWrite } from './multi-click-clipboard-guard'
 import { requestRenderUpTree } from './render-invalidation'
+import { shouldWriteSelectionToClipboard } from './selection-clipboard-dedup'
 import {
   type AnchoredRatioDragState,
   type AxisDragState,
@@ -146,7 +147,9 @@ function applyMultiClickInitialSelection(
     finishDragging: false,
   })
   requestRenderUpTree(selection.target)
-  copyToSystemClipboard(selection.selectedText)
+  if (shouldWriteSelectionToClipboard(selection.selectedText)) {
+    copyToSystemClipboard(selection.selectedText)
+  }
 }
 
 export function useMouseHandlers({
@@ -573,7 +576,7 @@ export function useMouseHandlers({
     const anchorIdx = drag.anchorAbsRow - startAbs
     const focusIdx = drag.focusAbsRow - startAbs
     const text = extractStreamText(lines, anchorIdx, anchorCol, focusIdx, drag.focusCol)
-    if (text.length > 0) {
+    if (text.length > 0 && shouldWriteSelectionToClipboard(text)) {
       copyToSystemClipboard(text)
       recordMultiClickClipboardWrite()
     }

package/src/app-runtime/use-renderer-bindings.ts

@@ -23,6 +23,10 @@ import {
 import { shouldSuppressSelectionCopy } from './multi-click-clipboard-guard'
 import { writeMacroExpansionToTab, writePasteToTab, writeToTab } from './pty-write'
 import { type OtuiSelection, resolveSelectionClipboardText } from './selection-clipboard'
+import {
+  resetSelectionClipboardDedup,
+  shouldWriteSelectionToClipboard,
+} from './selection-clipboard-dedup'
 import { applyViewportObservation, type ViewportObservation } from './selection-scroll'
 
 const BRACKETED_PASTE_ENABLE_SEQUENCE = '\x1b[?2004h'
@@ -239,7 +243,13 @@ export function useRendererBindings({
         textLength: selectedText.length,
       })
 
-      if (selection.isDragging === true || selectedText.length === 0) {
+      if (selection.isDragging === true) {
+        return
+      }
+      if (selectedText.length === 0) {
+        // Deselect → forget the last-written text so a deliberate reselection
+        // of the same text writes it again.
+        resetSelectionClipboardDedup()
         return
       }
 
@@ -254,7 +264,11 @@ export function useRendererBindings({
       }
 
       renderer.copyToClipboardOSC52(selectedText)
-      copyToSystemClipboard(selectedText)
+      if (shouldWriteSelectionToClipboard(selectedText)) {
+        copyToSystemClipboard(selectedText)
+      } else {
+        logInputDebug('app.selection.dedup', { textLength: selectedText.length })
+      }
     }
 
     renderer.prependInputHandler(handler)

package/src/app-runtime/use-terminal-resize.ts

@@ -19,6 +19,7 @@ import {
   forEachSplitPaneRect,
   toTerminalContentSize,
 } from '../state/layout-resize'
+import { getTreeForTab } from '../state/layout-tree'
 
 const MAIN_AREA_HORIZONTAL_CHROME = 2
 const MAIN_AREA_VERTICAL_PADDING = 0
@@ -161,6 +162,19 @@ export function useTerminalResize({
 
   const activeTabIdRef = useRef(state.activeTabId)
   activeTabIdRef.current = state.activeTabId
+  // Whether the active tab is part of a split. contentOriginRef must hold the
+  // whole terminal-area origin (root.tsx derives splitContentOrigin from it and
+  // SplitLayout re-adds each pane's bounds offset). When split, the active
+  // pane's measured box is pane-relative, so adopting it as the area origin
+  // would double-count the pane offset and push the hardware cursor off along
+  // the split axis. Track it so handleMeasure can skip the x/y overwrite.
+  const activeIsSplitRef = useRef(false)
+  {
+    const id = state.activeTabId
+    const tree =
+      id != null && id !== '' ? getTreeForTab(state.layoutTrees, state.tabGroupMap, id) : null
+    activeIsSplitRef.current = tree?.type === 'split'
+  }
   // Last size we pushed to the backend per tab, used to dedupe the measurement
   // loop so an unchanged box never re-triggers a resize.
   const measuredRef = useRef(new Map<string, { cols: number; rows: number }>())
@@ -187,7 +201,11 @@ export function useTerminalResize({
       const cols = Math.max(PTY_MIN_COLS, rect.cols)
       const rows = Math.max(PTY_MIN_ROWS, rect.rows)
       const isActive = tabId === activeTabIdRef.current
-      if (isActive) {
+      // Only adopt the measured box as the terminal-area origin when the active
+      // pane spans the whole area (no split). In a split the box is pane-local;
+      // keep the model-derived area origin from the terminalSize memo below so
+      // the hardware cursor isn't offset by the pane's bounds (see ref comment).
+      if (isActive && !activeIsSplitRef.current) {
         contentOriginRef.current = { cols, rows, x: rect.x, y: rect.y }
       }
       const prev = measuredRef.current.get(tabId)

package/src/pty/pty-manager.ts

@@ -341,6 +341,25 @@ export class PtyManager extends EventEmitter<PtyManagerEvents> {
         return false
       })
 
+      // Wire the emulator's reply channel back to the pty. When a program
+      // writes a query sequence — CPR (ESC[6n), Device Attributes (ESC[c),
+      // Device Status (ESC[5n), DECRQM, OSC color queries — xterm generates
+      // the answer and emits it via onData. Without forwarding it the program
+      // waits forever for a response it never gets (e.g. AWS CLI / Python
+      // prompt_toolkit warns "terminal doesn't support cursor position
+      // requests (CPR)"). These replies are produced synchronously inside the
+      // emulator.write() below, so write straight to the pty rather than
+      // queueing through the render path. The listener is disposed together
+      // with the emulator in finalize/dispose, mirroring registerCsiHandler.
+      //
+      // onBinary (legacy non-UTF-8 mouse reports) is intentionally not wired:
+      // bun-pty.write() only accepts a UTF-8 string so byte values >127 can't
+      // be transmitted faithfully, and the headless emulator is never fed raw
+      // mouse input, so it never generates those reports in the first place.
+      emulator.onData((reply) => {
+        session.pty.write(reply)
+      })
+
       pty.onData((data) => {
         logDebug('ptyManager.data', {
           byteLength: Buffer.byteLength(data, 'utf8'),