@cplieger/web-terminal-engine 5.0.5 → 5.0.8

package/jsr.json

@@ -1,6 +1,6 @@
 {
   "name": "@cplieger/web-terminal-engine",
-  "version": "5.0.5",
+  "version": "5.0.8",
   "license": "MPL-2.0",
   "exports": {
     ".": "./src/index.ts"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cplieger/web-terminal-engine",
-  "version": "5.0.5",
+  "version": "5.0.8",
   "type": "module",
   "scripts": {
     "typecheck": "tsc -p tsconfig.json",
@@ -21,7 +21,7 @@
     "esbuild": "0.28.2",
     "eslint": "10.8.1",
     "fast-check": "4.9.0",
-    "happy-dom": "20.11.2",
+    "happy-dom": "20.11.6",
     "pngjs": "7.0.0",
     "prettier": "3.9.6",
     "typescript": "npm:@typescript/typescript6@6.0.2",

package/src/connection.ts

@@ -782,7 +782,21 @@ function retransmitOutbox(sock: WebSocket, upgraded: boolean, st: ResumeState):
 }
 
 function scheduleReconnect(): void {
-  if (connState.status === "reconnecting") {
+  // Nothing to schedule when the state already owns a path back to a socket: a
+  // backoff timer is pending ("reconnecting"), or a socket exists that will
+  // drive its own retry from its close handler or its connect timeout
+  // ("connecting"/"connected"). Both callers set "disconnected" before calling
+  // cb.onClose(), so a consumer that reconnects from its own close hook lands
+  // here at "connecting" — and scheduling over it would OVERWRITE connState,
+  // hiding that socket from connect()'s double-call guard, which then never
+  // aborts it: the timer's connect() adds a second live socket with the first
+  // one's listeners still bound, i.e. the duplicate server connection and
+  // double delivery documented inside connect().
+  if (
+    connState.status === "reconnecting" ||
+    connState.status === "connecting" ||
+    connState.status === "connected"
+  ) {
     return;
   }
   const step = nextBackoffDelay(reconnectDelay);

package/src/store.ts

@@ -1826,6 +1826,17 @@ export class LineStore {
     if (this.highest < 0) {
       return;
     }
+    // No window established yet — the descriptor is still the initial base
+    // 0/height 0, whose bottom row is -1, below every index a line can hold.
+    // This whole pass rests on "the window's bottom row is the most recent line
+    // in the terminal", which says nothing at all when there is no window: a
+    // resume replays history BEFORE the screen frame describing the window, so
+    // taking -1 as the tail bound strands that replay and forgets it, silently
+    // (this path is not a trim, so no marker explains it). Same test `inWindow`
+    // and applyScroll's alt gate use to ask whether a window exists at all.
+    if (this.win.height <= 0) {
+      return;
+    }
     const windowBottom = this.win.base + this.win.height - 1;
     if (this.highest <= windowBottom) {
       return;
@@ -2008,6 +2019,18 @@ export class LineStore {
   }
 
   private updateWindow(msg: ScreenMessage): void {
+    // A base that is not a non-negative integer is not a screen position — the
+    // same validity test markUnconfirmedFrom applies to this very value — so it
+    // must not redefine the window's shape, for the same reason a rows-less
+    // frame must not: `base + height - 1` lands below every retained index, and
+    // truncateBelowWindow then hands the WHOLE store over as stranded past the
+    // live window. That is the silent wipe applyScreen's rows-less guard
+    // records, reached with rows present, and just as silent (this path is not a
+    // trim, so no marker explains the missing transcript). The frame's
+    // addressable rows still apply through applyLine's guard 1.
+    if (!Number.isInteger(msg.base) || msg.base < 0) {
+      return;
+    }
     const next: WindowState = {
       base: msg.base,
       height: msg.rows.length,