What the developer wrote
Authored by kdmukai
Remove dead code
What changed, and why it matters
This commit simply deletes an unused helper function that could generate a seed phrase from an image. There is no security fix or vulnerability here—it's routine cleanup of leftover code.
No action needed; this is a non-security code cleanup commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff removes generate_mnemonic_from_image() from src/seedsigner/helpers/mnemonic_generation.py. The function was already unused (the commit message and an inline comment both call it ‘dead code’) and its removal does not change any active code paths. No security behavior is altered.
Changed components
src/seedsigner/helpers/mnemonic_generation.pyInspect captured patch +0 / −11
diff --git a/src/seedsigner/helpers/mnemonic_generation.py b/src/seedsigner/helpers/mnemonic_generation.py
index b3e9068..444c3fe 100644
--- a/src/seedsigner/helpers/mnemonic_generation.py
+++ b/src/seedsigner/helpers/mnemonic_generation.py
@@ -110,14 +110,3 @@ def get_partial_final_word(coin_flips: str, wordlist_language_code: str = Settin
wordlist_index = int(binary_string, 2)
return Seed.get_wordlist(wordlist_language_code)[wordlist_index]
-
-
-
-# Note: This currently isn't being used since we're now chaining hashed bytes for the
-# image-based entropy and aren't just ingesting a single image.
-def generate_mnemonic_from_image(image, wordlist_language_code: str = SettingsConstants.WORDLIST_LANGUAGE__ENGLISH) -> list[str]:
- import hashlib
- hash = hashlib.sha256(image.tobytes())
-
- # Return as a list
- return bip39.mnemonic_from_bytes(hash.digest(), wordlist=Seed.get_wordlist(wordlist_language_code)).split()
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.