Skip to content

feat(webui)!: modernize module system and remove inline wasm#258

Merged
OttoAllmendinger merged 2 commits intomasterfrom
BTC-0.webui-fixes
Apr 22, 2026
Merged

feat(webui)!: modernize module system and remove inline wasm#258
OttoAllmendinger merged 2 commits intomasterfrom
BTC-0.webui-fixes

Conversation

@OttoAllmendinger
Copy link
Copy Markdown
Contributor

This PR modernizes the webui codebase by converting to ES modules and
removing inline wasm files.

Changes

  • Convert package.json to ES module type
  • Update webpack.config.js and extract-samples.js to use ES module
    syntax
  • Add proper __dirname handling for ES modules using import.meta
  • Fix ESLint issues: remove reassignments and void floating promises
  • Improve error handling with proper type checking in PSBT editor
  • Make router hash parsing use const instead of let
  • Remove inline wasm files (already in .gitignore)

Breaking Changes

This converts the package to ES modules, which may affect consumers
expecting CommonJS format.

Issue: BTC-0

OttoAllmendinger and others added 2 commits April 22, 2026 12:51
It is in .gitignore already

Issue: BTC-0
- Convert package.json to ES module type
- Update webpack.config.js and extract-samples.js to use ES module syntax
- Add proper __dirname handling for ES modules using import.meta
- Fix ESLint issues: remove reassignments and void floating promises
- Improve error handling with proper type checking in PSBT editor
- Make router hash parsing use const instead of let

Issue: BTC-0

Co-authored-by: llm-git <llm-git@ttll.de>
@OttoAllmendinger OttoAllmendinger marked this pull request as ready for review April 22, 2026 11:44
@OttoAllmendinger OttoAllmendinger requested a review from a team as a code owner April 22, 2026 11:44
@OttoAllmendinger OttoAllmendinger merged commit 66da398 into master Apr 22, 2026
16 checks passed
@OttoAllmendinger OttoAllmendinger deleted the BTC-0.webui-fixes branch April 22, 2026 15:42
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.

2 participants