fix: cached values inside attribute tags

.changeset/nine-pumas-raise.md

@@ -0,0 +1,5 @@
+---
+"@marko/tags-api-preview": patch
+---
+
+Fix issue with cached values inside attribute tags.

src/__tests__/fixtures/const/__snapshots__/const-mutation-error/web.render.error.expected.txt

@@ -1,2 +1 @@
-TypeError: Cannot add property fullName, object is not extensible
-    at /src/__tests__/fixture.ts
+TypeError: Cannot add property fullName, object is not extensible

src/__tests__/fixtures/get-and-set/__snapshots__/get-read-assign-to-missing-context/web.step-0.error.expected.txt

@@ -1,4 +1 @@
-Error: Assignment to constant variable.
-    at /src/__tests__/fire-event.ts
-    at /src/__tests__/fixtures/get-and-set/index.test.ts
-    at /src/__tests__/fixture.ts
+Error: Assignment to constant variable.

src/__tests__/fixtures/get-and-set/__snapshots__/get-set-assign-to-const-set/web.step-0.error.expected.txt

@@ -1,4 +1 @@
-Error: Assignment to constant variable.
-    at /src/__tests__/fire-event.ts
-    at /src/__tests__/fixtures/get-and-set/index.test.ts
-    at /src/__tests__/fixture.ts
+Error: Assignment to constant variable.

src/__tests__/fixtures/get-and-set/__snapshots__/get-set-assign-to-nested-const-set/web.step-0.error.expected.txt

@@ -1,4 +1 @@
-Error: Assignment to constant variable.
-    at /src/__tests__/fire-event.ts
-    at /src/__tests__/fixtures/get-and-set/index.test.ts
-    at /src/__tests__/fixture.ts
+Error: Assignment to constant variable.

src/__tests__/fixtures/misc/attribute-tags/__snapshots__/misc-attribute-tags/single/node.render.expected.html

@@ -0,0 +1,18 @@
+<div
+  data-key="hello"
+  name="Frank"
+>
+  Hello Frank!
+</div>
+<div
+  data-key="hello"
+  name="Frank"
+>
+  Hello Frank!
+</div>
+<div
+  data-key="hello"
+  name="John"
+>
+  Hello John!
+</div>

src/__tests__/fixtures/misc/attribute-tags/__snapshots__/misc-attribute-tags/single/web.render.expected.html

@@ -0,0 +1,18 @@
+<div
+  data-key="hello"
+  name="Frank"
+>
+  Hello Frank!
+</div>
+<div
+  data-key="hello"
+  name="Frank"
+>
+  Hello Frank!
+</div>
+<div
+  data-key="hello"
+  name="John"
+>
+  Hello John!
+</div>

src/__tests__/fixtures/misc/attribute-tags/components/test.marko

@@ -0,0 +1,17 @@
+<!-- use tags -->
+<for|key, value| in=input>
+  <if=value>
+    <if=Array.isArray(value)>
+      <for|item| of=value>
+        <div data-key=key ...item>
+          <${item}/>
+        </div>
+      </for>
+    </if>
+    <else>
+      <div data-key=key ...value>
+        <${value}/>
+      </div>
+    </else>
+  </if>
+</for>

src/__tests__/fixtures/misc/attribute-tags/index.test.ts

@@ -0,0 +1,5 @@
+import fixture from "../../../fixture";
+
+describe("misc attribute tags", () => {
+  describe("single", fixture("./templates/single.marko"));
+});

src/__tests__/fixtures/misc/attribute-tags/templates/single.marko

@@ -0,0 +1,17 @@
+<!-- use tags -->
+<test>
+  <@hello name="Frank" onClick() {
+    console.log('Hello Frank!');
+  }>
+    Hello Frank!
+  </@hello>
+</test>
+<test>
+  <for|name| of=["Frank", "John"]>
+    <@hello name=name onClick() {
+      console.log('Hello ' + name + '!');
+    }>
+      Hello ${name}!
+    </@hello>
+  </for>
+</test>

src/__tests__/fixtures/misc/hoisting/__snapshots__/misc-hoisting/error-maybe-sync-read-before-with-custom-tag/node.render.error.expected.txt

@@ -1,4 +1 @@
-ReferenceError: Cannot access 'x' before initialization
-    at /src/transform/hoist-tag-vars/index.ts
-    at /src/__tests__/fixtures/misc/hoisting/templates/error-hoisting-maybe-sync-read-before-with-custom-tag.marko
-    at /src/__tests__/fixture.ts
+ReferenceError: Cannot access 'x' before initialization

src/__tests__/fixtures/misc/hoisting/__snapshots__/misc-hoisting/error-maybe-sync-read-before-with-custom-tag/web.render.error.expected.txt

@@ -1,2 +1 @@
-ReferenceError: Cannot access 'x' before initialization
-    at /src/__tests__/fixtures/misc/hoisting/templates/error-hoisting-maybe-sync-read-before-with-custom-tag.marko
+ReferenceError: Cannot access 'x' before initialization

src/__tests__/fixtures/misc/hoisting/__snapshots__/misc-hoisting/error-maybe-sync-read-before/node.render.error.expected.txt

@@ -1,5 +1 @@
-ReferenceError: Cannot access 'x' before initialization
-    at /src/transform/hoist-tag-vars/index.ts
-    at /src/__tests__/fixtures/misc/hoisting/templates/error-hoisting-maybe-sync-read-before.marko
-    at /src/__tests__/fixtures/misc/hoisting/templates/error-hoisting-maybe-sync-read-before.marko
-    at /src/__tests__/fixture.ts
+ReferenceError: Cannot access 'x' before initialization

src/__tests__/fixtures/misc/hoisting/__snapshots__/misc-hoisting/error-maybe-sync-read-before/web.render.error.expected.txt

@@ -1,3 +1 @@
-ReferenceError: Cannot access 'x' before initialization
-    at /src/__tests__/fixtures/misc/hoisting/templates/error-hoisting-maybe-sync-read-before.marko
-    at /src/__tests__/fixtures/misc/hoisting/templates/error-hoisting-maybe-sync-read-before.marko
+ReferenceError: Cannot access 'x' before initialization

src/__tests__/fixtures/misc/native-tag-var/__snapshots__/misc-native-tag-vars/error-read-before-mount/node.render.error.expected.txt

@@ -1,2 +1 @@
-Error: Cannot read an element reference while rendering.
-    at /src/__tests__/fixture.ts
+Error: Cannot read an element reference while rendering.

src/__tests__/fixtures/misc/native-tag-var/__snapshots__/misc-native-tag-vars/error-read-before-mount/web.render.error.expected.txt

@@ -1,2 +1 @@
-Error: Cannot read an element reference while rendering.
-    at /src/__tests__/fixture.ts
+Error: Cannot read an element reference while rendering.

src/__tests__/snapshot.ts

@@ -144,8 +144,7 @@ function format(data: any): string {
           // eslint-disable-next-line no-control-regex
           .replace(/\x1B[[(?);]{0,2}(;?\d)*./g, "")
           .replaceAll(process.cwd(), "")
-          .replace(/\r?\n +at (?![/\\]src[/\\])[^\n]+$/gm, "")
-          .replace(/(\r?\n +at [/\\]src[/\\].*[^\n]):\d+:\d+$/gm, "$1")
+          .replace(/\r?\n +at [^\n]+$/gm, "")
       );
     }
   }

src/transform/wrapper-component.ts

@@ -1,5 +1,11 @@
 import { types as t } from "@marko/compiler";
-import { isNativeTag, getTagDef } from "@marko/babel-utils";
+import {
+  isNativeTag,
+  isTransparentTag,
+  isAttributeTag,
+  findParentTag,
+  getTagDef,
+} from "@marko/babel-utils";
 import { taglibId } from "../util/taglib-id";
 import isApi from "../util/is-api";
 
@@ -96,6 +102,14 @@ export function ensureLifecycle(tag: t.NodePath<t.MarkoTag>, client = true) {
     isNativeTag(root as t.NodePath<t.MarkoTag>)
   );
 
+  if (isAttributeTagLike(tag)) {
+    if (isTransparentTag(root as t.NodePath<t.MarkoTag>)) {
+      root = findParentTag(root as t.NodePath<t.MarkoTag>) || program;
+    }
+
+    root = findParentTag(root as t.NodePath<t.MarkoTag>) || program;
+  }
+
   if (root.node) {
     const roots = lifecycleRootsForProgram.get(program)!;
     let extra = root.node.extra;
@@ -182,3 +196,19 @@ function buildNestedLifecycle(tag: t.NodePath<t.MarkoTag>): t.Statement[] {
 
   return result;
 }
+
+function isAttributeTagLike(tag: t.NodePath<t.MarkoTag>) {
+  if (isAttributeTag(tag)) {
+    return true;
+  }
+
+  if (isTransparentTag(tag)) {
+    for (const child of tag.get("body").get("body")) {
+      if (child.isMarkoTag() && isAttributeTagLike(child)) {
+        return true;
+      }
+    }
+  }
+
+  return false;
+}