fix(cli): close twelve OpenTUI parity gaps found against ink

packages/cli/src/ui/components/messages/ToolMessage.tsx

@@ -1157,7 +1157,35 @@ export const TOOL_ARGS_INLINE_MAX_LINES = 2;
 
 /**
  * One-line JSON for the `ui.showToolCallArgs` row, or undefined when there is
- * nothing worth adding.
+ * nothing worth adding. Serializes `args`, then defers to
+ * {@link formatInlineToolArgsJson} for the row itself.
+ */
+export function formatInlineToolArgs(
+  args: Record<string, unknown> | undefined,
+  description: string,
+  uncapped: boolean,
+  rowWidth?: number,
+): string | undefined {
+  if (!args || Object.keys(args).length === 0) {
+    return undefined;
+  }
+
+  let json: string;
+  try {
+    json = JSON.stringify(args);
+  } catch {
+    // Circular or otherwise unserializable args — the header line is all we
+    // can honestly show.
+    return undefined;
+  }
+
+  return formatInlineToolArgsJson(json, description, uncapped, rowWidth);
+}
+
+/**
+ * The `ui.showToolCallArgs` row over an already-serialized `json`, so a renderer
+ * that carries the call's arguments as text (OpenTUI's `tool-args` event) draws
+ * the same row as this one rather than keeping a second copy of the policy.
  *
  * Skipped when `description` already IS the args JSON: MCP invocations return
  * `safeJsonStringify(params)` from `getDescription()`, so rendering both would
@@ -1179,25 +1207,12 @@ export const TOOL_ARGS_INLINE_MAX_LINES = 2;
  * component). When given, the row is bounded to `TOOL_ARGS_INLINE_MAX_LINES`
  * wrapped rows rather than by character count alone — see that constant.
  */
-export function formatInlineToolArgs(
-  args: Record<string, unknown> | undefined,
+export function formatInlineToolArgsJson(
+  json: string,
   description: string,
   uncapped: boolean,
   rowWidth?: number,
 ): string | undefined {
-  if (!args || Object.keys(args).length === 0) {
-    return undefined;
-  }
-
-  let json: string;
-  try {
-    json = JSON.stringify(args);
-  } catch {
-    // Circular or otherwise unserializable args — the header line is all we
-    // can honestly show.
-    return undefined;
-  }
-
   const trimmedDescription = description.trim();
   if (trimmedDescription.startsWith('{')) {
     try {

packages/cli/src/ui/opentui/dialogs-auth.test.tsx

@@ -149,6 +149,16 @@ async function typeText(text: string): Promise<void> {
   }
 }
 
+/** Every character in one act, as a burst out of a single pty read arrives. */
+async function typeBatched(text: string): Promise<void> {
+  const handler = lastKeyboardHandler();
+  await act(async () => {
+    for (const char of text) {
+      handler(baseKeyEvent({ name: char, sequence: char }));
+    }
+  });
+}
+
 async function pressEsc(): Promise<boolean> {
   const handler = mocks.state.inputHandlers.at(-1);
   if (!handler) throw new Error('no raw input handler registered');
@@ -445,6 +455,17 @@ describe('bracketed-paste into dialog inputs (#57)', () => {
     await press('return'); // baseUrl → apiKey
   }
 
+  it('keeps every character of a burst that shares one batch', async () => {
+    await runToApiKeyStep();
+    await typeBatched('sk-burst-key');
+    expect(
+      screen.getByText((_, element) => element?.textContent === 'sk-burst-key'),
+    ).toBeTruthy();
+    // the burst is what the wizard carries forward, not its last character
+    await press('return'); // apiKey → models
+    expect(screen.getByText(/Enter model IDs directly/)).toBeTruthy();
+  });
+
   it('inserts a paste into the API-key input and prevents default', async () => {
     await runToApiKeyStep();
     const event = await pasteText('sk-pasted-key');

packages/cli/src/ui/opentui/dialogs-auth.tsx

@@ -20,7 +20,7 @@
  *  - documentation/TOS links render as plain text (no OSC 8 in dialogs).
  */
 
-import { useCallback, useLayoutEffect, useMemo, useState } from 'react';
+import { useCallback, useLayoutEffect, useMemo, useRef, useState } from 'react';
 import { useKeyboard, usePaste, useRenderer } from '@opentui/react';
 import type { PasteEvent } from '@opentui/core';
 import { decodePasteBytes } from '@opentui/core';
@@ -239,18 +239,29 @@ function useLineInputKeys(
   onChange: (next: string) => void,
   onSubmit: () => void,
 ) {
+  // Key events can land in one React batch, where the value captured by the
+  // render that registered the handler is already stale by the second
+  // keystroke. The mirror is written synchronously so each event appends to
+  // what the previous one produced, and re-synced on render so a value set
+  // from anywhere else is not lost.
+  const latest = useRef(value);
+  latest.current = value;
+  const change = (next: string) => {
+    latest.current = next;
+    onChange(next);
+  };
   useKeyboard((key) => {
     const o = toOriginalKey(key);
     if (o.name === 'return' || o.name === 'enter') {
       onSubmit();
       return;
     }
     if (o.name === 'backspace' || o.name === 'delete') {
-      onChange(value.slice(0, -1));
+      change(latest.current.slice(0, -1));
       return;
     }
     if (isPrintableKeyInput(key)) {
-      onChange(value + key.sequence);
+      change(latest.current + key.sequence);
     }
   });
   // Bracketed pastes arrive as one PasteEvent with no keypress per character
@@ -262,7 +273,7 @@ function useLineInputKeys(
     const text = normalizePastedText(decodePasteBytes(event.bytes));
     if (!text) return;
     event.preventDefault();
-    onChange(value + text);
+    change(latest.current + text);
   });
 }