@cplieger/web-terminal-ui 8.6.0-dev.9 → 8.7.0-dev.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cplieger/web-terminal-ui",
-  "version": "8.6.0-dev.9",
+  "version": "8.7.0-dev.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",
@@ -44,32 +44,33 @@
     "verify": "bash scripts/verify.sh"
   },
   "peerDependencies": {
-    "@cplieger/web-terminal-engine": "^6.4.0-dev.4"
+    "@cplieger/web-terminal-engine": "^6.4.0"
   },
   "overrides": {
-    "qs": "^6.15.2"
+    "qs": "^6.15.2",
+    "minimatch": "^11.0.1"
   },
   "devDependencies": {
     "@eslint/js": "10.0.1",
-    "@stryker-mutator/core": "10.0.0",
-    "@stryker-mutator/typescript-checker": "10.0.0",
-    "@stryker-mutator/vitest-runner": "10.0.0",
-    "@types/node": "26.6.5",
+    "@stryker-mutator/core": "10.1.1",
+    "@stryker-mutator/typescript-checker": "10.1.1",
+    "@stryker-mutator/vitest-runner": "10.1.1",
+    "@types/node": "26.6.6",
     "@typescript/native": "npm:typescript@7.0.2",
-    "@vitest/browser": "5.0.5",
-    "@vitest/browser-playwright": "5.0.5",
-    "@vitest/coverage-v8": "5.0.5",
-    "eslint": "10.12.0",
+    "@vitest/browser": "5.1.0",
+    "@vitest/browser-playwright": "5.1.0",
+    "@vitest/coverage-v8": "5.1.0",
+    "eslint": "10.14.0",
     "fast-check": "4.10.2",
-    "html-validate": "11.16.2",
-    "knip": "6.41.0",
+    "html-validate": "11.18.0",
+    "knip": "6.44.1",
     "playwright": "1.64.1",
     "prettier": "3.9.10",
     "stylelint": "17.16.0",
     "stylelint-config-standard": "40.0.0",
     "typescript": "npm:@typescript/typescript6@6.0.2",
-    "typescript-eslint": "8.71.2",
-    "vitest": "5.0.5"
+    "typescript-eslint": "8.72.1",
+    "vitest": "5.1.0"
   },
   "publishConfig": {
     "access": "public"

package/src/features/clipboard.ts

@@ -1,5 +1,6 @@
 import { windowOf } from "../kernel/realm.js";
 import { selectionTextWithin } from "../kernel/selection.js";
+import { isShortcutLetter } from "../kernel/shortcut-letter.js";
 import type { TerminalFeature } from "../kernel/types.js";
 
 /** The value a peer reads through `ctx.use(clipboard())`. Both members are
@@ -62,11 +63,11 @@ export function clipboard(): TerminalFeature<ClipboardApi> {
         // Consuming plain Ctrl+V without preventDefault stops the kernel mapping
         // it to \x16 while the native paste still flows into the textarea, so no
         // clipboard read and no Firefox clipboard-read popup.
-        if (ev.ctrlKey && !ev.shiftKey && !ev.altKey && !ev.metaKey && ev.code === "KeyV") {
+        if (ev.ctrlKey && !ev.shiftKey && !ev.altKey && !ev.metaKey && isShortcutLetter(ev, "v")) {
           return true;
         }
         if (ev.ctrlKey && ev.shiftKey && !ev.altKey && !ev.metaKey) {
-          if (ev.code === "KeyC") {
+          if (isShortcutLetter(ev, "c")) {
             const sel = selectionTextWithin(surface);
             if (sel !== "") {
               copy(sel);
@@ -74,7 +75,7 @@ export function clipboard(): TerminalFeature<ClipboardApi> {
             ev.preventDefault();
             return true;
           }
-          if (ev.code === "KeyV") {
+          if (isShortcutLetter(ev, "v")) {
             paste();
             ev.preventDefault();
             return true;

package/src/kernel/a11y.ts

@@ -33,7 +33,7 @@ let REANNOUNCE_DELAY_MS = 100;
 
 /** @internal Test seam: re-set every later announcement after `ms`; returns the
  *  previous value so the caller can restore it. */
-// deadset:ignore DS1004 -- the browser tests shorten the re-announce delay so they need not wait out the real one
+// deadset:ignore DS1102 -- the browser tests shorten the re-announce delay so they need not wait out the real one
 export function setReannounceDelayMs(ms: number): number {
   const previous = REANNOUNCE_DELAY_MS;
   REANNOUNCE_DELAY_MS = ms;

package/src/kernel/pane.ts

@@ -14,6 +14,7 @@ import { createRegions } from "./regions.js";
 import { createAnnouncer, createPaneTablist, type PaneTablist } from "./a11y.js";
 import { createConnState } from "./conn-state.js";
 import { selectionTextWithin } from "./selection.js";
+import { isShortcutLetter } from "./shortcut-letter.js";
 import { primaryPointerFine } from "./keyboard-presence.js";
 import { windowOf } from "./realm.js";
 import {
@@ -77,6 +78,14 @@ function typedChar(ev: KeyboardEvent): string | null {
   return first !== undefined && String.fromCodePoint(first) === ev.key ? ev.key : null;
 }
 
+/** Ctrl+C (Shift allowed) or Ctrl+Insert: the chords that copy a selection. */
+function isCopyChord(ev: KeyboardEvent): boolean {
+  if (!ev.ctrlKey || ev.altKey || ev.metaKey) {
+    return false;
+  }
+  return ev.key === "Insert" || isShortcutLetter(ev, "c");
+}
+
 /** iOS sends U+00A0 where a space was typed; shared by every send path so one
  *  physical key cannot produce two byte sequences. */
 function normalizeTypedText(text: string): string {
@@ -986,8 +995,9 @@ function buildPaneInto(
       if (!ownsSelection() && !keysUnclaimed()) {
         return;
       }
-      // Features while the selection is STILL INTACT and before the modifier
-      // gate: clipboard's Ctrl+Shift+C is a modified key that reads the selection.
+      // Features while the selection is STILL INTACT and before the key is
+      // encoded: clipboard's Ctrl+Shift+C reads the selection, and its claim on
+      // Ctrl+V keeps 0x16 off the wire.
       if (runKeydownChain(ev)) {
         return;
       }
@@ -995,7 +1005,7 @@ function buildPaneInto(
       // beforeinput or input, `_tryToHandlePressesEvent:` in
       // https://github.com/WebKit/WebKit/blob/main/Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm),
       // so a dead key or an IME takes focus and passes through, ahead of the
-      // modifier gate that would keep Option+e.
+      // decision below, which drops a key that types nothing.
       if (
         ev.key === "Dead" ||
         ev.key === "Process" ||
@@ -1010,21 +1020,25 @@ function buildPaneInto(
         return;
       }
       // Decided before taking focus, because a bare modifier press (Shift on its
-      // way to Shift+click) must leave the selection alone. Computed only for an
-      // "ignore", so the encoder always wins a key it maps.
+      // way to Shift+click) must leave the selection alone. Modifiers alone never
+      // reject a key the encoder maps, as on the focused path.
       const result = mapKeyboardEvent(ev, engine.modes);
       const char = result.kind === "ignore" ? typedChar(ev) : null;
-      if (result.kind === "ignore" && char === null) {
-        return;
-      }
-      // Modified keys stay with the browser except an AltGr CHARACTER, the way
-      // some layouts type it; AltGraph alone is not a reliable signal (Firefox
-      // reports it for plain Option on macOS), and Alt+ArrowLeft is browser back.
-      const altGraphChar = char !== null && ev.getModifierState("AltGraph");
-      if ((ev.ctrlKey || ev.metaKey || ev.altKey) && !altGraphChar) {
-        return;
+      if (result.kind === "ignore") {
+        if (char === null) {
+          return;
+        }
+        // An unencoded character with Ctrl, Alt or Cmd is the browser's (Cmd+V,
+        // Ctrl+1). A desktop AltGr character needs no exception: it arrives with
+        // neither modifier (the engine's encodeKittyDisambiguate cites why), and
+        // Firefox sets AltGraph on every macOS Option chord.
+        if (ev.ctrlKey || ev.metaKey || ev.altKey) {
+          return;
+        }
       }
-      if (ev.key === "Tab") {
+      // Tab leaves the page (no keyboard trap); a copy chord over this
+      // terminal's selection is the browser's copy, not a key for the shell.
+      if (ev.key === "Tab" || (ownsSelection() && isCopyChord(ev))) {
         return;
       }
       doc.getSelection()?.removeAllRanges();

package/src/kernel/shortcut-letter.ts

@@ -0,0 +1,9 @@
+/** True when `ev` is the shortcut letter `letter` (lowercase a-z): its `key`, either
+ *  case, when the key types an ASCII letter (Dvorak's C is on KeyI), else its physical
+ *  `Key<letter>` (Russian С on KeyC, or a non-ASCII Latin letter such as é). */
+export function isShortcutLetter(ev: KeyboardEvent, letter: string): boolean {
+  if (ev.key.toLowerCase() === letter) {
+    return true;
+  }
+  return ev.code === `Key${letter.toUpperCase()}` && !/^[a-z]$/i.test(ev.key);
+}

package/src/viewport.ts

@@ -13,7 +13,7 @@ let SETTLE_MS = 350;
 
 /** @internal Test seam: arm every settle started from now on at `ms`; returns the
  *  previous value so the caller can restore it. */
-// deadset:ignore DS1004 -- the browser tests shorten the settle delay so they need not wait out the real one
+// deadset:ignore DS1102 -- the browser tests shorten the settle delay so they need not wait out the real one
 export function setSettleMs(ms: number): number {
   const previous = SETTLE_MS;
   SETTLE_MS = ms;