marko 6.1.11 → 6.1.13

package/dist/translator/index.js

@@ -113,8 +113,7 @@ let AccessorProp$1 = /* @__PURE__ */ function(AccessorProp) {
 	AccessorProp["CatchContent"] = "#CatchContent";
 	AccessorProp["ClosestBranch"] = "#ClosestBranch";
 	AccessorProp["ClosestBranchId"] = "#ClosestBranchId";
-	AccessorProp["Creating"] = "#Creating";
-	AccessorProp["Destroyed"] = "#Destroyed";
+	AccessorProp["Gen"] = "#Gen";
 	AccessorProp["DetachedAwait"] = "#DetachedAwait";
 	AccessorProp["EndNode"] = "#EndNode";
 	AccessorProp["Id"] = "#Id";
@@ -123,6 +122,7 @@ let AccessorProp$1 = /* @__PURE__ */ function(AccessorProp) {
 	AccessorProp["ParentBranch"] = "#ParentBranch";
 	AccessorProp["PendingEffects"] = "#PendingEffects";
 	AccessorProp["PendingRenders"] = "#PendingRenders";
+	AccessorProp["PendingScopes"] = "#PendingScopes";
 	AccessorProp["PlaceholderBranch"] = "#PlaceholderBranch";
 	AccessorProp["PlaceholderContent"] = "#PlaceholderContent";
 	AccessorProp["Renderer"] = "#Renderer";
@@ -182,15 +182,15 @@ function isNullableExpr(expr) {
 			case "**=": return false;
 			case "||=":
 			case "??=": return isNullableExpr(expr.right) || isNullableExpr(expr.left);
-			case "&&=": return isNullableExpr(expr.left) && isNullableExpr(expr.right);
+			case "&&=": return isNullableExpr(expr.left) || isNullableExpr(expr.right);
 			default: return true;
 		}
 		case "AwaitExpression": return isNullableExpr(expr.argument);
-		case "ConditionalExpression": return isNullableExpr(expr.consequent) && isNullableExpr(expr.alternate);
+		case "ConditionalExpression": return isNullableExpr(expr.consequent) || isNullableExpr(expr.alternate);
 		case "LogicalExpression": switch (expr.operator) {
 			case "||":
 			case "??": return isNullableExpr(expr.right) || isNullableExpr(expr.left);
-			case "&&": return isNullableExpr(expr.left) && isNullableExpr(expr.right);
+			case "&&": return isNullableExpr(expr.left) || isNullableExpr(expr.right);
 			default: return true;
 		}
 		case "ParenthesizedExpression": return isNullableExpr(expr.expression);
@@ -1052,8 +1052,7 @@ let AccessorProp = /* @__PURE__ */ function(AccessorProp) {
 	AccessorProp["CatchContent"] = "E";
 	AccessorProp["ClosestBranch"] = "F";
 	AccessorProp["ClosestBranchId"] = "G";
-	AccessorProp["Creating"] = "H";
-	AccessorProp["Destroyed"] = "I";
+	AccessorProp["Gen"] = "H";
 	AccessorProp["DetachedAwait"] = "V";
 	AccessorProp["EndNode"] = "K";
 	AccessorProp["Id"] = "L";
@@ -1062,6 +1061,7 @@ let AccessorProp = /* @__PURE__ */ function(AccessorProp) {
 	AccessorProp["ParentBranch"] = "N";
 	AccessorProp["PendingEffects"] = "J";
 	AccessorProp["PendingRenders"] = "W";
+	AccessorProp["PendingScopes"] = "Y";
 	AccessorProp["PlaceholderBranch"] = "P";
 	AccessorProp["PlaceholderContent"] = "Q";
 	AccessorProp["Renderer"] = "R";
@@ -2807,7 +2807,7 @@ function simplifyFunction(fn) {
 		case "FunctionDeclaration":
 		case "FunctionExpression":
 		case "ArrowFunctionExpression": return fn;
-		default: return _marko_compiler.types.functionExpression(null, fn.params, fn.body, fn.async, fn.generator);
+		default: return _marko_compiler.types.functionExpression(null, fn.params, fn.body, fn.generator, fn.async);
 	}
 }
 //#endregion
@@ -4181,10 +4181,14 @@ var native_tag_default = {
 				}
 				seen[attr.name] = attr;
 				if (injectNonce && attr.name === "nonce") injectNonce = false;
+				if (isEventOrChangeHandler(attr.name)) assertNativeHandlerAttr(tag, attr);
 				if (isEventHandler(attr.name)) {
 					valueExtra.isEffect = true;
 					hasEventHandlers = true;
-				} else if (!evaluate(attr.value).confident) hasDynamicAttributes = true;
+				} else {
+					assertValidNativeEventHandlerAttr(tag, attr);
+					if (!evaluate(attr.value).confident) hasDynamicAttributes = true;
+				}
 			} else if (_marko_compiler.types.isMarkoSpreadAttribute(attr)) {
 				valueExtra.isEffect = true;
 				hasEventHandlers = true;
@@ -4593,6 +4597,23 @@ function isInjectNonceTag(tagName) {
 		default: return false;
 	}
 }
+function assertNativeHandlerAttr(tag, attr) {
+	if ((0, _marko_compiler_babel_utils.computeNode)(attr.value)?.value) throw tag.hub.buildError(attr.value, `The \`${attr.name}\` ${isEventHandler(attr.name) ? "event handler" : "change handler"} on a [native tag](https://markojs.com/docs/reference/native-tag) must be a function or a falsey value (\`null\`, \`undefined\`, \`false\`, \`0\`, …).`, Error);
+}
+const lowercaseEventHandlerReg = /^on[a-z]/;
+function assertValidNativeEventHandlerAttr(tag, attr) {
+	if (!lowercaseEventHandlerReg.test(attr.name)) return;
+	const { value } = attr;
+	let invalid = _marko_compiler.types.isFunction(value);
+	if (!invalid) {
+		const { confident, computed } = evaluate(value);
+		invalid = confident && !!computed && typeof computed !== "string";
+	}
+	if (invalid) {
+		const suggestion = "on" + attr.name[2].toUpperCase() + attr.name.slice(3);
+		throw tag.hub.buildError(value, `The \`${attr.name}\` attribute on a [native tag](https://markojs.com/docs/reference/native-tag) must be a string or a falsey value (\`null\`, \`undefined\`, \`false\`, \`0\`, …). To attach an event listener, use the \`${suggestion}\` [event handler attribute](https://markojs.com/docs/reference/event-handling) instead.`, Error);
+	}
+}
 function getCanonicalTagName(tag) {
 	const tagName = getTagName(tag);
 	switch (tagName) {

package/package.json

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