MetaMask / MetaMask/core

Rename Controller metadata `anonymous` property

Offen
#643 1 Kommentar 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
team-assets team-wallet-framework wf-controller-tech-debt
Vorherrschende Sprache
TypeScript
Sterne
413
Forks
308
Ø Merge
1 T. 4 Std.
Gemergte PRs (30 T.)
253

Beschreibung

The `anonymous` property of the Base Controller v2 metadata state is meant to indicate whether the state is already anonymized or not, or if it's a function, it's meant as an anonymizing function.

This property is often misinterpreted to mean the _opposite_ of what it's intended to mean though. People have thought `anonymous: true` means that the data _should be anonymized_, rather than indicating that it has already been anonymized.

To clarify the meaning, this should be renamed to something easier to understand. Maybe `anonymized`?

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne damit, die Metadatendefinition von Base Controller v2 und jede Verwendung der Eigenschaft `anonymous` im gesamten TypeScript-Monorepo zu finden. Verfolge, wie der Boolean oder die Funktion interpretiert wird, bevor du die Verweise umbenennst; fertig ist es, wenn die Metadaten durchgängig den klareren Namen verwenden, ohne die bestehende Anonymisierungssemantik zu ändern.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
typescript
Bereich
api
Issue-Typ
Refactoring
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
38/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.