bug fix for lost view transition names introduced by 5.9.0 (#13907)

* bug fix for lost view transition names introduced by 5.9.0

* removed comment

Author: martrapp

.changeset/shy-showers-care.md

@@ -0,0 +1,5 @@
+---
+'astro': patch
+---
+
+Fixes a bug that caused view transition names to be lost.

packages/astro/e2e/view-transitions.test.js

@@ -738,8 +738,7 @@ test.describe('View Transitions', () => {
 		await expect(h, 'should have content').toHaveAttribute('style', 'background-color: green');
 		await expect(h, 'should have content').toHaveAttribute('data-other-name', 'value');
 		await expect(h, 'should have content').toHaveAttribute('data-astro-fake', 'value');
-		// TODO: check this assertion
-		// await expect(h, 'should have content').toHaveAttribute('data-astro-transition', 'forward');
+		await expect(h, 'should have content').toHaveAttribute('data-astro-transition', 'forward');
 		await expect(h, 'should have swap rest of data-astro-* attributes').toHaveAttribute(
 			'data-astro-transition-scope',
 			'scope-y',
@@ -1419,8 +1418,7 @@ test.describe('View Transitions', () => {
 		expect(loads.length, 'There should only be 1 page load').toEqual(1);
 	});
 
-	// TODO: investigate, it weirdly fails
-	test.skip('transition:name should be escaped correctly', async ({ page, astro }) => {
+	test('transition:name should be escaped correctly', async ({ page, astro }) => {
 		// view-transition-name errors on browser w/o native support
 		if (!(await nativeViewTransition(page))) return;
 		const expectedAnimations = new Set();
@@ -1584,8 +1582,7 @@ test.describe('View Transitions', () => {
 		await expect(p, 'should have content').toHaveText('Page 1');
 	});
 
-	// It weirdly fails
-	test.skip('animation get canceled when view transition is interrupted', async ({
+	test('animation get canceled when view transition is interrupted', async ({
 		page,
 		astro,
 	}) => {

packages/astro/package.json

@@ -113,7 +113,7 @@
     "test:integration": "astro-scripts test \"test/*.test.js\""
   },
   "dependencies": {
-    "@astrojs/compiler": "^2.12.0",
+    "@astrojs/compiler": "^2.12.1",
     "@astrojs/internal-helpers": "workspace:*",
     "@astrojs/markdown-remark": "workspace:*",
     "@astrojs/telemetry": "workspace:*",