Every captured commit receives deterministic security triage and a separate communication-quality score. Security candidates and broader second-pass signals receive full-patch Ollama analysis.
Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.
51/100 average clarity
33Strong · 80–100
316Adequate · 60–79
513Thin · 40–59
191Opaque · 0–39
33security candidates with opaque commit messaging
This commit fixes a mobile UI bug where pressing Cancel on a 'building transaction' dialog accidentally closed one too many screens. The fix moves the responsibility for closing the dialog into each screen's cancel handler, and prevents er…
No security-relevant signals in the diffUI-only navigation and error-dialog suppression changesNo cryptographic, authentication, authorization, or data-handling changes
This commit is a routine merge from the project's staging branch into a feature branch. The visible code changes fix small UI/UX bugs in Stack Wallet's send screens: clearing an internal address-validation cache when the form is reset, mak…
Clearing stale address-provider state on form reset reduces the risk of stale validation data influencing a subsequent transactionFee-form caching prevents repeated network calls and accidental fee UI flicker/state races on rebuildQR scan buttons are UX additions with no visible parser/validation logic changes
This commit fixes a desktop wallet bug where the send-fee form would repeatedly re-fetch fee estimates on every screen rebuild, wasting resources and possibly flickering. It now caches the fee estimate and only retries if the previous fetc…
No security-relevant signals present in diff or commit metadataChange is a UI/UX performance and reliability improvement
This commit merges a feature branch into another work-in-progress branch. It adds QR-code scanning buttons to two desktop token send screens and resets an internal address-validation state variable when the send form is cleared. There is n…
State reset added to clearForm() to avoid stale validation dataNew QR scan entry points in token send flows (UI feature, not a vulnerability signal)
This change fixes a UI state bug in the cryptocurrency wallet's send form. When a user clears the send form, the app now also resets whether the entered address is considered valid. Without this fix, the form could incorrectly keep showing…
UI state desynchronization between form field and validity indicatorPotential user confusion from stale valid-address stateNo input validation, cryptographic, or network-layer changes
This commit adds a QR code scanner button to two desktop cryptocurrency send forms. It is a straightforward user-interface feature addition with no apparent security relevance.
This commit adds a missing QR code scan button to two desktop token send screens in the Stack Wallet app. It is a straightforward user-interface fix that restores a feature already present on mobile and other send views; there is no indica…
This commit is purely a code-formatting cleanup. It changes line breaks and indentation in four user-interface files so the code is easier to read, but it does not change what the app actually does. There is no security relevance.
This commit fixes a UI bug in the desktop version of Stack Wallet where the send amount could be lost while the app was trying to estimate transaction fees. The fix keeps the amount 'alive' in memory so fee estimation does not accidentally…
State lifecycle issue in UI provider could lead to loss of user inputFix prevents fee estimation from silently resetting send amount to zeroNo cryptographic, authentication, or network security changes present
This commit updates the Stack Wallet app so that when a single payment is broken into multiple cryptocurrency transactions, the user sees a clear warning on the confirmation screen. It also changes the app's internal record-keeping to hand…
UI now warns users when a payment will be split into multiple on-chain transactionsInternal transaction model changed from single txid to list of txidsMultiple confirmation flows updated to iterate over all txids for notes and trade lookups
This commit fixes a performance bug in the desktop version of Stack Wallet where the app repeatedly re-fetched Bitcoin-style fee rates every time the on-screen fee form rebuilt. The change caches the fee result and only fetches again if th…
Unnecessary repeated network requests reducedFuture result cached to avoid re-execution on widget rebuildError path clears cache to allow retry
This commit fixes a UI state bug in Stack Wallet's send form. When a user clears the send form, the wallet now also resets whether the entered address is considered valid. Without this fix, the form could incorrectly keep showing an old ad…
Stale UI state after form resetAddress validity not synchronized with address field resetPotential UI confusion leading to incorrect send confirmation
This is a large feature/fix merge that restores and rewrites the Xelis (XEL) cryptocurrency integration in Stack Wallet. It swaps the old hand-rolled Xelis code for a new generated native interface (XWF), adds wallet restore/backup support…
Send-flow lifecycle hardening: prepared Xelis transactions are now discarded via cancelSend when the user cancels or the widget is disposedSession-generation checks prevent stale wallet handles from being used after close/reopenMutex serialization added around send preparation, balance, history, and rescan operations
This commit is a large merge that mainly adds integration tests for a desktop 'forgot password' reset feature and makes supporting code changes to safely shut down background database workers during that reset. It also removes a large set …
New integration tests exercise a destructive 'forgot password' data-wipe featureTests assert that password store and wallet key store are deleted on successful resetTests assert that wallet files are deleted while backup and tor state are preserved
This commit lets users type multi-line notes when editing transaction and trade notes, and fixes the desktop layout so long notes scroll instead of breaking the screen. It also swaps the old `mounted` check for the newer `context.mounted` …
This commit is a large merge that mainly removes old integration tests and adds new desktop 'forgot password' reset tests. It also adds a safe-shutdown path for background Firo cache workers and databases. The changes look like defensive h…
Added safe shutdown of Firo cache isolates/SQLite databases before reset exitNew integration tests verify desktop forgot-password reset deletes secrets and preserves backupsTest harness intercepts exit() and IOOverrides to observe reset side effects
This commit fixes the desktop "forgot password" reset flow in Stack Wallet. It adds integration tests that verify the app can securely wipe its own data when a user forgets the desktop password, and it updates the Firo cache worker to clos…
Desktop password reset now closes Firo cache workers and SQLite databases before deleting app data, reducing the risk of data leakage or corruption during wipeNew integration tests assert that a successful reset removes password store (hive/desktopdata.hive), wallet key store (isar/desktopStore.isar), and wallet files while preserving backups and tor stateFailed reset scenario leaves a .reset-pending marker and removes password/key stores first, preventing the reset from being undone after partial deletion
This commit is a large merge that mainly adds a new 'prove you own a Spark address' feature to the Stack Wallet app, plus some related fixes. It also updates a dependency that handles SOCKS5 proxy connections and changes how the app decide…
New cryptographic signing path added: SparkInterface.signMessage now delegates to Spark ownership proof creation using the wallet's private key and spark derivation path.Ownership proof code rejects view-only wallets and blank messages, and searches a 100-address lookahead for the requested address before signing.Dependency upgrade: socks5_proxy 1.0.3+dev.3 -> 2.1.1, which may change SOCKS5/Tor proxy behavior; a new test verifies hostname/onion routing through a fake SOCKS server.
This commit adds a new feature to Stack Wallet that lets users prove they own a Spark (privacy) address by generating a cryptographic ownership proof. It also improves the sign/verify screens so view-only wallets can still verify proofs, a…
New cryptographic proof generation using private key material (privateKeyHex, spendKeyIndex, diversifier) inside an isolateView-only wallet guard added for proof creation (throws if isViewOnly)Message whitespace now preserved for pasted/typed challenge messages, preventing proof/verification mismatches caused by silent trimming
This commit merges several changes into a development branch. The most notable security-relevant change is a fix for how the Trocador exchange service routes traffic: it now automatically uses Tor (an anonymity network) when the user has T…
Trocador exchange API previously forced clearnet (`isOnion: false`) at every call site, bypassing Tor even when enabledNew `_useTor` getter centralizes Tor routing decision based on app feature flag and user preferenceOnion service address rotated to a new v3 .onion hostname
Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.
AI review queuedfix switch on inherited typeby julian · 3ca1bfcc · Nov 13, 2025 · 3 filesMessage 45 · ThinLow 29Details
Commit message · julian
fix switch on inherited type
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 · Low 29/100
This commit refactors how the app decides which wallet setup path to use when restoring or verifying a wallet. It replaces exact class-name checks (like 'is this specifically a Monero wallet?') with broader category checks (like 'is this a CryptoNote-style wallet?'). This is mostly a code cleanup, but it could fix a subtle bug where a new or inherited wallet type would accidentally skip the restore-specific initialization and instead run a normal setup, potentially causing setup problems rather than direct theft of funds.
AI review queuedSWB tweaks/fixesby julian · 2dcd2b35 · Nov 13, 2025 · 3 filesMessage 28 · OpaqueLow 32Details
Commit message · julian
SWB tweaks/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 · Low 32/100
This commit fixes several bugs in Stack Wallet's backup restore flow. The most notable change is that retrying a failed wallet restore now properly re-initializes the wallet (including opening it for Cryptonote-based coins like Monero) before attempting recovery, instead of calling recover() on a wallet that may not be ready. It also moves an Android file-write operation to a background isolate to avoid blocking the user interface, and makes a wallet ID lookup more forgiving when a backed-up wallet no longer exists.
AI review queuedfix spark names not showing as confirmed in names viewby julian · 75865133 · Nov 13, 2025 · 1 fileMessage 50 · ThinInformational 17Details
Commit message · julian
fix spark names not showing as confirmed in names view
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 17/100
This commit fixes a display bug in the Stack Wallet app where Spark cryptocurrency names owned by the user were not showing as confirmed in the names view. The change simplifies how the wallet decides which Spark names belong to the user by removing a filter that excluded names already known locally. There is no indication this is a security vulnerability.
AI review queuedrefactor(spl): remove unneeded sol wallet token address providerby sneurlax · 4a860f04 · Nov 12, 2025 · 4 filesMessage 62 · AdequateInformational 15Details
Commit message · sneurlax
refactor(spl): remove unneeded sol wallet token address provider
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! 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 routine code cleanup. It removes a separate Solana-specific token address provider and folds its logic into the general wallet token address provider. There is no visible security fix or behavior change beyond simplifying where token addresses are read from.
AI review queuedrefactor(spl): don't create separate Amount objsby sneurlax · cc4ecbb0 · Nov 12, 2025 · 2 filesMessage 57 · ThinInformational 11Details
Commit message · sneurlax
refactor(spl): don't create separate Amount objs
57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 11/100
This commit is a small code cleanup in the Stack Wallet app. It changes how zero token balances are created so that only one zero-value object is reused instead of making four separate ones. There is no visible security problem or user-facing behavior change.
AI review queuedfix(spl): replace novel token balance provider to follow eth's exampleby sneurlax · a8906cf7 · Nov 12, 2025 · 6 filesMessage 62 · AdequateLow 32Details
Commit message · sneurlax
fix(spl): replace novel token balance provider to follow eth's example
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 32/100
This commit finishes a previously incomplete Solana token-balance feature. Before the change, the app showed a hard-coded zero balance for Solana tokens and could not refresh a token wallet on its own. After the change, real token balances are read from the Solana blockchain, stored in a local database, and displayed in the wallet. The change also makes the refresh button refresh the token balance instead of only the parent Solana wallet. There is no direct evidence in the commit of a security vulnerability, but it replaces a placeholder with real balance handling, which is a functional fix that could affect user funds if it had bugs.
AI review queuedfix(spl): prepare to replace novel token balance provider to be like ethby sneurlax · 87b3e5c0 · Nov 12, 2025 · 6 filesMessage 62 · AdequateLow 35Details
Commit message · sneurlax
fix(spl): prepare to replace novel token balance provider to be like eth
1/2, isar schema work next.
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 35/100
This commit changes how Stack Wallet displays Solana token balances. It replaces a live blockchain balance lookup with a temporary placeholder that always reports zero balance. The change is described by the developer as a preparatory step toward a database-backed design similar to Ethereum tokens. Because the balance shown to users will be zero until the follow-up work is completed, users could be misled about their token holdings, and send-flow checks that rely on this provider may behave incorrectly. There is no evidence in the commit that this is being exploited or that it was reported as a security issue.
✓ Subject identifies a change! Too few words to establish purpose! No meaningful explanatory body! Opaque security-relevant change
Why it was queued
cryptography-sensitive pathsigning or wallet path
AI analysis · Low 28/100
This commit updates the Monero wallet library used by Stack Wallet (cs_monero from version 2.0.0 to 3.1.0) and changes many wallet operations from synchronous to asynchronous. The visible changes are mostly mechanical: adding 'await' and 'Future' return types so the app waits for the underlying C++ Monero code to finish before continuing. There is also a small change to app-exit behavior on Linux and extra debug logging. The commit does not describe any security bug, and the diff itself does not show an obvious vulnerability. However, because it touches sensitive wallet operations (keys, balances, transaction keys, seeds, syncing) and upgrades a major dependency, it could indirectly affect security or stability if the new library version fixed or introduced issues not visible in this repository's diff.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 31/100
This commit fixes a crash in the desktop Solana token send screen. Previously, the app assumed a wallet value was always present and used a forced '!' operator, which would throw an error if the wallet data was still loading or missing. Now it checks for null and shows a 'Loading token data...' message instead of crashing.
AI review queuedfix(spl): set initial sync status according to parent walletby sneurlax · 5b977395 · Nov 11, 2025 · 2 filesMessage 62 · AdequateInformational 17Details
Commit message · sneurlax
fix(spl): set initial sync status according to parent wallet
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 17/100
This commit fixes a small UI consistency bug for Solana token wallets. Previously, the token wallet screen always showed itself as 'synced' when first opened, even if the parent Solana wallet was still syncing. Now it checks the parent wallet's actual sync state and shows 'syncing' if appropriate. There is no direct security vulnerability here; it is a correctness and user-experience fix.
AI review queuedfeat(spl): fetch sol token balance on refreshby sneurlax · 146a1572 · Nov 11, 2025 · 1 fileMessage 57 · ThinInformational 17Details
Commit message · sneurlax
feat(spl): fetch sol token balance on refresh
57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 17/100
This commit finishes a previously empty 'update balance' feature for Solana tokens in a crypto wallet. It now fetches the user's token balance from the Solana network when the wallet refreshes. The change is a normal feature completion and does not appear to introduce a security vulnerability, though it touches code that handles private wallet addresses and RPC communication.
AI review queuedfeat(spl): sol token fee estimationby sneurlax · cd97339f · Nov 11, 2025 · 1 fileMessage 57 · ThinInformational 19Details
Commit message · sneurlax
feat(spl): sol token fee estimation
57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 19/100
This commit replaces placeholder Solana token fee estimates with real ones fetched from the Solana network. Previously the wallet used a hard-coded 5000 lamport fee and threw an error when asked for fee details. Now it builds a sample token transfer transaction and asks the Solana RPC node how much that transaction would cost. This is a normal feature improvement and does not appear to introduce a security vulnerability.
AI review queuedfix(spl): amount formattingby sneurlax · 8ea00dfa · Nov 11, 2025 · 1 fileMessage 47 · ThinLow 42Details
Commit message · sneurlax
fix(spl): amount formatting
47/100 · ThinMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 42/100
This commit fixes how Solana token (SPL) amounts are displayed and confirmed in Stack Wallet's transaction confirmation screen. Previously, the app likely treated SPL tokens like Ethereum tokens, which could cause wrong decimal places, wrong token symbols, or even sending/confirming with incorrect values. The change adds separate code paths for Solana tokens so the correct token info is used for formatting, price lookup, and confirming the send.
AI review queuedfeat(spl): desktop sol token sendby sneurlax · fdcde9dc · Nov 11, 2025 · 3 filesMessage 57 · ThinInformational 17Details
Commit message · sneurlax
feat(spl): desktop sol token send
57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 17/100
This commit adds the ability to send Solana SPL tokens from the desktop version of Stack Wallet. It replaces a placeholder 'WIP' screen with a real send form and wires it up to Solana-specific wallet code. The change is a feature implementation, not a security fix. There is no evidence in the commit or supplied references that this addresses a vulnerability, breach, or disclosed security issue.
AI review queuedfix(spl): race condition fixby sneurlax · cda032bd · Nov 11, 2025 · 1 fileMessage 57 · ThinInformational 23Details
Commit message · sneurlax
fix(spl): race condition fix
57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: broader security terminologysecond-pass: security-sensitive path
AI analysis · Informational 23/100
This commit fixes a race condition in the Solana token transaction list widget. Previously, the widget tried to set up a database query and stream listener during initialization, before the wallet data was guaranteed to be ready. This could lead to crashes or a broken transaction list. The fix delays query setup until the wallet is available, and safely handles cases where the listener hasn't been created yet.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 33/100
This commit adds the first working code for sending Solana token (SPL) transfers inside Stack Wallet. It replaces a mock placeholder with real transaction building, signing, and broadcasting. The change is normal feature work, but it contains a known-incomplete helper for deriving recipient token addresses: the function falls back to returning the recipient's main wallet address instead of the correct associated token account, and the code itself admits this is a placeholder. If that fallback path is ever used, tokens could be sent to an address that does not actually hold the SPL token, risking loss of funds. The commit also exposes a wallet's signing keypair to a child token wallet, which is expected for this design but increases the attack surface if the token-wallet code is ever compromised.
AI review queueduse mobile_app_privacyby julian · 0a8b7a44 · Nov 7, 2025 · 5 filesMessage 18 · OpaqueInformational 19Details
Commit message · julian
use mobile_app_privacy
18/100 · OpaqueMessage clarity
✓ Subject identifies a change! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 19/100
This commit adds optional privacy hardening features to the Stack Wallet mobile app. Users can now enable a 'Cover in background' screen and, on Android, 'Disable screenshots.' These are defensive controls that make it harder for someone to capture the app's contents from the recent-apps switcher or via screenshots. There is no indication this commit fixes an active security flaw or introduces malicious behavior; it is a feature addition.
AI review queuedfeat(spl): implement balance fetching and fix ticker/symbol use in uiby sneurlax · a227e061 · Nov 6, 2025 · 6 filesMessage 62 · AdequateInformational 17Details
Commit message · sneurlax
feat(spl): implement balance fetching and fix ticker/symbol use in ui
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 17/100
This commit replaces placeholder/mock Solana token balances with real on-chain balance lookups and shows the correct token name and symbol in the user interface. It is a normal feature-completion change, not a security fix. There are no clear signs of malicious code, but the new code silently returns a zero balance whenever anything goes wrong, which could mislead users about their actual holdings.
AI review queuedui(spl): SPL token iconby sneurlax · c2ee6ecf · Nov 6, 2025 · 3 filesMessage 38 · OpaqueInformational 15Details
Commit message · sneurlax
ui(spl): SPL token icon
38/100 · OpaqueMessage clarity
✓ Subject identifies a change✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100
This commit is a routine user-interface polish: it adds the Solana coin icon asset, wires it up so Solana/SPL token screens show the correct icon, and replaces a placeholder TODO mint address with the actual token address. There is no security-relevant change.
AI review queuedfix(spl): Solana token specific transaction list widgetby sneurlax · bf7dacbc · Nov 6, 2025 · 4 filesMessage 62 · AdequateInformational 19Details
Commit message · sneurlax
fix(spl): Solana token specific transaction list widget
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 19/100
This commit adds a Solana-specific token transaction list widget and wires up the SolanaTokenAPI to use the wallet's real RPC client instead of a placeholder. It is a feature/fix for displaying SPL token transactions and checking token ownership. There is no direct evidence in the commit of a security vulnerability being patched.
Security candidatetemp fix text overflow on desktopby julian · 384e140b · Nov 6, 2025 · 1 fileMessage 30 · OpaqueInformational 15Details
Commit message · julian
temp fix text overflow on desktop
30/100 · OpaqueMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Names security-relevant behavior explicitly! No meaningful explanatory body! Contains work-in-progress language! Opaque security-relevant change
Why it was queued
memory safetysigning or wallet path
AI analysis · Informational 15/100
This commit is a minor user-interface tweak for the desktop version of Stack Wallet. It wraps a wallet name text widget so long names are truncated with an ellipsis instead of overflowing their container. There is no security relevance in the change.
AI review queuedconst HTTP constructorby julian · eb200eee · Nov 6, 2025 · 1 fileMessage 28 · OpaqueInformational 15Details
Commit message · julian
const HTTP constructor
28/100 · OpaqueMessage clarity
✓ Subject identifies a change! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100
This commit is a minor code cleanup in a Dart HTTP helper class. It adds a constant constructor and reformats some code onto fewer lines. There is no change to behavior, no bug fix, and no security relevance visible in the diff.
AI review queuedfix logical issueby julian · 2cdf6704 · Nov 5, 2025 · 1 fileMessage 28 · OpaqueLow 42Details
Commit message · julian
fix logical issue
28/100 · OpaqueMessage clarity
✓ Subject identifies a change! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Low 42/100
This commit fixes a simple but important logic bug in a Dart helper class. The code was checking whether a function was null, but then doing the exact opposite of what it should: when the function existed, it did nothing; when it was missing, it tried to call the missing function and would have crashed. The fix flips the condition so the existing function is called and the alternative code path is used only when arguments are needed. The bug appears to be a plain coding mistake rather than a deliberate security flaw, but in a wallet app any crash or unexpected behavior during sensitive operations is worth treating carefully.
AI review queuedfix android SAF SWBby julian · 2bc27922 · Nov 5, 2025 · 6 filesMessage 28 · OpaqueLow 33Details
Commit message · julian
fix android SAF SWB
28/100 · OpaqueMessage clarity
✓ Subject identifies a change! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Low 33/100
This commit changes how Stack Wallet's Android backup feature picks a save location. Instead of asking the user to choose a single file, it now asks the user to choose a folder. It also switches the auto-backup writer to use a helper that works with Android's Storage Access Framework (SAF), and it disables old-backup cleanup on Android because the SAF library cannot delete files. The change appears to be a bug fix for backups failing or misbehaving on Android due to file-path restrictions, not a fix for an active security vulnerability.
AI review queuedfix(spl): replace Ethereum-only transaction list with placeholderby sneurlax · fcd68bef · Nov 5, 2025 · 1 fileMessage 42 · ThinInformational 15Details
Commit message · sneurlax
fix(spl): replace Ethereum-only transaction list with placeholder
42/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body! Contains work-in-progress language
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit adds a brand-new desktop screen for sending Solana SPL tokens. It is a UI placeholder/replacement for an Ethereum-only transaction list, not a security patch. There is no evidence in the commit message or diff of a vulnerability being fixed.