@miaskiewicz/turbo-dom 0.1.42 → 0.1.44

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@miaskiewicz/turbo-dom",
-  "version": "0.1.42",
+  "version": "0.1.44",
   "description": "Faster, more spec-correct DOM for test runners — native html5ever (Rust/WASM) parser + lazy copy-on-write DOM. A drop-in-style alternative to jsdom/happy-dom for vitest & jest.",
   "license": "MIT",
   "main": "index.js",
@@ -70,7 +70,8 @@
     "prepublishOnly": "napi build --platform --release",
     "test:vitest": "vitest run -c test-vitest/vitest.config.mjs",
     "test:jest": "jest -c test-jest/jest.config.cjs",
-    "bench:query": "node bench/query.mjs"
+    "bench:query": "node bench/query.mjs",
+    "bench:scorecard": "node bench/scorecard.mjs"
   },
   "keywords": [
     "dom",

package/src/runtime/dom.mjs

@@ -98,9 +98,10 @@ export class Node extends EventTarget {
     if (this.__idx >= 0 && doc && doc.__buf) {
       const buf = doc.__buf;
       for (let c = buf.firstChild(this.__idx); c !== -1; c = buf.nextSib(c)) {
+        const nt = buf.nodeType(c); // read once, pass to __nodeAt (was re-read in its switch)
         // template content fragment is not a child — it's `.content`
-        if (buf.nodeType(c) === DOCUMENT_FRAGMENT_NODE && buf.tagName(c) === 'content') continue;
-        const child = doc.__nodeAt(c);
+        if (nt === DOCUMENT_FRAGMENT_NODE && buf.tagName(c) === 'content') continue;
+        const child = doc.__nodeAt(c, nt);
         child.parentNode = this;
         kids.push(child);
       }
@@ -1402,19 +1403,23 @@ export class Document extends Node {
   }
 
   // nodeAt: one handle per buffer index, memoized → preserves === identity.
-  __nodeAt(idx) {
+  // `nt` (nodeType) may be passed by callers that already read it (__children),
+  // avoiding a second buffer read; other callers leave it undefined.
+  __nodeAt(idx, nt) {
     if (idx < 0) return null;
     const cached = this.__cache[idx];
     if (cached !== undefined) return cached;
     const buf = this.__buf;
     let node;
-    switch (buf.nodeType(idx)) {
+    if (nt === undefined) nt = buf.nodeType(idx);
+    switch (nt) {
       case ELEMENT_NODE: {
-        node = new Element(this, buf.tagName(idx), buf.ns(idx));
+        const tag = buf.tagName(idx); // read once (was read twice — ctor + template check)
+        node = new Element(this, tag, buf.ns(idx));
         node.__idx = idx;
         node.__attrIdx = idx; // attrs lazy (constructor left __attrs undefined)
         // template content fragment: a child node typed 11 named "content"
-        if (buf.tagName(idx) === 'template') {
+        if (tag === 'template') {
           for (let c = buf.firstChild(idx); c !== -1; c = buf.nextSib(c)) {
             if (buf.nodeType(c) === DOCUMENT_FRAGMENT_NODE && buf.tagName(c) === 'content') {
               node.content = this.__nodeAt(c);