SW
← All projectsStack Wallet

Stack Wallet

Actively maintained multi-coin self-custody wallet with Monero support and on-device keys.

MoneroPrivacy protocolsSoftware walletsNormal
Repository coverage

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

338security candidates411second-pass queue749AI analyses
77commits · 30 days
175commits · 60 days
600commits · 180 days
1002commits · 365 days
Backfill bands
Sep 27 → Mar 31404 seen73 candidatesComplete
Mar 31 → Jul 29463 seen238 candidatesComplete
Jul 29 → Aug 2887 seen8 candidatesComplete
Aug 28 → Sep 2753 seen6 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.

51/100 average clarity
33Strong · 80–100
316Adequate · 60–79
513Thin · 40–59
191Opaque · 0–39
33security 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.
Navid Rahimi1017154
sneurlax339158241062
julian347112269044
Julian19741144043
Dan Miller831525053
Reuben Yap20318052
Cyrix12615211049
julian-CStack222049
levoncrypto24121043
Tritonn204412052
cassandras-lies413048
NyanCatTW1111045
Analysis record

Published AI watches

Last scanned 4 minutes ago

Informational 24 AI analysisMessage 73 · Adequate
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

Merge pull request #1469 from Cyrix126/building_dialog_cancel

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
94e6da66by Julian+68−8412 files
No security note in commit
Informational 20 AI analysisMessage 50 · Thin
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

Merge branch 'staging' into building_dialog_cancel

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
0485b166by Julian+194−56 files
No security note in commit
Informational 17 AI analysisMessage 58 · Thin
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

Merge pull request #1468 from Cyrix126/desktop_fee_form_refetch

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
62432946by Julian+176−52 files
No security note in commit
Informational 17 AI analysisMessage 50 · Thin
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

Merge branch 'staging' into desktop_fee_form_refetch

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)
8a72c82fby Julian+18−04 files
No security note in commit
Low 32 AI analysisMessage 73 · Adequate
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

Merge pull request #1467 from Cyrix126/clear_send_form_validity

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
d3383b4eby Julian+2−02 files
No security note in commit
Informational 15 AI analysisMessage 50 · Thin
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

Merge branch 'staging' into clear_send_form_validity

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.

22560a59by Julian+16−02 files
No security note in commit
Informational 20 AI analysisMessage 73 · Adequate
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

Merge pull request #1466 from Cyrix126/desktop_token_qr_button

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…

47f6d369by Julian+16−02 files
No security note in commit
Informational 15 AI analysisMessage 18 · Opaque
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

style: format

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.

ae45aba8by Cyrix126+47−524 files
No security note in commit
Informational 23 AI analysisMessage 62 · Adequate
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

fix: stop estimating the desktop fee for a zero amount

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
90c91bdeby Cyrix126+22−02 files
No security note in commit
Low 32 AI analysisMessage 45 · Thin
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

show split tx warning on confirm screen

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
80343a5aby Julian+376−23916 files
No security note in commit
Informational 18 AI analysisMessage 62 · Adequate
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

fix: stop refetching the desktop fee rates on every rebuild

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
a1ca9292by Cyrix126+154−52 files
No security note in commit
Low 34 AI analysisMessage 62 · Adequate
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

fix: reset the address validity when clearing the send form

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
43dd0477by Cyrix126+2−02 files
No security note in commit
Low 35 AI analysisMessage 73 · Adequate
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

Merge pull request #1450 from Ez3kiel-dev/fix/xelis-integration

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
ad945d43by Julian+4298−162634 files
No security note in commit
Low 32 AI analysisMessage 45 · Thin
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

Merge branch 'staging' into fix/xelis-integration

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
b0e5d35aby Julian+2678−330378 files
No security note in commit
Informational 19 AI analysisMessage 73 · Adequate
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

Merge pull request #1361 from cypherstack/fix/305-multiline-transaction-notes

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

5172e63eby Julian+402−1113 files
No security note in commit
Low 32 AI analysisMessage 50 · Thin
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

Merge branch 'staging' into fix/305-multiline-transaction-notes

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
21491edbby Julian+2352−319075 files
No security note in commit
Moderate 57 AI analysisMessage 73 · Adequate
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

Merge pull request #1455 from cypherstack/fix/desktop-pw-reset

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
055e6c6bby Julian+1077−297154 files
No security note in commit
Low 37 AI analysisMessage 45 · Thin
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

Merge branch 'staging' into fix/desktop-pw-reset

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.
d9b5cc02by Julian+1275−21923 files
No security note in commit
Low 34 AI analysisMessage 83 · Strong
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

Merge pull request #1439 from navidR/dev/navidr/spark-name-verification

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
a5411a50by Julian+1097−16616 files
No security note in commit
Low 39 AI analysisMessage 60 · Adequate
SW Stack WalletStack Wallet MoneroPrivacy protocolsSoftware wallets

Merge branch 'staging' into dev/navidr/spark-name-verification

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
92955848by Julian+122−507 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 queuedspark view only wallet electrumx cache clear option fixby julian · 4907ba1e · Nov 25, 2025 · 5 filesMessage 50 · ThinInformational 17Details
Commit message · julian

spark view only wallet electrumx cache clear option fix

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 is a small UI fix for the Stack Wallet app. It changes when the 'Clear electrumx cache' button is shown for Spark (Firo) wallets so that it also appears for certain 'view-only' wallets. The change mostly reorganizes on-screen settings and updates a few warning messages. There is no direct evidence in the commit of a security vulnerability being fixed.

AI review queuedspark transparent out limit increaseby julian · f8c94903 · Nov 25, 2025 · 1 fileMessage 45 · ThinInformational 19Details
Commit message · julian

spark transparent out limit increase

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 raises the per-transaction cap on sending Firo from a Spark (privacy) address to a regular transparent address from 10,000 Firo to 50,000 Firo. It is a configuration/limit change, not a code flaw. There is no indication it fixes a vulnerability; it simply aligns the wallet with a higher protocol-level limit.

AI review queuedchore: cleanupby sneurlax · 658ff805 · Nov 25, 2025 · 1 fileMessage 30 · OpaqueLow 26Details
Commit message · sneurlax

chore: cleanup

30/100 · OpaqueMessage clarity
✓ Subject identifies a change✓ Uses a recognizable type or scope! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Low 26/100

This commit simplifies how the wallet decides whether a Solana token uses the older SPL token standard or the newer Token-2022 standard. It replaces a precise check of the exact program address with a looser check: anything whose owner address starts with 'Token' is treated as Token-2022. It also removes an unused helper function for finding token metadata addresses and strips out some debug print statements. The change is described by the developer as a 'cleanup' with a low-priority TODO to improve the detection later.

AI review queuedupdate mwebdby julian · 0aea1b2f · Nov 24, 2025 · 3 filesMessage 18 · OpaqueInformational 11Details
Commit message · julian

update mwebd

18/100 · OpaqueMessage clarity
✓ Subject identifies a change! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: opaque commit messagesecond-pass: security-sensitive path
AI analysis · Informational 11/100

This commit updates a dependency called flutter_mwebd from version 0.0.1-pre.10 to 0.0.1-pre.11 and adds one debug log line when preparing a Mimblewimble (MWEB) cryptocurrency transaction. There is no description of why the dependency was updated or what changed in the new version. The added log line prints transaction preparation data, which could include sensitive details, but the commit itself does not show a fix for a clear security vulnerability.

AI review queuedIncrease spark mint tx size limit up to 250kbby julian · acd9de2f · Nov 24, 2025 · 1 fileMessage 45 · ThinInformational 18Details
Commit message · julian

Increase spark mint tx size limit up to 250kb

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

This commit increases the maximum allowed transaction size/weight for Spark mint transactions in the Stack Wallet app from 400,000 weight units to 1,000,000 weight units (effectively raising the size cap from roughly 100 KB to 250 KB). It is a configuration change that aligns the wallet with a recent Firo protocol update. There is no direct evidence in the commit that this fixes an exploitable vulnerability; it appears to be a compatibility/functional update to allow larger valid Spark mint transactions.

AI review queuedchore(spl): remove prints, log what's important, remove unnecessary docsby sneurlax · a1e4e84a · Nov 21, 2025 · 3 filesMessage 62 · AdequateInformational 15Details
Commit message · sneurlax

chore(spl): remove prints, log what's important, remove unnecessary docs

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 cleanup of Solana token wallet code. It removes debug print statements, trims verbose logging, and deletes lengthy documentation comments. There are no functional changes to how the wallet handles money, keys, or network calls, and no security bug is introduced or fixed.

AI review queuedfeat(spl): implement Token-2022 transfersby sneurlax · 75427be6 · Nov 21, 2025 · 2 filesMessage 57 · ThinLow 37Details
Commit message · sneurlax

feat(spl): implement Token-2022 transfers

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

This commit adds support for Solana Token-2022 transfers in Stack Wallet. It changes how the wallet detects which token program a token uses, validates sender and recipient token accounts before sending, and switches from a simple transfer instruction to a 'transferChecked' instruction that includes the token's mint and decimal count. The commit also replaces a placeholder address-derivation method with a real Solana Associated Token Account (ATA) derivation. The changes are mostly defensive, but they introduce new logic that relies on RPC responses and string matching on program IDs, which could be a source of bugs or security issues if not fully correct.

AI review queuedduo splash imageby julian · 31ca8592 · Nov 21, 2025 · 1 fileMessage 28 · OpaqueInformational 15Details
Commit message · julian

duo splash image

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 adds a single image file—a splash screen graphic for 'Stack Duo'—and changes no executable code. There is nothing here that affects security.

AI review queuedmweb thingsby julian · 80539b3d · Nov 21, 2025 · 1 fileMessage 0 · OpaqueLow 31Details
Commit message · julian

mweb things

0/100 · OpaqueMessage clarity
! Very short subject! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: opaque commit messagesecond-pass: security-sensitive path
AI analysis · Low 31/100

This commit updates the Mimblewimble (Mweb) wallet code in Stack Wallet. It adds a one-time scan of older transactions when restoring a wallet, fixes a bug where spent Mweb coins were not always marked as used, and changes how transaction fees are calculated. The fee-calculation change removes a previous workaround that added an extra 1 satoshi to every fee, which could have caused users to slightly overpay. There is no clear security vulnerability in the diff, but the changes touch sensitive wallet logic (secret scanning, UTXO state, and fee math), so bugs here could affect funds or privacy.

AI review queuedfix(spl): init rpc even when you skip opening the parent walletby sneurlax · 5b97929f · Nov 21, 2025 · 2 filesMessage 77 · AdequateLow 35Details
Commit message · sneurlax

fix(spl): init rpc even when you skip opening the parent wallet

applicable if the user selects "only sync selected wallets"

77/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 35/100

This commit fixes a bug in Stack Wallet's Solana token wallet support. When a user chose to sync only selected wallets and skipped opening the main Solana wallet, the token wallet would not initialize its connection to the Solana network (RPC client). The fix makes the token wallet explicitly initialize that connection during its own startup, so token balances and transactions can work even when the parent wallet was not opened first.

AI review queuedfix(spl): delegate chain refresh to parent walletby sneurlax · 553c2a84 · Nov 21, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · sneurlax

fix(spl): delegate chain refresh to parent wallet

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

A one-line change in the Solana token wallet code now asks the main Solana wallet to refresh the blockchain height instead of doing nothing. Previously, the token wallet's chain-height update was left as an unfinished 'to-do' task, which could have caused stale or missing chain-height information for token accounts. There is no direct evidence this was a security vulnerability, but relying on a placeholder no-op for chain-height updates is a reliability weakness.

AI review queuedui(spl): show tokens after sol wallet creation or restore on mobileby sneurlax · 6049da32 · Nov 21, 2025 · 2 filesMessage 50 · ThinInformational 15Details
Commit message · sneurlax

ui(spl): show tokens after sol wallet creation or restore on mobile

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 minor user-interface tweak for the Stack Wallet mobile app. After a user creates or restores a Solana wallet, the app now automatically shows the token-management screen, just as it already did for Ethereum wallets. There is no security issue here—only a small convenience improvement.

AI review queuedfix(spl): mobile fee selection ui fixby sneurlax · fe1fdf6e · Nov 21, 2025 · 2 filesMessage 57 · ThinLow 32Details
Commit message · sneurlax

fix(spl): mobile fee selection ui 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: security-sensitive path
AI analysis · Low 32/100

This commit fixes two mobile UI/behavior bugs in the Solana token send flow. First, it delays setting the default fee speed until after the screen is built, preventing a possible crash or inconsistent state. Second, it corrects which wallet object is used to estimate fees for non-Ethereum tokens (like Solana tokens), so fee estimates come from the right wallet instead of the currently selected Ethereum token wallet. The commit does not describe these as security issues, but using the wrong wallet for fee estimates could in theory lead to incorrect fees being shown or used.

AI review queuedfeat(spl): fee selectionby sneurlax · 977222dc · Nov 21, 2025 · 3 filesMessage 40 · ThinInformational 19Details
Commit message · sneurlax

feat(spl): fee selection

TODO dynamic estimation via api

40/100 · ThinMessage clarity
✓ Subject identifies a change✓ 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 user-facing fee selection feature for Solana and Solana token (SPL) sends in Stack Wallet. It replaces a hardcoded placeholder fee with live fee estimates based on network conditions, offering slow/average/fast tiers. There is no direct evidence this fixes a security vulnerability; it is a feature/UX improvement.

AI review queuedui(spl): mobile sol token wallet navby sneurlax · 61e01022 · Nov 20, 2025 · 2 filesMessage 60 · AdequateInformational 19Details
Commit message · sneurlax

ui(spl): mobile sol token wallet nav

fix(spl): fix navigation to token sub-wallet

60/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 19/100

This commit fixes how the mobile app navigates to Solana token wallets. Previously, tapping a Solana token card may have opened the wrong wallet view or skipped the token-specific screen. The change makes sure Solana tokens open their dedicated token view and that the wallets overview is shown when a Solana wallet has tokens. There is no indication this is a security fix; it appears to be a user-interface/navigation bug fix.

AI review queuedui(spl): desktop sol token wallet navby sneurlax · 40300024 · Nov 20, 2025 · 5 filesMessage 45 · ThinInformational 19Details
Commit message · sneurlax

ui(spl): desktop sol token wallet nav

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 adds desktop user-interface support for Solana SPL tokens in Stack Wallet. It lets users view and navigate to Solana token wallets the same way they already could for Ethereum tokens. The changes are almost entirely UI wiring and data-model generalization; there is no obvious security vulnerability in the diff, but the code does add new database lookups and token-loading paths that could later harbor bugs if the underlying token data is malformed or untrusted.

AI review queuedui(spl): desktop token details viewby sneurlax · 75895a25 · Nov 20, 2025 · 2 filesMessage 45 · ThinInformational 15Details
Commit message · sneurlax

ui(spl): desktop token details view

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 user-interface change for the desktop version of Stack Wallet. It makes the token name/header area on two desktop token detail pages respond to taps/clicks by opening a contract details screen. There is no indication this fixes or introduces a security problem.

AI review queuedduo splash imageby julian · 1a063db1 · Nov 20, 2025 · 1 fileMessage 28 · OpaqueInformational 15Details
Commit message · julian

duo splash image

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 adds a single new image file, a splash screen graphic for 'Stack Duo'. There is no code change and no apparent security relevance.

AI review queuedtxv2by julian · 234af02f · Nov 19, 2025 · 1 fileMessage 0 · OpaqueInformational 15Details
Commit message · julian

txv2

0/100 · OpaqueMessage clarity
! Very short subject! Too few words to establish purpose! 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 adds a single Dart getter that sets the Isar database transaction version to 2 for Solana token wallets. There is no visible security-relevant change in behavior, no bug fix, and no change to how funds, keys, or transactions are handled. It appears to be a schema or versioning update.

AI review queuedvariousby julian · 5d6a0ca4 · Nov 19, 2025 · 4 filesMessage 0 · OpaqueInformational 21Details
Commit message · julian

various

0/100 · OpaqueMessage clarity
! Very short subject! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: opaque commit messagesecond-pass: security-sensitive path
AI analysis · Informational 21/100

This commit is a code-quality refactor for Solana token handling in the Stack Wallet app. It replaces loose 'dynamic' types with concrete SplToken/EthContract types, removes unsafe casts, and makes the Solana token wallet constructor carry a full token object instead of separate fields. There is no direct evidence of a security vulnerability being fixed, but the changes remove patterns (dynamic casts and type confusion between Ethereum and Solana tokens) that can lead to crashes or incorrect balance/price display.

AI review queuedclean up token iconsby julian · 94cac105 · Nov 19, 2025 · 2 filesMessage 28 · OpaqueInformational 18Details
Commit message · julian

clean up token icons

28/100 · OpaqueMessage clarity
✓ Subject identifies a change! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 18/100

This commit tidies up how token icons are displayed in the Stack Wallet app. It switches Ethereum and Solana fallback icons from app-asset files to local files, adds a loading spinner while network icons load, and replaces a silent error with a logged error. There is no obvious security vulnerability here; it is primarily a UI cleanup.

AI review queuedhousekeeping (build runner, etc)by julian · 5832233c · Nov 19, 2025 · 12 filesMessage 35 · OpaqueInformational 15Details
Commit message · julian

housekeeping (build runner, etc)

35/100 · OpaqueMessage clarity
✓ Descriptive subject! 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 is routine housekeeping: it regenerates automatically generated Dart files (for database models and test mocks), updates a .gitignore rule, and tweaks a coin icon provider. There are no user-facing security fixes or obviously dangerous code changes in the diff itself.

AI review queuedremove unnecessary logging callby julian · 2cdfec96 · Nov 19, 2025 · 1 fileMessage 35 · OpaqueInformational 15Details
Commit message · julian

remove unnecessary logging call

35/100 · OpaqueMessage clarity
✓ Descriptive subject! 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

A single debug logging line that printed pending transaction data has been removed. There is no indication this was a security vulnerability—it appears to be a routine cleanup of leftover debug output.

AI review queuedfeat(spl): mobile custom token flowby sneurlax · 9a48267a · Nov 19, 2025 · 3 filesMessage 57 · ThinInformational 18Details
Commit message · sneurlax

feat(spl): mobile custom token flow

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

This commit refines how the Stack Wallet mobile app handles Solana (SPL) custom tokens. It separates default tokens from user-added custom tokens, passes the wallet ID when adding a custom token, and adds unique keys to token list items. There is no clear security vulnerability in the diff itself; it appears to be a feature/bug-fix improvement for token management.

AI review queuedfeat(spl): custom tokens with validation and error handlingby sneurlax · 0ecc5fd5 · Nov 19, 2025 · 2 filesMessage 62 · AdequateLow 26Details
Commit message · sneurlax

feat(spl): custom tokens with validation and error handling

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

This commit adds a new screen that lets users manually add custom Solana SPL tokens to their Stack Wallet. It checks the mint address format, tries to fetch token details from an API, and lets the user type in details if the API has no record. The change also replaces the old fixed list of Solana tokens with this custom-add flow. There is no clear security bug in the diff, but the new code trusts user-supplied token metadata and does not verify on-chain that the token really exists or that the wallet holds it, which could let a user be tricked into adding a fake or misleading token.