MetaMask / MetaMask/metamask-extension
Re-evaluate swallowing `persist`/`set` errors in the Persistence Manager
Open
INVALID-ISSUE-TEMPLATE
team-extension-platform
- Dominant language
- TypeScript
- Stars
- 13.2k
- Forks
- 5.6k
- Avg merge
- 2d 5h
- Merged PRs (30d)
- 451
Description
### What is this about?
See some workarounds applied in https://github.com/MetaMask/metamask-extension/pull/39643
that should be undone if we can figure out a good way to just let the error propagate instead of silently failing.
Contributor guide
Research direction
Start by reviewing the workaround in pull request 39643 and then locate the Persistence Manager code that handles persist and set errors. Determine how the errors are currently swallowed and what behavior is expected before removing the workaround; done means errors propagate instead of failing silently.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 28/100