Refactor wallet user API integration to use tw api #7987
Conversation
Migrates dashboard wallet user queries from manual fetch calls to the new @thirdweb-dev/api client, updating search, listing, and transformation logic in searchUsers.ts and useEmbeddedWallets.ts. Updates API client usage and types in sdk.gen.ts to match new endpoints and naming conventions. Adds @thirdweb-dev/api as a dependency in package.json. Closes BLD-243
|
|
Warning Rate limit exceeded@0xFirekeeper has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 0 minutes and 27 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
WalkthroughAdds Changes
Sequence Diagram(s)sequenceDiagram
participant UI as Dashboard UI
participant Hook as useEmbeddedWallets / searchUsers
participant API as @thirdweb-dev/api client
participant Svc as Thirdweb API
UI->>Hook: request users (filters, pagination)
Hook->>API: configure({ override: { baseUrl } })
Hook->>API: listUserWallets(query, { headers })
API->>Svc: GET /v1/wallets/user
Svc-->>API: 200 JSON or error
API-->>Hook: response
alt success
Hook->>Hook: transformToWalletUser(response.data)
Hook-->>UI: WalletUser[]
else error
Hook->>Hook: console.error(...)
Hook-->>UI: []
end
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Assessment against linked issues
Assessment against linked issues: Out-of-scope changes
✨ Finishing Touches
🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7987 +/- ##
=======================================
Coverage 56.53% 56.53%
=======================================
Files 904 904
Lines 58626 58626
Branches 4146 4146
=======================================
Hits 33145 33145
Misses 25375 25375
Partials 106 106
🚀 New features to boost your workflow:
|
size-limit report 📦
|
Moved THIRDWEB_API_HOST definition to a shared constants file and updated imports in searchUsers and useEmbeddedWallets to use the centralized constant. This improves maintainability and consistency across the codebase.
Changed the fallback THIRDWEB_API_HOST from 'https://api.thirdweb.com' to 'https://api.thirdweb-dev.com' to reflect the new API endpoint.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 5
🧹 Nitpick comments (8)
packages/api/src/client/sdk.gen.ts (1)
459-461: Typo: “signle” → “single” (public docs)Minor doc polish for Get Balance description.
- * Get native or ERC20 token balance for a wallet address. Can retrieve live balances for any ERC20 token on a signle chain, or native token balances across multiple chains. + * Get native or ERC20 token balance for a wallet address. Can retrieve live balances for any ERC20 token on a single chain, or native token balances across multiple chains.apps/dashboard/src/@/components/in-app-wallet-users-content/searchUsers.ts (2)
22-26: Stable user id: fallback to wallet address when profiles are emptyAn empty id breaks keys and selection. Prefer wallet address as a stable fallback.
- id: getProfileId(apiWallet.profiles[0]) || "", + id: getProfileId(apiWallet.profiles[0]) || apiWallet.address || "",
51-58: createdAt fallback can mislead; avoid “now” as synthetic creation timeUsing the current time skews sorting and UX. Omit the field or set null when missing.
- createdAt: apiWallet.createdAt || new Date().toISOString(), + ...(apiWallet.createdAt ? { createdAt: apiWallet.createdAt } : {}),If WalletUser requires createdAt, consider making it optional in the type used by this view.
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts (5)
2-6: Prefer lazy-loading the API client to keep bundles lean and avoid module side-effectsDefer importing
@thirdweb-dev/api(and running configuration) until inside the fetch path. This reduces initial dashboard bundle size and avoids global state at module import time.Apply within this hunk:
-import { configure, listUserWallets } from "@thirdweb-dev/api"; +// Defer '@thirdweb-dev/api' imports to the fetch path to reduce bundle size.Then inside the fetch function (outside this hunk), dynamically import and configure once:
let apiConfigured = false; async function getListUserWallets() { const mod = await import("@thirdweb-dev/api"); if (!apiConfigured) { mod.configure({ override: { baseUrl: THIRDWEB_API_HOST.replace(/\/+$/, "") } }); apiConfigured = true; } return mod.listUserWallets; }And replace direct calls with:
const listUserWallets = await getListUserWallets();
11-18: Base URL config: sanitize and mark this as a client module; avoid top-level side effects
- Sanitize trailing slashes to prevent
//v1/...joins.- This hook uses React Query and browser-only auth headers; declare
'use client'to avoid accidental server import.- Consider moving
configure(...)out of top-level (see previous comment).Apply within this hunk:
-const THIRDWEB_API_HOST = - process.env.NEXT_PUBLIC_THIRDWEB_API_HOST || "https://api.thirdweb.com"; +const THIRDWEB_API_HOST = (process.env.NEXT_PUBLIC_THIRDWEB_API_HOST ?? + "https://api.thirdweb.com").replace(/\/+$/, "");Additionally (outside this hunk), add at the top of the file:
'use client';
38-44: Minor: pull page size into a constant and consider optional filtersUse a
PAGE_SIZEconstant and thread optional filters (email/phone/address/id) when provided by callers.
59-67: Propagate richer errorsPreserve the original error when possible; fallback to a readable message rather than
"No data returned".- if (response.error || !response.data) { - const errorMessage = - typeof response.error === "string" - ? response.error - : "No data returned"; - throw new Error(errorMessage); - } + if (response.error || !response.data) { + const err = + response.error instanceof Error + ? response.error + : new Error( + typeof response.error === "string" + ? response.error + : "Request failed with no payload", + ); + throw err; + }
72-76: Improve log context (and consider routing to telemetry)Include
pageNumber,teamId, and client/ecosystem identifiers to speed up triage.- console.error("Failed to fetch wallets:", error); + console.error( + "Failed to fetch wallets", + { pageNumber, teamId, clientId, ecosystemSlug }, + error, + );
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (4)
apps/dashboard/package.json(1 hunks)apps/dashboard/src/@/components/in-app-wallet-users-content/searchUsers.ts(2 hunks)apps/dashboard/src/@/hooks/useEmbeddedWallets.ts(2 hunks)packages/api/src/client/sdk.gen.ts(20 hunks)
🧰 Additional context used
📓 Path-based instructions (5)
**/package.json
📄 CodeRabbit inference engine (AGENTS.md)
Track bundle budgets via
package.json#size-limit
Files:
apps/dashboard/package.json
**/*.{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
**/*.{ts,tsx}: Use explicit function declarations and explicit return types in TypeScript
Limit each file to one stateless, single‑responsibility function
Re‑use shared types from@/typeswhere applicable
Prefertypealiases overinterfaceexcept for nominal shapes
Avoidanyandunknownunless unavoidable; narrow generics when possible
Prefer composition over inheritance; use utility types (Partial, Pick, etc.)
Lazy‑import optional features and avoid top‑level side‑effects to reduce bundle size
Files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.tsapps/dashboard/src/@/components/in-app-wallet-users-content/searchUsers.tspackages/api/src/client/sdk.gen.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/dashboard/src/@/hooks/useEmbeddedWallets.tsapps/dashboard/src/@/components/in-app-wallet-users-content/searchUsers.tspackages/api/src/client/sdk.gen.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/dashboard/src/@/hooks/useEmbeddedWallets.tsapps/dashboard/src/@/components/in-app-wallet-users-content/searchUsers.ts
apps/{dashboard,playground}/**/*.{ts,tsx}
📄 CodeRabbit inference engine (AGENTS.md)
apps/{dashboard,playground}/**/*.{ts,tsx}: Import UI primitives from@/components/ui/_(e.g., Button, Input, Tabs, Card)
UseNavLinkfor internal navigation to get active state handling
Use Tailwind CSS for styling; no inline styles
Merge class names withcn()from@/lib/utilsfor conditional classes
Stick to design tokens (e.g., bg-card, border-border, text-muted-foreground)
Server Components must start withimport "server-only"; usenext/headers, server‑only env, heavy data fetching, andredirect()where appropriate
Client Components must start with'use client'; handle interactivity with hooks and browser APIs
Server-side data fetching: callgetAuthToken()from cookies, sendAuthorization: Bearer <token>header, and return typed results (avoidany)
Client-side data fetching: wrap calls in React Query with descriptive, stablequeryKeysand set sensiblestaleTime/cacheTime(≥ 60s default); keep tokens secret via internal routes or server actions
Do not importposthog-jsin server components (client-side only)
Files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.tsapps/dashboard/src/@/components/in-app-wallet-users-content/searchUsers.ts
🧠 Learnings (9)
📓 Common learnings
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/thirdweb/src/wallets/** : Support for in-app wallets (social/email login)
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/thirdweb/src/wallets/** : Unified `Wallet` and `Account` interfaces in wallet architecture
📚 Learning: 2025-07-18T19:19:55.613Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/thirdweb/src/wallets/** : Unified `Wallet` and `Account` interfaces in wallet architecture
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Interactive UI that relies on hooks (`useState`, `useEffect`, React Query, wallet hooks).
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:19:55.613Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/thirdweb/src/wallets/** : EIP-1193, EIP-5792, EIP-7702 standard support in wallet modules
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:19:55.613Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/thirdweb/src/wallets/** : Support for in-app wallets (social/email login)
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Anything that consumes hooks from `tanstack/react-query` or thirdweb SDKs.
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Use React Query (`tanstack/react-query`) for all client data fetching.
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-08-29T15:37:38.513Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: AGENTS.md:0-0
Timestamp: 2025-08-29T15:37:38.513Z
Learning: Applies to apps/{dashboard,playground}/**/*.{ts,tsx} : Client-side data fetching: wrap calls in React Query with descriptive, stable `queryKeys` and set sensible `staleTime/cacheTime` (≥ 60s default); keep tokens secret via internal routes or server actions
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:19:55.613Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to apps/{dashboard,playground-web}/**/*.{ts,tsx} : Wrap client-side data fetching calls in React Query (`tanstack/react-query`)
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
🧬 Code graph analysis (3)
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts (2)
packages/api/src/client/types.gen.ts (2)
ListUserWalletsResponses(627-844)ListUserWalletsData(601-614)packages/api/src/client/sdk.gen.ts (1)
listUserWallets(356-373)
apps/dashboard/src/@/components/in-app-wallet-users-content/searchUsers.ts (2)
packages/api/src/client/types.gen.ts (2)
ListUserWalletsResponses(627-844)ListUserWalletsData(601-614)packages/api/src/client/sdk.gen.ts (1)
listUserWallets(356-373)
packages/api/src/client/sdk.gen.ts (2)
packages/api/src/client/client/types.ts (1)
Options(191-199)packages/api/src/client/types.gen.ts (86)
InitiateAuthenticationData(3-87)InitiateAuthenticationResponses(100-164)InitiateAuthenticationErrors(89-98)CompleteAuthenticationData(169-303)CompleteAuthenticationResponses(316-338)CompleteAuthenticationErrors(305-314)SocialAuthenticationData(343-374)SocialAuthenticationErrors(376-381)GetMyWalletData(383-388)GetMyWalletResponses(405-596)GetMyWalletErrors(390-403)ListUserWalletsData(601-614)ListUserWalletsResponses(627-844)ListUserWalletsErrors(616-625)CreateUserWalletData(849-875)CreateUserWalletResponses(892-1083)CreateUserWalletErrors(877-890)ListServerWalletsData(1088-1096)ListServerWalletsResponses(1109-1326)ListServerWalletsErrors(1098-1107)CreateServerWalletData(1331-1344)CreateServerWalletResponses(1361-1552)CreateServerWalletErrors(1346-1359)SignMessageData(2160-2181)SignMessageResponses(2198-2210)SignMessageErrors(2183-2196)SignTypedDataData(2215-2282)SignTypedDataResponses(2299-2311)SignTypedDataErrors(2284-2297)SendTokensData(2316-2354)SendTokensResponses(2371-2383)SendTokensErrors(2356-2369)ListContractsData(2387-2401)ListContractsResponses(2422-2485)ListContractsErrors(2403-2420)DeployContractData(2490-2525)DeployContractResponses(2546-2566)DeployContractErrors(2527-2544)ReadContractData(2571-2602)ReadContractResponses(2619-2642)ReadContractErrors(2604-2617)WriteContractData(2647-2682)WriteContractResponses(2703-2715)WriteContractErrors(2684-2701)GetContractTransactionsData(2720-2779)GetContractTransactionsResponses(2800-2938)GetContractTransactionsErrors(2781-2798)GetContractEventsData(2943-3006)GetContractEventsResponses(3027-3117)GetContractEventsErrors(3008-3025)GetContractMetadataData(3122-3136)GetContractMetadataResponses(3161-3254)GetContractMetadataErrors(3138-3159)GetContractSignaturesData(3259-3273)GetContractSignaturesResponses(3298-3308)GetContractSignaturesErrors(3275-3296)GetTransactionByIdData(3313-3323)GetTransactionByIdResponses(3344-3416)GetTransactionByIdErrors(3325-3342)CreatePaymentData(3622-3669)CreatePaymentResponses(3690-3707)CreatePaymentErrors(3671-3688)GetPaymentHistoryData(3712-3719)GetPaymentHistoryResponses(3739-3807)GetPaymentHistoryErrors(3721-3734)PaymentsPurchaseData(3812-3828)PaymentsPurchaseResponses(3849-3861)PaymentsPurchaseErrors(3830-3847)ListTokensData(3866-3896)ListTokensResponses(3913-3956)ListTokensErrors(3898-3911)CreateTokenData(3960-4031)CreateTokenResponses(4052-4066)CreateTokenErrors(4033-4050)GetTokenOwnersData(4071-4094)GetTokenOwnersResponses(4115-4154)GetTokenOwnersErrors(4096-4113)BridgeSwapData(4159-4213)BridgeSwapResponses(4234-4247)BridgeSwapErrors(4215-4232)ChatData(4251-4310)ChatResponses(4312-4391)McpServerData(4395-4400)McpServerResponses(4402-4407)LlmsTxtData(4409-4414)LlmsTxtResponses(4416-4421)
⏰ 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). (6)
- GitHub Check: Size
- GitHub Check: E2E Tests (pnpm, esbuild)
- GitHub Check: E2E Tests (pnpm, vite)
- GitHub Check: Build Packages
- GitHub Check: Lint Packages
- GitHub Check: Analyze (javascript)
🔇 Additional comments (7)
packages/api/src/client/sdk.gen.ts (2)
350-373: Backend-only: listUserWallets requires x-secret-keySDK correctly models this as secret-key–only. Ensure dashboard code (searchUsers.ts) does not call this from the browser or with a user Bearer token. Route via server action/internal API that injects x-secret-key.
472-479: Include secret-key auth in security for server-side usageDocs say x-client-id or x-secret-key. The security section only lists x-client-id, which may prevent auto-auth via secret if your client supports it.
If your client auto-injects auth based on this array, add x-secret-key to security here (and similarly for getWalletTokens/Nfts/Transactions, getContractEvents/Transactions, listTokens, getTokenOwners) to match docs. Otherwise, confirm manual headers suffice.
apps/dashboard/src/@/components/in-app-wallet-users-content/searchUsers.ts (1)
56-57: Verify wallet type mappingHardcoding type: "enclave" for all addresses may be incorrect. Derive from available fields (e.g., presence of smartWalletAddress) or align with WalletUser’s allowed types.
Would you like a tiny helper that infers type = "smart" | "eoa" | "custodial" based on apiWallet fields?
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts (4)
20-23: Type extraction LGTMGood use of SDK-derived types to keep transformations honest.
120-134: Helper LGTMStraightforward narrowing across profile variants.
45-57: Verify auth header, cancellation support, and drop Content-Type
- Confirm that listUserWallets expects the SDK’s security header (e.g.
x-secret-key) rather than sendingAuthorization: Bearer <jwt>from the client to avoid 401s.- Check whether listUserWallets accepts an
AbortSignal; if so, forward React Query’ssignalto cancel in-flight requests.- Remove the unnecessary
"Content-Type": "application/json"on a GET.
79-118: Guarantee stable non-empty IDs, correct wallet mapping, and deterministic timestamps
- Fallback
idtoapiWallet.address/apiWallet.smartWalletAddressto avoid empty keys.- Map both EOA (
address) and smart wallets separately with the correcttype, using the API’screatedAt.- Simplify
detailstoRecord<string, string>.Verify that
APIWalletincludessmartWalletAddressand that these fields match theWalletUserspecification.
apps/dashboard/src/@/components/in-app-wallet-users-content/searchUsers.ts
Show resolved
Hide resolved
apps/dashboard/src/@/components/in-app-wallet-users-content/searchUsers.ts
Show resolved
Hide resolved
Updated the knip.json configuration to include @thirdweb-dev/api in the ignoreDependencies list for the dashboard app.
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/dashboard/src/@/hooks/useEmbeddedWallets.ts (1)
167-209: Use hasMore to terminate; avoid potential over-fetchRely on server pagination instead of zero-length heuristic.
- const res = await queryClient.fetchQuery<{ - users: WalletUser[]; - }>({ + const res = await queryClient.fetchQuery<{ + users: WalletUser[]; + hasMore?: boolean; + }>({ queryFn: fetchAccountList({ clientId, ecosystemSlug, teamId, jwt: authToken, pageNumber: page, }), queryKey: embeddedWalletsKeys.embeddedWallets( address || "", clientId || ecosystemSlug || "", page, ), }); - if (res.users.length === 0) { + if (!res.hasMore) { break; } page++; responses.push(...res.users);
♻️ Duplicate comments (1)
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts (1)
68-70: Return pagination.hasMore to avoid length-based loopsExpose server intent to terminate pagination.
- return { - users: response.data.result.wallets.map(transformToWalletUser), - }; + return { + users: response.data.result.wallets.map(transformToWalletUser), + hasMore: Boolean(response.data.result.pagination?.hasMore), + };
🧹 Nitpick comments (10)
apps/dashboard/knip.json (1)
11-11: Do we need to ignore "@thirdweb-dev/api"?It’s actively imported in dashboard code; removing it from ignoreDependencies helps Knip catch accidental removals. If Knip fails to detect type‑only usage, keep the ignore but add a comment explaining why.
apps/dashboard/src/@/constants/urls.ts (1)
1-2: Add fallback to old env var to smooth deploysKeep backward compatibility during the env rollout.
-export const THIRDWEB_API_HOST = - process.env.NEXT_PUBLIC_THIRDWEB_API_HOST || "https://api.thirdweb-dev.com"; +export const THIRDWEB_API_HOST = + process.env.NEXT_PUBLIC_THIRDWEB_API_HOST || + process.env.NEXT_PUBLIC_THIRDWEB_EWS_API_HOST || // fallback for older envs + "https://api.thirdweb-dev.com";apps/dashboard/src/@/hooks/useEmbeddedWallets.ts (8)
12-17: Avoid module-scoped client configuration duplicationConsider centralizing API client configure() in a single module (e.g., apiClient.ts) to prevent accidental overrides across imports.
39-42: Extract page size to a constant and accept overrideImproves reuse and testing; some lists may prefer 100.
- const queryParams: ListUserWalletsData["query"] = { - page: pageNumber, - limit: 50, // Keep the same page size - }; + const PAGE_SIZE = 50; + const queryParams: ListUserWalletsData["query"] = { + page: pageNumber, + limit: PAGE_SIZE, + };
45-56: Drop Content-Type on GETNot needed and occasionally triggers CORS preflight unnecessarily.
const response = await listUserWallets({ query: queryParams, headers: { Authorization: `Bearer ${jwt}`, - "Content-Type": "application/json", "x-thirdweb-team-id": teamId, ...(clientId && { "x-client-id": clientId }), ...(ecosystemSlug && { "x-ecosystem-id": `ecosystem.${ecosystemSlug}`, }), }, });
59-66: Preserve server error detailsSurface message/code if present for better debugging.
- if (response.error || !response.data) { - const errorMessage = - typeof response.error === "string" - ? response.error - : "No data returned"; - throw new Error(errorMessage); - } + if (response.error || !response.data) { + const err = response.error as + | string + | { message?: string; code?: string; error?: string }; + const msg = + typeof err === "string" + ? err + : err?.message || err?.error || "No data returned"; + throw new Error(msg); + }
120-132: Broaden profile ID fallback, keep deterministicIf API profiles can include other identifiers (e.g., userId), consider extending fallback. If not, ignore.
if ("id" in profile) { return profile.id; } else if ("credentialId" in profile) { return profile.credentialId; } else if ("identifier" in profile) { return profile.identifier; + } else if ("userId" in (profile as any)) { + return (profile as any).userId as string; }
144-158: Set sensible cache/stale timings and add explicit return typeMatch dashboard data-fetching guidelines.
- return useQuery({ + return useQuery<{ + users: WalletUser[]; + hasMore?: boolean; + }>({ enabled: !!address && !!(clientId || ecosystemSlug), + staleTime: 60_000, + gcTime: 5 * 60_000,
134-159: Add explicit hook return typesFor maintainability and DX, annotate the return types of both hooks.
Also applies to: 167-209
23-35: Auth token usage in clientEnsure jwt is ephemeral and scoped. Prefer proxying via internal API route/server action to keep tokens off the client.
Also applies to: 144-158, 167-209
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (4)
apps/dashboard/knip.json(1 hunks)apps/dashboard/src/@/components/in-app-wallet-users-content/searchUsers.ts(2 hunks)apps/dashboard/src/@/constants/urls.ts(1 hunks)apps/dashboard/src/@/hooks/useEmbeddedWallets.ts(2 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- apps/dashboard/src/@/components/in-app-wallet-users-content/searchUsers.ts
🧰 Additional context used
📓 Path-based instructions (4)
**/*.{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
**/*.{ts,tsx}: Use explicit function declarations and explicit return types in TypeScript
Limit each file to one stateless, single‑responsibility function
Re‑use shared types from@/typeswhere applicable
Prefertypealiases overinterfaceexcept for nominal shapes
Avoidanyandunknownunless unavoidable; narrow generics when possible
Prefer composition over inheritance; use utility types (Partial, Pick, etc.)
Lazy‑import optional features and avoid top‑level side‑effects to reduce bundle size
Files:
apps/dashboard/src/@/constants/urls.tsapps/dashboard/src/@/hooks/useEmbeddedWallets.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/dashboard/src/@/constants/urls.tsapps/dashboard/src/@/hooks/useEmbeddedWallets.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/dashboard/src/@/constants/urls.tsapps/dashboard/src/@/hooks/useEmbeddedWallets.ts
apps/{dashboard,playground}/**/*.{ts,tsx}
📄 CodeRabbit inference engine (AGENTS.md)
apps/{dashboard,playground}/**/*.{ts,tsx}: Import UI primitives from@/components/ui/_(e.g., Button, Input, Tabs, Card)
UseNavLinkfor internal navigation to get active state handling
Use Tailwind CSS for styling; no inline styles
Merge class names withcn()from@/lib/utilsfor conditional classes
Stick to design tokens (e.g., bg-card, border-border, text-muted-foreground)
Server Components must start withimport "server-only"; usenext/headers, server‑only env, heavy data fetching, andredirect()where appropriate
Client Components must start with'use client'; handle interactivity with hooks and browser APIs
Server-side data fetching: callgetAuthToken()from cookies, sendAuthorization: Bearer <token>header, and return typed results (avoidany)
Client-side data fetching: wrap calls in React Query with descriptive, stablequeryKeysand set sensiblestaleTime/cacheTime(≥ 60s default); keep tokens secret via internal routes or server actions
Do not importposthog-jsin server components (client-side only)
Files:
apps/dashboard/src/@/constants/urls.tsapps/dashboard/src/@/hooks/useEmbeddedWallets.ts
🧠 Learnings (8)
📓 Common learnings
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/thirdweb/src/wallets/** : Unified `Wallet` and `Account` interfaces in wallet architecture
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/thirdweb/src/wallets/** : EIP-1193, EIP-5792, EIP-7702 standard support in wallet modules
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/thirdweb/src/wallets/** : Support for in-app wallets (social/email login)
📚 Learning: 2025-07-18T19:19:55.613Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/thirdweb/src/wallets/** : Unified `Wallet` and `Account` interfaces in wallet architecture
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:19:55.613Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/thirdweb/src/wallets/** : EIP-1193, EIP-5792, EIP-7702 standard support in wallet modules
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Interactive UI that relies on hooks (`useState`, `useEffect`, React Query, wallet hooks).
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Anything that consumes hooks from `tanstack/react-query` or thirdweb SDKs.
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Use React Query (`tanstack/react-query`) for all client data fetching.
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-08-29T15:37:38.513Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: AGENTS.md:0-0
Timestamp: 2025-08-29T15:37:38.513Z
Learning: Applies to apps/{dashboard,playground}/**/*.{ts,tsx} : Client-side data fetching: wrap calls in React Query with descriptive, stable `queryKeys` and set sensible `staleTime/cacheTime` (≥ 60s default); keep tokens secret via internal routes or server actions
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:19:55.613Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to apps/{dashboard,playground-web}/**/*.{ts,tsx} : Wrap client-side data fetching calls in React Query (`tanstack/react-query`)
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
⏰ 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 (1)
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts (1)
144-146: Revisit address gatingListing users likely shouldn’t depend on active wallet address. If this was intentional, ignore.
Enhanced the fetchAccountList function and related hooks to handle paginated API responses using the hasMore flag. This ensures all wallet users are fetched across multiple pages, improving reliability and scalability of wallet data retrieval.
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (3)
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts (3)
68-71: Good: expose server pagination intentReturning
hasMorealigns the consumer loop with server pagination. Nice.
82-82: Stabilize user.id (avoid empty-string collisions)Fallback to the wallet address when no profile id exists to keep ids unique.
- id: getProfileId(apiWallet.profiles[0]) || "", + id: getProfileId(apiWallet.profiles[0]) || apiWallet.address || "",
112-112: Don’t fabricate createdAtAvoid setting
createdAtto client time; omit it when absent to prevent misleading data.- createdAt: apiWallet.createdAt || new Date().toISOString(), + createdAt: apiWallet.createdAt || undefined,
🧹 Nitpick comments (6)
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts (6)
12-17: Centralize API client configurationConfiguring the API client at module load is fine, but consider moving this to a single shared module (e.g., "@/lib/apiClient") to avoid divergent config if used elsewhere. Optionally lazy-load to keep bundles lean.
49-49: Drop Content-Type on GET
Content-Typeisn’t needed for a GET without a body.- "Content-Type": "application/json",
145-159: Set sensible cache policy for React QueryPer guidelines, add ≥60s freshness. Also consider a GC time.
return useQuery({ enabled: !!address && !!(clientId || ecosystemSlug), @@ queryKey: embeddedWalletsKeys.embeddedWallets( address || "", clientId || ecosystemSlug || "", page, ), + staleTime: 60_000, + gcTime: 5 * 60_000, });
179-207: Add a pagination safety cap to avoid infinite loopsGuard against bad
hasMoreresponses and empty pages.const responses: WalletUser[] = []; let page = 1; + const MAX_PAGES = 1000; - while (true) { + while (page <= MAX_PAGES) { @@ - if (!res.hasMore) { + if (!res.hasMore || res.users.length === 0) { break; } @@ page++; }
23-36: Annotate return types for clarityAdd an explicit page type and annotate the async factory’s return type.
type APIProfile = APIWallet["profiles"][0]; +type WalletListPage = { + users: WalletUser[]; + hasMore: boolean; +}; @@ -}) => { - return async () => { +}) => { + return async (): Promise<WalletListPage> => {
45-56: Hide tokens behind an internal API routePassing
authTokenfrom the client and calling the Thirdweb API directly exposes credentials in the browser. Prefer a server action or internal API route that reads the JWT viagetAuthToken()and forwards the request.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts(1 hunks)
🧰 Additional context used
📓 Path-based instructions (4)
**/*.{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
**/*.{ts,tsx}: Use explicit function declarations and explicit return types in TypeScript
Limit each file to one stateless, single‑responsibility function
Re‑use shared types from@/typeswhere applicable
Prefertypealiases overinterfaceexcept for nominal shapes
Avoidanyandunknownunless unavoidable; narrow generics when possible
Prefer composition over inheritance; use utility types (Partial, Pick, etc.)
Lazy‑import optional features and avoid top‑level side‑effects to reduce bundle size
Files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.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/dashboard/src/@/hooks/useEmbeddedWallets.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/dashboard/src/@/hooks/useEmbeddedWallets.ts
apps/{dashboard,playground}/**/*.{ts,tsx}
📄 CodeRabbit inference engine (AGENTS.md)
apps/{dashboard,playground}/**/*.{ts,tsx}: Import UI primitives from@/components/ui/_(e.g., Button, Input, Tabs, Card)
UseNavLinkfor internal navigation to get active state handling
Use Tailwind CSS for styling; no inline styles
Merge class names withcn()from@/lib/utilsfor conditional classes
Stick to design tokens (e.g., bg-card, border-border, text-muted-foreground)
Server Components must start withimport "server-only"; usenext/headers, server‑only env, heavy data fetching, andredirect()where appropriate
Client Components must start with'use client'; handle interactivity with hooks and browser APIs
Server-side data fetching: callgetAuthToken()from cookies, sendAuthorization: Bearer <token>header, and return typed results (avoidany)
Client-side data fetching: wrap calls in React Query with descriptive, stablequeryKeysand set sensiblestaleTime/cacheTime(≥ 60s default); keep tokens secret via internal routes or server actions
Do not importposthog-jsin server components (client-side only)
Files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
🧠 Learnings (8)
📓 Common learnings
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/thirdweb/src/wallets/** : Support for in-app wallets (social/email login)
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/thirdweb/src/wallets/** : Unified `Wallet` and `Account` interfaces in wallet architecture
📚 Learning: 2025-07-18T19:19:55.613Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/thirdweb/src/wallets/** : Unified `Wallet` and `Account` interfaces in wallet architecture
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:19:55.613Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to packages/thirdweb/src/wallets/** : EIP-1193, EIP-5792, EIP-7702 standard support in wallet modules
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Anything that consumes hooks from `tanstack/react-query` or thirdweb SDKs.
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Use React Query (`tanstack/react-query`) for all client data fetching.
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-08-29T15:37:38.513Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: AGENTS.md:0-0
Timestamp: 2025-08-29T15:37:38.513Z
Learning: Applies to apps/{dashboard,playground}/**/*.{ts,tsx} : Client-side data fetching: wrap calls in React Query with descriptive, stable `queryKeys` and set sensible `staleTime/cacheTime` (≥ 60s default); keep tokens secret via internal routes or server actions
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:19:55.613Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: CLAUDE.md:0-0
Timestamp: 2025-07-18T19:19:55.613Z
Learning: Applies to apps/{dashboard,playground-web}/**/*.{ts,tsx} : Wrap client-side data fetching calls in React Query (`tanstack/react-query`)
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
📚 Learning: 2025-07-18T19:20:32.530Z
Learnt from: CR
PR: thirdweb-dev/js#0
File: .cursor/rules/dashboard.mdc:0-0
Timestamp: 2025-07-18T19:20:32.530Z
Learning: Applies to dashboard/**/*client.tsx : Interactive UI that relies on hooks (`useState`, `useEffect`, React Query, wallet hooks).
Applied to files:
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts
🧬 Code graph analysis (1)
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts (3)
apps/dashboard/src/@/constants/urls.ts (1)
THIRDWEB_API_HOST(1-2)packages/api/src/client/types.gen.ts (2)
ListUserWalletsResponses(627-844)ListUserWalletsData(601-614)packages/api/src/client/sdk.gen.ts (1)
listUserWallets(356-373)
⏰ 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 (2)
apps/dashboard/src/@/hooks/useEmbeddedWallets.ts (2)
146-147: Verify address gating is intendedFetching user wallets may not require a connected
address. If admins should load lists without connecting, consider removing theaddressdependency fromenabled.
47-55: Auth scheme and header names confirmed
The/v1/wallets/userendpoint accepts a Bearer JWT (Authorization: Bearer <token>) alongsidex-thirdweb-team-idandx-ecosystem-id: ecosystem.<slug>. Nox-secret-keyheader is required when authenticating with a valid Bearer JWT. [1][4][6]
Migrates dashboard wallet user queries from manual fetch calls to the new @thirdweb-dev/api client, updating search, listing, and transformation logic in searchUsers.ts and useEmbeddedWallets.ts. Updates API client usage and types in sdk.gen.ts to match new endpoints and naming conventions. Adds @thirdweb-dev/api as a dependency in package.json.
Closes BLD-243
PR-Codex overview
This PR focuses on updating the API integration in the
dashboardapplication, enhancing wallet functionalities, and improving the handling of user data through the new API endpoints.Detailed summary
THIRDWEB_API_HOSTinurls.ts.listUserWalletsAPI for user wallet retrieval.package.jsonandpnpm-lock.yaml.Summary by CodeRabbit
New Features
Refactor
Chores