SW
← All projectsSparrow

Sparrow Wallet

Desktop Bitcoin wallet focused on security, privacy, multisignature, and hardware signers.

BitcoinHardware integrationSoftware walletsNormal
Repository coverage

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

50security candidates174second-pass queue175AI analyses
46commits · 30 days
53commits · 60 days
181commits · 180 days
281commits · 365 days
Backfill bands
Aug 5 → Feb 6100 seen9 candidatesComplete
Feb 6 → Jun 6128 seen15 candidatesComplete
Jun 6 → Jul 67 seen1 candidatesComplete
Jul 6 → Aug 546 seen8 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.

46/100 average clarity
0Strong · 80–100
20Adequate · 60–79
212Thin · 40–59
49Opaque · 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.
Craig Raw26846167546
doblon8312048
Michele Balistreri211048
nroktib111050
nzb-tuxxx111060
Liz Lightning201045
PeterXMR101045
Ian McKenzie100050
ottosch101050
craigraw100060
Analysis record

Published AI watches

Last scanned 53 minutes ago

Low 48 AI analysisMessage 50 · Thin
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

avoid deleting the backups of same-prefixed wallets

This commit fixes a bug in how Sparrow Wallet finds and deletes old wallet backups. Previously, the backup cleanup logic used loose pattern matching that could accidentally treat backups of one wallet as if they belonged to another wallet …

Incorrect backup selection could lead to deletion of another wallet's backup filesLoose filename prefix matching allowed same-prefixed wallet names to collideDate extraction regex accepted extra trailing characters, broadening matches
b6ed4ba7by Craig Raw+55−302 files
No security note in commit
Moderate 62 AI analysisMessage 60 · Adequate
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

verify proof of work on chain tips and warn when a tip goes stale

This commit adds safety checks to the Bitcoin wallet Sparrow when it receives block-chain tip announcements from an Electrum server. It now verifies that the announced block header is properly formatted, not timestamped too far in the futu…

Adds proof-of-work verification on server-announced chain tipsAdds future-timestamp rejection for announced chain tipsAdds staleness warning when no new mainnet tip is received for >2 hours
5ccc4902by Craig Raw+180−55 files
No security note in commit
Moderate 66 AI analysisMessage 50 · Thin
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

prevent a password change from re-encrypting wallets whose filenames share the same prefix

This commit fixes a bug in Sparrow Wallet's password-change feature. Previously, when a user changed the password on a wallet file whose name was a prefix of another wallet file (for example, 'Savings' and 'Savings.old'), the underlying H2…

Unintended re-encryption of sibling wallet files due to prefix matching in H2 ChangeFileEncryptionPotential wallet corruption or loss of access when changing a wallet passwordUse of isolated temporary directory to prevent cross-wallet file operations
24c6202eby Craig Raw+153−103 files
No security note in commit
Moderate 56 AI analysisMessage 50 · Thin
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

reject truncated and oversized tlv lengths when parsing keycard responses

This commit fixes a bug in how Sparrow Wallet reads data from Keycard hardware wallets. Previously, the code trusted the length declared inside a card response without checking whether that many bytes actually exist. A malicious or malfunc…

Out-of-bounds read / buffer over-read in TLV length parsingTruncated and oversized length values previously acceptedPotential memory-safety issue leading to zero-padded fabricated data
b0b9cc23by Craig Raw+160−52 files
No security note in commit
Low 42 AI analysisMessage 50 · Thin
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

reject extended private keys when creating a terminal watch only wallet, and show import errors

This commit fixes a bug in Sparrow Wallet's terminal (command-line) watch-only wallet creation. Previously, a user could accidentally paste an extended private key (xprv) into a dialog meant only for public keys or output descriptors, and …

Prevents accidental import of extended private keys into watch-only wallet contextSurfaces previously swallowed import errors to the user via error dialogChanges exception handling from ImportException-only to all exceptions
fa0d4841by Craig Raw+19−152 files
No security note in commit
Low 49 AI analysisMessage 50 · Thin
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

ensure imported keystore labels are truncated and unique

This commit fixes a bug in Sparrow Wallet's import of wallet labels. Previously, labels for multi-signature participants (keystores) could be too long or identical to each other, which could make the wallet file impossible to reopen. The c…

Denial-of-service via malformed imported label causing wallet file to become unopenableInput validation added for imported keystore labelsLength truncation and uniqueness enforcement for keystore labels
7a3f775aby Craig Raw+17−61 file
Vendor flagged security relevance
Moderate 61 AI analysisMessage 50 · Thin
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

always check and restrict existing wallets and backup directories to owner only permissions

This update makes Sparrow Wallet automatically tighten the file permissions on wallet and backup folders so only the computer's owner can read them. Previously, if those folders already existed with looser permissions (for example, created…

Permission hardening for sensitive wallet storage directoriesNew setOwnerOnlyDirectory helper enforcing rwx------ on existing directoriesBackup directory permissions now always restricted on startup
5d387765by Craig Raw+35−01 file
Vendor flagged security relevance
Low 25 AI analysisMessage 45 · Thin
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

add bitbox02 attestation failed dialog

This commit adds a user-facing warning dialog when a BitBox02 hardware wallet fails an attestation check. Attestation is a process that helps verify the device is genuine and not a counterfeit or tampered unit. Previously, a failed attesta…

Adds explicit user warning for failed hardware wallet attestationPrevents repeated warning dialogs during device enumeration using AtomicBooleanWarns user not to store funds on potentially counterfeit device
1a810c06by Craig Raw+16−12 files
No security note in commit
Moderate 56 AI analysisMessage 50 · Thin
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

improve validation of payjoin proposals, and accept a substituted payment output where a change output is present

This commit strengthens how Sparrow Wallet checks Payjoin proposals received from a payment receiver. Payjoin lets a receiver add their own inputs to a transaction to improve privacy. The changes add missing checks that could previously le…

Added minimum fee-rate enforcement on Payjoin proposalsAdded Taproot (P2TR) key-path and derived-public-key validation and propagationRestricted payment-output substitution to proposals that retain a change output
32f7e58fby Craig Raw+250−233 files
No security note in commit
Low 46 AI analysisMessage 50 · Thin
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

improve validation of legacy multipart qr part numbers

This commit tightens how Sparrow Wallet handles multi-part QR codes used to import older wallet data. It limits part numbers to four digits and rejects impossible values (like part 0 of 0, or part 5 of 2). The change prevents the app from …

Input validation added to QR multipart parserRegex quantifier tightened from unbounded to {1,4}Bounds check prevents m > n and zero/negative counts
059f1e88by Craig Raw+10−62 files
No security note in commit
Informational 18 AI analysisMessage 65 · Adequate
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

remove unused signature verification results in satochip and keycard signers, note where signatures are verified

This commit removes leftover code that checked whether signatures from hardware card signers were valid, but then threw away the result. The signatures are still verified later by a different part of the wallet when the signed transaction …

Removal of local signature verification calls in hardware signer code pathsAdded comments documenting that signature verification occurs later via PSBT.verifyCombinedSignatures()Unused boolean results indicate the removed checks were dead code rather than active security controls
866e9893by Craig Raw+5−132 files
No security note in commit
Moderate 60 AI analysisMessage 50 · Thin
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

improve validation of bip129, descriptor and unchained wallet imports

This commit hardens how Sparrow Wallet imports wallet files from other tools. It adds checks that reject malformed or inconsistent imports—such as a multisig setup that says it needs 3 signers but only provides 2 keys, or a coordinator fil…

Added wallet.checkWallet() validation after descriptor, BSMS, Caravan and terminal wallet creationBIP129 first-address verification prevents coordinator from supplying different keys to each signerBIP129 path restriction enforcement limits derivation to standard receive/change chains
3752540eby Craig Raw+469−429 files
Vendor flagged security relevance
Moderate 59 AI analysisMessage 60 · Adequate
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

only save certificates passing hostname verification as ca validated, and retain not yet valid certificates

This commit tightens how Sparrow Wallet saves and reuses TLS certificates for private servers. Previously, any certificate that passed a basic CA check was saved as 'CA validated' even if it didn't actually match the server's hostname. Tha…

Missing hostname verification before saving CA-trusted certificateBroad exception handling previously caused deletion of saved certificates on any CertificateExceptionCertificate pinning/trust-store logic for private Electrum servers
372ebf2fby Craig Raw+38−61 file
No security note in commit
Low 28 AI analysisMessage 35 · Opaque
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

remove ineffective eckey clear

This commit removes several calls that tried to wipe private-key material from memory. The developer says these wipes were 'ineffective' because the underlying ECKey object still held the secret bytes. The change is a cleanup, but it sligh…

Removal of explicit private-key zeroization callsCommit message labels prior zeroization as ineffectivePrivate key material remains referenced longer after use
1636c7e2by Craig Raw+2−1911 files
No security note in commit
High 72 AI analysisMessage 55 · Thin
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

verify fetched transactions match requested txid

This commit adds checks to make sure that when Sparrow Wallet asks an Electrum server for a specific Bitcoin transaction, the server actually returns the transaction that was requested. Before this change, a malicious or buggy server could…

Missing input validation on server-provided transaction dataPotential transaction substitution by malicious or compromised Electrum serverDefense-in-depth verification added at data deserialization boundary
3979694eby Craig Raw+100−163 files
No security note in commit
Informational 15 AI analysisMessage 28 · Opaque
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

rotate era logos

This commit simply rotates four logo images by 90 degrees. It changes only SVG graphics files and does not touch any code, cryptography, wallet logic, or network behavior. There is no security relevance.

2b9c3eb7by Craig Raw+4−44 files
No security note in commit
Informational 15 AI analysisMessage 38 · Opaque
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

bump to v2.5.4

This commit is a routine version bump from 2.5.3 to 2.5.4. It only changes version strings in four files: the build configuration, macOS packaging metadata, in-app version constant, and documentation for reproducible builds. There are no c…

ef2c2cdeby Craig Raw+4−44 files
No security note in commit
Informational 18 AI analysisMessage 50 · Thin
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

treat whitespace-only labels as blank on label import and export

This commit tightens how Sparrow Wallet treats labels made only of spaces or tabs during wallet label import and export. Previously, a label consisting solely of whitespace was considered a real label; now it is treated as blank and skippe…

Behavioral hardening: whitespace-only strings now treated as absent labelsNo input validation bypass, memory safety, or cryptographic changes observedPotential minor UI/data-integrity issue mitigated, not a vulnerability
0dce4783by Craig Raw+10−92 files
No security note in commit
Informational 19 AI analysisMessage 50 · Thin
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

add option to ignore dust on private key sweep

This commit adds a user-facing checkbox labeled 'Ignore dust' to the private-key sweep feature in Sparrow Wallet. When enabled, very small ('dust') unspent outputs linked to the swept key are excluded from the transaction. This is a usabil…

Adds user-controlled filtering of UTXOs by value thresholdUses existing 'dustAttackThreshold' configuration, implying prior anti-dust handling elsewhereNo input validation, cryptographic, or authorization changes
cf797ea0by nroktib+21−21 file
No security note in commit
Informational 19 AI analysisMessage 45 · Thin
SW SparrowSparrow Wallet BitcoinHardware integrationSoftware wallets

add opt-in xdg application directory support

This commit changes where Sparrow Wallet stores its files on macOS and Linux to optionally follow the XDG Base Directory Specification. It is a directory-layout migration, not a fix for a vulnerability. The change could create minor securi…

Directory migration logic with symlink creation and fallback search pathsInstance lock pointer now searched in both new XDG state home and legacy default homeNetwork directory rename logic creates/updates symlinks and may leave stale symlinks
2d667d11by Craig Raw+177−829 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.

AI review queuedfurther minor qr display dialog updatesby Craig Raw · 4724dc77 · Jan 28, 2026 · 3 filesMessage 45 · ThinInformational 15Details
Commit message · Craig Raw

further minor qr display dialog updates

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit is a minor user-interface polish update for the QR code display dialog in Sparrow Wallet. It changes the order of buttons in the dialog's button bar, renames a density toggle from 'Decrease/Increase Density' to 'Less/More Dense', and updates two small SVG icon graphics for the BBQR encoding option. There is no security-relevant change visible in the code.

Lower-priorityupdates for java 25 compatibilityby Craig Raw · 883558fd · Jan 28, 2026 · 2 filesMessage 45 · ThinTriage 0Details
Commit message · Craig Raw

updates for java 25 compatibility

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI review queuedavoid scanning for legacy addresses on bitbox02 with wallet discoveryby Craig Raw · 9d576bc4 · Jan 23, 2026 · 2 filesMessage 50 · ThinInformational 24Details
Commit message · Craig Raw

avoid scanning for legacy addresses on bitbox02 with wallet discovery

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 24/100

This commit changes Sparrow Wallet so that when it performs automatic wallet discovery with a BitBox02 hardware wallet, it no longer scans for old-style 'legacy' Bitcoin addresses (P2PKH). The change appears to be a compatibility or reliability fix for the BitBox02 device, not a security vulnerability patch. There is no evidence in the commit of an exploit, attacker, or disclosed security issue.

Security candidatefix trezor change detection on signingby Craig Raw · 49d807f3 · Jan 22, 2026 · 1 fileMessage 45 · ThinModerate 53Details
Commit message · Craig Raw

fix trezor change detection on signing

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing boundary
AI analysis · Moderate 53/100

This commit claims to fix how Sparrow Wallet detects 'change' outputs when signing transactions with a Trezor hardware wallet. Change outputs are coins sent back to the user's own wallet during a payment. If detection fails, the wallet or device might mislabel or mishandle those coins, which could confuse the user or in some designs affect how funds are verified. The actual code change is only one line in one file, but the diff was not provided, so we cannot verify what exactly was changed.

AI review queuedadd connected device wallet discovery with configurable number of accounts to scanby Craig Raw · 0c2ee669 · Jan 22, 2026 · 9 filesMessage 50 · ThinInformational 18Details
Commit message · Craig Raw

add connected device wallet discovery with configurable number of accounts to scan

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 18/100

This commit adds a new feature to Sparrow Wallet that lets users scan a connected hardware wallet for existing accounts and transactions across multiple script types. It is a feature addition, not a security fix. The code changes how wallet import events and discovery results are handled, moving from single-wallet to multi-wallet results. There is no direct evidence in the commit of a vulnerability being patched.

AI review queuedadd user agent to coingecko historical rates callby Liz Lightning · 85c81adc · Jan 21, 2026 · 1 fileMessage 45 · ThinInformational 19Details
Commit message · Liz Lightning

add user agent to coingecko historical rates call

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 19/100

This commit is a small cleanup that makes the CoinGecko historical exchange-rate request send the same browser-like User-Agent header that other exchange-rate calls already use. Previously, that one CoinGecko call sent no special headers. The change is unlikely to be a security fix on its own; it looks more like a reliability tweak to avoid being blocked by the CoinGecko API, which sometimes rejects requests without a User-Agent.

AI review queuedimprove qr encoding ui, save previous selection and add raw encoding for tx hexby Craig Raw · 04de8370 · Jan 20, 2026 · 13 filesMessage 50 · ThinInformational 18Details
Commit message · Craig Raw

improve qr encoding ui, save previous selection and add raw encoding for tx hex

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 18/100

This commit is a user-interface improvement for Sparrow Wallet's QR code sharing feature. It replaces a simple on/off toggle between two QR formats with a dropdown menu that also adds a new 'Raw' option for transaction hex, and it remembers the user's last chosen format. There is no indication this fixes a security vulnerability.

Security candidatefix handling of non-standard key derivations when writing output descriptorsby Craig Raw · ab99f1d3 · Jan 15, 2026 · 2 filesMessage 50 · ThinLow 32Details
Commit message · Craig Raw

fix handling of non-standard key derivations when writing output descriptors

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
secret or key materialsigning or wallet path
AI analysis · Low 32/100

This commit fixes how Sparrow Wallet writes output descriptors when a wallet uses a non-standard key derivation path. Previously, the code stripped the leading 'm/' from the derivation path in a simplistic way, which could produce incorrect descriptors for unusual paths. The fix now uses a dedicated parser/formatter (KeyDerivation.parsePath/writePath) to handle the path correctly. This is primarily a correctness/reliability fix, but incorrect descriptors could in theory lead to users backing up or sharing wrong wallet configuration data.

AI review queueduse precise fee rates estimate from mempool.spaceby Craig Raw · 53857389 · Jan 15, 2026 · 1 fileMessage 45 · ThinInformational 21Details
Commit message · Craig Raw

use precise fee rates estimate from mempool.space

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 21/100

This commit changes the Bitcoin fee-rate data source used by Sparrow Wallet from mempool.space's standard recommended-fee endpoint to its more precise fee-estimate endpoint. It is a minor data-source change, not a code-level security fix. There is no indication in the commit that this resolves a security vulnerability.

AI review queuedconvert from tilepane to gridpane to resolve mnemonic words layout issueby Craig Raw · 98576d40 · Jan 14, 2026 · 2 filesMessage 50 · ThinInformational 15Details
Commit message · Craig Raw

convert from tilepane to gridpane to resolve mnemonic words layout issue

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

This commit is a purely cosmetic UI change in the Sparrow Wallet desktop app. It swaps the JavaFX layout component used to display BIP39 mnemonic word fields from a TilePane to a GridPane so the 12 or 24 seed words line up in a tidy grid. There is no change to how seed words are generated, stored, validated, or protected, and no security-relevant behavior is introduced or fixed.

AI review queuedadd additional fee rate limit check for high fee transactionsby Craig Raw · 96f7d4bd · Jan 13, 2026 · 1 fileMessage 50 · ThinLow 45Details
Commit message · Craig Raw

add additional fee rate limit check for high fee transactions

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 45/100

This commit adds an extra safety check in Sparrow Wallet that warns users before broadcasting Bitcoin transactions with extremely high fee rates. Previously, the wallet only warned if the fee rate exceeded the top of a long-term fee rate range. Now it also warns if the fee rate is more than 100 times the default fee rate. This helps prevent users from accidentally overpaying transaction fees, which could be caused by user error, wallet misconfiguration, or potentially a malicious or compromised transaction.

Security candidateadd support for keycard via smart card interfaceby Michele Balistreri · 0c679627 · Jan 13, 2026 · 18 filesMessage 45 · ThinLow 25Details
Commit message · Michele Balistreri

add support for keycard via smart card interface

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
cryptography-sensitive pathsigning or wallet path
AI analysis · Low 25/100

This commit adds a brand-new feature to Sparrow Wallet: support for Keycard hardware wallets via a smart card interface. It introduces many new Java files that handle low-level smart card communication, secure channel encryption, PIN handling, key derivation, and signing. The change is a large feature addition (+3,250 lines) rather than a small bug fix. There is no direct evidence in the commit message or diff that this fixes a known security vulnerability, and no external references were provided. Some implementation details—such as hardcoded pairing passwords, a TODO comment about device certificate verification, and a fallback to a default derivation path—could become security concerns if misused, but they are not proven vulnerabilities on their own.

AI review queuedadd user agent to coingecko exchange source requestby Craig Raw · 0515d805 · Jan 13, 2026 · 1 fileMessage 50 · ThinInformational 19Details
Commit message · Craig Raw

add user agent to coingecko exchange source request

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 19/100

This commit changes how Sparrow Wallet asks CoinGecko for currency exchange rates. It now sends a fake web-browser 'User-Agent' header instead of none. This is likely a workaround because CoinGecko started blocking or rate-limiting requests that had no user agent. It is not a fix for a vulnerability in Sparrow itself, but it does make Sparrow's exchange-rate requests slightly more identifiable and could affect privacy if the same user agent is reused elsewhere.

AI review queuedavoid extraneous error logging on card enumerationby Craig Raw · cf15760d · Jan 13, 2026 · 2 filesMessage 50 · ThinInformational 11Details
Commit message · Craig Raw

avoid extraneous error logging on card enumeration

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 11/100

This commit makes two small logging and error-handling changes in code that talks to smart cards (hardware wallets). It downgrades one routine card-enumeration error from 'error' to 'info' level, and turns a swallowed CBOR parsing error into a thrown exception. There is no indication these changes fix a security vulnerability; they appear to be code-quality or diagnostic-noise improvements.

AI review queuedremove unnecessary lark .gitmodulesby Craig Raw · f28f15df · Jan 13, 2026 · 1 fileMessage 35 · OpaqueTriage 0Details
Commit message · Craig Raw

remove unnecessary lark .gitmodules

35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
Security candidateadd any missing key path information to psbts once signing wallet is chosenby Craig Raw · 34900d29 · Jan 3, 2026 · 2 filesMessage 50 · ThinLow 29Details
Commit message · Craig Raw

add any missing key path information to psbts once signing wallet is chosen

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing boundarysigning or wallet path
AI analysis · Low 29/100

This commit changes Sparrow Wallet so that, when a user chooses a wallet to sign a Bitcoin transaction (PSBT), the app automatically fills in any missing key-path details needed for signing. The change is small and appears to be a usability/fix improvement rather than a clear security patch. There is no vendor statement or external reference saying this fixes a vulnerability, so we cannot confidently label it as a security fix.

AI review queuedadd warning when entering outdated slip132 values into a watch-only wallet of a different script typeby Craig Raw · 430f2ee3 · Dec 23, 2025 · 1 fileMessage 50 · ThinLow 30Details
Commit message · Craig Raw

add warning when entering outdated slip132 values into a watch-only wallet of a different script type

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 30/100

This commit adds a warning message in Sparrow Wallet when a user pastes an older-style extended public key (SLIP132 format) into a watch-only wallet that uses a different Bitcoin address type. It does not change wallet logic or fix a vulnerability; it only alerts the user to a possible mismatch so they can correct it. The change is defensive UX, not a security patch.

AI review queuedminor trezor v2 fixesby Craig Raw · 74160ba3 · Dec 20, 2025 · 2 filesMessage 28 · OpaqueInformational 15Details
Commit message · Craig Raw

minor trezor v2 fixes

28/100 · OpaqueMessage clarity
✓ Subject identifies a change! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: opaque commit messagesecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit changes the left and right margins of a text input dialog box from 60 pixels to 65 pixels. It is purely a cosmetic/layout adjustment for the Trezor hardware wallet v2 integration and has no security relevance.

Security candidateimprove thp pairing flow, and add passphrase session supportby Craig Raw · 59d85cdd · Dec 18, 2025 · 2 filesMessage 50 · ThinInformational 18Details
Commit message · Craig Raw

improve thp pairing flow, and add passphrase session support

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
access controlsigning or wallet path
AI analysis · Informational 18/100

This commit improves the user interface for pairing a Trezor hardware wallet with Sparrow Wallet. It adds a numeric-only input filter for the pairing code, makes the text larger, and shows the device name in pairing messages. There is no clear security vulnerability in the changes.

AI review queuedadd trezor safe 7 noise config implementationby Craig Raw · 04064219 · Dec 17, 2025 · 2 filesMessage 45 · ThinInformational 20Details
Commit message · Craig Raw

add trezor safe 7 noise config implementation

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 20/100

This commit adds support for pairing the Trezor Safe 7 hardware wallet with Sparrow Wallet. It introduces a JavaFX dialog flow that asks the user to confirm pairing and enter a pairing code shown on the device. The change is a feature addition for a new hardware device and does not, on its own, appear to be a security vulnerability. The pairing secrets are stored in a local JSON file, similar to how the existing BitBox02 pairing is handled.

Lower-priorityupdate to macos-15-intel runnerby Craig Raw · 4ddd09fe · Dec 11, 2025 · 1 fileMessage 45 · ThinTriage 0Details
Commit message · Craig Raw

update to macos-15-intel runner

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
documentation-only discount
AI review queuedadd ledger nano gen5 supportby Craig Raw · a49edb7c · Dec 11, 2025 · 3 filesMessage 45 · ThinTriage 12Details
Commit message · Craig Raw

add ledger nano gen5 support

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
Lower-priorityupdate drongo for bip93 package exportby Craig Raw · 5275cab4 · Dec 9, 2025 · 1 fileMessage 45 · ThinTriage 0Details
Commit message · Craig Raw

update drongo for bip93 package export

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI review queuedadd codex32 importer to software keystore import workflowby Ian McKenzie · 4dcd463d · Dec 9, 2025 · 4 filesMessage 50 · ThinTriage 12Details
Commit message · Ian McKenzie

add codex32 importer to software keystore import workflow

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
Security candidateminor ui changes to master private key importby Craig Raw · f900f6dc · Dec 4, 2025 · 2 filesMessage 50 · ThinInformational 15Details
Commit message · Craig Raw

minor ui changes to master private key import

50/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Names security-relevant behavior explicitly! No meaningful explanatory body
Why it was queued
secret or key materialcryptography-sensitive pathsigning or wallet path
AI analysis · Informational 15/100

This commit makes two small user-interface tweaks to the screen where a user imports a master private key: it changes a title to say 'Enter master private key', makes the Import button the default button that activates when the user presses Enter, and adds a period to a description sentence. There is no security-relevant code change.