chore(deps): bump to wasm-bindgen 0.2.105#1947
Conversation
|
📝 WalkthroughWalkthroughA workspace dependency version bump of wasm-bindgen from 0.2.104 to 0.2.105 across Cargo.toml and corresponding platform-specific asset metadata updates for five architectures. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
packages/web-platform/web-style-transformer/scripts/wasm-bindgen (1)
1-102: Missing platform configuration in auto-generated file.The dotslash configuration is incomplete. While the file includes 5 of 6 available wasm-bindgen 0.2.105 release platforms with correct sizes, it is missing the
aarch64-unknown-linux-muslplatform that exists in the official GitHub release (size: 8426510 bytes).The file should either:
- Be regenerated with the complete set of platforms from the 0.2.105 release, or
- Document why this platform is intentionally excluded
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
Cargo.lockis excluded by!**/*.lock
📒 Files selected for processing (2)
Cargo.toml(1 hunks)packages/web-platform/web-style-transformer/scripts/wasm-bindgen(1 hunks)
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: colinaaa
Repo: lynx-family/lynx-stack PR: 1454
File: pnpm-workspace.yaml:46-46
Timestamp: 2025-08-07T04:00:59.645Z
Learning: In the lynx-family/lynx-stack repository, the webpack patch (patches/webpack5.101.0.patch) was created to fix issues with webpack5.99.9 but only takes effect on webpack5.100.0 and later versions. The patchedDependencies entry should use "webpack@^5.100.0" to ensure the patch applies to the correct version range.
Learnt from: gaoachao
Repo: lynx-family/lynx-stack PR: 1714
File: packages/react/transform/Cargo.toml:19-19
Timestamp: 2025-09-10T11:42:36.855Z
Learning: In packages/react/transform/Cargo.toml, the crate uses serde derive macros (#[derive(Serialize, Deserialize)]) in multiple files including src/esbuild.rs and src/swc_plugin_extract_str/mod.rs, so the "derive" feature is required when migrating to workspace dependencies.
📚 Learning: 2025-09-10T11:42:36.855Z
Learnt from: gaoachao
Repo: lynx-family/lynx-stack PR: 1714
File: packages/react/transform/Cargo.toml:19-19
Timestamp: 2025-09-10T11:42:36.855Z
Learning: In packages/react/transform/Cargo.toml, the crate uses serde derive macros (#[derive(Serialize, Deserialize)]) in multiple files including src/esbuild.rs and src/swc_plugin_extract_str/mod.rs, so the "derive" feature is required when migrating to workspace dependencies.
Applied to files:
Cargo.toml
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
CodSpeed Performance ReportMerging #1947 will improve performances by 7.05%Comparing Summary
Benchmarks breakdown
Footnotes
|
Web Explorer#6263 Bundle Size — 366.97KiB (~-0.01%).b544ba8(current) vs 12f6fb3 main#6252(baseline) Bundle metrics
Bundle size by type
Bundle analysis report Branch PupilTong:p/hw/wasm-bindgen-0-10... Project dashboard Generated by RelativeCI Documentation Report issue |
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.
Checklist