Commit 935cb58
authored
fix: build fails on Linux/Intel macOS due to missing native optional deps (#583)
* fix: declare platform-native optional deps so the frontend build resolves
The Linux (and macOS Intel) build of the frontend web bundle failed with:
Cannot find module '../lightningcss.linux-x64-gnu.node'
The committed lockfile only carried native package nodes for darwin-arm64
(it was generated on Apple Silicon). On any other platform the required
native binaries for lightningcss and @tailwindcss/oxide have no resolution
entry in the lockfile, so npm silently skips them (they're optional),
breaking the Tailwind/webpack build in npm ci and npm install alike
(npm/cli#4828).
Declare the needed platform binaries as direct optionalDependencies so npm
records their nodes in the lockfile and installs them reliably on each
matching platform. Covers the platforms this repo actually builds on:
linux-x64-gnu (build_linux.sh / build_deb.sh, _build-linux.yml) and macOS
arm64/x64 (build_dmg.sh, _build-dmg.yml). Their os/cpu constraints mean each
installs only where it applies.
Verified: a clean `npm ci` now installs the Linux natives and the frontend
bundle builds end to end.
* chore: add changeset for Linux/Intel macOS native deps fix1 parent 38e08f2 commit 935cb58
3 files changed
Lines changed: 138 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
63 | 63 | | |
64 | 64 | | |
65 | 65 | | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
66 | 74 | | |
67 | 75 | | |
68 | 76 | | |
| |||
0 commit comments