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
999commits · 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 16 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 queuedtry to ensure only one pop occursby Julian · dc1cca5c · Nov 5, 2025 · 4 filesMessage 57 · ThinLow 29Details
Commit message · Julian

try to ensure only one pop occurs

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Explains rationale or failure mode! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 29/100

This commit fixes a bug where scanning a QR code could accidentally close the same screen multiple times in rapid succession. The change wraps the scan handler so it only runs once, and also cancels the camera stream before closing the screen. Repeated pops could cause navigation glitches or minor app instability, but there is no direct evidence it could be used to steal funds or keys.

AI review queuedreplace mobile_scanner with an older library that does not use MLKit on androidby julian · 8ae4f2e0 · Nov 5, 2025 · 18 filesMessage 50 · ThinLow 40Details
Commit message · julian

replace mobile_scanner with an older library that does not use MLKit on android

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

This commit swaps the app's QR code scanning library on Android from one that uses Google's ML Kit to an older one that does not. The change is framed as a privacy move to avoid sending camera frames to Google's ML services. The diff also adds null checks on scanned QR results, which prevents some crashes when a scan returns no content. There is no direct evidence in the commit that this fixes an active security vulnerability, and the bulk of the diff is formatting changes unrelated to security.

AI review queuedfeat(spl): add Solana token handling in wallet send/receive viewby sneurlax · 66da8bb3 · Nov 5, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · sneurlax

feat(spl): add Solana token handling in wallet send/receive view

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 adds a placeholder UI message ('WIP') for sending Solana tokens on the desktop wallet view. It does not implement actual token-sending logic, nor does it change any security-sensitive code. The change is purely a UI stub to avoid falling through to the generic token-send widget for Solana wallets.

AI review queuedbitcoin legacy address toggle optionby julian · 3e6291c5 · Nov 5, 2025 · 6 filesMessage 45 · ThinInformational 19Details
Commit message · julian

bitcoin legacy address toggle option

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 a user-facing toggle that lets Bitcoin wallet owners choose whether to generate older-style "legacy" (P2PKH) addresses. Previously, the app hid legacy addresses for Bitcoin wallets when other address types were available. The change is a feature addition, not a bug fix, and does not appear to address any security vulnerability.

AI review queuedfix(spl): handle missing Ethereum token wallet in shared componentsby sneurlax · c72c9c15 · Nov 5, 2025 · 3 filesMessage 62 · AdequateInformational 22Details
Commit message · sneurlax

fix(spl): handle missing Ethereum token wallet in shared components

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

This commit is a defensive bug-fix in the desktop wallet UI. It prevents the app from crashing when a Solana token wallet is selected, because the shared UI components were originally written only for Ethereum-style token wallets. The changes replace hard assumptions (that a token wallet always has an Ethereum contract object) with null checks, try/catch guards, and placeholder labels. There is no evidence of an exploitable vulnerability; it is a robustness improvement.

AI review queuedoptimize/cleanup electrumx_interface wallet addresses saved on recover/rescanby julian · dd67c858 · Nov 4, 2025 · 3 filesMessage 50 · ThinLow 31Details
Commit message · julian

optimize/cleanup electrumx_interface wallet addresses saved on recover/rescan

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

This commit refactors how Stack Wallet saves wallet addresses during wallet recovery and rescanning for ElectrumX-based coins. It centralizes repeated address-gap-checking logic into a single helper function and changes how the highest used address index is tracked. The change appears intended to fix a bug where unused addresses beyond the gap limit could be incorrectly stored, which could lead to missing transactions or an incomplete wallet balance after restore. There is no clear evidence in the commit of a traditional security vulnerability such as theft of funds, but a bug in address discovery could affect wallet correctness and user funds visibility.

Security candidatebasic message signingby julian · 8a0f78e6 · Nov 4, 2025 · 16 filesMessage 28 · OpaqueLow 28Details
Commit message · julian

basic message signing

28/100 · OpaqueMessage clarity
✓ Subject identifies a change! No meaningful explanatory body! Opaque security-relevant change
Why it was queued
signing boundarycryptography-sensitive pathsigning or wallet path
AI analysis · Low 28/100

This commit adds a new 'Sign/Verify message' feature to Stack Wallet, letting users cryptographically sign messages with their wallet keys and verify signatures from others. It also corrects the message-prefix byte lengths for Dogecoin, Firo, and Namecoin so signatures match what other software expects. There is no clear security bug in the diff, but the signing code is new and touches private keys, so it deserves careful review.

AI review queuedfeat(spl): add Solana token selection in wallet token editorby sneurlax · 71178e68 · Nov 4, 2025 · 1 fileMessage 62 · AdequateInformational 14Details
Commit message · sneurlax

feat(spl): add Solana token selection in wallet token editor

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

This commit adds the ability for users to select and add Solana-based tokens (SPL tokens) inside the wallet's token editor. It is a feature addition that branches the existing Ethereum-only token flow to also support Solana wallets. There is no direct evidence in the commit that this fixes or introduces a security vulnerability; it is primarily a UI and data-handling change.

AI review queuedfeat(spl): add desktop Solana token (SPL) detail viewby sneurlax · a25e51ae · Nov 3, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · sneurlax

feat(spl): add desktop Solana token (SPL) detail view

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 adds a new desktop screen for viewing Solana token (SPL) details in Stack Wallet. It is purely a user-interface feature with no security-sensitive code, no network calls, no cryptographic operations, and no changes to permissions or data handling. There is nothing in the diff that suggests a security vulnerability or fix.

AI review queuedfeat(spl): implement Solana token selectionby sneurlax · 4fe52e02 · Nov 1, 2025 · 4 filesMessage 57 · ThinInformational 19Details
Commit message · sneurlax

feat(spl): implement Solana token selection

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 adds the ability for users to select Solana-based tokens (SPL tokens) in the Stack Wallet app. It is a feature implementation, not a security fix. The new code includes placeholder/stub functions that do not yet perform real Solana blockchain checks, and it adds debug logging that prints wallet addresses and selected token details during development. There is no direct evidence in the commit of a vulnerability being fixed or introduced.

AI review queuedfeat(spl): add Solana token storage and state mgmt providersby sneurlax · ba5492e3 · Nov 1, 2025 · 3 filesMessage 62 · AdequateInformational 15Details
Commit message · sneurlax

feat(spl): add Solana token storage and state mgmt providers

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 adds new code to support Solana SPL tokens in Stack Wallet. It introduces a way to store token mint addresses in wallet metadata and two placeholder data providers for retrieving those addresses. The actual token detail fetching is left as a TODO and returns an empty list. There is no evidence of a security fix or vulnerability being addressed.

Security candidatefeat(spl): flag/enable token support for Solanaby sneurlax · b9914576 · Oct 31, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · sneurlax

feat(spl): flag/enable token support for Solana

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
cryptography-sensitive pathsigning or wallet path
AI analysis · Informational 15/100

This commit simply turns on a feature flag indicating that the Solana wallet now supports SPL tokens. There is no code change that fixes a bug, closes a security hole, or changes how funds are handled. It is a normal product feature enablement.

AI review queuedfeat(spl): add Solana token (SPL) state mgmt providersby sneurlax · 70b7f51f · Oct 31, 2025 · 2 filesMessage 62 · AdequateInformational 15Details
Commit message · sneurlax

feat(spl): add Solana token (SPL) state mgmt providers

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 adds two new Dart files that manage UI state for Solana token wallets. One tracks which token wallet is currently active, and the other returns a hard-coded mock balance of 1000.00 for development purposes. There is no security-sensitive code, no cryptographic handling, no network calls, and no user data processing. It is a routine feature addition with no apparent security relevance.

AI review queuedfeat(spl): add mock SolanaTokenWallet implby sneurlax · 716813ce · Oct 31, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · sneurlax

feat(spl): add mock SolanaTokenWallet impl

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

This commit adds a new placeholder/mock implementation of a Solana token wallet for user-interface development. The code is explicitly labeled as a mock, contains only stub methods, and does not perform real transactions, balance lookups, or fee handling. There is no indication of a security flaw in the added code.

AI review queuedstandalone mwebd for windowsby Julian · 30730b89 · Oct 31, 2025 · 14 filesMessage 35 · OpaqueLow 35Details
Commit message · Julian

standalone mwebd for windows

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

This commit changes how the Stack Wallet app handles a helper program called mwebd on Windows. Instead of using a built-in library, it now extracts a separate mwebd.exe file from the app bundle, writes it to disk, checks its SHA256 hash, and runs it as a standalone process. The change also updates build instructions and dependency versions. There is no claim in the commit that this fixes a security issue, but the new code introduces a few security-relevant patterns worth reviewing: it executes a bundled binary, relies on a compile-time hash check, and uses a dynamic object wrapper to hide platform-specific server types.

AI review queuedfix: mwc list positionby Julian · 28cacaab · Oct 30, 2025 · 1 fileMessage 40 · ThinInformational 15Details
Commit message · Julian

fix: mwc list position

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

This commit simply moves one line in a configuration script so that a cryptocurrency called Mimblewimblecoin appears in a different position within an internal list. There is no security-relevant change visible in the diff.

AI review queuedprevent opening of already opened xmr/wow/sal walletsby Julian · 076fd922 · Oct 29, 2025 · 9 filesMessage 50 · ThinLow 42Details
Commit message · Julian

prevent opening of already opened xmr/wow/sal wallets

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

This commit changes how Stack Wallet initializes Monero-style (XMR, WOW, SAL) wallets. During setup, it now briefly opens the wallet, updates network node settings, immediately closes it, and clears the reference. The stated goal is to avoid leaving wallets open when they are already opened elsewhere. The change is defensive and cleanup-oriented rather than a clear fix for an active attack.

AI review queuedfix linux wow headerby Julian · 3e137fde · Oct 29, 2025 · 1 fileMessage 28 · OpaqueInformational 15Details
Commit message · Julian

fix linux wow header

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 updates a single commented-out dependency version in a template file used to configure the Linux build. It is a routine version bump for an optional Wownero cryptocurrency library and contains no active code changes or security fixes.

Security candidatefix dependenciesby Julian · c7b0591a · Oct 29, 2025 · 2 filesMessage 18 · OpaqueInformational 10Details
Commit message · Julian

fix dependencies

18/100 · OpaqueMessage clarity
✓ Subject identifies a change! Too few words to establish purpose! No meaningful explanatory body! Opaque security-relevant change
Why it was queued
cryptography-sensitive path
AI analysis · Informational 10/100

This commit updates a single dependency version number for a Wownero-related Flutter library from 2.0.0 to 2.0.2 in a template file. There is no indication in the commit itself that this fixes a security vulnerability. It appears to be a routine dependency bump.

Security candidateWIP extract some kind of CryptonoteWallet interfaceby Julian · 025404b6 · Oct 28, 2025 · 26 filesMessage 30 · OpaqueInformational 18Details
Commit message · Julian

WIP extract some kind of CryptonoteWallet interface

30/100 · OpaqueMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body! Contains work-in-progress language! Opaque security-relevant change
Why it was queued
cryptography-sensitive pathsigning or wallet path
AI analysis · Informational 18/100

This commit is a work-in-progress code cleanup that introduces a shared 'CryptonoteWallet' interface for Monero-like wallets (Monero, Wownero, Salvium). It replaces repeated checks for individual wallet classes with checks against the new shared interface. There is no direct security fix here; it is a refactoring change that could slightly reduce future bugs by making the code more consistent, but it also changes behavior in a few places by broadening which wallets match certain conditions.

Security candidateseparated xmw and wow libs. Dirty dirty minimal get-it-running messby Julian · a260a93d · Oct 28, 2025 · 22 filesMessage 50 · ThinLow 28Details
Commit message · Julian

separated xmw and wow libs. Dirty dirty minimal get-it-running mess

50/100 · ThinMessage clarity
✓ Specific, 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 28/100

This commit is a large refactor that splits the Wownero (WOW) cryptocurrency support out of the shared Monero (XMR) library path and into its own dedicated library (cs_wownero). It updates wallet creation, address validation, fee estimation, and restore logic to use the new Wownero-specific code instead of the old combined Monero/Wownero code. The commit message calls it a 'dirty dirty minimal get-it-running mess,' suggesting it is a rough, temporary implementation rather than a polished security fix.

Security candidateupdate libs for ios 26 compatby Julian · 64f40768 · Oct 27, 2025 · 2 filesMessage 45 · ThinInformational 2Details
Commit message · Julian

update libs for ios 26 compat

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

This commit updates two internal cryptocurrency plugin libraries (flutter_libepiccash and flutter_libmwc) for compatibility with iOS 26. No code diff is available, and the commit message gives no indication of a security fix. It appears to be a routine platform-compatibility dependency bump.

AI review queuedupdate mwebd libby Julian · bed9d1b8 · Oct 27, 2025 · 2 filesMessage 28 · OpaqueLow 25Details
Commit message · Julian

update mwebd lib

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

This commit simply bumps the version of a third-party library called flutter_mwebd from an early pre-release version (0.0.1-pre.8) to a slightly newer pre-release version (0.0.1-pre.10). The change appears in dependency lock files only. There is no description of why the update was made, no mention of security fixes, and no diff showing what actually changed inside the library. On its own, this commit does not show a vulnerability, but updating a pre-release library can sometimes include bug or security fixes.

Security candidateupdate submodules with macos framework fixesby Julian · bf970e21 · Oct 27, 2025 · 3 filesMessage 45 · ThinInformational 2Details
Commit message · Julian

update submodules with macos framework fixes

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

This commit only updates three submodule pointers (flutter_libepiccash, flutter_libmwc, frostdart) with a message saying they contain 'macos framework fixes'. No actual code changes or diff content are available, and no verified references describe what those submodule fixes do. There is no direct evidence this is a security patch.

AI review queuedensure tor singletonby Julian · 2299b45d · Oct 25, 2025 · 1 fileMessage 28 · OpaqueLow 27Details
Commit message · Julian

ensure tor singleton

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

This commit changes the Tor privacy service in Stack Wallet so that only one instance of the service is created and reused, rather than creating a fresh one each time. This is a code-quality and reliability fix. It does not by itself fix a known exploit, but running multiple Tor service instances could previously have caused inconsistent privacy states, resource waste, or subtle bugs that might weaken privacy guarantees.