[PRO-96] Playground: Update product grouping #8401
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
github-actions
bot
added
the
Playground
MananTank
changed the title
Member
Author
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. |
Contributor
WalkthroughRestructures the playground application information architecture by moving Payments into Bridge, deprecating Contracts, repositioning Account Abstraction and Transactions under Wallets, elevating x402 to a top-level category, and updating corresponding routing and documentation links across playground, dashboard, and portal sites. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Nav as Navigation
participant Playground as Playground App
participant Docs as Portal/Dashboard
rect rgb(220,240,250)
note over Nav,Playground: New navigation flow
User->>Nav: Click "Bridge" or a Bridge widget
Nav->>Playground: /bridge or /bridge/<widget>
Playground-->>User: Render Bridge overview or widget page
end
rect rgb(255,240,200)
note over User,Playground: Legacy route handling via redirects
User->>Playground: Navigate to /payments/* or /connect/pay/*
Playground->>Playground: Redirect (next.config) → /bridge or /bridge/<widget> (permanent: false)
Playground-->>User: Final Bridge/widget page
end
rect rgb(200,240,220)
note over Docs,Playground: Docs/dashboard links updated
Docs->>User: Article with updated link (/bridge, /wallets, /x402)
User->>Playground: Follow updated link
Playground-->>User: Correct page
end
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Warning Review ran into problems🔥 ProblemsErrors were encountered while retrieving linked issues. Errors (1)
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8401 +/- ##
=======================================
Coverage 54.82% 54.82%
=======================================
Files 919 919
Lines 60840 60840
Branches 4140 4140
=======================================
Hits 33356 33356
Misses 27382 27382
Partials 102 102
🚀 New features to boost your workflow:
|
Contributor
size-limit report 📦
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 3
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (3)
apps/playground-web/src/app/bridge/transaction-button/page.tsx (2)
52-56: Contract address doesn't match the chain.The contract address
0x3c499c542cEF5E3811e1192ce70d8cC03d5c3359is the USDC contract on Polygon, but the code specifiesarbitrumas the chain. This mismatch will cause the example to fail.Apply this diff to use the correct Arbitrum USDC address:
const usdcContract = getContract({ - address: "0x3c499c542cEF5E3811e1192ce70d8cC03d5c3359", + address: "0xaf88d065e77c8cC2239327C5EDb3A432268e5831", chain: arbitrum, client, });
83-96: Price label doesn't match the transaction.The second TransactionButton displays "Price: 0.1 ETH" but the transaction actually transfers 50 USDC (line 90), not ETH. This is misleading and will confuse users trying to understand the example.
Apply this diff to fix the inconsistency:
- <p> Price: 0.1 ETH </p> + <p> Price: 100 USDC </p> <TransactionButton client={client} transaction={() => { if (!account) { throw new Error("No wallet connected"); } return transfer({ contract: usdcContract, - amount: "50", + amount: "100", to: account.address, }); }} > - Buy VIP Pass + Buy Premium Pass </TransactionButton>apps/playground-web/src/app/bridge/components/RightSection.tsx (1)
118-132: Prevent invalid address crash in TransactionWidgetWhen no wallet is connected,
accountisundefined, sotobecomes an empty string.claimToimmediately normalizes this viagetAddressand throwsInvalidAddressError, crashing the entire playground before a user can connect. Please guard the render or fall back to a valid address (e.g., the configured receiver) before invokingclaimTo. For example:- <TransactionWidget + const recipient = + account?.address ?? props.options.payOptions.receiverAddress; + if (!recipient) { + embed = null; + } else { + embed = ( + <TransactionWidget ... - transaction={claimTo({ + transaction={claimTo({ contract: nftContract, quantity: 1n, - to: account?.address || "", + to: recipient, tokenId: 2n, })} ... /> - ); + ); + }
🧹 Nitpick comments (1)
apps/playground-web/src/app/data/pages-metadata.ts (1)
214-221: Update x402 feature link to the new routeWe now route users to
/x402, but this card still points at the legacy/payments/x402path. Even though the redirect covers it, we should link directly to the canonical destination to avoid extra hops and keep the metadata consistent with the new IA.- link: "/payments/x402", + link: "/x402",
📜 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 (22)
apps/playground-web/next.config.mjs(1 hunks)apps/playground-web/src/app/api/paywall/route.ts(1 hunks)apps/playground-web/src/app/bridge/buy-widget/page.tsx(1 hunks)apps/playground-web/src/app/bridge/checkout-widget/page.tsx(1 hunks)apps/playground-web/src/app/bridge/components/RightSection.tsx(1 hunks)apps/playground-web/src/app/bridge/page.tsx(1 hunks)apps/playground-web/src/app/bridge/swap-widget/page.tsx(1 hunks)apps/playground-web/src/app/bridge/transaction-button/page.tsx(2 hunks)apps/playground-web/src/app/bridge/transaction-widget/page.tsx(1 hunks)apps/playground-web/src/app/contracts/events/page.tsx(0 hunks)apps/playground-web/src/app/contracts/extensions/page.tsx(0 hunks)apps/playground-web/src/app/contracts/page.tsx(0 hunks)apps/playground-web/src/app/contracts/read/page.tsx(0 hunks)apps/playground-web/src/app/contracts/write/page.tsx(0 hunks)apps/playground-web/src/app/data/pages-metadata.ts(4 hunks)apps/playground-web/src/app/navLinks.ts(3 hunks)apps/playground-web/src/app/page.tsx(3 hunks)apps/playground-web/src/app/payments/page.tsx(0 hunks)apps/playground-web/src/app/transactions/page.tsx(1 hunks)apps/playground-web/src/app/x402/components/X402RightSection.tsx(1 hunks)apps/playground-web/src/app/x402/page.tsx(1 hunks)apps/playground-web/src/icons/ContractIcon.tsx(0 hunks)
💤 Files with no reviewable changes (7)
- apps/playground-web/src/app/contracts/page.tsx
- apps/playground-web/src/icons/ContractIcon.tsx
- apps/playground-web/src/app/payments/page.tsx
- apps/playground-web/src/app/contracts/events/page.tsx
- apps/playground-web/src/app/contracts/write/page.tsx
- apps/playground-web/src/app/contracts/extensions/page.tsx
- apps/playground-web/src/app/contracts/read/page.tsx
🧰 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:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/api/paywall/route.tsapps/playground-web/src/app/x402/components/X402RightSection.tsxapps/playground-web/src/app/bridge/checkout-widget/page.tsxapps/playground-web/src/app/transactions/page.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/bridge/transaction-widget/page.tsxapps/playground-web/src/app/bridge/buy-widget/page.tsxapps/playground-web/src/app/bridge/swap-widget/page.tsxapps/playground-web/src/app/data/pages-metadata.tsapps/playground-web/src/app/page.tsxapps/playground-web/src/app/bridge/transaction-button/page.tsxapps/playground-web/src/app/navLinks.ts
**/*.{ts,tsx,js,jsx}
📄 CodeRabbit inference engine (CLAUDE.md)
Load heavy dependencies inside async paths to keep initial bundle lean (lazy loading)
Files:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/api/paywall/route.tsapps/playground-web/src/app/x402/components/X402RightSection.tsxapps/playground-web/src/app/bridge/checkout-widget/page.tsxapps/playground-web/src/app/transactions/page.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/bridge/transaction-widget/page.tsxapps/playground-web/src/app/bridge/buy-widget/page.tsxapps/playground-web/src/app/bridge/swap-widget/page.tsxapps/playground-web/src/app/data/pages-metadata.tsapps/playground-web/src/app/page.tsxapps/playground-web/src/app/bridge/transaction-button/page.tsxapps/playground-web/src/app/navLinks.ts
apps/{dashboard,playground-web}/**/*.{ts,tsx}
📄 CodeRabbit inference engine (CLAUDE.md)
apps/{dashboard,playground-web}/**/*.{ts,tsx}: Import UI primitives from@/components/ui/*(Button, Input, Select, Tabs, Card, Sidebar, Badge, Separator) in dashboard and playground apps
UseNavLinkfor internal navigation with automatic active states in dashboard and playground apps
Use Tailwind CSS only – no inline styles or CSS modules
Usecn()from@/lib/utilsfor conditional class logic
Use design system tokens (e.g.,bg-card,border-border,text-muted-foreground)
Server Components (Node edge): Start files withimport "server-only";
Client Components (browser): Begin files with'use client';
Always callgetAuthToken()to retrieve JWT from cookies on server side
UseAuthorization: Bearerheader – never embed tokens in URLs
Return typed results (e.g.,Project[],User[]) – avoidany
Wrap client-side data fetching calls in React Query (@tanstack/react-query)
Use descriptive, stablequeryKeysfor React Query cache hits
ConfigurestaleTime/cacheTimein React Query based on freshness (default ≥ 60s)
Keep tokens secret via internal API routes or server actions
Never importposthog-jsin server components
Files:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/api/paywall/route.tsapps/playground-web/src/app/x402/components/X402RightSection.tsxapps/playground-web/src/app/bridge/checkout-widget/page.tsxapps/playground-web/src/app/transactions/page.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/bridge/transaction-widget/page.tsxapps/playground-web/src/app/bridge/buy-widget/page.tsxapps/playground-web/src/app/bridge/swap-widget/page.tsxapps/playground-web/src/app/data/pages-metadata.tsapps/playground-web/src/app/page.tsxapps/playground-web/src/app/bridge/transaction-button/page.tsxapps/playground-web/src/app/navLinks.ts
🧠 Learnings (56)
📓 Common learnings
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7888
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx:77-81
Timestamp: 2025-08-20T10:35:18.543Z
Learning: The webhooks/payments route exists at apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/payments/page.tsx and was added as part of the unified project layout changes.
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7888
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx:77-81
Timestamp: 2025-08-20T10:35:18.543Z
Learning: The webhooks/payments route exists at apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/payments/page.tsx and was added as part of the unified project layout PR #7888.
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Import UI primitives from `@/components/ui/*` (Button, Input, Select, Tabs, Card, Sidebar, Badge, Separator) in dashboard and playground apps
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/transactions/page.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/bridge/buy-widget/page.tsxapps/playground-web/src/app/bridge/swap-widget/page.tsxapps/playground-web/src/app/page.tsxapps/playground-web/src/app/navLinks.ts
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Server Components (Node edge): Start files with `import "server-only";`
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/api/paywall/route.tsapps/playground-web/src/app/x402/page.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Use `NavLink` for internal navigation with automatic active states in dashboard and playground apps
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/bridge/swap-widget/page.tsxapps/playground-web/src/app/navLinks.ts
📚 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 src/exports/react.native.ts : React Native specific exports are in `src/exports/react.native.ts`
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/x402/page.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Client Components (browser): Begin files with `'use client';`
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/navLinks.ts
📚 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 apps/{dashboard,playground-web}/**/feature/components/** : Group feature-specific components under `feature/components/*` with barrel `index.ts`
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/data/pages-metadata.tsapps/playground-web/src/app/page.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/components/**/index.ts : Group related components in their own folder and expose a single barrel `index.ts` where necessary.
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : Use `NavLink` (`@/components/ui/NavLink`) for internal navigation so active states are handled automatically.
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/navLinks.ts
📚 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 **/*.{ts,tsx} : Choose composition over inheritance; leverage utility types (`Partial`, `Pick`, etc.)
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsx
📚 Learning: 2025-09-05T19:45:05.006Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7984
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx:76-80
Timestamp: 2025-09-05T19:45:05.006Z
Learning: When verifying route existence for Next.js app router, search for the full nested path structure rather than using simple patterns. The AI summary often contains accurate information about file additions that should be referenced before running verification scripts.
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/next.config.mjs
📚 Learning: 2025-05-30T17:14:25.332Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 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:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/api/paywall/route.ts
📚 Learning: 2025-10-03T23:36:00.631Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8181
File: packages/thirdweb/src/react/web/ui/Bridge/BuyWidget.tsx:27-27
Timestamp: 2025-10-03T23:36:00.631Z
Learning: In packages/thirdweb/src/react/web/ui/Bridge/BuyWidget.tsx, the component intentionally uses a hardcoded English locale (connectLocaleEn) rather than reading from the provider, as BuyWidget is designed to be English-only and does not require internationalization support.
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/bridge/checkout-widget/page.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/bridge/transaction-widget/page.tsxapps/playground-web/src/app/bridge/buy-widget/page.tsxapps/playground-web/src/app/bridge/swap-widget/page.tsxapps/playground-web/src/app/bridge/transaction-button/page.tsx
📚 Learning: 2025-09-24T11:09:45.142Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8106
File: packages/thirdweb/src/script-exports/readme.md:6-7
Timestamp: 2025-09-24T11:09:45.142Z
Learning: For thirdweb Bridge Widget script exports, the module is exported with globalName: "BridgeWidget" in tsup config, making the global API `BridgeWidget.render()` rather than `window.thirdweb.BridgeWidget.render()`.
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/bridge/buy-widget/page.tsxapps/playground-web/src/app/bridge/swap-widget/page.tsx
📚 Learning: 2025-06-17T18:30:52.976Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 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:
apps/playground-web/src/app/bridge/components/RightSection.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 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:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/transactions/page.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/page.tsxapps/playground-web/src/app/bridge/transaction-button/page.tsxapps/playground-web/src/app/navLinks.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 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:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/x402/page.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{ts,tsx} : Redirect logic using `redirect()` from `next/navigation`.
Applied to files:
apps/playground-web/next.config.mjs
📚 Learning: 2025-05-26T16:31:02.480Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7152
File: apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/claim-conditions/shared-claim-conditions-page.tsx:43-49
Timestamp: 2025-05-26T16:31:02.480Z
Learning: In the thirdweb dashboard codebase, when `redirectToContractLandingPage()` is called, an explicit return statement is not required afterward because the function internally calls Next.js's `redirect()` which throws an error to halt execution.
Applied to files:
apps/playground-web/next.config.mjs
📚 Learning: 2025-06-18T02:02:21.427Z
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7363
File: apps/wallet-ui/next.config.mjs:58-73
Timestamp: 2025-06-18T02:02:21.427Z
Learning: User jnsdls prefers to explicitly list specific rewrite rules for clarity/documentation purposes, even when they are functionally redundant due to catch-all patterns. In PostHog rewrite configurations, they intentionally keep specific endpoint rules like "/_ph/decide" listed after catch-all rules like "/_ph/:path*" for documentation clarity, accepting the functional redundancy.
Applied to files:
apps/playground-web/next.config.mjs
📚 Learning: 2025-05-30T18:14:57.074Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7228
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/connect/in-app-wallets/settings/page.tsx:23-25
Timestamp: 2025-05-30T18:14:57.074Z
Learning: In the dashboard codebase, the `loginRedirect` function performs an actual page redirect that automatically stops execution, similar to Next.js `redirect()`. No return statement is needed after calling `loginRedirect` as it handles flow control internally.
Applied to files:
apps/playground-web/next.config.mjs
📚 Learning: 2025-06-05T13:59:49.886Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7285
File: apps/dashboard/src/app/(app)/(dashboard)/published-contract/components/uri-based-deploy.tsx:57-57
Timestamp: 2025-06-05T13:59:49.886Z
Learning: In the thirdweb dashboard Next.js app, when using loginRedirect() in server components, ensure to add a return statement after the redirect call to prevent further code execution and potential security issues.
Applied to files:
apps/playground-web/next.config.mjs
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Keep tokens secret via internal API routes or server actions
Applied to files:
apps/playground-web/src/app/api/paywall/route.tsapps/playground-web/src/app/page.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Prefer API routes or server actions to keep tokens secret; the browser only sees relative paths.
Applied to files:
apps/playground-web/src/app/api/paywall/route.ts
📚 Learning: 2025-08-20T10:35:18.543Z
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7888
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx:77-81
Timestamp: 2025-08-20T10:35:18.543Z
Learning: The webhooks/payments route exists at apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/payments/page.tsx and was added as part of the unified project layout changes.
Applied to files:
apps/playground-web/src/app/api/paywall/route.tsapps/playground-web/src/app/transactions/page.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/bridge/transaction-widget/page.tsxapps/playground-web/src/app/page.tsxapps/playground-web/src/app/navLinks.ts
📚 Learning: 2025-08-20T10:35:18.543Z
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7888
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx:77-81
Timestamp: 2025-08-20T10:35:18.543Z
Learning: The webhooks/payments route exists at apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/payments/page.tsx and was added as part of the unified project layout PR #7888.
Applied to files:
apps/playground-web/src/app/api/paywall/route.tsapps/playground-web/src/app/transactions/page.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/bridge/transaction-widget/page.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Use `Authorization: Bearer` header – never embed tokens in URLs
Applied to files:
apps/playground-web/src/app/api/paywall/route.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/api/**/*.{ts,tsx} : Pass the token in the `Authorization: Bearer` header – never embed it in the URL.
Applied to files:
apps/playground-web/src/app/api/paywall/route.ts
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Use design system tokens (e.g., `bg-card`, `border-border`, `text-muted-foreground`)
Applied to files:
apps/playground-web/src/app/api/paywall/route.tsapps/playground-web/src/app/transactions/page.tsxapps/playground-web/src/app/bridge/buy-widget/page.tsxapps/playground-web/src/app/data/pages-metadata.tsapps/playground-web/src/app/page.tsx
📚 Learning: 2025-09-17T11:02:13.528Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8044
File: packages/thirdweb/src/react/web/ui/Bridge/swap-widget/use-tokens.ts:15-17
Timestamp: 2025-09-17T11:02:13.528Z
Learning: The thirdweb `client` object is serializable and can safely be used in React Query keys, similar to the `contract` object.
Applied to files:
apps/playground-web/src/app/api/paywall/route.ts
📚 Learning: 2025-06-06T23:46:08.795Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7298
File: apps/dashboard/src/app/nebula-app/move-funds/move-funds.tsx:424-424
Timestamp: 2025-06-06T23:46:08.795Z
Learning: The thirdweb project has an ESLint rule that restricts direct usage of `defineChain`. When it's necessary to use `defineChain` directly, it's acceptable to disable the rule with `// eslint-disable-next-line no-restricted-syntax`.
Applied to files:
apps/playground-web/src/app/api/paywall/route.ts
📚 Learning: 2025-08-27T22:11:41.748Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7933
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/tokens/create/token/create-token-page-impl.tsx:465-473
Timestamp: 2025-08-27T22:11:41.748Z
Learning: In the token creation flow (create-token-page-impl.tsx), the createTokenOnUniversalBridge() call is intentionally not awaited (fire-and-forget pattern) to allow the token creation process to complete immediately without waiting for the bridge operation. This is a deliberate design choice to prioritize user experience by not blocking the main flow with supplementary bridge operations.
Applied to files:
apps/playground-web/src/app/api/paywall/route.ts
📚 Learning: 2025-05-20T18:54:15.781Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7081
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/assets/create/create-token-page-impl.tsx:110-118
Timestamp: 2025-05-20T18:54:15.781Z
Learning: In the thirdweb dashboard's token asset creation flow, the `transferBatch` function from `thirdweb/extensions/erc20` accepts the raw quantity values from the form without requiring explicit conversion to wei using `toUnits()`. The function appears to handle this conversion internally or is designed to work with the values in the format they're already provided.
Applied to files:
apps/playground-web/src/app/api/paywall/route.ts
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Use descriptive, stable `queryKeys` for React Query cache hits
Applied to files:
apps/playground-web/src/app/x402/components/X402RightSection.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Keep `queryKey` stable and descriptive for cache hits.
Applied to files:
apps/playground-web/src/app/x402/components/X402RightSection.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Wrap client-side data fetching calls in React Query (`tanstack/react-query`)
Applied to files:
apps/playground-web/src/app/x402/components/X402RightSection.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/layout.tsx : Building layout shells (`layout.tsx`) and top-level pages that mainly assemble data.
Applied to files:
apps/playground-web/src/app/transactions/page.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/bridge/transaction-widget/page.tsxapps/playground-web/src/app/page.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : Icons come from `lucide-react` or the project-specific `…/icons` exports – never embed raw SVG.
Applied to files:
apps/playground-web/src/app/transactions/page.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/data/pages-metadata.tsapps/playground-web/src/app/navLinks.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : For notices & skeletons rely on `AnnouncementBanner`, `GenericLoadingPage`, `EmptyStateCard`.
Applied to files:
apps/playground-web/src/app/transactions/page.tsxapps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/page.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Pages requiring fast transitions where data is prefetched on the client.
Applied to files:
apps/playground-web/src/app/transactions/page.tsx
📚 Learning: 2025-08-29T23:44:47.512Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7951
File: apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/_layout/contract-page-layout.tsx:38-38
Timestamp: 2025-08-29T23:44:47.512Z
Learning: The ContractPageLayout component in apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/_layout/contract-page-layout.tsx is not the root layout - it's nested within the dashboard layout which already handles footer positioning with min-h-dvh and AppFooter placement. The ContractPageLayout needs flex flex-col grow to properly participate in the parent's flex layout.
Applied to files:
apps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/bridge/transaction-widget/page.tsx
📚 Learning: 2025-08-07T17:24:31.965Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7812
File: apps/dashboard/src/app/(app)/team/~/~project/[[...paths]]/page.tsx:1-11
Timestamp: 2025-08-07T17:24:31.965Z
Learning: In Next.js App Router, page components (page.tsx files) are server components by default and do not require the "server-only" import directive. The "server-only" directive is primarily used for utility functions, API helpers, and data access modules that should never be included in the client bundle.
Applied to files:
apps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/bridge/page.tsx
📚 Learning: 2025-07-07T21:21:47.488Z
Learnt from: saminacodes
Repo: thirdweb-dev/js PR: 7543
File: apps/portal/src/app/pay/page.mdx:4-4
Timestamp: 2025-07-07T21:21:47.488Z
Learning: In the thirdweb-dev/js repository, lucide-react icons must be imported with the "Icon" suffix (e.g., ExternalLinkIcon, RocketIcon) as required by the new linting rule, contrary to the typical lucide-react convention of importing without the suffix.
Applied to files:
apps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/data/pages-metadata.tsapps/playground-web/src/app/navLinks.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : Layouts should reuse `SidebarLayout` / `FullWidthSidebarLayout` (`@/components/blocks/SidebarLayout`).
Applied to files:
apps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/navLinks.ts
📚 Learning: 2025-05-26T16:28:50.772Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7152
File: apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/tokens/shared-page.tsx:41-48
Timestamp: 2025-05-26T16:28:50.772Z
Learning: The `projectMeta` prop is not required for the server-rendered `ContractTokensPage` component in the tokens shared page, unlike some other shared pages where it's needed for consistency.
Applied to files:
apps/playground-web/src/app/bridge/transaction-widget/page.tsxapps/playground-web/src/app/page.tsx
📚 Learning: 2025-09-24T11:08:43.783Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8106
File: packages/thirdweb/src/react/web/ui/Bridge/bridge-widget/bridge-widget.tsx:34-41
Timestamp: 2025-09-24T11:08:43.783Z
Learning: In BridgeWidgetProps for packages/thirdweb/src/react/web/ui/Bridge/bridge-widget/bridge-widget.tsx, the Swap onError callback signature requires a non-undefined SwapPreparedQuote parameter (unlike Buy's onError which allows undefined quote). This is intentional - SwapWidget's onError is only called when a quote is available.
Applied to files:
apps/playground-web/src/app/bridge/buy-widget/page.tsxapps/playground-web/src/app/bridge/swap-widget/page.tsx
📚 Learning: 2025-10-17T22:59:11.867Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8278
File: apps/dashboard/src/@/components/blocks/BuyAndSwapEmbed.tsx:144-144
Timestamp: 2025-10-17T22:59:11.867Z
Learning: The BuyAndSwapEmbed component in apps/dashboard/src/@/components/blocks/BuyAndSwapEmbed.tsx cannot be used in testnets, so the hardcoded `is_testnet: false` in asset reporting calls is intentional and correct.
Applied to files:
apps/playground-web/src/app/bridge/buy-widget/page.tsx
📚 Learning: 2025-09-18T20:09:57.064Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8069
File: apps/playground-web/src/app/bridge/swap-widget/components/types.ts:3-12
Timestamp: 2025-09-18T20:09:57.064Z
Learning: The SwapWidgetPlaygroundOptions type should not include persistTokenSelections - the playground intentionally hardcodes persistTokenSelections={false} and doesn't expose it as a configurable option to users.
Applied to files:
apps/playground-web/src/app/bridge/swap-widget/page.tsx
📚 Learning: 2025-05-29T00:46:09.063Z
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7188
File: apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/accounts/components/accounts-count.tsx:15-15
Timestamp: 2025-05-29T00:46:09.063Z
Learning: In the accounts component at apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/accounts/components/accounts-count.tsx, the 3-column grid layout (md:grid-cols-3) is intentionally maintained even when rendering only one StatCard, as part of the design structure for this component.
Applied to files:
apps/playground-web/src/app/page.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : Reuse core UI primitives; avoid re-implementing buttons, cards, modals.
Applied to files:
apps/playground-web/src/app/page.tsx
📚 Learning: 2025-09-23T19:56:43.668Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8106
File: packages/thirdweb/src/react/web/ui/ConnectWallet/Modal/ConnectEmbed.tsx:482-485
Timestamp: 2025-09-23T19:56:43.668Z
Learning: In packages/thirdweb/src/react/web/ui/ConnectWallet/Modal/ConnectEmbed.tsx, the EmbedContainer uses width: "100vw" intentionally rather than "100%" - this is by design for the bridge widget embedding use case.
Applied to files:
apps/playground-web/src/app/bridge/transaction-button/page.tsx
📚 Learning: 2025-08-07T20:43:21.864Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7812
File: apps/dashboard/src/app/(app)/(dashboard)/published-contract/components/token-banner.tsx:48-60
Timestamp: 2025-08-07T20:43:21.864Z
Learning: In the TokenBanner component at apps/dashboard/src/app/(app)/(dashboard)/published-contract/components/token-banner.tsx, the Link components use target="_blank" with internal application routes (starting with "/") to open pages in new tabs within the same application. These internal links do not require rel="noopener noreferrer" security attributes, which are only needed for external URLs.
Applied to files:
apps/playground-web/src/app/navLinks.ts
📚 Learning: 2025-07-31T16:17:42.753Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7768
File: apps/playground-web/src/app/navLinks.ts:1-1
Timestamp: 2025-07-31T16:17:42.753Z
Learning: Configuration files that import and reference React components (like icon components from lucide-react) need the "use client" directive, even if they primarily export static data, because the referenced components need to be executed in a client context when used by other client components.
Applied to files:
apps/playground-web/src/app/navLinks.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/components/*.client.tsx : Client components must start with `'use client';` before imports.
Applied to files:
apps/playground-web/src/app/navLinks.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : When you need access to browser APIs (localStorage, window, IntersectionObserver etc.).
Applied to files:
apps/playground-web/src/app/navLinks.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : Prefer composable primitives over custom markup: `Button`, `Input`, `Select`, `Tabs`, `Card`, `Sidebar`, `Separator`, `Badge`.
Applied to files:
apps/playground-web/src/app/navLinks.ts
🧬 Code graph analysis (7)
apps/playground-web/src/app/transactions/page.tsx (1)
apps/playground-web/src/app/data/pages-metadata.ts (1)
FeatureCardMetadata(22-27)
apps/playground-web/src/app/bridge/page.tsx (3)
apps/playground-web/src/components/blocks/OverviewPage.tsx (1)
OverviewPage(4-40)apps/playground-web/src/app/data/pages-metadata.ts (1)
bridgeFeatureCards(116-151)apps/playground-web/src/icons/PayIcon.tsx (1)
PayIcon(1-21)
apps/playground-web/src/app/bridge/transaction-widget/page.tsx (2)
apps/playground-web/src/components/blocks/APIHeader.tsx (1)
PageLayout(49-70)apps/playground-web/src/app/bridge/transaction-widget/TransactionPlayground.tsx (1)
TransactionPlayground(32-48)
apps/playground-web/src/app/data/pages-metadata.ts (1)
apps/playground-web/src/icons/SmartAccountIcon.tsx (1)
SmartAccountIcon(1-21)
apps/playground-web/src/app/page.tsx (1)
apps/playground-web/src/app/data/pages-metadata.ts (5)
aiFeatureCards(199-212)walletsFeatureCards(29-93)bridgeFeatureCards(116-151)tokensFeatureCards(183-197)x402FeatureCards(214-222)
apps/playground-web/src/app/bridge/transaction-button/page.tsx (2)
apps/playground-web/src/app/bridge/buy-widget/page.tsx (1)
metadata(13-20)apps/playground-web/src/components/blocks/APIHeader.tsx (1)
PageLayout(49-70)
apps/playground-web/src/app/navLinks.ts (1)
apps/playground-web/src/components/blocks/full-width-sidebar-layout.tsx (1)
ShadcnSidebarLink(43-59)
⏰ 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). (3)
- GitHub Check: Size
- GitHub Check: Lint Packages
- GitHub Check: Analyze (javascript)
🔇 Additional comments (8)
apps/playground-web/src/app/bridge/transaction-button/page.tsx (1)
12-19: LGTM!The metadata structure is correct and follows the established pattern from other pages in the playground.
apps/playground-web/src/app/x402/components/X402RightSection.tsx (1)
55-57: LGTM!The template literal is cleaner and more idiomatic than string concatenation.
apps/playground-web/src/app/x402/page.tsx (1)
3-4: LGTM!The import path adjustments correctly reflect the directory restructuring.
apps/playground-web/src/app/bridge/page.tsx (1)
1-14: LGTM!The Bridge overview page follows the established pattern and correctly imports feature cards from the centralized metadata file.
apps/playground-web/src/app/page.tsx (1)
58-66: Verify removal of card limit.The previous implementation limited each section to 6 cards using
.slice(0, 6). Now all cards are rendered. This could make some sections large (e.g.,walletsFeatureCardshas 9 items based on the metadata).Was this intentional to display all features, or should the 6-card limit be retained for landing page brevity?
If the limit should be restored:
- {props.featureCards.map((card) => ( + {props.featureCards.slice(0, 6).map((card) => ( <FeatureCardapps/playground-web/src/app/navLinks.ts (3)
179-211: LGTM on Bridge restructuring!The Bridge submenu correctly replaces the previous Payments section and aligns with PR objectives. The widget names have been updated appropriately (e.g., "Buy Widget", "Checkout Widget", "Transaction Widget").
218-222: LGTM on x402 top-level link!The new top-level x402 entry with
DollarSignIconcorrectly implements the PR objective to give x402 its own category.
99-153: All navigation routes have corresponding pages — verification passed.The script confirms all 10 routes referenced in the navLinks.ts changes have matching page.tsx files:
- Transactions: 5/5 routes verified (overview, users, airdrop-tokens, mint-tokens, webhooks)
- Account Abstraction: 5/5 routes verified (overview, eip-4337, eip-7702, eip-5792, native-aa)
No issues found.
github-actions
bot
added
Dashboard
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
♻️ Duplicate comments (2)
apps/playground-web/src/app/bridge/transaction-button/page.tsx (2)
10-10: Fix grammar in description.The description has a grammatical error: "perform onchain transaction" should be "perform an onchain transaction" or "perform onchain transactions".
Apply this diff:
const description = - "Transaction Button component allows users to perform onchain transaction and prompts the user to fund the wallet if required from fiat or swap"; + "Transaction Button component allows users to perform an onchain transaction and prompts the user to fund the wallet if required from fiat or swap";
28-28: Update docsLink to reference the correct TransactionButton documentation.The current link points to the TypeScript reference, but this page documents the TransactionButton React component. The correct documentation URL should be
https://portal.thirdweb.com/react/v5/components/TransactionButton?utm_source=playground.Apply this diff:
- docsLink="https://portal.thirdweb.com/references/typescript/v5/TransactionButton" + docsLink="https://portal.thirdweb.com/react/v5/components/TransactionButton?utm_source=playground"
📜 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 (47)
apps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsx(1 hunks)apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx(1 hunks)apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx(2 hunks)apps/playground-web/next.config.mjs(2 hunks)apps/playground-web/src/app/api/paywall/route.ts(1 hunks)apps/playground-web/src/app/bridge/buy-widget/page.tsx(1 hunks)apps/playground-web/src/app/bridge/checkout-widget/page.tsx(1 hunks)apps/playground-web/src/app/bridge/components/RightSection.tsx(1 hunks)apps/playground-web/src/app/bridge/page.tsx(1 hunks)apps/playground-web/src/app/bridge/swap-widget/page.tsx(1 hunks)apps/playground-web/src/app/bridge/transaction-button/page.tsx(2 hunks)apps/playground-web/src/app/bridge/transaction-widget/page.tsx(1 hunks)apps/playground-web/src/app/contracts/events/page.tsx(0 hunks)apps/playground-web/src/app/contracts/extensions/page.tsx(0 hunks)apps/playground-web/src/app/contracts/page.tsx(0 hunks)apps/playground-web/src/app/contracts/read/page.tsx(0 hunks)apps/playground-web/src/app/contracts/write/page.tsx(0 hunks)apps/playground-web/src/app/data/pages-metadata.ts(4 hunks)apps/playground-web/src/app/navLinks.ts(3 hunks)apps/playground-web/src/app/page.tsx(3 hunks)apps/playground-web/src/app/payments/page.tsx(0 hunks)apps/playground-web/src/app/x402/components/X402RightSection.tsx(1 hunks)apps/playground-web/src/app/x402/page.tsx(2 hunks)apps/playground-web/src/icons/ContractIcon.tsx(0 hunks)apps/portal/src/app/bridge/fund/page.mdx(1 hunks)apps/portal/src/app/bridge/products/page.mdx(1 hunks)apps/portal/src/app/bridge/transactions/page.mdx(1 hunks)apps/portal/src/app/payments/page.mdx(1 hunks)apps/portal/src/app/payments/sidebar.tsx(1 hunks)apps/portal/src/app/react/v5/account-abstraction/get-started/page.mdx(3 hunks)apps/portal/src/app/react/v5/auth/page.mdx(2 hunks)apps/portal/src/app/react/v5/components/ConnectButton/page.mdx(1 hunks)apps/portal/src/app/react/v5/components/ConnectEmbed/page.mdx(1 hunks)apps/portal/src/app/react/v5/ecosystem-wallet/get-started/page.mdx(2 hunks)apps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdx(5 hunks)apps/portal/src/app/react/v5/in-app-wallet/get-started/page.mdx(1 hunks)apps/portal/src/app/react/v5/linking/page.mdx(1 hunks)apps/portal/src/app/react/v5/pay/fund-wallets/page.mdx(1 hunks)apps/portal/src/app/react/v5/pay/transaction/page.mdx(1 hunks)apps/portal/src/app/react/v5/reading-state/page.mdx(0 hunks)apps/portal/src/app/react/v5/social/page.mdx(1 hunks)apps/portal/src/app/react/v5/transactions/page.mdx(0 hunks)apps/portal/src/app/typescript/v5/ecosystem-wallet/page.mdx(2 hunks)apps/portal/src/app/typescript/v5/in-app-wallet/page.mdx(1 hunks)apps/portal/src/app/wallets/faq/page.mdx(2 hunks)apps/portal/src/app/x402/page.mdx(1 hunks)apps/portal/src/app/x402/sidebar.tsx(1 hunks)
💤 Files with no reviewable changes (9)
- apps/portal/src/app/react/v5/transactions/page.mdx
- apps/playground-web/src/app/contracts/page.tsx
- apps/portal/src/app/react/v5/reading-state/page.mdx
- apps/playground-web/src/app/contracts/extensions/page.tsx
- apps/playground-web/src/app/contracts/events/page.tsx
- apps/playground-web/src/app/contracts/read/page.tsx
- apps/playground-web/src/app/payments/page.tsx
- apps/playground-web/src/app/contracts/write/page.tsx
- apps/playground-web/src/icons/ContractIcon.tsx
✅ Files skipped from review due to trivial changes (4)
- apps/playground-web/src/app/bridge/buy-widget/page.tsx
- apps/portal/src/app/typescript/v5/in-app-wallet/page.mdx
- apps/portal/src/app/react/v5/in-app-wallet/get-started/page.mdx
- apps/portal/src/app/react/v5/linking/page.mdx
🚧 Files skipped from review as they are similar to previous changes (8)
- apps/playground-web/src/app/x402/page.tsx
- apps/playground-web/src/app/bridge/page.tsx
- apps/playground-web/src/app/x402/components/X402RightSection.tsx
- apps/playground-web/src/app/bridge/components/RightSection.tsx
- apps/playground-web/src/app/bridge/checkout-widget/page.tsx
- apps/playground-web/src/app/bridge/swap-widget/page.tsx
- apps/playground-web/src/app/page.tsx
- apps/playground-web/src/app/bridge/transaction-widget/page.tsx
🧰 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:
apps/portal/src/app/x402/sidebar.tsxapps/playground-web/src/app/bridge/transaction-button/page.tsxapps/playground-web/src/app/navLinks.tsapps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsxapps/portal/src/app/payments/sidebar.tsxapps/playground-web/src/app/api/paywall/route.tsapps/playground-web/src/app/data/pages-metadata.tsapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
**/*.{ts,tsx,js,jsx}
📄 CodeRabbit inference engine (CLAUDE.md)
Load heavy dependencies inside async paths to keep initial bundle lean (lazy loading)
Files:
apps/portal/src/app/x402/sidebar.tsxapps/playground-web/src/app/bridge/transaction-button/page.tsxapps/playground-web/src/app/navLinks.tsapps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsxapps/portal/src/app/payments/sidebar.tsxapps/playground-web/src/app/api/paywall/route.tsapps/playground-web/src/app/data/pages-metadata.tsapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
apps/{dashboard,playground-web}/**/*.{ts,tsx}
📄 CodeRabbit inference engine (CLAUDE.md)
apps/{dashboard,playground-web}/**/*.{ts,tsx}: Import UI primitives from@/components/ui/*(Button, Input, Select, Tabs, Card, Sidebar, Badge, Separator) in dashboard and playground apps
UseNavLinkfor internal navigation with automatic active states in dashboard and playground apps
Use Tailwind CSS only – no inline styles or CSS modules
Usecn()from@/lib/utilsfor conditional class logic
Use design system tokens (e.g.,bg-card,border-border,text-muted-foreground)
Server Components (Node edge): Start files withimport "server-only";
Client Components (browser): Begin files with'use client';
Always callgetAuthToken()to retrieve JWT from cookies on server side
UseAuthorization: Bearerheader – never embed tokens in URLs
Return typed results (e.g.,Project[],User[]) – avoidany
Wrap client-side data fetching calls in React Query (@tanstack/react-query)
Use descriptive, stablequeryKeysfor React Query cache hits
ConfigurestaleTime/cacheTimein React Query based on freshness (default ≥ 60s)
Keep tokens secret via internal API routes or server actions
Never importposthog-jsin server components
Files:
apps/playground-web/src/app/bridge/transaction-button/page.tsxapps/playground-web/src/app/navLinks.tsapps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsxapps/playground-web/src/app/api/paywall/route.tsapps/playground-web/src/app/data/pages-metadata.tsapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
🧠 Learnings (54)
📓 Common learnings
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7888
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx:77-81
Timestamp: 2025-08-20T10:35:18.543Z
Learning: The webhooks/payments route exists at apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/payments/page.tsx and was added as part of the unified project layout changes.
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7888
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx:77-81
Timestamp: 2025-08-20T10:35:18.543Z
Learning: The webhooks/payments route exists at apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/payments/page.tsx and was added as part of the unified project layout PR #7888.
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to apps/{dashboard,playground-web}/**/*.{ts,tsx} : Use `NavLink` for internal navigation with automatic active states in dashboard and playground apps
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7315
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/assets/create/nft/launch-nft.tsx:155-160
Timestamp: 2025-06-10T00:46:58.580Z
Learning: In the dashboard application, the route structure for team and project navigation is `/team/[team_slug]/[project_slug]/...` without a `/project/` segment. Contract links should be formatted as `/team/${teamSlug}/${projectSlug}/contract/${chainId}/${contractAddress}`.
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Use `NavLink` for internal navigation with automatic active states in dashboard and playground apps
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/portal/src/app/x402/page.mdxapps/playground-web/src/app/navLinks.tsapps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsxapps/portal/src/app/react/v5/components/ConnectEmbed/page.mdxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsxapps/portal/src/app/react/v5/auth/page.mdxapps/portal/src/app/payments/sidebar.tsxapps/portal/src/app/react/v5/social/page.mdxapps/portal/src/app/react/v5/components/ConnectButton/page.mdxapps/portal/src/app/react/v5/ecosystem-wallet/get-started/page.mdxapps/portal/src/app/react/v5/account-abstraction/get-started/page.mdxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Import UI primitives from `@/components/ui/*` (Button, Input, Select, Tabs, Card, Sidebar, Badge, Separator) in dashboard and playground apps
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/playground-web/src/app/navLinks.tsapps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsxapps/portal/src/app/react/v5/components/ConnectEmbed/page.mdxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsxapps/portal/src/app/react/v5/auth/page.mdxapps/portal/src/app/payments/sidebar.tsxapps/portal/src/app/react/v5/components/ConnectButton/page.mdxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
📚 Learning: 2025-08-20T10:35:18.543Z
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7888
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx:77-81
Timestamp: 2025-08-20T10:35:18.543Z
Learning: The webhooks/payments route exists at apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/payments/page.tsx and was added as part of the unified project layout changes.
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/portal/src/app/x402/page.mdxapps/playground-web/next.config.mjsapps/portal/src/app/react/v5/pay/fund-wallets/page.mdxapps/portal/src/app/bridge/transactions/page.mdxapps/portal/src/app/payments/page.mdxapps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsxapps/portal/src/app/react/v5/auth/page.mdxapps/portal/src/app/react/v5/pay/transaction/page.mdxapps/portal/src/app/payments/sidebar.tsxapps/playground-web/src/app/api/paywall/route.tsapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Server Components (Node edge): Start files with `import "server-only";`
Applied to files:
apps/portal/src/app/x402/sidebar.tsx
📚 Learning: 2025-08-20T10:35:18.543Z
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7888
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx:77-81
Timestamp: 2025-08-20T10:35:18.543Z
Learning: The webhooks/payments route exists at apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/payments/page.tsx and was added as part of the unified project layout PR #7888.
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/playground-web/next.config.mjsapps/portal/src/app/react/v5/pay/fund-wallets/page.mdxapps/portal/src/app/bridge/transactions/page.mdxapps/portal/src/app/payments/page.mdxapps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsxapps/portal/src/app/react/v5/auth/page.mdxapps/portal/src/app/react/v5/pay/transaction/page.mdxapps/portal/src/app/payments/sidebar.tsxapps/playground-web/src/app/api/paywall/route.tsapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Client Components (browser): Begin files with `'use client';`
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/playground-web/src/app/navLinks.tsapps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsxapps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Use design system tokens (e.g., `bg-card`, `border-border`, `text-muted-foreground`)
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsxapps/portal/src/app/react/v5/auth/page.mdxapps/portal/src/app/payments/sidebar.tsxapps/portal/src/app/react/v5/ecosystem-wallet/get-started/page.mdxapps/playground-web/src/app/data/pages-metadata.tsapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Use Tailwind CSS only – no inline styles or CSS modules
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{tsx} : Expose `className` prop on root element of components for overrides
Applied to files:
apps/portal/src/app/x402/sidebar.tsx
📚 Learning: 2025-07-07T21:21:47.488Z
Learnt from: saminacodes
Repo: thirdweb-dev/js PR: 7543
File: apps/portal/src/app/pay/page.mdx:4-4
Timestamp: 2025-07-07T21:21:47.488Z
Learning: In the thirdweb-dev/js repository, lucide-react icons must be imported with the "Icon" suffix (e.g., ExternalLinkIcon, RocketIcon) as required by the new linting rule, contrary to the typical lucide-react convention of importing without the suffix.
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/playground-web/src/app/navLinks.tsapps/portal/src/app/payments/sidebar.tsxapps/playground-web/src/app/data/pages-metadata.ts
📚 Learning: 2025-08-07T20:43:21.864Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7812
File: apps/dashboard/src/app/(app)/(dashboard)/published-contract/components/token-banner.tsx:48-60
Timestamp: 2025-08-07T20:43:21.864Z
Learning: In the TokenBanner component at apps/dashboard/src/app/(app)/(dashboard)/published-contract/components/token-banner.tsx, the Link components use target="_blank" with internal application routes (starting with "/") to open pages in new tabs within the same application. These internal links do not require rel="noopener noreferrer" security attributes, which are only needed for external URLs.
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/playground-web/src/app/bridge/transaction-button/page.tsxapps/playground-web/src/app/navLinks.tsapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsxapps/portal/src/app/bridge/fund/page.mdxapps/portal/src/app/react/v5/auth/page.mdxapps/portal/src/app/payments/sidebar.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
📚 Learning: 2025-06-18T04:30:04.326Z
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7365
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectFTUX/ProjectFTUX.tsx:16-17
Timestamp: 2025-06-18T04:30:04.326Z
Learning: Next.js Link component fully supports both internal and external URLs and works appropriately with all standard anchor attributes including target="_blank", rel="noopener noreferrer", etc. Using Link for external URLs is completely appropriate and recommended.
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/portal/src/app/payments/sidebar.tsx
📚 Learning: 2025-06-18T04:27:16.172Z
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7365
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectFTUX/ProjectFTUX.tsx:16-17
Timestamp: 2025-06-18T04:27:16.172Z
Learning: Next.js Link component supports external URLs without throwing errors. When used with absolute URLs (like https://...), it behaves like a regular anchor tag without client-side routing, but does not cause runtime crashes or errors as previously believed.
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/portal/src/app/payments/sidebar.tsx
📚 Learning: 2025-09-05T19:45:22.949Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7984
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/components/FeatureCard.client.tsx:93-99
Timestamp: 2025-09-05T19:45:22.949Z
Learning: In the thirdweb dashboard project, rel="noopener noreferrer" attributes are not required for external links with target="_blank" in Link components, as confirmed by MananTank.
Applied to files:
apps/portal/src/app/x402/sidebar.tsx
📚 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: Surface breaking changes prominently in PR descriptions
Applied to files:
apps/playground-web/src/app/bridge/transaction-button/page.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 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:
apps/playground-web/src/app/bridge/transaction-button/page.tsxapps/portal/src/app/typescript/v5/ecosystem-wallet/page.mdxapps/playground-web/src/app/navLinks.tsapps/portal/src/app/react/v5/pay/fund-wallets/page.mdxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsxapps/portal/src/app/react/v5/auth/page.mdxapps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdxapps/portal/src/app/react/v5/components/ConnectButton/page.mdxapps/portal/src/app/react/v5/ecosystem-wallet/get-started/page.mdxapps/portal/src/app/react/v5/account-abstraction/get-started/page.mdx
📚 Learning: 2025-10-03T23:36:00.631Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8181
File: packages/thirdweb/src/react/web/ui/Bridge/BuyWidget.tsx:27-27
Timestamp: 2025-10-03T23:36:00.631Z
Learning: In packages/thirdweb/src/react/web/ui/Bridge/BuyWidget.tsx, the component intentionally uses a hardcoded English locale (connectLocaleEn) rather than reading from the provider, as BuyWidget is designed to be English-only and does not require internationalization support.
Applied to files:
apps/playground-web/src/app/bridge/transaction-button/page.tsxapps/portal/src/app/react/v5/pay/fund-wallets/page.mdxapps/portal/src/app/bridge/transactions/page.mdxapps/portal/src/app/payments/page.mdxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsxapps/portal/src/app/bridge/products/page.mdxapps/portal/src/app/bridge/fund/page.mdxapps/portal/src/app/react/v5/pay/transaction/page.mdxapps/portal/src/app/payments/sidebar.tsx
📚 Learning: 2025-09-23T19:56:43.668Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8106
File: packages/thirdweb/src/react/web/ui/ConnectWallet/Modal/ConnectEmbed.tsx:482-485
Timestamp: 2025-09-23T19:56:43.668Z
Learning: In packages/thirdweb/src/react/web/ui/ConnectWallet/Modal/ConnectEmbed.tsx, the EmbedContainer uses width: "100vw" intentionally rather than "100%" - this is by design for the bridge widget embedding use case.
Applied to files:
apps/playground-web/src/app/bridge/transaction-button/page.tsxapps/portal/src/app/bridge/transactions/page.mdxapps/portal/src/app/payments/page.mdxapps/portal/src/app/react/v5/components/ConnectEmbed/page.mdxapps/portal/src/app/bridge/fund/page.mdx
📚 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/** : Support for in-app wallets (social/email login)
Applied to files:
apps/portal/src/app/typescript/v5/ecosystem-wallet/page.mdxapps/portal/src/app/wallets/faq/page.mdxapps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdxapps/portal/src/app/react/v5/social/page.mdxapps/portal/src/app/react/v5/ecosystem-wallet/get-started/page.mdxapps/portal/src/app/react/v5/account-abstraction/get-started/page.mdx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{ts,tsx} : Redirect logic using `redirect()` from `next/navigation`.
Applied to files:
apps/playground-web/next.config.mjsapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
📚 Learning: 2025-09-05T19:45:05.006Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7984
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx:76-80
Timestamp: 2025-09-05T19:45:05.006Z
Learning: When verifying route existence for Next.js app router, search for the full nested path structure rather than using simple patterns. The AI summary often contains accurate information about file additions that should be referenced before running verification scripts.
Applied to files:
apps/playground-web/next.config.mjs
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : Use `NavLink` (`@/components/ui/NavLink`) for internal navigation so active states are handled automatically.
Applied to files:
apps/playground-web/src/app/navLinks.tsapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
📚 Learning: 2025-07-31T16:17:42.753Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7768
File: apps/playground-web/src/app/navLinks.ts:1-1
Timestamp: 2025-07-31T16:17:42.753Z
Learning: Configuration files that import and reference React components (like icon components from lucide-react) need the "use client" directive, even if they primarily export static data, because the referenced components need to be executed in a client context when used by other client components.
Applied to files:
apps/playground-web/src/app/navLinks.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : Icons come from `lucide-react` or the project-specific `…/icons` exports – never embed raw SVG.
Applied to files:
apps/playground-web/src/app/navLinks.tsapps/playground-web/src/app/data/pages-metadata.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/components/*.client.tsx : Client components must start with `'use client';` before imports.
Applied to files:
apps/playground-web/src/app/navLinks.tsapps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : When you need access to browser APIs (localStorage, window, IntersectionObserver etc.).
Applied to files:
apps/playground-web/src/app/navLinks.tsapps/portal/src/app/react/v5/auth/page.mdxapps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : Prefer composable primitives over custom markup: `Button`, `Input`, `Select`, `Tabs`, `Card`, `Sidebar`, `Separator`, `Badge`.
Applied to files:
apps/playground-web/src/app/navLinks.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : Layouts should reuse `SidebarLayout` / `FullWidthSidebarLayout` (`@/components/blocks/SidebarLayout`).
Applied to files:
apps/playground-web/src/app/navLinks.tsapps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
📚 Learning: 2025-09-24T11:09:45.142Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8106
File: packages/thirdweb/src/script-exports/readme.md:6-7
Timestamp: 2025-09-24T11:09:45.142Z
Learning: For thirdweb Bridge Widget script exports, the module is exported with globalName: "BridgeWidget" in tsup config, making the global API `BridgeWidget.render()` rather than `window.thirdweb.BridgeWidget.render()`.
Applied to files:
apps/portal/src/app/payments/page.mdx
📚 Learning: 2025-07-15T08:25:44.584Z
Learnt from: Yash094
Repo: thirdweb-dev/js PR: 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.
Applied to files:
apps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsx
📚 Learning: 2025-07-10T10:18:33.238Z
Learnt from: arcoraven
Repo: thirdweb-dev/js PR: 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:
apps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsx
📚 Learning: 2025-06-10T00:46:58.580Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7315
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/assets/create/nft/launch-nft.tsx:155-160
Timestamp: 2025-06-10T00:46:58.580Z
Learning: In the dashboard application, the route structure for team and project navigation is `/team/[team_slug]/[project_slug]/...` without a `/project/` segment. Contract links should be formatted as `/team/${teamSlug}/${projectSlug}/contract/${chainId}/${contractAddress}`.
Applied to files:
apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
📚 Learning: 2025-09-17T11:02:13.528Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8044
File: packages/thirdweb/src/react/web/ui/Bridge/swap-widget/use-tokens.ts:15-17
Timestamp: 2025-09-17T11:02:13.528Z
Learning: The thirdweb `client` object is serializable and can safely be used in React Query keys, similar to the `contract` object.
Applied to files:
apps/portal/src/app/react/v5/auth/page.mdxapps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdxapps/playground-web/src/app/api/paywall/route.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 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:
apps/portal/src/app/react/v5/auth/page.mdxapps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdx
📚 Learning: 2025-05-30T17:14:25.332Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 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:
apps/portal/src/app/react/v5/auth/page.mdxapps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdxapps/playground-web/src/app/api/paywall/route.ts
📚 Learning: 2025-06-17T18:30:52.976Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 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:
apps/portal/src/app/react/v5/auth/page.mdxapps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Prefer API routes or server actions to keep tokens secret; the browser only sees relative paths.
Applied to files:
apps/portal/src/app/react/v5/auth/page.mdxapps/playground-web/src/app/api/paywall/route.ts
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Always call `getAuthToken()` to retrieve JWT from cookies on server side
Applied to files:
apps/portal/src/app/react/v5/auth/page.mdx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Create light wrappers (e.g. `fetchJson`) that automatically attach the JWT from cookies/session when calling internal API routes.
Applied to files:
apps/portal/src/app/react/v5/auth/page.mdxapps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{ts,tsx} : Export default async functions without `'use client';` – they run on the Node edge.
Applied to files:
apps/portal/src/app/react/v5/auth/page.mdx
📚 Learning: 2025-06-13T13:03:41.732Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7332
File: apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/public-pages/nft/overview/nft-drop-claim.tsx:82-90
Timestamp: 2025-06-13T13:03:41.732Z
Learning: The thirdweb `contract` object is serializable and can safely be used in contexts (e.g., React-Query keys) that require serializable values.
Applied to files:
apps/portal/src/app/react/v5/auth/page.mdx
📚 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:
apps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdxapps/portal/src/app/react/v5/account-abstraction/get-started/page.mdx
📚 Learning: 2025-09-17T11:14:35.659Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8044
File: packages/thirdweb/src/react/web/ui/Bridge/swap-widget/swap-ui.tsx:919-930
Timestamp: 2025-09-17T11:14:35.659Z
Learning: In the thirdweb codebase, useCustomTheme() hook can be used inside styled-components callbacks, contrary to the general React Rules of Hooks. This is a valid pattern in their implementation.
Applied to files:
apps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdx
📚 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/** : Smart wallets with account abstraction
Applied to files:
apps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdxapps/portal/src/app/react/v5/account-abstraction/get-started/page.mdx
📚 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/** : Unified `Wallet` and `Account` interfaces in wallet architecture
Applied to files:
apps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdxapps/portal/src/app/react/v5/account-abstraction/get-started/page.mdx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Keep tokens secret via internal API routes or server actions
Applied to files:
apps/playground-web/src/app/api/paywall/route.ts
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Use `Authorization: Bearer` header – never embed tokens in URLs
Applied to files:
apps/playground-web/src/app/api/paywall/route.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/api/**/*.{ts,tsx} : Pass the token in the `Authorization: Bearer` header – never embed it in the URL.
Applied to files:
apps/playground-web/src/app/api/paywall/route.ts
📚 Learning: 2025-06-06T23:46:08.795Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7298
File: apps/dashboard/src/app/nebula-app/move-funds/move-funds.tsx:424-424
Timestamp: 2025-06-06T23:46:08.795Z
Learning: The thirdweb project has an ESLint rule that restricts direct usage of `defineChain`. When it's necessary to use `defineChain` directly, it's acceptable to disable the rule with `// eslint-disable-next-line no-restricted-syntax`.
Applied to files:
apps/playground-web/src/app/api/paywall/route.ts
📚 Learning: 2025-08-27T22:11:41.748Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7933
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/tokens/create/token/create-token-page-impl.tsx:465-473
Timestamp: 2025-08-27T22:11:41.748Z
Learning: In the token creation flow (create-token-page-impl.tsx), the createTokenOnUniversalBridge() call is intentionally not awaited (fire-and-forget pattern) to allow the token creation process to complete immediately without waiting for the bridge operation. This is a deliberate design choice to prioritize user experience by not blocking the main flow with supplementary bridge operations.
Applied to files:
apps/playground-web/src/app/api/paywall/route.ts
📚 Learning: 2025-05-20T18:54:15.781Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7081
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/assets/create/create-token-page-impl.tsx:110-118
Timestamp: 2025-05-20T18:54:15.781Z
Learning: In the thirdweb dashboard's token asset creation flow, the `transferBatch` function from `thirdweb/extensions/erc20` accepts the raw quantity values from the form without requiring explicit conversion to wei using `toUnits()`. The function appears to handle this conversion internally or is designed to work with the values in the format they're already provided.
Applied to files:
apps/playground-web/src/app/api/paywall/route.ts
📚 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 test/src/test-wallets.ts : Predefined test accounts are in `test/src/test-wallets.ts`
Applied to files:
apps/portal/src/app/react/v5/account-abstraction/get-started/page.mdx
📚 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 apps/{dashboard,playground-web}/**/feature/components/** : Group feature-specific components under `feature/components/*` with barrel `index.ts`
Applied to files:
apps/playground-web/src/app/data/pages-metadata.ts
🧬 Code graph analysis (3)
apps/playground-web/src/app/bridge/transaction-button/page.tsx (2)
apps/playground-web/src/app/bridge/buy-widget/page.tsx (1)
metadata(13-20)apps/playground-web/src/components/blocks/APIHeader.tsx (1)
PageLayout(49-70)
apps/playground-web/src/app/navLinks.ts (1)
apps/playground-web/src/components/blocks/full-width-sidebar-layout.tsx (1)
ShadcnSidebarLink(43-59)
apps/playground-web/src/app/data/pages-metadata.ts (1)
apps/playground-web/src/icons/SmartAccountIcon.tsx (1)
SmartAccountIcon(1-21)
⏰ 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). (2)
- GitHub Check: Size
- GitHub Check: Analyze (javascript)
🔇 Additional comments (37)
apps/portal/src/app/react/v5/social/page.mdx (1)
26-26: No issues found with the updated playground URL.The change from
/connect/socialto/wallets/socialis consistent with the established pattern throughout the codebase. Multiple other wallet-related features already use the/wallets/path structure (auth, in-app-wallet, ecosystem-wallet, sign-in/button), confirming this is the valid URL format in the current playground architecture.apps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsx (1)
95-98: Playground link updates align with wallets flow.URL now targets the wallets sign-in button to match the new navigation structure—looks good.
apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx (1)
152-155: Playground footer link matches the wallets destination.This keeps the project sidebar footer consistent with the updated wallets-focused Playground paths.
apps/playground-web/src/app/api/paywall/route.ts (1)
5-5: Import path correctly updated for x402 reorganization.Verification confirms the
tokenconstant is properly exported from the new location atapps/playground-web/src/app/x402/components/constants.tswith the expecteddecimalsproperty. The usage on lines 45 and 61 will work correctly with this import.apps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdx (3)
85-85: Minor formatting: whitespace changes in code blocks.Lines 85, 89, 111, 121, and 133 add blank lines in code block examples. These appear to be minor formatting cleanups for consistency and readability.
Also applies to: 89-89, 111-111, 121-121, 133-133
23-23: Link path is valid and correctly configured.The path
/wallets/sponsor-gason line 23 correctly references the Account Abstraction page located atapps/portal/src/app/wallets/sponsor-gas/page.mdx. This route is properly structured for Next.js app router and no changes are required.
34-34: Playground URL path is valid—no changes needed.The updated playground URL
https://playground.thirdweb.com/account-abstraction/eip-4337is confirmed by the file structure atapps/playground-web/src/app/account-abstraction/eip-4337/page.tsx. The internal link/wallets/sponsor-gason line 23 is also valid and widely used throughout the codebase. Both changes align with the PR's playground reorganization.apps/playground-web/next.config.mjs (2)
62-62: LGTM! Clean migration of connect/pay routes to bridge.The updated destinations align with the payments → bridge migration strategy.
Also applies to: 67-67
161-189: I need to verify the complete redirect configuration context to assess whether the absence of a catch-all/payments/:path*redirect is intentional:All destination routes verified successfully; redirect structure is sound.
Next.js redirects are processed in order and patterns are anchored to the start, so exact path matches (like
/payments,/payments/x402, etc.) do not interfere with each other. The script confirmed all destination paths exist:
/bridge/✓/x402/✓/bridge/buy-widget/✓/bridge/checkout-widget/✓/bridge/transaction-widget/✓The absence of a catch-all
/payments/:path*redirect is an intentional design choice—unmapped payment paths will 404, which is standard practice during deprecation phases. No refactoring needed.apps/playground-web/src/app/navLinks.ts (5)
7-7: LGTM! Icon import follows naming convention.The
DollarSignIconimport follows the required "Icon" suffix convention for lucide-react icons.
99-125: All transaction routes verified and correctly configured.The Transactions submenu structure is properly organized and all five referenced routes exist in the app directory (
/transactions,/transactions/users,/transactions/airdrop-tokens,/transactions/mint-tokens, and/transactions/webhooks).
218-222: x402 top-level link is properly configured.The route exists at
apps/playground-web/src/app/x402/page.tsxand the link structure is correctly aligned with the PR objective to elevate x402 to its own category.
127-153: All account abstraction routes are correctly configured.All five routes referenced in the submenu have been verified to exist in the app directory. The navigation structure is properly aligned with the implemented pages.
179-211: Bridge routes verified with proper redirects in place.All six bridge routes are correctly implemented and discoverable. The migration from
/payments/*to/bridge/*is properly handled with comprehensive redirects configured innext.config.mjs:
/payments/fund-wallet→/bridge/buy-widget/payments/commerce→/bridge/checkout-widget/payments/transactions→/bridge/transaction-widget/payments→/bridge(root)New routes (
/bridge/swap-widget,/bridge/transaction-button) have no legacy equivalents and require no redirects. The navLinks structure is accurate and complete.apps/portal/src/app/bridge/products/page.mdx (1)
75-75: LGTM! Link updated correctly for Bridge restructuring.The Playground link has been correctly updated from
/payments/checkout-widgetto/bridge/checkout-widget, aligning with the PR's objective to move Payments functionality to Bridge.apps/portal/src/app/payments/sidebar.tsx (1)
16-16: LGTM! Playground link updated to Bridge.The Playground link has been correctly updated from
https://playground.thirdweb.com/paymentstohttps://playground.thirdweb.com/bridge, consistent with the PR's navigation restructuring.apps/portal/src/app/wallets/faq/page.mdx (2)
33-33: LGTM! Playground link updated to Wallets path.The Playground link has been correctly updated from
/connect/sign-in/buttonto/wallets/sign-in/button, aligning with the PR's navigation restructuring.
58-58: LGTM! Formatting cleanup.Trailing whitespace removed from the heading.
apps/portal/src/app/x402/page.mdx (1)
22-22: LGTM! Playground link updated to top-level x402 path.The Playground link has been correctly updated from
/payments/x402to/x402, aligning with the PR's objective to elevate x402 to its own top-level category.apps/portal/src/app/react/v5/components/ConnectEmbed/page.mdx (1)
33-33: LGTM! Playground link updated to Wallets path.The Playground link has been correctly updated from
/connect/sign-in/buttonto/wallets/sign-in/button, consistent with the PR's navigation restructuring.apps/portal/src/app/typescript/v5/ecosystem-wallet/page.mdx (2)
42-42: LGTM! Playground link updated to Wallets path.The Playground demo link has been correctly updated from
/connect/in-app-wallet/ecosystemto/wallets/ecosystem-wallet, aligning with the PR's navigation restructuring.
66-66: LGTM! Formatting cleanup.Trailing whitespace removed.
apps/portal/src/app/react/v5/pay/fund-wallets/page.mdx (1)
20-20: LGTM! Live demo link updated to Bridge buy-widget.The live demo link has been correctly updated from
/connect/pay/transactionsto/bridge/buy-widget, which is more appropriate for the "funding wallets" context and aligns with the PR's navigation restructuring.apps/portal/src/app/react/v5/pay/transaction/page.mdx (1)
20-20: LGTM!The playground link update from
/connect/pay/transactionsto/bridge/transaction-widgetcorrectly implements the navigation restructuring to move payment-related functionality under the Bridge category.apps/portal/src/app/payments/page.mdx (1)
117-117: LGTM!The playground link update maintains consistency with other payment-related documentation, correctly routing to the Bridge transaction widget demo.
apps/portal/src/app/bridge/transactions/page.mdx (1)
84-84: LGTM!The playground link update is consistent with the Bridge navigation restructuring and matches the pattern used across other transaction-related documentation.
apps/portal/src/app/bridge/fund/page.mdx (1)
72-72: LGTM!The playground link update from
/connect/pay/transactionsto/bridge/transaction-buttoncorrectly routes to the transaction button demo. Note that this intentionally usestransaction-buttonrather thantransaction-widget(used in other files), which is appropriate since this page demonstrates the BuyWidget for wallet funding rather than the TransactionWidget for payments.apps/portal/src/app/react/v5/account-abstraction/get-started/page.mdx (2)
103-158: LGTM!The minor whitespace and formatting changes are non-functional improvements to code consistency.
43-50: Destination route verified and approved.The /account-abstraction route exists with proper page handler configuration (page.tsx). The playground link updates are correct.
apps/portal/src/app/react/v5/auth/page.mdx (2)
21-28: LGTM!The playground link updates from
/connect/authto/wallets/authcorrectly consolidate authentication functionality under the Wallets category, consistent with other auth-related navigation changes in this PR.
41-44: LGTM!The minor whitespace changes are non-functional formatting improvements.
apps/portal/src/app/x402/sidebar.tsx (1)
16-16: LGTM! Playground route verified.The href update from
/payments/x402to/x402correctly implements the PR objective. Verification confirms the/x402route exists at the top level inapps/playground-web/src/app/x402/page.tsxand is properly configured in navigation and metadata.apps/portal/src/app/react/v5/components/ConnectButton/page.mdx (1)
30-30: Verified: destination route exists at/wallets/sign-in/button.The playground link update is confirmed correct. The actual page file exists at
apps/playground-web/src/app/wallets/sign-in/button/page.tsx, and the route is properly configured in both navigation and metadata files.apps/portal/src/app/react/v5/ecosystem-wallet/get-started/page.mdx (2)
39-39: Playground path links consolidated correctly.Both the markdown link and ArticleIconCard href are updated consistently to point to the new
/wallets/ecosystem-walletpath, aligning with the PR's restructuring of playground navigation. This ensures readers are directed to the same playground destination.Please verify that the new playground path
https://playground.thirdweb.com/wallets/ecosystem-walletis correctly configured and accessible as part of this restructuring work.Also applies to: 46-46
70-70: Minor formatting cleanup.Trailing whitespace removed from the custom UI reference text. Harmless and improves consistency.
apps/playground-web/src/app/data/pages-metadata.ts (2)
83-88: The Transactions and Account Abstraction cards are grouped underwalletsFeatureCardsfor semantic/feature organization, not because their routes are nested under/wallets/. The actual directory structure places both at the app root level (/transactionsand/account-abstraction), and the card links correctly point to these routes. Navigation will not break; the code is correct as is.Likely an incorrect or invalid review comment.
90-95: Account Abstraction link is correct; no changes needed.The review comment assumes Account Abstraction has been moved to the Wallets section, but the codebase shows account-abstraction remains at the root level as a separate section. The wallets directory contains only: auth, ecosystem-wallet, headless, in-app-wallet, sign-in, and social. The link
/account-abstractionin pages-metadata.ts correctly points to the existing route atapps/playground-web/src/app/account-abstraction/page.tsx.Likely an incorrect or invalid review comment.
apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx
Show resolved
Hide resolved
apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx
Show resolved
Hide resolved
Contributor
Merge activity
|
<!--
## 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 restructuring the `payments` and `contracts` sections of the codebase, consolidating them under a new `bridge` section, and updating various links and components to reflect the new structure.
### Detailed summary
- Deleted multiple files related to `payments` and `contracts`.
- Introduced new `bridge` components and pages.
- Updated links in various `.mdx` and `.tsx` files to point to the new `bridge` paths.
- Renamed components such as `BuyPlayground` to `BuyWidget`.
- Updated sidebar links for improved navigation.
- Adjusted feature cards to reflect the new structure and titles.
> ✨ 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
* **New Features**
* Reorganized navigation menu with new Bridge section consolidating payment widgets
* Added x402 as prominent top-level navigation item
* Expanded Account Abstraction section with new submenu items
* **Changes**
* Updated widget names and playground URLs across documentation and dashboard links
* Restructured Transactions and Account Abstraction into dedicated submenus under Wallets
* Renamed and relocated playground widget pages to Bridge section
* **Removals**
* Removed Contracts section from navigation and playground
* Removed standalone Payments overview page
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
graphite-app
bot
force-pushed
the
pro-96
branch
from
4e48ca5 to
be3316e
Compare
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
apps/playground-web/src/app/bridge/transaction-button/page.tsx (1)
83-96: Inconsistent code example: comment says ETH but code transfers USDC.The second TransactionButton has a comment stating "Price: 0.1 ETH" but the transaction transfers 50 USDC, identical to the first button. This inconsistency will confuse developers learning from this example.
Either update the comment to match the code, or if the intent is to demonstrate ETH transfers, update the transaction to actually send ETH:
Option 1: Fix the comment to match the USDC transfer:
- <p> Price: 0.1 ETH </p> + <p> Price: 50 USDC </p>Option 2: Update the transaction to actually send ETH (requires using a different transaction type, not the ERC20 transfer):
// Use native token transfer instead of ERC20 import { prepareTransaction, toWei } from "thirdweb"; <TransactionButton client={client} transaction={() => { if (!account) { throw new Error("No wallet connected"); } return prepareTransaction({ to: account.address, chain: arbitrum, client, value: toWei("0.1"), }); }} > Buy Premium Pass </TransactionButton>
♻️ Duplicate comments (2)
apps/playground-web/src/app/bridge/transaction-button/page.tsx (2)
10-10: Grammar error in description (already flagged).The description still has the grammatical error flagged in the previous review: "perform onchain transaction" should be "perform an onchain transaction" or "perform onchain transactions".
28-28: Incorrect documentation link (already flagged, but fix is wrong).The
docsLinkwas updated but is still incorrect. The previous review identified the correct URL ashttps://portal.thirdweb.com/react/v5/components/TransactionButton, but the current link points to/references/typescript/v5/TransactionButtoninstead of/react/v5/components/TransactionButton.Apply this diff to use the correct URL:
- docsLink="https://portal.thirdweb.com/references/typescript/v5/TransactionButton" + docsLink="https://portal.thirdweb.com/react/v5/components/TransactionButton?utm_source=playground"
🧹 Nitpick comments (2)
apps/playground-web/src/app/bridge/page.tsx (1)
5-14: Consider adding an explicit return type.The Page component would benefit from an explicit return type annotation for better type safety and documentation.
-export default function Page() { +export default function Page(): React.JSX.Element { return (apps/playground-web/src/app/bridge/checkout-widget/page.tsx (1)
22-22: Consider adding an explicit return type annotation.Per the TypeScript coding guidelines, function declarations should include explicit return types for better type safety and code clarity.
Apply this diff to add the return type:
-export default function Page() { +export default function Page(): React.JSX.Element { return (
📜 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 (47)
apps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsx(1 hunks)apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx(1 hunks)apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx(2 hunks)apps/playground-web/next.config.mjs(2 hunks)apps/playground-web/src/app/api/paywall/route.ts(1 hunks)apps/playground-web/src/app/bridge/buy-widget/page.tsx(1 hunks)apps/playground-web/src/app/bridge/checkout-widget/page.tsx(1 hunks)apps/playground-web/src/app/bridge/components/RightSection.tsx(1 hunks)apps/playground-web/src/app/bridge/page.tsx(1 hunks)apps/playground-web/src/app/bridge/swap-widget/page.tsx(1 hunks)apps/playground-web/src/app/bridge/transaction-button/page.tsx(2 hunks)apps/playground-web/src/app/bridge/transaction-widget/page.tsx(1 hunks)apps/playground-web/src/app/contracts/events/page.tsx(0 hunks)apps/playground-web/src/app/contracts/extensions/page.tsx(0 hunks)apps/playground-web/src/app/contracts/page.tsx(0 hunks)apps/playground-web/src/app/contracts/read/page.tsx(0 hunks)apps/playground-web/src/app/contracts/write/page.tsx(0 hunks)apps/playground-web/src/app/data/pages-metadata.ts(4 hunks)apps/playground-web/src/app/navLinks.ts(3 hunks)apps/playground-web/src/app/page.tsx(3 hunks)apps/playground-web/src/app/payments/page.tsx(0 hunks)apps/playground-web/src/app/x402/components/X402RightSection.tsx(1 hunks)apps/playground-web/src/app/x402/page.tsx(2 hunks)apps/playground-web/src/icons/ContractIcon.tsx(0 hunks)apps/portal/src/app/bridge/fund/page.mdx(1 hunks)apps/portal/src/app/bridge/products/page.mdx(1 hunks)apps/portal/src/app/bridge/transactions/page.mdx(1 hunks)apps/portal/src/app/payments/page.mdx(1 hunks)apps/portal/src/app/payments/sidebar.tsx(1 hunks)apps/portal/src/app/react/v5/account-abstraction/get-started/page.mdx(3 hunks)apps/portal/src/app/react/v5/auth/page.mdx(2 hunks)apps/portal/src/app/react/v5/components/ConnectButton/page.mdx(1 hunks)apps/portal/src/app/react/v5/components/ConnectEmbed/page.mdx(1 hunks)apps/portal/src/app/react/v5/ecosystem-wallet/get-started/page.mdx(2 hunks)apps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdx(5 hunks)apps/portal/src/app/react/v5/in-app-wallet/get-started/page.mdx(1 hunks)apps/portal/src/app/react/v5/linking/page.mdx(1 hunks)apps/portal/src/app/react/v5/pay/fund-wallets/page.mdx(1 hunks)apps/portal/src/app/react/v5/pay/transaction/page.mdx(1 hunks)apps/portal/src/app/react/v5/reading-state/page.mdx(0 hunks)apps/portal/src/app/react/v5/social/page.mdx(1 hunks)apps/portal/src/app/react/v5/transactions/page.mdx(0 hunks)apps/portal/src/app/typescript/v5/ecosystem-wallet/page.mdx(2 hunks)apps/portal/src/app/typescript/v5/in-app-wallet/page.mdx(1 hunks)apps/portal/src/app/wallets/faq/page.mdx(2 hunks)apps/portal/src/app/x402/page.mdx(1 hunks)apps/portal/src/app/x402/sidebar.tsx(1 hunks)
💤 Files with no reviewable changes (9)
- apps/playground-web/src/app/contracts/page.tsx
- apps/playground-web/src/app/contracts/read/page.tsx
- apps/portal/src/app/react/v5/reading-state/page.mdx
- apps/playground-web/src/app/payments/page.tsx
- apps/portal/src/app/react/v5/transactions/page.mdx
- apps/playground-web/src/app/contracts/events/page.tsx
- apps/playground-web/src/app/contracts/write/page.tsx
- apps/playground-web/src/app/contracts/extensions/page.tsx
- apps/playground-web/src/icons/ContractIcon.tsx
🚧 Files skipped from review as they are similar to previous changes (19)
- apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx
- apps/playground-web/src/app/bridge/swap-widget/page.tsx
- apps/portal/src/app/x402/page.mdx
- apps/portal/src/app/react/v5/ecosystem-wallet/get-started/page.mdx
- apps/portal/src/app/react/v5/in-app-wallet/get-started/page.mdx
- apps/portal/src/app/react/v5/social/page.mdx
- apps/portal/src/app/bridge/fund/page.mdx
- apps/dashboard/src/app/(app)/team/[team_slug]/(team)/TeamSidebarLayout.tsx
- apps/portal/src/app/wallets/faq/page.mdx
- apps/portal/src/app/bridge/products/page.mdx
- apps/portal/src/app/react/v5/account-abstraction/get-started/page.mdx
- apps/playground-web/src/app/bridge/transaction-widget/page.tsx
- apps/portal/src/app/react/v5/linking/page.mdx
- apps/portal/src/app/react/v5/components/ConnectButton/page.mdx
- apps/portal/src/app/react/v5/in-app-wallet/enable-gasless/page.mdx
- apps/playground-web/src/app/api/paywall/route.ts
- apps/playground-web/src/app/bridge/buy-widget/page.tsx
- apps/portal/src/app/payments/page.mdx
- apps/portal/src/app/bridge/transactions/page.mdx
🧰 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:
apps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/bridge/checkout-widget/page.tsxapps/playground-web/src/app/bridge/transaction-button/page.tsxapps/portal/src/app/x402/sidebar.tsxapps/playground-web/src/app/navLinks.tsapps/playground-web/src/app/x402/components/X402RightSection.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/page.tsxapps/playground-web/src/app/bridge/components/RightSection.tsxapps/portal/src/app/payments/sidebar.tsxapps/playground-web/src/app/data/pages-metadata.ts
**/*.{ts,tsx,js,jsx}
📄 CodeRabbit inference engine (CLAUDE.md)
Load heavy dependencies inside async paths to keep initial bundle lean (lazy loading)
Files:
apps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/bridge/checkout-widget/page.tsxapps/playground-web/src/app/bridge/transaction-button/page.tsxapps/portal/src/app/x402/sidebar.tsxapps/playground-web/src/app/navLinks.tsapps/playground-web/src/app/x402/components/X402RightSection.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/page.tsxapps/playground-web/src/app/bridge/components/RightSection.tsxapps/portal/src/app/payments/sidebar.tsxapps/playground-web/src/app/data/pages-metadata.ts
apps/{dashboard,playground-web}/**/*.{ts,tsx}
📄 CodeRabbit inference engine (CLAUDE.md)
apps/{dashboard,playground-web}/**/*.{ts,tsx}: Import UI primitives from@/components/ui/*(Button, Input, Select, Tabs, Card, Sidebar, Badge, Separator) in dashboard and playground apps
UseNavLinkfor internal navigation with automatic active states in dashboard and playground apps
Use Tailwind CSS only – no inline styles or CSS modules
Usecn()from@/lib/utilsfor conditional class logic
Use design system tokens (e.g.,bg-card,border-border,text-muted-foreground)
Server Components (Node edge): Start files withimport "server-only";
Client Components (browser): Begin files with'use client';
Always callgetAuthToken()to retrieve JWT from cookies on server side
UseAuthorization: Bearerheader – never embed tokens in URLs
Return typed results (e.g.,Project[],User[]) – avoidany
Wrap client-side data fetching calls in React Query (@tanstack/react-query)
Use descriptive, stablequeryKeysfor React Query cache hits
ConfigurestaleTime/cacheTimein React Query based on freshness (default ≥ 60s)
Keep tokens secret via internal API routes or server actions
Never importposthog-jsin server components
Files:
apps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/bridge/checkout-widget/page.tsxapps/playground-web/src/app/bridge/transaction-button/page.tsxapps/playground-web/src/app/navLinks.tsapps/playground-web/src/app/x402/components/X402RightSection.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/page.tsxapps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/data/pages-metadata.ts
🧠 Learnings (61)
📓 Common learnings
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7888
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx:77-81
Timestamp: 2025-08-20T10:35:18.543Z
Learning: The webhooks/payments route exists at apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/payments/page.tsx and was added as part of the unified project layout changes.
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7888
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx:77-81
Timestamp: 2025-08-20T10:35:18.543Z
Learning: The webhooks/payments route exists at apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/payments/page.tsx and was added as part of the unified project layout PR #7888.
📚 Learning: 2025-08-07T17:24:31.965Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7812
File: apps/dashboard/src/app/(app)/team/~/~project/[[...paths]]/page.tsx:1-11
Timestamp: 2025-08-07T17:24:31.965Z
Learning: In Next.js App Router, page components (page.tsx files) are server components by default and do not require the "server-only" import directive. The "server-only" directive is primarily used for utility functions, API helpers, and data access modules that should never be included in the client bundle.
Applied to files:
apps/playground-web/src/app/bridge/page.tsx
📚 Learning: 2025-08-20T10:35:18.543Z
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7888
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx:77-81
Timestamp: 2025-08-20T10:35:18.543Z
Learning: The webhooks/payments route exists at apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/payments/page.tsx and was added as part of the unified project layout changes.
Applied to files:
apps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/bridge/checkout-widget/page.tsxapps/portal/src/app/react/v5/pay/fund-wallets/page.mdxapps/playground-web/next.config.mjsapps/portal/src/app/typescript/v5/in-app-wallet/page.mdxapps/portal/src/app/x402/sidebar.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsxapps/portal/src/app/react/v5/pay/transaction/page.mdxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/page.tsxapps/portal/src/app/payments/sidebar.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/layout.tsx : Building layout shells (`layout.tsx`) and top-level pages that mainly assemble data.
Applied to files:
apps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/page.tsx
📚 Learning: 2025-08-20T10:35:18.543Z
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7888
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx:77-81
Timestamp: 2025-08-20T10:35:18.543Z
Learning: The webhooks/payments route exists at apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/webhooks/payments/page.tsx and was added as part of the unified project layout PR #7888.
Applied to files:
apps/playground-web/src/app/bridge/page.tsxapps/portal/src/app/react/v5/pay/fund-wallets/page.mdxapps/playground-web/next.config.mjsapps/portal/src/app/typescript/v5/in-app-wallet/page.mdxapps/portal/src/app/x402/sidebar.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsxapps/portal/src/app/react/v5/pay/transaction/page.mdxapps/playground-web/src/app/x402/page.tsxapps/portal/src/app/payments/sidebar.tsx
📚 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 apps/{dashboard,playground-web}/**/feature/components/** : Group feature-specific components under `feature/components/*` with barrel `index.ts`
Applied to files:
apps/playground-web/src/app/bridge/page.tsxapps/playground-web/src/app/page.tsxapps/playground-web/src/app/bridge/components/RightSection.tsxapps/playground-web/src/app/data/pages-metadata.ts
📚 Learning: 2025-10-03T23:36:00.631Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8181
File: packages/thirdweb/src/react/web/ui/Bridge/BuyWidget.tsx:27-27
Timestamp: 2025-10-03T23:36:00.631Z
Learning: In packages/thirdweb/src/react/web/ui/Bridge/BuyWidget.tsx, the component intentionally uses a hardcoded English locale (connectLocaleEn) rather than reading from the provider, as BuyWidget is designed to be English-only and does not require internationalization support.
Applied to files:
apps/playground-web/src/app/bridge/checkout-widget/page.tsxapps/portal/src/app/react/v5/pay/fund-wallets/page.mdxapps/playground-web/src/app/bridge/transaction-button/page.tsxapps/portal/src/app/react/v5/pay/transaction/page.mdxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/bridge/components/RightSection.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 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:
apps/portal/src/app/react/v5/pay/fund-wallets/page.mdxapps/playground-web/src/app/bridge/transaction-button/page.tsxapps/portal/src/app/typescript/v5/in-app-wallet/page.mdxapps/playground-web/src/app/navLinks.tsapps/portal/src/app/typescript/v5/ecosystem-wallet/page.mdxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/page.tsxapps/playground-web/src/app/bridge/components/RightSection.tsxapps/portal/src/app/react/v5/auth/page.mdx
📚 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: Surface breaking changes prominently in PR descriptions
Applied to files:
apps/playground-web/src/app/bridge/transaction-button/page.tsxapps/playground-web/src/app/data/pages-metadata.ts
📚 Learning: 2025-08-07T20:43:21.864Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7812
File: apps/dashboard/src/app/(app)/(dashboard)/published-contract/components/token-banner.tsx:48-60
Timestamp: 2025-08-07T20:43:21.864Z
Learning: In the TokenBanner component at apps/dashboard/src/app/(app)/(dashboard)/published-contract/components/token-banner.tsx, the Link components use target="_blank" with internal application routes (starting with "/") to open pages in new tabs within the same application. These internal links do not require rel="noopener noreferrer" security attributes, which are only needed for external URLs.
Applied to files:
apps/playground-web/src/app/bridge/transaction-button/page.tsxapps/portal/src/app/x402/sidebar.tsxapps/playground-web/src/app/navLinks.tsapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsxapps/portal/src/app/payments/sidebar.tsx
📚 Learning: 2025-09-23T19:56:43.668Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8106
File: packages/thirdweb/src/react/web/ui/ConnectWallet/Modal/ConnectEmbed.tsx:482-485
Timestamp: 2025-09-23T19:56:43.668Z
Learning: In packages/thirdweb/src/react/web/ui/ConnectWallet/Modal/ConnectEmbed.tsx, the EmbedContainer uses width: "100vw" intentionally rather than "100%" - this is by design for the bridge widget embedding use case.
Applied to files:
apps/playground-web/src/app/bridge/transaction-button/page.tsxapps/portal/src/app/react/v5/components/ConnectEmbed/page.mdx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{ts,tsx} : Redirect logic using `redirect()` from `next/navigation`.
Applied to files:
apps/playground-web/next.config.mjs
📚 Learning: 2025-09-05T19:45:05.006Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7984
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx:76-80
Timestamp: 2025-09-05T19:45:05.006Z
Learning: When verifying route existence for Next.js app router, search for the full nested path structure rather than using simple patterns. The AI summary often contains accurate information about file additions that should be referenced before running verification scripts.
Applied to files:
apps/playground-web/next.config.mjs
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Use `NavLink` for internal navigation with automatic active states in dashboard and playground apps
Applied to files:
apps/portal/src/app/typescript/v5/in-app-wallet/page.mdxapps/portal/src/app/x402/sidebar.tsxapps/playground-web/src/app/navLinks.tsapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsxapps/playground-web/src/app/x402/page.tsxapps/portal/src/app/react/v5/components/ConnectEmbed/page.mdxapps/playground-web/src/app/bridge/components/RightSection.tsxapps/portal/src/app/payments/sidebar.tsxapps/portal/src/app/react/v5/auth/page.mdx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Use design system tokens (e.g., `bg-card`, `border-border`, `text-muted-foreground`)
Applied to files:
apps/portal/src/app/typescript/v5/in-app-wallet/page.mdxapps/portal/src/app/x402/sidebar.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsxapps/playground-web/src/app/page.tsxapps/playground-web/src/app/data/pages-metadata.ts
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Import UI primitives from `@/components/ui/*` (Button, Input, Select, Tabs, Card, Sidebar, Badge, Separator) in dashboard and playground apps
Applied to files:
apps/portal/src/app/typescript/v5/in-app-wallet/page.mdxapps/portal/src/app/x402/sidebar.tsxapps/playground-web/src/app/navLinks.tsapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/page.tsxapps/playground-web/src/app/bridge/components/RightSection.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Use Tailwind CSS only – no inline styles or CSS modules
Applied to files:
apps/portal/src/app/typescript/v5/in-app-wallet/page.mdxapps/portal/src/app/x402/sidebar.tsxapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
📚 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/** : Support for in-app wallets (social/email login)
Applied to files:
apps/portal/src/app/typescript/v5/in-app-wallet/page.mdxapps/portal/src/app/typescript/v5/ecosystem-wallet/page.mdx
📚 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/** : Smart wallets with account abstraction
Applied to files:
apps/portal/src/app/typescript/v5/in-app-wallet/page.mdx
📚 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/** : Unified `Wallet` and `Account` interfaces in wallet architecture
Applied to files:
apps/portal/src/app/typescript/v5/in-app-wallet/page.mdx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Server Components (Node edge): Start files with `import "server-only";`
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/bridge/components/RightSection.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Client Components (browser): Begin files with `'use client';`
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/playground-web/src/app/navLinks.tsapps/playground-web/src/app/bridge/components/RightSection.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{tsx} : Expose `className` prop on root element of components for overrides
Applied to files:
apps/portal/src/app/x402/sidebar.tsx
📚 Learning: 2025-07-07T21:21:47.488Z
Learnt from: saminacodes
Repo: thirdweb-dev/js PR: 7543
File: apps/portal/src/app/pay/page.mdx:4-4
Timestamp: 2025-07-07T21:21:47.488Z
Learning: In the thirdweb-dev/js repository, lucide-react icons must be imported with the "Icon" suffix (e.g., ExternalLinkIcon, RocketIcon) as required by the new linting rule, contrary to the typical lucide-react convention of importing without the suffix.
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/playground-web/src/app/navLinks.tsapps/playground-web/src/app/x402/page.tsxapps/portal/src/app/payments/sidebar.tsxapps/playground-web/src/app/data/pages-metadata.ts
📚 Learning: 2025-06-18T04:30:04.326Z
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7365
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectFTUX/ProjectFTUX.tsx:16-17
Timestamp: 2025-06-18T04:30:04.326Z
Learning: Next.js Link component fully supports both internal and external URLs and works appropriately with all standard anchor attributes including target="_blank", rel="noopener noreferrer", etc. Using Link for external URLs is completely appropriate and recommended.
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/portal/src/app/payments/sidebar.tsx
📚 Learning: 2025-06-18T04:27:16.172Z
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7365
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectFTUX/ProjectFTUX.tsx:16-17
Timestamp: 2025-06-18T04:27:16.172Z
Learning: Next.js Link component supports external URLs without throwing errors. When used with absolute URLs (like https://...), it behaves like a regular anchor tag without client-side routing, but does not cause runtime crashes or errors as previously believed.
Applied to files:
apps/portal/src/app/x402/sidebar.tsxapps/portal/src/app/payments/sidebar.tsx
📚 Learning: 2025-09-05T19:45:22.949Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7984
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/components/FeatureCard.client.tsx:93-99
Timestamp: 2025-09-05T19:45:22.949Z
Learning: In the thirdweb dashboard project, rel="noopener noreferrer" attributes are not required for external links with target="_blank" in Link components, as confirmed by MananTank.
Applied to files:
apps/portal/src/app/x402/sidebar.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : Use `NavLink` (`@/components/ui/NavLink`) for internal navigation so active states are handled automatically.
Applied to files:
apps/playground-web/src/app/navLinks.ts
📚 Learning: 2025-07-31T16:17:42.753Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7768
File: apps/playground-web/src/app/navLinks.ts:1-1
Timestamp: 2025-07-31T16:17:42.753Z
Learning: Configuration files that import and reference React components (like icon components from lucide-react) need the "use client" directive, even if they primarily export static data, because the referenced components need to be executed in a client context when used by other client components.
Applied to files:
apps/playground-web/src/app/navLinks.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : Icons come from `lucide-react` or the project-specific `…/icons` exports – never embed raw SVG.
Applied to files:
apps/playground-web/src/app/navLinks.tsapps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/data/pages-metadata.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/components/*.client.tsx : Client components must start with `'use client';` before imports.
Applied to files:
apps/playground-web/src/app/navLinks.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : When you need access to browser APIs (localStorage, window, IntersectionObserver etc.).
Applied to files:
apps/playground-web/src/app/navLinks.tsapps/portal/src/app/react/v5/auth/page.mdx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : Prefer composable primitives over custom markup: `Button`, `Input`, `Select`, `Tabs`, `Card`, `Sidebar`, `Separator`, `Badge`.
Applied to files:
apps/playground-web/src/app/navLinks.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : Layouts should reuse `SidebarLayout` / `FullWidthSidebarLayout` (`@/components/blocks/SidebarLayout`).
Applied to files:
apps/playground-web/src/app/navLinks.tsapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsxapps/playground-web/src/app/x402/page.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Use descriptive, stable `queryKeys` for React Query cache hits
Applied to files:
apps/playground-web/src/app/x402/components/X402RightSection.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Wrap client-side data fetching calls in React Query (`tanstack/react-query`)
Applied to files:
apps/playground-web/src/app/x402/components/X402RightSection.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Keep `queryKey` stable and descriptive for cache hits.
Applied to files:
apps/playground-web/src/app/x402/components/X402RightSection.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Configure `staleTime`/`cacheTime` in React Query based on freshness (default ≥ 60s)
Applied to files:
apps/playground-web/src/app/x402/components/X402RightSection.tsx
📚 Learning: 2025-06-10T00:46:58.580Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7315
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/assets/create/nft/launch-nft.tsx:155-160
Timestamp: 2025-06-10T00:46:58.580Z
Learning: In the dashboard application, the route structure for team and project navigation is `/team/[team_slug]/[project_slug]/...` without a `/project/` segment. Contract links should be formatted as `/team/${teamSlug}/${projectSlug}/contract/${chainId}/${contractAddress}`.
Applied to files:
apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Use `Authorization: Bearer` header – never embed tokens in URLs
Applied to files:
apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx
📚 Learning: 2025-08-29T23:44:47.512Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7951
File: apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/_layout/contract-page-layout.tsx:38-38
Timestamp: 2025-08-29T23:44:47.512Z
Learning: The ContractPageLayout component in apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/_layout/contract-page-layout.tsx is not the root layout - it's nested within the dashboard layout which already handles footer positioning with min-h-dvh and AppFooter placement. The ContractPageLayout needs flex flex-col grow to properly participate in the parent's flex layout.
Applied to files:
apps/playground-web/src/app/x402/page.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : For notices & skeletons rely on `AnnouncementBanner`, `GenericLoadingPage`, `EmptyStateCard`.
Applied to files:
apps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/page.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 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:
apps/playground-web/src/app/x402/page.tsxapps/playground-web/src/app/bridge/components/RightSection.tsxapps/portal/src/app/react/v5/auth/page.mdx
📚 Learning: 2025-05-29T00:46:09.063Z
Learnt from: jnsdls
Repo: thirdweb-dev/js PR: 7188
File: apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/accounts/components/accounts-count.tsx:15-15
Timestamp: 2025-05-29T00:46:09.063Z
Learning: In the accounts component at apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/accounts/components/accounts-count.tsx, the 3-column grid layout (md:grid-cols-3) is intentionally maintained even when rendering only one StatCard, as part of the design structure for this component.
Applied to files:
apps/playground-web/src/app/page.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : Stick to design-tokens: background (`bg-card`), borders (`border-border`), muted text (`text-muted-foreground`) etc.
Applied to files:
apps/playground-web/src/app/page.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Pages requiring fast transitions where data is prefetched on the client.
Applied to files:
apps/playground-web/src/app/page.tsx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{tsx,jsx} : Reuse core UI primitives; avoid re-implementing buttons, cards, modals.
Applied to files:
apps/playground-web/src/app/page.tsx
📚 Learning: 2025-05-26T16:28:50.772Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7152
File: apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/tokens/shared-page.tsx:41-48
Timestamp: 2025-05-26T16:28:50.772Z
Learning: The `projectMeta` prop is not required for the server-rendered `ContractTokensPage` component in the tokens shared page, unlike some other shared pages where it's needed for consistency.
Applied to files:
apps/playground-web/src/app/page.tsx
📚 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 src/exports/react.native.ts : React Native specific exports are in `src/exports/react.native.ts`
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsx
📚 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 **/*.{ts,tsx} : Choose composition over inheritance; leverage utility types (`Partial`, `Pick`, etc.)
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsx
📚 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 **/*.{ts,tsx} : Write idiomatic TypeScript with explicit function declarations and return types
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsx
📚 Learning: 2025-05-30T17:14:25.332Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 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:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/portal/src/app/react/v5/auth/page.mdx
📚 Learning: 2025-09-24T11:09:45.142Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8106
File: packages/thirdweb/src/script-exports/readme.md:6-7
Timestamp: 2025-09-24T11:09:45.142Z
Learning: For thirdweb Bridge Widget script exports, the module is exported with globalName: "BridgeWidget" in tsup config, making the global API `BridgeWidget.render()` rather than `window.thirdweb.BridgeWidget.render()`.
Applied to files:
apps/playground-web/src/app/bridge/components/RightSection.tsx
📚 Learning: 2025-06-17T18:30:52.976Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 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:
apps/playground-web/src/app/bridge/components/RightSection.tsxapps/portal/src/app/react/v5/auth/page.mdx
📚 Learning: 2025-11-12T18:02:48.465Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8401
File: apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/payments/page.tsx:84-84
Timestamp: 2025-11-12T18:02:48.465Z
Learning: In the thirdweb-dev/js repository, when reviewing PRs that create new pages in apps/playground-web/src/app/, do not check production playground URLs with curl or similar tools to verify their existence. The pages are being created in the PR itself and will only be available after deployment. Focus on verifying the page files exist in the PR's codebase instead.
Applied to files:
apps/portal/src/app/payments/sidebar.tsx
📚 Learning: 2025-09-17T11:02:13.528Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 8044
File: packages/thirdweb/src/react/web/ui/Bridge/swap-widget/use-tokens.ts:15-17
Timestamp: 2025-09-17T11:02:13.528Z
Learning: The thirdweb `client` object is serializable and can safely be used in React Query keys, similar to the `contract` object.
Applied to files:
apps/portal/src/app/react/v5/auth/page.mdx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Prefer API routes or server actions to keep tokens secret; the browser only sees relative paths.
Applied to files:
apps/portal/src/app/react/v5/auth/page.mdx
📚 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 apps/{dashboard,playground-web}/**/*.{ts,tsx} : Always call `getAuthToken()` to retrieve JWT from cookies on server side
Applied to files:
apps/portal/src/app/react/v5/auth/page.mdx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Create light wrappers (e.g. `fetchJson`) that automatically attach the JWT from cookies/session when calling internal API routes.
Applied to files:
apps/portal/src/app/react/v5/auth/page.mdx
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
Repo: thirdweb-dev/js PR: 0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*.{ts,tsx} : Export default async functions without `'use client';` – they run on the Node edge.
Applied to files:
apps/portal/src/app/react/v5/auth/page.mdx
📚 Learning: 2025-06-13T13:03:41.732Z
Learnt from: MananTank
Repo: thirdweb-dev/js PR: 7332
File: apps/dashboard/src/app/(app)/(dashboard)/(chain)/[chain_id]/[contractAddress]/public-pages/nft/overview/nft-drop-claim.tsx:82-90
Timestamp: 2025-06-13T13:03:41.732Z
Learning: The thirdweb `contract` object is serializable and can safely be used in contexts (e.g., React-Query keys) that require serializable values.
Applied to files:
apps/portal/src/app/react/v5/auth/page.mdx
🧬 Code graph analysis (5)
apps/playground-web/src/app/bridge/page.tsx (3)
apps/playground-web/src/components/blocks/OverviewPage.tsx (1)
OverviewPage(4-40)apps/playground-web/src/app/data/pages-metadata.ts (1)
bridgeFeatureCards(119-154)apps/playground-web/src/icons/PayIcon.tsx (1)
PayIcon(1-21)
apps/playground-web/src/app/bridge/transaction-button/page.tsx (2)
apps/playground-web/src/app/bridge/buy-widget/page.tsx (1)
metadata(13-20)apps/playground-web/src/components/blocks/APIHeader.tsx (1)
PageLayout(49-70)
apps/playground-web/src/app/navLinks.ts (1)
apps/playground-web/src/components/blocks/full-width-sidebar-layout.tsx (1)
ShadcnSidebarLink(43-59)
apps/playground-web/src/app/page.tsx (1)
apps/playground-web/src/app/data/pages-metadata.ts (5)
aiFeatureCards(202-215)walletsFeatureCards(32-96)bridgeFeatureCards(119-154)tokensFeatureCards(186-200)x402FeatureCards(217-225)
apps/playground-web/src/app/data/pages-metadata.ts (1)
apps/playground-web/src/icons/SmartAccountIcon.tsx (1)
SmartAccountIcon(1-21)
⏰ 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, esbuild)
- GitHub Check: E2E Tests (pnpm, webpack)
- GitHub Check: Size
- GitHub Check: Unit Tests
- GitHub Check: Lint Packages
- GitHub Check: Build Packages
- GitHub Check: Analyze (javascript)
🔇 Additional comments (19)
apps/portal/src/app/react/v5/pay/fund-wallets/page.mdx (1)
20-20: Playground link updated correctly to bridge structure.The href has been successfully updated to point to the new bridge-based playground endpoint. Ensure the
/bridge/buy-widgetroute has been created in the playground application as part of this restructuring.apps/portal/src/app/react/v5/pay/transaction/page.mdx (1)
20-20: Playground link updated to new bridge endpoint.The href has been updated from the old payments/transactions path to the new
/bridge/transaction-widgetendpoint, consistent with the restructuring. Verify that this playground route exists and is accessible.apps/portal/src/app/react/v5/auth/page.mdx (2)
21-21: Live Playground link updates are consistent and correctly aligned with restructuring.Both the inline link (line 21) and ArticleIconCard href (line 28) have been updated from
/connect/authto/wallets/auth, reflecting the movement of auth functionality under Wallets as per PR objectives.Please verify that the new
https://playground.thirdweb.com/wallets/authpath exists and is accessible. Given that this is a portal documentation file pointing to an external playground, confirming the target route is live will ensure users don't encounter 404s.Also applies to: 28-28
41-41: Minor whitespace formatting is acceptable.The blank line additions (lines 41, 45) are minor cleanup that don't affect documentation rendering or functionality.
Also applies to: 45-45
apps/portal/src/app/typescript/v5/ecosystem-wallet/page.mdx (2)
42-42: Playground link correctly updated to reflect new Wallets structure.The ArticleIconCard href has been properly updated to
https://playground.thirdweb.com/wallets/ecosystem-wallet, aligning with the PR's restructuring to consolidate wallet-related features under a unified Wallets section.Please verify that the new
https://playground.thirdweb.com/wallets/ecosystem-walletpath exists and is accessible to users.
66-66: Minor whitespace adjustment is acceptable.The formatting change on line 66 is a minor cleanup with no functional impact.
apps/portal/src/app/typescript/v5/in-app-wallet/page.mdx (2)
30-30: Documentation description updated appropriately.The in-app wallet description has been refined to better reflect the wallet's capabilities, which is consistent with moving this feature to the Wallets section. The change looks good.
42-42: The updated playground link is valid and accessible.The path
https://playground.thirdweb.com/wallets/in-app-walletexists and is live as the In-App Wallets section of the playground, correctly reflecting the PR's restructuring to move in-app wallet functionality under the Wallets section. The link change is appropriate and the documentation update is verified.apps/playground-web/src/app/bridge/components/RightSection.tsx (1)
18-19: LGTM! Import paths updated correctly for the restructure.The import path changes from
../components/to./are consistent with the PR's objective to consolidate bridge-related components into a unified directory structure. TypeScript will verify these imports resolve correctly at build time.apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/components/ProjectSidebarLayout.tsx (1)
152-152: URL change verified as safe and functional.The new playground URL
https://playground.thirdweb.com/wallets/sign-in/buttonis live and returns HTTP 200. The old URL also redirects to the new one, ensuring no broken links. The change aligns with the PR's objective of moving sign-in functionality under the wallets category.apps/playground-web/src/app/page.tsx (2)
38-42: LGTM! Navigation restructure aligns with PR objectives.The updated feature section order (AI → Wallets → Bridge → Tokens → x402) and removal of the previous Contracts/Payments sections correctly implements the PRO-96 restructuring objectives.
58-58: LGTM! Full card display is appropriate.Removing the
.slice(0, 6)constraint to render all feature cards is appropriate for the restructured navigation, as section sizes vary (ranging from 1 to 9 cards) and the responsive grid layout handles this well.apps/playground-web/src/app/navLinks.ts (5)
99-124: LGTM! Transactions submenu correctly nested under Wallets.The new Transactions submenu structure aligns with the PR objective to "Move Transactions into Wallets" and provides clear organization for transaction-related features (user wallets, airdrops, minting, webhooks).
127-153: LGTM! Account Abstraction submenu correctly nested under Wallets.The Account Abstraction submenu structure correctly implements the PR objective to "Move Account Abstraction back to Wallets" and includes all relevant EIP standards (4337, 7702, 5792) plus Native AA support.
179-211: LGTM! Bridge section correctly replaces Payments.The renamed Bridge section with updated paths (
/bridge/*instead of/payments/*) correctly implements the "Move Payments to Bridge" objective and maintains all payment-related widgets (Swap, Buy, Checkout, Transaction Widget/Button).
218-222: LGTM! x402 elevated to top-level category.The standalone x402 link correctly implements the PR objective to "Place x402 into its own category" at the top level of the navigation.
99-222: All route paths verified and exist in the codebase.Verification confirms all 17 routes have corresponding page files:
- Transactions: 5/5 routes ✓
- Account Abstraction: 5/5 routes ✓
- Bridge: 6/6 routes ✓
- x402: 1/1 route ✓
apps/playground-web/src/app/bridge/checkout-widget/page.tsx (2)
7-7: LGTM! Title aligns with widget naming convention.The title change from "Checkout Component" to "Checkout Widget" is consistent with the broader PR objective of standardizing component naming with the "Widget" suffix.
17-17: Based on my analysis of the codebase:The icon and documentation link in the checkout-widget page are correct as-is and do not require changes.
The icon field in page metadata uses semantic values (what the feature does) rather than route-based values (where it's located). This pattern is consistent throughout the codebase:
- All bridge pages use
icon: "payments"because they are payment-related features- Other pages use
icon: "wallets"oricon: "transactions"based on their function- Even the standalone
/x402page usesicon: "payments"(not route-dependent)The docsLink pointing to
/paymentsis semantically appropriate for a payments checkout feature. The bridge reorganization changed the routing structure and information architecture, not the semantic classification of features.
| @@ -0,0 +1,14 @@ | |||
| import { OverviewPage } from "@/components/blocks/OverviewPage"; | |||
| import { PayIcon } from "@/icons/PayIcon"; | |||
Contributor
There was a problem hiding this comment.
Icon inconsistency between Bridge page and navigation.
The Bridge overview page uses PayIcon (line 2), but the navigation sidebar in apps/playground-web/src/app/navLinks.ts line 182 uses BringToFrontIcon for the Bridge submenu. This creates visual inconsistency in the UI.
Consider using BringToFrontIcon consistently:
-import { PayIcon } from "@/icons/PayIcon";
+import { BringToFrontIcon } from "lucide-react"; <OverviewPage
title="Bridge"
description="thirdweb Bridge lets developers swap and transfer any token across any chain instantly"
featureCards={bridgeFeatureCards}
- icon={PayIcon}
+ icon={BringToFrontIcon}
/>🤖 Prompt for AI Agents
In apps/playground-web/src/app/bridge/page.tsx around line 2, the file imports
PayIcon which creates a visual mismatch with the navigation (navLinks.ts line
182) that uses BringToFrontIcon; replace the PayIcon import with
BringToFrontIcon and update any JSX usage to reference BringToFrontIcon so the
Bridge page and sidebar use the same icon component across the app. Ensure the
import path matches where BringToFrontIcon is exported and remove the unused
PayIcon import.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR-Codex overview
This PR focuses on restructuring the payment and bridge-related components in the application, including renaming, moving files, and updating links to improve clarity and functionality.
Detailed summary
paymentsandcontracts.page.tsxfiles for better clarity (e.g.,BuyPlaygroundtoBuy Widget).bridgeinstead ofpayments.transactionsandaccount abstractionsections in the sidebar.Summary by CodeRabbit
New Features
Changes
Removals