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.
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 15/100
This commit fixes a single grammar typo in a code comment. The word 'host' was changed to 'the host' in a protobuf message description. No code behavior, security logic, or functionality was changed.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 15/100
This commit only adds a human-readable comment to a protobuf message definition. It does not change any code behavior, protocol logic, or security properties.
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Low 42/100
This commit fixes a subtle cleanup bug in the Bluetooth pairing function on Trezor hardware wallets. Before the fix, if a caller supplied an advertising name that was too long, the function returned early without unlocking an important system lock (irq_lock). Leaving that lock held could freeze or crash the device. The fix moves the length check before the lock is taken, so the bad-input path no longer skips the unlock.
Lower-priorityfix(core): truncate BLE name before invoking `ble_enter_pairing_mode()`by Roman Zeyde · 1345ef6e · Apr 8, 2026 · 2 filesMessage 77 · AdequateLow 34Details
Commit message · Roman Zeyde
fix(core): truncate BLE name before invoking `ble_enter_pairing_mode()`
Also, truncate when calling `switch_on()`.
77/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
AI analysis · Low 34/100
This commit fixes a bug in the Trezor hardware wallet's Bluetooth handling. Previously, the device name sent to the Bluetooth chip when entering pairing mode or switching Bluetooth on was not truncated to the maximum allowed length. This could cause the Bluetooth stack to receive an oversized name, potentially leading to unstable behavior, pairing failures, or memory corruption in the Bluetooth firmware. The fix ensures the name is shortened to the advertised-name limit before being passed along.
Will be used to allow choosing 2 options (instead of 3). In that case, the 2nd separator should not be rendered.
[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 path
AI analysis · Informational 18/100
This commit is a small UI layout change for the Trezor hardware wallet. It lets a screen that normally shows three word choices display only two choices by leaving one slot blank and hiding the separator line next to it. There is no indication this fixes a security bug; it appears to be a feature tweak for a future user interface.
Security candidatetest(core): test backup cancellation during `ResetDevice`by Roman Zeyde · c72a83e8 · Apr 7, 2026 · 3 filesMessage 87 · StrongInformational 15Details
Commit message · Roman Zeyde
test(core): test backup cancellation during `ResetDevice`
Previously, backup cancellation was tested only during `BackupDevice`.
[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
cryptography-sensitive path
AI analysis · Informational 15/100
This commit only adds new automated tests that simulate a user cancelling the backup step during device setup (ResetDevice). It does not change the actual Trezor firmware code, wallet logic, or fix any bug. It is a testing improvement with no direct security impact on shipped devices.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 15/100
This commit only updates the expected visual-output fingerprints used by the device's automated UI tests. It does not change any firmware, application, or security code. The test names were adjusted because the corresponding test code was likely refactored to add a 'normal' versus 'try_to_cancel' parameter. There is no user-facing or security-relevant change.
chore(core): move backup flow adapters to `tests.input_flows`
To be re-used in other tests as well.
[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
AI analysis · Informational 15/100
This commit is a routine test-code cleanup. It moves helper functions used only in automated device tests from one test file into a shared test helper file so they can be reused. No firmware behavior, cryptography, or user-facing functionality is changed.
Lower-priorityfix(core): ignore unexpected messages in backup flow during setupby Roman Zeyde · 69c51a63 · Apr 7, 2026 · 1 fileMessage 85 · StrongLow 45Details
Commit message · Roman Zeyde
fix(core): ignore unexpected messages in backup flow during setup
Changelog entry will be added after #6348 is fully resolved.
[no changelog]
85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Links an issue, advisory, or supporting reference
AI analysis · Low 45/100
This change wraps the wallet backup step during device setup in a helper that ignores unexpected messages. It likely prevents a setup/backup flow from being disrupted or aborted by stray or malicious messages sent to the Trezor while a backup is in progress, which could otherwise leave the device in an inconsistent state or confuse the user.
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
AI analysis · Informational 15/100
This is a small internal code cleanup in the Trezor firmware. It moves a debug-only helper function call directly to where it is used and removes an unnecessary wrapper. There is no user-facing change and no security fix.
AI review queuedchore(core): prefix constants with `_`by Roman Zeyde · db8dc485 · Apr 7, 2026 · 1 fileMessage 88 · StrongInformational 15Details
Commit message · Roman Zeyde
chore(core): prefix constants with `_`
It allows MicroPython to inline the constants and avoid allocations[^1].
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This is a routine code cleanup change in the Trezor firmware's Ethereum transaction signing module. It renames several internal constants to start with an underscore and wraps them with MicroPython's `const()` helper so the interpreter can optimize them. The numeric values and program logic are unchanged. There is no security fix or vulnerability here.
AI review queuedfix(python): avoid crash when scanning BLE multiple timesby Martin Milata · ca7efea8 · Apr 5, 2026 · 1 fileMessage 62 · AdequateInformational 20Details
Commit message · Martin Milata
fix(python): avoid crash when scanning BLE multiple times
[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
second-pass: broader security terminology
AI analysis · Informational 20/100
This commit fixes several small bugs in Trezor's Python library for Bluetooth (BLE) communication. The main fix prevents a crash when scanning for BLE devices multiple times, caused by a typo that iterated over dictionary values instead of items. Other changes add safety checks before reading or writing to a BLE device and improve error messages. There is no clear security vulnerability being patched; it looks like ordinary bug fixing and hardening.
fix(translations,cardano): translations for credentials phrase in cardano
- Refactored some TR string for better l10n. - Minor code adjustment around it.
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
credential or privilege statesigning or wallet path
AI analysis · Informational 15/100
This commit is a routine translation and user-interface wording update for the Cardano cryptocurrency flow on Trezor hardware wallets. It rephrases on-screen labels such as changing 'key hash' to 'Key hash' and replacing some message-signing prompts with a new credential description template. There is no indication of a security vulnerability being fixed or introduced.
- Added `wire_in` and `wire_out` options to THP messages.
[no changelog]
72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
AI analysis · Informational 15/100
This commit is a routine metadata update for Trezor's protocol-buffer message definitions. It adds directional labels (wire_in/wire_out) and a bitcoin-only flag to a set of Trezor-Host Protocol (THP) pairing and credential messages, plus regenerates the matching Rust code. There is no change to actual message handling logic, cryptography, or device behavior.
AI review queuedchore(deps): bump pygments from 2.19.2 to 2.20.0by dependabot[bot] · f0039f68 · Apr 2, 2026 · 1 fileMessage 88 · StrongInformational 15Details
Commit message · dependabot[bot]
chore(deps): bump pygments from 2.19.2 to 2.20.0
Bumps [pygments](https://github.com/pygments/pygments) from 2.19.2 to 2.20.0. - [Release notes](https://github.com/pygments/pygments/releases) - [Changelog](https://github.com/pygments/pygments/blob/master/CHANGES) - [Commits](https://github.com/pygments/pygments/compare/2.19.2...2.20.0)
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
This is an automated dependency update by Dependabot that bumps the Python syntax-highlighting library Pygments from version 2.19.2 to 2.20.0 in the project's lock file. There is no indication in the commit that this fixes a security issue, and Pygments is an indirect development-only dependency used for documentation or code display, not for handling secrets, cryptography, or user input in the hardware wallet firmware itself. On its own, this change does not present a security concern.
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 16/100
This is a small internal cleanup in the Trezor firmware that replaces direct access to a global variable (CURRENT_CONTEXT) with a safer helper function (get_context()). The change is described by the developers as a code hygiene improvement, not a security fix. It slightly reduces the risk of future bugs by making context access more controlled, but there is no direct evidence it fixes an exploitable vulnerability.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
authentication path
AI analysis · Informational 15/100
This commit only removes unused Python imports and simplifies a few conditional import blocks in unit test files. It does not change any production firmware code, cryptographic logic, or device behavior. There is no security issue here.
Extract context setup/teardown into `TestCaseWithContext`.
[no changelog]
82/100 · StrongMessage clarity
✓ 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
authentication path
AI analysis · Informational 15/100
This commit is a test-code cleanup only. It moves repeated setup and teardown code for unit tests into a shared helper class, TestCaseWithContext. There are no changes to the actual firmware, wallet logic, cryptography, or any code that runs on a real Trezor device. It cannot affect user funds, device security, or real-world attacks.
AI review queuedchore: extract constantsby Ioan Bizău · b5991262 · Apr 2, 2026 · 4 filesMessage 40 · ThinInformational 15Details
Commit message · Ioan Bizău
chore: extract constants
[no changelog]
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 is a simple code cleanup: it moves a small list of known Ethereum contract addresses and their human-readable names from one file to a new dedicated file. The actual addresses and names remain exactly the same, and no security behavior changes.
feat(tests, ethereum): add more clear signing fixtures.
[no changelog]
72/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
signing boundarysigning or wallet path
AI analysis · Informational 15/100
This commit only adds new test cases and fixes whitespace indentation in a test fixture file for Ethereum transaction signing. It does not change any firmware code that runs on the Trezor device or any production software. The new test cases include edge-case and fuzz-style scenarios (e.g., zero slippage, very large amounts, unknown tokens) to verify how transaction data is displayed to users. There is no security vulnerability introduced here.
AI review queuedfeat(ethereum): clear sign first 4kbby Ioan Bizău · 6c78de76 · Apr 2, 2026 · 4 filesMessage 57 · ThinLow 35Details
Commit message · Ioan Bizău
feat(ethereum): clear sign first 4kb
[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 pathsecond-pass: security-sensitive path
AI analysis · Low 35/100
This commit changes how Trezor handles Ethereum transaction data (calldata). Previously, clear signing only worked if the entire calldata fit in the first chunk sent by the host. Now the device actively requests and stores up to 4 KB of calldata so it can try clear signing on larger transactions. If clear signing succeeds, the user sees a human-readable confirmation instead of raw hex data. The change refactors the code to collect initial data before deciding whether to clear sign or fall back to blind signing. There is no direct evidence in the commit of a security vulnerability being fixed; it reads as a feature improvement.
✓ Subject identifies a change✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 15/100
This commit only updates a test data file (fixtures.json) that stores expected screen hashes for automated user-interface tests. It does not change any firmware, application, or cryptographic code that runs on Trezor devices. There is no security issue in this commit itself.
Security candidatefix(python): `trezorctl` should use recent THP credentials firstby Roman Zeyde · 280e51fd · Apr 2, 2026 · 2 filesMessage 97 · StrongInformational 24Details
Commit message · Roman Zeyde
fix(python): `trezorctl` should use recent THP credentials first
Otherwise, it may get stuck using an invalidated one (#6575).
97/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Explains rationale or failure mode✓ Links an issue, advisory, or supporting reference
Why it was queued
defensive validationcredential or privilege state
AI analysis · Informational 24/100
This commit fixes a minor bug in the Trezor command-line tool (trezorctl) used to manage paired device credentials. Previously, when listing saved credentials, the tool would try the oldest saved credential first. If that older credential had been invalidated on the device, the tool could get stuck trying to use it instead of a newer, still-valid one. The fix simply reverses the order so the most recently added credential is tried first. It is a usability/reliability fix rather than a security vulnerability that allows an attacker to steal funds or keys.
✓ Subject identifies a change✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
signing or wallet pathboot or update path
AI analysis · Informational 15/100
This commit only updates version numbers and translation metadata. It changes the firmware version from 2.11.1 to 2.11.2, the security monitor version from 1.0.8 to 1.0.9, and updates the translation files' version headers and signature metadata. There are no code changes that affect security or functionality.
feat(python): propagate method to choose backup handler
[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
AI analysis · Informational 15/100
This commit adds a new command-line option to the Trezor Python library that lets users choose between two backup methods (display on device screen, or a new 'N4W1' method) when setting up or backing up a hardware wallet. It is a feature addition, not a security fix.