Update componentize-py to 0.22.1#85
Merged
Merged
Conversation
This update includes a few small fixes and tweaks in upstream; more noticeably, however, it includes some changes that reduce the size of the wasm we generate (mostly by stripping bundled .so modules). Other cargo dependencies also received bumps to the current latest versions. In the future, we can hopefully move to pointing at crates.io or tags for componentize-py, but for now that isn't available. ``` Before: Permissions Size User Date Modified Name .rw-r--r--@ 43M paul.osborne 10 Apr 13:47 backend-requests.composed.wasm .rw-r--r--@ 43M paul.osborne 10 Apr 13:47 bottle-app.composed.wasm .rw-r--r--@ 50M paul.osborne 10 Apr 13:47 flask-app.composed.wasm .rw-r--r--@ 50M paul.osborne 10 Apr 13:47 game-of-life.composed.wasm After: .rw-r--r--@ 21M paul.osborne 10 Apr 13:50 backend-requests.composed.wasm .rw-r--r--@ 21M paul.osborne 10 Apr 13:50 bottle-app.composed.wasm .rw-r--r--@ 27M paul.osborne 10 Apr 13:50 flask-app.composed.wasm .rw-r--r--@ 27M paul.osborne 10 Apr 13:50 game-of-life.composed.wasm ```
erikrose
approved these changes
Apr 10, 2026
Member
erikrose
left a comment
erikrose
left a comment
There was a problem hiding this comment.
Great to glean the fruits of the new stuff! I just want to make sure the import wiggles don't make it in unless they're stable now. I generally see the wiggles upon re-generating the stubs.
| import weakref | ||
|
|
||
| from componentize_py_types import Result, Ok, Err, Some | ||
| from ..imports import async_io |
Member
There was a problem hiding this comment.
I've found that, unfortunately, the order of imports in stubs is unstable. I'd prefer to not keep committing changes that wiggle them back and forth unless we can show it's stable now. (Maybe we should run a fixer over them in the interrim.)
Member
Author
There was a problem hiding this comment.
This is actually changing due to changes in upstream to hopefully fix that issue: bytecodealliance/componentize-py@81d582a
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This update includes a few small fixes and tweaks in upstream; more noticeably, however, it includes some changes that reduce the size of the wasm we generate (mostly by stripping bundled .so modules).
Other cargo dependencies also received bumps to the current latest versions. In the future, we can hopefully move to pointing at crates.io or tags for componentize-py, but for now that isn't available.