CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

AI PR Analysis

🚫 Merge safe: false | 🟡 Risk: medium

Merge decision: A high-severity bugbot finding flags that selectWalletHomeOnboardingStepsEnabled may bypass the remote feature flag gate (the actual file shows the correct implementation, but the bugbot comment references a hardcoded true — this discrepancy needs resolution). Additionally, the Wallet view now conditionally hides AssetDetailsActions based on the new onboarding steps surface state, which is a behavioral regression risk for all users on the wallet home screen.

• Wallet home action buttons hidden for all eligible users: AssetDetailsActions (Buy/Swap/Send/Receive) is now conditionally rendered based on hideWalletMainActionsForPostOnboardingSteps state in Wallet/index.tsx. If the onboarding steps surface activates unexpectedly (e.g., due to feature flag or eligibility logic bugs), users lose access to core wallet actions.
• Feature flag selector ambiguity: Bugbot flagged selectWalletHomeOnboardingStepsEnabled as potentially hardcoded to true, bypassing remote gating. The actual file shows correct implementation, but the discrepancy between the bugbot finding and the code needs verification — if the flag was hardcoded at any point during development, it could affect users in production.
• AccountGroupBalance now has side effects: The component gained Redux dispatch logic (suppressing onboarding steps when balance > 0) and a callback prop. The useEffect that fires suppressWalletHomeOnboardingSteps('account_funded') runs on every render cycle where balance conditions are met — if hasBalanceFetched or balance selectors behave unexpectedly, this could permanently suppress the flow or fire repeatedly.
• State migration 132 is additive and safe: Migration correctly adds walletHomeOnboardingStepsEligible: false and walletHomeOnboardingSteps to existing state. No data loss risk.
• OnboardingSuccess prop change: backedUpSRP boolean prop replaced with successFlow enum in ManualBackupStep3. The old prop is removed — any other callers using backedUpSRP would break at compile time (TypeScript catches this).
• Placeholder locale strings: trade_subtitle contains placeholder text and notifications_subtitle duplicates fund_subtitle — users will see incorrect copy when the feature is enabled.
• Dimensions.get static capture: SLIDE_DISTANCE is computed once at module load, which will be wrong on foldable/split-screen devices.
• Test quality: Four assertions use toBeTruthy() instead of toBeOnTheScreen() per project guidelines.

View run

Codecov Report

❌ Patch coverage is 78.80795% with 96 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.50%. Comparing base (4797726) to head (b8a5fb5).
⚠️ Report is 1 commits behind head on main.

@@            Coverage Diff             @@
##             main   #28851      +/-   ##
==========================================
- Coverage   81.50%   81.50%   -0.01%     
==========================================
  Files        5328     5339      +11     
  Lines      141146   141577     +431     
  Branches    32157    32265     +108     
==========================================
+ Hits       115046   115393     +347     
- Misses      18239    18287      +48     
- Partials     7861     7897      +36     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🔍 Smart E2E Test Selection

• Selected E2E tags: SmokeAccounts, SmokeConfirmations, SmokeIdentity, SmokeNetworkAbstractions, SmokeNetworkExpansion, SmokeSwap, SmokeStake, SmokeWalletPlatform, SmokeMoney, SmokePerps, SmokeMultiChainAPI, SmokePredictions, SmokeSeedlessOnboarding, SmokeBrowser, SmokeSnaps
• Selected Performance tags: @PerformanceAccountList, @PerformanceOnboarding, @PerformanceLogin, @PerformanceSwaps, @PerformanceLaunch, @PerformanceAssetLoading, @PerformancePredict, @PerformancePreps
• Risk Level: high
• AI Confidence: %

View GitHub Actions results

Quality Gate passed

Issues
8 New issues
0 Accepted issues

Measures
0 Security Hotspots
85.6% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud