Every captured commit receives deterministic security triage and a separate communication-quality score. Security candidates and broader second-pass signals receive full-patch Ollama analysis.
Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.
This commit removes the SHA-1 hashing function from the regular Trezor firmware. SHA-1 is an old, weak hash algorithm that is no longer considered secure for sensitive uses. The change deletes the code that exposes SHA-1 to apps running on…
Removal of a deprecated cryptographic primitive (SHA-1) from the firmware API surfaceReduction of attack surface and prevention of future misuse of a collision-vulnerable hashNo direct vulnerability patch or memory-safety bug is present in the diff
This commit changes how a cryptographic library sets up secure connections. Previously, the code automatically calculated the public key from the private key. Now, the caller must provide the public key directly. This is a code-quality ref…
Cryptographic key handling changed: public key is now supplied rather than derivedPotential reduction of key-mismatch risk if caller provides correct public keyNew null-pointer check added for static_public_key
This commit simply reorders the arguments of an internal function called dh() and updates every place that calls it. The actual math and security behavior are unchanged; it is a code cleanup with no security effect.
This commit simply renames two groups of internal status labels (called enums) in the Trezor firmware's cryptographic code. The old names like WAITING_FOR_REQUEST1 were shared between two different parts of the code, so the developer gave …
This commit fixes three places in the Trezor firmware's cryptographic code where a memory-clearing function could be called with a NULL pointer. In practice, passing NULL to memzero is harmless on Trezor's platform (it does nothing), but i…
NULL pointer passed to memory-zeroing helper in cryptographic codeUndefined behavior in C standard library contractDefensive hardening in Noise protocol implementation
This commit is a code cleanup (refactor) for the cryptographic handshake code used in Trezor devices. It changes how the other party's long-term public key is returned to the caller: instead of storing it inside an internal state structure…
Removal of long-term public key storage from internal handshake stateCaller-supplied output buffer for remote static public key reduces internal secret retentionError-path memzero of returned key material on failure
This commit is a simple rename from 'ProjectProfile' to 'ProjectConfig' in two Rust source files. It only changes variable names, struct names, and error messages. There is no change to program logic, security behavior, or how data is hand…
This commit is a straightforward internal code refactor in Trezor's build tooling. It introduces a new ResolvedBuildArgs structure that centralizes default values for command-line build options before they are used. There is no change to u…
This commit is a straightforward internal refactor of the Trezor firmware build tool's command-line argument handling. It changes many on/off flags from plain booleans to optional booleans so they can explicitly be set to true, false, or l…
This commit is a straightforward internal code reorganization in Trezor's build tooling. It moves build-option definitions from one Rust source file to a new module and adds the ability to load those options from a configuration file. Ther…
This commit is a developer tooling change for the Trezor firmware build system. It introduces 'build presets'—named configuration bundles stored in TOML files—so developers can select common build settings with a single command-line flag i…
This commit is a routine update to a build-system manifest file used by Trezor's internal tooling. It reorganizes and re-labels feature flags (for example, moving options between 'Features', 'Debugging', and 'Signing' groups and simplifyin…
This commit adds a single configuration file for a Trezor developer tool (a VS Code extension). It only points the tool to existing folders in the repository and contains no executable code, no secrets, and no changes to firmware behavior.…
This commit is a simple internal code cleanup: it renames a Rust data structure called ResolvedBuild to ResolvedBuildFeatures and updates the places that use it. There are no functional changes, no security fixes, and no changes to how the…
This is a tiny code cleanup commit that replaces two calls to `ptr::null()` with Rust's `unwrap_or_default()` to silence automated Clippy lint warnings. It does not change what value is produced (still a null pointer when no salt is provid…
This commit is a straightforward internal code cleanup in Trezor's build tooling. It moves a data structure and two helper methods from one Rust source file to another and updates callers accordingly. There is no change to user-facing beha…
This commit only adds documentation. It updates the table of contents and adds a new markdown page plus a diagram explaining the project's 'xtask' build helper. No code, build scripts, or firmware logic was changed, so it cannot introduce …
This commit is a build-system refactoring for Trezor firmware. It moves the mapping of command-line build options to Rust/cargo features out of hard-coded Rust logic and into per-project TOML files. It also adds a small change so debug bui…
Build-option mapping now lives in project.toml files, increasing the attack surface for supply-chain/build-configuration tamperingMakefile change automatically enables debug-link for PYOPT=0 debug buildsValidation added to reject storage_insecure_testing_mode in production builds
This commit fixes a filename-renaming step in the project's automated nightly build pipeline. It ensures ARM64 emulator files get the correct name expected by another testing tool. There is no change to the actual Trezor firmware code, dev…
This is a routine housekeeping change to reduce clutter in Git's status output for a third-party component (the MicroPython submodule). There is no indication it changes any code that runs on the Trezor device or affects security.
chore(core/eckhart): overlap vertical menu and header
- Overlap the first button top padding with header when no subtitle is present [no changelog]
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
boot or update path
AI analysis · Informational 15/100
This is a user-interface layout tweak for the Trezor hardware wallet's new 'Eckhart' design. It makes the first menu button visually overlap the screen header when there is no subtitle, and adds logic so the header is drawn on top if its buttons are being pressed. There is no security-relevant change here.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 17/100
This commit adds a new feature to read pre-stored digital certificates from the Tropic secure chip inside Trezor hardware wallets. It does not change how keys are generated or signed, and it does not appear to fix a known bug or vulnerability. The new code is mostly about safely reading fixed-size data slots from the chip and exposing them to Python applications.
Security candidatefeat(core): Use correct pairing keys in Tropic handshake.by Andrew Kozlik · 41cd4bbf · Sep 18, 2025 · 2 filesMessage 62 · AdequateModerate 57Details
Commit message · Andrew Kozlik
feat(core): Use correct pairing keys in Tropic handshake.
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
cryptography-sensitive path
AI analysis · Moderate 57/100
This commit fixes how a Trezor hardware wallet selects cryptographic pairing keys when establishing a secure session with the Tropic secure chip. Previously the code always used a single fixed key slot (likely a factory/unprivileged key). After the change, it tries to use a privileged pairing key if available, and falls back to an unprivileged key if not. It also adds a small delay to avoid a 'chip busy' error from the Tropic chip. The change is security-relevant because using the wrong pairing key could weaken the secure channel or cause it to rely on a less trusted factory key, but the commit itself does not describe an active vulnerability or attack.
feat(python): Check Tropic fields to AuthenticityProof.
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
authentication path
AI analysis · Low 26/100
This commit updates the Trezor Python library's device-authentication code to support a new hardware security chip (Tropic) in addition to the existing Optiga chip. It adds Ed25519 signature verification, new root public keys for the T3W1 device, and a check that both chips' certificate chains are issued by the same root authority. The change is a feature addition rather than a fix for a known vulnerability, but it strengthens the authentication process by making it harder for a device with a valid certificate from one root to pass authentication using a mismatched certificate from another root.
Security candidatefeat(core): Use Tropic in AuthenticateDevice.by Andrew Kozlik · 1940bd56 · Sep 18, 2025 · 3 filesMessage 57 · ThinLow 26Details
Commit message · Andrew Kozlik
feat(core): Use Tropic in AuthenticateDevice.
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
authentication path
AI analysis · Low 26/100
This commit updates the Trezor hardware wallet's device-authentication feature so it can optionally use a second secure chip (Tropic) in addition to the existing Optiga chip. It refactors certificate parsing into a helper and renames the proof fields from generic names to optiga_* and tropic_*. There is no direct evidence in the commit that this fixes a security vulnerability; it appears to be a feature addition enabling a new hardware variant.
refactor(core): Move Tropic constants to tropic.h.
[no changelog]
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
cryptography-sensitive path
AI analysis · Informational 15/100
This is a routine code cleanup that moves Tropic chip constants (like which key slot is used for what) into a shared header file so multiple parts of the firmware can use the same definitions. It also updates the Python binding for signing to use the correct Ed25519 signature size and exposes two key-slot constants to Python. There is no security vulnerability here.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
boot or update path
AI analysis · Informational 11/100
This commit swaps out two pre-built binary firmware files for the T3W1 hardware model and updates the matching cryptographic hash values in a header file. The change is described as a routine chore with no changelog entry. There is no visible source code change, no security explanation, and no disclosed vulnerability.
Security candidatefeat(core/bootloader): stop charging indication when the battery is fully chargedby tychovrahe · 3d6c4354 · Sep 17, 2025 · 2 filesMessage 62 · AdequateInformational 15Details
Commit message · tychovrahe
feat(core/bootloader): stop charging indication when the battery is fully charged
[no changelog]
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
update trustboot or update path
AI analysis · Informational 15/100
This commit changes the bootloader's battery-charging LED behavior. When the battery is fully charged, the charging LED effect now turns off instead of continuing to show a charging animation. It also adds a small helper function for the Unix emulator build so it can compile. There is no security issue visible in this change.
Security candidatefix(core): differentiate wipe code from pin set upby Ioan Bizău · 08ea80c9 · Sep 17, 2025 · 23 filesMessage 62 · AdequateLow 28Details
Commit message · Ioan Bizău
fix(core): differentiate wipe code from pin set up
[no changelog]
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
access controlcryptography-sensitive pathsigning or wallet pathboot or update pathauthentication path
AI analysis · Low 28/100
This commit fixes a user-interface labeling bug where setting up a 'wipe code' (a special code that erases the device) was incorrectly shown using the same on-screen text and flow as setting up a regular PIN. The change renames the confirmation flow, adds a flag to distinguish wipe-code setup from PIN setup, and shows wipe-code-specific cancellation text on newer Trezor layouts. It is a UI clarity fix, not a cryptographic or access-control flaw.
chore(core/eckhart): make wipe code dependent on pin code
[no changelog]
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
access control
AI analysis · Low 26/100
This small change adjusts when a Trezor device shows that a 'wipe code' is set. Previously, the device menu would display the wipe-code status if the device was initialized, regardless of whether a PIN was set. After the change, the wipe-code status is only shown when both the device is initialized and a PIN is set. This is a UI/logic hardening change: it prevents a user from being misled into thinking a wipe code is active and usable when no PIN is set, because the wipe code feature depends on the PIN entry flow to trigger.
chore(core): ensure dependency between PIN and wipe code
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
access controlcryptography-sensitive pathsigning or wallet pathauthentication path
AI analysis · Low 47/100
This commit tightens the rules between a Trezor device's PIN and its 'wipe code' (a special code that erases the device). Previously, it was possible to end up in an inconsistent state: for example, you could set a wipe code even when no PIN was set, or remove the PIN while the wipe code was still active. The patch blocks both situations and shows explanatory on-screen messages. It also replaces some internal 'assert False' statements with explicit error raises, which is a defensive cleanup. The change is a hardening fix rather than a remote exploit.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
boot or update path
AI analysis · Informational 15/100
This commit updates the legal/regulatory information screen shown on a Trezor hardware wallet. It adds Japan to the list of regions, includes a new Japan icon, and adds the standard FCC compliance disclaimer text. There is no security-relevant change.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
boot or update path
AI analysis · Informational 15/100
This commit simply moves two small power-management files from one folder to another and updates the build scripts and import paths accordingly. The actual code behavior is unchanged. There is no security fix or vulnerability here.
it eats some functions from apps.base, and introduces an asynchronous notification mechanism for suspending via `notify_suspend()`
70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
Why it was queued
authentication path
AI analysis · Low 27/100
This commit is a code reorganization: it moves device locking, unlocking, homescreen selection, and suspend-related logic out of apps/base.py into a new module called apps/common/lock_manager.py. It also changes how the power button triggers suspend, using a new asynchronous notification mechanism (notify_suspend) instead of immediately locking the device. There is no direct evidence in the diff that this fixes a security vulnerability; it appears to be a feature/refactoring change.
Security candidatefeat(core/eckhart): two types of auto lock delay in device menuby Lukas Bielesch · 10078286 · Sep 16, 2025 · 17 filesMessage 62 · AdequateInformational 15Details
Commit message · Lukas Bielesch
feat(core/eckhart): two types of auto lock delay in device menu
[no changelog]
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
cryptography-sensitive pathsigning or wallet pathboot or update path
AI analysis · Informational 15/100
This commit is a user-interface feature change for the Trezor hardware wallet. It splits the single 'auto-lock delay' setting in the device menu into two separate entries: one for when the device is on battery/wireless charger and one for when it is connected to USB. There is no indication in the commit that this fixes a security bug or changes any security-critical behavior; it is purely a UI/UX update.
Security candidatefix(core): setup RGB LED also when PIN not setby obrusvit · 8a14db13 · Sep 16, 2025 · 1 fileMessage 57 · ThinInformational 18Details
Commit message · obrusvit
fix(core): setup RGB LED also when PIN not set
[no changelog]
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
access control
AI analysis · Informational 18/100
This commit fixes a small inconsistency in the Trezor hardware wallet's startup screen. When a user had not set a PIN, the device's RGB LED (colored status light) was not being configured according to the user's preference. The change makes sure the LED setting is applied in that case too. There is no direct security vulnerability here; it is a user-experience and settings-consistency fix.
Security candidatefeat(core): fade backlight even in case a background process is still runningby matejcik · 0aedc0c6 · Sep 16, 2025 · 1 fileMessage 85 · StrongInformational 23Details
Commit message · matejcik
feat(core): fade backlight even in case a background process is still running
when autolock_interrupts_workflow is False and we need to wait for e.g. transaction signing to finish before we can go to suspend
85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
Why it was queued
signing boundary
AI analysis · Informational 23/100
This small change moves where the screen backlight fade happens in the Trezor hardware wallet's lock/suspend logic. Previously, the screen dimmed only inside the suspend function. Now it also dims earlier when the device decides it should suspend but is still waiting for a background task (like signing a transaction) to finish. The goal is to avoid leaving the screen bright while the user is waiting for a long operation before the device can lock. This is a user-experience and minor privacy improvement, not a fix for a serious security flaw.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
boot or update path
AI analysis · Informational 3/100
This commit swaps in two updated binary firmware files for the Trezor T3W1 hardware wallet (a QA bootloader and a Bluetooth development binary) and updates the corresponding cryptographic hash list. The actual code inside the binaries is not shown, and there is no description of why the binaries changed. On its own, the commit does not contain any visible security bug or fix.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
boot or update path
AI analysis · Moderate 58/100
This commit adds a new security feature to Trezor hardware wallets that checks whether the Bluetooth chip (nRF) is running authentic, manufacturer-approved firmware before the main wallet software starts up. In production builds, if the Bluetooth chip fails this authenticity check, the device now shuts down with an error instead of continuing to boot. The change also separates the authentication code from general Bluetooth support so it can be enabled independently.
feat(core): send lock&unlock notifications when hard-locking
[no changelog]
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
boot or update path
AI analysis · Informational 22/100
This commit adds a notification system so the Trezor device can tell the connected computer when it becomes 'unlocked' (ready to accept commands) and 'locked' (hard-locked). It also exposes a way for the device's Python code to send these notifications. There is no obvious security bug in the change itself, but the new MicroPython binding does not validate that the event number is one of the defined values, so a bug or compromised app running inside the device could pass an out-of-range value to the underlying C notification function.
chore(core/bootloader): unify menu items with firmware
[no changelog]
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
update trustboot or update path
AI analysis · Informational 15/100
This commit simply renames two on-screen menu buttons in the bootloader user interface: 'Reboot' becomes 'Restart' and 'Power off' becomes 'Turn off'. There is no change to security logic, no bug fix, and no vulnerability addressed.
Security candidatefeat(eckhart): tutorial power screenby Ioan Bizău · 0e625a94 · Sep 15, 2025 · 10 filesMessage 57 · ThinInformational 15Details
Commit message · Ioan Bizău
feat(eckhart): tutorial power screen
[no changelog]
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 pathboot or update path
AI analysis · Informational 15/100
This commit adds a new tutorial screen to the Trezor hardware wallet's user interface that explains how to use the device's physical power button. It is a straightforward feature addition with no security relevance.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
boot or update path
AI analysis · Low 27/100
This commit adds a new device-to-host notification feature to Trezor hardware wallets. When the device boots (either in normal firmware or bootloader mode), it can now send a small BLE notification to a connected host announcing that it has started. The change introduces a new BLE characteristic for notifications and a small kernel module to format and send the message. There is no direct evidence in the commit that this fixes a security bug; it appears to be a feature addition. However, the new code path involves copying untrusted-length data into a fixed-size buffer and allocating/freeing memory in an embedded BLE stack, which are places where future bugs could hide.
feat(vendor): disable traceback allocation in `PYOPT=1` builds
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
boot or update path
AI analysis · Low 26/100
This commit changes the Trezor firmware's embedded MicroPython configuration so that detailed error tracebacks are no longer stored in optimized (PYOPT=1) builds. The stated goal is to avoid allocating traceback data in release builds, which can save memory and reduce information exposure if an error occurs. There is no direct evidence in the commit that this fixes an active security vulnerability, but it is a defensive hardening measure.
Security candidatefix(core): switch off BLE on power offby tychovrahe · 76869636 · Sep 12, 2025 · 1 fileMessage 57 · ThinLow 35Details
Commit message · tychovrahe
fix(core): switch off BLE on power off
[no changelog]
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
boot or update path
AI analysis · Low 35/100
This bootloader patch makes the device explicitly turn off its Bluetooth Low Energy (BLE) radio when the hardware receives a power-off command. Without this fix, the BLE chip might stay on during shutdown, which could waste battery, cause unexpected radio behavior, or keep the device discoverable longer than intended. The change is defensive and improves shutdown hygiene rather than fixing an obvious remote attack.