Skip to content

Conversation

@joaquim-verges
Copy link
Member

@joaquim-verges joaquim-verges commented Sep 14, 2025

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

[email protected]

Patch Changes

@thirdweb-dev/[email protected]


PR-Codex overview

This PR focuses on updating version numbers across multiple packages and includes changelog entries reflecting these updates, particularly for thirdweb, nebula, and wagmi-adapter.

Detailed summary

  • Deleted fuzzy-suits-wear.md from .changeset.
  • Updated version in packages/thirdweb/package.json from 5.105.46 to 5.105.47.
  • Updated version in packages/nebula/package.json from 0.2.46 to 0.2.47.
  • Updated version in packages/wagmi-adapter/package.json from 0.2.143 to 0.2.144.
  • Added changelog entries for nebula and thirdweb reflecting the new versions and associated changes.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Summary by CodeRabbit

  • Bug Fixes
    • Resolved an issue where fiat payments failed when no wallet was connected.
  • Documentation
    • Updated changelogs to include the latest patch details across packages.
  • Chores
    • Bumped package versions for thirdweb, nebula, and wagmi-adapter to the latest patch releases.
    • Cleaned up a redundant changeset entry related to the fiat payments fix.

@joaquim-verges joaquim-verges requested review from a team as code owners September 14, 2025 10:56
@vercel
Copy link

vercel bot commented Sep 14, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
docs-v2 Ready Ready Preview Comment Sep 14, 2025 11:25am
nebula Ready Ready Preview Comment Sep 14, 2025 11:25am
thirdweb_playground Ready Ready Preview Comment Sep 14, 2025 11:25am
thirdweb-www Ready Ready Preview Comment Sep 14, 2025 11:25am
wallet-ui Ready Ready Preview Comment Sep 14, 2025 11:25am

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 14, 2025

Walkthrough

Removed a changeset file, bumped package versions for nebula, thirdweb, and wagmi-adapter, and added corresponding changelog entries. No source code or public API changes.

Changes

Cohort / File(s) Summary
Changeset cleanup
.changeset/fuzzy-suits-wear.md
Deleted changeset entry referencing a thirdweb patch note.
Version bumps (package.json)
packages/nebula/package.json, packages/thirdweb/package.json, packages/wagmi-adapter/package.json
Incremented versions: nebula 0.2.46 → 0.2.47; thirdweb 5.105.46 → 5.105.47; wagmi-adapter 0.2.143 → 0.2.144.
Changelogs
packages/nebula/CHANGELOG.md, packages/thirdweb/CHANGELOG.md, packages/wagmi-adapter/CHANGELOG.md
Added entries: nebula 0.2.47 (deps updated to [email protected]); thirdweb 5.105.47 (fix fiat payments with no wallets connected); wagmi-adapter 0.2.144 header added.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

packages, SDK

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning The PR description includes autogenerated release notes and a PR-Codex summary that document the version bumps and changelog entries, but it does not follow the repository's required description_template: it lacks the specified PR title header format, the issue tag field, a "Notes for the reviewer" section, and a "How to test" section. These template sections are present in the description_template and are expected for reviewer context and testing instructions. Because those required template fields are missing, the description does not meet the repository's template requirements. Please update the PR description to include the repository template fields: a title following the specified format (e.g., "[SDK/Dashboard/Portal] Feature/Fix: Concise title"), the issue tag if applicable, a "Notes for the reviewer" section describing important details or rationale, and a "How to test" section with verification steps or test commands; you may keep the autogenerated release notes and PR-Codex content but add those template sections so reviewers have the expected context for review and testing.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title "Version Packages" is concise and directly related to the primary changeset, which updates package versions and changelogs across the repo. It communicates the main intent (a release/versioning PR) and accurately reflects the changes. The title is somewhat generic and does not follow the repository's preferred PR title format, which would make intent and scope clearer to reviewers.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch changeset-release/main

📜 Recent 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.

📥 Commits

Reviewing files that changed from the base of the PR and between befb58f and f14c85d.

📒 Files selected for processing (7)
  • .changeset/fuzzy-suits-wear.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/fuzzy-suits-wear.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)
  • packages/nebula/package.json
  • packages/wagmi-adapter/CHANGELOG.md
  • packages/thirdweb/package.json
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
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
⏰ 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, vite)
  • GitHub Check: E2E Tests (pnpm, esbuild)
  • GitHub Check: E2E Tests (pnpm, webpack)
  • GitHub Check: Unit Tests
  • GitHub Check: Lint Packages
  • GitHub Check: Size
  • GitHub Check: Analyze (javascript)
🔇 Additional comments (1)
packages/thirdweb/CHANGELOG.md (1)

3-8: Changelog entry looks good; patch bump is appropriate.

  • Version: 5.105.47 correctly follows 5.105.46.
  • Style and link formats are consistent with prior entries.

Please confirm:


Comment @coderabbitai help to get the list of available commands and usage tips.

@graphite-app
Copy link
Contributor

graphite-app bot commented Sep 14, 2025

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • merge-queue - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

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.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 14, 2025

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
thirdweb (esm) 63.96 KB (0%) 1.3 s (0%) 653 ms (+213.32% 🔺) 2 s
thirdweb (cjs) 361.44 KB (0%) 7.3 s (0%) 2.2 s (+16.38% 🔺) 9.4 s
thirdweb (minimal + tree-shaking) 5.73 KB (0%) 115 ms (0%) 323 ms (+3549.61% 🔺) 438 ms
thirdweb/chains (tree-shaking) 526 B (0%) 11 ms (0%) 178 ms (+4111.05% 🔺) 189 ms
thirdweb/react (minimal + tree-shaking) 19.15 KB (0%) 383 ms (0%) 207 ms (+1948.5% 🔺) 590 ms

@codecov
Copy link

codecov bot commented Sep 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.53%. Comparing base (778bf2c) to head (f14c85d).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8049   +/-   ##
=======================================
  Coverage   56.52%   56.53%           
=======================================
  Files         904      904           
  Lines       58864    58864           
  Branches     4166     4166           
=======================================
+ Hits        33275    33280    +5     
+ Misses      25483    25478    -5     
  Partials      106      106           
Flag Coverage Δ
packages 56.53% <ø> (+<0.01%) ⬆️
see 2 files with indirect coverage changes
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@coderabbitai coderabbitai bot left a 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 brief note under 0.2.144 for clarity.

Empty entry is fine, but a one‑liner helps consumers scanning the log.

 ## 0.2.144
+
+### Patch Changes
+- Version bump only via Changesets; no source changes in wagmi-adapter.
📜 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.

📥 Commits

Reviewing files that changed from the base of the PR and between 97e5080 and befb58f.

📒 Files selected for processing (7)
  • .changeset/fuzzy-suits-wear.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/fuzzy-suits-wear.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/nebula/package.json
  • packages/wagmi-adapter/package.json
  • packages/thirdweb/package.json
packages/wagmi-adapter/**

📄 CodeRabbit inference engine (CLAUDE.md)

Wagmi ecosystem integration is in packages/wagmi-adapter/

Files:

  • packages/wagmi-adapter/package.json
  • packages/wagmi-adapter/CHANGELOG.md
🧠 Learnings (2)
📚 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.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/package.json
  • packages/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). (9)
  • GitHub Check: Size
  • GitHub Check: E2E Tests (pnpm, vite)
  • GitHub Check: E2E Tests (pnpm, esbuild)
  • GitHub Check: E2E Tests (pnpm, webpack)
  • GitHub Check: Lint Packages
  • GitHub Check: Unit Tests
  • GitHub Check: Build Packages
  • GitHub Check: Socket Security: Pull Request Alerts
  • GitHub Check: Analyze (javascript)
🔇 Additional comments (9)
packages/nebula/package.json (2)

60-60: Nebula version bump looks good.

No API surface changes; aligns with dependency update to [email protected].


46-56: Add size-limit budget for packages/nebula or confirm central coverage

  • Repo search: "size-limit" exists at packages/thirdweb/package.json and packages/thirdweb/.size-limit.json; CI uses size-limit-action in .github/workflows/CI.yml. packages/nebula/package.json has no "size-limit" entry.
  • Action: either add a package.json#size-limit entry (or a .size-limit.json) to packages/nebula so it’s tracked, or confirm which central budget (packages/thirdweb or root) already includes nebula and that CI will cover it.
packages/wagmi-adapter/package.json (2)

58-58: Wagmi adapter version bump LGTM.

Peer range "^5.85.0" already admits thirdweb 5.105.47; no contract issues.


45-54: Add size-limit or confirm central config for packages/wagmi-adapter

packages/wagmi-adapter/package.json has no "size-limit". The repo contains packages/thirdweb/.size-limit.json and CI uses the size-limit action, while root package.json has no top-level "size-limit". Either add a per-package size-limit entry to packages/wagmi-adapter/package.json or confirm packages/thirdweb/.size-limit.json intentionally covers wagmi-adapter and will be picked up by CI.

packages/thirdweb/package.json (2)

420-420: Core package version bump LGTM.

Patch release matches the fiat-payments fix; no export changes.


317-345: Ensure size-limit budgets are configured and enforced in CI.
packages/thirdweb/.size-limit.json exists — confirm CI executes this package's "size" script (runs size-limit) to gate PRs; if it doesn't, add a workflow step to run the size check for packages/thirdweb.

packages/nebula/CHANGELOG.md (1)

3-9: Changelog entry reads correctly for a patch dependency bump.

Links and versions look consistent with the release notes.

Quick check that commit 97e5080 renders and matches PR #8048 when viewing on GitHub.

packages/thirdweb/CHANGELOG.md (1)

3-8: Changelog entry LGTM.

Formatting, links, and version placement match the existing style.

packages/wagmi-adapter/CHANGELOG.md (1)

3-4: Verified — CHANGELOG and package.json versions match (0.2.144)
packages/wagmi-adapter/package.json and packages/wagmi-adapter/CHANGELOG.md both list 0.2.144 as the topmost entry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packages SDK Involves changes to the thirdweb SDK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants