Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

packages/vue/__tests__/e2e/Transition.spec.ts (1)

1438-1477: ⚠️ Potential issue | 🟡 Minor

Always detach pageerror listener via try/finally.

If any assertion throws before Line 1477, the listener remains attached and can leak into subsequent tests.

Proposed fix

         const spy = vi.fn()
         const currentPage = page()
         currentPage.on('pageerror', spy)

-        await page().evaluate(() => {
+        try {
+          await page().evaluate(() => {
           const { createApp, ref } = (window as any).Vue
           createApp({
             template: `
               <div id="container">
                 <transition name="test">
                   <template v-if="show" #>
                     <div class="test">text</div>
                   </template>
                 </transition>
               </div>
               <button id="toggleBtn" `@click`="show = !show">button</button>
             `,
             setup: () => {
               const show = ref(true)
               return { show }
             },
           }).mount('#app')
-        })
+          })

-        expect(await html('#container')).toBe('<div class="test">text</div>')
+          expect(await html('#container')).toBe('<div class="test">text</div>')

-        await click('#toggleBtn')
-        await nextTick()
-        await click('#toggleBtn')
+          await click('#toggleBtn')
+          await nextTick()
+          await click('#toggleBtn')

-        expect(
-          await page().$$eval('#container .test', nodes =>
-            nodes.map(node => node.className),
-          ),
-        ).toStrictEqual(['test test-enter-from test-enter-active'])
+          expect(
+            await page().$$eval('#container .test', nodes =>
+              nodes.map(node => node.className),
+            ),
+          ).toStrictEqual(['test test-enter-from test-enter-active'])

-        await nextFrame()
-        await transitionFinish()
-        await nextFrame()
+          await nextFrame()
+          await transitionFinish()
+          await nextFrame()

-        expect(spy).not.toHaveBeenCalled()
-        currentPage.off('pageerror', spy)
-        expect(await html('#container')).toBe('<div class="test">text</div>')
+          expect(spy).not.toHaveBeenCalled()
+          expect(await html('#container')).toBe('<div class="test">text</div>')
+        } finally {
+          currentPage.off('pageerror', spy)
+        }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@packages/vue/__tests__/e2e/Transition.spec.ts` around lines 1438 - 1477, Wrap
the lifecycle of the pageerror listener (currentPage.on('pageerror', spy)) in a
try/finally inside the test so the listener is always removed with
currentPage.off('pageerror', spy) in the finally block; update the test around
the lines that call currentPage.on/currentPage.off and assertions (where spy is
used) to ensure the try contains the evaluate/click/expect/transition steps and
the finally calls currentPage.off('pageerror', spy) to avoid leaking the
listener into other tests.