[sahilc0]

Summary

• Fixes the loading SVG appearing twice on startup (slides left, new one enters from right)
• Root cause: two independent mechanisms both showing the Loading page — an inline render guard (line 172) and a navigate(Pages.Loading) call in a useEffect (line 112). The navigate call redundantly set direction: 'forward', triggering AnimatePresence to re-animate the already-visible Loading page.
• Replaced navigate(Pages.Loading) with a plain early return. The inline guard already renders Loading; the useEffect only needs to prevent premature navigation before wallet data loads.

Changes

• src/App.tsx — one line: if (!walletLoaded) return navigate(Pages.Loading) → if (!walletLoaded) return

Test plan

• Clear site data, reload on desktop — loading SVG should appear once, no slide animation
• Same test on mobile via HTTPS tunnel
• Returning user (has wallet, locked) — loading → unlock transition should work normally
• New user (no wallet) — loading → onboard transition should work normally

Summary by CodeRabbit

• Bug Fixes
  • Improved wallet initialization flow during app startup to prevent unnecessary redirect interruptions.

[cloudflare-workers-and-pages]

Deploying wallet-bitcoin with    Cloudflare Pages

Latest commit:	8bc6740
Status:	✅  Deploy successful!
Preview URL:	https://93302809.wallet-bitcoin.pages.dev
Branch Preview URL:	https://wt-fix-loading-double-flash.wallet-bitcoin.pages.dev

View logs

[cloudflare-workers-and-pages]

Deploying wallet-mutinynet with    Cloudflare Pages

Latest commit:	8bc6740
Status:	✅  Deploy successful!
Preview URL:	https://da081bdb.arkade-wallet.pages.dev
Branch Preview URL:	https://wt-fix-loading-double-flash.arkade-wallet.pages.dev

View logs

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: b9b4c253-52d3-48d9-a23a-30fabe0c2243

📥 Commits

Reviewing files that changed from the base of the PR and between ee6162d and 8bc6740.

📒 Files selected for processing (1)

• src/App.tsx

---

Walkthrough

Modified wallet-loading redirect logic in src/App.tsx: when walletLoaded is false, the effect now returns without navigation instead of redirecting to Pages.Loading, allowing subsequent checks for wallet.pubkey and initialized state to proceed.

Changes

Cohort / File(s)	Summary
Wallet Loading Redirect Logic src/App.tsx	Changed the conditional logic to return early without redirecting when wallet is not loaded, enabling downstream validation checks to execute in sequence rather than immediately routing to loading page.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

• pietro909
• bordalix

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title 'fix(ui): remove double loading animation on app startup' directly matches the main change: eliminating the double loading animation by removing the redundant navigation call in the useEffect, while keeping the inline rendering guard.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch wt-fix-loading-double-flash-20260309

---

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.}

[tiero]

tACK

[tiero]

This seems a nobrainer cc/ @Kukks