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.
Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.
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…
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
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
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
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
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
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
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
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
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
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
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
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
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
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)
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…
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
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
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
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
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.
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.
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.
✓ 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.
✓ 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.
✓ 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.
✓ 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.