CommitWatch RSS
SV
← Developer activityVerified account

Sander van Grieken

Public commit activity attributed with verified account confidence. This page describes observable work, not personal trustworthiness.

@accumulator on GitHub ↗
100 commits1 monitored projects10 candidates0 high-risk analyses
Project constellation

Where the commits appear

Amber nodes are monitored by CommitWatch. Gray nodes are sampled from authenticated GitHub public commit search and may not represent complete contribution history.

Monitored External sample
Projects connected to Sander van GriekenA visual map of monitored and externally discovered repositories.SVdeveloper100Electrum
Monitored evidence

CommitWatch projects

ElectrumElectrum100 commits10 candidates0 high risk
External discovery

Other public projects

No external sample loaded yet.

Authenticated GitHub discovery can enrich this profile.
Analyzed activity

Recent published watches

Message quality and risk characterize commits, never the person.

Low 41 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10799 from SomberNight/202608_android_secure_window

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

Adds visibility-gated secureWindow bindings to protect WIF keys and seed phrasesPrevents secureWindow from being cleared when multiple sensitive dialogs are stackedTargets Android screenshot/recents-thumbnail protection surface
4622390cby accumulator+17−05 files
Vendor flagged security relevance
Open full analysis →Source commit ↗
Informational 18 AI analysisMessage 50 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

qml: android on-screen keyboard submit trigger accepted

This commit tweaks the Android on-screen keyboard behavior in Electrum's QML wallet search screen. It sets the Enter key to 'Done' and hides the 'Create Wallet' button while the user is typing a search. There is no clear security problem h…

No security-relevant code paths modifiedUI-only change to on-screen keyboard and button visibilityNo input validation, authentication, or cryptographic logic changed
5fe0a292by Sander van Grieken+3−11 file
No security note in commit
Open full analysis →Source commit ↗
Low 29 AI analysisMessage 65 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

qml: add wallets search option, allow loading hidden wallet iff search string matches exact wallet name

This commit adds a search box to Electrum's mobile-style QML wallet list. It also changes how 'hidden' wallets (those whose filenames begin with a dot) are handled: previously they were blocked entirely, now they are allowed but kept out o…

GUI now permits loading wallets with dot-prefixed filenames that are otherwise hidden from the listExact-name match is required to load a hidden wallet, reducing accidental or brute-force discoveryValidation still rejects '..' prefixed filenames and path separators
836ef252by Sander van Grieken+51−52 files
No security note in commit
Open full analysis →Source commit ↗
Low 36 AI analysisMessage 58 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

qetxfinalizer: regex doesn't cover all, invalid Decimal -> 0 in TxFeeSlider.userFeeRate setter.

This commit fixes a crash in Electrum's mobile/QML fee slider. When a user typed a fee rate that the input regex allowed but Python's Decimal parser rejected, the app would crash with an 'InvalidOperation' exception instead of gracefully f…

Unhandled exception in user-input parsing pathUI crash / denial-of-service via malformed fee-rate inputInput validation gap between regex and Decimal parser
11155fdfby Sander van Grieken+5−21 file
No security note in commit
Open full analysis →Source commit ↗
Informational 21 AI analysisMessage 65 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

qml: QEWallet lifecycle fix, remove QEWallet from QEWallet.__instances on destruction.

This commit fixes a bookkeeping bug in Electrum's mobile/QML wallet interface. When a wallet object is destroyed, it was not being removed from an internal list that tracks all active wallet objects. The commit adds that cleanup step. The …

Lifecycle cleanup of shared instance registryPotential stale object reference / use-after-free class of issueNo explicit security claim in commit message
34f83a07by Sander van Grieken+3−01 file
No security note in commit
Open full analysis →Source commit ↗
Informational 15 AI analysisMessage 50 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

commands: add delete_channel(_backup) cli commands, analog to functionality already present in the GUIs

This commit adds two new command-line commands to Electrum, 'delete_channel' and 'delete_channel_backup', which already existed in the graphical user interface. It simply exposes existing wallet functionality through the command line, with…

8644b07fby Sander van Grieken+129−12 files
No security note in commit
Open full analysis →Source commit ↗
Informational 23 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

constants: add basic sanity check for servers.json

This commit adds a basic sanity check to Electrum's code that reads the built-in list of default servers (servers.json). Previously, if a user edited that JSON file incorrectly, the bad data would travel deeper into the program and cause a…

Input validation added to bundled configuration dataEarly-fail assertion to prevent downstream type errorsCommit message explicitly describes user error handling, not a security vulnerability
ab6308d6by Sander van Grieken+7−31 file
No security note in commit
Open full analysis →Source commit ↗
Informational 15 AI analysisMessage 45 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

lnwire: add bolt12 types to onion_wire

This commit adds data definitions for BOLT12 (a newer Lightning Network protocol for offers and invoices) to Electrum's wire format specification file. It is purely a schema addition with no executable code changes, no bug fixes, and no se…

52e681c8by Sander van Grieken+138−01 file
No security note in commit
Open full analysis →Source commit ↗
Moderate 57 AI analysisMessage 65 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

lnmsg: fix parsing of nested complex types where the leaf objects

This commit fixes a bug in Electrum's Lightning Network message parser. When reading nested structured data, the parser could keep consuming more bytes than intended because it didn't stop after reaching the expected number of items. The f…

Parser over-consumption bug in network message deserializationNested complex type boundary enforcement addedUnexpectedEndOfStream validation added for under-read scenarios
2e89915cby Sander van Grieken+8−01 file
No security note in commit
Open full analysis →Source commit ↗
Low 25 AI analysisMessage 88 · Strong
EL ElectrumElectrum BitcoinSoftware wallets

lnmsg: add new primitive type `bip340sig`, add tlv merkle root calculation and schnorr-sign over tlvs, implicit en/decode utf8 fields, schnorr signature verification.

This commit adds new Lightning Network message handling code to Electrum. It introduces a new BIP-340 Schnorr signature field type, computes Merkle roots over TLV (type-length-value) records, signs and verifies those roots, and adds implic…

New cryptographic signing/verification path addedNew BIP-340 tagged-hash Merkle root constructionSignature TLVs excluded from the Merkle root they sign
759ef10dby Sander van Grieken+160−111 file
No security note in commit
Open full analysis →Source commit ↗
Informational 12 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

common_qt: move submarine swap support code from qt gui to common_qt as SubmarineSwapMixin

This commit is a straightforward internal code refactor. It moves the user-interface code that handles 'submarine swaps' (a way to exchange on-chain Bitcoin for Lightning funds, or vice versa) out of the Qt desktop dialog and into a shared…

No security-relevant keywords in commit title or messageNo changes to cryptographic primitives, wallet signing, or network protocol parsingRefactor only: code moved between files with equivalent control flow
7c728344by Sander van Grieken+154−833 files
No security note in commit
Open full analysis →Source commit ↗
Informational 15 AI analysisMessage 45 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

qt: move wait_for_swap_transport to SwapManager

This commit simply moves an existing helper function from the Qt user-interface layer into the core swap-management module. The code's behavior is unchanged; it is a routine internal refactoring with no visible security implications.

1a1516acby Sander van Grieken+31−312 files
No security note in commit
Open full analysis →Source commit ↗
Informational 13 AI analysisMessage 50 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

swaps: encapsulate swap transport ongoing_connection_attempt future into transport

This commit is a small internal code cleanup in Electrum's submarine swap feature. It moves a 'future' object (a handle for an in-progress network connection attempt) from the transaction dialog class into the swap transport class itself. …

1fae2da5by Sander van Grieken+19−162 files
No security note in commit
Open full analysis →Source commit ↗
Informational 13 AI analysisMessage 50 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

qt: refactor message for change-to-lightning swap to backend submarine_swaps.py

This commit is a straightforward code cleanup: it moves the logic that decides what message to show users about 'sending change to Lightning' from the Qt GUI dialog into a shared backend module. The visible behavior is essentially unchange…

ffec9df1by Sander van Grieken+40−332 files
No security note in commit
Open full analysis →Source commit ↗
Informational 15 AI analysisMessage 45 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

lnutil: add to_tlv_bytes() to LnFeatures

This commit adds a small helper method that converts Lightning Network feature flags into a compact byte format used in modern Lightning protocol messages. It is a routine, additive code change with no security relevance visible in the com…

a7d552adby Sander van Grieken+19−02 files
No security note in commit
Open full analysis →Source commit ↗
Informational 15 AI analysisMessage 45 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

lnutil: add blinded path feature flag

This commit simply adds two new named constants for an existing Lightning network feature (route blinding/blinded paths) to Electrum's internal feature-flag list. It does not change any active code paths, parsing logic, or security behavio…

7ed84375by Sander van Grieken+6−01 file
No security note in commit
Open full analysis →Source commit ↗
Informational 15 AI analysisMessage 55 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

lnonion: factor out next_blinding_from_shared_secret

This commit is a simple code cleanup: it extracts a small, repeated calculation into a shared helper function. The math used to derive the next blinding key is unchanged, and there is no indication of a security fix or behavior change.

37b6fe3dby Sander van Grieken+12−102 files
No security note in commit
Open full analysis →Source commit ↗
Informational 15 AI analysisMessage 65 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

lnonion: rm is_onion_message param from process_onion_packet

This is a small internal code cleanup in Electrum's Lightning onion message handling. It removes a redundant function parameter and lets the code figure out the same thing from an existing parameter. There is no security fix or behavior ch…

63e0258bby Sander van Grieken+3−33 files
No security note in commit
Open full analysis →Source commit ↗
Informational 15 AI analysisMessage 50 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

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

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 securi…

5a5c1e1fby Sander van Grieken+30−01 file
No security note in commit
Open full analysis →Source commit ↗
Informational 15 AI analysisMessage 45 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

android: remove unneeded dl-ndk-ci.sh

This commit simply deletes an unused helper script that downloaded an Android NDK build artifact during continuous integration. There is no security-relevant change, no vulnerability fix, and no indication the script was ever part of shipp…

83b67700by Sander van Grieken+0−81 file
No security note in commit
Open full analysis →Source commit ↗
Wrong identity?Names can collide and public author strings can be misleading.Contact commitwatch@karma-x.io →