marko 6.3.34 → 6.3.35

package/dist/translator/index.js

@@ -126,6 +126,7 @@ var accessor_prefix_debug_exports = /* @__PURE__ */ __exportAll({
 	ControlledValue: () => ControlledValue$1,
 	DynamicHTMLLastChild: () => DynamicHTMLLastChild$1,
 	EventAttributes: () => EventAttributes$1,
+	IdFallback: () => IdFallback$1,
 	KeyedScopes: () => KeyedScopes$1,
 	Lifecycle: () => Lifecycle$1,
 	Promise: () => Promise$2,
@@ -141,6 +142,7 @@ const ControlledType$1 = "ControlledType:";
 const ControlledValue$1 = "ControlledValue:";
 const DynamicHTMLLastChild$1 = "DynamicHTMLLastChild:";
 const EventAttributes$1 = "EventAttributes:";
+const IdFallback$1 = "IdFallback:";
 const KeyedScopes$1 = "KeyedScopes:";
 const Lifecycle$1 = "Lifecycle:";
 const Promise$2 = "Promise:";
@@ -203,6 +205,7 @@ const StartNode$1 = "#StartNode";
 const Subscriptions$1 = "#Subscriptions";
 const TagVariable$1 = "#TagVariable";
 const TagVariableChange$2 = "#TagVariableChange";
+const Owner$1 = "owner";
 //#endregion
 //#region src/translator/util/evaluate.ts
 function evaluate(value) {
@@ -933,6 +936,7 @@ var accessor_prefix_exports = /* @__PURE__ */ __exportAll({
 	ControlledValue: () => "G",
 	DynamicHTMLLastChild: () => "H",
 	EventAttributes: () => "I",
+	IdFallback: () => "J",
 	KeyedScopes: () => "O",
 	Lifecycle: () => "K",
 	Promise: () => "L",
@@ -1937,7 +1941,10 @@ function assertExclusiveAttrs(attrs, onError = throwErr) {
 			(exclusiveAttrs ||= []).push("checkedValueChange");
 			if ("checked" in attrs) exclusiveAttrs.push("checked");
 		}
-		if (attrs.valueChange) (exclusiveAttrs ||= []).push("valueChange");
+		if (attrs.valueChange) {
+			(exclusiveAttrs ||= []).push("valueChange");
+			if ("checked" in attrs && !exclusiveAttrs.includes("checked")) exclusiveAttrs.push("checked");
+		}
 		if (exclusiveAttrs && exclusiveAttrs.length > 1) onError(`The attributes ${joinWithAnd(exclusiveAttrs)} are mutually exclusive.`);
 	}
 }
@@ -1957,8 +1964,8 @@ function joinWithAnd(a) {
 function _unescaped(val) {
 	return val ? val + "" : val === 0 ? "0" : "";
 }
-const unsafeXMLReg = /[<&]/g;
-const replaceUnsafeXML = (c) => c === "&" ? "&amp;" : "&lt;";
+const unsafeXMLReg = /[<&\r]/g;
+const replaceUnsafeXML = (c) => c === "&" ? "&amp;" : c === "<" ? "&lt;" : "&#13;";
 const escapeXMLStr = (str) => unsafeXMLReg.test(str) ? str.replace(unsafeXMLReg, replaceUnsafeXML) : str;
 function _escape(val) {
 	return val ? escapeXMLStr(val + "") : val === 0 ? "0" : "";
@@ -1987,6 +1994,9 @@ function getContext(key) {
 function getState() {
 	return $chunk.boundary.state;
 }
+function rendererKey(renderer) {
+	return renderer?.["owner"] === void 0 ? renderer?.["id"] || renderer : renderer["id"] + " " + renderer[Owner$1];
+}
 function getScopeById(scopeId) {
 	if (scopeId !== void 0) return $chunk.boundary.state.scopes.get(scopeId);
 }
@@ -2081,7 +2091,7 @@ function _attr_textarea_value(scopeId, nodeAccessor, value, valueChange, seriali
 }
 function _textarea_value(value) {
 	const escaped = _escape(normalizeStrAttrValue(value));
-	return escaped[0] === "\n" || escaped[0] === "\r" ? "\n" + escaped : escaped;
+	return escaped[0] === "\n" ? "\n" + escaped : escaped;
 }
 function _attr_input_value(scopeId, nodeAccessor, value, valueChange, serializeType) {
 	if (valueChange) writeControlledScope(2, scopeId, nodeAccessor, void 0, valueChange, serializeType);
@@ -2276,7 +2286,7 @@ let _dynamic_tag = (scopeId, accessor, tag, inputOrArgs, content, inputIsArgs, s
 		}
 	}
 	if (rendered) {
-		if (shouldResume) writeScope(scopeId, { [ConditionalRenderer$1 + accessor]: renderer?.["id"] || renderer });
+		if (shouldResume) writeScope(scopeId, { [ConditionalRenderer$1 + accessor]: rendererKey(renderer) });
 	} else _scope_id();
 	return result;
 };
@@ -3012,10 +3022,25 @@ function setSectionSerializedValue(section, prop, expression) {
 }
 function setBindingSerializedValue(section, binding, expression, prefix) {
 	const reason = getSerializeReason(section, binding, prefix);
-	if (reason) getSerializedAccessors(section).set(prefix === void 0 ? getScopeAccessor(binding) : getPrefixedScopeAccessor(binding, prefix), {
+	if (reason) if (prefix === void 0) getSerializedAccessors(section).set(getScopeAccessor(binding), {
 		expression,
 		reason
 	});
+	else {
+		const accessor = getPrefixedScopeAccessor(binding, prefix);
+		getSerializedAccessors(section).set(accessor, {
+			expression,
+			reason
+		});
+		if (!isOptimize() && prefix === getAccessorPrefix().TagVariableChange) {
+			const { root, access } = getDebugScopeAccess(binding);
+			setSectionDebugVar(section, accessor, `${root.name + access}Change`, root.loc);
+		}
+	}
+}
+const [getSectionDebugVars] = createSectionState("sectionDebugVars", () => /* @__PURE__ */ new Map());
+function setSectionDebugVar(section, accessor, name, loc) {
+	if (!isOptimize()) getSectionDebugVars(section).set(accessor, loc ? [name, `${loc.start.line}:${loc.start.column + 1}`] : [name]);
 }
 const nonAnalyzedForceSerializedSection = /* @__PURE__ */ new WeakSet();
 function setSerializedValue(section, key, expression) {
@@ -3354,7 +3379,7 @@ function writeSignals(section) {
 		if (signal.build) {
 			let value = signal.build();
 			const buildsEagerForward = _marko_compiler.types.isIdentifier(value);
-			if (!value || !signal.register && _marko_compiler.types.isFunction(value) && _marko_compiler.types.isBlockStatement(value.body) && !value.body.body.length) return;
+			if (!value || !signal.register && !signal.referenced && _marko_compiler.types.isFunction(value) && _marko_compiler.types.isBlockStatement(value.body) && !value.body.body.length) return;
 			if (_marko_compiler.types.isCallExpression(value)) replaceNullishAndEmptyFunctionsWith0(value.arguments);
 			if (signal.register) value = callRuntime("_var_resume", _marko_compiler.types.stringLiteral(getResumeRegisterId(section, signal.referencedBindings, "var")), value);
 			if (buildsEagerForward) forEach(signal.forwards, writeSignal);
@@ -3401,7 +3426,8 @@ function writeRegisteredFns() {
 					params = [localsIdentifier];
 					if (registeredFn.referencedBindings || registeredFn.referencesScope) prologue = [_marko_compiler.types.variableDeclaration("const", [_marko_compiler.types.variableDeclarator(scopeIdentifier, _marko_compiler.types.memberExpression(localsIdentifier, _marko_compiler.types.identifier(getAccessorProp().Owner)))])];
 				} else params = [scopeIdentifier];
-				fn = _marko_compiler.types.functionDeclaration(_marko_compiler.types.identifier(registeredFn.id), params, _marko_compiler.types.blockStatement(toReturnedFunction(registeredFn.node, prologue)));
+				const body = toReturnedFunction(registeredFn.node, prologue);
+				fn = _marko_compiler.types.variableDeclaration("const", [_marko_compiler.types.variableDeclarator(_marko_compiler.types.identifier(registeredFn.id), _marko_compiler.types.arrowFunctionExpression(params, body.length === 1 && body[0].type === "ReturnStatement" ? body[0].argument : _marko_compiler.types.blockStatement(body)))]);
 			} else if (registeredFn.node.type === "FunctionDeclaration" && registeredFn.node.id?.name === registeredFn.id) fn = registeredFn.node;
 			else fn = _marko_compiler.types.functionDeclaration(_marko_compiler.types.identifier(registeredFn.id), registeredFn.node.params, registeredFn.node.body.type === "BlockStatement" ? registeredFn.node.body : _marko_compiler.types.blockStatement([_marko_compiler.types.returnStatement(registeredFn.node.body)]), registeredFn.node.generator, registeredFn.node.async);
 			statements.push(fn);
@@ -3507,6 +3533,7 @@ function writeHTMLResumeStatements(path) {
 		const writeScopeArgs = [scopeIdIdentifier, _marko_compiler.types.objectExpression(serializedProperties)];
 		if (debug) {
 			writeScopeArgs.push(_marko_compiler.types.stringLiteral(path.hub.file.opts.filenameRelative), section.loc && section.loc.start.line != null ? _marko_compiler.types.stringLiteral(`${section.loc.start.line}:${section.loc.start.column + 1}`) : _marko_compiler.types.numericLiteral(0));
+			for (const [accessor, varLoc] of getSectionDebugVars(section)) (debugVars ||= []).push(toObjectProperty(accessor, _marko_compiler.types.valueToNode(varLoc)));
 			if (debugVars) writeScopeArgs.push(_marko_compiler.types.objectExpression(debugVars));
 		}
 		body.push(_marko_compiler.types.expressionStatement(getExprIfSerialized(section, sectionSerializeReason, writeScopeBuilder ? writeScopeBuilder(callRuntime("_scope", ...writeScopeArgs)) : callRuntime("_scope", ...writeScopeArgs))));
@@ -3560,7 +3587,7 @@ function replaceAssignedNode(node) {
 		case "UpdateExpression": {
 			const { extra } = node.argument;
 			if (isAssignedBindingExtra(extra)) {
-				let builtAssignment = getBuildAssignment(extra)?.(extra.section, _marko_compiler.types.binaryExpression(node.operator === "++" ? "+" : "-", createScopeReadExpression(extra.assignment, extra.section), _marko_compiler.types.numericLiteral(1)));
+				let builtAssignment = getBuildAssignment(extra)?.(extra.section, _marko_compiler.types.binaryExpression(node.operator === "++" ? "+" : "-", _marko_compiler.types.unaryExpression("+", createScopeReadExpression(extra.assignment, extra.section)), _marko_compiler.types.numericLiteral(1)));
 				if (builtAssignment) {
 					if (!node.prefix) {
 						builtAssignment = _marko_compiler.types.binaryExpression(node.operator === "++" ? "-" : "+", builtAssignment, _marko_compiler.types.numericLiteral(1));
@@ -4753,7 +4780,7 @@ function buildContent(body) {
 		}
 		if (dynamicSerializeReason) body.node.body.unshift(getScopeReasonDeclaration(bodySection));
 		else body.node.body.unshift(_marko_compiler.types.expressionStatement(callRuntime("_scope_reason")));
-		return callRuntime(serialized ? "_content_resume" : "_content", _marko_compiler.types.stringLiteral(getResumeRegisterId(bodySection, "content")), _marko_compiler.types.arrowFunctionExpression(body.node.params, _marko_compiler.types.blockStatement(body.node.body)), serialized ? getScopeIdIdentifier(getSection(getAttributeTagParent(body.parentPath))) : void 0);
+		return callRuntime(serialized ? "_content_resume" : "_content", _marko_compiler.types.stringLiteral(getResumeRegisterId(bodySection, "content")), _marko_compiler.types.arrowFunctionExpression(body.node.params, _marko_compiler.types.blockStatement(body.node.body)), getScopeIdIdentifier(getSection(getAttributeTagParent(body.parentPath))));
 	} else {
 		if (isSectionRendererElided(bodySection)) return;
 		return _marko_compiler.types.callExpression(_marko_compiler.types.identifier(bodySection.name), bodySection.referencedLocalClosures ? [scopeIdentifier, _marko_compiler.types.objectExpression(toArray(bodySection.referencedLocalClosures, (ref) => {
@@ -4849,6 +4876,11 @@ var native_tag_default = {
 				else if (!_marko_compiler.types.isMarkoText(child)) throw tag.hub.buildError(child, `Only text is allowed inside a \`<${tagName}>\`.`);
 			}
 			relatedControllable ||= getRelatedControllable(tagName, seen);
+			const valueChangeEval = tagName === "input" && seen.valueChange ? evaluate(seen.valueChange.value) : void 0;
+			if (valueChangeEval && !(valueChangeEval.confident && valueChangeEval.computed == null) && getInputValueMode(seen.type) === "attribute") {
+				const type = evaluate(seen.type.value).computed;
+				throw tag.hub.buildError(seen.valueChange, `\`valueChange\` cannot be used on a \`type="${type}"\` \`<input>\` — user interaction can never change its \`value\`.` + (/^[cr]/i.test(type) ? " Bind `checked` or `checkedValue` instead." : ""));
+			}
 			if (relatedControllable && relatedControllable.attrs[1]) hasEventHandlers = true;
 			if (node.var || hasDynamicAttributes || hasEventHandlers || textPlaceholders || injectNonce || seen.content && tagName !== "meta" && !node.body.body.length || isDynamicControllable(relatedControllable)) {
 				const tagExtra = node.extra ??= {};
@@ -4952,6 +4984,20 @@ var native_tag_default = {
 				const usedAttrs = getUsedAttrs(tagName, tag.node);
 				const { staticAttrs, staticControllable, staticContentAttr, skipExpression, injectNonce } = usedAttrs;
 				let { spreadExpression } = usedAttrs;
+				if (!isOptimize() && nodeBinding) {
+					const changeAttr = staticControllable?.attrs[1];
+					if (changeAttr) {
+						const handler = evaluate(changeAttr.value);
+						if (!(handler.confident && handler.computed == null)) setSectionDebugVar(tagSection, getPrefixedScopeAccessor(nodeBinding, getAccessorPrefix().ControlledHandler), changeAttr.name, changeAttr.loc);
+					} else if (spreadExpression) {
+						const spreads = tag.node.attributes.filter((attr) => _marko_compiler.types.isMarkoSpreadAttribute(attr));
+						const spreadLoc = spreads.length === 1 && spreads[0].value.loc;
+						if (spreadLoc && spreadLoc.start.index != null) {
+							const name = "..." + tag.hub.file.code.slice(spreadLoc.start.index, spreadLoc.end.index);
+							for (const prefix of [getAccessorPrefix().ControlledHandler, getAccessorPrefix().EventAttributes]) if (prefix !== getAccessorPrefix().ControlledHandler || getSpreadControllableValueProps(tagName)) setSectionDebugVar(tagSection, getPrefixedScopeAccessor(nodeBinding, prefix), name, spreadLoc);
+						}
+					}
+				}
 				if (tagName === "select" && (staticControllable || spreadExpression)) flushBefore(tag);
 				write`<${tagName}`;
 				if (injectNonce) write`${callRuntime("_attr_nonce")}`;
@@ -5128,7 +5174,7 @@ var native_tag_default = {
 										stmt = _marko_compiler.types.expressionStatement(callRuntime(`_attr_${name}_items`, nodeExpr, _marko_compiler.types.objectExpression(props)));
 									}
 								}
-								if (stmt) addStatement("render", tagSection, valueReferences, stmt, !!meta.dynamicItems);
+								if (stmt) addStatement("render", tagSection, valueReferences, stmt, true);
 							}
 							break;
 						}
@@ -6019,7 +6065,7 @@ function getChangeHandler(tag, attr) {
 		if (!existingChangedAttr) {
 			const bindingIdentifierPath = binding.path.getOuterBindingIdentifierPaths()[binding.identifier.name];
 			let changeAttrExpr = bindingIdentifierPath ? bindingIdentifierPath.parentPath === binding.path ? buildChangeHandlerFunction(attr.value, modifier) : bindingIdentifierPath.parentPath.isObjectProperty() ? getChangeHandlerFromObjectPattern(bindingIdentifierPath.parentPath) : void 0 : void 0;
-			if (!changeAttrExpr) throw tag.hub.buildError(attr.value, bindingIdentifierPath?.parentPath?.isArrayPattern() ? `Cannot two-way bind to \`${attr.value.name}\` because it comes from array destructuring, which has no change handler. Use object destructuring or pass an explicit \`${changeAttrName}\` attribute.` : "Unable to bind to value.");
+			if (!changeAttrExpr) throw tag.hub.buildError(attr.value, bindingIdentifierPath?.parentPath?.isArrayPattern() ? `Cannot two-way bind to \`${attr.value.name}\` because it comes from array destructuring, which has no change handler. Use object destructuring or pass an explicit \`${changeAttrName}\` attribute.` : bindingIdentifierPath?.parentPath?.isObjectProperty({ computed: true }) ? `Cannot two-way bind to \`${attr.value.name}\` because it is destructured with a dynamically computed key, so its change handler cannot be derived. Use a static key or pass an explicit \`${changeAttrName}\` attribute.` : "Unable to bind to value.");
 			if (modifier && _marko_compiler.types.isIdentifier(changeAttrExpr)) changeAttrExpr = _marko_compiler.types.logicalExpression("&&", changeAttrExpr, buildModifierForwarder(_marko_compiler.types.cloneNode(changeAttrExpr), modifier));
 			const changeHandlerAttr = _marko_compiler.types.markoAttribute(changeAttrName, changeAttrExpr);
 			handlerByModifier.set(modifierKey, changeHandlerAttr);
@@ -6060,25 +6106,21 @@ function buildChangeHandlerFunction(id, modifier) {
 function getChangeHandlerFromObjectPattern(parent) {
 	let changeKey;
 	const pattern = parent.parentPath;
-	if (parent.node.computed) {
-		changeKey = generateUidIdentifier("dynamicChange");
-		pattern.pushContainer("properties", _marko_compiler.types.objectProperty(_marko_compiler.types.binaryExpression("+", parent.get("key").node, _marko_compiler.types.stringLiteral("Change")), changeKey, true));
-	} else {
-		const searchKey = `${getStringOrIdentifierValue(parent.get("key"))}Change`;
-		for (const prop of pattern.get("properties")) if (prop.isObjectProperty()) {
-			const propKey = prop.get("key");
-			const propValue = prop.get("value");
-			if (!prop.node.computed && getStringOrIdentifierValue(propKey) === searchKey && propValue.isIdentifier()) {
-				changeKey = propValue.node;
-				break;
-			}
+	const keyName = getStaticKeyName(parent.node);
+	if (keyName === void 0) return;
+	const searchKey = `${keyName}Change`;
+	for (const prop of pattern.get("properties")) if (prop.isObjectProperty()) {
+		const propValue = prop.get("value");
+		if (getStaticKeyName(prop.node) === searchKey && propValue.isIdentifier()) {
+			changeKey = propValue.node;
+			break;
 		}
-		if (!changeKey) pattern.unshiftContainer("properties", _marko_compiler.types.objectProperty(_marko_compiler.types.stringLiteral(searchKey), changeKey = generateUidIdentifier(searchKey)));
 	}
+	if (!changeKey) pattern.unshiftContainer("properties", _marko_compiler.types.objectProperty(_marko_compiler.types.stringLiteral(searchKey), changeKey = generateUidIdentifier(searchKey)));
 	return changeKey;
 }
-function getStringOrIdentifierValue(path) {
-	return getLiteralName(path.node);
+function getStaticKeyName(prop) {
+	return prop.computed && !_marko_compiler.types.isStringLiteral(prop.key) ? void 0 : getLiteralName(prop.key);
 }
 function getLiteralName(node) {
 	switch (node.type) {
@@ -6272,7 +6314,7 @@ function crawlSectionsAndSetBinding(tag, binding, properties, skip) {
 }
 //#endregion
 //#region src/translator/util/translate-var.ts
-function translateVar(tag, initialValue, kind = "const") {
+function translateVar(tag, initialValue, kind = "const", statements) {
 	const { node: { var: tagVar } } = tag;
 	if (!tagVar) return;
 	const tagSection = getOrCreateSection(tag);
@@ -6307,7 +6349,9 @@ function translateVar(tag, initialValue, kind = "const") {
 			restPath.remove();
 		}
 	});
-	tag.insertBefore(_marko_compiler.types.variableDeclaration(kind, [_marko_compiler.types.variableDeclarator(tagVar, initialValue)]));
+	const declaration = _marko_compiler.types.variableDeclaration(kind, [_marko_compiler.types.variableDeclarator(tagVar, initialValue)]);
+	if (statements) statements.push(declaration);
+	else tag.insertBefore(declaration);
 }
 function getDestructurePattern(id) {
 	let cur = id;
@@ -6362,13 +6406,13 @@ function knownTagTranslateHTML(tag, tagIdentifier, contentSection, propTree) {
 		statements: []
 	};
 	const childScopeBinding = tagExtra[kChildScopeBinding];
-	if (getSerializeReason(section, childScopeBinding)) {
+	const childScopeSerializeReason = getSerializeReason(section, childScopeBinding);
+	let varStatement;
+	if (childScopeSerializeReason) {
 		const peekScopeId = generateUidIdentifier(childScopeBinding?.name);
-		const peekScopeDeclaration = _marko_compiler.types.variableDeclaration("const", [_marko_compiler.types.variableDeclarator(peekScopeId, callRuntime("_peek_scope_id"))]);
-		if (tagVar) tag.insertBefore(peekScopeDeclaration);
-		else statements.push(peekScopeDeclaration);
+		statements.push(_marko_compiler.types.variableDeclaration("const", [_marko_compiler.types.variableDeclarator(peekScopeId, callRuntime("_peek_scope_id"))]));
 		setBindingSerializedValue(section, childScopeBinding, callRuntime("_existing_scope", peekScopeId));
-		if (tagVar) statements.push(_marko_compiler.types.expressionStatement(callRuntime("_var", getScopeIdIdentifier(section), getScopeAccessorLiteral(tag.node.extra[kChildOffsetScopeBinding$1]), peekScopeId, _marko_compiler.types.stringLiteral(getResumeRegisterId(section, tagVar.extra?.binding, "var")))));
+		if (tagVar) varStatement = _marko_compiler.types.expressionStatement(callRuntime("_var", getScopeIdIdentifier(section), getScopeAccessorLiteral(tag.node.extra[kChildOffsetScopeBinding$1]), peekScopeId, _marko_compiler.types.stringLiteral(getResumeRegisterId(section, tagVar.extra?.binding, "var"))));
 	}
 	if (contentSection.paramReasonGroups) {
 		let childSerializeReasonExpr;
@@ -6406,8 +6450,10 @@ function knownTagTranslateHTML(tag, tagIdentifier, contentSection, propTree) {
 		if (!tag.node.arguments?.length || properties.length) renderArgs.push(propsToExpression(properties));
 		return renderArgs;
 	};
-	if (tagVar) translateVar(tag, callExpression(tagIdentifier, ...getArgs()), "let");
-	else statements.push(callStatement(tagIdentifier, ...getArgs()));
+	if (tagVar) {
+		translateVar(tag, callExpression(tagIdentifier, ...getArgs()), "let", statements);
+		if (varStatement) statements.push(varStatement);
+	} else statements.push(callStatement(tagIdentifier, ...getArgs()));
 	for (const replacement of tag.replaceWithMultiple(statements)) replacement.skip();
 }
 function knownTagTranslateDOM(tag, propTree, getBindingIdentifier, callSetup) {
@@ -6417,7 +6463,8 @@ function knownTagTranslateDOM(tag, propTree, getBindingIdentifier, callSetup) {
 	if (node.var) {
 		const varBinding = node.var.extra.binding;
 		const source = initValue(varBinding);
-		source.register = !!getSerializeReason(tagSection, childScopeBinding) || !signalHasStatements(source);
+		source.register = !!getSerializeReason(tagSection, childScopeBinding);
+		source.referenced = true;
 		source.buildAssignment = (valueSection, value) => {
 			const changeArgs = [createScopeReadExpression(childScopeBinding, valueSection), value];
 			if (!isOptimize()) changeArgs.push(_marko_compiler.types.stringLiteral(varBinding.name));
@@ -6994,9 +7041,9 @@ function trackVarReferences(tag, type, upstreamAlias) {
 		const section = getOrCreateSection(tag);
 		let canonicalUpstreamAlias = upstreamAlias && getCanonicalBinding(upstreamAlias);
 		if (canonicalUpstreamAlias) {
-			const { excludeProperties } = canonicalUpstreamAlias;
+			const { excludeProperties, restOffset } = canonicalUpstreamAlias;
 			if (excludeProperties !== void 0) canonicalUpstreamAlias = canonicalUpstreamAlias.upstreamAlias;
-			createBindingsAndTrackReferences(tagVar, canonicalUpstreamAlias.type, tag.scope, section, canonicalUpstreamAlias, void 0, excludeProperties);
+			createBindingsAndTrackReferences(tagVar, canonicalUpstreamAlias.type, tag.scope, section, canonicalUpstreamAlias, void 0, excludeProperties, restOffset);
 			return canonicalUpstreamAlias;
 		}
 		createBindingsAndTrackReferences(tagVar, type, tag.scope, section, void 0, void 0, void 0);
@@ -7080,12 +7127,13 @@ function trackReferencesForBinding(babelBinding, binding) {
 				const left = ref.get("left");
 				if (left.isIdentifier()) trackReference(left, binding);
 			}
-		}
+		} else if (ref.isForXStatement()) throw ref.get("left").buildCodeFrameError(`\`${binding.name}\` is a [tag variable](https://markojs.com/docs/reference/language#tag-variables), so a \`for...${ref.isForOfStatement() ? "of" : "in"}\` cannot assign to it. Loop into a local variable and assign \`${binding.name}\` from it instead.`);
 	}
 }
 function trackAssignment(assignment, binding) {
 	const fnParent = getFnParent(assignment);
 	if (!fnParent) throw assignment.buildCodeFrameError(`\`${binding.name}\` is a tag ${binding.type === 3 ? "parameter" : "variable"} and can only be assigned within a script or function.`);
+	if (binding.type === 4) throw assignment.buildCodeFrameError(`\`${binding.name}\` is a tag parameter and cannot be assigned to.`);
 	const fnRoot = getFnRoot(fnParent);
 	const fnExtra = fnRoot && (fnRoot.node.extra ??= {});
 	const section = getOrCreateSection(assignment);
@@ -7145,14 +7193,14 @@ function createBindingsAndTrackReferences(lVal, type, scope, section, upstreamAl
 		}
 		case "ArrayPattern": {
 			const patternBinding = (property ? upstreamAlias.propertyAliases.get(property) : upstreamAlias) || ((lVal.extra ??= {}).binding = createBinding(generateUid(property || "pattern"), type, section, upstreamAlias, property, excludeProperties, lVal.loc));
-			let i = -1;
+			let index = (restOffset || 0) - 1;
 			for (const element of lVal.elements) {
-				i++;
+				index++;
 				if (element) {
 					if (element.type === "RestElement") {
-						excludeProperties = i > 0 ? addNumericPropertiesUntil(excludeProperties, i) : void 0;
-						createBindingsAndTrackReferences(element.argument, type, scope, section, patternBinding, void 0, excludeProperties, i);
-					} else if (_marko_compiler.types.isLVal(element)) createBindingsAndTrackReferences(element, type, scope, section, patternBinding, `${i}`, void 0);
+						excludeProperties = index > 0 ? addNumericPropertiesUntil(excludeProperties, index) : void 0;
+						createBindingsAndTrackReferences(element.argument, type, scope, section, patternBinding, void 0, excludeProperties, index);
+					} else if (_marko_compiler.types.isLVal(element)) createBindingsAndTrackReferences(element, type, scope, section, patternBinding, `${index}`, void 0);
 				}
 			}
 			break;
@@ -8419,8 +8467,8 @@ var define_default = {
 			if (isOutputHTML()) {
 				flushInto(tag);
 				writeHTMLResumeStatements(tag.get("body"));
+				translateVar(tag, propsToExpression(translatedAttrs.properties), "const", translatedAttrs.statements);
 				tag.insertBefore(translatedAttrs.statements);
-				translateVar(tag, propsToExpression(translatedAttrs.properties));
 			} else {
 				if (_marko_compiler.types.isIdentifier(node.var)) {
 					const babelBinding = tag.scope.getBinding(node.var.name);
@@ -8642,7 +8690,7 @@ var id_default = {
 			const source = initValue(node.var.extra.binding);
 			if (valueAttr) {
 				const { value } = valueAttr;
-				addValue(section, value.extra?.referencedBindings, source, _marko_compiler.types.logicalExpression("||", value, id));
+				addValue(section, value.extra?.referencedBindings, source, _marko_compiler.types.logicalExpression("||", value, evaluate(value).confident ? id : callRuntime("_id", scopeIdentifier, _marko_compiler.types.stringLiteral(getPrefixedScopeAccessor(node.var.extra.binding, getAccessorPrefix().IdFallback)))));
 			} else addValue(section, void 0, source, id);
 			tag.remove();
 		}
@@ -9114,6 +9162,7 @@ function checkStyleInterpolations(tag) {
 	let stringQuote = "";
 	let inComment = false;
 	let groupDepth = 0;
+	let urlDepth = 0;
 	let blockDepth = 0;
 	let valueColon = false;
 	let runPlaceholder;
@@ -9128,6 +9177,7 @@ function checkStyleInterpolations(tag) {
 		const child = body[i];
 		if (_marko_compiler.types.isMarkoPlaceholder(child)) {
 			if (stringQuote) throw tag.hub.buildError(child, styleStringMsg);
+			if (urlDepth) throw tag.hub.buildError(child, styleUrlMsg);
 			if (!runPlaceholder) {
 				runPlaceholder = child;
 				runAfterColon = valueColon;
@@ -9155,9 +9205,14 @@ function checkStyleInterpolations(tag) {
 				inComment = true;
 				j++;
 			} else if (c === "\"" || c === "'") stringQuote = c;
-			else if (c === "(" || c === "[") groupDepth++;
-			else if (c === ")" || c === "]") {
-				if (groupDepth) groupDepth--;
+			else if (c === "(" || c === "[") {
+				groupDepth++;
+				if (!urlDepth && cssUrlBefore.test(text.slice(0, j + 1)) && !cssQuotedArg.test(text.slice(j + 1))) urlDepth = groupDepth;
+			} else if (c === ")" || c === "]") {
+				if (groupDepth) {
+					if (urlDepth === groupDepth) urlDepth = 0;
+					groupDepth--;
+				}
 			} else if (!groupDepth) switch (c) {
 				case ":":
 					if (blockDepth) valueColon = true;
@@ -9182,10 +9237,13 @@ const styleInterpolationMsg = "A `${...}` interpolation in a [`<style>` tag](htt
 const styleSelectorMsg = `${styleInterpolationMsg} only resolves in a declaration value, not in a selector or at-rule prelude. For a native html [\`<style>\` tag](https://markojs.com/docs/reference/core-tag#style) use the \`html-style\` core tag instead.`;
 const stylePropertyMsg = `${styleInterpolationMsg} cannot be used as a property name. For a native html [\`<style>\` tag](https://markojs.com/docs/reference/core-tag#style) use the \`html-style\` core tag instead.`;
 const styleStringMsg = `${styleInterpolationMsg} is not substituted inside a quoted CSS string — the literal text \`var(--…)\` would be rendered instead of the value. For a native html [\`<style>\` tag](https://markojs.com/docs/reference/core-tag#style) use the \`html-style\` core tag instead.`;
+const styleUrlMsg = `${styleInterpolationMsg} is not substituted inside an unquoted \`url()\` — the raw url token consumes the \`var(--…)\` text literally, invalidating the declaration. Move the whole \`url(...)\` into the interpolated value. For a native html [\`<style>\` tag](https://markojs.com/docs/reference/core-tag#style) use the \`html-style\` core tag instead.`;
 const styleGluedMsg = `${styleInterpolationMsg} CSS does not re-tokenize, so a unit written directly after it (eg \`\${x}px\`) becomes the invalid \`var(--…)px\`. Move the unit into the interpolated value (so it resolves to eg \`"10px"\`) or use \`calc(var(--…) * 1px)\`.`;
 const styleGluedBeforeMsg = `${styleInterpolationMsg} CSS does not re-tokenize, so text written directly before it (eg \`10\${x}\`) merges with the \`var(--…)\` into a single invalid token. Add whitespace before the interpolation or move the text into the interpolated value.`;
 const cssGluedValue = /^(?:[%.\d]|(?:p[xtc]|in|[cm]m|q|r?em|ex|ch|r?lh|v[whib]|vmin|vmax|fr|deg|g?rad|turn|m?s|k?hz|dp(?:i|cm|px)|cq[whib]|cqmin|cqmax)(?![\w-]))/i;
 const cssGluedBefore = /[\w%]$/;
+const cssUrlBefore = /(?:^|[^\w-])url\($/i;
+const cssQuotedArg = /^\s*["']/;
 //#endregion
 //#region src/translator/core/style.ts
 const STYLE_EXT_REG = /^style((?:\.[a-zA-Z0-9$_-]+)+)?/;
@@ -9534,6 +9592,27 @@ function isBeforeHtmlOrHead(documentType) {
 	return false;
 }
 //#endregion
+//#region src/translator/visitors/export-declaration.ts
+var export_declaration_default = {
+	analyze(exportDecl) {
+		const { node } = exportDecl;
+		const { source } = node;
+		if (source) {
+			const tagImport = (0, _marko_compiler_babel_utils.resolveTagImport)(exportDecl, source.value);
+			if (tagImport) {
+				(node.extra ??= {}).tagImport = tagImport;
+				const tags = exportDecl.hub.file.metadata.marko.tags;
+				if (!tags.includes(tagImport)) tags.push(tagImport);
+			}
+		}
+	},
+	translate: { exit(exportDecl) {
+		const { node } = exportDecl;
+		const tagImport = node.extra?.tagImport;
+		if (tagImport) node.source.value = tagImport;
+	} }
+};
+//#endregion
 //#region src/translator/visitors/import-declaration.ts
 const triggerRegExp = /\s*([\w-]+)\s*([^?|]+?)?\s*(?:\?([^|]*?))?\s*(?:\||$)/g;
 const [getHtmlLoadWrapped] = createProgramState(() => /* @__PURE__ */ new Map());
@@ -10287,7 +10366,7 @@ var dynamic_tag_default = {
 				if (tag.node.var) {
 					const varBinding = tag.node.var.extra.binding;
 					tagVarSignal = initValue(varBinding);
-					tagVarSignal.register = true;
+					tagVarSignal.register = tagVarSignal.referenced = true;
 					tagVarSignal.buildAssignment = (valueSection, value) => {
 						const changeArgs = [_marko_compiler.types.memberExpression(getScopeExpression(tagVarSignal.section, valueSection), _marko_compiler.types.stringLiteral(getAccessorPrefix().BranchScopes + getScopeAccessor(nodeBinding)), true), value];
 						if (!isOptimize()) changeArgs.push(_marko_compiler.types.stringLiteral(varBinding.name));
@@ -10488,7 +10567,7 @@ function isTagsAPI(file = (0, _marko_compiler_babel_utils.getFile)()) {
 	let { featureType } = programExtra;
 	if (!featureType) {
 		const lookup = (0, _marko_compiler_babel_utils.getTaglibLookup)(file);
-		const tagsDir = getTagsDir(file.opts.filename);
+		const tagsDir = getTagsDir(lookup, file.opts.filename);
 		const state = {};
 		if (tagsDir && !lookup.manualTagsDirs?.has(tagsDir)) addFeature(state, Tags, "Template file within a tags directory", program);
 		scanBody(state, program.get("body"));
@@ -10496,24 +10575,10 @@ function isTagsAPI(file = (0, _marko_compiler_babel_utils.getFile)()) {
 	}
 	return featureType === Tags;
 }
-function getTagsDir(filename) {
-	const pathSeparator = /\/|\\/.exec(filename)?.[0];
-	if (pathSeparator) {
-		let previousIndex = filename.length - 1;
-		while (previousIndex > 0) {
-			const index = filename.lastIndexOf(pathSeparator, previousIndex);
-			switch (previousIndex - index) {
-				case 4:
-					if (filename.startsWith("tags", index + 1)) return filename.slice(0, index + 5);
-					break;
-				case 10:
-					if (filename.startsWith("components", index + 1)) return false;
-					break;
-			}
-			previousIndex = index - 1;
-		}
-	}
-	return false;
+function getTagsDir(lookup, filename) {
+	let nearest;
+	for (const dir of lookup.discoveryDirs || []) if (filename.startsWith(dir) && (filename[dir.length] === "/" || filename[dir.length] === "\\") && (!nearest || dir.length > nearest.length)) nearest = dir;
+	return !!nearest && /[/\\]tags$/.test(nearest) && nearest;
 }
 function scanBody(state, body) {
 	if (body?.length) for (const child of body) switch (child.type) {
@@ -10801,6 +10866,8 @@ const visitors = extractVisitors({
 	Function: function_default,
 	ReferencedIdentifier: referenced_identifier_default,
 	ImportDeclaration: import_declaration_default,
+	ExportNamedDeclaration: export_declaration_default,
+	ExportAllDeclaration: export_declaration_default,
 	MarkoDocumentType: document_type_default,
 	MarkoDeclaration: declaration_default,
 	MarkoCDATA: cdata_default,

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

@@ -9,6 +9,7 @@ export interface Signal {
     section: Section;
     build: undefined | (() => t.Expression | undefined);
     register?: boolean;
+    referenced?: boolean;
     values: Array<{
         signal: Signal;
         value: t.Expression;
@@ -37,6 +38,7 @@ export declare const getTryHasPlaceholder: (section: Section) => true | undefine
 export declare function setSectionOwnerResumedByMarker(section: Section): void;
 export declare function setSectionSerializedValue(section: Section, prop: AccessorProp, expression: t.Expression): void;
 export declare function setBindingSerializedValue(section: Section, binding: Binding, expression: t.Expression, prefix?: AccessorPrefix): void;
+export declare function setSectionDebugVar(section: Section, accessor: string, name: string, loc: t.SourceLocation | null | undefined): void;
 export declare function setSerializedValue(section: Section, key: string, expression: t.Expression): void;
 export declare function addWriteScopeBuilder(section: Section, builder: (writeCall: t.Expression) => t.Expression): void;
 export declare const getHTMLSectionStatements: (section: Section) => t.Statement[];

package/dist/translator/util/translate-var.d.ts

@@ -1,2 +1,2 @@
 import { types as t } from "@marko/compiler";
-export default function translateVar(tag: t.NodePath<t.MarkoTag>, initialValue: t.Expression, kind?: "let" | "const"): void;
+export default function translateVar(tag: t.NodePath<t.MarkoTag>, initialValue: t.Expression, kind?: "let" | "const", statements?: t.Statement[]): void;

package/dist/translator/visitors/export-declaration.d.ts

@@ -0,0 +1,8 @@
+import { types as t } from "@marko/compiler";
+declare const _default: {
+    analyze(this: unknown, exportDecl: t.NodePath<t.ExportAllDeclaration | t.ExportNamedDeclaration>): void;
+    translate: {
+        exit(this: unknown, exportDecl: t.NodePath<t.ExportAllDeclaration | t.ExportNamedDeclaration>): void;
+    };
+};
+export default _default;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "marko",
-  "version": "6.3.34",
+  "version": "6.3.35",
   "description": "Optimized runtime for Marko templates.",
   "keywords": [
     "api",
@@ -48,14 +48,14 @@
     }
   },
   "dependencies": {
-    "@marko/compiler": "^5.41.18",
+    "@marko/compiler": "^5.42.0",
     "csstype": "^3.2.3",
     "fastest-levenshtein": "^1.0.16",
     "magic-string": "^0.30.21"
   },
   "devDependencies": {
-    "@marko/runtime-tags": "npm:marko@6.3.34",
-    "marko": "5.39.32"
+    "@marko/runtime-tags": "npm:marko@6.3.35",
+    "marko": "5.39.33"
   },
   "engines": {
     "node": ">=22"

package/tags/let.d.marko

@@ -1,8 +1,7 @@
 /** File for types only, not actual implementation **/
 
-export interface Input<T, K = T> {
-  value?: T;
-  valueChange?: ((newValue: K) => void) | false | null;
-}
+export type Input<T = undefined, K = T> =
+  | { value: T; valueChange?: ((newValue: K) => void) | false | null }
+  | { value?: undefined; valueChange?: ((newValue: K) => void) | false | null };
 
 return=(input.value as T) valueChange=(input.valueChange as (newValue: K) => void)