Skip to content

fix(renovate): skip browser downloads during artifact update - #884

Draft
l2ysho wants to merge 1 commit into
masterfrom
claude/dep-triage-c08844
Draft

fix(renovate): skip browser downloads during artifact update#884
l2ysho wants to merge 1 commit into
masterfrom
claude/dep-triage-c08844

Conversation

@l2ysho

@l2ysho l2ysho commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Every Renovate PR that bumps a package.json dependency (puppeteer, eslint, typescript, patch/minor group, camoufox-js, lock file maintenance, etc.) is failing CI with ERR_PNPM_OUTDATED_LOCKFILE, because the renovate/artifacts step never manages to write back an updated pnpm-lock.yaml.
  • Confirmed locally: checking out the affected branches and running pnpm install --no-frozen-lockfile resolves cleanly and produces a valid lockfile — the dependency graph is fine, so the problem is in Renovate's own environment, not the repo.
  • Root cause: puppeteer/playwright postinstall scripts download a full browser binary, which appears to fail (network-restricted sandbox) inside Renovate's own artifact-update run.
  • Fix: set PUPPETEER_SKIP_DOWNLOAD / PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD via Renovate's env config. This only affects the install command Renovate itself runs to compute the lockfile — real CI workflows are untouched and still download real browsers for tests. Verified the resulting lockfile is identical with or without the download.

Test plan

…update

pnpm/npm's postinstall hook for puppeteer and playwright downloads a
browser binary, which appears to fail in Renovate's own sandbox and
breaks its lockfile regeneration step (renovate/artifacts). Every PR
that bumps a package.json dependency then fails CI with
ERR_PNPM_OUTDATED_LOCKFILE, even though pnpm install resolves fine
locally. These env vars only apply to Renovate's own install command,
not real CI, so tests still exercise real browsers.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@github-actions github-actions Bot added this to the 146th sprint - Tooling team milestone Aug 7, 2026
@github-actions github-actions Bot added the t-tooling Issues with this label are in the ownership of the tooling team. label Aug 7, 2026
@l2ysho l2ysho added adhoc Ad-hoc unplanned task added during the sprint. t-builders Issues owned by the Builders team. low priority Low priority issues to be done eventually. and removed t-tooling Issues with this label are in the ownership of the tooling team. labels Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. low priority Low priority issues to be done eventually. t-builders Issues owned by the Builders team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants