@cplieger/web-terminal-ui 6.1.5 → 7.0.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cplieger/web-terminal-ui",
-  "version": "6.1.5",
+  "version": "7.0.1",
   "type": "module",
   "description": "Reference touch-first browser UI for @cplieger/web-terminal-engine: a composable kernel + feature modules (tabs, activity monitor, mobile toolbar, context menu, IME, predictive echo, viewport handling) and CSS. One createTerminal() call.",
   "license": "MPL-2.0",
@@ -40,7 +40,7 @@
     "verify": "bash scripts/verify.sh"
   },
   "peerDependencies": {
-    "@cplieger/web-terminal-engine": "^3.10.0 || ^4.0.0 || ^5.0.0"
+    "@cplieger/web-terminal-engine": "^5.0.3"
   },
   "overrides": {
     "qs": "^6.15.2"

package/src/features/context-menu.ts

@@ -188,9 +188,6 @@ export function contextMenu(opts: ContextMenuOptions = {}): TerminalFeature {
             clipboard.paste();
           });
         }
-        if (menu.childElementCount === 0) {
-          return; // nothing to offer
-        }
         // Make visible (so it has measurable size), then place it within the
         // visible viewport via the shared point-anchored positioner (clamp to the
         // visual viewport; flip above the finger near the bottom edge).

package/src/features/tabs/index.ts

@@ -244,7 +244,12 @@ async function createSessionHonouringRetry(
   isTornDown: () => boolean,
 ): Promise<SessionInfo> {
   const startedAt = Date.now();
-  let lastAnnouncedAt = 0;
+  // The elapsed reading of the last announcement, or null when nothing has been
+  // announced yet. NOT a 0 sentinel: elapsed is legitimately 0 when the server
+  // refuses inside the first millisecond (a host on the same machine), and a 0
+  // sentinel read that as "never spoken" and re-announced on the next refusal —
+  // the one thing the throttle below exists to prevent.
+  let lastAnnouncedAt: number | null = null;
   for (;;) {
     try {
       return await api.create();
@@ -266,7 +271,7 @@ async function createSessionHonouringRetry(
       // only said "not yet" -- and the first announcement fires before any retry
       // has happened at all, so a user two seconds into a boot was told
       // "retrying" about a request that had been made exactly once.
-      if (lastAnnouncedAt === 0 || elapsed - lastAnnouncedAt >= CREATE_RETRY_REANNOUNCE_MS) {
+      if (lastAnnouncedAt === null || elapsed - lastAnnouncedAt >= CREATE_RETRY_REANNOUNCE_MS) {
         lastAnnouncedAt = elapsed;
         const reason = err.serverMessage ?? "Server is not ready yet";
         ctx.toast(`${reason}; waiting…`, 8000);

package/src/kernel/kernel.ts

@@ -601,7 +601,13 @@ function buildTerminal(
 
   connection.init({
     computeSize: render.computeSize,
-    getHaveThrough: render.getHighestIndex,
+    // getHaveThrough is deliberately NOT supplied. The engine defaults it to its
+    // own renderer (engine >= 5.0.3), which is the only party that knows which
+    // store is currently bound and which of its rows the server has confirmed.
+    // Supplying it from here is what claimed rows the application had merely
+    // DRAWN — `render.getHighestIndex` is the window's bottom row — and left a
+    // frozen copy of the composer parked in scrollback on almost every reattach.
+    // An explicit member wins over the default, so re-adding one reopens that.
     // --- Demand-paged scrollback (engine docs/paged-scrollback.md §4-5) ---
     // The transport is store-blind and viewport-blind by design, so every one of
     // these forwards a decision only the renderer can make. Without them the
@@ -784,29 +790,29 @@ function buildTerminal(
   );
 
   /**
-   * Applies the part of a `mapKeyboardEvent` result that BOTH keydown paths (the
-   * focused textarea below, and the document-level type-to-focus fallback in the
-   * focus-strategy section) handle identically. Returns false only for "ignore",
-   * the one arm the two callers resolve differently: the textarea lets its own
-   * `input` event deliver the character, while the fallback has no `input` event
-   * coming and must send the character itself.
+   * Applies a `mapKeyboardEvent` result. BOTH keydown paths — the focused textarea
+   * below, and the document-level type-to-focus fallback in the focus-strategy
+   * section — handle every arm identically, including "ignore": the textarea lets
+   * its own `input` event deliver the character, and the fallback has already
+   * resolved that case before it gets here, computing the typed character itself
+   * because no `input` event is coming for a key that targeted the body.
    *
-   * Switching over the whole `KeyboardResult` union in one place is what keeps
-   * the two paths from drifting — handling only "send" in the fallback silently
-   * dropped Shift+PageUp, which is exactly what a reader who just selected some
-   * scrollback presses next.
+   * Switching over the whole `KeyboardResult` union in one place is what keeps the
+   * two paths from drifting — handling only "send" in the fallback silently dropped
+   * Shift+PageUp, which is exactly what a reader who just selected some scrollback
+   * presses next.
    */
-  function applyMappedKey(ev: KeyboardEvent, result: keyboard.KeyboardResult): boolean {
+  function applyMappedKey(ev: KeyboardEvent, result: keyboard.KeyboardResult): void {
     switch (result.kind) {
       case "send":
         ev.preventDefault();
         sendText(result.bytes);
-        return true;
+        return;
       case "scroll-up": {
         ev.preventDefault();
         const h = termWrap.clientHeight;
         termWrap.scrollTop = Math.max(0, termWrap.scrollTop - h);
-        return true;
+        return;
       }
       case "scroll-down": {
         ev.preventDefault();
@@ -817,10 +823,10 @@ function buildTerminal(
         // reasoning as the engine's bottom writes.
         const max = Math.max(0, termWrap.scrollHeight - h);
         termWrap.scrollTop = Math.min(max, termWrap.scrollTop + h);
-        return true;
+        return;
       }
       case "ignore":
-        return false;
+        return;
     }
   }
 
@@ -836,9 +842,9 @@ function buildTerminal(
           return;
         }
       }
-      // A false return ("ignore") needs nothing here: the `input` listener above
-      // delivers the character, which is also how IME and dead-key composition
-      // output reaches the terminal.
+      // Nothing more is needed for an "ignore": the `input` listener above delivers
+      // the character, which is also how IME and dead-key composition output
+      // reaches the terminal.
       applyMappedKey(ev, mapKeyboardEvent(ev, modes));
     },
     { signal },