Skip to content

fix: update all non-major dependencies - #580

Merged
kylehoehns merged 2 commits into
mainfrom
fix/non-major-deps-with-parcel-watcher
Sep 9, 2026
Merged

fix: update all non-major dependencies#580
kylehoehns merged 2 commits into
mainfrom
fix/non-major-deps-with-parcel-watcher

Conversation

@kylehoehns

Copy link
Copy Markdown
Contributor

This does the same dependency updates as #575, plus a one-line config fix that makes the build pass. Open this instead of #575.

Why #575 is stuck

#575 bumps jest from 30.4.2 to 30.5.1. That new version of jest depends on a package called @parcel/watcher, which runs a build script when it installs.

We use pnpm 11. pnpm 11 refuses to run a package's build script unless you have listed that package as allowed. If it finds one that isn't listed, it stops with an error instead of just warning you. So pnpm install fails and the Verify Code check goes red:

[ERR_PNPM_IGNORED_BUILDS] Ignored build scripts: @parcel/watcher@2.6.0

Renovate will not merge a pull request whose checks are failing, so #575 has been sitting there.

The fix

Add @parcel/watcher to the allow lists in pnpm-workspace.yaml:

onlyBuiltDependencies:
  - '@parcel/watcher'
  - esbuild
  - unrs-resolver
allowBuilds:
  '@parcel/watcher': true
  esbuild: true
  unrs-resolver: true

It goes in both lists because the two lists are for two different pnpm versions. pnpm 11 reads allowBuilds. pnpm 10 reads the older onlyBuiltDependencies. Listing it twice means someone still running pnpm 10 locally gets the same result as CI. The existing entries follow the same pattern.

How I checked

Ran pnpm verify locally on this branch using pnpm 11, the same version CI uses:

  • Install succeeded, and @parcel/watcher built.
  • Lint, format check, type-check, and build all passed.
  • Tests: 34 suites passed, 265 tests passed.

Before the fix, the same command failed at the install step with the error above.

One thing to know

I have not changed anything about how @parcel/watcher is used. Nothing imports it directly. Jest pulls it in for watch mode. Allowing its build script only lets it compile its native helper during install, which is what it already does in every other project that uses jest.

After this merges

Close #575. Renovate will see the dependencies are already up to date and will not reopen it.

🤖 Generated with Claude Code

https://claude.ai/code/session_019dUo24dvyCizU9GbU9dTww

renovate Bot and others added 2 commits September 9, 2026 14:00
The jest 30.4.2 -> 30.5.1 bump in this branch pulls in @parcel/watcher
via jest-haste-map. pnpm 11 hard-errors on any package whose build
scripts it has not been told to run, so `pnpm install --frozen-lockfile`
fails the Verify Code check with:

    [ERR_PNPM_IGNORED_BUILDS] Ignored build scripts: @parcel/watcher@2.6.0

Add it to both lists for the same reason the existing entries are in
both: pnpm 11 reads allowBuilds, pnpm 10 reads onlyBuiltDependencies,
and contributors may still be on 10.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019dUo24dvyCizU9GbU9dTww
@kylehoehns
kylehoehns merged commit 85f7f05 into main Sep 9, 2026
1 check passed
@kylehoehns
kylehoehns deleted the fix/non-major-deps-with-parcel-watcher branch September 9, 2026 14:29
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