Skip to content

fix: don't rely on node_modules to detect @playwright/test version#1228

Open
hwartig wants to merge 2 commits intomainfrom
harry/red-167-browsers-launch-error
Open

fix: don't rely on node_modules to detect @playwright/test version#1228
hwartig wants to merge 2 commits intomainfrom
harry/red-167-browsers-launch-error

Conversation

@hwartig
Copy link
Contributor

@hwartig hwartig commented Feb 18, 2026

The version of @playwright/test in node_modules might be out of sync with what is required in package.json/package-lock.json. In those rare cases the cli will report an incorrect playwright version to the API which will cause checks to fail as checks rely on the playwright version noted in lock files.

Affected Components

  • CLI
  • Create CLI

New Dependency Submission

Added snyk-nodejs-lockfile-parser to parse lockfiles to detect installed playwright version.

The version of @playwright/test in node_modules might be out of sync
with what is required in package.json/package-lock.json. In those rare
cases the cli will report an incorrect playwright version to the API
which will cause checks to fail as checks rely on the playwright version
noted in lock files.
@hwartig hwartig force-pushed the harry/red-167-browsers-launch-error branch from 3c1da33 to fc2c64c Compare February 18, 2026 19:22
@hwartig hwartig requested a review from sorccu February 18, 2026 20:27
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.

1 participant

Comments