fix(provider): added a balance load check to prevent balance popup

[jigar-arc10]

Summary by CodeRabbit

• Bug Fixes

  • Strengthened wallet balance checks to require balance data be fully loaded before validation and notifications.
  • Prevented premature insufficient-balance popups and blocked progression until balance state is known.

• New Features

  • Next button shows a loading hint and is disabled while wallet balance is loading to improve setup flow.

[coderabbitai]

Walkthrough

Added explicit loading-state tracking for wallet balances in ServerAccess. The component now requires balance data to be loaded before performing balance checks, gating the Next action and showing a loading hint until balances are available.

Changes

Cohort / File(s)	Summary
Balance loading & null-safety apps/provider-console/src/components/become-provider/ServerAccess.tsx	Added isBalanceLoaded = walletBalances !== null. Tightened hasEnoughBalance to require walletBalances !== null and minimum balance. Updated balance-popup trigger, handleNextClick early-return when not loaded, and useEffect/handler dependency arrays. UI now shows a loading hint and disables Next until balances load.

Sequence Diagram(s)

sequenceDiagram
  participant UI as ServerAccess UI
  participant Wallet as WalletBalancesStore
  participant Flow as Navigation/Next

  Note over Wallet,UI `#D3E4FF`: Initial load
  UI->>Wallet: request balances
  Wallet-->>UI: balances = null (loading)
  UI->>UI: show "loading balance" hint, disable Next

  Note over Wallet,UI `#D3FFD8`: When data arrives
  Wallet-->>UI: balances (loaded)
  UI->>UI: isBalanceLoaded = true
  alt balance >= MIN_BALANCE
    UI->>Flow: allow Next (proceed)
  else balance < MIN_BALANCE
    UI->>UI: show insufficient balance popup
    UI-xFlow: block Next
  end

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• Single-file change but affects control flow, hooks (useEffect) and event handler dependencies.
• Pay extra attention to:
  • Correct dependency list updates for useEffect and useCallback/handleNextClick.
  • Edge cases where walletBalances transitions from null to a value; ensure UI state updates and popups behave as intended.
  • Accessibility/state of the Next button while loading.

Poem

🐰 I waited by the wallet gate,

until the balances did state,
"Loaded!" cried the tiny flag,
then onward hopped the next-step wag,
no nulls to trip my fluffy fate. 🥕✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: adding a balance load check to prevent balance popup issues in the provider console.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/provider-console/min-balance-issue

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e2703e1 and 8a951b4.

📒 Files selected for processing (1)

• apps/provider-console/src/components/become-provider/ServerAccess.tsx (4 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• apps/provider-console/src/components/become-provider/ServerAccess.tsx

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: test-build

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46.85%. Comparing base (f784754) to head (8a951b4).
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2200      +/-   ##
==========================================
- Coverage   47.19%   46.85%   -0.35%     
==========================================
  Files        1023     1013      -10     
  Lines       29078    28729     -349     
  Branches     7584     7543      -41     
==========================================
- Hits        13724    13461     -263     
+ Misses      14961    14884      -77     
+ Partials      393      384       -9     

Flag	Coverage Δ		*Carryforward flag
api	81.94% <ø> (ø)		Carriedforward from e2703e1
deploy-web	25.96% <ø> (ø)		Carriedforward from e2703e1
log-collector	?
notifications	87.94% <ø> (ø)		Carriedforward from e2703e1
provider-console	81.48% <ø> (ø)
provider-proxy	85.28% <ø> (ø)		Carriedforward from e2703e1

*This pull request uses carry forward flags. Click here to find out more.
see 10 files with indirect coverage changes

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f784754 and e2703e1.

📒 Files selected for processing (1)

• apps/provider-console/src/components/become-provider/ServerAccess.tsx (3 hunks)

🧰 Additional context used

📓 Path-based instructions (2)

**/*.{ts,tsx}

📄 CodeRabbit inference engine (.cursor/rules/general.mdc)

Never use type any or cast to type any. Always define the proper TypeScript types.

Files:

• apps/provider-console/src/components/become-provider/ServerAccess.tsx

**/*.{js,ts,tsx}

📄 CodeRabbit inference engine (.cursor/rules/general.mdc)

**/*.{js,ts,tsx}: Never use deprecated methods from libraries.
Don't add unnecessary comments to the code

Files:

• apps/provider-console/src/components/become-provider/ServerAccess.tsx

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: test-build

🔇 Additional comments (2)

apps/provider-console/src/components/become-provider/ServerAccess.tsx (2)

28-29: Good addition of explicit loading state.

The separation of isBalanceLoaded and hasEnoughBalance correctly distinguishes between three states: not loaded, loaded but insufficient, and loaded with sufficient balance. This prevents premature balance checks.

---

41-44: Correctly gates the balance popup on loading state.

The condition isBalanceLoaded && !hasEnoughBalance ensures the popup only displays when wallet data has loaded and the balance is actually insufficient, preventing premature popup triggers.