Skip to content
This repository was archived by the owner on Mar 11, 2026. It is now read-only.

Fix: Issue with bundler wasm not loading in the browser environment#60

Merged
Wizdave97 merged 26 commits into
mainfrom
fix/bundle-wasm
Jun 4, 2025
Merged

Fix: Issue with bundler wasm not loading in the browser environment#60
Wizdave97 merged 26 commits into
mainfrom
fix/bundle-wasm

Conversation

@owonwo

@owonwo owonwo commented May 15, 2025

Copy link
Copy Markdown
Contributor
  • chore: lazy load wasm
  • chore: disabled cjs bundle in favour of esm and fixed issue with webpack loading wasm

@owonwo owonwo changed the title fix/bundle wasm Fix: Issue with bundler wasm not loading on the Web May 15, 2025
@owonwo owonwo requested a review from Wizdave97 May 15, 2025 14:56
@Wizdave97

Wizdave97 commented May 27, 2025

Copy link
Copy Markdown
Member

Let's fix the commit history here; Use merge strategy when pulling changes from main instead of rebase.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think we should split this wasm module into its own package entirely

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It imports correctly now, the wasm plugin for the vite test runner was missing initially.

@Wizdave97 Wizdave97 May 27, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Publishing it as a separate package means users might still have to set up plugins in some cases for the wasm to work correctly, whereas enforcing esm and node >= 23 means it would just work like every other package.

@owonwo owonwo force-pushed the fix/bundle-wasm branch from 0330087 to d82fe6a Compare May 27, 2025 13:53
@owonwo owonwo changed the title Fix: Issue with bundler wasm not loading on the Web Fix: Issue with bundler wasm not loading in the browser environment May 28, 2025
@owonwo

owonwo commented Jun 4, 2025

Copy link
Copy Markdown
Contributor Author

@Wizdave97 I've skipped failing test cases.

@Wizdave97 Wizdave97 merged commit d4efe49 into main Jun 4, 2025
1 check passed
@Wizdave97 Wizdave97 deleted the fix/bundle-wasm branch June 4, 2025 11:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants