chore: add Sentry snapshot with masked data

[tommasini]

Description

This PR introduces snapshot with masked data for metamask mobile.
Here is an example of masked data on sentry

{"accounts": {"reloadAccounts": false}, "alert": {"autodismiss": null, "content": null, "data": null, "isVisible": false}, "bookmarks": [], "browser": {"activeTab": null, "favicons": [], "history": [], "tabs": [], "visitedDappsByHostname": {}, "whitelist": []}, "collectibles": {"favorites": {}, "isNftFetchingProgress": false}, "engine": {"backgroundState": {"AccountTrackerController": [Object], "AccountsController": [Object], "AddressBookController": [Object], "ApprovalController": [Object], "AssetsContractController": [Object], "CurrencyRateController": [Object], "GasFeeController": [Object], "KeyringController": [Object], "LoggingController": [Object], "NetworkController": [Object], "NftController": [Object], "NftDetectionController": "object", "PPOMController": [Object], "PermissionController": [Object], "PhishingController": [Object], "PreferencesController": [Object], "SmartTransactionsController": [Object], "SnapController": [Object], "SubjectMetadataController": [Object], "SwapsController": [Object], "TokenBalancesController": "object", "TokenDetectionController": [Object], "TokenListController": [Object], "TokenRatesController": [Object], "TokensController": [Object], "TransactionController": [Object]}}, "experimentalSettings": {"securityAlertsEnabled": true}, "fiatOrders": "object", "infuraAvailability": {"isBlocked": false}, "inpageProvider": {"networkId": "1"}, "legalNotices": {"newPrivacyPolicyToastClickedOrClosed": true, "newPrivacyPolicyToastShownDate": null}, "modals": {"collectibleContractModalVisible": false, "dappTransactionModalVisible": false, "networkModalVisible": false, "receiveAsset": undefined, "receiveModalVisible": false, "shouldNetworkSwitchPopToWallet": true, "signMessageModalVisible": true}, "navigation": {"currentBottomNavRoute": "Wallet", "currentRoute": "Login"}, "networkOnboarded": {"networkOnboardedState": {}, "networkState": {"nativeToken": "", "networkType": "", "networkUrl": "", "showNetworkOnboarding": false}, "switchedNetwork": {"networkStatus": false, "networkUrl": ""}}, "notification": {"notification": {"notificationsSettings": [Object]}, "notifications": []}, "onboarding": {"events": []}, "privacy": {"approvedHosts": {}, "revealSRPTimestamps": []}, "rpcEvents": {"signingEvent": {"eventStage": "idle", "rpcName": ""}}, "sdk": {"approvedHosts": {}, "connections": {}, "dappConnections": {}, "wc2Metadata": undefined}, "security": {"allowLoginWithRememberMe": false, "automaticSecurityChecksEnabled": false, "dataCollectionForMarketing": null, "hasUserSelectedAutomaticSecurityCheckOption": false, "isAutomaticSecurityChecksModalOpen": false, "isNFTAutoDetectionModalViewed": false}, "settings": {"basicFunctionalityEnabled": true, "hideZeroBalanceTokens": false, "lockTime": 30000, "primaryCurrency": "ETH", "searchEngine": "DuckDuckGo", "useBlockieIcon": true}, "signatureRequest": "object", "smartTransactions": {"optInModalAppVersionSeen": null}, "swaps": "object", "transaction": "object", "transactionMetrics": "object", "user": {"ambiguousAddressEntries": "object", "appTheme": "os", "backUpSeedphraseVisible": false, "gasEducationCarouselSeen": false, "initialScreen": "", "isAuthChecked": false, "loadingMsg": "", "loadingSet": false, "passwordSet": true, "protectWalletModalVisible": false, "seedphraseBackedUp": true, "userLoggedIn": true}, "wizard": {"step": 1}}

Related issues

Fixes: https://github.com/MetaMask/mobile-planning/issues/1843

Manual testing steps

1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I’ve followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

[github-actions]

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.

[tommasini]

The failed quality gate do not seem accurate maskKey === undefined it's needed and not redundant!

[NicolasMassart]

The failed quality gate do not seem accurate maskKey === undefined it's needed and not redundant!

The maskKey === undefined check might not be necessary if you want to handle it in the same way as false. So if you use maskKey == false both false and undefined values will be covered. At least that's my understanding.

[tommasini]

That would mean that maskKey == false would cover for both? It's not what it seems, but most likely !maskKey would cover for both, but since there is an else scenario, Im not sure if we should do that, we are not accounting for key that are null or empty strings or 0, those values would fall into the else and throw an error!

I bring that method from extension

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 66.66667% with 7 lines in your changes missing coverage. Please review.

Project coverage is 49.68%. Comparing base (ea14ef7) to head (bb06e5c).
Report is 29 commits behind head on main.

Files	Patch %	Lines
app/util/sentry/utils.js	66.66%	7 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10241      +/-   ##
==========================================
+ Coverage   49.50%   49.68%   +0.17%     
==========================================
  Files        1438     1450      +12     
  Lines       34674    34909     +235     
  Branches     3853     3950      +97     
==========================================
+ Hits        17166    17344     +178     
- Misses      16426    16463      +37     
- Partials     1082     1102      +20     

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

[tommasini]

@jpcloureiro Thanks for the review!!!! I believe I addressed/answered everything!

[jpcloureiro]

lgtm

[NicolasMassart]

Looks good to me.

[NicolasMassart]

Looks good to me.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
1 Accepted issue

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

See analysis details on SonarCloud