Version Packages #8340
Version Packages #8340
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. |
|
Warning Rate limit exceeded@joaquim-verges has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 11 minutes and 14 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (9)
WalkthroughVersion and changelog updates across multiple packages (wagmi-demo, nebula, thirdweb, wagmi-adapter), plus deletion of a resolved changeset file that previously recorded the thirdweb minor release and removal of Bridge.routes / useBridgeRoutes. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes
Pre-merge checks and finishing touches❌ Failed checks (2 warnings)
✅ Passed checks (1 passed)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
packages/wagmi-adapter/CHANGELOG.md (1)
3-4: Add changelog documentation for the 0.2.175 release.The entry for version 0.2.175 is empty and lacks any Patch Changes documentation. Compare this to the wagmi-demo (v0.0.20) and nebula (v0.2.78) changelogs in the same release, which both document their dependency updates. Consider adding a Patch Changes section to document the upgrade or any relevant changes, even if minimal.
📜 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/rotten-apes-sing.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/rotten-apes-sing.md
🧰 Additional context used
📓 Path-based instructions (2)
packages/wagmi-adapter/**
📄 CodeRabbit inference engine (CLAUDE.md)
Wagmi ecosystem integration is in
packages/wagmi-adapter/
Files:
packages/wagmi-adapter/CHANGELOG.mdpackages/wagmi-adapter/package.json
**/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
🧠 Learnings (6)
📓 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-08-29T15:37:38.513Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: AGENTS.md:0-0
Timestamp: 2025-08-29T15:37:38.513Z
Learning: Applies to .changeset/*.md : Version bump rules: patch for non‑API changes; minor for new/modified public API
Applied to files:
packages/nebula/CHANGELOG.mdapps/wagmi-demo/CHANGELOG.mdapps/wagmi-demo/package.jsonpackages/thirdweb/CHANGELOG.md
📚 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/CHANGELOG.mdapps/wagmi-demo/CHANGELOG.mdapps/wagmi-demo/package.jsonpackages/wagmi-adapter/package.json
📚 Learning: 2025-08-29T15:37:38.513Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: AGENTS.md:0-0
Timestamp: 2025-08-29T15:37:38.513Z
Learning: Applies to packages/thirdweb/exports/** : Export all public API via `packages/thirdweb/exports/`, grouped by feature
Applied to files:
packages/thirdweb/package.jsonpackages/nebula/package.jsonpackages/wagmi-adapter/package.json
📚 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/thirdweb/src/exports/** : Export everything via `exports/` directory, grouped by feature in the SDK public API
Applied to files:
packages/thirdweb/package.jsonpackages/nebula/package.jsonpackages/wagmi-adapter/package.json
📚 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: Surface breaking changes prominently in PR descriptions
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). (7)
- GitHub Check: E2E Tests (pnpm, esbuild)
- GitHub Check: E2E Tests (pnpm, vite)
- GitHub Check: Build Packages
- GitHub Check: Unit Tests
- GitHub Check: Size
- GitHub Check: E2E Tests (pnpm, webpack)
- GitHub Check: Lint Packages
🔇 Additional comments (6)
apps/wagmi-demo/package.json (1)
4-4: Version bump aligns with changelog and dependency updates.Patch version bump from 0.0.19 to 0.0.20 is appropriate for dependency-only changes.
packages/thirdweb/package.json (1)
433-433: Minor version bump correctly reflects Bridge API removal.Version 5.110.6 → 5.111.0 is appropriate for the removal of Bridge.routes and useBridgeRoutes, which are breaking changes to the public API.
packages/wagmi-adapter/package.json (1)
59-59: Version bump is correct and dependency constraints are satisfied.Patch version bump from 0.2.174 to 0.2.175 is appropriate. The peerDependency constraint on thirdweb@^5.85.0 is satisfied by the new thirdweb@5.111.0.
packages/nebula/package.json (1)
60-60: Version bump is correct for dependency update.Patch version bump from 0.2.77 to 0.2.78 is appropriate for the thirdweb@5.111.0 dependency update. Workspace dependency resolution will pick up the new version automatically.
apps/wagmi-demo/CHANGELOG.md (1)
3-9: Changelog entry is complete and well-documented.Version 0.0.20 changelog entry correctly documents the dependency updates to thirdweb@5.111.0 and @thirdweb-dev/wagmi-adapter@0.2.175 with proper links and formatting.
packages/nebula/CHANGELOG.md (1)
3-8: Changelog entry is properly formatted with correct dependency documentation.Version 0.2.78 changelog correctly documents the thirdweb@5.111.0 dependency update with proper link and formatting following the changesets convention.
| ## 5.111.0 | ||
|
|
||
| ### Minor Changes | ||
|
|
||
| - [#8335](https://github.com/thirdweb-dev/js/pull/8335) [`c83f1c6`](https://github.com/thirdweb-dev/js/commit/c83f1c6f7a38c8a9f424798e6e726e1d037ed042) Thanks [@MananTank](https://github.com/MananTank)! - Remove `Bridge.routes` and `useBridgeRoutes` | ||
|
|
There was a problem hiding this comment.
🧩 Analysis chain
Call out BREAKING removal and add migration notes
Removing Bridge.routes and useBridgeRoutes is a breaking API change but it’s listed under Minor Changes with no migration path. Please:
- Mark this as BREAKING in the entry title/body.
- Add a brief “Migration” section pointing devs to supported replacements:
- Use
Bridge.Buy.quote/prepareorBridge.Sell.quote/preparefor route discovery and execution. - For single‑leg transfers, use
Bridge.Transfer.prepare. - Use
Bridge.chainsandBridge.tokensto enumerate supported chains/tokens.
- Use
Example (suggested edit):
- Remove `Bridge.routes` and `useBridgeRoutes`
+ BREAKING: Removed `Bridge.routes` and `useBridgeRoutes`.
+ Migration:
+ - For quotes and execution, use:
+ `Bridge.Buy.quote/prepare` or `Bridge.Sell.quote/prepare`
+ - For direct same‑chain transfers, use:
+ `Bridge.Transfer.prepare`
+ - For discovery, use:
+ `Bridge.chains()` and `Bridge.tokens()`
+ See PR #8335 for details.Also verify no packages/docs still reference the removed symbols before publishing. Based on learnings.
🏁 Script executed:
#!/bin/bash
# Search for remaining references to the removed Bridge APIs
echo "=== Searching for Bridge.routes references ==="
rg -n 'Bridge\.routes' --type=ts --type=tsx --type=js --type=jsx -C2
echo ""
echo "=== Searching for useBridgeRoutes references ==="
rg -n 'useBridgeRoutes' --type=ts --type=tsx --type=js --type=jsx -C2
echo ""
echo "=== Checking for Bridge export/type definition ==="
rg -n 'export.*Bridge' packages/thirdweb -C2 | head -50Length of output: 3669
🏁 Script executed:
#!/bin/bash
# Check the specific commit/PR for more context
echo "=== Checking PR #8335 and commit context ==="
git log --oneline --all | grep -i "bridge.routes\|useBridgeRoutes" | head -10
echo ""
echo "=== Looking for BREAKING or migration notes in recent CHANGELOG ==="
head -50 packages/thirdweb/CHANGELOG.mdLength of output: 3594
🏁 Script executed:
#!/bin/bash
# Search for Bridge.Buy, Bridge.Sell, Bridge.Transfer, Bridge.chains, Bridge.tokens mentioned in migration
echo "=== Checking suggested replacements exist ==="
rg -n 'Bridge\.(Buy|Sell|Transfer|chains|tokens)' packages/thirdweb --type=ts --type=tsx | head -20Length of output: 73
🏁 Script executed:
#!/bin/bash
# Corrected search without problematic file type flags
echo "=== Searching for Bridge.routes references (all files) ==="
rg -n 'Bridge\.routes' packages/thirdweb -C2 2>&1 | head -30
echo ""
echo "=== Searching for useBridgeRoutes references ==="
rg -n 'useBridgeRoutes' packages/thirdweb -C2 2>&1 | head -30Length of output: 3871
🏁 Script executed:
#!/bin/bash
# Verify the suggested replacement APIs are exported
echo "=== Checking bridge exports in index.ts ==="
cat packages/thirdweb/src/bridge/index.ts
echo ""
echo "=== Checking for Buy, Sell, Transfer, chains, tokens ==="
ls -la packages/thirdweb/src/bridge/ | grep -E 'Buy|Sell|Transfer|Token|Chain'Length of output: 1551
🏁 Script executed:
#!/bin/bash
# Search git history for migration notes or deprecation messages
echo "=== Checking commit message for migration guidance ==="
git show --stat c83f1c6f 2>/dev/null | head -50
echo ""
echo "=== Looking for BREAKING keyword in CHANGELOG history ==="
grep -i "BREAKING" packages/thirdweb/CHANGELOG.md | head -10Length of output: 3492
BREAKING change miscategorized — move to Breaking Changes section and add migration path
Removing Bridge.routes and useBridgeRoutes is a breaking change (confirmed in the commit message), but it's listed under Minor Changes. Move this entry to a Breaking Changes section and add a migration path for users:
- For route discovery and execution → use
Bridge.Buy.quote/prepareorBridge.Sell.quote/prepare - For direct same-chain transfers → use
Bridge.Transfer.prepare - For enumerating chains and tokens → use
Bridge.chains()andBridge.tokens()
All replacement APIs are already exported and available. Verify internal docs/guides have been updated accordingly before publishing.
🤖 Prompt for AI Agents
In packages/thirdweb/CHANGELOG.md around lines 3 to 8, the removal of
Bridge.routes and useBridgeRoutes is incorrectly listed under "Minor Changes";
move that entry into a new "Breaking Changes" section and update the entry to
include a concise migration path: replace route discovery/execution with
Bridge.Buy.quote/prepare or Bridge.Sell.quote/prepare, replace direct same-chain
transfers with Bridge.Transfer.prepare, and replace enumeration with
Bridge.chains() and Bridge.tokens(); also add a note that the replacement APIs
are exported and available and confirm any internal docs/guides are updated
before publishing.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8340 +/- ##
=======================================
Coverage 54.87% 54.87%
=======================================
Files 919 919
Lines 60653 60653
Branches 4117 4119 +2
=======================================
Hits 33283 33283
Misses 27268 27268
Partials 102 102
🚀 New features to boost your workflow:
|
size-limit report 📦
|
joaquim-verges
force-pushed
the
changeset-release/main
branch
from
02ed92b to
6819ebe
Compare
joaquim-verges
force-pushed
the
changeset-release/main
branch
from
6819ebe to
54d6fe2
Compare
joaquim-verges
force-pushed
the
changeset-release/main
branch
from
54d6fe2 to
cd62ee6
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@5.111.0
Minor Changes
c83f1c6Thanks @MananTank! - RemoveBridge.routesanduseBridgeRoutesPatch Changes
219fc69Thanks @joaquim-verges! - Update x402 flows to use remote /accepts endpoint@thirdweb-dev/nebula@0.2.78
Patch Changes
c83f1c6,219fc69]:@thirdweb-dev/wagmi-adapter@0.2.175
wagmi-inapp@0.0.20
Patch Changes
c83f1c6,219fc69]:PR-Codex overview
This PR focuses on updating various package versions and their changelogs, reflecting new releases and dependency updates for several components, including
thirdweb,wagmi-adapter, andnebula.Detailed summary
.changeset/rotten-apes-sing.mdand.changeset/smart-sheep-doubt.md.versioninpackages/thirdweb/package.jsonfrom5.110.6to5.111.0.versioninapps/wagmi-demo/package.jsonfrom0.0.19to0.0.20.versioninpackages/nebula/package.jsonfrom0.2.77to0.2.78.versioninpackages/wagmi-adapter/package.jsonfrom0.2.174to0.2.175.packages/nebula(0.2.78) andapps/wagmi-demo(0.0.20).packages/thirdweb(5.111.0) with minor changes and patch updates.