-
Notifications
You must be signed in to change notification settings - Fork 607
Version Packages #8300
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 #8300
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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. |
WalkthroughVersion and changelog bumps across multiple packages and an app (thirdweb, wagmi-adapter, nebula, wagmi-demo); plus removal of two processed changeset files. No source-code logic changes are included. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro Disabled knowledge base sources:
📒 Files selected for processing (10)
💤 Files with no reviewable changes (2)
🚧 Files skipped from review as they are similar to previous changes (5)
🧰 Additional context used📓 Path-based instructions (1)**/package.json📄 CodeRabbit inference engine (AGENTS.md)
Files:
🧠 Learnings (2)📓 Common learnings📚 Learning: 2025-07-18T19:19:55.613ZApplied to files:
🔇 Additional comments (3)
Warning Review ran into problems🔥 ProblemsErrors were encountered while retrieving linked issues. Errors (1)
Comment |
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: 1
🧹 Nitpick comments (1)
apps/wagmi-demo/package.json (1)
28-28: Pin @wagmi/cli to a specific version for reproducibility.Using
"latest"for the @wagmi/cli devDependency can lead to non-reproducible builds and unexpected breaking changes. Consider pinning to a specific version, e.g.,"@wagmi/cli": "2.x.x".Apply this diff to improve build reproducibility:
- "@wagmi/cli": "latest", + "@wagmi/cli": "2.2.0",(Verify the latest compatible version of @wagmi/cli and adjust the pinned version accordingly.)
📜 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 (9)
.changeset/light-signs-send.md(0 hunks)apps/wagmi-demo/CHANGELOG.md(1 hunks)apps/wagmi-demo/package.json(1 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/light-signs-send.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/nebula/package.jsonapps/wagmi-demo/package.jsonpackages/wagmi-adapter/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)
📓 Common learnings
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/`
📚 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:
apps/wagmi-demo/CHANGELOG.mdpackages/wagmi-adapter/package.jsonpackages/wagmi-adapter/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: Size
- GitHub Check: E2E Tests (pnpm, webpack)
- GitHub Check: E2E Tests (pnpm, esbuild)
- GitHub Check: Unit Tests
- GitHub Check: Build Packages
- GitHub Check: Lint Packages
- GitHub Check: Analyze (javascript)
🔇 Additional comments (6)
packages/nebula/CHANGELOG.md (1)
3-9: Changelog entry for v0.2.72 looks good.The new version entry is properly formatted with commit reference and dependency documentation.
packages/nebula/package.json (1)
60-60: Version bump to 0.2.72 is consistent with changelog.apps/wagmi-demo/CHANGELOG.md (1)
3-9: Changelog entry for v0.0.14 looks good.The version entry properly documents dependency updates to both thirdweb and wagmi-adapter.
packages/thirdweb/package.json (1)
433-433: Version bump to 5.110.1 is correct.packages/wagmi-adapter/package.json (1)
59-59: Version bump is appropriate for this release.The patch version increment (0.2.169) reflects the dependency update to [email protected], which satisfies the peer dependency range
^5.85.0. The devDependency@wagmi/[email protected]also satisfies its peer requirement.apps/wagmi-demo/package.json (1)
4-4: Version bump is appropriate for this release.The patch version increment (0.0.14) reflects upstream dependency updates to [email protected] and @thirdweb-dev/[email protected]. All type dependencies are correctly aligned with the pinned React version.
| ## 0.2.169 | ||
|
|
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.
Missing changelog entry content for v0.2.169.
The new version entry for 0.2.169 has no documentation. Per the PR objectives, [email protected] is being released and should include at least a "### Patch Changes" section explaining the update. Compare this to the well-documented entries in packages/nebula/CHANGELOG.md (which notes dependency updates to [email protected]) and apps/wagmi-demo/CHANGELOG.md (which documents dependency updates to both thirdweb and wagmi-adapter).
Consider adding a patch notes section:
## 0.2.169
+
+### Patch Changes
+
+- Updated dependencies [[`e9788096d6...`](https://github.com/thirdweb-dev/js/commit/e9788096d665afef7c2a933cea5d6f522fc67273)]:
+ - dependencies updated (if any)Alternatively, if this is intentionally a no-op release, consider documenting that explicitly.
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| ## 0.2.169 | |
| ## 0.2.169 | |
| ### Patch Changes | |
| - Updated dependencies [[`e9788096d6...`](https://github.com/thirdweb-dev/js/commit/e9788096d665afef7c2a933cea5d6f522fc67273)]: | |
| - dependencies updated (if any) |
🤖 Prompt for AI Agents
In packages/wagmi-adapter/CHANGELOG.md around lines 3–4, the 0.2.169 entry is
empty; add a proper changelog entry by inserting a "### Patch Changes" section
describing what changed for v0.2.169 (e.g., dependency updates, bug fixes, or
other small changes), mirror the style used in packages/nebula/CHANGELOG.md or
apps/wagmi-demo/CHANGELOG.md for formatting and specificity, and if this release
intentionally contains no changes, explicitly state that with a short "No
changes (no-op release)" note under "### Patch Changes".
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8300 +/- ##
==========================================
- Coverage 54.92% 54.90% -0.03%
==========================================
Files 919 919
Lines 60665 60664 -1
Branches 4141 4127 -14
==========================================
- Hits 33322 33308 -14
- Misses 27240 27254 +14
+ Partials 103 102 -1
🚀 New features to boost your workflow:
|
size-limit report 📦
|
9fabe90 to
fe56a0d
Compare
fe56a0d to
2979584
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
85fe949,e978809,899051f]:[email protected]
Patch Changes
#8299
85fe949Thanks @alecananian! - [React Native] Fixed Send Funds screen not allowing decimal amounts#8296
e978809Thanks @MananTank! - Do not require connecting wallet inBuyWidgetifreceiverAddressis set#8298
899051fThanks @joaquim-verges! - Add EIP-5792 support for EIP1193.toProvider()@thirdweb-dev/[email protected]
[email protected]
Patch Changes
85fe949,e978809,899051f]:PR-Codex overview
This PR focuses on updating version numbers across multiple packages and includes patch changes for
thirdweb,wagmi-adapter, andwagmi-demo, along with their respective changelogs.Detailed summary
thirdwebversion from5.110.0to5.110.1.wagmi-demoversion from0.0.13to0.0.14.nebulaversion from0.2.71to0.2.72.wagmi-adapterversion from0.2.168to0.2.169.thirdweb,wagmi-demo, andnebuladetailing patch changes.Summary by CodeRabbit
New Features
Bug Fixes
Chores