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.
Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.
Lower-prioritychore(core): use pre-processed files for coverageby obrusvit · 74d62622 · Apr 23, 2026 · 5 filesMessage 100 · StrongInformational 15Details
Commit message · obrusvit
chore(core): use pre-processed files for coverage
- the resulting code coverage reports should be more precise in result - CI job reuses the build files so that the job doesn't have to make frozen build again
[no changelog]
100/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Mentions testing or verification
AI analysis · Informational 15/100
This is a routine developer tooling change that improves how code-coverage reports are generated for the Trezor firmware. It makes coverage measurements more accurate by using the same preprocessed source files that are actually compiled into test builds, and avoids redundant rebuilds in CI. There is no user-facing change and no security relevance.
✓ 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 15/100
This commit simply updates a Python testing tool called 'coverage' from version 5.5 to 7.13.5. The coverage tool is only used during software testing to measure which parts of the code were executed by tests. It is not part of the actual Trezor device firmware or wallet software that users interact with. There is no indication this update fixes or introduces any security issue.
build(core): exclude haptic & led code if possible
- also sort the features in the relevant function for better readability
[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 harmless build-system cleanup. It teaches the firmware build scripts to remove haptic (vibration) and RGB LED code from devices that do not have those hardware features, and it sorts the list of feature flags alphabetically for easier reading. There is no security bug being fixed here.
Security candidaterefactor(rust/trezor-thp): channel ids are allocated by applicationby Martin Milata · f36c2eab · Apr 23, 2026 · 8 filesMessage 62 · AdequateLow 28Details
Commit message · Martin Milata
refactor(rust/trezor-thp): channel ids are allocated by application
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 28/100
This commit is a code refactor in Trezor's Rust THP (Trezor Host Protocol) library. It moves responsibility for assigning channel IDs from the internal library mux to the application using the library. The change also removes the requirement that credential stores/cloners implement the Clone trait, and adds a helper allocator for consecutive channel IDs. There is no direct evidence in the commit or supplied references that this fixes an active security vulnerability; it appears to be an architectural cleanup that may make misuse harder but also places more correctness burden on the application.
Lower-priorityrefactor(rust/trezor-thp): type aliasesby Martin Milata · bc5d97a1 · Apr 23, 2026 · 4 filesMessage 57 · ThinInformational 15Details
Commit message · Martin Milata
refactor(rust/trezor-thp): type aliases
57/100 · ThinMessage clarity
✓ 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 is a straightforward code cleanup in the Trezor firmware's Rust implementation of the Trezor Host Protocol (THP). It replaces repeated uses of a generic type `Channel<Host, B>` or `Channel<Device, B>` with shorter type aliases `Channel<B>` inside the host and device channel modules. No behavior of the code changes, and there are no security implications.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 11/100
This commit adds a new 'reset' method to two internal Rust channel objects in the Trezor firmware's THP (Trezor Host Protocol) code. The method simply restores the channel to its freshly-created state, discarding any pending messages or connection bookkeeping. There is no indication in the commit that this fixes a security bug; it reads as a routine code cleanup or preparation for future use.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 11/100
This commit is a code cleanup in Trezor's secure transport layer. It centralizes how error messages are sent and removes an outdated workaround where the device had to manually craft an error packet. The change also makes decryption failures on the device side send a proper error response instead of silently closing the channel. There is no indication this fixes an active security vulnerability; it appears to be a refactoring and robustness improvement.
refactor(rust/trezor-thp): remember handshake pairing state
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 · Low 26/100
This commit refactors how a Trezor hardware wallet's Rust-based secure channel code remembers whether a pairing handshake succeeded. It moves the pairing-state information from a temporary handshake object into the long-lived channel object, adds a check that the handshake is fully done before completing, and exposes the remote device's public key and pairing state to the rest of the application. The changes look like a defensive cleanup rather than an active vulnerability fix, but they close a small window where pairing state could be lost or queried incorrectly.
refactor(rust/trezor-thp): keep track of retransmission attempts
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 18/100
This commit refactors the internal state tracking of the Trezor hardware wallet's THP (Trezor Host Protocol) Rust code so the channel can count how many times it has retransmitted a message. It also exposes that retry count to callers and adds a helper function for calculating how long to wait before retrying. The change is defensive: it makes the protocol more observable and easier to limit in the future, but it does not by itself fix a known exploitable bug.
✓ 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 changes how numbers appear in log messages for the Trezor hardware protocol (THP) Rust code. It switches decimal formatting (e.g., '123') to hexadecimal formatting (e.g., '0x7b') for control bytes, channel IDs, and error codes. There are no functional code changes, no security fixes, and no behavior changes.
Lower-priorityfeat(core): fix repetitive words in comments and some typos.by kopecdav · 23dd66d9 · Apr 23, 2026 · 3 filesMessage 77 · AdequateInformational 15Details
Commit message · kopecdav
feat(core): fix repetitive words in comments and some typos.
✓ 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 commit only fixes typos and duplicated words in code comments. No actual program code was changed, so it cannot affect device behavior or introduce security issues.
(cherry picked from commit 601327a80794a4e6fdbd394776eee32126bdb68c)
62/100 · AdequateMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope✓ Provides an explanatory body
AI analysis · Informational 15/100
This commit simply adds a new firmware version number (2.11.1) to a release metadata file. It is a routine administrative update with no code changes and no security relevance visible in the diff.
AI review queuedchore(core): sign translationsby Ioan Bizău · 39c051b7 · Apr 23, 2026 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Ioan Bizău
chore(core): sign translations
[no changelog]
(cherry picked from commit cbc91cf723769ed45c2390265f3aebdf6c932028)
62/100 · AdequateMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
signing or wallet pathtranslation-only discountsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit simply adds a new digital signature entry to a JSON file that records approved translation bundles for the Trezor hardware wallet. There is no code change, no bug fix, and no visible security issue. It appears to be routine release housekeeping: signing the translation data for firmware version 2.11.1.0.
(cherry picked from commit da6439c958fa115a977298b53b64dae2c1a24e80)
62/100 · AdequateMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope✓ Provides an explanatory body
AI analysis · Informational 15/100
This commit only updates a timestamp and a matching version number used for Ethereum network definition files. There are no code logic changes, no bug fixes, and no security-related modifications visible in the diff.
Lower-prioritychore: update signed secmonby Ioan Bizău · a19528f5 · Apr 23, 2026 · 1 fileMessage 70 · AdequateInformational 3Details
Commit message · Ioan Bizău
chore: update signed secmon
Built from 601327a80794a4e6fdbd394776eee32126bdb68c
(cherry picked from commit ef73d69c06a0496a06ca849ed4f84d9beafe5c2c)
70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
AI analysis · Informational 3/100
This commit replaces a binary file called secmon.bin for the T3W1 hardware model. The file is a compiled security monitor firmware image, but the actual code changes inside it are not visible in the commit. There is no description of any security fix or vulnerability, so we cannot determine from this commit alone whether it addresses a security issue.
Lower-prioritydocs(core): changelog for 2.11.1by Ioan Bizău · d41cfd95 · Apr 23, 2026 · 14 filesMessage 72 · AdequateInformational 15Details
Commit message · Ioan Bizău
docs(core): changelog for 2.11.1
(cherry picked from commit dea5e0f25f58c8cd3e9d55c1a9a88f2ecac5ebd4)
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 purely a documentation change: it assembles the release notes for Trezor firmware version 2.11.1 by moving text fragments from temporary changelog files into the final Markdown changelogs. No source code, cryptographic logic, or device behavior is modified, so it cannot introduce or fix a security vulnerability on its own.
AI review queuedchore(ethereum): Move stablecoin yielding from debug to production.by PrisionMike · 59c8a6b6 · Apr 23, 2026 · 4 filesMessage 62 · AdequateInformational 18Details
Commit message · PrisionMike
chore(ethereum): Move stablecoin yielding from debug to production.
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 18/100
This commit moves an Ethereum feature called "stablecoin yielding" from debug-only mode into production. It adds user-interface flows for certain ERC-4626 vault interactions, such as depositing USDC into a yield-bearing vault. The change itself is a feature promotion, not a fix for a known security flaw. There is no evidence in the commit that it addresses an active vulnerability or that it introduces one.
Security candidatefeat(ethereum): add support for erc-4626 withdraw and redeem flowby PrisionMike · b30022e5 · Apr 22, 2026 · 10 filesMessage 95 · StrongLow 41Details
Commit message · PrisionMike
feat(ethereum): add support for erc-4626 withdraw and redeem flow
- Updates vault with latest test - Adds concept of safe_tx - blind sign unsafe transaction with unkown vaults - clear sign safe transaction everywhere - refactor
[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
cryptography-sensitive pathsigning or wallet path
AI analysis · Low 41/100
This commit expands Trezor's Ethereum 'clear signing' support to include ERC-4626 vault withdraw and redeem transactions, not just deposits. For transactions where the receiver or owner differs from the signer on an unknown vault, the device now falls back to blind signing instead of refusing outright. For known Trezor-listed vaults, mismatches still trigger a hard error. The change is a feature addition with intentional security trade-offs rather than a clear-cut vulnerability patch.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit only changes user-facing text strings and their translation metadata for Ethereum vault (ERC-4626) screens. It adds labels like 'Redeem', 'Claim to', 'Withdraw amount', and removes unused strings like 'Your address'. There is no code logic change, no security fix, and no vulnerability introduced.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 2/100
This commit updates a bundled third-party library called libtropic from version 3.1.0 to version 3.2.1 inside the Trezor firmware repository. The actual code changes are not visible because only a submodule pointer was changed (one line modified in the vendor/libtropic path). There is no information in the commit itself saying this fixes a security problem, and no independent references were provided. On its own, this looks like a routine maintenance update.
Lower-priorityrefactor(core): scope menu details' layoutby Roman Zeyde · 5c7b29df · Apr 22, 2026 · 1 fileMessage 72 · AdequateInformational 12Details
Commit message · Roman Zeyde
refactor(core): scope menu details' layout
Also rename `Details.factory` to `_interact`.
[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 12/100
This is a small internal code cleanup in the Trezor firmware's user interface code. It renames a variable and ensures a temporary layout object is properly cleaned up after use. There is no indication this fixes a security vulnerability or introduces one.
May be related to the following OOM: https://github.com/trezor/trezor-firmware/actions/runs/24751684853/job/72416000050
[no changelog]
88/100 · StrongMessage clarity
✓ 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
AI analysis · Informational 11/100
This is a tiny code cleanup that wraps a UI menu layout call in a Python `with` statement so the object is automatically cleaned up after use. The commit message suggests it may help prevent an out-of-memory (OOM) crash during automated tests, but it does not describe a security vulnerability and no exploit path is visible.
refactor(core): scope payment request text confirmation layouts
May be related to the following OOM: https://github.com/trezor/trezor-firmware/actions/runs/24751684853/job/72416000050
[no changelog]
93/100 · StrongMessage clarity
✓ Specific, 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
cryptography-sensitive path
AI analysis · Informational 18/100
This commit is a small code cleanup in the Trezor hardware wallet's user-interface code. It changes how on-screen payment-request confirmation dialogs are created and destroyed, wrapping them in a 'with' block so they are cleaned up promptly. The developer notes this may be related to an out-of-memory (OOM) issue seen in automated tests, but the commit does not claim to fix a security vulnerability and no independent security references are provided.
✓ 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 is a straightforward internal refactoring that adds a new Rust logging bridge. It does not change any security-critical behavior: logging is already disabled in release builds, and the new code is only compiled into debug builds. There is no indication this fixes or introduces a security vulnerability.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 16/100
This commit fixes a logging bug in Trezor firmware's debug console output. Previously, the code used a width specifier to print a log source name, which could read past the end of a string that isn't null-terminated. The fix switches to a precision specifier that respects the known length. This is a defensive correctness fix in debug-only logging code; there is no direct evidence it is exploitable as a security vulnerability.