@stacksjs/stx 0.2.366 → 0.2.368

package/dist/{chunk-jf9mv5tq.js → chunk-kaf0x9rk.js}

@@ -2606,8 +2606,8 @@ catch (e) {
             return expressionCallsSignal(expression, prop)
               || expressionUsesSignalApi(expression, prop);
           });
-          var fn = __stxCompile(['__scope__'], 'with(__scope__) { return (' + expression + ') }');
-          value = fn(unwrapScope);
+          var fn = __stxCompileScoped(unwrapScope, [], 'return (' + expression + ')');
+          value = fn(...__stxArgs(fn, unwrapScope));
           if (value && typeof value === 'function' && (value._isSignal || value._isDerived)) value = value();
         }
         catch (e) {
@@ -2993,8 +2993,8 @@ else if (part) {
 
         // Use auto-unwrap proxy (Feature #1)
         const unwrapScope = createExpressionAutoUnwrapProxy(activeScope, expr);
-        const fn = __stxCompile(['__scope__'], 'with(__scope__) { return (' + expr + ') }');
-        var value = maybeUnwrapSignal(fn(unwrapScope));
+        const fn = __stxCompileScoped(unwrapScope, [], 'return (' + expr + ')');
+        var value = maybeUnwrapSignal(fn(...__stxArgs(fn, unwrapScope)));
         noteExprSuccess(expr);
         return value;
       }
@@ -3002,8 +3002,8 @@ catch (e) {
         // Auto-unwrap can break explicit signal calls like errorData().message
         // Retry without auto-unwrap so signal functions remain callable
         try {
-          const fn = __stxCompile(['__scope__'], 'with(__scope__) { return (' + expr + ') }');
-          var retried = maybeUnwrapSignal(fn(activeScope));
+          const fn = __stxCompileScoped(activeScope, [], 'return (' + expr + ')');
+          var retried = maybeUnwrapSignal(fn(...__stxArgs(fn, activeScope)));
           noteExprSuccess(expr);
           return retried;
         }
@@ -3520,8 +3520,8 @@ catch (e) {
         return direct();
       try {
         var unwrapScope = createExpressionAutoUnwrapProxy(capturedScope, expr);
-        var fn = __stxCompile(['__scope__'], 'with(__scope__) { return (' + expr + ') }');
-        return fn(unwrapScope);
+        var fn = __stxCompileScoped(unwrapScope, [], 'return (' + expr + ')');
+        return fn(...__stxArgs(fn, unwrapScope));
       }
       catch (e) {
         if (!(e instanceof ReferenceError) && !(e instanceof TypeError)) console.warn('[STX] ' + attrName + ' get error:', expr, e);
@@ -4138,8 +4138,12 @@ else if (typeof value === 'string') {
     let templateContent;
     if (isTemplate) {
       templateContent = el.content;
+      deferRowConditionals(templateContent);
     }
 else {
+      // Before the source is cloned into the row template, so that clone does
+      // not build the conditional elements either.
+      deferRowConditionals(el);
       const wrapper = stxHost.clone(el);
       wrapper.removeAttribute('@for');
       wrapper.removeAttribute(':for');
@@ -4246,10 +4250,12 @@ catch (e) {
           return expressionCallsSignal(expression, prop)
             || expressionUsesSignalApi(expression, prop);
         });
-        // new Function body is non-strict, so with() works \u2014 only accessed
-        // properties trigger the proxy's get trap and register as dependencies
-        const fn = __stxCompile(['__scope__'], 'with(__scope__) { return ' + expression + ' }');
-        return fn(unwrapScope);
+        // Compiled over the names the expression mentions (__stxCompileScoped),
+        // so only those are read and register as dependencies. It used with()
+        // over the proxy, which JavaScriptCore resolves through proxy traps on
+        // every identifier - 36x slower than V8 for a list of message rows.
+        const fn = __stxCompileScoped(unwrapScope, [], 'return ' + expression + '');
+        return fn(...__stxArgs(fn, unwrapScope));
       } catch (e) {
         if (!(e instanceof ReferenceError) && !(e instanceof TypeError)) console.warn('[STX] Expression error:', expression, e);
         return '';
@@ -4643,6 +4649,50 @@ catch (e) {
     });
   }
 
+  // A :for row is a clone of its template, so a conditional element in the row
+  // is built for every row whether or not its condition ever holds; :if then
+  // parks the false ones. In WebKit a <video> or <audio> sets up a media player
+  // as it is created: a message list with one media slot per attachment took
+  // 1.8s to render instead of 18ms, and every row paid for elements it never
+  // showed. Before any row is cloned, each eligible conditional element in the
+  // row template is moved into a <template> carrying its condition. Template
+  // content is inert, so a cloned row builds the element only when the
+  // condition holds (bindIf clones template content on the first true).
+  //
+  // Left alone: an if/else chain (its branches must stay siblings), an element
+  // that is also a loop, a component (its setup script is a sibling), and
+  // anything holding a script.
+  var IF_ATTRS = [':if', 'x-if', '@if'];
+  var FOR_ATTRS = [':for', 'x-for', '@for'];
+  function deferRowConditionals(root) {
+    if (!root || !root.querySelectorAll || typeof document === 'undefined' || !document.createElement) return;
+    // A walk, not a selector: escaping ':' and '@' in an attribute selector
+    // through this generated source is easy to get wrong, and wrong throws.
+    var candidates = Array.prototype.slice.call(root.querySelectorAll('*'));
+    for (var i = 0; i < candidates.length; i++) {
+      var el = candidates[i];
+      if (el.tagName === 'TEMPLATE' || !el.parentNode) continue;
+      var attr = null;
+      for (var a = 0; a < IF_ATTRS.length; a++) {
+        if (el.hasAttribute(IF_ATTRS[a])) { attr = IF_ATTRS[a]; break; }
+      }
+      if (!attr) continue;
+      var skip = false;
+      for (var f = 0; f < FOR_ATTRS.length; f++) {
+        if (el.hasAttribute(FOR_ATTRS[f])) skip = true;
+      }
+      if (skip || getElseAttrInfo(el)) continue;
+      var next = el.nextElementSibling;
+      if (next && (getElseAttrInfo(next) || next.tagName === 'SCRIPT')) continue;
+      if (el.hasAttribute('data-stx-scope') || el.hasAttribute('x-data') || el.querySelector('script')) continue;
+      var holder = document.createElement('template');
+      holder.setAttribute(attr, el.getAttribute(attr));
+      el.removeAttribute(attr);
+      el.parentNode.insertBefore(holder, el);
+      holder.content.appendChild(el);
+    }
+  }
+
   var __bindIfCounter = 0;
   // \u2500\u2500 x-else / x-else-if chain support (stacksjs/stx#1734) \u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500
   // Returns { name, terminal } for an else/else-if attribute present on el,
@@ -4765,14 +4815,14 @@ catch (e) {
       var scope = { ...globalHelpers, ...capturedComponentScope, ...(b.capturedElementScope || {}) };
       try {
         var unwrapScope = createExpressionAutoUnwrapProxy(scope, expression);
-        // new Function body is non-strict, so with() works.
-        var fn = __stxCompile(['__scope__'], 'with(__scope__) { return ' + expression + ' }');
-        return fn(unwrapScope);
+        // Compiled over the names the expression mentions (see __stxCompileScoped).
+        var fn = __stxCompileScoped(unwrapScope, [], 'return ' + expression + '');
+        return fn(...__stxArgs(fn, unwrapScope));
       }
 catch (e1) {
         try {
-          var fn2 = __stxCompile(['__scope__'], 'with(__scope__) { return ' + expression + ' }');
-          return fn2(scope);
+          var fn2 = __stxCompileScoped(scope, [], 'return ' + expression + '');
+          return fn2(...__stxArgs(fn2, scope));
         }
 catch (e2) {
           if (!(e2 instanceof ReferenceError) && !(e2 instanceof TypeError)) console.warn('[STX] Expression error:', expression, e2);
@@ -4980,22 +5030,12 @@ catch (e) { /* a disposer of its own is not this branch's problem */ }
     el.removeAttribute(attrName);
 
     if (isTemplate) {
-      // For templates, we need to handle the content fragment
-      const content = el.content;
-      currentNodes = Array.from(content.childNodes).map(n => stxHost.clone(n));
-      // Insert cloned content initially
-      // Capture the anchor once. Re-reading placeholder.nextSibling after
-      // every insertion points at the node just inserted and reverses a
-      // multi-root template branch.
-      const initialAnchor = placeholder.nextSibling;
-      currentNodes.forEach(node => stxHost.insert(parent, node, initialAnchor));
-      // The end of this branch's range, bound at the same time as the content
-      // it closes (#1955). Needed here too, not only on re-show: the first
-      // hide of a branch that was never toggled removes THIS content, and a
-      // nested :for renders rows that are siblings of these clones rather than
-      // among them.
-      endMarker = stxHost.anchor('stx-if-end');
-      stxHost.insert(parent, endMarker, initialAnchor);
+      // Nothing is cloned until the condition holds: the effect below clones
+      // template.content on the first true and binds it then. Every branch
+      // used to be cloned and inserted up front and the false ones removed, so
+      // a false branch still built its elements - in WebKit a <video> or
+      // <audio> sets up a media player as it is created.
+      isInserted = false;
       stxHost.remove(el); // Remove the template element itself
     }
 
@@ -5015,15 +5055,15 @@ catch (e) { /* a disposer of its own is not this branch's problem */ }
       try {
         const scope = { ...globalHelpers, ...capturedComponentScope, ...(capturedElementScope || {}) };
         const unwrapScope = createExpressionAutoUnwrapProxy(scope, expression);
-        const fn = __stxCompile(['__scope__'], 'with(__scope__) { return ' + expression + ' }');
-        return fn(unwrapScope);
+        const fn = __stxCompileScoped(unwrapScope, [], 'return ' + expression + '');
+        return fn(...__stxArgs(fn, unwrapScope));
       }
 catch (e1) {
         try {
           // Retry without the unwrap proxy so call-syntax (signal()) works.
           const scope2 = { ...globalHelpers, ...capturedComponentScope, ...(capturedElementScope || {}) };
-          const fn2 = __stxCompile(['__scope__'], 'with(__scope__) { return ' + expression + ' }');
-          return fn2(scope2);
+          const fn2 = __stxCompileScoped(scope2, [], 'return ' + expression + '');
+          return fn2(...__stxArgs(fn2, scope2));
         }
 catch (e2) {
           if (!(e2 instanceof ReferenceError) && !(e2 instanceof TypeError)) console.warn('[STX] Expression error:', expression, e2);

package/dist/{chunk-9n4wz4p2.js → chunk-n7sq6rpd.js}

@@ -1,2 +1,2 @@
 // @bun
-import{Ip as b,Jp as c,Kp as d,Lp as e,mo as a}from"./chunk-y3gd1g9s.js";import"./chunk-jc6wrxrh.js";import"./chunk-2xpdcr8c.js";import"./chunk-v90kxg77.js";import"./chunk-4arms0am.js";import"./chunk-135mme4g.js";import"./chunk-y4j6br2e.js";import"./chunk-w1asc0zd.js";import"./chunk-kd95t585.js";import"./chunk-rg473fqn.js";import"./chunk-xk402w4a.js";import"./chunk-dnard9gp.js";import"./chunk-vgx5077x.js";import"./chunk-met6wtbk.js";import"./chunk-ta6z7k4p.js";import"./chunk-zxptxz2b.js";import"./chunk-7racewc5.js";import"./chunk-e03m4har.js";import"./chunk-vsqx756d.js";import"./chunk-m8kem5ca.js";export{d as validateClientScript,c as processOnceDirective,b as processJsonDirective,e as processDirectives,a as injectRouterScript};
+import{Ip as b,Jp as c,Kp as d,Lp as e,mo as a}from"./chunk-pdhkzprw.js";import"./chunk-jc6wrxrh.js";import"./chunk-2xpdcr8c.js";import"./chunk-v90kxg77.js";import"./chunk-4arms0am.js";import"./chunk-135mme4g.js";import"./chunk-dmzzbw55.js";import"./chunk-w1asc0zd.js";import"./chunk-kd95t585.js";import"./chunk-htd9bf38.js";import"./chunk-xk402w4a.js";import"./chunk-dnard9gp.js";import"./chunk-vgx5077x.js";import"./chunk-met6wtbk.js";import"./chunk-ta6z7k4p.js";import"./chunk-zxptxz2b.js";import"./chunk-7racewc5.js";import"./chunk-e03m4har.js";import"./chunk-vsqx756d.js";import"./chunk-m8kem5ca.js";export{d as validateClientScript,c as processOnceDirective,b as processJsonDirective,e as processDirectives,a as injectRouterScript};

package/dist/{chunk-ksfnb7rd.js → chunk-nyw7c3e3.js}

@@ -1,2 +1,2 @@
 // @bun
-import{Kj as b,Lj as c,Mj as d,Nj as e,Oj as f,Pj as g,Qj as h,Rj as i,Sj as j,Tj as k,Uj as l,Vj as m,Wj as n,Xj as o}from"./chunk-2wbs79rn.js";import{ul as a}from"./chunk-qjvny3wf.js";import"./chunk-y3gd1g9s.js";import"./chunk-jc6wrxrh.js";import"./chunk-2xpdcr8c.js";import"./chunk-v90kxg77.js";import"./chunk-4arms0am.js";import"./chunk-135mme4g.js";import"./chunk-y4j6br2e.js";import"./chunk-w1asc0zd.js";import"./chunk-kd95t585.js";import"./chunk-rg473fqn.js";import"./chunk-xk402w4a.js";import"./chunk-dnard9gp.js";import"./chunk-vgx5077x.js";import"./chunk-met6wtbk.js";import"./chunk-ta6z7k4p.js";import"./chunk-zxptxz2b.js";import"./chunk-7racewc5.js";import"./chunk-e03m4har.js";import"./chunk-vsqx756d.js";import"./chunk-m8kem5ca.js";export{b as templateCache,j as registerServeModuleBundle,c as readFileCached,e as invalidateFileCacheByDir,d as invalidateFileCache,o as hashFilePath,k as getServeModuleBundle,i as getServeClientAsset,g as getCachedSignalsRuntime,h as getCachedRouterScript,l as clearDevCaches,m as checkCache,n as cacheTemplate,a as cacheEngineFingerprint,f as __setSignalsGeneratorsForTest};
+import{Kj as b,Lj as c,Mj as d,Nj as e,Oj as f,Pj as g,Qj as h,Rj as i,Sj as j,Tj as k,Uj as l,Vj as m,Wj as n,Xj as o}from"./chunk-dxw41d46.js";import{ul as a}from"./chunk-qjvny3wf.js";import"./chunk-pdhkzprw.js";import"./chunk-jc6wrxrh.js";import"./chunk-2xpdcr8c.js";import"./chunk-v90kxg77.js";import"./chunk-4arms0am.js";import"./chunk-135mme4g.js";import"./chunk-dmzzbw55.js";import"./chunk-w1asc0zd.js";import"./chunk-kd95t585.js";import"./chunk-htd9bf38.js";import"./chunk-xk402w4a.js";import"./chunk-dnard9gp.js";import"./chunk-vgx5077x.js";import"./chunk-met6wtbk.js";import"./chunk-ta6z7k4p.js";import"./chunk-zxptxz2b.js";import"./chunk-7racewc5.js";import"./chunk-e03m4har.js";import"./chunk-vsqx756d.js";import"./chunk-m8kem5ca.js";export{b as templateCache,j as registerServeModuleBundle,c as readFileCached,e as invalidateFileCacheByDir,d as invalidateFileCache,o as hashFilePath,k as getServeModuleBundle,i as getServeClientAsset,g as getCachedSignalsRuntime,h as getCachedRouterScript,l as clearDevCaches,m as checkCache,n as cacheTemplate,a as cacheEngineFingerprint,f as __setSignalsGeneratorsForTest};