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
133commits · 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-prioritylnpeer: simplify channel_type, as it is now "assumed"by SomberNight · efa4d06e · Apr 30, 2026 · 1 fileMessage 50 · ThinLow 26Details
Commit message · SomberNight

lnpeer: simplify channel_type, as it is now "assumed"

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

This commit tightens how Electrum's Lightning code handles channel setup messages. Previously, the code tolerated missing optional fields (channel_type) in some situations; now it requires them to be present and rejects the connection if they are missing. This is a defensive hardening change that reduces the chance of a peer tricking the wallet into opening a channel with unexpected or unsafe settings.

Lower-prioritylnpeer: add property for "config"by SomberNight · be2096f8 · Apr 30, 2026 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · SomberNight

lnpeer: add property for "config"

instead of "self.network.config" or "self.lnworker.config" (mixed),
use "self.config", which points to "self.lnworker.config"

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

This commit is a small internal cleanup in Electrum's Lightning networking code. It introduces a single `self.config` shortcut so the code always reads settings from the same place instead of sometimes using `self.network.config` and sometimes `self.lnworker.config`. There is no user-facing change and no security fix.

Lower-priorityLnFeatures: rename OPTION_ANCHORS_ZERO_FEE_HTLC to OPTION_ANCHORSby SomberNight · 4357ef2f · Apr 30, 2026 · 5 filesMessage 65 · AdequateInformational 15Details
Commit message · SomberNight

LnFeatures: rename OPTION_ANCHORS_ZERO_FEE_HTLC to OPTION_ANCHORS

- to follow the same change in the BOLTS
- both for LnFeatures and for ChannelType

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

This commit is a simple renaming of a Lightning Network feature flag to match a recent change in the official protocol specifications (BOLTS). It replaces the old name 'OPTION_ANCHORS_ZERO_FEE_HTLC' with the shorter name 'OPTION_ANCHORS' everywhere it appears. The numeric bit positions and behavior are unchanged, so this is a non-functional code cleanup with no security relevance.

Lower-priorityqml: history list: don't update fiat values during drag scollby f321x · 3d8fabac · Apr 30, 2026 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · f321x

qml: history list: don't update fiat values during drag scoll

Don't update the fiat values during scrolling through the history
with the thing on the right side (drag scrolling), this should
improve the performance of the list when doing a quick drag scroll.

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 performance optimization for the mobile-style QML history list in the Electrum Bitcoin wallet. It stops the fiat (local currency) value labels from being recalculated while the user is drag-scrolling, and refreshes them only when the scroll finishes. There is no security-relevant change.

Lower-priorityqml: set sourceSize on HistoryItemDelegate iconby f321x · efda94ae · Apr 30, 2026 · 1 fileMessage 88 · StrongInformational 15Details
Commit message · f321x

qml: set sourceSize on HistoryItemDelegate icon

From the Qt docs (https://doc.qt.io/qt-6/qml-qtquick-image.html#sourceSize-prop):
```
Unlike the width and height properties, which scale the painting of the image, this property sets the maximum number of pixels stored for the loaded image so that large images do not use more memory than necessary.
```

When we want to display e.g. the clock icon in iconSizeLarge we only need to keep
iconSizeLarge with/height pixels in memory instead of the whole 256x256 icon.

88/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Links an issue, advisory, or supporting reference
AI analysis · Informational 15/100

This commit is a minor memory-use optimization for the Electrum mobile/QML user interface. It tells Qt to keep only the needed icon size in memory instead of loading the full 256x256 image. There is no security issue here.

Lower-priorityqml: history list: improve scrolling speedby f321x · 341a4f7b · Apr 30, 2026 · 2 filesMessage 80 · StrongInformational 14Details
Commit message · f321x

qml: history list: improve scrolling speed

Disables the Material design ovelay background on HistoryItemDelegate
to prevent flickering when scrolling and loading the app.

Sets reuseItems and cacheBuffer in history ElListView to reuse delegate
instances instead of destroying and re-creating them each time they
become (in-)visible. This makes scrolling through a large history
smoother.

80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
AI analysis · Informational 14/100

This commit is a user-interface performance tweak for Electrum's mobile-style QML history screen. It makes scrolling through transaction history smoother by reusing list row objects instead of constantly creating and destroying them, and by disabling a visual press effect that was causing flicker. There is no security-relevant change here.

Lower-prioritycommands: add list_channel_htlcs command to list failed, inflight and settled HTLCs for a channelby Sander van Grieken · 5a5c1e1f · Apr 29, 2026 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · Sander van Grieken

commands: add list_channel_htlcs command to list failed, inflight and settled HTLCs for a channel

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

This commit adds a new read-only command that lets users list payment details (HTLCs) for one of their own Lightning channels. It does not change how money moves, does not add new network exposure, and does not appear to introduce a security vulnerability.

Lower-priorityqml: QERequestDetails: handle _wallet = None in callbackby f321x · e1153265 · Apr 29, 2026 · 1 fileMessage 73 · AdequateInformational 22Details
Commit message · f321x

qml: QERequestDetails: handle _wallet = None in callback

Hanndle `QERequestDetails._wallet` being `None` in the
`on_event_request_status` callback to fix #10617.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Links an issue, advisory, or supporting reference
AI analysis · Informational 22/100

This is a small bug-fix patch for Electrum's mobile-style QML user interface. It prevents a crash when a payment-request status update arrives after the wallet object has already been cleared. The change simply checks that the wallet reference still exists before comparing it. There is no direct evidence in the commit that this is a security vulnerability, and the most likely effect is a harmless UI crash rather than a security exploit.

Security candidateSECURITY.md: enable "private vuln reports" on GitHubby SomberNight · a271e2f1 · Apr 28, 2026 · 1 fileMessage 86 · StrongInformational 15Details
Commit message · SomberNight

SECURITY.md: enable "private vuln reports" on GitHub

ref https://docs.github.com/en/code-security/how-tos/report-and-fix-vulnerabilities/privately-reporting-a-security-vulnerability

86/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference✓ Names security-relevant behavior explicitly
Why it was queued
explicit security languagedocumentation-only discount
AI analysis · Informational 15/100

This commit only updates the project's SECURITY.md file to add a new preferred way for researchers to privately report security vulnerabilities through GitHub's built-in reporting feature. It does not change any application code, fix a bug, or address an active security issue. It is a documentation/process change only.

Lower-priorityrelease notes: 4.7.2: add links to security disclosuresby SomberNight · bd5ac019 · Apr 28, 2026 · 1 fileMessage 55 · ThinTriage 0Details
Commit message · SomberNight

release notes: 4.7.2: add links to security disclosures

55/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Names security-relevant behavior explicitly! No meaningful explanatory body
AI review queuedcommands: export_lightning_preimage: add comment about wallet passwordby SomberNight · 3399c20a · Apr 28, 2026 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · SomberNight

commands: export_lightning_preimage: add comment about wallet password

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
second-pass: broader security terminology
AI analysis · Informational 15/100

This commit only adds a documentation comment to the export_lightning_preimage command explaining that exporting a Lightning payment preimage does not require the wallet password. It changes no code behavior, no access controls, and no security logic. There is no vulnerability in this patch.

Lower-prioritylazy_trampoline: adapt unit testby ThomasV · 187ea806 · Apr 28, 2026 · 2 filesMessage 45 · ThinInformational 12Details
Commit message · ThomasV

lazy_trampoline: adapt unit test

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 12/100

This commit only moves and adapts a unit test for Electrum's Lightning trampoline routing feature. It changes where a test case lives and how many payment attempts are expected, but it does not fix any user-facing bug or change any production security behavior. There is no indication this is a security patch.

Lower-prioritylazy trampoline:by ThomasV · f3a8dd61 · Apr 28, 2026 · 3 filesMessage 53 · ThinLow 35Details
Commit message · ThomasV

lazy trampoline:

If a trampoline forwarder fails to find a path, it may return a list
of trampolines it knows how to reach, so that clients can add these
trampolines to their route.

The list of trampolines and fees is written in the error data of
the 'update_fail_htlc' message.

53/100 · ThinMessage clarity
✓ Subject identifies a change✓ Provides detailed explanatory context✓ Explains rationale or failure mode! Too few words to establish purpose
AI analysis · Low 35/100

This commit changes how Electrum routes Lightning trampoline payments. When a trampoline forwarder cannot find a path, it can now return a list of alternative trampolines it knows, along with their fees, inside the payment failure message. The sender then uses that list to build a better route. The change also removes the old fixed two-trampoline-hop option and adjusts when fee/expiry errors are raised during forwarding. It is a protocol/routing improvement, not a clearly disclosed security fix.

Lower-prioritycreate_routes_for_payment: allow trampoline forwarding without channel_db if there is a direct pathby ThomasV · df5c8c4c · Apr 28, 2026 · 1 fileMessage 50 · ThinLow 34Details
Commit message · ThomasV

create_routes_for_payment: allow trampoline forwarding without channel_db if there is a direct path

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

This commit changes how Electrum's Lightning wallet builds payment routes when using trampoline routing. It adds a shortcut: if the payment recipient is directly connected to the wallet via a channel, Electrum can create a direct route without needing the full network channel database. The change also moves the fee-budget check out of the general route builder so it applies to both direct and non-direct routes. There is no explicit security bug in the diff, but the change touches payment routing logic and fee checks, which are security-sensitive.

AI review queuedbolt11: follow-up renamesby SomberNight · b0a5e201 · Apr 27, 2026 · 2 filesMessage 35 · OpaqueInformational 15Details
Commit message · SomberNight

bolt11: follow-up renames

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 simple code cleanup that renames leftover references from 'LnAddr' to 'BOLT11Addr' and 'lndecode' to 'decode_bolt11_invoice' in comments and docstrings. It makes no functional changes to the code and has no security relevance.

AI review queuedqml: BalanceDetails: fix typoby SomberNight · d2700dfb · Apr 27, 2026 · 1 fileMessage 35 · OpaqueInformational 15Details
Commit message · SomberNight

qml: BalanceDetails: fix typo

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 is a one-character typo fix in a user-interface file for Electrum's QML (mobile-style) GUI. The button that starts a Lightning swap was checking the wrong property name, which likely meant the button stayed disabled even when a Lightning receive balance existed. There is no security issue here—just a UI bug that may have prevented users from clicking a swap button.

AI review queuedqt, watchtower: cleanup importsby f321x · 560d90e8 · Apr 27, 2026 · 2 filesMessage 35 · OpaqueInformational 15Details
Commit message · f321x

qt, watchtower: cleanup imports

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 removes unused Python imports from two files: a Qt channel details dialog and a watchtower server plugin. It is a routine code cleanup with no functional changes and no security relevance.

AI review queuedlnaddr: rename LnAddr -> bolt11by f321x · 306cac19 · Apr 27, 2026 · 15 filesMessage 58 · ThinInformational 15Details
Commit message · f321x

lnaddr: rename LnAddr -> bolt11

The LnAddr, lndecode and lnencode naming didn't imply that it is
bolt 11 specific, making it confusing to work with, now that there are
also bolt 12 "lnaddr".
Renaming it to *bolt11* creates a clear separation to bolt 12 things and
reduces mental load.

This commit is pure renaming (using the PyCharm IDE refactor function),
except for the removal of the `object` inheritance of LnAddr/BOLT11Addr,
this is Python 2 legacy.

58/100 · ThinMessage clarity
✓ Descriptive subject✓ 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 large but purely cosmetic rename in the Electrum Bitcoin wallet. It renames the internal module `lnaddr.py` to `bolt11.py` and renames classes/functions from names like `LnAddr` and `lndecode` to `BOLT11Addr` and `decode_bolt11_invoice`. The goal is to avoid confusion with the newer BOLT 12 Lightning invoice format. The actual code logic is unchanged, apart from removing the obsolete `object` base class from one class. There is no security fix or vulnerability introduced here.

AI review queuedcrash_reporter: detect more altcoin-forks, don't send reportsby SomberNight · 3534f62b · Apr 27, 2026 · 1 fileMessage 91 · StrongInformational 23Details
Commit message · SomberNight

crash_reporter: detect more altcoin-forks, don't send reports

A new crash report [0] looks to be from a fork where the altcoin has the same genesis block [1] as Bitcoin.
But ofc "branding" is important, so they updated the URLs to point to their fork -- so we could also test against that.

[0]: https://github.com/spesmilo/electrum/issues/10610
[1]: https://github.com/palladium-coin/pallectrum/blob/0df81c6af8d36f3549739576681d3df9969e32fd/electrum/constants.py#L194

91/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
Why it was queued
second-pass: broader security terminology
AI analysis · Informational 23/100

This commit tightens a check in Electrum's crash reporter so it refuses to send crash reports to Electrum's servers when the software is an altcoin fork. Previously, forks that kept Bitcoin's genesis block but changed the project's source-code URL could still spam Electrum with crash reports. The change adds a URL check to catch more forks. It is a defensive hardening patch, not a fix for a vulnerability in Electrum itself.

Lower-priorityqml: add default topPadding to ElTextAreaby f321x · 9079badf · Apr 27, 2026 · 2 filesMessage 76 · AdequateInformational 15Details
Commit message · f321x

qml: add default topPadding to ElTextArea

I noticed all ElTextAreas seem to look better with some topPadding,
so it makes more sense to add it as default to ElTextArea.
Followup to https://github.com/spesmilo/electrum/pull/10579

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 15/100

This commit is a purely cosmetic user-interface tweak. It moves a padding setting from one specific dialog into a shared text-area component so all text areas look consistent. There is no security relevance.

Lower-priorityqml: ElTextArea: hide placeholder text on user inputby f321x · 68726370 · Apr 27, 2026 · 1 fileMessage 53 · ThinInformational 17Details
Commit message · f321x

qml: ElTextArea: hide placeholder text on user input

When the user enters something into the ElTextArea the placeholder
text should be hidden so it doesn't conflict with the user input.
This affects e.g. the lnurl pay dialog.

53/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context! Contains work-in-progress language
AI analysis · Informational 17/100

This is a minor user-interface fix for Electrum's mobile/QML app. It makes placeholder hint text disappear once the user starts typing in a multi-line text box, so the hint doesn't visually overlap with what the user wrote. There is no security issue here.

Lower-prioritysetup.py: "qml_gui" extra: update pyqt versionby SomberNight · 96a3345a · Apr 24, 2026 · 2 filesMessage 45 · ThinInformational 12Details
Commit message · SomberNight

setup.py: "qml_gui" extra: update pyqt version

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

This commit updates the recommended PyQt6 version for Electrum's newer QML-based graphical interface. It loosens the version pin from below 6.6 to approximately 6.10, and removes notes saying 6.4 worked best because 6.5 was needed for a missing module. On its own, this is a routine dependency bump, not a clear security fix. However, pinning to an older PyQt6 branch can carry unpatched Qt/PyQt bugs, so moving to a newer supported line is generally healthier for security.

Lower-prioritytrampoline: _allocate_fee_budget_among_route: followup commentby SomberNight · b483e0d1 · Apr 24, 2026 · 1 fileMessage 40 · ThinInformational 15Details
Commit message · SomberNight

trampoline: _allocate_fee_budget_among_route: followup comment

40/100 · ThinMessage clarity
✓ Specific, descriptive subject! No meaningful explanatory body
AI analysis · Informational 15/100

This commit only renames an internal variable from `coeff` to `amt_in_coeff` and adds a clarifying comment. The mathematical behavior is unchanged, and there is no security fix or functional change.

Lower-prioritytest_lnrouter: add unittests for tramp fee allocationby f321x · 06fd0889 · Apr 24, 2026 · 1 fileMessage 83 · StrongInformational 15Details
Commit message · f321x

test_lnrouter: add unittests for tramp fee allocation

Adds `TestAllocateFeeBudget` for trampoline route fee allocation
coverage.

Co-Authored-By: SomberNight <somber.night@protonmail.com>

83/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
AI analysis · Informational 15/100

This commit only adds new unit tests for an existing fee-budget allocation function used in Lightning trampoline routing. It does not change any production code, so it cannot introduce a security vulnerability or fix one directly.

Lower-prioritytrampoline: handle edges with known fees in allocationby f321x · 6933faee · Apr 24, 2026 · 1 fileMessage 73 · AdequateLow 31Details
Commit message · f321x

trampoline: handle edges with known fees in allocation

Handle `TrampolineEdge` with known fees when allocating fees to
`TrampolineEdge` with `PLACEHOLDER_FEE` during trampoline route
construction.

This allows to create a mixed route from edges where we know the exact
feerates (e.g. provided through lazy trampoline) and evenly spread the
remaining budget between the edges where the fees are unknown (`PLACEHOLDER_FEE`).

Co-Authored-By: SomberNight <somber.night@protonmail.com>

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

This commit fixes how Electrum calculates fees for trampoline Lightning payments when some route edges have known fees and others are unknown placeholders. Previously, the code ignored known fees and split the entire budget evenly among placeholder edges, which could either overpay or exceed the intended fee budget. The new code solves a small linear equation so the final total fee stays within budget. This is a correctness and fee-budget safety improvement, not a direct theft-of-funds bug, but getting it wrong could cause payments to fail or users to pay more than expected.