-
Notifications
You must be signed in to change notification settings - Fork 567
Version Packages #7776
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 #7776
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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. |
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. Caution Review failedThe pull request is closed. WalkthroughThis change updates the version numbers and changelogs for the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (9)
✨ Finishing Touches🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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/wagmi-adapter/CHANGELOG.md (1)
3-4
: Add a “### Patch Changes” stub for consistencyEvery previous entry includes a “### Patch Changes” header (even when empty). Keeping that structure avoids accidental parsing issues in tooling that relies on this format.
## 0.2.124 +### Patch Changes +- Version bump only – automated release via PR #7776.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (7)
.changeset/thin-days-add.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/thin-days-add.md
🧰 Additional context used
📓 Path-based instructions (1)
packages/wagmi-adapter/**
📄 CodeRabbit Inference Engine (CLAUDE.md)
Wagmi ecosystem integration is in
packages/wagmi-adapter/
Files:
packages/wagmi-adapter/CHANGELOG.md
packages/wagmi-adapter/package.json
🧠 Learnings (7)
📓 Common learnings
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
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/`
Learnt from: MananTank
PR: thirdweb-dev/js#7356
File: apps/nebula/src/app/not-found.tsx:1-1
Timestamp: 2025-06-17T18:30:52.976Z
Learning: In the thirdweb/js project, the React namespace is available for type annotations (like React.FC) without needing to explicitly import React. This is project-specific configuration that differs from typical TypeScript/React setups.
Learnt from: Yash094
PR: thirdweb-dev/js#7608
File: apps/dashboard/src/@/api/team.ts:96-96
Timestamp: 2025-07-15T08:25:44.584Z
Learning: In the thirdweb dashboard codebase, the `getTeamById` function was actually a wrapper around `getTeamBySlug`, so removing it and calling `getTeamBySlug` directly doesn't change functionality. The `LAST_USED_TEAM_ID` cookie and related variables were storing slugs all along, not IDs.
📚 Learning: applies to packages/wagmi-adapter/** : wagmi ecosystem integration is in `packages/wagmi-adapter/`...
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.md
packages/wagmi-adapter/package.json
📚 Learning: applies to packages/thirdweb/src/exports/** : export everything via `exports/` directory, grouped by...
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/wagmi-adapter/package.json
packages/thirdweb/package.json
packages/nebula/package.json
📚 Learning: applies to packages/thirdweb/src/exports/** : every public symbol must have comprehensive tsdoc with...
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/** : Every public symbol must have comprehensive TSDoc with at least one `@example` block that compiles and custom annotation tags (`@beta`, `@internal`, `@experimental`)
Applied to files:
packages/wagmi-adapter/package.json
packages/thirdweb/package.json
packages/nebula/package.json
📚 Learning: in the thirdweb/js project, the react namespace is available for type annotations (like react.fc) wi...
Learnt from: MananTank
PR: thirdweb-dev/js#7356
File: apps/nebula/src/app/not-found.tsx:1-1
Timestamp: 2025-06-17T18:30:52.976Z
Learning: In the thirdweb/js project, the React namespace is available for type annotations (like React.FC) without needing to explicitly import React. This is project-specific configuration that differs from typical TypeScript/React setups.
Applied to files:
packages/wagmi-adapter/package.json
packages/thirdweb/package.json
packages/nebula/package.json
packages/thirdweb/CHANGELOG.md
📚 Learning: surface breaking changes prominently in pr descriptions...
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
📚 Learning: the thirdwebbarchart component in apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(side...
Learnt from: arcoraven
PR: thirdweb-dev/js#7505
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/analytics/components/WebhookAnalyticsCharts.tsx:186-204
Timestamp: 2025-07-10T10:18:33.238Z
Learning: The ThirdwebBarChart component in apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/analytics/components/WebhookAnalyticsCharts.tsx does not accept standard accessibility props like `aria-label` and `role` in its TypeScript interface, causing compilation errors when added.
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: Unit Tests
- GitHub Check: E2E Tests (pnpm, vite)
- GitHub Check: E2E Tests (pnpm, webpack)
- GitHub Check: Build Packages
- GitHub Check: Size
- GitHub Check: Lint Packages
- GitHub Check: Analyze (javascript)
🔇 Additional comments (5)
packages/thirdweb/CHANGELOG.md (1)
3-8
: Confirm commit hash length formatMost previous changelog entries use the short 7-character commit hash for readability.
This entry links to the full 40-character hash (e1ad7daf5401c4ddd3661eceed15e8c4f481d28e
).
For consistency, consider truncating the hash to the first 7 characters or, if the long form is now the new convention, update earlier entries accordingly.packages/thirdweb/package.json (1)
412-412
: Version bump acknowledgedPatch version updated to
5.105.27
; no other manifest changes. Looks good.packages/nebula/CHANGELOG.md (1)
3-9
: Changelog entry looks correctEntry accurately reflects the dependency bump and references the fixing commit.
packages/nebula/package.json (1)
60-60
: Nebula package version bump confirmedVersion advanced to
0.2.27
; aligns with the changelog. No issues.packages/wagmi-adapter/package.json (1)
58-58
: LGTM – version updated to 0.2.124No further changes required.
size-limit report 📦
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7776 +/- ##
=======================================
Coverage 56.34% 56.35%
=======================================
Files 905 905
Lines 58796 58796
Branches 4142 4143 +1
=======================================
+ Hits 33130 33135 +5
+ Misses 25561 25556 -5
Partials 105 105
🚀 New features to boost your workflow:
|
da2dc97
to
8168899
Compare
8168899
to
4ac8d32
Compare
4ac8d32
to
9b04879
Compare
9b04879
to
f3e1151
Compare
## 0.2.124 | ||
|
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.
The changelog entry for version 0.2.124
appears to be empty. Consider adding details about the changes in this version, following the same format used in other package changelogs that include patch changes and links to relevant PRs. This helps maintain consistency in documentation and provides users with important information about what changed between versions.
Spotted by Diamond
Is this helpful? React 👍 or 👎 to let us know.
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
fd4414b
,9a3ebe1
,e5326c4
,e1ad7da
]:[email protected]
Patch Changes
#7803
fd4414b
Thanks @MananTank! - UpdategetTotalClaimedSupply
andgetTotalUnclaimedSupply
erc721 extensions to considerstartTokenId
if the contract hasstartTokenId
set.#7789
9a3ebe1
Thanks @joaquim-verges! - Add transaction hash to max wait time error#7785
e5326c4
Thanks @alecananian! - AddeduseSiweAuth
to the React Native exports#7774
e1ad7da
Thanks @MananTank! - Fixthirdweb/insight
import typescript error@thirdweb-dev/[email protected]
PR-Codex overview
This PR focuses on updating package versions and changelogs for various packages, specifically
thirdweb
,nebula
, andwagmi-adapter
. It includes minor version increments and updates to dependencies.Detailed summary
.changeset
markdown files.version
inpackages/thirdweb/package.json
from5.105.26
to5.105.27
.version
inpackages/nebula/package.json
from0.2.26
to0.2.27
.version
inpackages/wagmi-adapter/package.json
from0.2.123
to0.2.124
.nebula
andthirdweb
, detailing patch changes and contributions from various authors.Summary by CodeRabbit
@thirdweb-dev/nebula
,@thirdweb-dev/wagmi-adapter
, andthirdweb
.