marko 6.0.0-next.3.34 → 6.0.0-next.3.36

package/dist/debug/dom.js

@@ -44,7 +44,6 @@ __export(dom_exports, {
   controllable_textarea_value_effect: () => controllable_input_value_effect,
   createRenderer: () => createRenderer,
   createRendererWithOwner: () => createRendererWithOwner,
-  createScope: () => createScope,
   createTemplate: () => createTemplate,
   data: () => data,
   dynamicClosure: () => dynamicClosure,

package/dist/debug/dom.mjs

@@ -2072,7 +2072,6 @@ export {
   controllable_input_value_effect as controllable_textarea_value_effect,
   createRenderer,
   createRendererWithOwner,
-  createScope,
   createTemplate,
   data,
   dynamicClosure,

package/dist/dom/template.d.ts

@@ -1,3 +1,3 @@
 import type { Template } from "../common/types";
 import { createRenderer } from "./renderer";
-export declare const createTemplate: (templateId: string, template: string, walks?: string | undefined, setup?: ((scope: import("../dom").Scope) => void) | undefined, getArgs?: (() => import("./signals").Signal<unknown>) | undefined) => Template;
+export declare const createTemplate: (templateId: string, template: string, walks?: string | undefined, setup?: ((scope: import("../common/types").Scope) => void) | undefined, getArgs?: (() => import("./signals").Signal<unknown>) | undefined) => Template;

package/dist/dom.d.ts

@@ -1,6 +1,5 @@
 export { attrTag, attrTags } from "./common/attr-tag";
 export { forIn, forOf, forTo } from "./common/for";
-export type { Scope } from "./common/types";
 export { getAbortSignal, resetAbortSignal } from "./dom/abort-signal";
 export { compat } from "./dom/compat";
 export { conditional, conditionalOnlyChild, loopIn, loopOf, loopTo, } from "./dom/control-flow";
@@ -10,6 +9,5 @@ export { on } from "./dom/event";
 export { run } from "./dom/queue";
 export { createRenderer, createRendererWithOwner, dynamicTagAttrs, } from "./dom/renderer";
 export { init, nodeRef, register, registerBoundSignal, registerSubscriber, } from "./dom/resume";
-export { createScope } from "./dom/scope";
 export { conditionalClosure, dynamicClosure, effect, inChild, intersection, intersections, loopClosure, nextTagId, setTagVar, setTagVarChange, state, tagVarSignal, tagVarSignalChange, value, } from "./dom/signals";
 export { createTemplate } from "./dom/template";

package/dist/dom.js

@@ -41,7 +41,6 @@ __export(dom_exports, {
   controllable_textarea_value_effect: () => controllable_input_value_effect,
   createRenderer: () => createRenderer,
   createRendererWithOwner: () => createRendererWithOwner,
-  createScope: () => createScope,
   createTemplate: () => createTemplate,
   data: () => data,
   dynamicClosure: () => dynamicClosure,

package/dist/dom.mjs

@@ -1311,7 +1311,6 @@ export {
   controllable_input_value_effect as controllable_textarea_value_effect,
   createRenderer,
   createRendererWithOwner,
-  createScope,
   createTemplate,
   data,
   dynamicClosure,

package/dist/translator/index.js

@@ -626,19 +626,22 @@ function toMemberExpression(object, key, optional) {
 
 // src/translator/util/runtime.ts
 var pureFunctions = [
-  "createTemplate",
+  "conditional",
+  "conditionalClosure",
+  "conditionalOnlyChild",
   "createRenderer",
   "createRendererWithOwner",
-  "value",
-  "state",
+  "createTemplate",
+  "dynamicClosure",
+  "dynamicTagAttrs",
+  "inChild",
   "intersection",
   "loopClosure",
-  "conditionalClosure",
-  "dynamicClosure",
-  "loopOf",
   "loopIn",
+  "loopOf",
   "loopTo",
-  "conditional"
+  "state",
+  "value"
 ];
 function importRuntime(name2) {
   const { output } = getMarkoOpts();
@@ -3114,7 +3117,9 @@ var program_default = {
       currentProgramPath = program;
       startSection(program);
       trackParamsReferences(program, 2 /* input */);
-      program.node.params[0].extra.binding.nullable = false;
+      if (program.node.params[0]?.extra?.binding) {
+        program.node.params[0].extra.binding.nullable = false;
+      }
       const { scope } = program;
       (program.node.extra ??= {}).domExports = {
         template: scope.generateUid("template_"),
@@ -4569,6 +4574,7 @@ var native_tag_default = {
               );
             }
           }
+          setForceResumeScope(section);
           translateVar(
             tag,
             callRuntime(
@@ -8008,14 +8014,14 @@ var function_default = {
     if (markoRoot && (markoRoot.isMarkoPlaceholder() || markoRoot.isMarkoScriptlet() && markoRoot.node.target === "server")) {
       return;
     }
+    if (isMarkoAttribute(markoRoot) && ((0, import_babel_utils37.isNativeTag)(markoRoot.parentPath) && /^on[A-Z-]/.test(markoRoot.node.name) || isCoreTagName(markoRoot.parentPath, "script") || isCoreTagName(markoRoot.parentPath, "lifecycle") || isCoreTagName(markoRoot.parentPath, "for"))) {
+      return;
+    }
     const { node } = fn;
     const extra = node.extra ??= {};
     const name2 = extra.name = fn.node.id?.name || (isMarkoAttribute(markoRoot) ? markoRoot.node.default ? import_compiler47.types.toIdentifier(
       markoRoot.parentPath.has("var") ? markoRoot.parentPath.get("var") : markoRoot.parentPath.get("name")
     ) : markoRoot.node.name : import_compiler47.types.isVariableDeclarator(fn.parent) && import_compiler47.types.isIdentifier(fn.parent.id) ? fn.parent.id.name : import_compiler47.types.isObjectMethod(node) && import_compiler47.types.isIdentifier(node.key) ? node.key.name : "anonymous");
-    if (isMarkoAttribute(markoRoot) && ((0, import_babel_utils37.isNativeTag)(markoRoot.parentPath) && /^on[A-Z-]/.test(markoRoot.node.name) || isCoreTagName(markoRoot.parentPath, "script") || isCoreTagName(markoRoot.parentPath, "lifecycle") || isCoreTagName(markoRoot.parentPath, "for"))) {
-      return;
-    }
     const {
       markoOpts,
       opts: { filename }
@@ -8370,6 +8376,7 @@ var import_compiler51 = require("@marko/compiler");
 var import_babel_utils40 = require("@marko/compiler/babel-utils");
 var import_path4 = __toESM(require("path"));
 var kChildScopeBinding = Symbol("custom tag child scope");
+var kChildAttrExprs = Symbol("custom tag child attribute expressions");
 var custom_tag_default = {
   analyze: {
     enter(tag) {
@@ -8398,12 +8405,14 @@ var custom_tag_default = {
         void 0,
         tagExtra
       );
+      tagExtra[kChildAttrExprs] = /* @__PURE__ */ new Set([tagExtra]);
       const childFile = (0, import_babel_utils40.loadFileForTag)(tag);
       if (childFile.opts.filename === tag.hub.file.opts.filename) {
         mergeReferences(section, tag.node, getAllTagReferenceNodes(tag.node));
       } else {
         const childProgramExtra = childFile.ast.program.extra;
         analyzeAttrs(
+          tagExtra,
           section,
           tag,
           childProgramExtra?.domExports.params?.props?.[0]
@@ -8442,40 +8451,51 @@ function translateHTML(tag) {
   }
   const tagVar = node.var;
   const section = getSection(tag);
-  const childScopeBinding = node.extra[kChildScopeBinding];
-  const peekScopeId = tag.scope.generateUidIdentifier(childScopeBinding?.name);
-  tag.insertBefore(
-    import_compiler51.types.variableDeclaration("const", [
-      import_compiler51.types.variableDeclarator(peekScopeId, callRuntime("peekNextScope"))
-    ])
-  );
-  getSerializedScopeProperties(section).set(
-    getScopeAccessorLiteral(childScopeBinding),
-    callRuntime("writeExistingScope", peekScopeId)
-  );
   const inputExport = (0, import_babel_utils40.loadFileForTag)(tag)?.ast.program.extra?.domExports?.params?.props?.[0];
   const { properties, statements } = inputExport ? translateAttrs(tag, inputExport.props) : {
     properties: [],
     statements: []
   };
-  if (tagVar) {
-    statements.push(
-      import_compiler51.types.expressionStatement(
-        callRuntime(
-          "setTagVar",
-          getScopeIdIdentifier(section),
-          peekScopeId,
-          import_compiler51.types.stringLiteral(
-            getResumeRegisterId(
-              section,
-              node.var.extra?.binding,
-              // TODO: node.var is not always an identifier.
-              "var"
+  let providesStatefulAttrs = false;
+  for (const expr of tag.node.extra[kChildAttrExprs]) {
+    if (isReferencedExtra(expr) && isStatefulReferences(expr.referencedBindings)) {
+      providesStatefulAttrs = true;
+      break;
+    }
+  }
+  if (providesStatefulAttrs || tagVar) {
+    const childScopeBinding = node.extra[kChildScopeBinding];
+    const peekScopeId = tag.scope.generateUidIdentifier(
+      childScopeBinding?.name
+    );
+    tag.insertBefore(
+      import_compiler51.types.variableDeclaration("const", [
+        import_compiler51.types.variableDeclarator(peekScopeId, callRuntime("peekNextScope"))
+      ])
+    );
+    getSerializedScopeProperties(section).set(
+      getScopeAccessorLiteral(childScopeBinding),
+      callRuntime("writeExistingScope", peekScopeId)
+    );
+    if (tagVar) {
+      statements.push(
+        import_compiler51.types.expressionStatement(
+          callRuntime(
+            "setTagVar",
+            getScopeIdIdentifier(section),
+            peekScopeId,
+            import_compiler51.types.stringLiteral(
+              getResumeRegisterId(
+                section,
+                node.var.extra?.binding,
+                // TODO: node.var is not always an identifier.
+                "var"
+              )
             )
           )
         )
-      )
-    );
+      );
+    }
   }
   if (node.extra.tagNameNullable) {
     const contentProp = getTranslatedBodyContentProperty(properties);
@@ -8617,7 +8637,7 @@ function getTagRelativePath(tag) {
   }
   return relativePath;
 }
-function analyzeAttrs(section, tag, templateExport) {
+function analyzeAttrs(rootTagExtra, section, tag, templateExport) {
   if (!templateExport) {
     dropReferences(getAllTagReferenceNodes(tag.node));
     return;
@@ -8653,7 +8673,7 @@ function analyzeAttrs(section, tag, templateExport) {
           const childAttrExports = templateExport.props[attrTagMeta.name];
           if (childAttrExports) {
             if (childAttrExports.props && !attrTagMeta.dynamic) {
-              analyzeAttrs(section, child, childAttrExports);
+              analyzeAttrs(rootTagExtra, section, child, childAttrExports);
             } else {
               analyzeDynamicChildGroup(attrTagMeta.group, child);
             }
@@ -8677,8 +8697,12 @@ function analyzeAttrs(section, tag, templateExport) {
         }
       }
     }
-    for (const { firstTag, referenceNodes } of nodeReferencesByGroup.values()) {
-      mergeReferences(section, firstTag.node, referenceNodes);
+    for (const {
+      firstTag: { node },
+      referenceNodes
+    } of nodeReferencesByGroup.values()) {
+      mergeReferences(section, node, referenceNodes);
+      rootTagExtra[kChildAttrExprs].add(node.extra);
     }
   }
   const { attributes } = tag.node;
@@ -8696,6 +8720,8 @@ function analyzeAttrs(section, tag, templateExport) {
       spreadReferenceNodes.push(attr2.value);
     } else if (import_compiler51.types.isMarkoSpreadAttribute(attr2)) {
       spreadReferenceNodes = [attr2.value];
+    } else {
+      rootTagExtra[kChildAttrExprs].add(attr2.value.extra ??= {});
     }
   }
   if (spreadReferenceNodes) {

package/dist/translator/visitors/tag/custom-tag.d.ts

@@ -1,9 +1,11 @@
 import { types as t } from "@marko/compiler";
 import { type Binding } from "../../util/references";
 declare const kChildScopeBinding: unique symbol;
+declare const kChildAttrExprs: unique symbol;
 declare module "@marko/compiler/dist/types" {
     interface MarkoTagExtra {
         [kChildScopeBinding]?: Binding;
+        [kChildAttrExprs]?: Set<t.NodeExtra>;
     }
 }
 declare const _default: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "marko",
-  "version": "6.0.0-next.3.34",
+  "version": "6.0.0-next.3.36",
   "description": "Optimized runtime for Marko templates.",
   "keywords": [
     "api",