fix: serialize reason improvements

Author: DylanPiercey

.changeset/weak-fans-fail.md

@@ -0,0 +1,6 @@
+---
+"@marko/translator-interop-class-tags": patch
+"@marko/runtime-tags": patch
+---
+
+Track reasons for function closure serialization and add appropriate guards.

.changeset/wise-suns-hang.md

@@ -0,0 +1,5 @@
+---
+"@marko/runtime-tags": patch
+---
+
+Track downstream serialize reasons for attribute tags.

packages/runtime-tags/src/__tests__/fixtures/at-tag-inside-if-tag/__snapshots__/dom.expected/template.hydrate.js

@@ -1,3 +1 @@
-// size: 74 (min) 72 (brotli)
-(_._content_resume("b0", "Goodbye", "b"),
-  _._content_resume("b1", "Hello", "b"));
+// size: 0

packages/runtime-tags/src/__tests__/fixtures/at-tag-inside-if-tag/__snapshots__/dom.expected/template.js

@@ -2,8 +2,8 @@ export const $template = `<!>${_customTag_template}`;
 export const $walks = /* over(1), beginChild, _customTag_walks, endChild */`b/${_customTag_walks}&`;
 import { $setup as _customTag, $thing2 as _customTag_input_thing, $template as _customTag_template, $walks as _customTag_walks } from "./tags/custom-tag/index.marko";
 import * as _ from "@marko/runtime-tags/debug/dom";
-const $thing_content2 = _._content_resume("__tests__/template.marko_2_content", "Goodbye", /* over(1) */"b");
-const $thing_content = _._content_resume("__tests__/template.marko_1_content", "Hello", /* over(1) */"b");
+const $thing_content2 = /* @__PURE__ */_._content("__tests__/template.marko_2_content", "Goodbye", /* over(1) */"b");
+const $thing_content = /* @__PURE__ */_._content("__tests__/template.marko_1_content", "Hello", /* over(1) */"b");
 export function $setup($scope) {
   _customTag($scope["#childScope/0"]);
 }

packages/runtime-tags/src/__tests__/fixtures/at-tag-inside-if-tag/__snapshots__/html.expected/template.js

@@ -11,18 +11,18 @@ export default _._template("__tests__/template.marko", input => {
   if (x) {
     $thing = _.attrTag({
       x: 1,
-      content: _._content_resume("__tests__/template.marko_1_content", () => {
+      content: _._content("__tests__/template.marko_1_content", () => {
         const $scope1_id = _._scope_id();
         _._html("Hello");
-      }, $scope0_id)
+      })
     });
   } else {
     $thing = _.attrTag({
       x: 2,
-      content: _._content_resume("__tests__/template.marko_2_content", () => {
+      content: _._content("__tests__/template.marko_2_content", () => {
         const $scope2_id = _._scope_id();
         _._html("Goodbye");
-      }, $scope0_id)
+      })
     });
   }
   _._set_serialize_reason({

packages/runtime-tags/src/__tests__/fixtures/at-tags-dynamic-and-static/__snapshots__/dom.expected/template.hydrate.js

@@ -1,9 +1 @@
-// size: 112 (min) 88 (brotli)
-_._content_closures(_._content_resume("b0", "<!>:<!>", "%c%b"), {
-  2($scope, a) {
-    _._text($scope[0], a);
-  },
-  3($scope, v) {
-    _._text($scope[1], v);
-  },
-});
+// size: 0

packages/runtime-tags/src/__tests__/fixtures/at-tags-dynamic-and-static/__snapshots__/dom.expected/template.js

@@ -2,15 +2,15 @@ export const $template = `<!>${_hello_template}<!>`;
 export const $walks = /* over(1), beginChild, _hello_walks, endChild, over(1) */`b/${_hello_walks}&b`;
 import * as _ from "@marko/runtime-tags/debug/dom";
 import { $setup as _hello, $input_other as _hello_input_other, $input_item as _hello_input_item, $template as _hello_template, $walks as _hello_walks } from "./tags/hello/index.marko";
-const $item_content = _._content_closures(_._content_resume("__tests__/template.marko_2_content", "<!>:<!>", /* replace, over(2), replace, over(1) */"%c%b"), {
+const $other_content = /* @__PURE__ */_._content("__tests__/template.marko_2_content", "other", /* over(1) */"b");
+const $item_content = /* @__PURE__ */_._content_closures(/* @__PURE__ */_._content("__tests__/template.marko_1_content", "<!>:<!>", /* replace, over(2), replace, over(1) */"%c%b"), {
   a($scope, a) {
     _._text($scope["#text/0"], a);
   },
   v($scope, v) {
     _._text($scope["#text/1"], v);
   }
 });
-const $other_content = /* @__PURE__ */_._content("__tests__/template.marko_1_content", "other", /* over(1) */"b");
 export function $setup($scope) {
   _hello($scope["#childScope/0"]);
   _hello_input_other($scope["#childScope/0"], _.attrTag({

packages/runtime-tags/src/__tests__/fixtures/at-tags-dynamic-and-static/__snapshots__/html.expected/template.js

@@ -8,23 +8,17 @@ export default _._template("__tests__/template.marko", input => {
     b: 2
   }, (a, v) => {
     $item = _.attrTags($item, {
-      content: _._content_resume("__tests__/template.marko_2_content", () => {
-        const $scope2_id = _._scope_id();
-        _._html(`${_._escape(a)}${_._el_resume($scope2_id, "#text/0")}:<!>${_._escape(v)}${_._el_resume($scope2_id, "#text/1")}`);
-        _._scope($scope2_id, {
-          a,
-          v
-        }, "__tests__/template.marko", "3:8", {
-          a: "2:10",
-          v: "2:13"
-        });
-      }, $scope0_id)
+      content: _._content("__tests__/template.marko_1_content", () => {
+        const $scope1_id = _._scope_id();
+        _._html(`${_._escape(a)}${_._el_resume($scope1_id, "#text/0")}:<!>${_._escape(v)}${_._el_resume($scope1_id, "#text/1")}`);
+        _._scope($scope1_id, {}, "__tests__/template.marko", "3:8");
+      })
     });
   });
   _hello({
     other: _.attrTag({
-      content: _._content("__tests__/template.marko_1_content", () => {
-        const $scope1_id = _._scope_id();
+      content: _._content("__tests__/template.marko_2_content", () => {
+        const $scope2_id = _._scope_id();
         _._html("other");
       })
     }),

packages/runtime-tags/src/__tests__/fixtures/at-tags-dynamic-and-static/__snapshots__/resume.expected.md

@@ -19,14 +19,8 @@
     <script>
       WALKER_RUNTIME("M")("_");
       M._.r = [_ =&gt; (_.a = [0, 3,
-      {
-        a: "a",
-        v: 1
-      }, 1,
-      {
-        a: "b",
-        v: 2
-      }])]
+      {}, 1,
+      {}])]
     </script>
   </body>
 </html>

packages/runtime-tags/src/__tests__/fixtures/at-tags-dynamic-and-static/__snapshots__/ssr.expected.md

@@ -1,6 +1,6 @@
 # Write
 ```html
-  a<!--M_*4 #text/0-->:<!>1<!--M_*4 #text/1-->b<!--M_*6 #text/0-->:<!>2<!--M_*6 #text/1-->other<script>WALKER_RUNTIME("M")("_");M._.r=[_=>(_.a=[0,3,{a:"a",v:1},1,{a:"b",v:2}])]</script>
+  a<!--M_*4 #text/0-->:<!>1<!--M_*4 #text/1-->b<!--M_*6 #text/0-->:<!>2<!--M_*6 #text/1-->other<script>WALKER_RUNTIME("M")("_");M._.r=[_=>(_.a=[0,3,{},1,{}])]</script>
 ```
 
 # Render End
@@ -24,14 +24,8 @@
     <script>
       WALKER_RUNTIME("M")("_");
       M._.r = [_ =&gt; (_.a = [0, 3,
-      {
-        a: "a",
-        v: 1
-      }, 1,
-      {
-        a: "b",
-        v: 2
-      }])]
+      {}, 1,
+      {}])]
     </script>
   </body>
 </html>