Skip to content

Fix/upgrade playwright go v0.6100.0 - #1532

Open
danilodeLuca wants to merge 4 commits into
Versent:masterfrom
danilodeLuca:fix/upgrade-playwright-go-v0.6100.0
Open

Fix/upgrade playwright go v0.6100.0#1532
danilodeLuca wants to merge 4 commits into
Versent:masterfrom
danilodeLuca:fix/upgrade-playwright-go-v0.6100.0

Conversation

@danilodeLuca

@danilodeLuca danilodeLuca commented Jul 9, 2026

Copy link
Copy Markdown

Fixes: #1531

Problem
Versions of playwright-go before v0.6100.0 download the Playwright driver from playwright.azureedge.net/builds/driver/..., which Microsoft retired on 2026-07-08. That causes 404s on fresh installs — the same issue reported in antigravity-cli#566.

Changes made
Upgraded dependency from github.com/playwright-community/playwright-go v0.4702.0 → github.com/mxschmitt/playwright-go v0.6100.0

v0.6100.0 assembles the driver from npm (playwright-core) + nodejs.org instead of the deprecated CDN (playwright-go#615)
The module path moved from playwright-community to mxschmitt in this release

Updated imports in:

  • pkg/provider/browser/browser.go
  • pkg/provider/browser/browser_test.go
  • mocks/Page.go, mocks/Request.go, mocks/Response.go

Fixed API break — StorageState() now takes options instead of a plain path string:

browser.go
Lines 139-142
		_, err := context.StorageState(playwright.BrowserContextStorageStateOptions{
			Path: playwright.String(storageStatePath),
		})

Updated README link to the new module location.

danilodeLuca and others added 3 commits July 9, 2026 13:47
Microsoft retired Playwright driver zip distribution on all CDN endpoints,
breaking fresh installs for browser IDP users on playwright-go < v0.6100.0.

Co-authored-by: Cursor <cursoragent@cursor.com>
Comment thread go.mod Outdated
v0.6000.0 still uses the retired driver CDN; only v0.6100.0+
assembles the driver from npm + nodejs.org. That release requires
the mxschmitt module path.

Co-authored-by: Cursor <cursoragent@cursor.com>

@leonidasKap leonidasKap left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

lgtm - I 'd recommend a local build and smoke test. Thank you

@danilodeLuca

Copy link
Copy Markdown
Author

@leonidasKap Thanks — did a local build and smoke test on this branch (mxschmitt/playwright-go v0.6100.0):

Build

  • go build -o bin/saml2aws ./cmd/saml2aws succeeded
  • Binary starts (--version / --help OK)

Tests

  • go test ./pkg/provider/browser/... passed

Playwright smoke (CDN fix) Fresh driver dir (no existing cache):

  • playwright.Install() succeeded (driver assembled via npm + nodejs.org)
  • Headless Chromium launch + page navigation succeeded
  • Confirmed the old CDN is still dead (playwright.azureedge.net/builds/driver/... → 404)

Looks good on my side.

@dosmanak

Copy link
Copy Markdown

Hi. What is the blocker here?

@danilodeLuca

Copy link
Copy Markdown
Author

@dosmanak i dont reallt know, we are waiting for some approver/committer to give a review on it.

@raulbtmonteiro

raulbtmonteiro commented Sep 2, 2026

Copy link
Copy Markdown

Hi @tinaboyce @srizzling @leonidasKap ! Do you know if there is something missing for this PR to get a review?

@srizzling

Copy link
Copy Markdown
Contributor

Hey sorry I've left Versent.. so don't have rights to merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

5 participants