marko 6.0.132 → 6.0.134

package/dist/debug/dom.js

@@ -1121,7 +1121,7 @@ function _attr_input_value(scope, nodeAccessor, value, valueChange) {
   const normalizedValue = normalizeStrProp(value);
   scope["ControlledHandler:" /* ControlledHandler */ + nodeAccessor] = valueChange;
   if (valueChange) {
-    scope["ControlledType:" /* ControlledType */ + nodeAccessor] = 0 /* InputChecked */;
+    scope["ControlledType:" /* ControlledType */ + nodeAccessor] = 2 /* InputValue */;
     scope["ControlledValue:" /* ControlledValue */ + nodeAccessor] = value;
     if (el.isConnected) {
       setValueAndUpdateSelection(el, normalizedValue);

package/dist/debug/dom.mjs

@@ -1015,7 +1015,7 @@ function _attr_input_value(scope, nodeAccessor, value, valueChange) {
   const normalizedValue = normalizeStrProp(value);
   scope["ControlledHandler:" /* ControlledHandler */ + nodeAccessor] = valueChange;
   if (valueChange) {
-    scope["ControlledType:" /* ControlledType */ + nodeAccessor] = 0 /* InputChecked */;
+    scope["ControlledType:" /* ControlledType */ + nodeAccessor] = 2 /* InputValue */;
     scope["ControlledValue:" /* ControlledValue */ + nodeAccessor] = value;
     if (el.isConnected) {
       setValueAndUpdateSelection(el, normalizedValue);

package/dist/dom.js

@@ -682,7 +682,7 @@ function _attr_input_checkedValue_script(scope, nodeAccessor) {
 }
 function _attr_input_value(scope, nodeAccessor, value, valueChange) {
   let el = scope[nodeAccessor], normalizedValue = normalizeStrProp(value);
-  scope["E" /* ControlledHandler */ + nodeAccessor] = valueChange, valueChange ? (scope["F" /* ControlledType */ + nodeAccessor] = 0 /* InputChecked */, scope["G" /* ControlledValue */ + nodeAccessor] = value, el.isConnected ? setValueAndUpdateSelection(el, normalizedValue) : el.defaultValue = normalizedValue) : (scope["F" /* ControlledType */ + nodeAccessor] = 5 /* None */, el.defaultValue = normalizedValue);
+  scope["E" /* ControlledHandler */ + nodeAccessor] = valueChange, valueChange ? (scope["F" /* ControlledType */ + nodeAccessor] = 2 /* InputValue */, scope["G" /* ControlledValue */ + nodeAccessor] = value, el.isConnected ? setValueAndUpdateSelection(el, normalizedValue) : el.defaultValue = normalizedValue) : (scope["F" /* ControlledType */ + nodeAccessor] = 5 /* None */, el.defaultValue = normalizedValue);
 }
 function _attr_input_value_script(scope, nodeAccessor) {
   let el = scope[nodeAccessor];

package/dist/dom.mjs

@@ -579,7 +579,7 @@ function _attr_input_checkedValue_script(scope, nodeAccessor) {
 }
 function _attr_input_value(scope, nodeAccessor, value, valueChange) {
   let el = scope[nodeAccessor], normalizedValue = normalizeStrProp(value);
-  scope["E" /* ControlledHandler */ + nodeAccessor] = valueChange, valueChange ? (scope["F" /* ControlledType */ + nodeAccessor] = 0 /* InputChecked */, scope["G" /* ControlledValue */ + nodeAccessor] = value, el.isConnected ? setValueAndUpdateSelection(el, normalizedValue) : el.defaultValue = normalizedValue) : (scope["F" /* ControlledType */ + nodeAccessor] = 5 /* None */, el.defaultValue = normalizedValue);
+  scope["E" /* ControlledHandler */ + nodeAccessor] = valueChange, valueChange ? (scope["F" /* ControlledType */ + nodeAccessor] = 2 /* InputValue */, scope["G" /* ControlledValue */ + nodeAccessor] = value, el.isConnected ? setValueAndUpdateSelection(el, normalizedValue) : el.defaultValue = normalizedValue) : (scope["F" /* ControlledType */ + nodeAccessor] = 5 /* None */, el.defaultValue = normalizedValue);
 }
 function _attr_input_value_script(scope, nodeAccessor) {
   let el = scope[nodeAccessor];

package/dist/translator/index.js

@@ -1028,8 +1028,8 @@ var Sorted = class {
       return false;
     }
     for (let i = subLen; i--; ) {
-      const supIndex = this.findIndex(superset, subset[i]);
-      if (supIndex === -1 || supLen - supIndex <= i) return false;
+      const found = findIndexSorted(this.compare, superset, subset[i]);
+      if (found === -1 || supLen - found <= i) return false;
     }
     return true;
   }
@@ -1773,7 +1773,7 @@ function getOrCreateSection(path7) {
   let cur = path7;
   while (true) {
     if (cur.type === "Program" || cur.type === "MarkoTagBody" && !cur.node.attributeTags && !isNativeNode(cur.parentPath)) {
-      return (path7.node.extra ??= {}).section = startSection(cur);
+      return startSection(cur);
     }
     cur = cur.parentPath;
   }
@@ -2370,9 +2370,7 @@ var entry_builder_default = {
     };
     const programExtra = file.path.node.extra;
     const { analyzedTags } = file.metadata.marko;
-    state.imports.push(
-      (0, import_babel_utils12.resolveRelativePath)(entryFile, file.opts.filename)
-    );
+    state.imports.push((0, import_babel_utils12.resolveRelativePath)(entryFile, file.opts.filename));
     if (programExtra.isInteractive || programExtra.needsCompat) {
       state.init = true;
     }
@@ -3462,8 +3460,16 @@ function getBindingGetterIdentifier(binding) {
   return identifier;
 }
 function getSignal(section, referencedBindings, name2 = generateSignalName(referencedBindings)) {
-  if (!Array.isArray(referencedBindings) && referencedBindings?.type === 7 /* constant */) {
-    return getSignal(section, void 0);
+  if (referencedBindings && !Array.isArray(referencedBindings)) {
+    if (referencedBindings.type === 7 /* constant */) {
+      return getSignal(section, void 0);
+    }
+    if (referencedBindings.type !== 4 /* local */ && referencedBindings.section !== section) {
+      const canonicalReference = getCanonicalBinding(referencedBindings);
+      if (canonicalReference !== referencedBindings) {
+        return getSignal(section, canonicalReference);
+      }
+    }
   }
   const signals = getSignals(section);
   let signal = signals.get(referencedBindings);
@@ -3508,9 +3514,9 @@ function getSignal(section, referencedBindings, name2 = generateSignalName(refer
           scopeOffset && getScopeAccessorLiteral(scopeOffset, true)
         );
       };
-    } else if (referencedBindings.section !== section && bindingUtil.find(section.referencedClosures, referencedBindings)) {
+    } else if (referencedBindings.section !== section && sectionUtil.has(referencedBindings.closureSections, section)) {
       signal.build = () => {
-        const closure = getCanonicalBinding(referencedBindings);
+        const closure = referencedBindings;
         const render = getSignalFn(signal);
         const closureSignalBuilder = getClosureSignalBuilder(section);
         if (closureSignalBuilder && !isDynamicClosure(section, closure)) {
@@ -3692,7 +3698,7 @@ function getSignalFn(signal) {
       )
     );
   });
-  if (isValue && binding.sources && binding.type !== 4 /* local */) {
+  if (isValue && binding.sources) {
     let dynamicClosureArgs;
     let dynamicClosureSignalIdentifier;
     forEach(binding.closureSections, (closureSection) => {
@@ -8604,18 +8610,11 @@ function trackVarReferences(tag, type, upstreamAlias) {
     return tagVar.extra?.binding;
   }
 }
-function trackParamsReferences(body, type, upstreamAlias) {
+function trackParamsReferences(body, type) {
   const params = body.node.params;
   if (body.node.body.length && params.length) {
-    const canonicalUpstreamAlias = upstreamAlias && getCanonicalBinding(upstreamAlias);
-    let section;
-    if (canonicalUpstreamAlias) {
-      section = canonicalUpstreamAlias.section;
-      type = canonicalUpstreamAlias.type;
-    } else {
-      section = getOrCreateSection(body);
-    }
-    const paramsBinding = canonicalUpstreamAlias || ((body.node.extra ??= {}).binding = createBinding(
+    const section = getOrCreateSection(body);
+    const paramsBinding = (body.node.extra ??= {}).binding = createBinding(
       generateUid("params"),
       type,
       section,
@@ -8623,8 +8622,11 @@ function trackParamsReferences(body, type, upstreamAlias) {
       void 0,
       void 0,
       params[0].loc
-    ));
-    section.params = paramsBinding;
+    );
+    const bodySection = getSectionForBody(body);
+    if (bodySection) {
+      bodySection.params = paramsBinding;
+    }
     for (let i = 0; i < params.length; i++) {
       const param = params[i];
       if (param.type === "RestElement") {
@@ -9044,20 +9046,20 @@ function finalizeReferences() {
     );
     for (const { isEffect, section: section2 } of binding.reads) {
       if (section2 !== binding.section) {
-        const canonicalUpstreamAlias = getCanonicalBinding(binding);
-        canonicalUpstreamAlias.closureSections = sectionUtil.add(
-          canonicalUpstreamAlias.closureSections,
-          section2
-        );
         if (binding.type === 4 /* local */) {
           section2.referencedLocalClosures = bindingUtil.add(
             section2.referencedLocalClosures,
             binding
           );
         } else {
+          const canonicalUpstreamAlias = getCanonicalBinding(binding);
+          canonicalUpstreamAlias.closureSections = sectionUtil.add(
+            canonicalUpstreamAlias.closureSections,
+            section2
+          );
           section2.referencedClosures = bindingUtil.add(
             section2.referencedClosures,
-            binding
+            canonicalUpstreamAlias
           );
           addOwnerSerializeReason(
             section2,
@@ -10292,12 +10294,13 @@ var define_default = {
       dropNodes(getAllTagReferenceNodes(tag.node));
       return;
     }
-    trackParamsReferences(tagBody, 3 /* param */);
+    const paramsBinding = trackParamsReferences(tagBody, 3 /* param */);
     setTagDownstream(tag, varBinding);
     if (bodySection) {
       if (import_compiler43.types.isIdentifier(tag.node.var)) {
         const babelBinding = tag.scope.getBinding(tag.node.var.name);
         let allDirectReferences = true;
+        let allHaveInput = true;
         for (const ref of babelBinding.referencePaths) {
           if (isReferenceHoisted(babelBinding.path, ref)) {
             throw ref.buildCodeFrameError(
@@ -10306,12 +10309,18 @@ var define_default = {
           }
           if (ref.parent.type === "MarkoTag" && ref.parent.name === ref.node) {
             (ref.parent.extra ??= {}).defineBodySection = bodySection;
+            allHaveInput &&= !ref.parent.arguments?.length;
             dropNodes(ref.parent.name);
           } else {
+            allHaveInput = false;
             allDirectReferences = false;
           }
         }
         if (allDirectReferences) {
+          const inputAlias = allHaveInput && paramsBinding?.propertyAliases.get("0");
+          if (inputAlias) {
+            inputAlias.nullable = false;
+          }
           dropNodes(getAllTagReferenceNodes(tag.node));
           return;
         }

package/dist/translator/util/references.d.ts

@@ -94,7 +94,7 @@ export declare function createBinding(name: string, type: Binding["type"], secti
 export declare function getOrCreatePropertyAlias(binding: Binding, property: string): Binding;
 export declare function trackDomVarReferences(tag: t.NodePath<t.MarkoTag>, binding: Binding): Binding | undefined;
 export declare function trackVarReferences(tag: t.NodePath<t.MarkoTag>, type: BindingType, upstreamAlias?: Binding["upstreamAlias"]): Binding | undefined;
-export declare function trackParamsReferences(body: t.NodePath<t.MarkoTagBody | t.Program>, type: BindingType, upstreamAlias?: Binding["upstreamAlias"]): Binding | undefined;
+export declare function trackParamsReferences(body: t.NodePath<t.MarkoTagBody | t.Program>, type: BindingType): Binding | undefined;
 export declare function trackHoistedReference(referencePath: t.NodePath<t.Identifier>, binding: Binding): void;
 export declare function isReferenceHoisted(bindingPath: t.NodePath, reference: t.NodePath): boolean;
 export declare function setReferencesScope(path: t.NodePath<any>): void;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "marko",
-  "version": "6.0.132",
+  "version": "6.0.134",
   "description": "Optimized runtime for Marko templates.",
   "keywords": [
     "api",
@@ -48,7 +48,7 @@
     "build": "node -r ~ts ./scripts/bundle.ts"
   },
   "dependencies": {
-    "@marko/compiler": "^5.39.46",
+    "@marko/compiler": "^5.39.48",
     "csstype": "^3.1.3",
     "magic-string": "^0.30.17"
   },