Semver aware WASI module resolution when dealing with imports

Open
#349 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
56/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Active
Tech stack
javascript, rust, wasm

Research direction

Start at ComponentizeJS's WASI import-resolution path and trace the existing direct-match handling for imports such as wasi:http/types@0.2.6. Define the compatible semver behavior around exact matches, upgrades within the supported WASI 2.x track, and rejection of incompatible older versions. Done means the stated import resolves against newer compatible worlds while jco transpile remains out of scope.

Written by the indexing model from the issue text.

Description

enhancement

Right now when building components with ComponentizeJS, we deal with references WASI types via direct match only.

This is usually fine but forces certain versions (the ones supported by StarlingMonkey below) to be used. Ideally we should be able to do an exact match where possible and the fall back to a compatible semver track (usually allowing upgrades but not downgrades) similar to what wasmtime would do.

As WASI 2.x is unlikely to change going forward I think we can now do this pretty definitively here, with minimal changes required for StarlingMonkey as well.

Also note that while this issue is probably going to be unnecessary when the "new" ComponentizeJS lands, this issue is here for tracking purposes just to make sure this

A simple use case is being able to write wasi:http components against any version of 2.x and having that compile (and eventually jco transpile, but that's out of scope here!) properly. Essentially the following should never fail:

import { Fields } from "wasi:http/types@0.2.6";

(Even if the underlying world is 2.10/2.12, but this should fail if the underlying world is 2.4 probably)

Maybe it should be possible to disable this feature (i.e. supporting very specific imports), but IMO this would be a big DX improvement that is worth doing (maybe we emit some warnings or something if we're worried about it).

Dominant language
Rust
Stars
392
Forks
54
Avg merge
2d 18h
Merged PRs (30d)
3

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from bytecodealliance/ComponentizeJS

All issues in bytecodealliance/ComponentizeJS

Similar issues

More Rust issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.