Skip to content

fix: only bundle workspace package files for PlaywrightChecks#1220

Merged
sorccu merged 2 commits intomainfrom
simo/sim-191-lockfiles-should-not-be-included-for-browserchecks-and
Feb 12, 2026
Merged

fix: only bundle workspace package files for PlaywrightChecks#1220
sorccu merged 2 commits intomainfrom
simo/sim-191-lockfiles-should-not-be-included-for-browserchecks-and

Conversation

@sorccu
Copy link
Member

@sorccu sorccu commented Feb 12, 2026

Affected Components

  • CLI
  • Create CLI
  • Test
  • Docs
  • Examples
  • Other

Notes for the Reviewer

This PR changes the resolver to not include workspace package files (such as the lockfile) for checks running in restricted mode. All checks other than PlaywrightChecks run in restricted mode, and the workspace files bring no benefit there. They unnecessarily increase the payload size which can cause issues in repositories with extremely large lockfiles.

If you are seeing the following error when using CLI v7.0.0, you are affected by the issue:

Unable to run checks: Combined test length of setup script and test is too long. The returned setup variables or the test might be too long

Resolves #[issue-number]

New Dependency Submission

@sorccu sorccu added the build Issue regarding building and packaging label Feb 12, 2026
@github-actions
Copy link

🎉 Experimental release successfully published on npm

npm install checkly@0.0.0-pr.1220.e997535

@sorccu sorccu merged commit 5e0547e into main Feb 12, 2026
8 of 10 checks passed
@sorccu sorccu deleted the simo/sim-191-lockfiles-should-not-be-included-for-browserchecks-and branch February 12, 2026 19:16
sorccu added a commit that referenced this pull request Feb 12, 2026
* fix: only bundle workspace package files for PlaywrightChecks

* fix(tests): update tests for check types using restricted mode
sorccu added a commit that referenced this pull request Feb 12, 2026
* fix: only bundle workspace package files for PlaywrightChecks

* fix(tests): update tests for check types using restricted mode
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Issue regarding building and packaging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant