0xMiden / 0xMiden/protocol

Rename `seed` to `id_salt` to avoid confusion with wallet **seed** phrases

Offen
#1,992 5 Kommentare 2 Reaktionen 1 zugewiesene Person Beansprucht von @Peponks9 Auf GitHub ansehen
Vorherrschende Sprache
Rust
Sterne
132
Forks
167
Ø Merge
1 T. 23 Std.
Gemergte PRs (30 T.)
110

Beschreibung

The term `seed` in Miden refers to a value used during account ID generation. This is fundamentally different from how "seed" is used in Ethereum/Bitcoin, where it refers to BIP39 mnemonic phrases for hierarchical key derivation and wallet recovery.

This terminology overlap is confusing for developers coming from the Ethereum ecosystem.

I (re-)realized this when reviewing https://github.com/0xMiden/miden-base/pull/1805/, and there the `AccountDataExport` actually includes "seed", which is counterintuitive.

### Proposal

Rename `seed` -> `account_salt`

Alternatives:
- `id_salt` - shorter but less explicit, might also be confusing
- `account_id_salt` (my second favorite)
- `account_genesis_salt` - emphasizes one-time use, but maybe too verbose?

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

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