Skip to content

Replace wasm-opt in bindings dep with local version#283

Open
wulfraem wants to merge 2 commits into
mainfrom
chore/use-local-wasm-opt
Open

Replace wasm-opt in bindings dep with local version#283
wulfraem wants to merge 2 commits into
mainfrom
chore/use-local-wasm-opt

Conversation

@wulfraem

Copy link
Copy Markdown
Contributor

Description of change

PR removes outdated wasm-opt installed via NPM, and add the requirement to have wasm-opt installed locally, which allows using newer versions, that are compatible with the latest Rust toolchain versions.

This updates the wasm-opt used in the build process to its latest version.

In related projects, we had an error related to newer Rust versions and outdated wasm-opt version ([parse exception: invalid code after misc prefix: 17 (at ...)]) thrown during build, which was fixed by the same changes. The error does not seem to be thrown at the moment (checked in CI), but possibly will have been with future changes in the code. Using an up-to-date wasm-opt version should prevent this.

Links to any relevant issues

See related issue.

Type of change

  • Bug fix (a non-breaking change which fixes an issue)
  • Enhancement (a non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Fix
  • Chore

How the change has been tested

Same changes fixed wasm-opt error in identity project (see related issue above).

Change checklist

  • I have followed the contribution guidelines for this project
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation

- replaced with local installation instructions
- see also [related PR](iotaledger/identity#1823)
@wulfraem wulfraem added enhancement New feature or request No changelog labels Jun 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request No changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant