EL
← All projectsElectrum

Electrum

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

BitcoinSoftware walletsNormal
Repository coverage

1153 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.

92security candidates261second-pass queue1007AI analyses
127commits · 30 days
262commits · 60 days
545commits · 180 days
1002commits · 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.

66/100 average clarity
220Strong · 80–100
549Adequate · 60–79
340Thin · 40–59
44Opaque · 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.
ThomasV137891162
SomberNight39330362063
f321x40325387070
Sander van Grieken10910101057
ghost4342517069
Felix25410072
user323062
ekzyis222053
accumulator613071
Roman Zeyde515053
Sasha Zykov515075
Aaron Fiore414068
Analysis record

Published AI watches

Last scanned 43 minutes ago

Low 29 AI analysisMessage 58 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10992 from f321x/dedup_is_seed_wizard

This commit simplifies the seed-validation logic in Electrum's Qt wallet wizard by replacing a local copy of the rules with a call to an existing shared validation method. The change appears to be a code-cleanup refactor that removes dupli…

Refactor of seed-validation logic in wallet creation wizardRemoval of duplicated seed-type checks in favor of centralized validationAdded assertion constraining seed type to 'electrum'
554daca6by ghost43+2−81 file
No security note in commit
Low 29 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10982 from spesmilo/wizard_2fa_seed_redirect

This commit changes Electrum's wallet creation wizard so that if a user tries to restore a normal 'standard' wallet but pastes a seed phrase that actually belongs to a two-factor authentication (2FA) wallet, the wizard automatically redire…

UX-level safety fix preventing wallet-type/seed mismatchState mutation: wizard_data['wallet_type'] changed from 'standard' to '2fa' based on seed typeNo cryptographic, network, or privilege changes
310c9dd0by Felix+102−133 files
No security note in commit
Informational 12 AI analysisMessage 68 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

qt: wizard: dedup seed validation rules

This commit is a small code cleanup in Electrum's wallet setup wizard. It removes a duplicate copy of the rules used to check whether a recovery seed is valid and instead reuses an existing method. There is no indication this fixes a secur…

89d0de06by f321x+2−81 file
No security note in commit
Informational 15 AI analysisMessage 70 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

tests: wizard: cover standard -> 2fa redirection

This commit only adds and reorganizes automated test code for Electrum's wallet creation wizard. It does not change any production wallet logic, cryptography, or network handling. The tests verify that when a user tries to restore a 'stand…

70b4d27cby f321x+68−91 file
No security note in commit
Informational 15 AI analysisMessage 78 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10988 from f321x/update_security_review_ci_model

This commit updates Electrum's internal CI (continuous integration) script that runs an automated security review using Anthropic's Claude Code tool. It changes the AI model version used for reviews from 'claude-opus-5' to 'claude-opus-5-5…

CI hardening: detects and reports AI model downgrades during automated security reviewNo changes to application code, cryptography, network protocol, or build artifactsNo privilege escalation, injection, or data-exfiltration vectors introduced by the diff
638fbba8by Felix+75−111 file
No security note in commit
Informational 12 AI analysisMessage 90 · Strong
EL ElectrumElectrum BitcoinSoftware wallets

ci: security review: warn if the model got downgraded

This commit updates Electrum's own CI security-review script. It adds detection and warnings when the automated code reviewer (Claude Code) silently falls back to a different AI model, for example after a content-policy refusal. It does no…

CI-only changeNo modification of wallet, crypto, networking, or build artifactsAdds detection for AI model fallback/downgrade in automated security review
ace2ca7aby f321x+74−101 file
No security note in commit
Low 44 AI analysisMessage 68 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10987 from spesmilo/fix_10986

This commit fixes a bug where Electrum failed to recognize certain already-signed Bitcoin transactions as complete. Specifically, for native SegWit inputs, Bitcoin Core can produce a finalized PSBT with a valid witness but an empty scriptS…

Logic error in transaction completeness detectionNative SegWit witness handling edge casePSBT interoperability issue with Bitcoin Core
bef80421by ghost43+11−02 files
No security note in commit
Low 35 AI analysisMessage 81 · Strong
EL ElectrumElectrum BitcoinSoftware wallets

transaction: treat native segwit input with non-empty witness as complete

This commit fixes a bug where Electrum incorrectly treated finalized Bitcoin transactions as incomplete. Some wallet software (like Bitcoin Core and Sparrow) creates native SegWit transactions that omit an empty placeholder field Electrum …

Logic error in transaction completeness detectionPotential denial of service / user funds stuck due to refusal to broadcast valid finalized transactionInteroperability failure with Bitcoin Core and Sparrow PSBT output
e9bb42e6by ThomasV+11−02 files
No security note in commit
Moderate 68 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10981 from spesmilo/do_not_sign_txin_with_witness

This change prevents Electrum from re-signing Bitcoin transaction inputs that already contain a witness (the data proving the input was authorized). The patch fixes a bug where Electrum could incorrectly sign a non-SegWit input as if it we…

Incorrect signature algorithm selection for non-SegWit inputs when witness data is presentPSBT handling edge case where pre-existing witness data influences signing pathPotential invalid signature production during transaction signing
9da4c342by ThomasV+10−21 file
No security note in commit
Informational 15 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10974 from SomberNight/202609_nostr_psbt_xpub

This commit only adds a long code comment explaining a design choice in Electrum's PSBT-over-Nostr plugin. It does not change any program behavior. The comment documents that the plugin derives each cosigner's Nostr secret key from their B…

Documentation-only changeExplicit design tradeoff disclosure: cosigners can derive each other's Nostr secret keys from shared xpubsNo functional code change
46408a78by ThomasV+13−01 file
No security note in commit
Low 42 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10980 from spesmilo/add_input_from_network_do_not_ignore_exceptions

This is a one-line bugfix in Electrum's transaction handling. Previously, the code accidentally swallowed (hid) almost all exceptions when fetching extra transaction data from the network, only re-raising errors that were both network-rela…

Exception swallowing bug fixedNetwork/transaction input enrichment logic changedSilent failure mode removed
bb9d3280by ghost43+1−11 file
No security note in commit
Moderate 60 AI analysisMessage 58 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10970 from f321x/fix_bolt11_bugs_2

This commit fixes several bugs in Electrum's handling of BOLT11 Lightning invoices. The most user-visible fixes are: stricter validation of invoice amounts (rejecting zero, negative, sub-millisatoshi, and leading-zero amounts), correct pad…

Stricter BOLT11 amount validation prevents acceptance of zero, negative, sub-millisatoshi, and leading-zero amountsFixed int_to_data5 padding bug that corrupted small timestamp values during invoice round-tripMalformed 'r' routing tags now raise exceptions instead of being silently skipped
4b92c0f2by ghost43+82−939 files
No security note in commit
Moderate 61 AI analysisMessage 80 · Strong
EL ElectrumElectrum BitcoinSoftware wallets

add_info_from_network: do not swallow exceptions

Electrum's transaction builder had a bug where it would ignore certain consistency errors when fetching extra transaction data from the network. A malicious or misbehaving server could supply a wrong input amount, and Electrum would silent…

Swallowed exception leading to use of attacker-controlled input valuePSBT input consistency check bypassedPotential malicious Electrum server influence on transaction signing
3f67b21bby ThomasV+1−11 file
Vendor flagged security relevance
Informational 15 AI analysisMessage 60 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

bolt11: remove tag arg from get_routing_info

This is a small code cleanup change. A function that reads routing hints from Lightning invoices no longer takes a tag argument because only one type of routing hint ('r') is now supported. All callers are updated accordingly. There is no …

e78fba28by f321x+12−127 files
No security note in commit
Moderate 62 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10940 from accumulator/fix_10937

This commit tightens how Electrum parses Bitcoin payment requests (BOLT11 lightning invoices and BIP21 URIs). It turns previously uncaught internal errors into proper validation failures, rejects malformed invoice fields that used to be si…

Stricter input validation for externally supplied BOLT11 invoices and BIP21 URIsPreviously uncaught exceptions (ValueError, UnicodeDecodeError, ecc errors) are now wrapped in domain-specific decode exceptionsMalformed fallback addresses are skipped rather than aborting or crashing
a57ef6e7by Felix+707−9610 files
No security note in commit
Low 40 AI analysisMessage 36 · Opaque
EL ElectrumElectrum BitcoinSoftware wallets

bolt11: followup #10940

This commit tightens how Electrum parses BOLT11 Lightning invoices. It now rejects duplicate 'n' (node pubkey) tags instead of silently keeping only the first one, and it rejects invoice timestamps that are negative or too far in the futur…

Stricter validation of invoice timestamp boundsDuplicate 'n' tag now raises an exception instead of being silently droppedFollow-up to prior PR #10940, indicating a recent area of security-sensitive review
3026fbdbby f321x+14−232 files
No security note in commit
Low 32 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10737 from sashazykov/hw-close-wallet-thread-leak

This commit fixes a shutdown bug in Electrum's hardware wallet support. Previously, if disconnecting from a hardware wallet failed, the cleanup thread could keep running, which could crash the application when closing. The fix stops the th…

Process abort/crash at shutdown due to leaked QThreadException during cleanup not handled, potentially causing abnormal terminationHardware wallet client close made best-effort with error logging
5b77ca46by Felix+9−22 files
No security note in commit
Informational 23 AI analysisMessage 58 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10958 from SomberNight/202609_deps

This commit is a routine dependency update for the Electrum Bitcoin wallet. It bumps versions of Python, OpenSSL, secp256k1, and various Python packages used in builds and runtime. The commit itself does not fix any known security bug in E…

Dependency version bumps for cryptographic libraries (OpenSSL, libsecp256k1, electrum-ecc)Routine maintenance commit with no explicit security claimHash-only changes in deterministic requirements files
8bd8c3fbby ghost43+57−5612 files
No security note in commit
Informational 21 AI analysisMessage 58 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10965 from SomberNight/202609_header_checkpoints

This commit only adds new Bitcoin block header checkpoints to Electrum's built-in chain data files. Checkpoints are hardcoded reference points that help the wallet verify it is following the real blockchain and not a fake one. Updating the…

Hardcoded blockchain checkpoints extended for all supported networksNo executable code or cryptographic logic changedNo bug fix, privilege change, or input validation change present
9e0b7e64by ghost43+276−04 files
No security note in commit
Informational 15 AI analysisMessage 58 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

update block header checkpoints

This commit simply refreshes Electrum's built-in list of Bitcoin blockchain checkpoints. Checkpoints are known-good block hashes that help the wallet verify it is following the real chain. The update adds newer checkpoints for mainnet, sig…

e11c579bby SomberNight+276−04 files
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-prioritytests: lnpeer: test_reject_payment_for_expired_invoiceby f321x · 7840df2e · Nov 27, 2025 · 1 fileMessage 65 · AdequateInformational 12Details
Commit message · f321x

tests: lnpeer: test_reject_payment_for_expired_invoice

Test that lnpeer is rejecting incoming htlcs for invoices that are
already expired.

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Provides an explanatory body✓ Mentions testing or verification
AI analysis · Informational 12/100

This commit only adds a new automated test to Electrum's Lightning networking code. The test checks that the software correctly refuses payments for expired invoices. It does not change any production code, so it cannot by itself introduce or fix a security issue in running software.

Lower-prioritytests: lnpeer: test_reject_invalid_min_final_cltv_deltaby f321x · da5f5990 · Nov 27, 2025 · 1 fileMessage 73 · AdequateInformational 12Details
Commit message · f321x

tests: lnpeer: test_reject_invalid_min_final_cltv_delta

Add `test_reject_invalid_min_final_cltv_delta` which is supposed to test
that the peer rejects incoming htlcs with final cltv delta differing
from what has been requested in the lightning invoice.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Provides detailed explanatory context✓ Mentions testing or verification
AI analysis · Informational 12/100

This commit only adds a new automated test to Electrum's Lightning code. The test checks that a payment is rejected when the sender uses a shorter safety window (called 'final cltv delta') than the receiver's invoice requested. There is no change to production code, so by itself this commit does not fix or introduce a security issue. It may be related to a prior or future fix, but this commit alone is just a test.

AI review queuedlnpeer/lnworker: refactor htlc_switchby f321x · 0f314d1d · Nov 27, 2025 · 11 filesMessage 58 · ThinModerate 59Details
Commit message · f321x

lnpeer/lnworker: refactor htlc_switch

refactor `htlc_switch` to new architecture to make it more robust
against partial settlement of htlc sets and increase maintainability.
Htlcs are now processed in two steps, first the htlcs are collected into
sets from the channels, and potentially failed on their own already.
Then a second loop iterates over the htlc sets and finalizes only on
whole sets.

# Conflicts:
# electrum/lnpeer.py

58/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Moderate 59/100

This is a large internal refactor of Electrum's Lightning code that handles incoming multi-part payments (MPP). The change restructures how individual HTLCs are collected into sets and how those sets are validated, forwarded, or settled. The commit message frames it as improving robustness and maintainability, not as a security fix. The diff shows new consistency checks for trampoline onions, stricter state transitions for payment sets, and a wallet database upgrade to migrate old state. There is no vendor statement that this fixes a known vulnerability, and no CVE or researcher attribution is present in the supplied materials.

Lower-priorityswaps: followup 10303by f321x · 3d0ba336 · Nov 27, 2025 · 1 fileMessage 51 · ThinLow 35Details
Commit message · f321x

swaps: followup 10303

small followup replacing tx height integers with const variables and
considering claim tx broadcast too if there is an unconfirmed parent
(height -1).

51/100 · ThinMessage clarity
✓ Subject identifies a change✓ Provides detailed explanatory context
AI analysis · Low 35/100

This is a small cleanup change in Electrum's submarine swap (atomic swap) code. It replaces hardcoded transaction height numbers with named constants and slightly adjusts when a claim transaction can be broadcast. The change allows a claim transaction to be considered 'already broadcast' even if its parent transaction is still unconfirmed. This is a follow-up to a previous change and appears aimed at preventing duplicate broadcasts or premature broadcasting, not at introducing a vulnerability.

Lower-priorityFix: For inner trampoline onions amt_to_forward can be larger than the htlc amountby f321x · 41d391a6 · Nov 27, 2025 · 2 filesMessage 100 · StrongModerate 61Details
Commit message · f321x

Fix: For inner trampoline onions amt_to_forward can be larger than the htlc amount

Add unittest to TestPeerForwarding which sends a multi trampoline
payment.

Wait another htlc_switch iteration in tests because trampolines might have different delays

100/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Mentions testing or verification
AI analysis · Moderate 61/100

This commit fixes a bug in Electrum's Lightning Network code where multi-trampoline payments could incorrectly fail. In normal payments, the amount specified inside the onion must match the HTLC amount. But for trampoline routing, an inner trampoline onion can legitimately request a larger amount than the current HTLC because the payment is split across multiple trampoline forwarders. The old code enforced the strict match check even for these inner trampoline onions, causing valid payments to be rejected with a 'final incorrect HTLC amount' error. The fix moves the check into a method that only applies to final (non-trampoline-forwarding) HTLCs and skips it for inner trampoline onions.

Lower-priorityqml: (trivial) QEConfig.formatMilliSats: conv unused fallback to assertby SomberNight · bc2bd485 · Nov 27, 2025 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · SomberNight

qml: (trivial) QEConfig.formatMilliSats: conv unused fallback to assert

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

This is a tiny code cleanup in Electrum's QML (mobile-style) user interface. A function that formats tiny Bitcoin amounts used to silently return '---' if it received an unexpected input type. The change removes that fallback and instead adds an assertion that will crash the program if the wrong type is ever passed. This is a defensive coding change, not a security fix, and it does not introduce a vulnerability.

Lower-priorityln: fix race when doing concurrent ln paymentsby f321x · 196cc33c · Nov 27, 2025 · 2 filesMessage 68 · AdequateLow 37Details
Commit message · f321x

ln: fix race when doing concurrent ln payments

There is a race when initiating multiple lightning payments concurrently
(e.g. when doing a reverse swap with prepayment + swap payment).
suggest_splits might overallocate
split amounts for a channel as the splitting of both invoice amounts runs
concurrently and before acutal htlcs that reduce the channels balance
have been added to the channel yet. This results in a "not
enough balance" PaymentFailure once we try to send the htlcs and the
other payment attempt already reduced the available balance of the
channel.

This fix takes a lock from splitting the amount until the htlcs are
put on the channel, so suggest_splits always acts on the correct channel
balance.

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

This commit fixes a timing bug in Electrum's Lightning payments. If a user started multiple Lightning payments at the same time, the wallet could plan routes using outdated channel balances, promising more money than was actually available. When the payments were then sent, one would fail with a 'not enough balance' error. The fix adds a lock so that balance checks and route creation happen one payment at a time, preventing the stale-balance problem.

Lower-priorityrun_electrum: proxy command line option implies proxy enabled (fixes #10324)by Sander van Grieken · 3a9e3848 · Nov 27, 2025 · 1 fileMessage 58 · ThinLow 49Details
Commit message · Sander van Grieken

run_electrum: proxy command line option implies proxy enabled (fixes #10324)

58/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Links an issue, advisory, or supporting reference! No meaningful explanatory body
AI analysis · Low 49/100

This commit fixes a bug where users who typed a proxy setting on the command line could believe their traffic was being routed through that proxy, but Electrum was not actually enabling proxy mode. The fix makes specifying a proxy automatically turn proxy mode on. If left unfixed, a user might leak network traffic outside their intended proxy, which matters for privacy but is not a direct remote exploit.

Lower-priorityqml: show lightning invoice amounts with msat precision, allow msat precision entry for no-amount lightning invoicesby Sander van Grieken · c519083b · Nov 27, 2025 · 4 filesMessage 50 · ThinInformational 18Details
Commit message · Sander van Grieken

qml: show lightning invoice amounts with msat precision, allow msat precision entry for no-amount lightning invoices

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

This commit changes the mobile/QML user interface of the Electrum Bitcoin wallet so that Lightning invoice amounts are shown and entered with millisecond precision (one-thousandth of a satoshi) instead of whole satoshis. It is a precision/usability improvement for the QML GUI, not a fix for a vulnerability. There is no indication in the commit that this resolves a security issue.

Lower-priorityqeconfig: fix btc amount regex, add msat regex property, add regex testsby Sander van Grieken · e137c888 · Nov 27, 2025 · 2 filesMessage 60 · AdequateLow 25Details
Commit message · Sander van Grieken

qeconfig: fix btc amount regex, add msat regex property, add regex tests

60/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Low 25/100

This commit tightens a Bitcoin amount input validation regex used in Electrum's QML (mobile/desktop) GUI. The old regex lacked start (^) and end ($) anchors, meaning it could match invalid strings that merely contained a valid-looking number somewhere inside. The fix anchors the pattern and makes the decimal fraction optional. A new variant for milli-satoshi precision is also added, along with unit tests. The practical security impact is limited because the regex is only one layer of validation and likely used for UI feedback rather than final payment authorization.

Lower-priorityqml: refactor invoice amount checks, msat precision for lightning. add qeconfig unit tests for conversion methods.by Sander van Grieken · 93c9dd9d · Nov 27, 2025 · 3 filesMessage 75 · AdequateLow 32Details
Commit message · Sander van Grieken

qml: refactor invoice amount checks, msat precision for lightning.
add qeconfig unit tests for conversion methods.

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

This commit fixes a precision bug in Electrum's mobile/QML interface when handling very small Lightning payments. Previously, amounts were rounded to whole satoshis too early, which could cause the app to reject valid Lightning invoices that require millisecond-precision amounts, or to miscalculate whether the user has enough balance. The change also cleans up duplicate balance-check code and adds unit tests.

Lower-prioritysimple_config: factor out self.decimal_point and self.get_decimal_point() in favor of self.BTC_AMOUNTS_DECIMAL_POINTby Sander van Grieken · 0f2a41e0 · Nov 27, 2025 · 8 filesMessage 50 · ThinInformational 15Details
Commit message · Sander van Grieken

simple_config: factor out self.decimal_point and self.get_decimal_point() in favor of self.BTC_AMOUNTS_DECIMAL_POINT

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

This is a routine internal code cleanup in Electrum's settings module. It replaces an old shortcut property (`self.decimal_point`) and its getter method with direct use of a configuration variable (`self.BTC_AMOUNTS_DECIMAL_POINT`). The behavior is unchanged; there is no security issue visible in the commit.

Lower-priorityqt: move pubkey_to_q_icon in qt.utilby f321x · 3b80795e · Nov 27, 2025 · 3 filesMessage 60 · AdequateInformational 15Details
Commit message · f321x

qt: move pubkey_to_q_icon in qt.util

Move the pubkey_to_q_icon utility function from qt.swap_dialog in
qt.util.

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

This commit is a simple code cleanup: it moves a small helper function that creates a colored icon from a public key from one file to a shared utility file. There is no change in behavior, no bug fix, and no security relevance.

Lower-priorityqt: expose swaps to address as Submarine Paymentsby f321x · 8a70fdcb · Nov 27, 2025 · 7 filesMessage 68 · AdequateLow 27Details
Commit message · f321x

qt: expose swaps to address as Submarine Payments

Exposes reverse submarine swaps to an external/specific address in the
TxEditor gui as "Submarine Payments". The user can enter a onchain
address in the Send Tab and then pay it from the lightning balance in
the send tab by enabling the Submarine Payments option in the TxEditor
dialog menu and switching to the Submarine Payment tab in the Tab bar.

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

This commit adds a new optional feature in Electrum's desktop (Qt) GUI called 'Submarine Payments'. It lets users pay a regular on-chain Bitcoin address directly from their Lightning balance by doing a reverse submarine swap through a third-party swap provider. The change is a feature addition, not a fix for a known vulnerability. It exposes existing swap logic in the transaction editor and adds UI controls, configuration, and cleanup handling. There is no direct evidence in the commit or supplied references that this resolves a security bug or introduces a new attack.

AI review queuedswaps: allow reverse swaps to external addressby f321x · a0455f83 · Nov 27, 2025 · 3 filesMessage 68 · AdequateLow 29Details
Commit message · f321x

swaps: allow reverse swaps to external address

Implement logic to claim a reverse swap funding output to any given
address. This allows to do onchain payments to external recipients
through a submarine swap.

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 · Low 29/100

This commit adds a feature to Electrum that lets users route reverse submarine swaps to an external Bitcoin address, effectively turning a Lightning payment into an on-chain payment to someone else. The change renames a database field, adds wallet-upgrade logic, and introduces a new code path that builds and broadcasts a claim transaction. There is no direct evidence in the commit of a security vulnerability, but the new code handles private keys, transaction signing, and broadcasting, so it is a sensitive area worth reviewing carefully.

Lower-prioritylnonion: add OnionFailureCode.from_int() helperby SomberNight · d7bc6170 · Nov 26, 2025 · 1 fileMessage 45 · ThinInformational 14Details
Commit message · SomberNight

lnonion: add OnionFailureCode.from_int() helper

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 14/100

This commit is a small internal code cleanup in Electrum's Lightning network code. It moves an existing conversion of raw integer failure codes into a named helper method, without changing behavior. There is no user-facing or security-relevant change.

Lower-prioritylnonion: work around immutability of OnionHopsDataSingleby Sander van Grieken · 30f2bc9e · Nov 25, 2025 · 1 fileMessage 50 · ThinLow 32Details
Commit message · Sander van Grieken

lnonion: work around immutability of OnionHopsDataSingle

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

This commit fixes a bug where the code tried to modify a data object that had been made read-only (immutable). The fix creates a new copy of the object with the needed changes instead of changing the original. This is a defensive repair to prevent runtime errors, not a clear-cut security patch, though immutability violations can sometimes hide or cause subtle bugs in payment routing.

Lower-priorityonion_message: move encrypt_onionmsg_tlv_hops_data() to lnonion as encrypt_hops_recipient_data() and add support payloads other than onionmsg_tlvby Sander van Grieken · 23d5ed87 · Nov 25, 2025 · 3 filesMessage 73 · AdequateInformational 15Details
Commit message · Sander van Grieken

onion_message: move encrypt_onionmsg_tlv_hops_data() to lnonion as encrypt_hops_recipient_data()
and add support payloads other than onionmsg_tlv

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

This commit is a straightforward internal code refactor. It moves a helper function that encrypts routing data from one file to another, renames it, and generalizes it so it can handle two slightly different data formats. There is no indication it fixes a security bug or introduces a vulnerability.

Lower-prioritylnonion: add helper properties minor refactoringby f321x · a6e103b6 · Nov 25, 2025 · 3 filesMessage 65 · AdequateInformational 12Details
Commit message · f321x

lnonion:
add helper properties
minor refactoring

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 12/100

This commit is a straightforward internal cleanup in Electrum's Lightning Network code. It adds helper properties to make reading onion packet fields more consistent and slightly reorders how payment data is assembled. There is no indication it fixes a security bug or introduces a vulnerability.

AI review queuedadd helper method: run_htlc_switch_iterationby f321x · c65ec6b2 · Nov 25, 2025 · 1 fileMessage 35 · OpaqueInformational 15Details
Commit message · f321x

add helper method: run_htlc_switch_iteration

no code modification, indent-only

35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100

This commit is a pure code restructure: it extracts the inner body of Electrum's Lightning HTLC switching loop into a new private helper method named `_run_htlc_switch_iteration`. The commit message explicitly says 'no code modification, indent-only', and the diff matches that claim. There are no functional changes, no security fixes, and no new behavior introduced.

Lower-priorityremove plugin payserver (moved to spesmilo/electrum-payserver)by ThomasV · d36b753c · Nov 25, 2025 · 6 filesMessage 50 · ThinInformational 15Details
Commit message · ThomasV

remove plugin payserver (moved to spesmilo/electrum-payserver)

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

This commit simply deletes the 'payserver' plugin from the Electrum repository and moves it to a separate project. There is no security fix or vulnerability patch here—just code relocation.

Lower-priorityfix tests: follow-up prevby SomberNight · 4fedab8b · Nov 21, 2025 · 1 fileMessage 55 · ThinInformational 15Details
Commit message · SomberNight

fix tests: follow-up prev

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 is a tiny follow-up patch to a test file only. It changes one expected value in a unit test from the integer 1 to the raw byte string b"\x01" so the test correctly matches what the actual code returns. It does not change any production code, network behavior, or wallet logic, and has no security relevance.

Lower-priorityonion_message: num_hops in blinded path is byte, not int.by Sander van Grieken · d92b0c74 · Nov 21, 2025 · 1 fileMessage 50 · ThinInformational 22Details
Commit message · Sander van Grieken

onion_message: num_hops in blinded path is byte, not int.

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

This commit fixes a data-format bug in Electrum's Lightning onion-message handling. The number of hops in a 'blinded path' was being stored as a plain integer, but the protocol expects a single byte. The patch changes it to a bytes object. A second change only improves an error message when no reply path can be created. The fix is small and appears to prevent interoperability or parsing problems rather than a direct theft-of-funds vulnerability.

Lower-prioritytrampoline: when sending payment, log "is_legacy" (e2e vs pay2legacy)by SomberNight · c288656d · Nov 21, 2025 · 2 filesMessage 50 · ThinInformational 15Details
Commit message · SomberNight

trampoline: when sending payment, log "is_legacy" (e2e vs pay2legacy)

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

This commit only adds extra information to log messages when Electrum sends a Lightning payment. It does not change any payment logic, permissions, or security behavior. One log line now includes the payment hash, and another debug log records whether the payment uses a legacy trampoline relay. There is no security issue visible in the diff.

Lower-prioritycli: payto: fix feerate parsingby f321x · 114c48e4 · Nov 19, 2025 · 2 filesMessage 76 · AdequateLow 45Details
Commit message · f321x

cli: payto: fix feerate parsing

Feerate is passed to `Commands._get_fee_policy()` as str which then
tried to multiply the string by 1000. Now it first casts the string to
`Decimal` and multiplies the decimal.

Fixes https://github.com/spesmilo/electrum/issues/10315

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 · Low 45/100

This commit fixes a bug in Electrum's command-line 'payto' command where a transaction fee rate typed in as text (a string) was being incorrectly handled. The old code tried to multiply the text string by 1000, which in Python repeats the string instead of doing math, producing a wildly wrong fee policy. The fix converts the string to a number (Decimal) first, then multiplies. This could cause users who set a custom fee rate via the CLI to get an absurdly large or malformed fee value, potentially overpaying fees or having transaction creation fail.