MetaMask / MetaMask/metamask-extension

[State Sync Redesign 2.4] Replace `this.memStore` with direct messenger subscriptions

Open
#40,055 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

After Issues 2.1, 2.2, 2.3, and Epic 1 Issue 1.3, `this.memStore` has no remaining consumers. Remove it.

## Steps

1. Verify no remaining `memStore` consumers (all migrated in Issues 2.1-2.3 and Epic 1)
2. Remove `this.memStore` construction and the ~68-line controller list in `updateStructure`
3. Remove `ComposableObservableStore` import (if `this.store` also removed, see Issue 2.5)

## Files

- `app/scripts/metamask-controller.js`

## Risk

Low — all consumers already migrated.

## Dependencies

Issues 2.1, 2.2, 2.3, and Epic 1 Issue 1.3

## Context

Part of [State Sync Redesign — Epic 2: Background cleanup].

Contributor guide

Open the contributing guide

Research direction

Start in app/scripts/metamask-controller.js by verifying that Issues 2.1–2.3 and Epic 1 Issue 1.3 left no memStore consumers. Remove the this.memStore construction and the approximately 68-line controller list in updateStructure, then remove the ComposableObservableStore import if it is no longer needed; check Issue 2.5 if this.store is also removed.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
backend
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.