@cplieger/web-terminal-engine 6.2.0 → 6.4.0-dev.3

package/jsr.json

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cplieger/web-terminal-engine",
-  "version": "6.2.0",
+  "version": "6.4.0-dev.3",
   "type": "module",
   "scripts": {
     "typecheck": "tsc -p tsconfig.json",
@@ -22,18 +22,18 @@
     "@stryker-mutator/vitest-runner": "10.0.0",
     "@types/pngjs": "6.0.5",
     "@typescript/native": "npm:typescript@7.0.2",
-    "@vitest/browser-playwright": "5.0.3",
-    "@vitest/coverage-v8": "5.0.3",
+    "@vitest/browser-playwright": "5.0.4",
+    "@vitest/coverage-v8": "5.0.4",
     "esbuild": "0.28.2",
     "eslint": "10.12.0",
     "fast-check": "4.10.2",
-    "knip": "6.40.0",
+    "knip": "6.41.0",
     "playwright": "1.64.0",
     "pngjs": "7.0.0",
     "prettier": "3.9.9",
     "typescript": "npm:@typescript/typescript6@6.0.2",
     "typescript-eslint": "8.71.1",
-    "vitest": "5.0.3"
+    "vitest": "5.0.4"
   },
   "overrides": {
     "qs": "^6.15.2"

package/src/render.ts

@@ -1352,10 +1352,9 @@ export function createRenderer(opts: RendererOptions): Renderer {
     if (el === null) {
       return null;
     }
+    // Non-negative: rowAtViewportTop answers only with a CONTENT row, and
+    // isContentRow requires a parsed index that resolves back to that element.
     const abs = rowAbs(el);
-    if (abs < 0) {
-      return null;
-    }
     return {
       abs,
       screenTop: el.offsetTop - scroll.currentScrollTop(),
@@ -1468,10 +1467,10 @@ export function createRenderer(opts: RendererOptions): Renderer {
       abs >= gap.hi
         ? Math.max(gap.lo, gap.hi - budget, floor) // approaching from BELOW the gap's top
         : Math.max(gap.lo, floor); // approaching from above: start at the gap's low edge
+    // At least 1, so no caller has to re-check it: every gap the store returns
+    // satisfies `hi > lo` and `hi > pagingFloor`, and `budget` is at least 1,
+    // which puts `fromAbs` strictly below `gap.hi` on both approaches.
     const maxLines = Math.min(budget, gap.hi - fromAbs);
-    if (maxLines < 1) {
-      return;
-    }
     requestHistoryFn(fromAbs, maxLines);
   }