@marko/runtime-tags 0.3.41 → 0.3.42

package/dist/common/types.d.ts

@@ -103,6 +103,5 @@ export declare enum ControlledType {
     InputValue = 2,
     SelectValue = 3,
     DetailsOrDialogOpen = 4,
-    None = 5,
-    Pending = 6
+    None = 5
 }

package/dist/debug/dom.js

@@ -140,9 +140,8 @@ function stringifyClassObject(name, value2) {
 function styleValue(value2) {
   return toDelimitedString(value2, ";", stringifyStyleObject);
 }
-var NON_DIMENSIONAL = /^(--|ta|or|li|z)|n-c|i(do|nk|m|t)|w$|we/;
 function stringifyStyleObject(name, value2) {
-  return value2 || value2 === 0 ? `${name}:${typeof value2 === "number" && value2 && !NON_DIMENSIONAL.test(name) ? value2 + "px" : value2}` : "";
+  return value2 || value2 === 0 ? `${name}:${typeof value2 === "number" && value2 && !/^(--|ta|or|li|z)|cou|nk|it|ag|we|do|w$/.test(name) ? value2 + "px" : value2}` : "";
 }
 function toDelimitedString(val, delimiter, stringify) {
   switch (typeof val) {
@@ -468,12 +467,10 @@ function controllable_input_checked_effect(scope, nodeAccessor) {
   syncControllable(el, "input", hasCheckboxChanged, () => {
     const checkedChange = scope[nodeAccessor + ";" /* ControlledHandler */];
     if (checkedChange) {
-      scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */;
-      checkedChange(el.checked);
+      const newValue = el.checked;
+      el.checked = !newValue;
+      checkedChange(newValue);
       run();
-      if (scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */) {
-        el.checked = !el.checked;
-      }
     }
   });
 }
@@ -494,24 +491,20 @@ function controllable_input_checkedValue_effect(scope, nodeAccessor) {
     const checkedValueChange = scope[nodeAccessor + ";" /* ControlledHandler */];
     if (checkedValueChange) {
       const oldValue = scope[nodeAccessor + ":" /* ControlledValue */];
-      scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */;
-      checkedValueChange(
-        Array.isArray(oldValue) ? updateList(oldValue, el.value, el.checked) : el.checked ? el.value : void 0
-      );
-      run();
-      if (scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */) {
-        if (el.name && el.type[0] === "r") {
-          for (const radio of el.getRootNode().querySelectorAll(
-            `[type=radio][name=${CSS.escape(el.name)}]`
-          )) {
-            if (radio.form === el.form) {
-              radio.checked = Array.isArray(oldValue) ? oldValue.includes(radio.value) : oldValue === radio.value;
-            }
+      const newValue = Array.isArray(oldValue) ? updateList(oldValue, el.value, el.checked) : el.checked ? el.value : void 0;
+      if (el.name && el.type[0] === "r") {
+        for (const radio of el.getRootNode().querySelectorAll(
+          `[type=radio][name=${CSS.escape(el.name)}]`
+        )) {
+          if (radio.form === el.form) {
+            radio.checked = Array.isArray(oldValue) ? oldValue.includes(radio.value) : oldValue === radio.value;
           }
-        } else {
-          el.checked = !el.checked;
         }
+      } else {
+        el.checked = !el.checked;
       }
+      checkedValueChange(newValue);
+      run();
     }
   });
 }
@@ -540,16 +533,14 @@ function controllable_input_value_effect(scope, nodeAccessor) {
   syncControllable(el, "input", hasValueChanged, (ev) => {
     const valueChange = scope[nodeAccessor + ";" /* ControlledHandler */];
     if (valueChange) {
-      scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */;
-      if (ev) inputType = ev.inputType;
-      valueChange(el.value);
+      const newValue = el.value;
+      inputType = ev?.inputType;
+      setValueAndUpdateSelection(
+        el,
+        scope[nodeAccessor + ":" /* ControlledValue */]
+      );
+      valueChange(newValue);
       run();
-      if (scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */) {
-        setValueAndUpdateSelection(
-          el,
-          scope[nodeAccessor + ":" /* ControlledValue */]
-        );
-      }
       inputType = "";
     }
   });
@@ -576,18 +567,16 @@ function controllable_select_value_effect(scope, nodeAccessor) {
   const onChange = () => {
     const valueChange = scope[nodeAccessor + ";" /* ControlledHandler */];
     if (valueChange) {
-      scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */;
-      valueChange(
-        Array.isArray(scope[nodeAccessor + ":" /* ControlledValue */]) ? Array.from(el.selectedOptions, toValueProp) : el.value
+      const newValue = Array.isArray(
+        scope[nodeAccessor + ":" /* ControlledValue */]
+      ) ? Array.from(el.selectedOptions, toValueProp) : el.value;
+      setSelectOptions(
+        el,
+        scope[nodeAccessor + ":" /* ControlledValue */],
+        valueChange
       );
+      valueChange(newValue);
       run();
-      if (scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */) {
-        setSelectOptions(
-          el,
-          scope[nodeAccessor + ":" /* ControlledValue */],
-          valueChange
-        );
-      }
     }
   };
   if (!el._) {
@@ -643,12 +632,10 @@ function controllable_detailsOrDialog_open_effect(scope, nodeAccessor) {
     () => {
       const openChange = scope[nodeAccessor + ";" /* ControlledHandler */];
       if (openChange && hasChanged()) {
-        scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */;
-        openChange(el.open);
+        const newValue = el.open;
+        el.open = !newValue;
+        openChange(newValue);
         run();
-        if (scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */) {
-          el.open = !el.open;
-        }
       }
     }
   );

package/dist/debug/dom.mjs

@@ -54,9 +54,8 @@ function stringifyClassObject(name, value2) {
 function styleValue(value2) {
   return toDelimitedString(value2, ";", stringifyStyleObject);
 }
-var NON_DIMENSIONAL = /^(--|ta|or|li|z)|n-c|i(do|nk|m|t)|w$|we/;
 function stringifyStyleObject(name, value2) {
-  return value2 || value2 === 0 ? `${name}:${typeof value2 === "number" && value2 && !NON_DIMENSIONAL.test(name) ? value2 + "px" : value2}` : "";
+  return value2 || value2 === 0 ? `${name}:${typeof value2 === "number" && value2 && !/^(--|ta|or|li|z)|cou|nk|it|ag|we|do|w$/.test(name) ? value2 + "px" : value2}` : "";
 }
 function toDelimitedString(val, delimiter, stringify) {
   switch (typeof val) {
@@ -382,12 +381,10 @@ function controllable_input_checked_effect(scope, nodeAccessor) {
   syncControllable(el, "input", hasCheckboxChanged, () => {
     const checkedChange = scope[nodeAccessor + ";" /* ControlledHandler */];
     if (checkedChange) {
-      scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */;
-      checkedChange(el.checked);
+      const newValue = el.checked;
+      el.checked = !newValue;
+      checkedChange(newValue);
       run();
-      if (scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */) {
-        el.checked = !el.checked;
-      }
     }
   });
 }
@@ -408,24 +405,20 @@ function controllable_input_checkedValue_effect(scope, nodeAccessor) {
     const checkedValueChange = scope[nodeAccessor + ";" /* ControlledHandler */];
     if (checkedValueChange) {
       const oldValue = scope[nodeAccessor + ":" /* ControlledValue */];
-      scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */;
-      checkedValueChange(
-        Array.isArray(oldValue) ? updateList(oldValue, el.value, el.checked) : el.checked ? el.value : void 0
-      );
-      run();
-      if (scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */) {
-        if (el.name && el.type[0] === "r") {
-          for (const radio of el.getRootNode().querySelectorAll(
-            `[type=radio][name=${CSS.escape(el.name)}]`
-          )) {
-            if (radio.form === el.form) {
-              radio.checked = Array.isArray(oldValue) ? oldValue.includes(radio.value) : oldValue === radio.value;
-            }
+      const newValue = Array.isArray(oldValue) ? updateList(oldValue, el.value, el.checked) : el.checked ? el.value : void 0;
+      if (el.name && el.type[0] === "r") {
+        for (const radio of el.getRootNode().querySelectorAll(
+          `[type=radio][name=${CSS.escape(el.name)}]`
+        )) {
+          if (radio.form === el.form) {
+            radio.checked = Array.isArray(oldValue) ? oldValue.includes(radio.value) : oldValue === radio.value;
           }
-        } else {
-          el.checked = !el.checked;
         }
+      } else {
+        el.checked = !el.checked;
       }
+      checkedValueChange(newValue);
+      run();
     }
   });
 }
@@ -454,16 +447,14 @@ function controllable_input_value_effect(scope, nodeAccessor) {
   syncControllable(el, "input", hasValueChanged, (ev) => {
     const valueChange = scope[nodeAccessor + ";" /* ControlledHandler */];
     if (valueChange) {
-      scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */;
-      if (ev) inputType = ev.inputType;
-      valueChange(el.value);
+      const newValue = el.value;
+      inputType = ev?.inputType;
+      setValueAndUpdateSelection(
+        el,
+        scope[nodeAccessor + ":" /* ControlledValue */]
+      );
+      valueChange(newValue);
       run();
-      if (scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */) {
-        setValueAndUpdateSelection(
-          el,
-          scope[nodeAccessor + ":" /* ControlledValue */]
-        );
-      }
       inputType = "";
     }
   });
@@ -490,18 +481,16 @@ function controllable_select_value_effect(scope, nodeAccessor) {
   const onChange = () => {
     const valueChange = scope[nodeAccessor + ";" /* ControlledHandler */];
     if (valueChange) {
-      scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */;
-      valueChange(
-        Array.isArray(scope[nodeAccessor + ":" /* ControlledValue */]) ? Array.from(el.selectedOptions, toValueProp) : el.value
+      const newValue = Array.isArray(
+        scope[nodeAccessor + ":" /* ControlledValue */]
+      ) ? Array.from(el.selectedOptions, toValueProp) : el.value;
+      setSelectOptions(
+        el,
+        scope[nodeAccessor + ":" /* ControlledValue */],
+        valueChange
       );
+      valueChange(newValue);
       run();
-      if (scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */) {
-        setSelectOptions(
-          el,
-          scope[nodeAccessor + ":" /* ControlledValue */],
-          valueChange
-        );
-      }
     }
   };
   if (!el._) {
@@ -557,12 +546,10 @@ function controllable_detailsOrDialog_open_effect(scope, nodeAccessor) {
     () => {
       const openChange = scope[nodeAccessor + ";" /* ControlledHandler */];
       if (openChange && hasChanged()) {
-        scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */;
-        openChange(el.open);
+        const newValue = el.open;
+        el.open = !newValue;
+        openChange(newValue);
         run();
-        if (scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */) {
-          el.open = !el.open;
-        }
       }
     }
   );

package/dist/debug/html.js

@@ -114,9 +114,8 @@ function stringifyClassObject(name, value) {
 function styleValue(value) {
   return toDelimitedString(value, ";", stringifyStyleObject);
 }
-var NON_DIMENSIONAL = /^(--|ta|or|li|z)|n-c|i(do|nk|m|t)|w$|we/;
 function stringifyStyleObject(name, value) {
-  return value || value === 0 ? `${name}:${typeof value === "number" && value && !NON_DIMENSIONAL.test(name) ? value + "px" : value}` : "";
+  return value || value === 0 ? `${name}:${typeof value === "number" && value && !/^(--|ta|or|li|z)|cou|nk|it|ag|we|do|w$/.test(name) ? value + "px" : value}` : "";
 }
 function toDelimitedString(val, delimiter, stringify) {
   switch (typeof val) {
@@ -646,15 +645,11 @@ function writeRoot(state, root) {
   if (writeProp(state, root, null, "")) {
     const rootRef = state.refs.get(root);
     if (rootRef) {
-      ensureId(state, rootRef);
-    }
-    if (assigned.size) {
-      if (assigned.delete(rootRef)) {
-        assigned.add(rootRef);
-        writeAssigned(state);
-      } else {
+      const rootId = ensureId(state, rootRef);
+      if (assigned.size) {
+        assigned.delete(rootRef);
         writeAssigned(state);
-        buf.push("," + rootRef.id);
+        buf.push("," + rootRef.assigns + rootId);
       }
     }
     result = "(";
@@ -674,8 +669,10 @@ function writeRoot(state, root) {
 }
 function writeAssigned(state) {
   for (const valueRef of state.assigned) {
-    state.buf.push("," + valueRef.assigns + (valueRef.init || valueRef.id));
-    valueRef.init = "";
+    if (valueRef.assigns || valueRef.init) {
+      state.buf.push("," + valueRef.assigns + (valueRef.init || valueRef.id));
+      valueRef.init = "";
+    }
   }
 }
 function writeProp(state, val, parent, accessor) {
@@ -743,6 +740,7 @@ function writeRegistered(state, val, parent, accessor, { access, scope }) {
     state.refs.set(val, fnRef);
     if (scopeRef) {
       if (isCircular(parent, scopeRef)) {
+        state.assigned.add(parent);
         state.assigned.add(fnRef);
         fnRef.init = access + "(" + ensureId(state, scopeRef) + ")";
         fnRef.assigns += ensureId(state, parent) + toAccess(accessor) + "=";
@@ -756,6 +754,7 @@ function writeRegistered(state, val, parent, accessor, { access, scope }) {
       const scopeRef2 = parent && state.refs.get(scope);
       const scopeId = scopeRef2 && ensureId(state, scopeRef2);
       if (scopeId && assigns !== state.assigned.size) {
+        state.assigned.add(parent);
         state.assigned.add(fnRef);
         fnRef.init = access + "(" + scopeId + ")";
         fnRef.assigns += ensureId(state, parent) + toAccess(accessor) + "=";
@@ -1271,26 +1270,31 @@ function writeObjectProps(state, val, ref) {
     }
   }
   if (hasSymbolIterator(val)) {
-    state.buf.push(sep + "[Symbol.iterator]:");
-    sep = ",";
-    if (!writeReferenceOr(
-      state,
-      writeNever,
-      val[Symbol.iterator],
-      ref,
-      "Symbol.iterator"
-    )) {
-      const arrayRef = new Reference(
-        ref,
-        null,
-        state.flush,
-        null,
-        nextRefAccess(state)
-      );
-      state.buf.push("(a=>()=>a.values())(" + arrayRef.id + "=");
-      writeArray(state, [...val], arrayRef);
-      state.buf.push(")");
+    const iterArr = [...val];
+    switch (iterArr.length) {
+      case 0:
+        state.buf.push(sep + "*[Symbol.iterator](){}");
+        break;
+      case 1:
+        state.buf.push(
+          sep + "*[Symbol.iterator](){yield " + (iterArr[0] === val ? "this" : ensureId(state, ref)) + "}"
+        );
+        break;
+      default: {
+        const iterRef = new Reference(
+          ref,
+          null,
+          state.flush,
+          null,
+          nextRefAccess(state)
+        );
+        state.buf.push(sep + "*[(" + iterRef.id + "=");
+        writeArray(state, iterArr, iterRef);
+        state.buf.push(",Symbol.iterator)](){yield*" + iterRef.id + "}");
+        break;
+      }
     }
+    sep = ",";
   }
   return sep;
 }

package/dist/debug/html.mjs

@@ -32,9 +32,8 @@ function stringifyClassObject(name, value) {
 function styleValue(value) {
   return toDelimitedString(value, ";", stringifyStyleObject);
 }
-var NON_DIMENSIONAL = /^(--|ta|or|li|z)|n-c|i(do|nk|m|t)|w$|we/;
 function stringifyStyleObject(name, value) {
-  return value || value === 0 ? `${name}:${typeof value === "number" && value && !NON_DIMENSIONAL.test(name) ? value + "px" : value}` : "";
+  return value || value === 0 ? `${name}:${typeof value === "number" && value && !/^(--|ta|or|li|z)|cou|nk|it|ag|we|do|w$/.test(name) ? value + "px" : value}` : "";
 }
 function toDelimitedString(val, delimiter, stringify) {
   switch (typeof val) {
@@ -564,15 +563,11 @@ function writeRoot(state, root) {
   if (writeProp(state, root, null, "")) {
     const rootRef = state.refs.get(root);
     if (rootRef) {
-      ensureId(state, rootRef);
-    }
-    if (assigned.size) {
-      if (assigned.delete(rootRef)) {
-        assigned.add(rootRef);
-        writeAssigned(state);
-      } else {
+      const rootId = ensureId(state, rootRef);
+      if (assigned.size) {
+        assigned.delete(rootRef);
         writeAssigned(state);
-        buf.push("," + rootRef.id);
+        buf.push("," + rootRef.assigns + rootId);
       }
     }
     result = "(";
@@ -592,8 +587,10 @@ function writeRoot(state, root) {
 }
 function writeAssigned(state) {
   for (const valueRef of state.assigned) {
-    state.buf.push("," + valueRef.assigns + (valueRef.init || valueRef.id));
-    valueRef.init = "";
+    if (valueRef.assigns || valueRef.init) {
+      state.buf.push("," + valueRef.assigns + (valueRef.init || valueRef.id));
+      valueRef.init = "";
+    }
   }
 }
 function writeProp(state, val, parent, accessor) {
@@ -661,6 +658,7 @@ function writeRegistered(state, val, parent, accessor, { access, scope }) {
     state.refs.set(val, fnRef);
     if (scopeRef) {
       if (isCircular(parent, scopeRef)) {
+        state.assigned.add(parent);
         state.assigned.add(fnRef);
         fnRef.init = access + "(" + ensureId(state, scopeRef) + ")";
         fnRef.assigns += ensureId(state, parent) + toAccess(accessor) + "=";
@@ -674,6 +672,7 @@ function writeRegistered(state, val, parent, accessor, { access, scope }) {
       const scopeRef2 = parent && state.refs.get(scope);
       const scopeId = scopeRef2 && ensureId(state, scopeRef2);
       if (scopeId && assigns !== state.assigned.size) {
+        state.assigned.add(parent);
         state.assigned.add(fnRef);
         fnRef.init = access + "(" + scopeId + ")";
         fnRef.assigns += ensureId(state, parent) + toAccess(accessor) + "=";
@@ -1189,26 +1188,31 @@ function writeObjectProps(state, val, ref) {
     }
   }
   if (hasSymbolIterator(val)) {
-    state.buf.push(sep + "[Symbol.iterator]:");
-    sep = ",";
-    if (!writeReferenceOr(
-      state,
-      writeNever,
-      val[Symbol.iterator],
-      ref,
-      "Symbol.iterator"
-    )) {
-      const arrayRef = new Reference(
-        ref,
-        null,
-        state.flush,
-        null,
-        nextRefAccess(state)
-      );
-      state.buf.push("(a=>()=>a.values())(" + arrayRef.id + "=");
-      writeArray(state, [...val], arrayRef);
-      state.buf.push(")");
+    const iterArr = [...val];
+    switch (iterArr.length) {
+      case 0:
+        state.buf.push(sep + "*[Symbol.iterator](){}");
+        break;
+      case 1:
+        state.buf.push(
+          sep + "*[Symbol.iterator](){yield " + (iterArr[0] === val ? "this" : ensureId(state, ref)) + "}"
+        );
+        break;
+      default: {
+        const iterRef = new Reference(
+          ref,
+          null,
+          state.flush,
+          null,
+          nextRefAccess(state)
+        );
+        state.buf.push(sep + "*[(" + iterRef.id + "=");
+        writeArray(state, iterArr, iterRef);
+        state.buf.push(",Symbol.iterator)](){yield*" + iterRef.id + "}");
+        break;
+      }
     }
+    sep = ",";
   }
   return sep;
 }

package/dist/dom.js

@@ -121,9 +121,8 @@ function stringifyClassObject(name, value2) {
 function styleValue(value2) {
   return toDelimitedString(value2, ";", stringifyStyleObject);
 }
-var NON_DIMENSIONAL = /^(--|ta|or|li|z)|n-c|i(do|nk|m|t)|w$|we/;
 function stringifyStyleObject(name, value2) {
-  return value2 || value2 === 0 ? `${name}:${typeof value2 == "number" && value2 && !NON_DIMENSIONAL.test(name) ? value2 + "px" : value2}` : "";
+  return value2 || value2 === 0 ? `${name}:${typeof value2 == "number" && value2 && !/^(--|ta|or|li|z)|cou|nk|it|ag|we|do|w$/.test(name) ? value2 + "px" : value2}` : "";
 }
 function toDelimitedString(val, delimiter, stringify) {
   switch (typeof val) {
@@ -333,7 +332,10 @@ function controllable_input_checked_effect(scope, nodeAccessor) {
   let el = scope[nodeAccessor];
   syncControllable(el, "input", hasCheckboxChanged, () => {
     let checkedChange = scope[nodeAccessor + ";" /* ControlledHandler */];
-    checkedChange && (scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */, checkedChange(el.checked), run(), scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */ && (el.checked = !el.checked));
+    if (checkedChange) {
+      let newValue = el.checked;
+      el.checked = !newValue, checkedChange(newValue), run();
+    }
   });
 }
 function controllable_input_checkedValue(scope, nodeAccessor, checkedValue, checkedValueChange, value2) {
@@ -350,17 +352,15 @@ function controllable_input_checkedValue_effect(scope, nodeAccessor) {
   syncControllable(el, "input", hasCheckboxChanged, () => {
     let checkedValueChange = scope[nodeAccessor + ";" /* ControlledHandler */];
     if (checkedValueChange) {
-      let oldValue = scope[nodeAccessor + ":" /* ControlledValue */];
-      if (scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */, checkedValueChange(
-        Array.isArray(oldValue) ? updateList(oldValue, el.value, el.checked) : el.checked ? el.value : void 0
-      ), run(), scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */)
-        if (el.name && el.type[0] === "r")
-          for (let radio of el.getRootNode().querySelectorAll(
-            `[type=radio][name=${CSS.escape(el.name)}]`
-          ))
-            radio.form === el.form && (radio.checked = Array.isArray(oldValue) ? oldValue.includes(radio.value) : oldValue === radio.value);
-        else
-          el.checked = !el.checked;
+      let oldValue = scope[nodeAccessor + ":" /* ControlledValue */], newValue = Array.isArray(oldValue) ? updateList(oldValue, el.value, el.checked) : el.checked ? el.value : void 0;
+      if (el.name && el.type[0] === "r")
+        for (let radio of el.getRootNode().querySelectorAll(
+          `[type=radio][name=${CSS.escape(el.name)}]`
+        ))
+          radio.form === el.form && (radio.checked = Array.isArray(oldValue) ? oldValue.includes(radio.value) : oldValue === radio.value);
+      else
+        el.checked = !el.checked;
+      checkedValueChange(newValue), run();
     }
   });
 }
@@ -372,10 +372,13 @@ function controllable_input_value_effect(scope, nodeAccessor) {
   let el = scope[nodeAccessor];
   isResuming && (scope[nodeAccessor + ":" /* ControlledValue */] = el.defaultValue), syncControllable(el, "input", hasValueChanged, (ev) => {
     let valueChange = scope[nodeAccessor + ";" /* ControlledHandler */];
-    valueChange && (scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */, ev && (inputType = ev.inputType), valueChange(el.value), run(), scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */ && setValueAndUpdateSelection(
-      el,
-      scope[nodeAccessor + ":" /* ControlledValue */]
-    ), inputType = "");
+    if (valueChange) {
+      let newValue = el.value;
+      inputType = ev?.inputType, setValueAndUpdateSelection(
+        el,
+        scope[nodeAccessor + ":" /* ControlledValue */]
+      ), valueChange(newValue), run(), inputType = "";
+    }
   });
 }
 function controllable_select_value(scope, nodeAccessor, value2, valueChange) {
@@ -391,13 +394,16 @@ function controllable_select_value(scope, nodeAccessor, value2, valueChange) {
 function controllable_select_value_effect(scope, nodeAccessor) {
   let el = scope[nodeAccessor], onChange = () => {
     let valueChange = scope[nodeAccessor + ";" /* ControlledHandler */];
-    valueChange && (scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */, valueChange(
-      Array.isArray(scope[nodeAccessor + ":" /* ControlledValue */]) ? Array.from(el.selectedOptions, toValueProp) : el.value
-    ), run(), scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */ && setSelectOptions(
-      el,
-      scope[nodeAccessor + ":" /* ControlledValue */],
-      valueChange
-    ));
+    if (valueChange) {
+      let newValue = Array.isArray(
+        scope[nodeAccessor + ":" /* ControlledValue */]
+      ) ? Array.from(el.selectedOptions, toValueProp) : el.value;
+      setSelectOptions(
+        el,
+        scope[nodeAccessor + ":" /* ControlledValue */],
+        valueChange
+      ), valueChange(newValue), run();
+    }
   };
   el._ || new MutationObserver(() => {
     let value2 = scope[nodeAccessor + ":" /* ControlledValue */];
@@ -433,7 +439,10 @@ function controllable_detailsOrDialog_open_effect(scope, nodeAccessor) {
     hasChanged,
     () => {
       let openChange = scope[nodeAccessor + ";" /* ControlledHandler */];
-      openChange && hasChanged() && (scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */, openChange(el.open), run(), scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */ && (el.open = !el.open));
+      if (openChange && hasChanged()) {
+        let newValue = el.open;
+        el.open = !newValue, openChange(newValue), run();
+      }
     }
   );
 }

package/dist/dom.mjs

@@ -38,9 +38,8 @@ function stringifyClassObject(name, value2) {
 function styleValue(value2) {
   return toDelimitedString(value2, ";", stringifyStyleObject);
 }
-var NON_DIMENSIONAL = /^(--|ta|or|li|z)|n-c|i(do|nk|m|t)|w$|we/;
 function stringifyStyleObject(name, value2) {
-  return value2 || value2 === 0 ? `${name}:${typeof value2 == "number" && value2 && !NON_DIMENSIONAL.test(name) ? value2 + "px" : value2}` : "";
+  return value2 || value2 === 0 ? `${name}:${typeof value2 == "number" && value2 && !/^(--|ta|or|li|z)|cou|nk|it|ag|we|do|w$/.test(name) ? value2 + "px" : value2}` : "";
 }
 function toDelimitedString(val, delimiter, stringify) {
   switch (typeof val) {
@@ -250,7 +249,10 @@ function controllable_input_checked_effect(scope, nodeAccessor) {
   let el = scope[nodeAccessor];
   syncControllable(el, "input", hasCheckboxChanged, () => {
     let checkedChange = scope[nodeAccessor + ";" /* ControlledHandler */];
-    checkedChange && (scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */, checkedChange(el.checked), run(), scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */ && (el.checked = !el.checked));
+    if (checkedChange) {
+      let newValue = el.checked;
+      el.checked = !newValue, checkedChange(newValue), run();
+    }
   });
 }
 function controllable_input_checkedValue(scope, nodeAccessor, checkedValue, checkedValueChange, value2) {
@@ -267,17 +269,15 @@ function controllable_input_checkedValue_effect(scope, nodeAccessor) {
   syncControllable(el, "input", hasCheckboxChanged, () => {
     let checkedValueChange = scope[nodeAccessor + ";" /* ControlledHandler */];
     if (checkedValueChange) {
-      let oldValue = scope[nodeAccessor + ":" /* ControlledValue */];
-      if (scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */, checkedValueChange(
-        Array.isArray(oldValue) ? updateList(oldValue, el.value, el.checked) : el.checked ? el.value : void 0
-      ), run(), scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */)
-        if (el.name && el.type[0] === "r")
-          for (let radio of el.getRootNode().querySelectorAll(
-            `[type=radio][name=${CSS.escape(el.name)}]`
-          ))
-            radio.form === el.form && (radio.checked = Array.isArray(oldValue) ? oldValue.includes(radio.value) : oldValue === radio.value);
-        else
-          el.checked = !el.checked;
+      let oldValue = scope[nodeAccessor + ":" /* ControlledValue */], newValue = Array.isArray(oldValue) ? updateList(oldValue, el.value, el.checked) : el.checked ? el.value : void 0;
+      if (el.name && el.type[0] === "r")
+        for (let radio of el.getRootNode().querySelectorAll(
+          `[type=radio][name=${CSS.escape(el.name)}]`
+        ))
+          radio.form === el.form && (radio.checked = Array.isArray(oldValue) ? oldValue.includes(radio.value) : oldValue === radio.value);
+      else
+        el.checked = !el.checked;
+      checkedValueChange(newValue), run();
     }
   });
 }
@@ -289,10 +289,13 @@ function controllable_input_value_effect(scope, nodeAccessor) {
   let el = scope[nodeAccessor];
   isResuming && (scope[nodeAccessor + ":" /* ControlledValue */] = el.defaultValue), syncControllable(el, "input", hasValueChanged, (ev) => {
     let valueChange = scope[nodeAccessor + ";" /* ControlledHandler */];
-    valueChange && (scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */, ev && (inputType = ev.inputType), valueChange(el.value), run(), scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */ && setValueAndUpdateSelection(
-      el,
-      scope[nodeAccessor + ":" /* ControlledValue */]
-    ), inputType = "");
+    if (valueChange) {
+      let newValue = el.value;
+      inputType = ev?.inputType, setValueAndUpdateSelection(
+        el,
+        scope[nodeAccessor + ":" /* ControlledValue */]
+      ), valueChange(newValue), run(), inputType = "";
+    }
   });
 }
 function controllable_select_value(scope, nodeAccessor, value2, valueChange) {
@@ -308,13 +311,16 @@ function controllable_select_value(scope, nodeAccessor, value2, valueChange) {
 function controllable_select_value_effect(scope, nodeAccessor) {
   let el = scope[nodeAccessor], onChange = () => {
     let valueChange = scope[nodeAccessor + ";" /* ControlledHandler */];
-    valueChange && (scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */, valueChange(
-      Array.isArray(scope[nodeAccessor + ":" /* ControlledValue */]) ? Array.from(el.selectedOptions, toValueProp) : el.value
-    ), run(), scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */ && setSelectOptions(
-      el,
-      scope[nodeAccessor + ":" /* ControlledValue */],
-      valueChange
-    ));
+    if (valueChange) {
+      let newValue = Array.isArray(
+        scope[nodeAccessor + ":" /* ControlledValue */]
+      ) ? Array.from(el.selectedOptions, toValueProp) : el.value;
+      setSelectOptions(
+        el,
+        scope[nodeAccessor + ":" /* ControlledValue */],
+        valueChange
+      ), valueChange(newValue), run();
+    }
   };
   el._ || new MutationObserver(() => {
     let value2 = scope[nodeAccessor + ":" /* ControlledValue */];
@@ -350,7 +356,10 @@ function controllable_detailsOrDialog_open_effect(scope, nodeAccessor) {
     hasChanged,
     () => {
       let openChange = scope[nodeAccessor + ";" /* ControlledHandler */];
-      openChange && hasChanged() && (scope[nodeAccessor + "=" /* ControlledType */] = 6 /* Pending */, openChange(el.open), run(), scope[nodeAccessor + "=" /* ControlledType */] === 6 /* Pending */ && (el.open = !el.open));
+      if (openChange && hasChanged()) {
+        let newValue = el.open;
+        el.open = !newValue, openChange(newValue), run();
+      }
     }
   );
 }

package/dist/html.js

@@ -99,9 +99,8 @@ function stringifyClassObject(name, value) {
 function styleValue(value) {
   return toDelimitedString(value, ";", stringifyStyleObject);
 }
-var NON_DIMENSIONAL = /^(--|ta|or|li|z)|n-c|i(do|nk|m|t)|w$|we/;
 function stringifyStyleObject(name, value) {
-  return value || value === 0 ? `${name}:${typeof value == "number" && value && !NON_DIMENSIONAL.test(name) ? value + "px" : value}` : "";
+  return value || value === 0 ? `${name}:${typeof value == "number" && value && !/^(--|ta|or|li|z)|cou|nk|it|ag|we|do|w$/.test(name) ? value + "px" : value}` : "";
 }
 function toDelimitedString(val, delimiter, stringify) {
   switch (typeof val) {
@@ -479,7 +478,11 @@ function writeRoot(state, root) {
   let { buf, assigned } = state, hadBuf = buf.length !== 0, result = "";
   if (hadBuf && buf.push(","), writeProp(state, root, null, "")) {
     let rootRef = state.refs.get(root);
-    rootRef && ensureId(state, rootRef), assigned.size && (assigned.delete(rootRef) ? (assigned.add(rootRef), writeAssigned(state)) : (writeAssigned(state), buf.push("," + rootRef.id))), result = "(", buf.push(")");
+    if (rootRef) {
+      let rootId = ensureId(state, rootRef);
+      assigned.size && (assigned.delete(rootRef), writeAssigned(state), buf.push("," + rootRef.assigns + rootId));
+    }
+    result = "(", buf.push(")");
   } else
     hadBuf && (buf.pop(), writeAssigned(state)), result = "{", buf.push("}");
   for (let chunk of buf)
@@ -488,7 +491,7 @@ function writeRoot(state, root) {
 }
 function writeAssigned(state) {
   for (let valueRef of state.assigned)
-    state.buf.push("," + valueRef.assigns + (valueRef.init || valueRef.id)), valueRef.init = "";
+    (valueRef.assigns || valueRef.init) && (state.buf.push("," + valueRef.assigns + (valueRef.init || valueRef.id)), valueRef.init = "");
 }
 function writeProp(state, val, parent, accessor) {
   switch (typeof val) {
@@ -530,13 +533,13 @@ function writeRegistered(state, val, parent, accessor, { access, scope }) {
     );
     if (state.refs.set(val, fnRef), scopeRef) {
       if (isCircular(parent, scopeRef))
-        return state.assigned.add(fnRef), fnRef.init = access + "(" + ensureId(state, scopeRef) + ")", fnRef.assigns += ensureId(state, parent) + toAccess(accessor) + "=", !1;
+        return state.assigned.add(parent), state.assigned.add(fnRef), fnRef.init = access + "(" + ensureId(state, scopeRef) + ")", fnRef.assigns += ensureId(state, parent) + toAccess(accessor) + "=", !1;
       state.buf.push(access + "(" + ensureId(state, scopeRef) + ")");
     } else {
       let pos = state.buf.push(""), assigns = state.assigned.size;
       writeProp(state, scope, parent, "");
       let scopeRef2 = parent && state.refs.get(scope), scopeId = scopeRef2 && ensureId(state, scopeRef2);
-      scopeId && assigns !== state.assigned.size ? (state.assigned.add(fnRef), fnRef.init = access + "(" + scopeId + ")", fnRef.assigns += ensureId(state, parent) + toAccess(accessor) + "=") : (state.buf[pos - 2] += access + "(", state.buf.push(")"));
+      scopeId && assigns !== state.assigned.size ? (state.assigned.add(parent), state.assigned.add(fnRef), fnRef.init = access + "(" + scopeId + ")", fnRef.assigns += ensureId(state, parent) + toAccess(accessor) + "=") : (state.buf[pos - 2] += access + "(", state.buf.push(")"));
     }
   } else
     state.buf.push(access);
@@ -821,21 +824,30 @@ function writeObjectProps(state, val, ref) {
         escapedKey
       ) ? sep = "," : state.buf.pop();
     }
-  if (hasSymbolIterator(val) && (state.buf.push(sep + "[Symbol.iterator]:"), sep = ",", !writeReferenceOr(
-    state,
-    writeNever,
-    val[Symbol.iterator],
-    ref,
-    "Symbol.iterator"
-  ))) {
-    let arrayRef = new Reference(
-      ref,
-      null,
-      state.flush,
-      null,
-      nextRefAccess(state)
-    );
-    state.buf.push("(a=>()=>a.values())(" + arrayRef.id + "="), writeArray(state, [...val], arrayRef), state.buf.push(")");
+  if (hasSymbolIterator(val)) {
+    let iterArr = [...val];
+    switch (iterArr.length) {
+      case 0:
+        state.buf.push(sep + "*[Symbol.iterator](){}");
+        break;
+      case 1:
+        state.buf.push(
+          sep + "*[Symbol.iterator](){yield " + (iterArr[0] === val ? "this" : ensureId(state, ref)) + "}"
+        );
+        break;
+      default: {
+        let iterRef = new Reference(
+          ref,
+          null,
+          state.flush,
+          null,
+          nextRefAccess(state)
+        );
+        state.buf.push(sep + "*[(" + iterRef.id + "="), writeArray(state, iterArr, iterRef), state.buf.push(",Symbol.iterator)](){yield*" + iterRef.id + "}");
+        break;
+      }
+    }
+    sep = ",";
   }
   return sep;
 }

package/dist/html.mjs

@@ -20,9 +20,8 @@ function stringifyClassObject(name, value) {
 function styleValue(value) {
   return toDelimitedString(value, ";", stringifyStyleObject);
 }
-var NON_DIMENSIONAL = /^(--|ta|or|li|z)|n-c|i(do|nk|m|t)|w$|we/;
 function stringifyStyleObject(name, value) {
-  return value || value === 0 ? `${name}:${typeof value == "number" && value && !NON_DIMENSIONAL.test(name) ? value + "px" : value}` : "";
+  return value || value === 0 ? `${name}:${typeof value == "number" && value && !/^(--|ta|or|li|z)|cou|nk|it|ag|we|do|w$/.test(name) ? value + "px" : value}` : "";
 }
 function toDelimitedString(val, delimiter, stringify) {
   switch (typeof val) {
@@ -400,7 +399,11 @@ function writeRoot(state, root) {
   let { buf, assigned } = state, hadBuf = buf.length !== 0, result = "";
   if (hadBuf && buf.push(","), writeProp(state, root, null, "")) {
     let rootRef = state.refs.get(root);
-    rootRef && ensureId(state, rootRef), assigned.size && (assigned.delete(rootRef) ? (assigned.add(rootRef), writeAssigned(state)) : (writeAssigned(state), buf.push("," + rootRef.id))), result = "(", buf.push(")");
+    if (rootRef) {
+      let rootId = ensureId(state, rootRef);
+      assigned.size && (assigned.delete(rootRef), writeAssigned(state), buf.push("," + rootRef.assigns + rootId));
+    }
+    result = "(", buf.push(")");
   } else
     hadBuf && (buf.pop(), writeAssigned(state)), result = "{", buf.push("}");
   for (let chunk of buf)
@@ -409,7 +412,7 @@ function writeRoot(state, root) {
 }
 function writeAssigned(state) {
   for (let valueRef of state.assigned)
-    state.buf.push("," + valueRef.assigns + (valueRef.init || valueRef.id)), valueRef.init = "";
+    (valueRef.assigns || valueRef.init) && (state.buf.push("," + valueRef.assigns + (valueRef.init || valueRef.id)), valueRef.init = "");
 }
 function writeProp(state, val, parent, accessor) {
   switch (typeof val) {
@@ -451,13 +454,13 @@ function writeRegistered(state, val, parent, accessor, { access, scope }) {
     );
     if (state.refs.set(val, fnRef), scopeRef) {
       if (isCircular(parent, scopeRef))
-        return state.assigned.add(fnRef), fnRef.init = access + "(" + ensureId(state, scopeRef) + ")", fnRef.assigns += ensureId(state, parent) + toAccess(accessor) + "=", !1;
+        return state.assigned.add(parent), state.assigned.add(fnRef), fnRef.init = access + "(" + ensureId(state, scopeRef) + ")", fnRef.assigns += ensureId(state, parent) + toAccess(accessor) + "=", !1;
       state.buf.push(access + "(" + ensureId(state, scopeRef) + ")");
     } else {
       let pos = state.buf.push(""), assigns = state.assigned.size;
       writeProp(state, scope, parent, "");
       let scopeRef2 = parent && state.refs.get(scope), scopeId = scopeRef2 && ensureId(state, scopeRef2);
-      scopeId && assigns !== state.assigned.size ? (state.assigned.add(fnRef), fnRef.init = access + "(" + scopeId + ")", fnRef.assigns += ensureId(state, parent) + toAccess(accessor) + "=") : (state.buf[pos - 2] += access + "(", state.buf.push(")"));
+      scopeId && assigns !== state.assigned.size ? (state.assigned.add(parent), state.assigned.add(fnRef), fnRef.init = access + "(" + scopeId + ")", fnRef.assigns += ensureId(state, parent) + toAccess(accessor) + "=") : (state.buf[pos - 2] += access + "(", state.buf.push(")"));
     }
   } else
     state.buf.push(access);
@@ -742,21 +745,30 @@ function writeObjectProps(state, val, ref) {
         escapedKey
       ) ? sep = "," : state.buf.pop();
     }
-  if (hasSymbolIterator(val) && (state.buf.push(sep + "[Symbol.iterator]:"), sep = ",", !writeReferenceOr(
-    state,
-    writeNever,
-    val[Symbol.iterator],
-    ref,
-    "Symbol.iterator"
-  ))) {
-    let arrayRef = new Reference(
-      ref,
-      null,
-      state.flush,
-      null,
-      nextRefAccess(state)
-    );
-    state.buf.push("(a=>()=>a.values())(" + arrayRef.id + "="), writeArray(state, [...val], arrayRef), state.buf.push(")");
+  if (hasSymbolIterator(val)) {
+    let iterArr = [...val];
+    switch (iterArr.length) {
+      case 0:
+        state.buf.push(sep + "*[Symbol.iterator](){}");
+        break;
+      case 1:
+        state.buf.push(
+          sep + "*[Symbol.iterator](){yield " + (iterArr[0] === val ? "this" : ensureId(state, ref)) + "}"
+        );
+        break;
+      default: {
+        let iterRef = new Reference(
+          ref,
+          null,
+          state.flush,
+          null,
+          nextRefAccess(state)
+        );
+        state.buf.push(sep + "*[(" + iterRef.id + "="), writeArray(state, iterArr, iterRef), state.buf.push(",Symbol.iterator)](){yield*" + iterRef.id + "}");
+        break;
+      }
+    }
+    sep = ",";
   }
   return sep;
 }

package/dist/translator/index.js

@@ -229,9 +229,8 @@ function stringifyClassObject(name2, value) {
 function styleValue(value) {
   return toDelimitedString(value, ";", stringifyStyleObject);
 }
-var NON_DIMENSIONAL = /^(--|ta|or|li|z)|n-c|i(do|nk|m|t)|w$|we/;
 function stringifyStyleObject(name2, value) {
-  return value || value === 0 ? `${name2}:${typeof value === "number" && value && !NON_DIMENSIONAL.test(name2) ? value + "px" : value}` : "";
+  return value || value === 0 ? `${name2}:${typeof value === "number" && value && !/^(--|ta|or|li|z)|cou|nk|it|ag|we|do|w$/.test(name2) ? value + "px" : value}` : "";
 }
 function toDelimitedString(val, delimiter, stringify) {
   switch (typeof val) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@marko/runtime-tags",
-  "version": "0.3.41",
+  "version": "0.3.42",
   "description": "Optimized runtime for Marko templates.",
   "keywords": [
     "api",