-
Notifications
You must be signed in to change notification settings - Fork 617
Version Packages #8357
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 #8357
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. |
WalkthroughCoordinated release updates: removed a processed changeset, bumped versions and added changelog entries for packages Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes
Possibly related PRs
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (1 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
size-limit report 📦
|
bb83f88 to
ddfeb7e
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 (2)
packages/thirdweb/CHANGELOG.md (2)
7-8: Style nit: wrap the property name in backticks.For consistency with prior entries that format identifiers/props as code, consider:
- - [#8358] (...) Thanks [@joaquim-verges]! - Add support for extraMetadata in x402 payment requirements + - [#8358] (...) Thanks [@joaquim-verges]! - Add support for `extraMetadata` in x402 payment requirements
9-10: Optional clarity: add brief scope/context.“Update implementations, fix deployment” is a bit vague. If possible, note the affected area (e.g., “contracts/deploy,” “SDK publish pipeline”) for quicker triage when scanning the log.
📜 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/open-icons-start.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/open-icons-start.md
✅ Files skipped from review due to trivial changes (2)
- packages/wagmi-adapter/package.json
- packages/nebula/CHANGELOG.md
🚧 Files skipped from review as they are similar to previous changes (3)
- apps/wagmi-demo/package.json
- packages/wagmi-adapter/CHANGELOG.md
- apps/wagmi-demo/CHANGELOG.md
🧰 Additional context used
📓 Path-based instructions (1)
**/package.json
📄 CodeRabbit inference engine (AGENTS.md)
Track bundle budgets via
package.json#size-limit
Files:
packages/nebula/package.jsonpackages/thirdweb/package.json
🧠 Learnings (6)
📓 Common learnings
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Surface breaking changes prominently in PR descriptions
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-08-29T15:37:38.513Z
Learning: Surface breaking changes prominently in PR descriptions
Learnt from: CR
Repo: thirdweb-dev/js PR: 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
Repo: thirdweb-dev/js PR: 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/nebula/package.jsonpackages/thirdweb/package.json
📚 Learning: 2025-07-18T19:19:55.613Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 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/nebula/package.jsonpackages/thirdweb/package.json
📚 Learning: 2025-07-18T19:19:55.613Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/thirdweb/src/wallets/** : EIP-1193, EIP-5792, EIP-7702 standard support in wallet modules
Applied to files:
packages/thirdweb/CHANGELOG.md
📚 Learning: 2025-08-29T15:37:38.513Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-08-29T15:37:38.513Z
Learning: Surface breaking changes prominently in PR descriptions
Applied to files:
packages/thirdweb/CHANGELOG.md
📚 Learning: 2025-08-29T15:37:38.513Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 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/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, esbuild)
- GitHub Check: E2E Tests (pnpm, webpack)
- GitHub Check: Size
- GitHub Check: Unit Tests
- GitHub Check: E2E Tests (pnpm, vite)
- GitHub Check: Build Packages
- GitHub Check: Lint Packages
- GitHub Check: Analyze (javascript)
🔇 Additional comments (3)
packages/nebula/package.json (1)
60-60: Patch version bump is correctly applied.The version is bumped to 0.2.80 for a patch release documenting the updated dependency to thirdweb@5.111.2. The workspace reference on line 7 ensures the nebula package will resolve to the updated thirdweb version correctly.
packages/thirdweb/package.json (1)
433-433: Patch version bump is correctly applied.The version is bumped to 5.111.2 for a patch release aligning with the PR objectives (extraMetadata support in x402 payment requirements and deployment fixes). No changes to dependencies, exports, or public API surface.
However, per the project's coding guidelines, bundle size compliance must be verified against the
size-limitbudget defined inpackage.json. Please verify locally that this patch release remains within the established bundle size limits before merging.packages/thirdweb/CHANGELOG.md (1)
3-6: LGTM on section structure and placement.Version ordering, headings, and “Patch Changes” formatting are consistent with prior entries. Links and short hashes read well.
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/nebula@0.2.80
Patch Changes
1d54cff,fd04eef]:thirdweb@5.111.2
Patch Changes
#8358
1d54cffThanks @joaquim-verges! - Add support for extraMetadata in x402 payment requirements#8356
fd04eefThanks @kumaryash90! - Update implementations, fix deployment@thirdweb-dev/wagmi-adapter@0.2.177
wagmi-inapp@0.0.22
Patch Changes
1d54cff,fd04eef]:PR-Codex overview
This PR focuses on updating version numbers and changelogs across several packages, reflecting dependency updates and new features.
Detailed summary
versioninpackages/thirdweb/package.jsonfrom5.111.1to5.111.2versioninapps/wagmi-demo/package.jsonfrom0.0.21to0.0.22versioninpackages/nebula/package.jsonfrom0.2.79to0.2.80versioninpackages/wagmi-adapter/package.jsonfrom0.2.176to0.2.177nebula,wagmi-demo, andthirdweb, including dependency updates tothirdweb@5.111.2and@thirdweb-dev/wagmi-adapter@0.2.177Summary by CodeRabbit
New Features
Bug Fixes
Chores