Skip to content
This repository was archived by the owner on Jun 1, 2026. It is now read-only.

Add gem_keystore crate with BIP39 recovery phrase suggestions#1154

Closed
0xh3rman wants to merge 1 commit into
mainfrom
recovery-phrase-suggest
Closed

Add gem_keystore crate with BIP39 recovery phrase suggestions#1154
0xh3rman wants to merge 1 commit into
mainfrom
recovery-phrase-suggest

Conversation

@0xh3rman
Copy link
Copy Markdown
Collaborator

Introduce gem_keystore as the foundation crate for keystore and key management work. Initial surface is a Mnemonic helper that exposes BIP39 English word suggestions, wired through gemstone via UniFFI as suggest_recovery_phrase_words.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a new gem_keystore crate and a keystore module within the gemstone package to provide mnemonic word suggestions based on the BIP39 standard. The implementation includes a suggest_recovery_phrase_words function exported via UniFFI for cross-platform support. Feedback indicates that the suggestion logic should normalize input to lowercase to ensure case-insensitive matching against the wordlist. Additionally, there is a concern regarding performance overhead when returning the full 2048-word list for empty prefixes during FFI serialization, suggesting a limit should be applied.

Comment thread crates/gem_keystore/src/mnemonic.rs
Introduce gem_keystore as the foundation crate for keystore and key
management work. Initial surface is a Mnemonic helper that exposes
BIP39 English word suggestions, wired through gemstone via UniFFI as
suggest_recovery_phrase_words.
@0xh3rman 0xh3rman force-pushed the recovery-phrase-suggest branch from 50d5711 to ad6e6e8 Compare May 26, 2026 13:03
@0xh3rman 0xh3rman marked this pull request as ready for review May 26, 2026 13:37
@0xh3rman 0xh3rman closed this May 28, 2026
@0xh3rman
Copy link
Copy Markdown
Collaborator Author

will be included in full keystore pr

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant