marko 6.0.63 → 6.0.64

package/dist/debug/dom.js

@@ -1059,9 +1059,16 @@ function _var(scope, childAccessor, signal) {
 }
 var _return = (scope, value) => scope["#TagVariable" /* TagVariable */]?.(value);
 function _return_change(scope, changeHandler) {
-  scope["#TagVariableChange" /* TagVariableChange */] = changeHandler;
+  if (changeHandler) {
+    scope["#TagVariableChange" /* TagVariableChange */] = changeHandler;
+  }
 }
-var _var_change = (scope, value) => scope["#TagVariableChange" /* TagVariableChange */]?.(value);
+var _var_change = true ? (scope, value, name = "This") => {
+  if (typeof scope["#TagVariableChange" /* TagVariableChange */] !== "function") {
+    throw new TypeError(`${name} is a readonly tag variable.`);
+  }
+  scope["#TagVariableChange" /* TagVariableChange */](value);
+} : (scope, value) => scope["#TagVariableChange" /* TagVariableChange */]?.(value);
 var tagIdsByGlobal = /* @__PURE__ */ new WeakMap();
 function _id({ $global }) {
   const id = tagIdsByGlobal.get($global) || 0;

package/dist/debug/dom.mjs

@@ -962,9 +962,16 @@ function _var(scope, childAccessor, signal) {
 }
 var _return = (scope, value) => scope["#TagVariable" /* TagVariable */]?.(value);
 function _return_change(scope, changeHandler) {
-  scope["#TagVariableChange" /* TagVariableChange */] = changeHandler;
+  if (changeHandler) {
+    scope["#TagVariableChange" /* TagVariableChange */] = changeHandler;
+  }
 }
-var _var_change = (scope, value) => scope["#TagVariableChange" /* TagVariableChange */]?.(value);
+var _var_change = true ? (scope, value, name = "This") => {
+  if (typeof scope["#TagVariableChange" /* TagVariableChange */] !== "function") {
+    throw new TypeError(`${name} is a readonly tag variable.`);
+  }
+  scope["#TagVariableChange" /* TagVariableChange */](value);
+} : (scope, value) => scope["#TagVariableChange" /* TagVariableChange */]?.(value);
 var tagIdsByGlobal = /* @__PURE__ */ new WeakMap();
 function _id({ $global }) {
   const id = tagIdsByGlobal.get($global) || 0;

package/dist/dom/signals.d.ts

@@ -17,8 +17,8 @@ export declare function _closure_get<T>(valueAccessor: Accessor, fn: Signal<T>,
 };
 export declare function _var(scope: Scope, childAccessor: Accessor, signal: Signal<unknown>): void;
 export declare const _return: (scope: Scope, value: unknown) => any;
-export declare function _return_change(scope: Scope, changeHandler: (value: unknown) => void): void;
-export declare const _var_change: (scope: Scope, value: unknown) => any;
+export declare function _return_change(scope: Scope, changeHandler?: ((value: unknown) => void) | null | false): void;
+export declare const _var_change: (scope: Scope, value: unknown, name?: string) => void;
 export declare function _id({ $global }: Scope): string;
 export declare function _script(id: string, fn: (scope: Scope) => void): (scope: Scope) => void;
 export declare function _hoist(...path: Accessor[]): (scope: Scope) => (...args: unknown[]) => any;

package/dist/dom.js

@@ -684,7 +684,7 @@ function _var(scope, childAccessor, signal) {
 }
 var _return = (scope, value) => scope.e?.(value);
 function _return_change(scope, changeHandler) {
-  scope.f = changeHandler;
+  changeHandler && (scope.f = changeHandler);
 }
 var _var_change = (scope, value) => scope.f?.(value), tagIdsByGlobal = /* @__PURE__ */ new WeakMap();
 function _id({ $global }) {

package/dist/dom.mjs

@@ -590,7 +590,7 @@ function _var(scope, childAccessor, signal) {
 }
 var _return = (scope, value) => scope.e?.(value);
 function _return_change(scope, changeHandler) {
-  scope.f = changeHandler;
+  changeHandler && (scope.f = changeHandler);
 }
 var _var_change = (scope, value) => scope.f?.(value), tagIdsByGlobal = /* @__PURE__ */ new WeakMap();
 function _id({ $global }) {

package/dist/translator/index.js

@@ -3135,7 +3135,11 @@ var return_default = {
           setSectionSerializedValue(
             section,
             getAccessorProp().TagVariableChange,
-            attrs.valueChange
+            import_compiler18.types.logicalExpression(
+              "||",
+              attrs.valueChange,
+              import_compiler18.types.unaryExpression("void", import_compiler18.types.numericLiteral(0))
+            )
           );
         }
         if (attrs.value) {
@@ -9570,7 +9574,7 @@ var let_default = {
         "The [`<let>` tag](https://next.markojs.com/docs/reference/core-tag#let) variable cannot be destructured."
       );
     }
-    if (valueChangeAttr && (0, import_babel_utils38.computeNode)(valueChangeAttr.value)) {
+    if (valueChangeAttr && (0, import_babel_utils38.computeNode)(valueChangeAttr.value)?.value) {
       throw tag.get("attributes").find((attr) => attr.node === valueChangeAttr).get("value").buildCodeFrameError(
         "The [`<let>` tag](https://next.markojs.com/docs/reference/core-tag#let) [`valueChange=` attribute](https://next.markojs.com/docs/reference/core-tag#controllable-let) must be a function."
       );
@@ -9621,7 +9625,11 @@ var let_default = {
           setBindingSerializedValue(
             section,
             binding,
-            valueChangeAttr.value,
+            import_compiler44.types.logicalExpression(
+              "||",
+              valueChangeAttr.value,
+              import_compiler44.types.unaryExpression("void", import_compiler44.types.numericLiteral(0))
+            ),
             getAccessorPrefix().TagVariableChange
           );
         }
@@ -10734,6 +10742,7 @@ var custom_tag_default = {
         throw tag.get("name").buildCodeFrameError("Unable to resolve file for tag.");
       }
       const varBinding = trackVarReferences(tag, 5 /* derived */);
+      const mutatesTagVar = !!(tag.node.var?.type === "Identifier" && tag.scope.getBinding(tag.node.var.name)?.constantViolations.length);
       if (varBinding) {
         varBinding.scopeOffset = tagExtra[kChildOffsetScopeBinding] = createBinding("#scopeOffset", 0 /* dom */, section);
       }
@@ -10745,7 +10754,7 @@ var custom_tag_default = {
           addBindingSerializeReasonExpr(
             section,
             childScopeBinding,
-            varSerializeReason
+            mutatesTagVar || varSerializeReason
           );
         }
       } else {
@@ -10771,7 +10780,7 @@ var custom_tag_default = {
           addBindingSerializeReasonExpr(
             section,
             childScopeBinding,
-            varSerializeReason
+            mutatesTagVar || varSerializeReason
           );
         }
         if (childExtra.inputSerializeReasons) {
@@ -10987,16 +10996,21 @@ function translateDOM(tag) {
   );
   const inputExport = childExports.input;
   if (node.var) {
+    const varBinding = node.var.extra.binding;
     const source = initValue(
       // TODO: support destructuring
-      node.var.extra.binding
+      varBinding
     );
     source.register = true;
     source.buildAssignment = (valueSection, value) => {
-      return import_compiler56.types.callExpression(importRuntime("_var_change"), [
+      const changeArgs = [
         createScopeReadExpression(valueSection, childScopeBinding),
         value
-      ]);
+      ];
+      if (!isOptimize()) {
+        changeArgs.push(import_compiler56.types.stringLiteral(varBinding.name));
+      }
+      return import_compiler56.types.callExpression(importRuntime("_var_change"), changeArgs);
     };
     addStatement(
       "render",
@@ -11741,13 +11755,14 @@ var dynamic_tag_default = {
         const signal = getSignal(section, nodeBinding, "dynamicTag");
         let tagVarSignal;
         if (tag.node.var) {
+          const varBinding = tag.node.var.extra.binding;
           tagVarSignal = initValue(
             // TODO: support destructuring
-            tag.node.var.extra.binding
+            varBinding
           );
           tagVarSignal.register = true;
           tagVarSignal.buildAssignment = (valueSection, value) => {
-            return import_compiler57.types.callExpression(importRuntime("_var_change"), [
+            const changeArgs = [
               import_compiler57.types.memberExpression(
                 getScopeExpression(tagVarSignal.section, valueSection),
                 import_compiler57.types.stringLiteral(
@@ -11756,7 +11771,11 @@ var dynamic_tag_default = {
                 true
               ),
               value
-            ]);
+            ];
+            if (!isOptimize()) {
+              changeArgs.push(import_compiler57.types.stringLiteral(varBinding.name));
+            }
+            return import_compiler57.types.callExpression(importRuntime("_var_change"), changeArgs);
           };
         }
         signal.build = () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "marko",
-  "version": "6.0.63",
+  "version": "6.0.64",
   "description": "Optimized runtime for Marko templates.",
   "keywords": [
     "api",

package/tags/let.d.marko

@@ -2,7 +2,7 @@
 
 export interface Input<T> {
   value: T;
-  valueChange?: (newValue: T) => void;
+  valueChange?: ((newValue: T) => void) | false | null;
 }
 
 return=input.value valueChange=input.valueChange!