MG
← All projectsMAGIC Grants

Skylight Wallet

Modern open-source self-custody Monero light wallet using Monero LWS.

MoneroPrivacy protocolsSoftware walletsNormal
Repository coverage

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

39security candidates178second-pass queue217AI analyses
77commits · 30 days
98commits · 60 days
139commits · 180 days
292commits · 365 days
Backfill bands
Sep 27 → Mar 31160 seen18 candidatesComplete
Mar 31 → Jul 2928 seen4 candidatesComplete
Jul 29 → Aug 2834 seen3 candidatesComplete
Aug 28 → Sep 2748 seen9 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.

37/100 average clarity
0Strong · 80–100
10Adequate · 60–79
151Thin · 40–59
138Opaque · 0–39
17security 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.
Keeqler25135185236
Justin Ehrenhofer42428240
Licaon_Kter302034
SamsungGalaxyPlayer202035
jermanuts100045
Analysis record

Published AI watches

Last scanned 19 minutes ago

Low 32 AI analysisMessage 58 · Thin
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Merge pull request #178 from MAGICGrants/2.1.0-release-fixes

This is a routine version-2.1.0 bug-fix merge for the Skylight Monero wallet. The visible changes fix small packaging and platform-detection issues, add a new automated TLS test suite, and update pinned internal library versions. There is …

New native TLS integration test workflow covering all shipped platformsCA bundle asset handling moved into wallet-core (assets/cacert.pem removed from app asset list, copyCacertToAppDocumentsDir removed)Debian launcher LD_LIBRARY_PATH no longer includes empty trailing entry
320c02ceby Justin Ehrenhofer+383−362427 files
No security note in commit
Informational 3 AI analysisMessage 0 · Opaque
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Update pins

This commit only updates version numbers and the pinned Git commit references (called 'pins') for several software libraries the project depends on. No actual code in this repository was changed. The commit message simply says 'Update pins…

Dependency pin update to new commit hashes in external repositoriesNo source code changes in the skylight-wallet repository itselfNo commit message or in-diff indication of security relevance
7125d971by Justin Ehrenhofer+25−252 files
No security note in commit
Informational 0 AI analysisMessage 45 · Thin
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Merge branch 'main' into 2.1.0-release-fixes

This commit is a routine Git merge that brings the latest changes from the 'main' branch into a release-fixes branch. The only changed files are precompiled binary libraries for Monero wallet support on Android, iOS, Linux, and Windows. No…

50b25b5eby Justin Ehrenhofer+0−07 files
No security note in commit
Informational 18 AI analysisMessage 35 · Opaque
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Skip fetching unused submodules

This commit changes three build scripts so they only download two specific submodules ('monero' and 'lwsf') instead of all submodules. The stated reason is reliability: unused submodules for other coins can cause build failures when their …

Build script change limiting submodule checkout scopeReduced fetch of third-party dependencies during buildNo direct vulnerability or exploit mechanism introduced
8c5b00d3by Justin Ehrenhofer+9−33 files
No security note in commit
Informational 0 AI analysisMessage 58 · Thin
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Merge pull request #179 from MAGICGrants/update-moneroc-libs

This commit only updates precompiled Monero library files (binary .so and .dll files) across Android, iOS, Linux, and Windows. No source code changes are shown, and no description of what changed in the libraries is provided. We cannot det…

3df9967aby Justin Ehrenhofer+0−07 files
No security note in commit
Informational 0 AI analysisMessage 35 · Opaque
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Update monero_c libraries

This commit only updates precompiled Monero wallet library files (binary .so and .dll files) across Android, iOS, Linux, and Windows. No source code changes are shown, and no security-related information is provided in the commit title or …

2cf30607by SamsungGalaxyPlayer+0−07 files
No security note in commit
Informational 15 AI analysisMessage 28 · Opaque
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Fix desktop builds

This commit fixes broken build pipelines for Linux and Windows desktop releases. It pins the Rust toolchain version used during the Linux build and installs the NASM assembler on Windows so that a cryptography library can compile. There is…

e0eaa15fby Justin Ehrenhofer+11−22 files
No security note in commit
Low 34 AI analysisMessage 58 · Thin
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Merge pull request #176 from MAGICGrants/desktop-ui

This is a large feature merge that adds a desktop user interface, re-enables Linux and Windows release builds, and makes several Android build and security-related changes. The most notable security-relevant change is a fix in the Android …

Android MainActivity blocks route/deeplink intent injection by returning null initial route and disabling deeplink handlingAndroid build split into Play and FOSS source sets to keep Google Play review library out of F-Droid/GitHub APKsNew StoreReview method channels on Android and iOS
d6d9d318by Justin Ehrenhofer+3922−151454 files
Vendor flagged security relevance
Informational 15 AI analysisMessage 0 · Opaque
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Bump build

This commit only changes the app's version number in a configuration file, bumping it from 2.0.0+410 to 2.1.0+411. There are no code changes, no security fixes, and no behavior changes visible in the diff.

993a1147by Justin Ehrenhofer+1−11 file
No security note in commit
Low 46 AI analysisMessage 45 · Thin
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Updates for Monero 0.18.5.3

This commit updates the Skylight Wallet app to work with Monero 0.18.5.3, refreshes several internal library versions, re-enables Linux and Windows release builds, and adds two Android safeguards that prevent other apps or adb commands fro…

Exported Android MainActivity previously accepted route-bearing intents that could bypass App LockNew getInitialRoute() and shouldHandleDeeplinking() overrides neutralize route/deep-link injection on AndroidSubmodule/package bumps to monero_c and wallet-core may include undisclosed security fixes for Monero 0.18.5.3
5f5eea3eby Justin Ehrenhofer+37−296 files
No security note in commit
Informational 0 AI analysisMessage 58 · Thin
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Merge pull request #175 from MAGICGrants/update-moneroc-libs

This commit updates pre-compiled Monero wallet library files across Android, iOS, Linux, and Windows. The actual code changes are inside binary files, so the diff shows no readable source changes. There is no information in the commit titl…

424f9588by Justin Ehrenhofer+0−07 files
No security note in commit
Informational 0 AI analysisMessage 35 · Opaque
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Update monero_c libraries

This commit only updates precompiled Monero library files (binary .so and .dll files) across Android, iOS, Linux, and Windows. No source code changes are shown, and no commit message or vendor reference explains what changed in these libra…

1e620a30by SamsungGalaxyPlayer+0−07 files
No security note in commit
Informational 15 AI analysisMessage 50 · Thin
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Show fiat API failure as a warning triangle by the balance; aligns with Spice

This commit is a cosmetic user-interface change. It swaps a text-based fiat exchange-rate error message for a warning-triangle icon with a tooltip and shows the coin balance more clearly when the fiat rate is unavailable. There is no secur…

09609000by Keeqler+64−541 file
No security note in commit
Informational 15 AI analysisMessage 35 · Opaque
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Fix mobile screen transitions

This commit changes how screen transitions (animations) work in a mobile/desktop wallet app. It disables animated transitions on desktop entirely and keeps them only between navigation-bar screens on mobile. There is no security-relevant c…

9456bbbdby Keeqler+3−11 file
No security note in commit
Informational 18 AI analysisMessage 50 · Thin
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Fix create password screen showing up on mobile; format

This commit fixes a UI bug where mobile users were incorrectly shown a 'create wallet password' screen that should only appear on desktop. On mobile, the app now skips that screen and creates or restores the wallet directly, relying on the…

Flow change: mobile wallet creation/restoration bypasses app-level password screenMobile now relies on device app lock instead of an in-app passwordDuplicate-submission guard added via _committing flag
6483d8a1by Keeqler+163−729 files
No security note in commit
Informational 15 AI analysisMessage 73 · Adequate
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Merge remote-tracking branch 'origin/send-wallet-core' into brightness-fix

This commit is a routine merge that moves fiat-currency handling into a shared library and adds a 'switch amount unit' feature on the send screen. There is no security-relevant change visible in the diff.

1f4209d3by Keeqler+77−10512 files
No security note in commit
Informational 23 AI analysisMessage 58 · Thin
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Merge pull request #173 from MAGICGrants/review-prompt

This commit adds an in-app store review prompt. After a successful cryptocurrency send, it marks the user as eligible, and the next time they open the wallet home screen it may ask for a Google Play or App Store rating. The code deliberate…

Third-party SDK inclusion gated by build flavor (Google Play only)Install-source check before invoking Play review APIF-Droid reproducible-build compatibility via source-set exclusion and recipe deletion
6528c1cbby Keeqler+165−19 files
No security note in commit
Informational 17 AI analysisMessage 68 · Adequate
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Merge branch 'desktop-ui' into brightness-fix

This is a large merge commit that brings a new desktop user interface into the Skylight Wallet app. Most of the changes are UI layout, new desktop-specific screens, updated text strings, and build script tweaks. There is no obvious securit…

Large feature merge with 43 changed files and thousands of linesBuild script updates pinned appimagetool SHA256 and filenameNew desktop UI screens added; no security-critical logic visible
2932c7e0by Keeqler+3592−143843 files
No security note in commit
Informational 15 AI analysisMessage 0 · Opaque
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Bump build

This commit only increases the app's internal build number from 409 to 410 in a configuration file. There are no code changes, no bug fixes, and no security-related modifications visible in the diff.

dcb087efby Keeqler+1−11 file
No security note in commit
Informational 15 AI analysisMessage 0 · Opaque
MG MAGIC GrantsSkylight Wallet MoneroPrivacy protocolsSoftware wallets

Bump build

This commit only increases the app's internal build number from 408 to 409 in a configuration file. No code, dependencies, or security settings were changed. There is no security relevance.

e69ac4eeby Keeqler+1−11 file
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.

Lower-priorityLWS string improvements (#10)by Justin Ehrenhofer · 4dba7c99 · Nov 21, 2025 · 3 filesMessage 66 · AdequateTriage 0Details
Commit message · Justin Ehrenhofer

LWS string improvements (#10)

* WIP: LWS string improvements

* Generate l10n

---------

Co-authored-by: Keeqler <33733651+Keeqler@users.noreply.github.com>

66/100 · AdequateMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
Lower-priorityFee loading optimizations and handle insufficient balance when loading feesby Keeqler · b127d9c5 · Nov 20, 2025 · 1 fileMessage 50 · ThinTriage 0Details
Commit message · Keeqler

Fee loading optimizations and handle insufficient balance when loading fees

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Lower-priorityFix some dialogs too wide on desktopby Keeqler · 09aa0255 · Nov 13, 2025 · 2 filesMessage 45 · ThinTriage 0Details
Commit message · Keeqler

Fix some dialogs too wide on desktop

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI review queuedPrompt for wallet password on desktopby Keeqler · c6768610 · Nov 13, 2025 · 13 filesMessage 45 · ThinLow 44Details
Commit message · Keeqler

Prompt for wallet password on desktop

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: broader security terminologysecond-pass: security-sensitive path
AI analysis · Low 44/100

This commit adds a password prompt for the Skylight Wallet app when running on desktop computers (Linux, Windows, macOS). Previously, the app only used the phone's biometric/PIN lock, which isn't reliably available on desktop. The change makes desktop users create and enter a wallet password to protect their funds. It is a security improvement, not a vulnerability fix, though the password is held in memory while the app runs and is not persisted on desktop.

Security candidateFix some screens too wide on desktopby Keeqler · d033aa20 · Nov 11, 2025 · 3 filesMessage 45 · ThinInformational 15Details
Commit message · Keeqler

Fix some screens too wide on desktop

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
seed or entropy pathsigning or wallet path
AI analysis · Informational 15/100

This commit is a routine user-interface layout fix. It limits how wide three wallet setup screens can stretch on desktop computers by wrapping them in a centered container with a maximum width. There is no security-relevant change.

AI review queuedMake layouts adapt to larger screensby Keeqler · dfb2e047 · Nov 7, 2025 · 16 filesMessage 45 · ThinInformational 18Details
Commit message · Keeqler

Make layouts adapt to larger screens

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 is a routine UI polish for the Skylight Monero wallet. It makes screens adapt better to large monitors and tablets, adds a desktop copy-address button, fixes a macOS notification bug, and replaces a full-screen transaction details page with a popup dialog. There is no clear security vulnerability in the changes, but one small behavior change is worth noting: the old receive screen copied the address silently when tapped, while the new code shows a confirmation message. Most of the diff is reformatting and layout work.

AI review queuedMake features work on Linuxby Keeqler · 800d4f87 · Nov 5, 2025 · 16 filesMessage 45 · ThinLow 31Details
Commit message · Keeqler

Make features work on Linux

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

This commit adds Linux support to a Monero wallet app. Most changes are ordinary cross-platform plumbing, but two items deserve a closer look: a new helper script edits a binary shared library to clear an 'executable stack' flag, and the app now stores wallet files, Tor data, and logs in the user's home directory under a hidden folder on Linux. The commit itself does not describe these as security fixes, and there is no external advisory or researcher attribution supplied.

AI review queuedBump build numberby Keeqler · 2f07a7f1 · Oct 31, 2025 · 1 fileMessage 28 · OpaqueInformational 15Details
Commit message · Keeqler

Bump build number

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 only increases the app's build number from 3 to 4 in a configuration file. It makes no code changes and has no security relevance.

AI review queuedPotential fix for possible crash when wallet is deletedby Keeqler · 151f6625 · Oct 31, 2025 · 1 fileMessage 50 · ThinInformational 23Details
Commit message · Keeqler

Potential fix for possible crash when wallet is deleted

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: broader security terminologysecond-pass: security-sensitive path
AI analysis · Informational 23/100

This is a one-line defensive fix in a Dart wallet app. It prevents the app from notifying UI listeners about a connection change if the underlying wallet object has already been deleted. The change likely avoids a crash or inconsistent UI state when a wallet is removed, but it is not a security vulnerability that an attacker can directly exploit.

Security candidateAdd subaddresses limit reached checkby Keeqler · 11ee322e · Oct 31, 2025 · 8 filesMessage 45 · ThinLow 36Details
Commit message · Keeqler

Add subaddresses limit reached check

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
privacy or spend-authorization protocolsigning or wallet path
AI analysis · Low 36/100

This commit adds a safety check to a cryptocurrency wallet so that, when the server can no longer create new subaddresses, the app warns the user and falls back to reusing an already-used subaddress instead of silently failing or leaking privacy. It is a defensive/privacy-hardening change, not an obvious vulnerability fix, but it addresses a real privacy edge case: address reuse weakens transaction privacy.

Security candidateMinor refactoring and subaddress check result cachingby Keeqler · 791f0f67 · Oct 30, 2025 · 8 filesMessage 50 · ThinInformational 14Details
Commit message · Keeqler

Minor refactoring and subaddress check result caching

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
privacy or spend-authorization protocolseed or entropy pathsigning or wallet path
AI analysis · Informational 14/100

This commit is a routine cleanup of a Monero wallet app. It mainly moves repeated startup steps into a single helper function, caches whether the server supports subaddresses in local app storage, and improves loading indicators on the connection setup screen. There is no obvious security bug introduced, but the change is not purely cosmetic because it alters when and how the app checks server capabilities and stores that result.

AI review queuedBump build numberby Keeqler · 75dd1c91 · Oct 29, 2025 · 1 fileMessage 28 · OpaqueInformational 15Details
Commit message · Keeqler

Bump build number

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 only increases the app's build number from 2 to 3 in a configuration file. There are no code changes, no security fixes, and no functional changes of any kind.

Security candidateAdd subaddress support checkby Keeqler · 03c9f0c5 · Oct 29, 2025 · 11 filesMessage 35 · OpaqueLow 37Details
Commit message · Keeqler

Add subaddress support check

35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body! Opaque security-relevant change
Why it was queued
privacy or spend-authorization protocolsigning or wallet path
AI analysis · Low 37/100

This commit adds a feature that checks whether the user's chosen Monero light-wallet server supports subaddresses. Before this change, the app could show a subaddress even when the server did not support it, which could cause incoming transactions to be missed. The change also removes a stored user preference that always defaulted to showing the primary address, and instead auto-detects server capability and defaults to showing a subaddress when supported. The most notable security-relevant side effect is that the app now sends the wallet's secret view key to the light-wallet server during the check, and it does so over plain HTTP unless the user has enabled SSL/Tor. That is a privacy-sensitive design choice, but it is consistent with how Monero light wallets already operate.

AI review queuedAdd READMEby Keeqler · a7101ae7 · Oct 24, 2025 · 1 fileMessage 0 · OpaqueInformational 15Details
Commit message · Keeqler

Add README

0/100 · OpaqueMessage clarity
! Very short subject! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
documentation-only discountsecond-pass: opaque commit message
AI analysis · Informational 15/100

This commit only adds and updates the project's README file. It introduces no code changes, no configuration changes, and no functional modifications to the wallet application. There is no security relevance in the diff itself.

AI review queuedAdd loading spinner in seed screenby Keeqler · d7bb5eca · Oct 24, 2025 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · Keeqler

Add loading spinner in seed screen

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 adds a visual loading spinner to the wallet restore screen and prevents the restore button from triggering multiple restore attempts at once. It is a user-experience improvement, not a security fix.

AI review queuedBump build numberby Keeqler · 35ab4051 · Oct 23, 2025 · 1 fileMessage 28 · OpaqueInformational 15Details
Commit message · Keeqler

Bump build number

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 only changes the app's build number from 1 to 2 in a configuration file. It does not modify any code, fix any behavior, or affect security in any way.

AI review queuedAdd tx priority selectby Keeqler · a439703d · Oct 23, 2025 · 7 filesMessage 28 · OpaqueInformational 20Details
Commit message · Keeqler

Add tx priority select

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

This commit adds a transaction fee priority selector to a Monero wallet app. It lets users choose Low, Normal, or High transaction priority and shows estimated fees. The change also moves OpenAlias address resolution onto a background thread. There is no direct evidence in the commit that this fixes a security vulnerability; it appears to be a user-facing feature improvement.

AI review queuedAdd missing currency symbolsby Keeqler · 9f121c39 · Oct 22, 2025 · 2 filesMessage 35 · OpaqueInformational 15Details
Commit message · Keeqler

Add missing currency symbols

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 simply adds correct currency symbols (like £ for British pounds or ¥ for Japanese yen) to the wallet's display. It is a user-interface improvement with no security relevance.

AI review queuedFix not connecting after restoreby Keeqler · c32dfc54 · Oct 22, 2025 · 1 fileMessage 45 · ThinInformational 21Details
Commit message · Keeqler

Fix not connecting after restore

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

This commit fixes a bug where restoring a wallet from a recovery phrase left the wallet in a disconnected state. After the fix, the app automatically refreshes the wallet, loads account statistics, and connects to the network daemon once the restore completes. There is no indication this was a security vulnerability; it appears to be a functional bug fix for wallet connectivity.

Security candidateSet up android app bundle signingby Keeqler · 9597a310 · Oct 22, 2025 · 5 filesMessage 45 · ThinInformational 15Details
Commit message · Keeqler

Set up android app bundle signing

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

This commit is a routine build configuration change. It sets up proper Android release signing (so the app can be published on app stores) and updates the app's package name and copyright strings across Android, iOS, macOS, and Windows from placeholder/example values to the real project identity (MAGIC Grants / Skylight Wallet). There is no security vulnerability here.

AI review queuedMinor changesby Keeqler · df381bf7 · Oct 21, 2025 · 2 filesMessage 18 · OpaqueInformational 15Details
Commit message · Keeqler

Minor changes

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

This commit makes two tiny cleanups: it removes two harmless warning log messages that were printed when a background timer ran before a wallet was opened, and it removes the 'const' keyword from one UI padding value. There is no security issue here.

AI review queuedDemo modeby Keeqler · 93fedb26 · Oct 21, 2025 · 1 fileMessage 0 · OpaqueLow 27Details
Commit message · Keeqler

Demo mode

0/100 · OpaqueMessage clarity
! Very short subject! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Low 27/100

This commit adds a 'demo mode' to the wallet app. When the app is built with a special environment flag and the user types 'demo' as the server address, the app skips the real network connection test and pretends it succeeded. This is a development/testing convenience, but it could let someone bypass the actual connection check if they can trigger or build the app in demo mode.

AI review queuedFix minor memory leak after wallet is deletedby Keeqler · 3cae1bff · Oct 17, 2025 · 1 fileMessage 45 · ThinInformational 23Details
Commit message · Keeqler

Fix minor memory leak after wallet is deleted

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

This commit fixes a minor memory leak that could occur after a wallet is deleted in the Skylight Wallet app. It adds safety checks so background timer tasks stop trying to work with a wallet that no longer exists, and it prevents an error in one stats-loading task from crashing the whole refresh process. The changes are defensive and improve stability, but they do not appear to be a security fix for an actively exploitable vulnerability.

AI review queuedWipe address book and pending outgoing txs list when wallet is deletedby Keeqler · a3f079fb · Oct 17, 2025 · 1 fileMessage 50 · ThinLow 46Details
Commit message · Keeqler

Wipe address book and pending outgoing txs list when wallet is deleted

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

This commit fixes a cleanup bug: when a user deletes their wallet, the app now also removes the saved address book (contacts) and any pending outgoing transactions from phone storage. Before this fix, those details could remain on the device after the wallet was supposedly deleted, which is a privacy and data-retention issue rather than a direct money-stealing bug.

Lower-priorityFix color of loading iconsby Keeqler · 8c67dd70 · Oct 17, 2025 · 4 filesMessage 45 · ThinTriage 0Details
Commit message · Keeqler

Fix color of loading icons

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body