EL
← All projectsElectrum

Electrum

Long-running lightweight Bitcoin wallet for desktop and mobile platforms.

BitcoinSoftware walletsNormal
Repository coverage

935 commits in the local evidence base

Every captured commit receives deterministic security triage and a separate communication-quality score. Security candidates and broader second-pass signals receive full-patch Ollama analysis.

67security candidates208second-pass queue929AI analyses
61commits · 30 days
116commits · 60 days
465commits · 180 days
931commits · 365 days
Backfill bands
Aug 5 → Feb 6470 seen11 candidatesComplete
Feb 6 → Jun 6346 seen10 candidatesComplete
Jun 6 → Jul 653 seen2 candidatesComplete
Jul 6 → Aug 554 seen3 candidatesComplete
Commit communication

Does the history explain itself?

Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.

65/100 average clarity
194Strong · 80–100
415Adequate · 60–79
287Thin · 40–59
39Opaque · 0–39
2security candidates with opaque commit messaging
Read the scoring rubric →
Developer activity

Who is changing the project?

Public Git author strings; identities are not independently verified.

DeveloperCommitsCandidatesAnalyzedHigh riskMessage avg.
ThomasV73272156
SomberNight35226350063
f321x37121371070
Sander van Grieken991098057
user323062
Roman Zeyde515053
Sasha Zykov414076
Aaron Fiore414068
ghost43312068
Róbert Nagy111098
Ferdinando Ametrano111096
Felix302074
Analysis record

Published AI watches

Last scanned 53 minutes ago

Informational 15 AI analysisMessage 63 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10816 from accumulator/qml_auth_log_improvement

This commit is a minor logging cleanup in the mobile/QML authentication helper. It changes the wording and placement of debug/error log messages so they reveal the function name only after safely retrieving it, and avoids logging a raw tup…

8dddae68by ghost43+4−41 file
No security note in commit
Moderate 51 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10798 from SomberNight/202608_base43

This commit fixes a performance weakness in Electrum's handling of large Bitcoin transactions encoded in 'base43'. The base43 encoding and decoding functions run in quadratic time, meaning the work grows much faster than the input size. An…

Quadratic-time base43 encode/decode can be triggered on attacker-controlled inputDenial-of-service via CPU exhaustion / UI freeze on large transaction stringsInput-length cap added as a defensive guard, not a full algorithmic fix
0e9f1a0dby Felix+25−92 files
No security note in commit
Low 41 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10799 from SomberNight/202608_android_secure_window

This update tightens a privacy feature on Electrum's Android app that prevents the screen from being captured while sensitive Bitcoin private keys or seed phrases are visible. Before, the protection could be accidentally turned off when mu…

Adds visibility-gated secureWindow bindings to protect WIF keys and seed phrasesPrevents secureWindow from being cleared when multiple sensitive dialogs are stackedTargets Android screenshot/recents-thumbnail protection surface
4622390cby accumulator+17−05 files
Vendor flagged security relevance
Informational 15 AI analysisMessage 98 · Strong
EL ElectrumElectrum BitcoinSoftware wallets

docs: add Coldcard Mk3 seed-entropy security notice to plugin README (#10805)

This commit only adds a documentation warning to Electrum's Coldcard plugin README. It does not change any executable code, cryptographic logic, or wallet behavior. The warning tells users that Coldcard/Coinkite disclosed a firmware bug in…

Documentation-only changeReferences an external hardware-wallet firmware advisoryNo code, cryptography, or plugin behavior modified
2c2a40b6by Róbert Nagy+4−01 file
Vendor flagged security relevance
Low 48 AI analysisMessage 81 · Strong
EL ElectrumElectrum BitcoinSoftware wallets

qml/android: protect WIF keys from screenshots in more places

This commit tightens screen-shot protection for private Bitcoin wallet keys (WIF keys and seed phrases) in Electrum's Android/QML user interface. It also fixes a bug where one screen (AddressDetails) was leaving the 'secure window' flag pe…

Adds FLAG_SECURE-style screenshot protection to additional private-key dialogsFixes a bug where secureWindow was never unset in AddressDetailsUses QML Binding 'when' with restoreMode semantics to handle stacked secure dialogs safely
739cba5dby SomberNight+17−05 files
Vendor flagged security relevance
Moderate 60 AI analysisMessage 92 · Strong
EL ElectrumElectrum BitcoinSoftware wallets

lnpeer: chan_reest: ctn overflow: force-close instead of disconnect

This commit fixes a bug in Electrum's Lightning Network code. Previously, if a malicious or malfunctioning peer sent a channel re-establishment message with extremely large commitment numbers (over 2^48), the local program would hit an int…

Addition of explicit overflow check on untrusted peer-supplied commitment countersReplacement of assertion-failure/disconnect behavior with force-close on misbehaviorNew test cases for ctn overflow in both next_local_ctn and oldest_unrevoked_remote_ctn
b3e441f8by SomberNight+22−32 files
No security note in commit
Informational 18 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10791 from SomberNight/202608_wallet_sign_message2

This change moves the trimming of leading/trailing spaces in message-signing fields from the user-interface code into the shared wallet code. For the graphical interfaces (Qt and QML), spaces are still stripped automatically for convenienc…

Behavior change in message signing/verification input handlingCLI now preserves whitespace, which could affect signature validity for messages that intentionally contain leading or trailing whitespaceNo memory-safety, cryptographic, or authentication flaw visible in the diff
b1aa52d7by ghost43+27−185 files
No security note in commit
Informational 19 AI analysisMessage 96 · Strong
EL ElectrumElectrum BitcoinSoftware wallets

wallet: sign_message: strip whitespaces in GUIs, do not strip in CLI

This commit tidies up how Electrum handles extra spaces around Bitcoin addresses, messages, and signatures when signing or verifying messages. Previously, different parts of the program (desktop GUI, mobile-style GUI, command line) behaved…

Behavior normalization across GUI and CLI interfacesNo cryptographic primitive changedNo privilege boundary crossed
4c3064f5by SomberNight+27−185 files
No security note in commit
Informational 22 AI analysisMessage 50 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

wallet: decrypt_message: nicer error msg if pubkey is unrelated

This commit improves the error handling in Electrum's message decryption feature. Previously, if a user tried to decrypt a message using a public key that did not belong to their wallet, the code could proceed and likely fail with a confus…

Input validation added for public key before cryptographic operationUser-facing exception replaces potential internal failureDefensive check prevents use of unrelated key material
92e938f4by SomberNight+5−01 file
No security note in commit
Informational 16 AI analysisMessage 83 · Strong
EL ElectrumElectrum BitcoinSoftware wallets

qt: main_window: use TaskThread for each of sign/verify/encrypt/decrypt

This commit moves four message-crypto operations (sign, verify, encrypt, decrypt) in Electrum's Qt wallet window onto a background worker thread so they all behave the same way. It also lets the background thread surface user-facing errors…

Moved blocking wallet crypto calls off the GUI thread, reducing UI freeze / DoS surfaceRemoved local exception swallowing for encrypt_message; errors now propagate through the window's on_error handlerAdded defensive RuntimeError guard for deleted Qt widget in async callback
4a9a199fby SomberNight+28−151 file
No security note in commit
Low 29 AI analysisMessage 35 · Opaque
EL ElectrumElectrum BitcoinSoftware wallets

wallet: sign_message: force kwargs

This commit changes how Electrum's message-signing function is called so that callers must explicitly name each argument (address, message, password) rather than passing them by position. The main practical effect is to prevent accidental …

API hardening: keyword-only arguments prevent positional argument swapNo explicit security bug fix or vulnerability disclosure in commit message or diffNo changes to cryptographic logic, password handling, or message parsing
e1eb2a34by SomberNight+9−44 files
No security note in commit
Low 39 AI analysisMessage 45 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

wallet: sign_message: consolidate checks from UIs

This commit moves input-validation checks for signing, verifying, encrypting, and decrypting messages out of the user-interface code and into the shared wallet logic. It also adds type checks so that non-text inputs are rejected earlier. T…

Input validation moved from UI controllers into core wallet logicType checks added at the CLI command entry points for message-crypto operationsUserFacingException used consistently for address, wallet, script-type, and key-format errors
48893760by SomberNight+89−396 files
No security note in commit
Low 27 AI analysisMessage 50 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

wallet: decrypt_message: consolidate checks from UIs

This commit moves input-validation and wallet-type checks for message decryption out of the user-interface and command-line layers and into the core wallet code. It also removes a faster, separate decryption path for imported wallets so al…

Input validation consolidated into a single core methodWatching-only wallet check moved from UI to wallet layerMultisig wallet unsupported-operation check moved from UI to wallet layer
9a0cc0d6by SomberNight+25−173 files
No security note in commit
Informational 15 AI analysisMessage 50 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

wallet: refactor: add encrypt_message method, consolidated from UIs

This commit is a simple code cleanup: it moves the message-encryption logic from two separate places (the command-line tool and the Qt graphical interface) into a single shared method on the wallet class. The actual encryption behavior is …

No change to cryptographic primitives or parametersNo change to trust boundaries or input sourcesPure refactor consolidating duplicate code
5e9c8df0by SomberNight+26−183 files
No security note in commit
Informational 13 AI analysisMessage 50 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

wallet: refactor: add verify_message method, consolidated from UIs

This commit is a routine code cleanup: it moves message-signature verification out of three user-interface files into a single shared helper method in the wallet module. There is no security-relevant behavior change visible in the diff. Th…

No security-relevant functional changeRefactoring only: code deduplicationException handling broadened safely (ValueError covers binascii.Error and unicode errors)
ff5bbb04by SomberNight+26−304 files
No security note in commit
Informational 15 AI analysisMessage 60 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

qt: main_window: add type hints to sign/verify, encrypt/decrypt

This commit only adds Python type hints and renames some function parameters for clarity in the Electrum desktop wallet's sign/verify and encrypt/decrypt message dialogs. It does not change what the code actually does, how it processes use…

0906fc20by SomberNight+45−181 file
No security note in commit
Informational 18 AI analysisMessage 50 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

coldcard: fix get_soft_device_id() discarding its return value

This is a tiny one-line bug fix in Electrum's Coldcard hardware wallet plugin. A helper method that is supposed to fetch a software-based device identifier was calling the parent implementation but accidentally throwing away the result ins…

Missing return value causes method to always return NoneAffects hardware wallet device identification onlyNo change to signing, encryption, or authentication code
d7500508by Felipe Micaroni Lalli+1−11 file
No security note in commit
Informational 19 AI analysisMessage 96 · Strong
EL ElectrumElectrum BitcoinSoftware wallets

qml: strip whitespace from message before signing, as in qt gui

This commit fixes a minor consistency bug in Electrum's newer QML (mobile-style) user interface. When signing a message, the app now removes accidental spaces at the start or end of the message and address, just like the older desktop Qt i…

Behavioral inconsistency between GUI implementations could cause user confusion or failed verificationNo cryptographic weakness introduced; change is input normalizationNo memory-unsafe code, no privilege changes, no network changes
72507328by Ferdinando Ametrano+3−01 file
No security note in commit
Low 48 AI analysisMessage 85 · Strong
EL ElectrumElectrum BitcoinSoftware wallets

config: don't save "hidden wallet" paths in CURRENT_WALLET cv

This commit fixes a privacy leak in the Electrum wallet app. Previously, if a user opened a 'hidden wallet' (one whose filename starts with a dot, like a secret file), Electrum would remember it as the last-used wallet and could reopen it …

Privacy leak: hidden wallet filename persisted in config and could be auto-reopenedInformation disclosure: hidden wallet path exposed in recently-opened listBehavioral fix: setter silently drops sensitive value rather than persisting it
c43cf8e4by SomberNight+14−23 files
No security note in commit
Low 28 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

lnrouter: liquidity hints: add extra penalty if amt near cannot_send

This commit tweaks how Electrum's Lightning Network payment routing avoids channels where a previous payment attempt already failed. Previously, the router would treat a channel almost at its known failure limit almost the same as a channe…

Routing/liquidity-hint logic changed to penalize amounts close to a known cannot_send thresholdAdds defensive assertions on num_inflight_htlcs type and rangeSelf-described by author as a 'completely naive bandaid' / heuristic improvement
ee75ea31by SomberNight+12−21 file
No security note in commit
Repository ledger

Explore captured commits

Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.

Lower-priorityqt: channels_list: add tooltip to New Channel btnby f321x · 71bc9af5 · Dec 22, 2025 · 1 fileMessage 72 · AdequateInformational 15Details
Commit message · f321x

qt: channels_list: add tooltip to New Channel btn

Adds tooltips to the "New Channel" button so users understand why it is
disabled and what it does when it is enabled.

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Explains rationale or failure mode
AI analysis · Informational 15/100

This commit only adds helpful hover text (tooltips) to the 'New Channel' button in Electrum's user interface. It explains why the button is disabled when Lightning isn't available and what the button does when it is enabled. There is no security change.

Lower-prioritylnpeer: fix callback type hintby f321x · 14977e4c · Dec 19, 2025 · 1 fileMessage 60 · AdequateInformational 15Details
Commit message · f321x

lnpeer: fix callback type hint

asyncio.create_task expects a Coroutine, not all Awaitables are
Coroutines.

60/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 15/100

This commit changes only a Python type hint in the Lightning Network peer code. It replaces 'Awaitable' with 'Coroutine' in a function signature annotation to better match what asyncio.create_task expects. Type hints do not affect runtime behavior, so this cannot by itself introduce or fix a security vulnerability. It is a code-quality/correctness improvement for static analysis tools and developers.

AI review queuedlnpeer: fix callback exception handlerby f321x · ff615965 · Dec 19, 2025 · 1 fileMessage 68 · AdequateLow 25Details
Commit message · f321x

lnpeer: fix callback exception handler

The done_callback for the callback tasks in _run_htlc_switch_iteration
tried to access mpp_sets by key but they might already have been deleted
when the callback is called, causing an KeyError. Instead forward the
exceptions to the crash reporter so we get notice of them and they get
logged correctly.

```
20251219T131356.946565Z | ERROR | asyncio | Exception in callback Peer._run_htlc_switch_iteration.<locals>.<lambda>(<Task finishe.../util.py:1773>) at /home/user/code/electrum-fork/electrum/lnpeer.py:2907
handle: <Handle Peer._run_htlc_switch_iteration.<locals>.<lambda>(<Task finishe.../util.py:1773>) at /home/user/code/electrum-fork/electrum/lnpeer.py:2907 created at /usr/lib64/python3.14/asyncio/events.py:94>
source_traceback: Object created at (most recent call last):
File "/usr/lib64/python3.14/threading.py", line 1082, in _bootstrap_inner
self._context.run(self.run)
File "/home/user/code/electrum-fork/electrum/util.py", line 1145, in run_with_except_hook
run_original(*args2, **kwargs2)
File "/usr/lib64/python3.14/threading.py", line 1024, in run
self._target(*self._args, **self._kwargs)
File "/home/user/code/electrum-fork/electrum/util.py", line 1705, in run_event_loop
loop.run_until_complete(stopping_fut)
File "/usr/lib64/python3.14/asyncio/base_events.py", line 706, in run_until_complete
self.run_forever()
File "/usr/lib64/python3.14/asyncio/base_events.py", line 677, in run_forever
self._run_once()
File "/usr/lib64/python3.14/asyncio/base_events.py", line 2038, in _run_once
handle._run()
File "/usr/lib64/python3.14/asyncio/events.py", line 94, in _run
self._context.run(self._callback, *self._args)
Traceback (most recent call last):
File "/usr/lib64/python3.14/asyncio/events.py", line 94, in _run
self._context.run(self._callback, *self._args)
~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/user/code/electrum-fork/electrum/lnpeer.py", line 2909, in <lambda>
f"{self.lnworker.received_mpp_htlcs[pk]=}", exc_info=t.exception()) if t.exception() else None
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^
KeyError: '0000980000010001:1'
```

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
second-pass: broader security terminology
AI analysis · Low 25/100

This commit fixes a bug in Electrum's Lightning payment handling. When a background callback task finished, the code tried to read a payment record that might already have been deleted, causing a KeyError. The fix removes that unsafe lookup and instead sends any callback failure to the crash reporter. It is a robustness fix rather than a security vulnerability; the error was noisy and could hide real callback failures, but there is no evidence it could be exploited to steal funds or bypass protections.

Lower-prioritytests: lnpeer/onion_msg: rm dead codeby SomberNight · 08f101a7 · Dec 19, 2025 · 2 filesMessage 55 · ThinInformational 15Details
Commit message · SomberNight

tests: lnpeer/onion_msg: rm dead code

55/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 15/100

This commit only removes unused test helper code from two test files. It does not change any production wallet or network code, so it cannot affect real users' funds or security.

Lower-priorityinterface: remove prefix from donation addressesby f321x · c56c2796 · Dec 19, 2025 · 1 fileMessage 68 · AdequateInformational 21Details
Commit message · f321x

interface: remove prefix from donation addresses

Some servers seem to serve their donation address with `bitcoin:`
prefix, preventing them from getting donations. This removes the prefix.

E.g.:
```
262.63 | I | interface.[2ex.digitaleveryware.com:50002] | invalid donation address from server: 'bitcoin:bc1q0q6lcajak7r8h8c8luzp0zj35v0usqd4pdzqjg'
```

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 21/100

This commit fixes a small bug where Electrum servers that include 'bitcoin:' in front of their donation address were being rejected as invalid. The change strips that prefix before checking the address. It is a minor usability fix for donations, not a serious security issue, though it does slightly tighten input validation by ensuring the server response is a string.

Lower-prioritydaemon: reset CURRENT_WALLET if the wallet gets deletedby f321x · 91b3a4a5 · Dec 18, 2025 · 1 fileMessage 85 · StrongInformational 19Details
Commit message · f321x

daemon: reset CURRENT_WALLET if the wallet gets deleted

Set SimpleConfig.CURRENT_WALLET = None if the wallet gets deleted,
otherwise we try to open it on the next startup which will show an error
message in QML.

85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
AI analysis · Informational 19/100

This is a small bug-fix patch. When a user deletes a wallet, Electrum now clears its 'remembered last wallet' setting so the app doesn't try to reopen the deleted wallet on next launch and show an error message. It is a user-experience fix, not a security vulnerability.

Lower-priorityqml: fix incorrect index when deleting walletby f321x · 6bb8afeb · Dec 18, 2025 · 1 fileMessage 68 · AdequateInformational 16Details
Commit message · f321x

qml: fix incorrect index when deleting wallet

QEWalletListModel.remove_wallet was calling beginRemoveRows with i
instead of remove as index, causing it to not delete the wallet from the
list and the wallet list becoming broken after deleting a wallet.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 16/100

This is a one-line bug fix in Electrum's mobile/QML wallet interface. When a user deleted a wallet from the list, the wrong row index was passed to Qt's list-removal machinery, so the wallet stayed in the list and the displayed list became corrupted. The fix uses the correct index. There is no direct security impact: it is a UI consistency bug, not a vulnerability that lets an attacker steal funds or run code.

AI review queuedqml: OpenWalletDialog: load any wallet if password matchesby f321x · aee0f8fb · Dec 18, 2025 · 5 filesMessage 73 · AdequateLow 38Details
Commit message · f321x

qml: OpenWalletDialog: load any wallet if password matches

If the user has wallets with different passwords (non-unified pw) and
enters a password on startup that fails to unlock the recently used
wallet this change will automatically open any other wallet if there
is another wallet that can be unlocked with this password.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
signing or wallet pathsecond-pass: broader security terminologysecond-pass: security-sensitive path
AI analysis · Low 38/100

This commit changes the mobile/QML version of Electrum so that, when the app starts and the user's entered password does not unlock the most recently used wallet, the app will automatically try to open any other wallet on the device that can be unlocked with that same password. The goal is to help users who have multiple wallets with different passwords. However, it means a password intended for one wallet may silently open a different wallet, potentially showing balances, transaction history, and addresses the user did not expect. It also adds a stored config flag that remembers whether all wallets previously used the same password.

AI review queuedqml: LoadingWalletDialog: fix ressource leakby f321x · ba379b7d · Dec 18, 2025 · 1 fileMessage 68 · AdequateInformational 23Details
Commit message · f321x

qml: LoadingWalletDialog: fix ressource leak

When closing the OpenWalletDialog without unlocking a wallet the
LoadingWalletDialog wouldn't get properly cleaned up as the
LoadingWalletDialog.visible was never set true. This causes the
connections to accumulate and the callbacks won't get unregistered after
closing the LoadingWalletDialog again.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 23/100

This commit fixes a resource leak in Electrum's mobile-style QML user interface. If a user closed the wallet unlock dialog without actually unlocking a wallet, a hidden 'LoadingWalletDialog' would keep running in the background. Its internal signal connections would pile up and never be cleaned up, which could gradually slow down or destabilize the app. The fix explicitly destroys the dialog when it was never shown, so its cleanup code runs.

AI review queuedqml: show wallet list as root if no wallet is loadedby f321x · 70084750 · Dec 18, 2025 · 4 filesMessage 73 · AdequateInformational 15Details
Commit message · f321x

qml: show wallet list as root if no wallet is loaded

Shows Wallets.qml as root if no wallet is loaded and removes the logic
for no loaded wallet from the WalletMainView as WalletMainView won't be
shown anymore without a Daemon.currentWallet.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit is a user-interface cleanup for Electrum's mobile/QML app. It changes the app so that when no wallet is open, the user sees a wallet list screen instead of a 'no wallet loaded' placeholder inside the main wallet view. It also moves handling of incoming payment links (URI intents) from the wallet view to the top-level app so the link is not lost if a wallet isn't open yet. There is no security fix here.

Lower-priorityln: require LnFeatures.OPTION_CHANNEL_TYPE as bolts now mandate itby SomberNight · aab22a23 · Dec 18, 2025 · 3 filesMessage 81 · StrongInformational 24Details
Commit message · SomberNight

ln: require LnFeatures.OPTION_CHANNEL_TYPE as bolts now mandate it

This simplifies some code.

following https://github.com/lightning/bolts/commit/9d456b1c4a6c8e05a6b5b5edbc6c10f7b4b8e4de

81/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
AI analysis · Informational 24/100

This commit updates Electrum's Lightning code to follow a newer version of the Lightning Network specification (BOLTs). It makes the 'channel type' feature mandatory instead of optional when opening or accepting Lightning channels. The change removes old conditional code paths and simplifies the logic. There is no direct evidence in the commit that this fixes an active security vulnerability; it appears to be a protocol-compliance and code-simplification change.

Lower-prioritylnutil: rm remnants of old unsupported original ANCHOR_OUTPUTS optionby SomberNight · 85a45f9b · Dec 18, 2025 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · SomberNight

lnutil: rm remnants of old unsupported original ANCHOR_OUTPUTS option

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 15/100

This commit removes leftover code for an obsolete and never-supported Lightning Network feature called 'anchor outputs.' It is a cleanup change with no apparent security relevance.

Lower-prioritynetwork: disconnected servers: do not filter out bookmarked raw ":t"by SomberNight · cb3c286f · Dec 18, 2025 · 4 filesMessage 81 · StrongInformational 19Details
Commit message · SomberNight

network: disconnected servers: do not filter out bookmarked raw ":t"

In the GUI, when displaying the list of disconnected servers, we were
filtering to only ":s" servers. Instead now we also show ":t" servers
if they are bookmarked.

Also, if bookmarked, we also show disconnected .onion servers, even
when not using a Tor proxy.

fixes https://github.com/spesmilo/electrum/issues/10374

81/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
AI analysis · Informational 19/100

This is a small user-interface fix in the Electrum Bitcoin wallet. Previously, the list of 'disconnected servers' shown in the network settings only displayed secure ':s' servers and hid plain-text ':t' servers and .onion servers, even if the user had bookmarked them. The change makes bookmarked plain-text and .onion servers visible again so users can see and manage servers they deliberately saved. It does not change how the wallet actually connects to servers or any security-critical networking logic.

Lower-prioritynetwork: dedupe qt/qml get_disconnected_server_addrs codeby SomberNight · 21fa6114 · Dec 18, 2025 · 3 filesMessage 50 · ThinInformational 15Details
Commit message · SomberNight

network: dedupe qt/qml get_disconnected_server_addrs code

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 15/100

This commit is a simple internal code cleanup. It moves duplicated logic for listing disconnected servers from two user-interface files into a single shared helper method in the core networking module. There is no change in behavior visible to users and no security fix or vulnerability introduced.

AI review queuedqml: enforce single password on password changeby f321x · 02abc0e6 · Dec 18, 2025 · 1 fileMessage 68 · AdequateInformational 19Details
Commit message · f321x

qml: enforce single password on password change

If a qml user with non-uniform wallet passwords tries to change their wallet
password this will force them to change it to a password that is used by
at least one other wallet. This will guide them towards a single wallet
password and prevents the number of different passwords from increasing.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
signing or wallet pathsecond-pass: broader security terminologysecond-pass: security-sensitive path
AI analysis · Informational 19/100

This commit changes the mobile-style QML wallet settings screen so that when a user tries to change their wallet password, the app encourages (and on Android enforces) using the same password already used by another wallet. It is a user-experience/policy change, not a security fix or vulnerability. There is no evidence of a security bug being patched.

Lower-priorityqml: QERequestDetails process request update via event loop. This allows backend to process all callbacks before we start querying the payment databaseby Sander van Grieken · 0d812500 · Dec 18, 2025 · 1 fileMessage 81 · StrongLow 31Details
Commit message · Sander van Grieken

qml: QERequestDetails process request update via event loop.
This allows backend to process all callbacks before we start querying the payment database

should fix #10116

81/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
AI analysis · Low 31/100

This commit changes how the Electrum mobile/QML wallet listens for payment request status updates. It switches from a direct callback to one processed through the Qt event loop, so the backend can finish its own bookkeeping before the user interface reads the payment database. The stated goal is to fix a reported bug (#10116), likely a race condition where the UI shows stale or inconsistent payment state. There is no direct evidence in the commit that this is a security vulnerability, but race conditions between payment status and database queries can in principle lead to incorrect UI state or user decisions.

AI review queuedqml: disable 'Create Wallet' before first unlockby f321x · 378a9e61 · Dec 18, 2025 · 3 filesMessage 80 · StrongLow 30Details
Commit message · f321x

qml: disable 'Create Wallet' before first unlock

If the user has not unlocked any wallet yet and tries to create a new
wallet in the overview a dialog will prompt them to first unlock an
existing wallet in order to be able to create a new wallet.

This ensures they remember at least one password so they can complete
the wizard. The wizard will ask them for an existing password later and
it would be annoying for the user to go through all steps (writing down
the seed etc.) only to find out they need a password they don't
remember. This way they can reinstall the app right before going through
the wizard.

80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
signing or wallet pathsecond-pass: broader security terminologysecond-pass: security-sensitive path
AI analysis · Low 30/100

This commit changes the Electrum mobile/QML wallet interface so that users cannot create a new wallet until they have first unlocked an existing wallet. The goal is to prevent a frustrating situation where a user goes through the entire new-wallet setup (for example, writing down a recovery seed) and only afterward discovers they must enter an existing wallet password they have forgotten. The change also fixes a related state issue where the single-password setting was not being updated after a password change. There is no direct evidence in the commit that this fixes an active security vulnerability; it reads as a usability and defensive hardening improvement.

Lower-priorityqt: don't store python tuple in a qt QVariant.by Sander van Grieken · f2e6243e · Dec 18, 2025 · 1 fileMessage 76 · AdequateInformational 21Details
Commit message · Sander van Grieken

qt: don't store python tuple in a qt QVariant.

This is an educated guess in fixing #10335, Qt can be very picky w.r.t. wrapping types,
and we have additional complexity in the PyQt/SIP layer.

as the address index is only used to facilitate sorting, convert address index tuple
to a sortable string instead.

This assumes uniformity in the length of address indexes.

76/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
AI analysis · Informational 21/100

This commit fixes a crash or misbehavior in Electrum's Qt address list. The developer suspects that storing a Python tuple inside a Qt data container (QVariant) was causing problems, so they converted the address index tuple into a plain sortable string before storing it. There is no direct evidence in the commit that this is a security vulnerability; it appears to be a stability/bug-fix change.

Lower-priorityqml: store current wallet when switching to already open walletby Sander van Grieken · 7d5d5197 · Dec 18, 2025 · 1 fileMessage 50 · ThinInformational 18Details
Commit message · Sander van Grieken

qml: store current wallet when switching to already open wallet

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 18/100

This is a small bugfix in Electrum's background service (daemon). When a user switched to a wallet that was already open, the daemon forgot to update its record of which wallet is currently active. The patch makes it remember the active wallet in that case. There is no direct evidence this is a security vulnerability, but it could contribute to confused state or unexpected behavior in the wallet user interface.

Lower-priorityqml: allow manual editing of fee/feerate also improve warning box styling consistency across finalizers, add CPFP new feerate > old feerate check, add relayfee checks for CPFP, DSCancel, proper warning string for no dynamic fee estimatesby Sander van Grieken · 7ef605ee · Dec 18, 2025 · 9 filesMessage 73 · AdequateLow 32Details
Commit message · Sander van Grieken

qml: allow manual editing of fee/feerate
also improve warning box styling consistency across finalizers,
add CPFP new feerate > old feerate check,
add relayfee checks for CPFP, DSCancel,
proper warning string for no dynamic fee estimates

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Low 32/100

This commit adds a new 'Manual' fee mode to Electrum's QML (mobile-style) GUI, letting users type in an exact fee or fee rate. It also tightens safety checks for fee-bumping actions (CPFP, RBF bump, and RBF cancel) by warning when the new fee is too low to be relayed by the network or, in the case of CPFP, lower than the parent transaction's fee rate. The changes are mostly defensive UI hardening rather than a fix for an active vulnerability.

Lower-priorityqml: add FeePicker manual fee/feerate input validatorsby Sander van Grieken · fffcf4a9 · Dec 18, 2025 · 1 fileMessage 50 · ThinInformational 23Details
Commit message · Sander van Grieken

qml: add FeePicker manual fee/feerate input validators

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 23/100

This commit adds input validators to two manual fee input fields in Electrum's QML (mobile-style) user interface. One field now only accepts whole numbers, and the other only accepts numbers with at most one decimal place. This is a defensive hardening change that reduces the chance a user can type malformed or unexpected values into fee fields, which could previously lead to confusing errors or invalid transactions.

Lower-priorityqml: don't catch-all when making tx for sweep finalizerby Sander van Grieken · 4cab0ced · Dec 18, 2025 · 1 fileMessage 50 · ThinInformational 22Details
Commit message · Sander van Grieken

qml: don't catch-all when making tx for sweep finalizer

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 22/100

This commit tightens error handling in Electrum's mobile/QML wallet when creating a transaction to sweep funds. Previously, the code caught every possible exception and only logged the error, which could hide unexpected failures and leave the user with a generic or misleading warning. Now it only catches two specific, expected errors (no fee estimates and insufficient funds) and lets other errors surface normally. This is a defensive hardening change, not a fix for a known active exploit.

Lower-priorityqml: improve HelpDialog stylingby Sander van Grieken · 72daff3a · Dec 17, 2025 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · Sander van Grieken

qml: improve HelpDialog styling

remove misalignment between pane and dialog, add bottom padding

50/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides an explanatory body
AI analysis · Informational 15/100

This commit is a purely cosmetic update to a help dialog in Electrum's mobile-style QML user interface. It adjusts spacing, alignment, background colors, and padding so the dialog looks better. There is no security relevance.

Lower-priorityqml: fix FeePolicy regression ref 840243e0290e1550927d7d91b8c62dc00e40dedeby Sander van Grieken · 5a49161a · Dec 17, 2025 · 1 fileMessage 75 · AdequateLow 40Details
Commit message · Sander van Grieken

qml: fix FeePolicy regression
ref 840243e0290e1550927d7d91b8c62dc00e40dede

75/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification
AI analysis · Low 40/100

This commit fixes a regression in Electrum's QML (mobile/lightweight) user interface where creating a 'sweep' transaction accidentally ignored the user's chosen fee policy and passed no fee settings at all. The fix restores the intended behavior so the transaction respects whatever fee policy the user selected. The actual security impact is moderate and mostly user-facing: users might have gotten unexpectedly slow confirmations or unexpectedly high fees, rather than a direct theft-of-funds bug.

AI review queuedqml: enforce use of existing password for wallet creationby f321x · 3b028b06 · Dec 17, 2025 · 7 filesMessage 73 · AdequateInformational 19Details
Commit message · f321x

qml: enforce use of existing password for wallet creation

When creating a new wallet in a Electrum instance with existing wallets
this change forces the user to reuse a password of any existing wallet
if `SimpleConfig.WALLET_USE_SINGLE_PASSWORD` is True.
This prevents the amount of different passwords from increasing and
guides the user towards a single wallet password (the intended default).

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
signing or wallet pathsecond-pass: broader security terminologysecond-pass: security-sensitive path
AI analysis · Informational 19/100

This commit is a hardening and usability change for Electrum's mobile/QML wallet creation wizard. When the app is configured to prefer a single password across wallets, it now forces the user to reuse an existing wallet password when creating a new wallet, instead of allowing a new password. It also renames an internal configuration flag and exposes helper functions to check how many wallets a password can unlock. There is no direct security vulnerability being patched; it is a defensive measure to reduce password fragmentation.