Skip to content

CI: pin Playwright for the audit, move to Node 22 - #8

Merged
Kninsay merged 1 commit into
mainfrom
fix/ci-audit
Nov 2, 2026
Merged

Kninsay merged 1 commit into
mainfrom
fix/ci-audit

Conversation

@Kninsay

@Kninsay Kninsay commented Nov 2, 2026

Copy link
Copy Markdown
Collaborator

The first run on main failed at the audit step: playwright was not a dependency, so scripts/audit.mjs could not import it on the runner.

  • Pin playwright 1.63.0 as a dev dependency and install its browser with npx playwright install, so the browser build always matches the library.
  • Node 20 is past end of life. .node-version and engines move to 22, which Cloudflare Pages also reads.

Checked locally: build (116 pages, privacy scan and link check clean) and audit pass.


Generated by Claude Code

The audit step failed on GitHub because playwright was not a dependency.
Pin it to the exact version used locally so the browser build matches.
Node 20 is past end of life; build on Node 22 here and on Cloudflare Pages.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Gnmfunw3PUpLkhSGZWPpfn
@Kninsay
Kninsay merged commit f80beaf into main Nov 2, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants