merge master into dev

[abhishek-nexgen-dev]

No description provided.

[gemini-code-assist]

Code Review

This pull request introduces a comprehensive billing and credit management system, featuring a community wallet, transaction tracking, usage analytics with visualizations, and automated recharge capabilities. It also incorporates E2E testing with Playwright and refines several UI components. Feedback highlights a critical missing import for the BarChart3 icon and a naming collision for the useWalletTransactions hook. Additionally, improvements are suggested for the robustness of initials calculation, React key usage in charts, the timing of idempotency key generation to prevent duplicate payments, and the frequency logic of the auto-recharge system.

[gemini-code-assist]

The BarChart3 icon is used on line 64 but is not included in the imports from lucide-react. This will cause a runtime error when the sidebar items are rendered.

  Wallet as WalletIcon,
  BarChart3,

[gemini-code-assist]

A hook with the same name useWalletTransactions is already defined in src/features/Billing/v1/hooks/useWallet.ts. This naming collision can lead to confusion and potential bugs during import. Consider renaming this hook to something more descriptive, such as useWalletTransactionsById.

[gemini-code-assist]

The initials calculation can produce "undefined" strings if communityName contains multiple consecutive spaces, as part[0] on an empty string is undefined. Adding a filter to remove empty strings from the split result will make this more robust.

Suggested change

	.map((part) => part[0])
	.join("")
	.split(" ")
	.filter(Boolean)
	.map((part) => part[0])

[gemini-code-assist]

Using the array index as a key for Cell components is not recommended in React as it can lead to rendering issues if the data changes. Use a unique identifier from the data instead, such as item.category.

Suggested change

	{breakdown.map((_, i) => (
	<Cell key={i} fill={COLORS[i % COLORS.length]} />
	))}
	{breakdown.map((item, i) => (
	<Cell key={item.category} fill={COLORS[i % COLORS.length]} />
	))}

[gemini-code-assist]

The idempotency key is generated inside the handlePay function. If the user clicks the "Pay" button multiple times before the request completes (and before the button is disabled), each click will generate a new UUID, resulting in multiple distinct payment attempts. To ensure true idempotency, the key should be generated once when the modal opens or when the payment intent is first formed, and then reused for any retries of that specific intent.

[gemini-code-assist]

The idempotency key for auto-recharge is based on the current date (YYYY-MM-DD). This implementation restricts auto-recharge to once per day per wallet. If a community consumes credits rapidly and needs multiple recharges in a single day, subsequent attempts will be blocked by the idempotency check. Consider using a more granular timestamp or a unique session ID to allow multiple recharges when necessary.