@miaskiewicz/turbo-dom 0.1.23 → 0.1.25

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@miaskiewicz/turbo-dom",
-  "version": "0.1.23",
+  "version": "0.1.25",
   "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",

package/src/runtime/collections.mjs

@@ -5,23 +5,32 @@ function makeLive(getArray, extra = {}) {
   const target = function () {};
   return new Proxy(target, {
     get(_t, key) {
-      const arr = getArray();
-      if (key === 'length') return arr.length;
-      if (key === 'item') return (i) => arr[i] ?? null;
-      if (key === 'forEach') return (cb, thisArg) => arr.forEach(cb, thisArg);
-      if (key === 'entries') return () => arr.entries();
-      if (key === 'keys') return () => arr.keys();
-      if (key === 'values') return () => arr[Symbol.iterator]();
-      if (key === Symbol.iterator) return () => arr[Symbol.iterator]();
-      if (key === 'toString') return () => '[object NodeList]';
-      if (key in extra) return extra[key](arr);
-      if (typeof key === 'string' && /^\d+$/.test(key)) return arr[Number(key)] ?? undefined;
+      if (typeof key === 'string') {
+        // hot path: indexed access coll[i]. A property key starting with a digit
+        // is a numeric index — detect via charCode, no regex, no string-compare
+        // chain. getArray() is called only when actually needed.
+        const c = key.charCodeAt(0);
+        if (c >= 48 && c <= 57) return getArray()[+key];
+        if (key === 'length') return getArray().length;
+        if (key === 'item') return (i) => getArray()[i] ?? null;
+        if (key === 'forEach') return (cb, thisArg) => getArray().forEach(cb, thisArg);
+        if (key === 'entries') return () => getArray().entries();
+        if (key === 'keys') return () => getArray().keys();
+        if (key === 'values') return () => getArray()[Symbol.iterator]();
+        if (key === 'toString') return () => '[object NodeList]';
+        if (key in extra) return extra[key](getArray());
+        return undefined;
+      }
+      if (key === Symbol.iterator) return () => getArray()[Symbol.iterator]();
       return undefined;
     },
     has(_t, key) {
-      const arr = getArray();
-      if (typeof key === 'string' && /^\d+$/.test(key)) return Number(key) < arr.length;
-      return key === 'length' || key === 'item' || key === 'forEach' || key in extra;
+      if (typeof key === 'string') {
+        const c = key.charCodeAt(0);
+        if (c >= 48 && c <= 57) return +key < getArray().length;
+        return key === 'length' || key === 'item' || key === 'forEach' || key in extra;
+      }
+      return false;
     },
     ownKeys() {
       const arr = getArray();
@@ -30,8 +39,11 @@ function makeLive(getArray, extra = {}) {
     getOwnPropertyDescriptor(_t, key) {
       const arr = getArray();
       if (key === 'length') return { configurable: true, enumerable: false, value: arr.length };
-      if (typeof key === 'string' && /^\d+$/.test(key) && Number(key) < arr.length) {
-        return { configurable: true, enumerable: true, value: arr[Number(key)] };
+      if (typeof key === 'string') {
+        const c = key.charCodeAt(0);
+        if (c >= 48 && c <= 57 && +key < arr.length) {
+          return { configurable: true, enumerable: true, value: arr[+key] };
+        }
       }
       return undefined;
     },

package/src/runtime/dom.mjs

@@ -338,7 +338,7 @@ export class Element extends Node {
     super(ownerDocument);
     this.localName = localName;
     this.__ns = namespace;            // '', 'svg', 'math'
-    this.__attrs = [];                // [{name, value, prefix}]
+    this.__attrs = undefined;         // lazily built (buffer or []) on first attr touch
     this.__attrIdx = -1;              // buffer index for lazy attr inflation
     this.content = null;              // <template> content fragment
     this.shadowRoot = null;           // open shadow root, if attached
@@ -1106,7 +1106,7 @@ export class Document extends Node {
       case ELEMENT_NODE: {
         node = new Element(this, buf.tagName(idx), buf.ns(idx));
         node.__idx = idx;
-        node.__attrIdx = idx; node.__attrs = undefined; // lazy: build on first attr access
+        node.__attrIdx = idx; // attrs lazy (constructor left __attrs undefined)
         // template content fragment: a child node typed 11 named "content"
         if (buf.tagName(idx) === 'template') {
           for (let c = buf.firstChild(idx); c !== -1; c = buf.nextSib(c)) {

package/src/runtime/events.mjs

@@ -114,13 +114,16 @@ function normalizeOptions(options) {
 
 export class EventTarget {
   constructor() {
-    // type -> array of { callback, capture, once, passive }
-    this.__listeners = new Map();
+    // type -> array of { callback, capture, once, passive }. Lazily created on
+    // first addEventListener — most inflated nodes never get a listener, so this
+    // skips a Map allocation per node (inflation is a top hot path).
+    this.__listeners = null;
   }
 
   addEventListener(type, callback, options) {
     if (callback == null) return;
     const o = normalizeOptions(options);
+    if (!this.__listeners) this.__listeners = new Map();
     let list = this.__listeners.get(type);
     if (!list) { list = []; this.__listeners.set(type, list); }
     // dedupe on (callback, capture) per spec
@@ -129,6 +132,7 @@ export class EventTarget {
   }
 
   removeEventListener(type, callback, options) {
+    if (!this.__listeners) return;
     const o = normalizeOptions(options);
     const list = this.__listeners.get(type);
     if (!list) return;