fix: migrate playwright-go to mxschmitt/playwright-go (CDN shutdown)

[kdacosta0]

Summary

• Migrate playwright-community/playwright-go → mxschmitt/playwright-go (one-line change)
• The Playwright /builds/driver CDN (azureedge) has been shut down — playwright install fails with 404
• The module moved to mxschmitt/playwright-go which downloads the driver from npm + nodejs.org instead (PR #615)

Related PRs

• sigstore-e2e: fix: migrate playwright-go to mxschmitt/playwright-go v0.6100.0 sigstore-e2e#80
• releases: https://github.com/securesign/releases/pull/291
• Jira: SECURESIGN-4984

🤖 Generated with Claude Code

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.06%. Comparing base (fac73b4) to head (0443851).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2003   +/-   ##
=======================================
  Coverage   54.06%   54.06%           
=======================================
  Files         235      235           
  Lines       14480    14480           
=======================================
  Hits         7828     7828           
  Misses       5838     5838           
  Partials      814      814           

Flag	Coverage Δ
e2e	76.36% <ø> (ø)
unit	28.98% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[qodo-for-securesign]

Code Review by Qodo

Sorry, something went wrong

We weren't able to complete the code review on our side. Please try again

[qodo-for-securesign]

PR Summary by Qodo

Fix CI Playwright install by switching to mxschmitt/playwright-go

🐞 Bug fix ⚙️ Configuration changes 🕐 Less than 10 minutes

AI Description

• Switch Playwright Go module used in CI to mxschmitt/playwright-go.
• Avoid failing playwright install due to the deprecated /builds/driver CDN.
• Keep e2e UI test setup unchanged aside from the install source.

Diagram

graph TD
  A["GitHub Actions: main.yml"] --> B["e2e job: Playwright install"] --> C["go run mxschmitt/playwright-go"] --> D["Download driver (npm/nodejs.org)"] --> E["Run Go e2e tests"]

Loading

High-Level Assessment

The following are alternative approaches to this PR:

1. Use Node Playwright install in CI (npm/npx)

• ➕ Aligns directly with upstream Playwright distribution and tooling
• ➕ Decouples driver install from Go module behavior
• ➖ Adds/expands Node toolchain usage in the workflow
• ➖ Requires adjusting current Go-based install step and potentially caching strategy

2. Vendor/cache Playwright drivers internally (artifact/mirror)

• ➕ Eliminates reliance on external availability and upstream distribution changes
• ➕ Improves reproducibility in restricted environments
• ➖ Operational overhead to mirror and update drivers
• ➖ More complex workflow and storage management

Recommendation: Proceed with the current approach: switching to mxschmitt/playwright-go is the lowest-change fix that restores playwright install after the CDN shutdown while keeping the existing Go-based workflow intact. Consider a mirrored/cached driver strategy only if external download reliability becomes an ongoing issue.

Files changed (1) +1 / -1

Other (1) +1 / -1

main.ymlUpdate Playwright install command to use mxschmitt/playwright-go +1/-1

Update Playwright install command to use mxschmitt/playwright-go

• Replaces the Go module path used for 'playwright install --with-deps' in the e2e workflow step. This avoids 404s caused by the deprecated Playwright driver CDN used by the previous module.

.github/workflows/main.yml