marko 6.3.23 → 6.3.25

package/dist/html.js

@@ -262,7 +262,7 @@ let empty = [], rest = Symbol(), unsafeStyleAttrReg = /[\\;]/g, replaceUnsafeSty
 				_html(`</${renderer}>`);
 			}
 			let childScope = getScopeById(branchId), needsScript = childScope && (childScope.Ia || childScope.Ea);
-			needsScript && (writeScope(branchId, { R: renderer }), _script(branchId, "d")), (shouldResume || needsScript) && _html(state.mark("'", scopeId + " " + accessor + " " + branchId));
+			needsScript && _script(branchId, "d"), (shouldResume || needsScript) && _html(state.mark("'", scopeId + " " + accessor + " " + branchId));
 		})();
 	} else {
 		shouldResume && _html(state.mark("[", ""));
@@ -516,7 +516,7 @@ function writeScopesRoot(state, flushes) {
 	let { buf } = state, nextSlotId = -1, fillIndex = -1;
 	for (let flush of flushes) {
 		let scopeId = flush[0], scope = flush[1], ref = state.refs.get(scope) || newScopeReference(state, scope, scopeId), openIndex = buf.push("") - 1;
-		writeObjectProps(state, flush[2], ref) ? (buf[openIndex] = nextSlotId === -1 ? "[" + scopeId + ",{" : (scopeId > nextSlotId ? "," + (scopeId - nextSlotId) : "") + ",{", fillIndex === -1 && (fillIndex = openIndex), nextSlotId = scopeId + 1, buf.push("}")) : buf.pop();
+		writeObjectProps(state, flush[2], ref) ? (buf[openIndex] = nextSlotId === -1 ? "[" + scopeId + ",{" : (scopeId === nextSlotId ? "" : "," + (scopeId - nextSlotId)) + ",{", fillIndex === -1 && (fillIndex = openIndex), nextSlotId = scopeId + 1, buf.push("}")) : buf.pop();
 	}
 	nextSlotId !== -1 && buf.push("]");
 	let extras = "";

package/dist/html.mjs

@@ -262,7 +262,7 @@ let empty = [], rest = Symbol(), unsafeStyleAttrReg = /[\\;]/g, replaceUnsafeSty
 				_html(`</${renderer}>`);
 			}
 			let childScope = getScopeById(branchId), needsScript = childScope && (childScope.Ia || childScope.Ea);
-			needsScript && (writeScope(branchId, { R: renderer }), _script(branchId, "d")), (shouldResume || needsScript) && _html(state.mark("'", scopeId + " " + accessor + " " + branchId));
+			needsScript && _script(branchId, "d"), (shouldResume || needsScript) && _html(state.mark("'", scopeId + " " + accessor + " " + branchId));
 		})();
 	} else {
 		shouldResume && _html(state.mark("[", ""));
@@ -515,7 +515,7 @@ function writeScopesRoot(state, flushes) {
 	let { buf } = state, nextSlotId = -1, fillIndex = -1;
 	for (let flush of flushes) {
 		let scopeId = flush[0], scope = flush[1], ref = state.refs.get(scope) || newScopeReference(state, scope, scopeId), openIndex = buf.push("") - 1;
-		writeObjectProps(state, flush[2], ref) ? (buf[openIndex] = nextSlotId === -1 ? "[" + scopeId + ",{" : (scopeId > nextSlotId ? "," + (scopeId - nextSlotId) : "") + ",{", fillIndex === -1 && (fillIndex = openIndex), nextSlotId = scopeId + 1, buf.push("}")) : buf.pop();
+		writeObjectProps(state, flush[2], ref) ? (buf[openIndex] = nextSlotId === -1 ? "[" + scopeId + ",{" : (scopeId === nextSlotId ? "" : "," + (scopeId - nextSlotId)) + ",{", fillIndex === -1 && (fillIndex = openIndex), nextSlotId = scopeId + 1, buf.push("}")) : buf.pop();
 	}
 	nextSlotId !== -1 && buf.push("]");
 	let extras = "";

package/dist/translator/index.js

@@ -110,6 +110,7 @@ var accessor_prefix_debug_exports = /* @__PURE__ */ __exportAll({
 	ClosureSignalIndex: () => ClosureSignalIndex$1,
 	ConditionalRenderer: () => ConditionalRenderer$1,
 	ControlledHandler: () => ControlledHandler$1,
+	ControlledObserver: () => ControlledObserver$1,
 	ControlledType: () => ControlledType$1,
 	ControlledValue: () => ControlledValue$1,
 	DynamicHTMLLastChild: () => DynamicHTMLLastChild$1,
@@ -123,6 +124,7 @@ const BranchScopes$3 = "BranchScopes:";
 const ClosureScopes$1 = "ClosureScopes:";
 const ClosureSignalIndex$1 = "ClosureSignalIndex:";
 const ConditionalRenderer$1 = "ConditionalRenderer:";
+const ControlledObserver$1 = "ControlledObserver:";
 const ControlledHandler$1 = "ControlledHandler:";
 const ControlledType$1 = "ControlledType:";
 const ControlledValue$1 = "ControlledValue:";
@@ -238,7 +240,7 @@ function isNullableExpr(expr) {
 			case "|=":
 			case "**=": return false;
 			case "||=":
-			case "??=": return isNullableExpr(expr.right) || isNullableExpr(expr.left);
+			case "??=": return isNullableExpr(expr.right);
 			case "&&=": return isNullableExpr(expr.left) || isNullableExpr(expr.right);
 			default: return true;
 		}
@@ -246,7 +248,7 @@ function isNullableExpr(expr) {
 		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.right);
 			case "&&": return isNullableExpr(expr.left) || isNullableExpr(expr.right);
 			default: return true;
 		}
@@ -908,6 +910,7 @@ var accessor_prefix_exports = /* @__PURE__ */ __exportAll({
 	ClosureSignalIndex: () => "C",
 	ConditionalRenderer: () => "D",
 	ControlledHandler: () => "E",
+	ControlledObserver: () => "N",
 	ControlledType: () => "F",
 	ControlledValue: () => "G",
 	DynamicHTMLLastChild: () => "H",
@@ -2071,10 +2074,7 @@ let _dynamic_tag = (scopeId, accessor, tag, inputOrArgs, content, inputIsArgs, s
 			}
 			const childScope = getScopeById(branchId);
 			const needsScript = childScope && (childScope["EventAttributes:a"] || childScope["ControlledHandler:a"]);
-			if (needsScript) {
-				writeScope(branchId, { [Renderer$1]: renderer });
-				_script(branchId, "d");
-			}
+			if (needsScript) _script(branchId, "d");
 			if (shouldResume || needsScript) _html(state.mark("'", scopeId + " " + accessor + " " + branchId));
 		};
 		renderNative();
@@ -2527,7 +2527,7 @@ function visitInternal(section) {
 }
 function nCodeString(code, number) {
 	switch (code) {
-		case 67: return toCharString(number, code, 20);
+		case 67: return toCharString(number, code, 25);
 		case 97: return toCharString(number, code, 10);
 		case 107: return toCharString(number, code, 10);
 		default: throw new Error(`Unexpected walk code: ${code}`);
@@ -2543,6 +2543,16 @@ function toCharString(number, startCode, rangeSize) {
 	result += String.fromCharCode(startCode + number);
 	return result;
 }
+/** `_content` strips trailing exit codes before it walks, so a renderer that
+* reaches the runtime through it never needs them on the wire. */
+function trimTrailingExits(walks) {
+	if (!_marko_compiler.types.isStringLiteral(walks)) return walks;
+	const value = walks.value.replace(/[^\0-1]+$/, "");
+	return value === walks.value ? walks : value ? withLeadingComment(_marko_compiler.types.stringLiteral(value), getComment(walks)) : void 0;
+}
+function getComment(node) {
+	return node.leadingComments?.[0]?.value.trim() || "";
+}
 function getWalkString(section) {
 	if (section.content?.endType === 1) getSteps(section).push(0, 1);
 	visitInternal(section);
@@ -3615,7 +3625,8 @@ var dom_default = { translate: {
 			if (childSection !== section) {
 				const tagParamsSignal = childSection.params && initValue(childSection.params);
 				const tagParamsIdentifier = tagParamsSignal && signalHasStatements(tagParamsSignal) ? tagParamsSignal.identifier : void 0;
-				const { walks, writes, decls } = getSectionMeta(childSection);
+				const { writes, decls } = getSectionMeta(childSection);
+				const walks = trimTrailingExits(getSectionMeta(childSection).walks);
 				const setup = getSetup(childSection);
 				const written = writeSignals(childSection);
 				const setupIdentifier = setup && written.has(setup) ? setup.identifier : void 0;
@@ -4748,8 +4759,12 @@ var native_tag_default = {
 				}
 				if (spreadExpression) {
 					const canHaveAttrContent = !(isTextOnly || isOpenOnly || hasChildren || staticContentAttr);
-					if (skipExpression) addStatement("render", tagSection, tagExtra.referencedBindings, _marko_compiler.types.expressionStatement(callRuntime(canHaveAttrContent ? "_attrs_partial_content" : "_attrs_partial", scopeIdentifier, visitAccessor, spreadExpression, skipExpression)));
-					else addStatement("render", tagSection, tagExtra.referencedBindings, _marko_compiler.types.expressionStatement(callRuntime(canHaveAttrContent ? "_attrs_content" : "_attrs", scopeIdentifier, visitAccessor, spreadExpression)));
+					const name = tag.get("name");
+					const staticName = name.isStringLiteral() ? name.node.value : void 0;
+					const controllable = controllableClaimFor(staticName);
+					if (skipExpression) addStatement("render", tagSection, tagExtra.referencedBindings, _marko_compiler.types.expressionStatement(callRuntime(canHaveAttrContent ? "_attrs_partial_content" : "_attrs_partial", scopeIdentifier, visitAccessor, spreadExpression, skipExpression, controllable && importRuntime(controllable))));
+					else addStatement("render", tagSection, tagExtra.referencedBindings, _marko_compiler.types.expressionStatement(callRuntime(canHaveAttrContent ? "_attrs_content" : "_attrs", scopeIdentifier, visitAccessor, spreadExpression, controllable && importRuntime(controllable))));
+					enableControllable(controllableScriptLatchFor(staticName));
 					addStatement("effect", tagSection, tagExtra.referencedBindings, _marko_compiler.types.expressionStatement(callRuntime("_attrs_script", scopeIdentifier, visitAccessor)), false);
 				}
 				if (staticContentAttr) addStatement("render", tagSection, staticContentAttr.value.extra?.referencedBindings, _marko_compiler.types.expressionStatement(callRuntime("_attr_content", scopeIdentifier, visitAccessor, staticContentAttr.value)), void 0, true);
@@ -5159,6 +5174,40 @@ function isDynamicControllable(controllable) {
 function buildUndefined$1() {
 	return _marko_compiler.types.unaryExpression("void", _marko_compiler.types.numericLiteral(0));
 }
+/** How a statically named tag claims its controlled attrs during render;
+* nothing for the many tags that control nothing. */
+function controllableClaimFor(tagName) {
+	switch (tagName) {
+		case "input": return "_controllable_input";
+		case "textarea": return "_controllable_textarea";
+		case "select": return "_controllable_select";
+		case "details":
+		case "dialog": return "_controllable_open";
+	}
+}
+/** The resume-pass latch for a tag: `_attrs_script` resolves the control kind
+* at run time, and a run-time tag name can be any of them. */
+function controllableScriptLatchFor(tagName) {
+	switch (tagName) {
+		case "input": return "_enable_controllable_input";
+		case "textarea": return "_enable_controllable_textarea";
+		case "select": return "_enable_controllable_select";
+		case "details":
+		case "dialog": return "_enable_controllable_open";
+		case void 0: return "_enable_controllable";
+	}
+}
+const enabledControllables = /* @__PURE__ */ new WeakMap();
+/** Emits the latch once per program, like `_enable_catch`. */
+function enableControllable(helper) {
+	if (!helper) return;
+	const program = (0, _marko_compiler_babel_utils.getProgram)().node;
+	let enabled = enabledControllables.get(program);
+	if (!enabled) enabledControllables.set(program, enabled = /* @__PURE__ */ new Set());
+	if (enabled.has(helper)) return;
+	enabled.add(helper);
+	program.body.push(_marko_compiler.types.expressionStatement(callRuntime(helper)));
+}
 //#endregion
 //#region src/translator/util/is-only-child-in-parent.ts
 const kOnlyChildInParent = Symbol("only child in parent");
@@ -9781,13 +9830,25 @@ var dynamic_tag_default = {
 					const argsOrInput = hasTagArgs ? _marko_compiler.types.arrayExpression(args) : args[0];
 					if (!_marko_compiler.types.isObjectExpression(argsOrInput) || argsOrInput.properties.length) signal.extraArgs = [_marko_compiler.types.arrowFunctionExpression([], statements.length ? _marko_compiler.types.blockStatement(statements.concat(_marko_compiler.types.returnStatement(argsOrInput))) : argsOrInput)];
 				}
-				if (!isClassAPI) enableDynamicTagResume(tag);
+				if (!isClassAPI) {
+					enableDynamicTagResume(tag);
+					enableDynamicTagControllables(tag);
+				}
 				addValue(section, tagExtra.referencedBindings, signal, tagExpression);
 				tag.remove();
 			}
 		}
 	}
 };
+const controlledAttrs = /^(?:value|checked(?:Value)?|open)(?:Change)?$/;
+/** The name resolves at run time, so any control kind is possible. */
+function enableDynamicTagControllables(tag) {
+	if (analyzeTagNameType(tag, true) === 1) return;
+	for (const attr of tag.node.attributes) if (attr.type === "MarkoSpreadAttribute" || attr.type === "MarkoAttribute" && controlledAttrs.test(attr.name)) {
+		enableControllable(controllableScriptLatchFor(void 0));
+		return;
+	}
+}
 function enableDynamicTagResume(tag) {
 	const program = (0, _marko_compiler_babel_utils.getProgram)().node;
 	if (!importedDynamicTagResume.has(program) && analyzeTagNameType(tag, true) !== 1) {

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

@@ -11,4 +11,7 @@ export declare function exit(path: t.NodePath<any>): void;
 export declare function enterShallow(path: t.NodePath<any>): void;
 export declare function injectWalks(tag: t.NodePath<t.MarkoTag>, name: string, expr?: Walk): void;
 export declare function visit(path: t.NodePath<t.MarkoTag | t.MarkoPlaceholder | t.Program>, code: VisitCodes): void;
+/** `_content` strips trailing exit codes before it walks, so a renderer that
+ * reaches the runtime through it never needs them on the wire. */
+export declare function trimTrailingExits(walks: t.Expression | undefined): t.ArrayExpression | t.ArrowFunctionExpression | t.AssignmentExpression | t.AwaitExpression | t.BigIntLiteral | t.BinaryExpression | t.BindExpression | t.BooleanLiteral | t.CallExpression | t.ClassExpression | t.ConditionalExpression | t.DecimalLiteral | t.DoExpression | t.FunctionExpression | t.Identifier | t.Import | t.ImportExpression | t.JSXElement | t.JSXFragment | t.LogicalExpression | t.MarkoParseError | t.MemberExpression | t.MetaProperty | t.ModuleExpression | t.NewExpression | t.NullLiteral | t.NumericLiteral | t.ObjectExpression | t.OptionalCallExpression | t.OptionalMemberExpression | t.ParenthesizedExpression | t.PipelineBareFunction | t.PipelinePrimaryTopicReference | t.PipelineTopicExpression | t.RecordExpression | t.RegExpLiteral | t.SequenceExpression | t.StringLiteral | t.Super | t.TSAsExpression | t.TSInstantiationExpression | t.TSNonNullExpression | t.TSSatisfiesExpression | t.TSTypeAssertion | t.TaggedTemplateExpression | t.TemplateLiteral | t.ThisExpression | t.TopicReference | t.TupleExpression | t.TypeCastExpression | t.UnaryExpression | t.UpdateExpression | t.YieldExpression | undefined;
 export declare function getWalkString(section: Section): t.Expression | undefined;

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

@@ -1,5 +1,6 @@
 import { types as t } from "@marko/compiler";
 import { type Binding } from "../../util/references";
+import { type DOMRuntimeHelpers } from "../../util/runtime";
 export declare const kNativeTagBinding: unique symbol;
 export declare const kSkipEndTag: unique symbol;
 declare const kTagContentAttr: unique symbol;
@@ -20,3 +21,11 @@ declare const _default: {
     };
 };
 export default _default;
+/** How a statically named tag claims its controlled attrs during render;
+ * nothing for the many tags that control nothing. */
+export declare function controllableClaimFor(tagName: string | undefined): "_controllable_input" | "_controllable_open" | "_controllable_select" | "_controllable_textarea" | undefined;
+/** The resume-pass latch for a tag: `_attrs_script` resolves the control kind
+ * at run time, and a run-time tag name can be any of them. */
+export declare function controllableScriptLatchFor(tagName: string | undefined): "_enable_controllable" | "_enable_controllable_input" | "_enable_controllable_open" | "_enable_controllable_select" | "_enable_controllable_textarea" | undefined;
+/** Emits the latch once per program, like `_enable_catch`. */
+export declare function enableControllable(helper: DOMRuntimeHelpers | undefined): void;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "marko",
-  "version": "6.3.23",
+  "version": "6.3.25",
   "description": "Optimized runtime for Marko templates.",
   "keywords": [
     "api",
@@ -54,8 +54,8 @@
     "magic-string": "^0.30.21"
   },
   "devDependencies": {
-    "@marko/runtime-tags": "npm:marko@6.3.23",
-    "marko": "5.39.28"
+    "marko": "5.39.29",
+    "@marko/runtime-tags": "npm:marko@6.3.25"
   },
   "engines": {
     "node": ">=22"