Skip to content

Conversation

@lcovar
Copy link
Contributor

@lcovar lcovar commented Nov 13, 2025

The exports field was preventing consumers from importing internal WASM wrapper files like 'dist/browser/js/wasm/wasm_utxo.js', which caused build failures in bitgo-ui during the migration from wasm-miniscript to wasm-utxo.

The wasm-miniscript package did not have an exports field, allowing all files to be accessible. Removing the exports field from wasm-utxo restores this behavior and allows bitgo-ui to use the existing webpack configuration without modifications.

This change enables:

  • Direct imports of WASM initialization wrapper files
  • Compatibility with existing build tooling that expects deep imports
  • Drop-in replacement for wasm-miniscript without code changes

Resolves module resolution errors when building bitgo-ui with wasm-utxo.

Ticket: BTC-0

The exports field was preventing consumers from importing internal
WASM wrapper files like 'dist/browser/js/wasm/wasm_utxo.js', which
caused build failures in bitgo-ui during the migration from
wasm-miniscript to wasm-utxo.

The wasm-miniscript package did not have an exports field, allowing
all files to be accessible. Removing the exports field from wasm-utxo
restores this behavior and allows bitgo-ui to use the existing webpack
configuration without modifications.

This change enables:
- Direct imports of WASM initialization wrapper files
- Compatibility with existing build tooling that expects deep imports
- Drop-in replacement for wasm-miniscript without code changes

Resolves module resolution errors when building bitgo-ui with wasm-utxo.

Ticket: BTC-0
@lcovar lcovar requested a review from a team as a code owner November 13, 2025 00:21
@lcovar lcovar merged commit ec1f42c into master Nov 13, 2025
1 check passed
@lcovar lcovar deleted the BTC-0-no-exports branch November 13, 2025 00:34
@OttoAllmendinger
Copy link
Contributor

dist/browser/js/wasm/wasm_utxo.js

can we add the required exports field so that it bitgo-ui can access this?

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.

5 participants