Skip to content

chore: pin puppeteer to a single version for GHA#3419

Merged
kumilingus merged 2 commits into
clientIO:masterfrom
zbynekstara:puppeteer-version-fix
Jul 7, 2026
Merged

chore: pin puppeteer to a single version for GHA#3419
kumilingus merged 2 commits into
clientIO:masterfrom
zbynekstara:puppeteer-version-fix

Conversation

@zbynekstara

@zbynekstara zbynekstara commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Description

Resolves an intermittent issue with the publish-joint-react-docs.yml GitHub Action caused by grunt-contrib-qunit@7.0.1 depending on puppeteer@^19.7.0 (which gets resolved as 19.11.1), which is different from the puppeteer everything else pulls in (24.22.0). This triggers a second download of Chrome.

Yarn link step runs these two donwloads concurrently, causing a race condition between the 24.22.0 installer and the 19.11.1 installer. If the two installs overlap at the wrong instant, the action fails.

grunt-contrib-qunit just needs a working puppeteer to drive headless QUnit, and its API (puppeteer.launch) is stable across 19→24 of Puppeteer.

Copilot AI 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.

Pull request overview

This PR aims to eliminate intermittent failures in the Publish Joint React Docs GitHub Action by ensuring only a single Puppeteer version is resolved, preventing concurrent Chromium downloads/installs during yarn install.

Changes:

  • Adds a root-level Yarn resolutions override intended to force grunt-contrib-qunit’s Puppeteer dependency to use puppeteer@24.22.0.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json

Copilot AI 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.

Pull request overview

Copilot reviewed 1 out of 2 changed files in this pull request and generated no new comments.

@kumilingus
kumilingus self-requested a review July 7, 2026 16:25
@kumilingus
kumilingus merged commit 8e81238 into clientIO:master Jul 7, 2026
3 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.

3 participants