Skip to content

Enhances the wallet setup experience with a strong focus on security, accessibility, and developer ergonomics#549

Merged
jamespepper81 merged 12 commits into
mainfrom
dev
Feb 10, 2026
Merged

Enhances the wallet setup experience with a strong focus on security, accessibility, and developer ergonomics#549
jamespepper81 merged 12 commits into
mainfrom
dev

Conversation

@jamespepper81
Copy link
Copy Markdown
Contributor

This pull request enhances the wallet setup experience with a strong focus on security, accessibility, and developer ergonomics. The most significant changes include improved handling of test/fallback mnemonics, automatic clipboard clearing for sensitive data, expanded accessibility support, and developer-friendly debug logging. The AnimatedPressable component is also updated with richer accessibility props and improved documentation.

Wallet Setup Security & UX Improvements:

  • Introduced clear fallback test mnemonics (FALLBACK_MNEMONIC_12, FALLBACK_MNEMONIC_24) and warnings to users when a test mnemonic is used, ensuring users are aware not to use these for real funds. [1] [2]
  • Clipboard is now automatically cleared 60 seconds after copying the recovery phrase, with user alerts and cleanup to reduce the risk of sensitive data leakage. [1] [2]
  • Improved error handling and developer logging for wallet service loading and mnemonic generation, including more robust fallback logic and detailed logs in development mode without exposing sensitive data. [1] [2] [3]

Accessibility Enhancements:

  • Added accessibilityRole, accessibilityLabel, accessibilityHint, and accessibilityState to interactive elements (buttons, checkboxes, links) throughout the wallet setup screen for improved screen reader and keyboard navigation support. [1] [2] [3] [4] [5] [6]
  • Increased the size of checkboxes for better touch accessibility.

AnimatedPressable Component Improvements:

  • Extended AnimatedPressable to support accessibility props (accessibilityLabel, accessibilityRole, accessibilityHint), testID, and hitSlop, and added comprehensive JSDoc comments and usage example for better developer experience and testability. [1] [2] [3] [4]

These changes collectively make the wallet setup process safer for users, more accessible, and easier to maintain and debug for developers.

Copilot AI and others added 12 commits February 10, 2026 10:58
Co-authored-by: jamespepper81 <84083764+jamespepper81@users.noreply.github.com>
Co-authored-by: jamespepper81 <84083764+jamespepper81@users.noreply.github.com>
Co-authored-by: jamespepper81 <84083764+jamespepper81@users.noreply.github.com>
Co-authored-by: jamespepper81 <84083764+jamespepper81@users.noreply.github.com>
Co-authored-by: jamespepper81 <84083764+jamespepper81@users.noreply.github.com>
…sx-file

Code quality: Fix security and accessibility issues in wallet-setup.tsx
…ce console.log guards

Co-authored-by: jamespepper81 <84083764+jamespepper81@users.noreply.github.com>
…able

Co-authored-by: jamespepper81 <84083764+jamespepper81@users.noreply.github.com>
…nimatedpressable

Enhance AnimatedPressable: accessibility, performance, production safety
Removed hitSlop prop due to TypeScript error.
@jamespepper81 jamespepper81 merged commit bac4703 into main Feb 10, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants