-
Notifications
You must be signed in to change notification settings - Fork 629
Version Packages #8554
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
base: main
Are you sure you want to change the base?
Version Packages #8554
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughRemoved three changeset files and added/updated changelog entries and package.json version bumps across several packages and an app; no source code or exported API declarations were changed. 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)
Comment |
7064dc8 to
f7c9199
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: 1
📜 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 (10)
.changeset/ninety-trains-tell.md(0 hunks).changeset/violet-needles-fly.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 (2)
- .changeset/ninety-trains-tell.md
- .changeset/violet-needles-fly.md
🧰 Additional context used
📓 Path-based instructions (2)
**/*.{js,jsx,ts,tsx,json}
📄 CodeRabbit inference engine (AGENTS.md)
Biome governs formatting and linting; its rules live in biome.json. Run
pnpm fix&pnpm lintbefore committing, ensure there are no linting errors
Files:
packages/wagmi-adapter/package.jsonpackages/nebula/package.jsonapps/wagmi-demo/package.jsonpackages/thirdweb/package.json
packages/thirdweb/package.json
📄 CodeRabbit inference engine (CLAUDE.md)
Track bundle budgets via
package.json#size-limit
Files:
packages/thirdweb/package.json
⏰ 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: E2E Tests (pnpm, webpack)
- GitHub Check: Unit Tests
- GitHub Check: Size
- GitHub Check: Build Packages
- GitHub Check: Lint Packages
- GitHub Check: Analyze (javascript)
🔇 Additional comments (7)
packages/nebula/package.json (1)
60-60: Version bump looks consistent with a patch release.
No concerns in this file beyond the version increment.apps/wagmi-demo/package.json (1)
4-4: Private app version bump is fine.
No other changes/risk in this file.packages/wagmi-adapter/package.json (1)
59-59: Patch version bump looks correct; peer range remains compatible.
[email protected]is withinpeerDependencies.thirdweb: ^5.85.0, so this release bump is consistent.packages/thirdweb/package.json (1)
433-433: Version bump only; please ensure bundle budgets still pass.Since
packages/thirdweb/package.jsontracks bundle budgets viasize-limit, it's worth confirmingpnpm -C packages/thirdweb sizeandpnpm lintremain green. (CI will also verify these on the PR via the size and lint jobs.)packages/thirdweb/CHANGELOG.md (1)
3-10: Changelog entry looks consistent and correctly scoped to patch notes.
5.116.2is added at the top, uses the same heading structure as prior entries, and clearly attributes PRs/commits/authors for the two fixes.packages/nebula/CHANGELOG.md (1)
3-9: Changelog entry looks consistent and complete for a dependency-only patch release.apps/wagmi-demo/CHANGELOG.md (1)
3-10: Changelog entry is clear and matches the established “Updated dependencies” format.
| ## 0.2.201 | ||
|
|
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.
Avoid empty changelog sections; add at least a minimal “Patch Changes” note for 0.2.201.
Suggested patch:
## 0.2.201
+
+### Patch Changes
+
+- No user-facing changes.📝 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.201 | |
| ## 0.2.201 | |
| ### Patch Changes | |
| - No user-facing changes. |
🤖 Prompt for AI Agents
In packages/wagmi-adapter/CHANGELOG.md around lines 3 to 4, the 0.2.201 entry is
empty; add a minimal "Patch Changes" section under the version header with
one-line note describing the patch (e.g., "Patch Changes: minor fixes and
documentation updates" or the actual fix summary). Update the file so 0.2.201
includes the "Patch Changes" heading and a concise bullet or sentence, commit
the change.
size-limit report 📦
|
f7c9199 to
cd790c9
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8554 +/- ##
==========================================
- Coverage 54.47% 54.45% -0.02%
==========================================
Files 922 922
Lines 61361 61361
Branches 4149 4146 -3
==========================================
- Hits 33425 33416 -9
- Misses 27835 27845 +10
+ Partials 101 100 -1
🚀 New features to boost your workflow:
|
cd790c9 to
ad13a2c
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-14: Changelog entry looks correct; consider normalizing wording/capitalization (“UI”).
New5.116.2section is well-formed and links look consistent. Minor nit: “wide connect ui” → “wide connect UI” to match surrounding style.
📜 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 (11)
.changeset/eager-loops-obey.md(0 hunks).changeset/ninety-trains-tell.md(0 hunks).changeset/violet-needles-fly.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 (3)
- .changeset/violet-needles-fly.md
- .changeset/eager-loops-obey.md
- .changeset/ninety-trains-tell.md
🚧 Files skipped from review as they are similar to previous changes (4)
- packages/thirdweb/package.json
- packages/wagmi-adapter/CHANGELOG.md
- packages/nebula/package.json
- apps/wagmi-demo/package.json
🧰 Additional context used
📓 Path-based instructions (1)
**/*.{js,jsx,ts,tsx,json}
📄 CodeRabbit inference engine (AGENTS.md)
Biome governs formatting and linting; its rules live in biome.json. Run
pnpm fix&pnpm lintbefore committing, ensure there are no linting errors
Files:
packages/wagmi-adapter/package.json
⏰ 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: E2E Tests (pnpm, esbuild)
- GitHub Check: E2E Tests (pnpm, webpack)
- GitHub Check: E2E Tests (pnpm, vite)
- GitHub Check: Build Packages
- GitHub Check: Lint Packages
- GitHub Check: Unit Tests
- GitHub Check: Analyze (javascript)
🔇 Additional comments (3)
packages/wagmi-adapter/package.json (1)
59-59: Version bump looks consistent for release automation.Only metadata change; aligns with the release PR intent.
apps/wagmi-demo/CHANGELOG.md (1)
1-10: No action needed — changelog header and dependencies are correct.The package name
wagmi-inappin the changelog header matchesapps/wagmi-demo/package.json, version0.0.46is correct, and the declared dependency versions (@thirdweb-dev/[email protected]) align with the entry. The app usesworkspace:*references, which is appropriate for this monorepo structure.packages/nebula/CHANGELOG.md (1)
3-8: Changelog entry looks good.The new version 0.2.104 entry is properly formatted and consistent with existing entries. The commit references and dependency update to [email protected] align with the PR objectives. The patch version bump is appropriate for dependency-only updates.
ad13a2c to
fa097ae
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-13: Changelog entry looks correct; tiny wording nitThe 5.116.2 entry correctly lists the four PRs described in the PR summary and matches existing formatting. The only tiny consistency nit is “wide connect ui” in Line 11 — elsewhere in the changelog similar phrases use “UI” uppercase. Consider changing
wide connect ui→wide connect UIfor consistency, but it’s purely editorial.
📜 Review details
Configuration used: Organization 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 (12)
.changeset/eager-loops-obey.md(0 hunks).changeset/ninety-trains-tell.md(0 hunks).changeset/sixty-clubs-fix.md(0 hunks).changeset/violet-needles-fly.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 (4)
- .changeset/ninety-trains-tell.md
- .changeset/eager-loops-obey.md
- .changeset/violet-needles-fly.md
- .changeset/sixty-clubs-fix.md
✅ Files skipped from review due to trivial changes (1)
- apps/wagmi-demo/CHANGELOG.md
🚧 Files skipped from review as they are similar to previous changes (4)
- packages/wagmi-adapter/package.json
- packages/thirdweb/package.json
- packages/wagmi-adapter/CHANGELOG.md
- apps/wagmi-demo/package.json
🧰 Additional context used
📓 Path-based instructions (1)
**/*.{js,jsx,ts,tsx,json}
📄 CodeRabbit inference engine (AGENTS.md)
Biome governs formatting and linting; its rules live in biome.json. Run
pnpm fix&pnpm lintbefore committing, ensure there are no linting errors
Files:
packages/nebula/package.json
⏰ 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, webpack)
- GitHub Check: E2E Tests (pnpm, esbuild)
- GitHub Check: Build Packages
- GitHub Check: Lint Packages
- GitHub Check: Unit Tests
- GitHub Check: E2E Tests (pnpm, vite)
- GitHub Check: Size
- GitHub Check: Analyze (javascript)
🔇 Additional comments (2)
packages/nebula/package.json (1)
60-60: LGTM! Version bump is correct.The patch version increment from 0.2.103 to 0.2.104 aligns with the automated release process and matches the corresponding CHANGELOG entry.
packages/nebula/CHANGELOG.md (1)
3-9: LGTM! Changelog entry is well-formed.The new 0.2.104 release entry correctly documents the dependency update to [email protected] with proper commit references, consistent with the automated release process.
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
5ec84bd,31e24f1,151127d,38f4a89]:[email protected]
Patch Changes
#8552
5ec84bdThanks @MananTank! - Token Selection UX improvements in SwapWidget#8551
31e24f1Thanks @MananTank! - Alphabetically sort the chains in SwapWidget UI#8556
151127dThanks @MananTank! - Update in-app wallet icon in wide connect ui#8553
38f4a89Thanks @MananTank! - Fix Spinner animation not working sometimes in Next.js@thirdweb-dev/[email protected]
[email protected]
Patch Changes
5ec84bd,31e24f1,151127d,38f4a89]:PR-Codex overview
This PR focuses on updating version numbers across multiple packages, including
thirdweb,wagmi-demo, andwagmi-adapter, while also documenting changes in their respective changelogs.Detailed summary
.changesetmarkdown files.versioninpackages/thirdweb/package.jsonfrom5.116.1to5.116.2.versioninapps/wagmi-demo/package.jsonfrom0.0.45to0.0.46.versioninpackages/nebula/package.jsonfrom0.2.103to0.2.104.versioninpackages/wagmi-adapter/package.jsonfrom0.2.200to0.2.201.packages/nebulaandapps/wagmi-demofor new versions, noting updates to dependencies.packages/thirdweb/CHANGELOG.mdfor version5.116.2, highlighting specific improvements and fixes.Summary by CodeRabbit
New Features
Bug Fixes
✏️ Tip: You can customize this high-level summary in your review settings.