@cplieger/web-terminal-ui 7.0.0 → 7.0.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cplieger/web-terminal-ui",
-  "version": "7.0.0",
+  "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",

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

@@ -790,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();
@@ -823,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;
     }
   }
 
@@ -842,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 },