Skip to content

fix(ci): flox upgrade to resolve nodejs package conflict - #43

Closed
garbas wants to merge 1 commit into
mainfrom
fix/flox-nodejs-conflict
Closed

fix(ci): flox upgrade to resolve nodejs package conflict#43
garbas wants to merge 1 commit into
mainfrom
fix/flox-nodejs-conflict

Conversation

@garbas

@garbas garbas commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Problem

The Lighthouse CI is failing for every PR (including #42) at the Activate Flox environment step:

❌ ERROR: 'nodejs' conflicts with 'nodejs'.
Both packages provide the file 'include/node/common.gypi'

The manifest installs nodejs explicitly and clean-css-cli (which bundles its own nodejs). Both resolved at priority 5 and collide, so the environment fails to build. This is broken on main, which means the CI's base-branch comparison step also fails — no PR can go green until main is fixed.

Fix

Ran flox upgrade, which re-locks the environment and resolves the conflict. It also bumps a few packages:

  • google-lighthouse 12.8.2 -> 13.3.0
  • nodejs 22.21.1 -> 24.15.0
  • just 1.43.1 -> 1.51.0
  • imagemagick 7.1.2-8 -> 7.1.2-23

Verification

Locally, flox activate now builds successfully and node, lighthouse, just and cleancss all resolve (node v24.15.0, lighthouse 13.3.0).

Only .flox/env/manifest.{toml,lock} change — no site content touched.

The Lighthouse CI fails at 'flox activate' with:
  'nodejs' conflicts with 'nodejs' — both packages provide
  include/node/common.gypi

The explicit nodejs install and the nodejs bundled by clean-css-cli
both resolved at priority 5 and collided, breaking the environment on
main. Running 'flox upgrade' re-locks the environment and resolves the
conflict, also bumping:
  google-lighthouse 12.8.2 -> 13.3.0
  nodejs            22.21.1 -> 24.15.0
  just              1.43.1  -> 1.51.0
  imagemagick       7.1.2-8 -> 7.1.2-23

Verified locally: 'flox activate' now builds and node resolves to
v24.15.0.
@netlify

netlify Bot commented Jun 12, 2026

Copy link
Copy Markdown

Deploy Preview for planetnix ready!

Name Link
🔨 Latest commit e0a974b
🔍 Latest deploy log https://app.netlify.com/projects/planetnix/deploys/6a2c4d255eec020008c44e46
😎 Deploy Preview https://deploy-preview-43--planetnix.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@garbas

garbas commented Jun 12, 2026

Copy link
Copy Markdown
Contributor Author

Superseded — folding the Flox env fix into #42 (per request to keep it in one PR). #42 also patches the Lighthouse workflow to measure the base branch with the PR's fixed Flox env, so it no longer depends on main being fixed first.

@garbas garbas closed this Jun 12, 2026
@garbas
garbas deleted the fix/flox-nodejs-conflict branch June 12, 2026 18:21
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