Skip to content

refactor(@angular/build): remove oxc WASI shim - #34646

Merged
dgp1130 merged 1 commit into
angular:mainfrom
alan-agius4:refactor/remove-oxc-wasi-shim
Nov 3, 2026
Merged

dgp1130 merged 1 commit into
angular:mainfrom
alan-agius4:refactor/remove-oxc-wasi-shim

Conversation

@alan-agius4

Copy link
Copy Markdown
Collaborator

The upstream issue with @napi-rs/wasm-runtime and @emnapi/core WASI fallback resolution has been fixed in @napi-rs/wasm-runtime@1.2.7 (napi-rs/napi-rs#3680), making the temporary WASI shim no longer necessary.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes the oxc-wasi-shim.ts file and its associated imports across several files, including the i18n inliner worker, OXC transform, and traversal tools. This shim was originally introduced as a workaround for upstream WebAssembly/WASI binding incompatibilities. There are no review comments to evaluate, so I have no feedback to provide.

@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Nov 3, 2026
@alan-agius4
alan-agius4 requested a review from dgp1130 November 3, 2026 17:38
The upstream issue with @napi-rs/wasm-runtime and @emnapi/core WASI fallback resolution has been fixed in @napi-rs/wasm-runtime@1.2.7 (napi-rs/napi-rs#3680), making the temporary WASI shim no longer necessary.
@alan-agius4
alan-agius4 force-pushed the refactor/remove-oxc-wasi-shim branch from 2578c24 to 5affb76 Compare November 3, 2026 17:50
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Nov 3, 2026
@dgp1130
dgp1130 merged commit d9e3e22 into angular:main Nov 3, 2026
40 checks passed
@dgp1130

dgp1130 commented Nov 3, 2026

Copy link
Copy Markdown
Collaborator

This PR was merged into the repository. The changes were merged into the following branches:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: @angular/build target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants