MetaMask / MetaMask/metamask-extension

Re-evaluate swallowing `persist`/`set` errors in the Persistence Manager

Open
#39,674 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.