fix: support assignment patterns via pre-analyze

Author: DylanPiercey

.changeset/eight-badgers-join.md

@@ -0,0 +1,5 @@
+---
+"@marko/runtime-tags": patch
+---
+
+Fix support for assignment patterns.

packages/runtime-tags/src/__tests__/fixtures/function-tag-var-default-registration/__snapshots__/csr-sanitized.expected.md

@@ -12,6 +12,6 @@ container.querySelector("button").click();
 ```
 ```html
 <button>
-  before
+  after
 </button>
 ```

packages/runtime-tags/src/__tests__/fixtures/function-tag-var-default-registration/__snapshots__/csr.expected.md

@@ -16,6 +16,12 @@ container.querySelector("button").click();
 ```
 ```html
 <button>
-  before
+  after
 </button>
 ```
+
+# Mutations
+```
+REMOVE #text in button
+INSERT button/#text
+```

packages/runtime-tags/src/__tests__/fixtures/function-tag-var-default-registration/__snapshots__/dom.expected/template.js

@@ -1,15 +1,18 @@
 export const $template = "<button>before</button>";
 export const $walks = /* get, over(1) */" b";
-function updateText(ev) {
-  ev.target.textContent = "after";
-}
+const updateText = $updateText;
 import * as _ from "@marko/runtime-tags/debug/dom";
-const $pattern2 = /* @__PURE__ */_._const("$pattern", ($scope, $pattern) => $onClick($scope, $pattern.onClick));
-const $onClick__script = _._script("__tests__/template.marko_0_onClick", ($scope, {
+const $pattern2 = /* @__PURE__ */_._const("$pattern", ($scope, $pattern) => $onClick2($scope, $pattern.onClick));
+const $onClick3__script = _._script("__tests__/template.marko_0_onClick", ($scope, {
   onClick
 }) => _._on($scope["#button/0"], "click", onClick));
-const $onClick = /* @__PURE__ */_._const("onClick", $onClick__script);
+const $onClick3 = /* @__PURE__ */_._const("onClick", $onClick3__script);
+const $onClick2 = /* @__PURE__ */_._const("$onClick", ($scope, $onClick) => $onClick3($scope, void 0 !== $onClick ? $onClick : updateText));
 export function $setup($scope) {
   $pattern2($scope, {});
 }
+function $updateText(ev) {
+  ev.target.textContent = "after";
+}
+_._resume("__tests__/template.marko_0/updateText", $updateText);
 export default /* @__PURE__ */_._template("__tests__/template.marko", $template, $walks, $setup);

packages/runtime-tags/src/__tests__/fixtures/function-tag-var-default-registration/__snapshots__/html.expected/template.js

@@ -1,12 +1,14 @@
 function updateText(ev) {
   ev.target.textContent = "after";
 }
+_._resume(updateText, "__tests__/template.marko_0/updateText");
 import * as _ from "@marko/runtime-tags/debug/html";
 export default _._template("__tests__/template.marko", input => {
   const $scope0_id = _._scope_id();
   const {
-    onClick = updateText
+    onClick: $onClick
   } = {};
+  const onClick = void 0 !== $onClick ? $onClick : updateText;
   _._html(`<button>before</button>${_._el_resume($scope0_id, "#button/0")}`);
   _._script($scope0_id, "__tests__/template.marko_0_onClick");
   _._scope($scope0_id, {

packages/runtime-tags/src/__tests__/fixtures/param-destructure-default/__snapshots__/.name-cache.json

@@ -0,0 +1,5 @@
+{
+  "vars": {
+    "props": {}
+  }
+}

packages/runtime-tags/src/__tests__/fixtures/param-destructure-default/__snapshots__/csr-sanitized.expected.md

@@ -0,0 +1,33 @@
+# Render
+```html
+<div
+  class="a"
+>
+  0 object
+</div>
+<div
+  class="a"
+>
+  1 object
+</div>
+<div
+  class="a"
+>
+  2 undefined
+</div>
+<div
+  class="b"
+>
+  0 object
+</div>
+<div
+  class="b"
+>
+  1 object
+</div>
+<div
+  class="b"
+>
+  2 undefined
+</div>
+```

packages/runtime-tags/src/__tests__/fixtures/param-destructure-default/__snapshots__/csr.expected.md

@@ -0,0 +1,40 @@
+# Render
+```html
+<!---->
+<div
+  class="a"
+>
+  0 object
+</div>
+<div
+  class="a"
+>
+  1 object
+</div>
+<div
+  class="a"
+>
+  2 undefined
+</div>
+<div
+  class="b"
+>
+  0 object
+</div>
+<div
+  class="b"
+>
+  1 object
+</div>
+<div
+  class="b"
+>
+  2 undefined
+</div>
+<!---->
+```
+
+# Mutations
+```
+INSERT #comment0, div0, div1, div2, div3, div4, div5, #comment1
+```

packages/runtime-tags/src/__tests__/fixtures/param-destructure-default/__snapshots__/dom.expected/template.hydrate.js

@@ -0,0 +1 @@
+// size: 0

packages/runtime-tags/src/__tests__/fixtures/param-destructure-default/__snapshots__/dom.expected/template.js

@@ -0,0 +1,48 @@
+const $ChildB_content__walks = /* next(1), replace, over(2), replace, out(1) */"D%c%l",
+  $ChildB_content__template = "<div class=b><!> <!></div>",
+  $ChildA_content__walks = /* next(1), replace, over(2), replace, out(1) */"D%c%l",
+  $ChildA_content__template = "<div class=a><!> <!></div>";
+export const $template = `<!>${$ChildA_content__template}${$ChildA_content__template}${$ChildA_content__template}${$ChildB_content__template}${$ChildB_content__template}${$ChildB_content__template}<!>`;
+export const $walks = /* over(1), beginChild, $ChildA_content__walks, endChild, beginChild, $ChildA_content__walks, endChild, beginChild, $ChildA_content__walks, endChild, beginChild, $ChildB_content__walks, endChild, beginChild, $ChildB_content__walks, endChild, beginChild, $ChildB_content__walks, endChild, over(1) */`b/${$ChildA_content__walks}&/${$ChildA_content__walks}&/${$ChildA_content__walks}&/${$ChildB_content__walks}&/${$ChildB_content__walks}&/${$ChildB_content__walks}&b`;
+import * as _ from "@marko/runtime-tags/debug/dom";
+const $ChildB_content__$pattern = /* @__PURE__ */_._const("$pattern2", ($scope, $pattern2) => $ChildB_content__$bar($scope, $pattern2.bar));
+const $ChildB_content__bar = /* @__PURE__ */_._const("bar", ($scope, bar) => _._text($scope["#text/0"], bar));
+const $ChildB_content__$bar = /* @__PURE__ */_._const("$bar2", ($scope, $bar2) => $ChildB_content__bar($scope, void 0 !== $bar2 ? $bar2 : 1));
+const $ChildB_content__$foo = ($scope, $foo2) => {
+  $ChildB_content__$pattern($scope, void 0 !== $foo2 ? $foo2 : {
+    bar: 2
+  });
+};
+const $ChildB_content__foo = /* @__PURE__ */_._const("foo", ($scope, foo) => {
+  _._text($scope["#text/1"], typeof foo);
+  $ChildB_content__$foo($scope, foo);
+});
+const $ChildB_content__$params = /* @__PURE__ */_._const("$params3", ($scope, $params3) => $ChildB_content__input($scope, $params3[0]));
+const $ChildB_content__input = /* @__PURE__ */_._const("input", ($scope, input) => $ChildB_content__foo($scope, input.foo));
+const $ChildA_content__$pattern = /* @__PURE__ */_._const("$pattern", ($scope, $pattern) => $ChildA_content__$bar($scope, $pattern.bar));
+const $ChildA_content__bar = /* @__PURE__ */_._const("bar", ($scope, bar) => _._text($scope["#text/0"], bar));
+const $ChildA_content__$bar = /* @__PURE__ */_._const("$bar", ($scope, $bar) => $ChildA_content__bar($scope, void 0 !== $bar ? $bar : 1));
+const $ChildA_content__$foo = ($scope, $foo) => {
+  $ChildA_content__$pattern($scope, void 0 !== $foo ? $foo : {
+    bar: 2
+  });
+};
+const $ChildA_content__foo = /* @__PURE__ */_._const("foo", ($scope, foo) => {
+  _._text($scope["#text/1"], typeof foo);
+  $ChildA_content__$foo($scope, foo);
+});
+const $ChildA_content__$params = /* @__PURE__ */_._const("$params2", ($scope, $params2) => $ChildA_content__$temp($scope, $params2?.[0]));
+const $ChildA_content__$temp = /* @__PURE__ */_._const("$temp", ($scope, $temp) => $ChildA_content__foo($scope, $temp.foo));
+export function $setup($scope) {
+  $ChildA_content__foo($scope["#childScope/0"], {
+    bar: 0
+  });
+  $ChildA_content__foo($scope["#childScope/1"], {});
+  $ChildA_content__foo($scope["#childScope/2"], void 0);
+  $ChildB_content__foo($scope["#childScope/3"], {
+    bar: 0
+  });
+  $ChildB_content__foo($scope["#childScope/4"], {});
+  $ChildB_content__foo($scope["#childScope/5"], void 0);
+}
+export default /* @__PURE__ */_._template("__tests__/template.marko", $template, $walks, $setup);