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✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 15/100
This commit simply deletes an unused helper class called BusyDecoder from a cryptography file in the Trezor firmware. There is no change to active code, no bug fix, and no security-relevant behavior change.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Moderate 60/100
This commit replaces ordinary byte-string comparisons with a constant-time equality check (consteq) when verifying authentication tags in Trezor's THP (Trezor Host Protocol) cryptography. In theory, a standard '==' comparison can leak timing information that might help an attacker forge or tamper with encrypted messages. The change is defensive and reduces the risk of timing side-channels, but the commit itself does not claim a fix for any known exploitable vulnerability.
Lower-prioritychore(core): simplify `interact()` type annotationby Roman Zeyde · 86cbd6ac · Apr 17, 2026 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde
chore(core): simplify `interact()` type annotation
[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 is a minor code cleanup that simplifies a type annotation and removes a now-unnecessary runtime type check. It does not change security behavior or fix any vulnerability.
Lower-prioritychore(core): improve `loop.Task` type annotationby Roman Zeyde · 1e4152e5 · Apr 17, 2026 · 3 filesMessage 57 · ThinInformational 15Details
Commit message · Roman Zeyde
chore(core): improve `loop.Task` type annotation
[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 commit only changes Python type annotations for the loop.Task class, adding a generic type parameter [None]. It does not alter any runtime behavior, logic, or security checks in the Trezor firmware code.
chore(docs): adjust description of `core` in readme
- Adjusted description of `core`, because it contains firmware implementation for Trezor Safe models, too - not only Trezor T. - Added few minor readme fixes.
[no changelog]
97/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
documentation-only discount
AI analysis · Informational 15/100
This commit only updates the project's README file. It expands the description of the 'core' folder to mention additional Trezor Safe hardware models and makes minor grammar and formatting fixes. There are no code or security changes.
Lower-priorityfix(python): support receiving piggybacked ACKsby Martin Milata · c5d9cc4a · Apr 16, 2026 · 2 filesMessage 57 · ThinLow 32Details
Commit message · Martin Milata
fix(python): support receiving piggybacked ACKs
[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 · Low 32/100
This commit fixes the Trezor Python host library so it correctly handles a protocol optimization called 'piggybacked ACKs.' In the Trezor THP (Trezor Host Protocol), a data message can also carry an acknowledgment (ACK) bit, meaning one side can confirm it received the previous message while simultaneously sending new data. Previously, the Python library apparently treated such combined messages as invalid ACKs and would keep waiting or retry, which could cause communication failures or timeouts. The change lets the library accept the ACK part of such a message, save the data part for the next read, and continue normally.
✓ 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 adds new user-visible text labels for two backup types ('N4W1 backup' and 'Wordlist backup') to the Trezor firmware's translation system. It does not change any security logic, cryptographic code, or user-flow behavior. The strings are empty on most device layouts and only populated for the 'Eckhart' layout. The translation signature metadata was also updated as part of the normal build process.
Lower-priorityrefactor(core): move `show_invalid_mnemonic()` into `_process_words()`by Roman Zeyde · b7b3fe66 · Apr 16, 2026 · 1 fileMessage 89 · StrongInformational 15Details
Commit message · Roman Zeyde
refactor(core): move `show_invalid_mnemonic()` into `_process_words()`
In order to simplify `_DisplayHandler` interface.
[no changelog]
89/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
AI analysis · Informational 15/100
This is a small internal code cleanup in the Trezor firmware's device-recovery flow. It moves the 'invalid mnemonic' warning screen from one helper object into another function and switches the control flow from returning an awaitable to raising a custom internal exception. There is no user-visible behavior change and no security fix or vulnerability introduced.
AI review queuedchore(core): remove personal todo markersby cepetr · dd5c5e58 · Apr 16, 2026 · 6 filesMessage 37 · OpaqueInformational 15Details
Commit message · cepetr
chore(core): remove personal todo markers
[no changelog]
37/100 · OpaqueMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body! Contains work-in-progress language
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100
This commit is a routine cleanup that removes informal developer notes ('!@# TODO' markers) from six source files and makes one tiny code improvement (using an unsigned integer conversion function). There is no security-relevant change and no indication of a vulnerability being fixed.
Lower-priorityci(core): increase coverage threshold back to 85%by Roman Zeyde · 1bcf9318 · Apr 15, 2026 · 1 fileMessage 67 · AdequateInformational 15Details
Commit message · Roman Zeyde
ci(core): increase coverage threshold back to 85%
[no changelog]
67/100 · AdequateMessage clarity
✓ 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
documentation-only discount
AI analysis · Informational 15/100
This commit simply raises a test-coverage threshold from 84% to 85% for all hardware models in the project's continuous-integration workflow. It removes a temporary exception that had been made for one model (T3B1). There is no change to the firmware code that runs on devices, no bug fix, and no security-related content.
feat(core): update translation parser support for blob V2.
[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
translation-only discount
AI analysis · Low 27/100
This commit updates the Trezor firmware's translation (language) file parser to support a new version 2 format. It adds support for reading font kerning data (small spacing adjustments between letter pairs) and relaxes a strict check that previously rejected any trailing data after the font section. The change is a feature update, not a documented security fix, but it touches low-level binary parsing code where mistakes can create security risks.
feat(core): add font kernings for non-ascii characters.
[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 11/100
This commit updates how the Trezor device calculates spacing (kerning) between pairs of text characters so it also works for non-ASCII characters used in translations. It is a UI rendering feature change, not a security fix.
fix(core): fix horizontal positioning with kerning
[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 17/100
This commit fixes a text-rendering bug in the Trezor hardware wallet's user interface. Previously, when laying out text on screen, the code measured each character's width individually but forgot to add the small spacing adjustments (called kerning) between certain pairs of letters. This could cause text lines to be slightly misaligned or overflow their intended boundaries. The fix tracks the previous character and adds the correct kerning amount to each character's width during layout.
feat(core): extract kernings from ttf files, update mako template.
[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 feature addition to the font-generation tooling for the Trezor hardware wallet firmware. It teaches the build-time font generator to read kerning (letter-spacing adjustment) data from TrueType font files and include that data in the generated Rust font tables. There is no indication this change fixes a security bug or introduces a security-relevant behavior.
✓ Descriptive subject✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 15/100
This commit only updates a large test data file (tests/ui_tests/fixtures.json) used for UI regression testing. It changes expected screen-shot hashes for many test cases but does not modify any firmware, application, or cryptographic code. There is no indication of a security fix or vulnerability.
Lower-priorityfeat(core/tools): update gen_font script to extract kernings for special lang characters.by kopecdav · 72f762c9 · Apr 15, 2026 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · kopecdav
feat(core/tools): update gen_font script to extract kernings for special lang characters.
[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 updates an internal font-generation tool so that it also extracts character spacing (kerning) data for non-English characters. It is a build-time developer script change with no effect on device firmware at runtime and no security relevance.
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 21/100
This commit refactors how font kerning (fine-tuning space between letter pairs) is stored and looked up in the Trezor firmware. It changes the data structure from a flat list of triplets to a two-level index/pairs table, adds a Rust feature flag to enable or disable kerning, and updates the font generation tools to extract kerning from modern OpenType GPOS tables. The changes are primarily a data-format and build-tooling improvement, not a fix for a known vulnerability. The commit does not describe itself as security-related.
Lower-priorityfeat(trezorlib): Update translation blob generator to V2 with extra kerning table.by kopecdav · aab89cff · Apr 15, 2026 · 1 fileMessage 62 · AdequateInformational 18Details
Commit message · kopecdav
feat(trezorlib): Update translation blob generator to V2 with extra kerning table.
[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
translation-only discount
AI analysis · Informational 18/100
This commit updates the Trezor library's translation-blob generator from version 1 to version 2. The main change is adding support for a new 'kerning table' that adjusts spacing between font characters. It also adjusts how font glyph data is read from JSON files. There is no direct evidence in the commit that this fixes a security vulnerability; it appears to be a feature update for font rendering.
Lower-priorityfix(ui/fonts): make font kernings optional.by kopecdav · 4ce22ff1 · Apr 15, 2026 · 3 filesMessage 57 · ThinInformational 15Details
Commit message · kopecdav
fix(ui/fonts): make font kernings optional.
[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 commit changes the font system so that kerning (spacing adjustments between letter pairs) can be omitted when generating a font. It is a build/code-generation feature change, not a security fix or vulnerability.
feat(core): add kernings into rust drawing functions.
[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 font kerning support to Trezor's Rust-based UI text rendering. Kerning adjusts the spacing between specific pairs of characters to make text look better. There is no security issue visible in the change; it is a normal typography feature.
✓ 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 regenerates font data files for the Trezor firmware user interface. It adds optional kerning (letter-spacing adjustment) tables to several fonts, gated behind a compile-time feature flag named `ui_font_kerning`. Other fonts simply add a `None` kerning placeholder under the same feature flag. There is no indication of a security fix or vulnerability being addressed.
Security candidatefeat(ui/fonts): add kernings field into rust font files FontInfo struct.by kopecdav · 47622946 · Apr 15, 2026 · 21 filesMessage 62 · AdequateInformational 15Details
Commit message · kopecdav
feat(ui/fonts): add kernings field into rust font files FontInfo struct.
[no changelog]
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 15/100
This commit is a routine user-interface change: it adds a new optional 'kernings' field to font description structures in the Trezor firmware's Rust code. Every font is simply set to 'None' (no kerning data), so nothing actually changes in behavior. There is no security issue visible in this patch.
✓ 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 pathtranslation-only discount
AI analysis · Informational 15/100
This commit is a routine regeneration of font data files used for displaying translated text on Trezor hardware wallets. It only changes glyph bitmap data for accented characters in various languages and font styles. There is no executable code change, no change to cryptographic logic, and no indication of a security fix or vulnerability.
✓ Descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
boot or update pathauthentication path
AI analysis · Informational 15/100
This commit is a test-only cleanup that renames parameters, refactors emulator selection logic, and fixes test helpers for navigating on-screen keyboards during recovery tests. It does not change any firmware, device, or production code, and there is no indication it fixes or introduces a security vulnerability.
tests(upgrade): migrate upgrade suites to direct EmulatorWrapper flow
[no changelog]
60/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
boot or update pathauthentication path
AI analysis · Informational 15/100
This commit only changes automated test code for firmware upgrade scenarios. It does not modify the Trezor firmware itself, wallet logic, or any code that runs on user devices. There is no security-relevant change to end users.