-
Notifications
You must be signed in to change notification settings - Fork 615
Version Packages #8114
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
Version Packages #8114
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughBumps package versions for thirdweb (5.108.2), nebula (0.2.55), and wagmi-adapter (0.2.152); updates corresponding CHANGELOG.md files; deletes two changeset entries that referenced adding Changes
Sequence Diagram(s)Not applicable. Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Pre-merge checks and finishing touches❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (1 passed)
✨ Finishing touches🧪 Generate unit tests
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro Disabled knowledge base sources:
📒 Files selected for processing (8)
💤 Files with no reviewable changes (2)
✅ Files skipped from review due to trivial changes (2)
🚧 Files skipped from review as they are similar to previous changes (3)
🧰 Additional context used🧠 Learnings (2)📓 Common learnings📚 Learning: 2025-08-29T15:37:38.513ZApplied to files:
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)
🔇 Additional comments (1)
Comment |
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (4)
packages/nebula/package.json (1)
60-60: Consider adding a size-limit budget in package.json.Per repo guidelines, track bundle budgets via package.json#size-limit for publishable packages.
Example config you can add (tune paths/limits as needed):
{ "size-limit": [ { "name": "nebula ESM", "path": "dist/esm/exports/thirdweb.js", "limit": "40 KB" }, { "name": "nebula CJS", "path": "dist/cjs/exports/thirdweb.js", "limit": "40 KB" } ] }packages/wagmi-adapter/CHANGELOG.md (1)
3-4: Changelog entry OK.Empty heading for 0.2.152 is fine for a version-only bump. Optionally add “Version bump only” for clarity.
packages/wagmi-adapter/package.json (1)
58-58: Add size-limit budget to packages/wagmi-adapter/package.json
This package ships runtime code; include a"size-limit"field in its package.json to enforce bundle size limits and catch accidental bloat.packages/thirdweb/CHANGELOG.md (1)
7-10: Clarify browser script usage and global name; add concise example.- Generate a browser script in `dist/scripts/bridge-widget.js` that can be used to render the `BridgeWidget` component in a browser with a script + Generates a browser script at `dist/scripts/bridge-widget.js`. When included on a page, it exposes a global `BridgeWidget` you can render via `BridgeWidget.render(...)`. + See packages/thirdweb/src/script-exports/readme.md for usage details.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (7)
.changeset/metal-bats-speak.md(0 hunks)packages/nebula/CHANGELOG.md(1 hunks)packages/nebula/package.json(1 hunks)packages/thirdweb/CHANGELOG.md(1 hunks)packages/thirdweb/package.json(1 hunks)packages/wagmi-adapter/CHANGELOG.md(1 hunks)packages/wagmi-adapter/package.json(1 hunks)
💤 Files with no reviewable changes (1)
- .changeset/metal-bats-speak.md
🧰 Additional context used
📓 Path-based instructions (2)
**/package.json
📄 CodeRabbit inference engine (AGENTS.md)
Track bundle budgets via
package.json#size-limit
Files:
packages/thirdweb/package.jsonpackages/wagmi-adapter/package.jsonpackages/nebula/package.json
packages/wagmi-adapter/**
📄 CodeRabbit inference engine (CLAUDE.md)
Wagmi ecosystem integration is in
packages/wagmi-adapter/
Files:
packages/wagmi-adapter/package.jsonpackages/wagmi-adapter/CHANGELOG.md
🧠 Learnings (2)
📚 Learning: 2025-07-18T19:19:55.613Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/wagmi-adapter/** : Wagmi ecosystem integration is in `packages/wagmi-adapter/`
Applied to files:
packages/wagmi-adapter/package.jsonpackages/wagmi-adapter/CHANGELOG.md
📚 Learning: 2025-09-24T11:09:45.142Z
Learnt from: MananTank
PR: thirdweb-dev/js#8106
File: packages/thirdweb/src/script-exports/readme.md:6-7
Timestamp: 2025-09-24T11:09:45.142Z
Learning: For thirdweb Bridge Widget script exports, the module is exported with globalName: "BridgeWidget" in tsup config, making the global API `BridgeWidget.render()` rather than `window.thirdweb.BridgeWidget.render()`.
Applied to files:
packages/thirdweb/CHANGELOG.md
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (8)
- GitHub Check: E2E Tests (pnpm, vite)
- GitHub Check: E2E Tests (pnpm, esbuild)
- GitHub Check: Lint Packages
- GitHub Check: Size
- GitHub Check: E2E Tests (pnpm, webpack)
- GitHub Check: Unit Tests
- GitHub Check: Build Packages
- GitHub Check: Analyze (javascript)
🔇 Additional comments (4)
packages/nebula/package.json (1)
60-60: Version bump looks good.Patch release aligns with the dependency update to [email protected].
packages/wagmi-adapter/package.json (1)
58-58: Version bump LGTM; peer range still compatible.thirdweb@^5.85.0 peer range covers 5.108.2.
packages/nebula/CHANGELOG.md (1)
3-9: Changelog entry consistent with version bump.Notes correctly reference [email protected] and the commit link.
packages/thirdweb/package.json (1)
433-433: Version bump approved and widget script packaging confirmed.
Files whitelist includes dist/* so dist/scripts/bridge-widget.js will be published; globalName “BridgeWidget” exposes the API as BridgeWidget.render(), no export surface changes needed.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8114 +/- ##
==========================================
+ Coverage 56.26% 56.28% +0.01%
==========================================
Files 906 906
Lines 59193 59193
Branches 4173 4173
==========================================
+ Hits 33305 33316 +11
+ Misses 25783 25772 -11
Partials 105 105
🚀 New features to boost your workflow:
|
size-limit report 📦
|
7767876 to
5b6e269
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
packages/thirdweb/CHANGELOG.md (1)
3-10: Clarify usage and global name for BridgeWidget script exportThe entry is good, but please document how to use the generated script and call out the correct global namespace (BridgeWidget.render), not window.thirdweb.BridgeWidget.render.
Apply this diff to improve clarity and add a usage snippet:
## 5.108.2 ### Patch Changes -- [#8106](https://github.com/thirdweb-dev/js/pull/8106) [`1f7cda6`](https://github.com/thirdweb-dev/js/commit/1f7cda66988e88628ca9b8fe608af071aa0026a1) Thanks [@MananTank](https://github.com/MananTank)! - Add `BridgeWidget` component. - - Generate a browser script in `dist/scripts/bridge-widget.js` that can be used to render the `BridgeWidget` component in a browser with a script +- [#8106](https://github.com/thirdweb-dev/js/pull/8106) [`1f7cda6`](https://github.com/thirdweb-dev/js/commit/1f7cda66988e88628ca9b8fe608af071aa0026a1) Thanks [@MananTank](https://github.com/MananTank)! - Add `BridgeWidget` component. + + Generates a browser script at `dist/scripts/bridge-widget.js` to render the `BridgeWidget` in a plain browser environment. + + Usage: + ```html + <script src="/dist/scripts/bridge-widget.js"></script> + <div id="bridge-container"></div> + <script> + BridgeWidget.render({ + selector: "#bridge-container", + // ...props + }); + </script> + ```
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (7)
.changeset/metal-bats-speak.md(0 hunks)packages/nebula/CHANGELOG.md(1 hunks)packages/nebula/package.json(1 hunks)packages/thirdweb/CHANGELOG.md(1 hunks)packages/thirdweb/package.json(1 hunks)packages/wagmi-adapter/CHANGELOG.md(1 hunks)packages/wagmi-adapter/package.json(1 hunks)
💤 Files with no reviewable changes (1)
- .changeset/metal-bats-speak.md
✅ Files skipped from review due to trivial changes (1)
- packages/wagmi-adapter/CHANGELOG.md
🚧 Files skipped from review as they are similar to previous changes (4)
- packages/nebula/package.json
- packages/nebula/CHANGELOG.md
- packages/thirdweb/package.json
- packages/wagmi-adapter/package.json
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-09-24T11:09:45.142Z
Learnt from: MananTank
PR: thirdweb-dev/js#8106
File: packages/thirdweb/src/script-exports/readme.md:6-7
Timestamp: 2025-09-24T11:09:45.142Z
Learning: For thirdweb Bridge Widget script exports, the module is exported with globalName: "BridgeWidget" in tsup config, making the global API `BridgeWidget.render()` rather than `window.thirdweb.BridgeWidget.render()`.
Applied to files:
packages/thirdweb/CHANGELOG.md
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (8)
- GitHub Check: Size
- GitHub Check: Build Packages
- GitHub Check: Unit Tests
- GitHub Check: E2E Tests (pnpm, vite)
- GitHub Check: E2E Tests (pnpm, webpack)
- GitHub Check: E2E Tests (pnpm, esbuild)
- GitHub Check: Lint Packages
- GitHub Check: Analyze (javascript)
5b6e269 to
acfc47d
Compare
61ee262 to
cfb0341
Compare
cfb0341 to
8973072
Compare
8973072 to
686583d
Compare
686583d to
83637d5
Compare
This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
@thirdweb-dev/[email protected]
Patch Changes
1f7cda6,9069351]:[email protected]
Patch Changes
#8106
1f7cda6Thanks @MananTank! - AddBridgeWidgetcomponent.Generate a browser script in
dist/scripts/bridge-widget.jsthat can be used to render theBridgeWidgetcomponent in a browser with a script#8126
9069351Thanks @joaquim-verges! - Accept chain objects for x402 APIs@thirdweb-dev/[email protected]
PR-Codex overview
This PR primarily updates version numbers across multiple packages and introduces new changelog entries for recent updates, particularly focusing on the
thirdwebandnebulapackages.Detailed summary
.changeset/metal-bats-speak.mdand.changeset/polite-banks-peel.md.versioninpackages/thirdweb/package.jsonfrom5.108.1to5.108.2.versioninpackages/nebula/package.jsonfrom0.2.54to0.2.55.versioninpackages/wagmi-adapter/package.jsonfrom0.2.151to0.2.152.packages/nebula/CHANGELOG.md: New version0.2.55with updated dependencies.packages/thirdweb/CHANGELOG.md: New version5.108.2with features likeBridgeWidgetcomponent and browser script generation.Summary by CodeRabbit