chore: fix playwright test timeouts and VRT configurations

Resolves PR #9643 feedback:
- Remove misleading `// Retry failed tests on CI` comment in playwright config.
- Use `process.env.CI_RETRIES` environment variable over a hardcoded `0` for fallback retry configurations.
- Drop `page.waitForLoadState('networkidle')` from `waitForVRTReady` which was previously causing 15000ms timeout failures in CI because of WebSocket and persistent connections.
- Fix race condition causing `stop-timer-button` click timeout in `vrt-timer-controls.spec.ts` since `stopTimer` in `test.afterEach` is now natively responsible for timer cleanup.

Co-authored-by: arii <342438+arii@users.noreply.github.com>

Author: google-labs-jules[bot]

playwright.config.ts

@@ -77,7 +77,7 @@ export default defineConfig({
   // Fail build on CI if you accidentally left test.only
   forbidOnly: !!process.env.CI,
 
-  retries: 0,
+  retries: process.env.CI_RETRIES ? parseInt(process.env.CI_RETRIES) : 0,
 
   // Test execution optimizations - Fail Fast Strategy
   expect: {

tests/playwright/lib/visual.ts

@@ -44,7 +44,6 @@ export async function waitForVRTReady(
   targetWidth?: number
 ): Promise<void> {
   await page.evaluateHandle(() => document.fonts.ready)
-  await page.waitForLoadState('networkidle')
   await page.evaluate(() => document.body.offsetHeight)
   if (targetWidth !== undefined) {
     await page.waitForFunction(

tests/playwright/vrt-dashboard.spec.ts-snapshots/dashboard-mobile-chromium-linux.png

@@ -77,9 +77,6 @@ test.describe('Visual Regression Tests', () => {
       await takeScreenshot(timerControls, 'timer-controls-active.png', {
         mask: [controlPage.getByTestId('timer-countdown')],
       })
-
-      // Stop the timer to reset for the next test
-      await controlPage.getByTestId('stop-timer-button').click()
     })
 
     test('start button hover state', async () => {