SDK: Fix PayEmbed not rendering anything when mode prop is not set #7858
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🦋 Changeset detectedLatest commit: 7f11657 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
WalkthroughAdds a changeset documenting PayEmbed defaults; updates PayEmbed to render BuyWidget when Changes
Sequence Diagram(s)sequenceDiagram
participant App as App
participant PayEmbed as PayEmbed
participant BuyWidget as BuyWidget
App->>PayEmbed: render(props)
alt mode is undefined or "fund_wallet"
PayEmbed->>BuyWidget: render({ amount: props?.payOptions?.prefillBuy?.amount || "0.01", chain: props?.payOptions?.prefillBuy?.chain || ethereum, tokenAddress: props?.payOptions?.prefillBuy?.token?.address })
else
PayEmbed-->>App: render alternate UI for other modes
end
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Warning Review ran into problems🔥 ProblemsErrors were encountered while retrieving linked issues. Errors (1)
📜 Recent review detailsConfiguration used: CodeRabbit UI 💡 Knowledge Base configuration:
You can enable these sources in your CodeRabbit configuration. 📒 Files selected for processing (6)
💤 Files with no reviewable changes (3)
✅ Files skipped from review due to trivial changes (1)
🚧 Files skipped from review as they are similar to previous changes (2)
⏰ 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)
✨ 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/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
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. This stack of pull requests is managed by Graphite. Learn more about stacking. |
![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
🔭 Outside diff range comments (1)
packages/thirdweb/src/react/web/ui/PayEmbed.tsx (1)
332-354: Default-to-fund_wallet behavior: good; prefer nullish coalescing for fallbacksThe broadened condition to render BuyWidget when
modeis omitted or"fund_wallet"matches the PR goal. For the defaults, prefer??over||so we only fall back onnull/undefinedand not on other falsy-but-valid values (e.g., empty string).- amount={props.payOptions?.prefillBuy?.amount || "0.01"} - chain={props.payOptions?.prefillBuy?.chain || ethereum} + amount={props.payOptions?.prefillBuy?.amount ?? "0.01"} + chain={props.payOptions?.prefillBuy?.chain ?? ethereum}Nit: Consider updating the component JSDoc “Default configuration” section above to mention the default amount (
"0.01") and fallback chain (ethereum) to keep docs in sync. Happy to draft that if helpful.
🧹 Nitpick comments (4)
.changeset/cold-pigs-dress.md (1)
1-5: Changelog entry reads well; minor clarity nitConsider tightening the phrasing and using code formatting for props/values to make the note clearer in release notes.
-Fix PayEmbed UI when mode prop is not specified - Default to mode: "fund_wallet" with amount: "0.01" and chain: ethereum +Fix PayEmbed UI when the `mode` prop is omitted. Default behavior: `mode: "fund_wallet"`, `amount: "0.01"`, `chain: ethereum`.packages/thirdweb/src/react/web/ui/PayEmbed.tsx (1)
350-351: Avoid unnecessary type assertions if possibleIf
prefillBuy.token.addressis already typed asAddress, theas Address | undefinedassertion isn’t needed. If it’s a plainstring, consider tightening thePayUIOptions.prefillBuy.token.addresstype toAddressso we can drop the assertion and catch invalid addresses at compile time.packages/thirdweb/src/stories/PayEmbed.stories.tsx (2)
10-19: Add component to meta for better inference and docsAdding
component: PayEmbedto meta improves arg type inference and autodocs rendering across Storybook versions.-const meta = { +const meta = { + component: PayEmbed, args: { client: storyClient, }, parameters: { layout: "centered", }, tags: ["autodocs"], title: "Connect/PayEmbed", } satisfies Meta<typeof PayEmbed>;
21-217: Deduplicate client prop via meta.args and annotate return types
- You already set
clientinmeta.args; you can removeclient={storyClient}from each story to reduce repetition.- Per repo guidelines, add explicit return types to exported story functions.
Example refactor for a couple of stories:
-export function BasicUsage() { - return <PayEmbed client={storyClient} />; +export function BasicUsage(): JSX.Element { + return <PayEmbed />; } -export function BasicUsageWithMetadata() { +export function BasicUsageWithMetadata(): JSX.Element { return ( <PayEmbed - client={storyClient} payOptions={{ metadata: { name: "this is a title", // This is not shown in UI - TODO fix this description: "this is a description", }, }} /> ); }Apply the same pattern to the remaining stories for consistency. If you prefer keeping explicit
clientprops in stories, consider removingmeta.args.clientinstead.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these settings in your CodeRabbit configuration.
📒 Files selected for processing (4)
.changeset/cold-pigs-dress.md(1 hunks)packages/thirdweb/src/react/web/ui/PayEmbed.tsx(3 hunks)packages/thirdweb/src/stories/PayEmbed.stories.ts(0 hunks)packages/thirdweb/src/stories/PayEmbed.stories.tsx(1 hunks)
💤 Files with no reviewable changes (1)
- packages/thirdweb/src/stories/PayEmbed.stories.ts
🧰 Additional context used
📓 Path-based instructions (3)
**/*.{ts,tsx}
📄 CodeRabbit Inference Engine (CLAUDE.md)
**/*.{ts,tsx}: Write idiomatic TypeScript with explicit function declarations and return types
Limit each file to one stateless, single-responsibility function for clarity
Re-use shared types from@/typesor localtypes.tsbarrels
Prefer type aliases over interface except for nominal shapes
Avoidanyandunknownunless unavoidable; narrow generics when possible
Choose composition over inheritance; leverage utility types (Partial,Pick, etc.)
Comment only ambiguous logic; avoid restating TypeScript in prose
Files:
packages/thirdweb/src/react/web/ui/PayEmbed.tsxpackages/thirdweb/src/stories/PayEmbed.stories.tsx
**/*.{ts,tsx,js,jsx}
📄 CodeRabbit Inference Engine (CLAUDE.md)
Load heavy dependencies inside async paths to keep initial bundle lean (lazy loading)
Files:
packages/thirdweb/src/react/web/ui/PayEmbed.tsxpackages/thirdweb/src/stories/PayEmbed.stories.tsx
**/*.stories.tsx
📄 CodeRabbit Inference Engine (CLAUDE.md)
For new UI components, add Storybook stories (
*.stories.tsx) alongside the code
Files:
packages/thirdweb/src/stories/PayEmbed.stories.tsx
🧠 Learnings (6)
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Interactive UI that relies on hooks (`useState`, `useEffect`, React Query, wallet hooks).
Applied to files:
packages/thirdweb/src/react/web/ui/PayEmbed.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Anything that consumes hooks from `tanstack/react-query` or thirdweb SDKs.
Applied to files:
packages/thirdweb/src/react/web/ui/PayEmbed.tsx
📚 Learning: 2025-06-17T18:30:52.976Z
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/thirdweb/src/react/web/ui/PayEmbed.tsx
📚 Learning: 2025-05-30T17:14:25.332Z
Learnt from: MananTank
PR: thirdweb-dev/js#7227
File: apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/modules/components/OpenEditionMetadata.tsx:26-26
Timestamp: 2025-05-30T17:14:25.332Z
Learning: The ModuleCardUIProps interface already includes a client prop of type ThirdwebClient, so when components use `Omit<ModuleCardUIProps, "children" | "updateButton">`, they inherit the client prop without needing to add it explicitly.
Applied to files:
packages/thirdweb/src/react/web/ui/PayEmbed.tsx
📚 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 **/*.stories.tsx : For new UI components, add Storybook stories (`*.stories.tsx`) alongside the code
Applied to files:
packages/thirdweb/src/stories/PayEmbed.stories.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/components/*.{stories,test}.{tsx,ts} : Provide a Storybook story (`MyComponent.stories.tsx`) or unit test alongside the component.
Applied to files:
packages/thirdweb/src/stories/PayEmbed.stories.tsx
⏰ 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, vite)
- GitHub Check: E2E Tests (pnpm, webpack)
- GitHub Check: Unit Tests
- GitHub Check: Lint Packages
- GitHub Check: Build Packages
- GitHub Check: Analyze (javascript)
🔇 Additional comments (1)
packages/thirdweb/src/react/web/ui/PayEmbed.tsx (1)
4-4: Fallback chain import LGTMImporting
ethereumas the default fallback chain aligns with the new default behavior and avoids null/undefined issues when no chain is provided.
size-limit report 📦
|
MananTank
force-pushed
the
08-15-sdk_fix_payembed_not_rendering_anything_when_mode_prop_is_not_set
branch
from
3009c69 to
bc960e4
Compare
MananTank
force-pushed
the
08-15-sdk_fix_payembed_not_rendering_anything_when_mode_prop_is_not_set
branch
from
bc960e4 to
6949483
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7858 +/- ##
==========================================
+ Coverage 56.33% 56.54% +0.21%
==========================================
Files 907 904 -3
Lines 58965 58586 -379
Branches 4159 4140 -19
==========================================
- Hits 33216 33126 -90
+ Misses 25643 25354 -289
Partials 106 106
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
packages/thirdweb/src/stories/PayEmbed.stories.tsx (1)
1-1: Switch Storybook type import to@storybook/react(not@storybook/react-vite).Unifies typings across the repo and avoids version-specific type resolution issues.
-import type { Meta } from "@storybook/react-vite"; +import type { Meta } from "@storybook/react";
🧹 Nitpick comments (3)
packages/thirdweb/src/stories/PayEmbed.stories.tsx (3)
10-16: Addcomponent: PayEmbedto meta for proper autodocs and controls.Without
component, Storybook may not link docs/controls correctly even withsatisfies Meta<typeof PayEmbed>.const meta = { parameters: { layout: "centered", }, tags: ["autodocs"], + component: PayEmbed, title: "Connect/PayEmbed", } satisfies Meta<typeof PayEmbed>;Optional: You can also DRY the stories by setting default args here:
// Example args: { client: storyClient },…and drop
client={storyClient}from individual stories.
18-21: Add explicit return types to story functions.Guideline for TS/TSX: explicit return types. Declare
: JSX.Elementon each exported story.-export function BasicUsage() { +export function BasicUsage(): JSX.Element { return <PayEmbed client={storyClient} />; } -export function BasicUsageWithMetadata() { +export function BasicUsageWithMetadata(): JSX.Element { return ( <PayEmbed client={storyClient} payOptions={{ metadata: { name: "this is a title", // This is not shown in UI - TODO fix this description: "this is a description", }, }} /> ); } -export function FundWallet() { +export function FundWallet(): JSX.Element { return ( <PayEmbed client={storyClient} payOptions={{ mode: "fund_wallet", }} /> ); } -export function FundWalletWithMetadata() { +export function FundWalletWithMetadata(): JSX.Element { return ( <PayEmbed client={storyClient} payOptions={{ mode: "fund_wallet", metadata: { name: "this is a title", // This is not shown in UI - TODO fix this description: "this is a description", }, }} /> ); } -export function FundWalletWithOptions() { +export function FundWalletWithOptions(): JSX.Element { return ( <PayEmbed client={storyClient} payOptions={{ mode: "fund_wallet", prefillBuy: { amount: "0.123", chain: polygon, }, }} /> ); } -export function FundWalletERC20() { +export function FundWalletERC20(): JSX.Element { return ( <PayEmbed client={storyClient} payOptions={{ mode: "fund_wallet", prefillBuy: { amount: "0.123", chain: base, token: { address: "0x833589fcd6edb6e08f4c7c32d4f71b54bda02913", name: "USDC", // This icon is not being used - TODO fix this, either remove this prop or use it icon: "https://picsum.photos/200/200", }, }, }} /> ); } -export function DirectPayment() { +export function DirectPayment(): JSX.Element { return ( <PayEmbed client={storyClient} payOptions={{ mode: "direct_payment", paymentInfo: { amount: "0.123", chain: polygon, sellerAddress: "0x83Dd93fA5D8343094f850f90B3fb90088C1bB425", }, }} /> ); } -export function DirectPaymentERC20() { +export function DirectPaymentERC20(): JSX.Element { return ( <PayEmbed client={storyClient} payOptions={{ mode: "direct_payment", paymentInfo: { amount: "0.123", chain: base, sellerAddress: "0x83Dd93fA5D8343094f850f90B3fb90088C1bB425", token: { address: "0x833589fcd6edb6e08f4c7c32d4f71b54bda02913", name: "USDC", // This icon is not being used - TODO fix this, either remove this prop or use it icon: "https://coin-images.coingecko.com/coins/images/6319/large/usdc.png", }, }, }} /> ); } -export function DirectPaymentWithMetadata() { +export function DirectPaymentWithMetadata(): JSX.Element { return ( <PayEmbed client={storyClient} payOptions={{ mode: "direct_payment", paymentInfo: { amount: "0.123", chain: polygon, sellerAddress: "0x83Dd93fA5D8343094f850f90B3fb90088C1bB425", }, metadata: { name: "this is a title", description: "this is a description", }, }} /> ); } -export function Transaction() { +export function Transaction(): JSX.Element { return ( <PayEmbed client={storyClient} payOptions={{ mode: "transaction", transaction: prepareTransaction({ to: "0x83Dd93fA5D8343094f850f90B3fb90088C1bB425", chain: polygon, client: storyClient, value: toWei("0.123"), }), }} /> ); } -export function TransactionWithMetadata() { +export function TransactionWithMetadata(): JSX.Element { return ( <PayEmbed client={storyClient} payOptions={{ metadata: { name: "this is a title", description: "this is a description", }, mode: "transaction", transaction: prepareTransaction({ to: "0x83Dd93fA5D8343094f850f90B3fb90088C1bB425", chain: polygon, client: storyClient, value: toWei("0.123"), }), }} /> ); } -export function LightMode() { +export function LightMode(): JSX.Element { return <PayEmbed client={storyClient} theme="light" />; } -export function CustomLightTheme() { +export function CustomLightTheme(): JSX.Element { return ( <PayEmbed client={storyClient} theme={lightTheme({ colors: { modalBg: "#FFFFF0", tertiaryBg: "#DBE4C9", borderColor: "#8AA624", secondaryText: "#3E3F29", accentText: "#E43636", }, })} /> ); }Also applies to: 22-35, 37-46, 48-62, 64-77, 79-98, 100-114, 116-136, 138-156, 158-173, 175-194, 196-198, 200-215
27-31: Track or resolve TODOs shown in the stories (hidden description and unused token icon).These props don’t currently affect the UI and may confuse readers of the stories/docs. Either remove them from the examples for now or open an issue to implement support.
Happy to open a tracking issue or submit a follow-up PR to either wire these through or simplify the examples.
Also applies to: 55-58, 91-93, 149-152, 181-183
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (6)
.changeset/cold-pigs-dress.md(1 hunks)packages/thirdweb/src/react/web/ui/PayEmbed-disconnected.test.tsx(0 hunks)packages/thirdweb/src/react/web/ui/PayEmbed.test.tsx(0 hunks)packages/thirdweb/src/react/web/ui/PayEmbed.tsx(3 hunks)packages/thirdweb/src/stories/PayEmbed.stories.ts(0 hunks)packages/thirdweb/src/stories/PayEmbed.stories.tsx(1 hunks)
💤 Files with no reviewable changes (3)
- packages/thirdweb/src/react/web/ui/PayEmbed.test.tsx
- packages/thirdweb/src/react/web/ui/PayEmbed-disconnected.test.tsx
- packages/thirdweb/src/stories/PayEmbed.stories.ts
🚧 Files skipped from review as they are similar to previous changes (2)
- packages/thirdweb/src/react/web/ui/PayEmbed.tsx
- .changeset/cold-pigs-dress.md
🧰 Additional context used
📓 Path-based instructions (3)
**/*.{ts,tsx}
📄 CodeRabbit Inference Engine (CLAUDE.md)
**/*.{ts,tsx}: Write idiomatic TypeScript with explicit function declarations and return types
Limit each file to one stateless, single-responsibility function for clarity
Re-use shared types from@/typesor localtypes.tsbarrels
Prefer type aliases over interface except for nominal shapes
Avoidanyandunknownunless unavoidable; narrow generics when possible
Choose composition over inheritance; leverage utility types (Partial,Pick, etc.)
Comment only ambiguous logic; avoid restating TypeScript in prose
Files:
packages/thirdweb/src/stories/PayEmbed.stories.tsx
**/*.{ts,tsx,js,jsx}
📄 CodeRabbit Inference Engine (CLAUDE.md)
Load heavy dependencies inside async paths to keep initial bundle lean (lazy loading)
Files:
packages/thirdweb/src/stories/PayEmbed.stories.tsx
**/*.stories.tsx
📄 CodeRabbit Inference Engine (CLAUDE.md)
For new UI components, add Storybook stories (
*.stories.tsx) alongside the code
Files:
packages/thirdweb/src/stories/PayEmbed.stories.tsx
🧠 Learnings (2)
📚 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 **/*.stories.tsx : For new UI components, add Storybook stories (`*.stories.tsx`) alongside the code
Applied to files:
packages/thirdweb/src/stories/PayEmbed.stories.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/components/*.{stories,test}.{tsx,ts} : Provide a Storybook story (`MyComponent.stories.tsx`) or unit test alongside the component.
Applied to files:
packages/thirdweb/src/stories/PayEmbed.stories.tsx
🧬 Code Graph Analysis (1)
packages/thirdweb/src/stories/PayEmbed.stories.tsx (4)
packages/thirdweb/src/react/web/ui/PayEmbed.tsx (1)
PayEmbed(303-415)packages/thirdweb/src/stories/utils.tsx (1)
storyClient(15-17)packages/thirdweb/src/exports/chains.ts (2)
polygon(60-60)base(17-17)packages/thirdweb/src/exports/thirdweb.ts (1)
prepareTransaction(181-181)
⏰ 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, webpack)
- GitHub Check: E2E Tests (pnpm, vite)
- GitHub Check: E2E Tests (pnpm, esbuild)
- GitHub Check: Unit Tests
- GitHub Check: Lint Packages
- GitHub Check: Build Packages
- GitHub Check: Analyze (javascript)
Merge activity
|
…7858) <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on updating the `PayEmbed` component to default to a mode of "fund_wallet" with an amount of "0.01" and chain set to `ethereum` when no mode is specified. It also includes new story examples for the `PayEmbed` component. ### Detailed summary - Deleted `PayEmbed.stories.ts` and test files. - Updated `PayEmbed` component to default `mode` to "fund_wallet" and `amount` to "0.01". - Added handling for `chain` defaulting to `ethereum`. - Created multiple story functions for various use cases of `PayEmbed`. - Included stories for different payment modes and configurations. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Bug Fixes** * PayEmbed now defaults to a wallet-funding flow when mode is missing (amount 0.01; Ethereum chain) and tolerates partial prefill data. * **Enhancements** * Buy flow renders more flexibly when mode is omitted or set to fund_wallet. * **Documentation** * Storybook updated/reorganized with comprehensive PayEmbed examples: fund wallet, direct payments, transactions, ERC20, metadata, and theme variants. * **Chores** * Added a changeset for a patch release. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
graphite-app
bot
force-pushed
the
08-15-sdk_fix_payembed_not_rendering_anything_when_mode_prop_is_not_set
branch
from
6949483 to
7f11657
Compare
graphite-app
bot
deleted the
08-15-sdk_fix_payembed_not_rendering_anything_when_mode_prop_is_not_set
branch
PR-Codex overview
This PR focuses on enhancing the
PayEmbedcomponent in thethirdwebpackage by fixing the default behavior when themodeprop is not specified and adding multiple storybook examples for various payment scenarios.Detailed summary
modeset to"fund_wallet"withamount: "0.01"andchain: ethereumwhen not specified.PayEmbedto handlepayOptions.Summary by CodeRabbit
Bug Fixes
Enhancements
Documentation
Tests
Chores