Skip to content

Conversation

@christian-byrne
Copy link
Contributor

@christian-byrne christian-byrne commented Aug 30, 2025

Summary

Fix TypeScript compilation error in i18n collection workflow by updating playwright.i18n.config.ts to use modern defineConfig API.

Problem

The i18n workflow was failing with:

SyntaxError: TypeScript 'declare' fields must first be transformed by @babel/plugin-transform-typescript

This occurred because playwright.i18n.config.ts was using the older PlaywrightTestConfig interface instead of defineConfig() which provides proper Vite integration for TypeScript compilation.

Solution

  • Replace PlaywrightTestConfig interface with defineConfig() API
  • Maintains all existing i18n collection functionality
  • Aligns with main playwright.config.ts which already uses defineConfig

Fixes the remaining issue after PR #5266

┆Issue is synchronized with this Notion page by Unito

…mpilation

Update playwright.i18n.config.ts to use defineConfig() instead of PlaywrightTestConfig
interface to fix TypeScript compilation issues with 'declare' fields.

The defineConfig API provides proper Vite integration for TypeScript/Babel
compilation, fixing the error: 'TypeScript declare fields must first be
transformed by @babel/plugin-transform-typescript'

Fixes compilation error in i18n collection workflow.
@christian-byrne christian-byrne requested a review from a team as a code owner August 30, 2025 22:07
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 30, 2025
@github-actions
Copy link

github-actions bot commented Aug 30, 2025

🎨 Storybook Build Status

Build failed!

⏰ Completed at: 08/30/2025, 10:07:34 PM UTC

🔗 Links


⚠️ Please check the workflow logs for error details.

@github-actions
Copy link

github-actions bot commented Aug 30, 2025

🎭 Playwright Test Results

All tests passed across all browsers!

⏰ Completed at: 08/30/2025, 10:30:41 PM UTC

📊 Test Reports by Browser


🎉 Your tests are passing across all browsers!

@christian-byrne
Copy link
Contributor Author

Merging so I can unblock release. Feel free to review/comment still and I'll address in followup.

@christian-byrne christian-byrne merged commit 2e72988 into main Aug 30, 2025
17 checks passed
@christian-byrne christian-byrne deleted the fix/i18n-playwright-defineconfig branch August 30, 2025 22:33
@benceruleanlu benceruleanlu mentioned this pull request Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:testing size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants