fix: walletconnect clean up session

[jaymengxy]

Related Issue

Closes #1102 #1054 #1092

Summary of Changes

Need Regression Testing

• Yes
• No

Risk Assessment

• Low
• Medium
• High

Additional Notes

Screenshots (if applicable)

[github-actions]

PR Summary

Enabled previously commented-out WalletConnect session cleanup functionality and improved profile fragment stability by adding fragment lifecycle checks and safer activity access. The changes ensure proper session management in WalletConnect and prevent potential crashes in the profile UI.

Changes

File	Summary
app/src/main/java/com/flowfoundation/wallet/manager/walletconnect/WalletConnect.kt	Uncommented and enabled the cleanupActiveSessions() functionality that runs before pairing. This includes error handling and a 500ms delay to ensure proper cleanup completion.
app/src/main/java/com/flowfoundation/wallet/page/profile/presenter/ProfileFragmentPresenter.kt	Added fragment attachment check (isAdded) to prevent UI updates on detached fragments. Replaced requireActivity() with safer activity access when checking night mode theme, reducing potential crash scenarios.

---

autogenerated by presubmit.ai

[github-actions]

🚨 Pull request needs attention.

Review Summary

Commits Considered (1)

• 30eab70: fix: walletconnect clean up session

Files Processed (2)

• app/src/main/java/com/flowfoundation/wallet/manager/walletconnect/WalletConnect.kt (1 hunk)
• app/src/main/java/com/flowfoundation/wallet/page/profile/presenter/ProfileFragmentPresenter.kt (2 hunks)

Actionable Comments (1)

• app/src/main/java/com/flowfoundation/wallet/page/profile/presenter/ProfileFragmentPresenter.kt [165-167]

  possible bug: "Potential null pointer exception when accessing activity"

Skipped Comments (1)

• app/src/main/java/com/flowfoundation/wallet/manager/walletconnect/WalletConnect.kt [116-117]

  enhancement: "Consider making delay duration configurable"