fix(oxc): fall back to wasm#921
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview The lockfile reflects that shift: WASM-related packages are no longer marked optional at the root install graph, while Reviewed by Cursor Bugbot for commit 8fed169. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 8fed169. Configure here.
| "@node-core/ui-components": "^1.7.0", | ||
| "@orama/orama": "^3.1.18", | ||
| "@orama/ui": "^1.5.4", | ||
| "@oxc-parser/binding-wasm32-wasi": "^0.140.0", |
There was a problem hiding this comment.
WASM binding version mismatch
High Severity
The new direct dependency on @oxc-parser/binding-wasm32-wasi is pinned to ^0.140.0 while oxc-parser remains at ^0.139.0. OXC expects parser and platform bindings to share the same release. When the optional nested 0.139.0 WASM package is not installed, Node resolves the hoisted 0.140.0 binding for a 0.139.0 parser—the exact fallback path this change targets.
Reviewed by Cursor Bugbot for commit 8fed169. Configure here.
|
cc @Boshen howdy from Node.js! Any idea on how to allow a WASM fallback without npm throwing invalid architecture errors? |
|
In the meantime, I'm going to swap us for a WASM parser like swc or yuku |
|
https://stackblitz.com/edit/oxc-parser?file=package.json miss click and I cannot discard my approval |
|
|
That wouldn't help, would it? Our consumer (node core) runs |
|
Wait ... why do you need this fallback when Which architecture are we missing, I can add those instead. |
I assumed (possibly incorrectly) that by manually installing the WASM binding, oxc would 'fall back' to it if no bindings were found.
Thank you, that would be great (if it's not too much work, of course)! In our CI, the following runners failed:
If this is too difficult, we can also throw an error that doc generation is not available on these machines (cc @nodejs/web-infra wdyt?). Although, adding these architectures would allow us to use |


As titled