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.
feat(translations): allow dev-signed translation blobs in non-production builds
Changed the feature lock on accepting dev signed translation blobs from `debug` to `dev_keys` (new). This is enabled for all non-production builds so QA can test translations even on production ready, unsigned builds. [no changelog]
95/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
boot or update path
AI analysis · Low 25/100
This change lets non-production firmware builds accept translation files signed with development keys, not just official production keys. It is meant to help QA test translations on unsigned builds. Production builds still require official signatures, so an end-user running official firmware is not directly affected. The main risk is that a misconfigured build could accidentally ship with the relaxed setting enabled.
- New refresh device menu result used to refresh the menu after BLE connection change - Use static submenus in device menu for possibility to refresh to a particular menu screen - Add init screen param to device_menu FwUI function. After each device menu action, update the init screen or break out of the menu loop - Static back navigation, the previous design assumed the initial screen was always the root. - Don't require confirmation for toggle buttons [no changelog]
80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
Why it was queued
cryptography-sensitive pathboot or update path
AI analysis · Informational 17/100
This commit refreshes the on-device settings menu for the Trezor Safe 7 (Eckhart layout). It lets the menu restart on a specific submenu after actions like disconnecting a Bluetooth device or toggling haptics/LED, and removes the confirmation prompt for some toggle buttons. The changes are almost entirely user-interface plumbing; there is no obvious security bug, but the new loop and direct hardware calls deserve a quick sanity check.
- add builder functions for menu items - VecExt trait for menu items - use u8 instead of usize where possible [no changelog]
80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
Why it was queued
cryptography-sensitive pathboot or update path
AI analysis · Informational 15/100
This commit is a routine internal code cleanup for the Trezor firmware's on-device menu screen. It changes some internal number types from usize to u8, adds helper functions for building menu items, and makes the code more readable. There is no indication of a security fix or vulnerability being addressed.
Security candidatefix(core): use translation strings instead of literalsby Roman Zeyde · 5c16f75e · Sep 11, 2025 · 15 filesMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde
fix(core): use translation strings instead of literals
[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 replaces hard-coded English text strings in the Bluetooth pairing user interface with references to the device's translation system. It does not change what the device does, only how the text is loaded, so it can be translated into other languages later. There is no security bug being fixed here.
feat(core/bootloader): add LED effect to bootloader pairing workflow.
[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 adds colored LED lighting effects during the Bluetooth pairing process in the Trezor bootloader. It is purely a user-interface change to give visual feedback when the device is in pairing mode. There is no indication it fixes or introduces any security vulnerability.
Security candidatefeat(core/bootloader): add red LED to workflow_wipe_device.by kopecdav · 9de30971 · Sep 11, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · kopecdav
feat(core/bootloader): add red LED to workflow_wipe_device.
[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 change simply turns the device's LED red while the user is confirming a factory wipe, then turns it off afterward. It is a user-experience/visual indicator change in the bootloader and does not alter security logic, data handling, or access controls.
feat(core) add LED charging effect in hibernation.
[no changelog]
50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
boot or update path
AI analysis · Informational 15/100
This commit changes the Trezor bootloader's hibernation charging indicator from a solid blue LED to a pulsing 'charging' LED effect. There is no security-relevant change visible in the diff. It is a user-experience/visual feature.
Security candidatefeat(eckhart): menu item subtext can be a marqueeby Ioan Bizău · f4b52d8c · Sep 11, 2025 · 2 filesMessage 57 · ThinInformational 15Details
Commit message · Ioan Bizău
feat(eckhart): menu item subtext can be a marquee
[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 · Informational 15/100
This commit is a user-interface feature change for the Trezor hardware wallet's Eckhart design. It replaces a static text truncation behavior with a scrolling marquee animation for long subtext labels in menu items. There is no security-relevant change visible in the code.
✓ 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 23/100
This commit increases a timeout in the Trezor bootloader from 100 to 300 ticks while wiping Bluetooth pairing information ('bonds'). A too-short timeout could cause the wipe operation to fail or behave inconsistently if the Bluetooth chip takes longer to respond. The change is a hardening fix, not a clear-cut vulnerability patch, and the commit message does not describe any security impact.
chore(crypto): update secp256k1 submodule to 0.7.0
[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 11/100
This commit updates a bundled cryptographic library (secp256k1-zkp) from one version to another. No actual code changes are shown, and the commit message does not describe any security fix. On its own, this looks like a routine dependency maintenance update.
✓ 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 commit changes two fixed text strings inside the ChaCha20 code from C string literals to explicit character arrays. The strings are public constants used by the ChaCha20 algorithm ('expand 32-byte k' and 'expand 16-byte k'). The commit title says it fixes a CI build error, and there is no evidence in the diff or commit message of any security bug, memory issue, or behavior change beyond how the compiler stores these constants.
✓ 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 newer pre-built binary files for the T3W1 hardware model's bootloader and Bluetooth firmware, plus updates the recorded cryptographic hashes that the build system expects. The change log says '[no changelog]' and the title calls it a routine chore. There is no visible source-code change, no description of a security fix, and no independent evidence that this update addresses a vulnerability.
Security candidatefix(core/bootloader): account for slow nRF startup - wait before displaying the start buttonby tychovrahe · 2a4ddb4f · Sep 10, 2025 · 5 filesMessage 62 · AdequateLow 26Details
Commit message · tychovrahe
fix(core/bootloader): account for slow nRF startup - wait before displaying the start button
[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 · Low 26/100
This bootloader-only change adds a short wait and a temporary logo screen when a Trezor device has no firmware installed and Bluetooth (BLE) is enabled. The goal is to give the wireless nRF chip time to finish starting up before the user sees the 'Start' setup button. Without this delay, the device could briefly show the start button while the Bluetooth state is still unknown, which might lead to a confusing or inconsistent setup experience. There is no direct evidence in the commit that this fixes an exploitable security vulnerability.
chore(core): raise exception for invalid ThpCreateNewSession messages [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
authentication path
AI analysis · Low 37/100
This commit tightens how the Trezor hardware wallet handles passphrase-protected sessions. Previously, a host could supply a passphrase in a message even when the device was configured to require the passphrase only on the device. Now the firmware raises an error in that case, preventing the host from silently overriding the 'always on device' setting. The change also adds a debug-only warning when a session is created with a passphrase while passphrase protection is disabled on the device.
Security candidatetest(python): add device test for passphrase on device [no changelog]by M1nd3r · 8cc43871 · Sep 9, 2025 · 1 fileMessage 87 · StrongInformational 15Details
Commit message · M1nd3r
test(python): add device test for passphrase on device [no changelog]
87/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Mentions testing or verification
Why it was queued
access control
AI analysis · Informational 15/100
This commit only adds a new automated test for an existing feature. It does not change the actual Trezor firmware or its security behavior. The test checks that when a user enables 'passphrase always entered on device,' the device correctly rejects passphrases sent from the computer and accepts passphrases typed directly on the device. There is no security fix or vulnerability introduced here.
Security candidatechore(core): update type annotation for `button_request` in `trezorui_api` [no changelog]by M1nd3r · 296422e4 · Sep 9, 2025 · 2 filesMessage 77 · AdequateInformational 15Details
Commit message · M1nd3r
chore(core): update type annotation for `button_request` in `trezorui_api` [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 commit only changes a type annotation in documentation and generated mock files. It replaces `tuple[int, str]` with `tuple[ButtonRequestType, str]` for a function that returns button request information. No actual code behavior changes, and there is no security relevance.
Security candidatechore(core): use Python's `NotImplementedError` for missing layoutsby Roman Zeyde · 9f7df35d · Sep 8, 2025 · 7 filesMessage 77 · AdequateInformational 15Details
Commit message · Roman Zeyde
chore(core): use Python's `NotImplementedError` for missing layouts
Also, remove method-specific error messages, since it could be deduced from the exception traceback.
[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
cryptography-sensitive pathboot or update path
AI analysis · Informational 15/100
This is a routine code cleanup in the Trezor firmware's user-interface layer. It replaces a custom 'not implemented' error message with Python's standard NotImplementedError and removes some method-specific error strings. There is no security-relevant change: the same functions still fail in the same situations, only the error type and wording differ.
Security candidatefix(core): mark THP `app_name` and `host_name` with yellowby Roman Zeyde · c5fc0c09 · Sep 8, 2025 · 15 filesMessage 70 · AdequateInformational 19Details
Commit message · Roman Zeyde
fix(core): mark THP `app_name` and `host_name` with yellow
Fixes #5557.
[no changelog]
70/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Links an issue, advisory, or supporting reference! No meaningful explanatory body
Why it was queued
cryptography-sensitive pathsigning or wallet pathboot or update path
AI analysis · Informational 19/100
This commit changes how the Trezor hardware wallet displays pairing and connection prompts for its new Bluetooth-like 'THP' protocol. Instead of building the prompt sentence in Python code, it now uses a dedicated Rust UI screen that highlights the app name and host name in yellow. This is a user-experience and anti-phishing improvement, not a fix for a remotely exploitable vulnerability. The change only affects on-screen text formatting and which UI component is used; it does not change cryptographic checks, authorization logic, or data parsing.
chore(core): rename `OpTextLayout::add_text` to `add_text_with_font`
[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 pathboot or update path
AI analysis · Informational 15/100
This commit is a simple rename of a Rust function from `add_text` to `add_text_with_font` across several UI layout files. It does not change any behavior, logic, or security properties of the code. It is a routine code cleanup with no user-visible or security impact.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
cryptography-sensitive pathparser or protocol path
AI analysis · Informational 15/100
This commit is a routine build-system housekeeping change for the Trezor Python library. It replaces the old setup.py/requirements.txt packaging with a modern pyproject.toml and the uv build tool, drops Python 3.8 support, and changes how the package version is read. There is no security-relevant code change and no indication this fixes or introduces a vulnerability.
Security candidatestyle: update to black 25 style, silence new flake8 warningsby matejcik · 87f5f12d · Sep 8, 2025 · 38 filesMessage 50 · ThinInformational 15Details
Commit message · matejcik
style: update to black 25 style, silence new flake8 warnings
[no changelog]
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 pathboot or update pathauthentication path
AI analysis · Informational 15/100
This commit is purely a code-style update. It reformats Python source files to match the latest version of the Black formatter, changes uppercase hex escapes in byte strings to lowercase, removes unnecessary 'global' declarations, and adjusts some 'with' statements and trailing commas to satisfy updated flake8 rules. There are no functional changes to the firmware, tests, or tooling.
After: ``` print-type-size type: `ui::component::swipe_detect::SwipeConfig`: 5 bytes, alignment: 1 bytes print-type-size field `.page_axis`: 1 bytes print-type-size field `.up`: 1 bytes print-type-size field `.down`: 1 bytes print-type-size field `.left`: 1 bytes print-type-size field `.right`: 1 bytes ```
Before: ``` print-type-size type: `ui::component::swipe_detect::SwipeConfig`: 36 bytes, alignment: 4 bytes print-type-size field `.up`: 8 bytes print-type-size field `.down`: 8 bytes print-type-size field `.left`: 8 bytes print-type-size field `.right`: 8 bytes print-type-size field `.page_axis`: 1 bytes print-type-size end padding: 3 bytes ```
[no changelog]
80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
Why it was queued
boot or update pathauthentication path
AI analysis · Informational 15/100
This commit is a straightforward internal code cleanup that reduces the memory used by swipe gesture settings. It replaces a flexible but bulky structure storing animation duration with a simple on/off-style enum that only supports the two durations actually used (normal 333 ms and immediate 0 ms). There is no user-visible behavior change and 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 28/100
This commit changes the build scripts for Trezor hardware wallets so that when building in a non-optimized debug mode (PYOPT=0), a USB debug interface feature is automatically requested. This debug interface is normally used only during development and testing to control and inspect the device. The change itself is small and appears intentional, but enabling debug interfaces in production-like builds can be a security concern if misused. There is no evidence in the commit that this fixes a known security vulnerability; it looks like a build configuration fix.
✓ 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 adjusts how text labels are positioned and rendered on the bootloader screen of a Trezor hardware wallet model. It changes the vertical position of progress text, switches from a low-level text-drawing call to a reusable UI label component that supports multi-line wrapping, and adds explicit line breaks in two user-facing messages. There is no security-relevant change visible in the diff.
✓ 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 only increments version numbers in four header files after an internal release. There are no code, logic, or security changes visible in the diff.