Skip to content

fix(create-plugin): pin @playwright/test to 1.61.x - #2806

Merged
tolzhabayev merged 2 commits into
mainfrom
fix/pin-playwright-1-61
Jul 29, 2026
Merged

fix(create-plugin): pin @playwright/test to 1.61.x#2806
tolzhabayev merged 2 commits into
mainfrom
fix/pin-playwright-1-61

Conversation

@tolzhabayev

@tolzhabayev tolzhabayev commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

What this PR does / why we need it:

Playwright 1.62 made unresolvable tsconfig extends paths a fatal error (microsoft/playwright#41571). Its config loader can't resolve the package specifier in the scaffolded "extends": "@grafana/tsconfig", it only probes paths relative to the extending file, so playwright test now crashes on startup for every scaffolded plugin that floats to 1.62. Before 1.62 the unresolved extends was silently swallowed, which is why this never surfaced.

This pins @playwright/test to ~1.61.1 in the template. Existing plugins manage their own playwright version through renovate/dependabot and won't merge a broken bump, so no migration.

Upstream regression report: microsoft/playwright#41989, fixed by microsoft/playwright#42005 which hasn't shipped in a release yet. The pin comes off once it does, tracked in #2805.

Which issue(s) this PR fixes:

Part of #2805 (stays open to track removing the pin)

Special notes for your reviewer:

First seen in grafana/grafana-plugin-examples#721 where all five e2e matrix jobs fail with Failed to resolve "extends" path "@grafana/tsconfig". Verified by scaffolding a plugin from this branch: @playwright/test resolves to 1.61.1 and playwright test --list loads the config and tests cleanly.

@tolzhabayev tolzhabayev self-assigned this Jul 27, 2026
@tolzhabayev
tolzhabayev marked this pull request as ready for review July 27, 2026 10:24
@tolzhabayev
tolzhabayev requested a review from a team as a code owner July 27, 2026 10:24
@tolzhabayev
tolzhabayev requested review from eledobleefe and jackw and removed request for a team July 27, 2026 10:24
@grafana-catalog-project-bot grafana-catalog-project-bot Bot moved this from 📬 Triage to 🔬 In review in Grafana Catalog Team Jul 27, 2026
@tolzhabayev

Copy link
Copy Markdown
Collaborator Author

@jackw ptal

Comment thread packages/create-plugin/src/codemods/migrations/scripts/013-pin-playwright.ts Outdated
@tolzhabayev
tolzhabayev force-pushed the fix/pin-playwright-1-61 branch from 88988e4 to 1b9af99 Compare July 29, 2026 07:53
@tolzhabayev
tolzhabayev requested a review from jackw July 29, 2026 07:53
jackw
jackw previously approved these changes Jul 29, 2026

@jackw jackw left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

@jackw

jackw commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Urgh... the update command smoke test once again bites us! 😞 Shall we disable it for the time being?

@tolzhabayev

Copy link
Copy Markdown
Collaborator Author

Urgh... the update command smoke test once again bites us! 😞 Shall we disable it for the time being?

How about we pin it too for now instead of disabling it? 2500b77

@tolzhabayev
tolzhabayev requested a review from jackw July 29, 2026 10:08

@jackw jackw left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

@tolzhabayev
tolzhabayev merged commit 772bf50 into main Jul 29, 2026
33 checks passed
@tolzhabayev
tolzhabayev deleted the fix/pin-playwright-1-61 branch July 29, 2026 13:45
@github-project-automation github-project-automation Bot moved this from 🔬 In review to 🚀 Shipped in Grafana Catalog Team Jul 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🚀 Shipped

Development

Successfully merging this pull request may close these issues.

2 participants