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(translations): update to blanking rules [no changelog]by Michal Kazda · 7b6a5a3c · May 19, 2026 · 1 fileMessage 77 · AdequateInformational 15Details
Commit message · Michal Kazda
chore(translations): update to blanking rules [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
translation-only discount
AI analysis · Informational 15/100
This commit only updates a JSON file that controls which on-screen text strings are allowed to be left blank (untranslated) for different Trezor device screen layouts. It adds and reorganizes string identifiers so the translation-blanking rules match the latest user interface. There is no change to security logic, cryptography, or how the device handles secrets.
✓ 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 is a routine translation update. It removes a few unused text strings, changes some translations to use per-device-style wording, and updates the internal offsets in generated translation tables. There is no code that handles secrets, cryptography, user authentication, or device commands being changed. It is not a security patch.
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
translation-only discount
AI analysis · Informational 15/100
This commit simply removes four unused English translation text strings from a language file. There is no code change, no functional change, and no security relevance.
Security candidatechore(translations): update signatures fixtures and some strings - fixes a failing T3W1 test - other cs string adjustments [no changelog]by Michal Kazda · 9f5f454a · May 18, 2026 · 3 filesMessage 95 · StrongInformational 15Details
Commit message · Michal Kazda
chore(translations): update signatures fixtures and some strings - fixes a failing T3W1 test - other cs string adjustments [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
signing boundarysigning or wallet path
AI analysis · Informational 15/100
This commit updates Czech translation strings and refreshes the digital signatures that authenticate those translations, plus updates expected test screenshots. There is no code change that affects security behavior, cryptography, or how the device protects secrets. It is a routine localization maintenance patch.
AI review queuedchore(translations): sync Crowdin translations [no changelog]by Michal Kazda · 4602c0c3 · May 18, 2026 · 6 filesMessage 77 · AdequateInformational 15Details
✓ Specific, descriptive subject✓ Names a concrete action or component✓ 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 only updates translated text strings in Czech, German, Spanish, French, and Portuguese, plus the translation signature metadata. There are no code, logic, or security-related changes.
Lower-prioritychore(translations): smarter crowdin.py merge - replace /n/n with /n/r - skip English during merge [no changelog]by Michal Kazda · d7f9d3e6 · May 18, 2026 · 1 fileMessage 77 · AdequateInformational 15Details
Commit message · Michal Kazda
chore(translations): smarter crowdin.py merge - replace /n/n with /n/r - skip English during merge [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
translation-only discount
AI analysis · Informational 15/100
This is a routine internal tooling change for managing translation files. It adjusts how translation text downloaded from the Crowdin service is cleaned up before being saved, and skips processing English-language 'translations' because they are empty and would overwrite the existing English file. There is 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
documentation-only discount
AI analysis · Informational 15/100
This commit only updates a documentation file (OPTIONS.rst) to list two newly added command-line options. It does not change any executable code, cryptographic logic, or security-sensitive behavior. There is no security issue in the commit itself.
AI review queueddocs(core): update Trezor Core referencesby Roman Zeyde · a6eeac12 · May 17, 2026 · 2 filesMessage 72 · AdequateInformational 15Details
Commit message · Roman Zeyde
docs(core): update Trezor Core references
Also, add TS7 in `common/defs/README.md`
[no changelog]
72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
signing or wallet pathdocumentation-only discountsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit only updates documentation text. It replaces the product name 'Trezor T' with 'Trezor Core' in two README/markdown files and adds a new device model code (T3W1 for Trezor Safe 7) to a support list. No code, logic, or security behavior is changed.
Lower-prioritychore(translations): updates to English strings - mainly adding periods where approprite for Delizia and Eckhart [no changelog]by Michal Kazda · 26f0f851 · May 15, 2026 · 4 filesMessage 85 · StrongInformational 15Details
Commit message · Michal Kazda
chore(translations): updates to English strings - mainly adding periods where approprite for Delizia and Eckhart [no changelog]
85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
AI analysis · Informational 15/100
This commit only changes user-visible text strings in the Trezor firmware. It adds or removes periods and makes tiny wording tweaks (for example, 'Language changed successfully' became 'Language changed successfully.') for specific device themes called Delizia and Eckhart. There is no code behavior change, no new feature, and no security fix.
AI review queuedchore(common): add BTC signet definitionby Roman Zeyde · 73dd7733 · May 15, 2026 · 10 filesMessage 57 · ThinInformational 15Details
Commit message · Roman Zeyde
chore(common): add BTC signet definition
[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 · Informational 15/100
This commit adds support for Bitcoin Signet, a public testing network for Bitcoin, to Trezor firmware. It is a routine feature addition with no security-relevant code changes. The commit only adds a new coin definition, updates generated coin-info tables, and adjusts internal tooling to treat Signet like the existing Testnet and Regtest networks.
Lower-prioritychore(python): allow ignoring xpub magic in trezorctlby Roman Zeyde · 5976e757 · May 15, 2026 · 1 fileMessage 77 · AdequateInformational 18Details
Commit message · Roman Zeyde
chore(python): allow ignoring xpub magic in trezorctl
Output descriptors don't use ypub/zpubs.
[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
AI analysis · Informational 18/100
This commit adds a new command-line option to the Trezor Python tool (trezorctl) that lets users request a public key (xpub) without the usual version/magic bytes being adjusted for the specific coin and script type. This is described as a convenience for working with 'output descriptors', a modern Bitcoin wallet format. There is no direct security bug here; it is a feature change in a client-side utility.
Lower-prioritychore(deps): bump urllib3 from 2.6.3 to 2.7.0by M1nd3r · 1e27d5df · May 15, 2026 · 2 filesMessage 57 · ThinInformational 15Details
Commit message · M1nd3r
chore(deps): bump urllib3 from 2.6.3 to 2.7.0
[no changelog]
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 is a routine dependency update that bumps the Python urllib3 library from version 2.6.3 to 2.7.0. It only changes package metadata files (pyproject.toml and uv.lock) and contains no code changes. There is no indication in the commit of any security issue being fixed.
chore(docs): add mdbook mermaid build processor for diagrams support
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
documentation-only discount
AI analysis · Informational 15/100
This commit adds a third-party diagram-drawing library (Mermaid) to the project's documentation build. It only touches documentation configuration and static JavaScript files used to render diagrams in the generated docs. There is no change to the Trezor firmware, wallet code, or any security-sensitive component.
Lower-prioritychore(deps): decrease number of dependenciesby M1nd3r · 2df3304c · May 15, 2026 · 1 fileMessage 80 · StrongInformational 15Details
Commit message · M1nd3r
chore(deps): decrease number of dependencies
- Updating the following packages drops related dependencies.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
AI analysis · Informational 15/100
This commit is a routine housekeeping update to the project's Python dependency lock file. It upgrades four development or utility packages to newer versions that no longer need certain sub-dependencies, so those sub-packages are removed. There is no indication of a security fix or vulnerability being addressed.
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
AI analysis · Informational 18/100
This commit updates the list of recognized Tron TRC-20 tokens in Trezor firmware. It adds six new tokens (WBT, WTRX, SUNOLD, AINFT, sTRX, Klever), restructures how token addresses are stored, and adds internal checks that the stored byte addresses match the public TronScan addresses. There is no security vulnerability visible in the change itself.
Security candidatebuild: prevent uv from using packages newer than 30 daysby matejcik · b4b7f5cf · May 14, 2026 · 6 filesMessage 98 · StrongLow 30Details
Commit message · matejcik
build: prevent uv from using packages newer than 30 days
- Limits exposure of the repo to supply chain compromise -- as long as you use uv. - Downgraded Mako (path traversal vulnerability is irrelevant for us). - If a relevant security is available, we'll have to whitelist that package -- use `exclude-newer-package` (https://docs.astral.sh/uv/reference/settings/#exclude-newer-package). - Switched buildsystem declaration from hatchling to flit everywhere.
[no changelog]
98/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✓ Names security-relevant behavior explicitly
Why it was queued
explicit security languageupdate trust
AI analysis · Low 30/100
This commit is a hardening change to Trezor's build tooling. It tells the Python package manager 'uv' not to use any package uploaded to PyPI in the last 30 days, which reduces the risk of accidentally pulling in a freshly compromised dependency. It also downgrades the 'Mako' template library from 1.3.12 to 1.3.10 and switches the build system from 'hatchling' to 'flit' in several sub-projects. The commit message explicitly says the goal is to limit supply-chain compromise exposure.
Lower-prioritychore(ci): remove unnecessary use of fullDeps nix-shellby M1nd3r · 1faa1a94 · May 14, 2026 · 6 filesMessage 62 · AdequateInformational 15Details
Commit message · M1nd3r
chore(ci): remove unnecessary use of fullDeps nix-shell
[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 is a routine cleanup of the project's automated build and test configuration. It removes unnecessary uses of a special 'full dependencies' mode in the Nix shell for some CI jobs, while explicitly adding it where it is actually needed. There is no change to the Trezor firmware code that runs on devices, and no security issue is present.
Lower-prioritychore(ci): run tropic model_server in uv environmentby M1nd3r · 709940ff · May 14, 2026 · 1 fileMessage 85 · StrongInformational 15Details
Commit message · M1nd3r
chore(ci): run tropic model_server in uv environment
- It is not necessary to rely on poetry from ts-tvl, it can be run directly in the uv environment from trezor-firmware.
[no changelog]
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
documentation-only discount
AI analysis · Informational 15/100
This is a routine internal cleanup of the project's automated testing setup. It changes how a test-support program (the 'Tropic model server') is started during continuous integration (CI) runs, switching from a Poetry-based Python environment inside a vendored dependency directory to the project's own 'uv' Python environment. There is no change to the firmware, wallet logic, cryptography, or any code shipped to users.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 20/100
This commit fixes a misleading error message in the Monero key image sync feature on Trezor hardware wallets. Previously, when the cryptographic hash check failed, the device incorrectly reported 'Invalid number of outputs' instead of 'Invalid hash'. This is a user-facing string correction with no direct security vulnerability in the underlying check itself.
✓ 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 only changes a user-facing label on Trezor hardware wallets. The old text 'Review details to claim rewards.' is replaced with 'Claim rewards from Merkl.xyz' in translation files and generated Rust code. There is no code behavior change, no bug fix, and no security relevance.
Security candidatefeat(ethereum): clear signing support for nested array and byte32by PrisionMike · ba620e9a · May 14, 2026 · 7 filesMessage 62 · AdequateLow 28Details
Commit message · PrisionMike
feat(ethereum): clear signing support for nested array and byte32
[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
signing boundarysigning or wallet path
AI analysis · Low 28/100
This commit adds support in Trezor's Ethereum 'clear signing' feature for two new data shapes: nested arrays (arrays inside arrays) and fixed 32-byte values (bytes32). It also renames an internal helper function from `_request_definitions` to `request_definitions`. The changes are framed as a normal feature addition with no changelog entry. There is no direct evidence in the commit that this fixes a security vulnerability, but any change to transaction parsing logic can affect how the device interprets maliciously crafted data.
✓ 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 39/100
This commit adds support for displaying human-readable confirmation screens when users claim rewards through the Merkl/claim.xyz rewards distributor on Ethereum. It replaces a placeholder that previously forced blind signing for these transactions. The change lets the Trezor show the token and amount being claimed instead of just raw transaction data, but only for a specific known distributor address and only when the claim is for the signer themselves.
✓ 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 adds a new automated CI check that verifies the project's Python dependency lock file (uv.lock) is up-to-date. It does not change any firmware code, fix a bug, or alter security-critical behavior. It is purely a build/development hygiene improvement.
Security candidaterefactor(core, crypto): extract `consteq` for general useby M1nd3r · 1b051d71 · May 14, 2026 · 4 filesMessage 62 · AdequateInformational 12Details
Commit message · M1nd3r
refactor(core, crypto): extract `consteq` for general use
[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 12/100
This commit adds a new utility function called `consteq` that compares two chunks of memory in a way that takes the same amount of time regardless of how similar the chunks are. It is intended to help prevent timing-based attacks in the future, but the commit itself only extracts and compiles the helper; it does not change any existing security-sensitive comparison or fix a known vulnerability.
Security candidatechore(core): add consteq check directly into aesgcm.finish when decryptingby M1nd3r · 4e0245cc · May 14, 2026 · 3 filesMessage 62 · AdequateLow 47Details
Commit message · M1nd3r
chore(core): add consteq check directly into aesgcm.finish when decrypting
[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 · Low 47/100
This commit hardens how Trezor devices verify AES-GCM authentication tags during decryption. Previously, callers computed the tag and then compared it separately using a constant-time helper. Now the comparison is performed inside the cryptographic finish routine itself, using a constant-time equality check. This reduces the risk that a future caller forgets to verify the tag or uses a non-constant-time comparison, which could allow an attacker to tamper with encrypted messages or recover secrets through timing analysis.