Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): update jco versions #92

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

vados-cosmonic
Copy link
Contributor

Feature or Problem

This PR:

  • Updates the versions of jco and componentize-js in use
  • Updates code to use jco guest-types
  • Updates usage of types to be more ergonomic

Weirdly enough, it looks like the JS examples are broken when run w/ wash dev, but work with wasmtime serve...

I suggest we take in this change (since it's not any more broken), then get around to figuring out this error:

2025-02-05T18:42:47.768753Z ERROR wasmcloud_host::wasmbus: failed to scale component component_ref=file:///path/to/http-hello-world/build/http_hello_world_s.wasm component_id=Ymdzvb-typescript-http-hello-world err=component imports instance `wasi:http/[email protected]`, but a matching implementation was not found in the linker

Caused by:
    0: instance export `[method]incoming-body.stream` has the wrong type
    1: type mismatch with results
    2: resource type mismatch

Related Issues

Release Information

Consumer Impact

Testing

Unit Test(s)

Acceptance or Integration

Manual Verification

@vados-cosmonic vados-cosmonic requested review from a team as code owners February 5, 2025 18:50
@vados-cosmonic
Copy link
Contributor Author

Also, ignore the 0.2.0 note -- even if you update to 0.2.2 (which I have) the error is still present

@vados-cosmonic
Copy link
Contributor Author

vados-cosmonic commented Feb 5, 2025

This PR is blocked by this upstream issue (and the release of a new host w/ the fix):

wasmCloud/wasmCloud#4063

@vados-cosmonic vados-cosmonic force-pushed the chore(deps)=update-jco-versions branch from 6914b5d to 3d58bd5 Compare February 12, 2025 06: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.

1 participant