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-priorityci: simplify release-emu.ymlby Martin Milata · d9ee4759 · May 29, 2026 · 2 filesMessage 57 · ThinInformational 15Details
Commit message · Martin Milata
ci: simplify release-emu.yml
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
documentation-only discount
AI analysis · Informational 15/100
This commit is a routine cleanup of the GitHub Actions workflow that builds Trezor emulator releases. It removes unused matrix dimensions (like ASan variants), drops environment variables that were no longer needed, changes a default so the bootloader build step is skipped unless explicitly requested, and passes the device model into the build command. There is no change to the firmware code that runs on user devices, no fix for a vulnerability, and no security-relevant behavior change.
✓ 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 removes the full build directory path from the firmware binary by switching from __FILE__ to __FILE_NAME__ in two error-handling locations. The change reduces information leakage but does not fix a memory corruption or logic bug. It is a hardening/information-disclosure reduction patch, not a fix for an actively exploitable vulnerability.
Lower-prioritybuild: build-docker.sh: adjust paths for xtaskby Martin Milata · 7e4fecf0 · May 29, 2026 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · Martin Milata
build: build-docker.sh: adjust paths for xtask
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 updates a build script to look for compiled firmware files in a new directory path (`build-xtask/artifacts/...`) instead of the old path (`build/...`). It is a routine build-system path adjustment with no apparent security relevance.
Lower-prioritychore(core): update fixtures for T3W1by Martin Milata · 5185f493 · May 29, 2026 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · Martin Milata
chore(core): update fixtures for T3W1
[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 updates expected test result fingerprints (hashes) in a UI test fixtures file for the T3W1 device model. No actual product code, firmware logic, or user-facing behavior was changed. It is a routine test-maintenance change with no security relevance.
Lower-prioritydocs(core): Document absence of SD salt overwrite.by Andrew Kozlik · b9f92ba8 · May 29, 2026 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Andrew Kozlik
docs(core): Document absence of SD salt overwrite.
[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 only adds comments explaining why the code does not overwrite SD card salt files with random data before deleting them. It changes no behavior, logic, or security controls. The documentation argues that overwriting is unnecessary because the encrypted data key stored inside the device is already wiped when the salt is changed, so an old recovered salt cannot decrypt anything.
AI review queuedchore(deps): bump urllib3 in /tools/automatic_battery_testerby dependabot[bot] · 2bda8865 · May 29, 2026 · 1 fileMessage 93 · StrongInformational 15Details
Commit message · dependabot[bot]
chore(deps): bump urllib3 in /tools/automatic_battery_tester
Bumps [urllib3](https://github.com/urllib3/urllib3) from 2.6.3 to 2.7.0. - [Release notes](https://github.com/urllib3/urllib3/releases) - [Changelog](https://github.com/urllib3/urllib3/blob/main/CHANGES.rst) - [Commits](https://github.com/urllib3/urllib3/compare/2.6.3...2.7.0)
✓ 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
This is a routine automated dependency update by Dependabot that bumps the urllib3 library from version 2.6.3 to 2.7.0 inside a small internal testing tool (automatic_battery_tester). The change is a single line in a requirements file. There is no indication in the commit that this fixes a security issue, and the tool is not part of the Trezor firmware that runs on user devices.
Lower-prioritychore: drop `oldNixpkgs` from `shell.nix`by M1nd3r · 4501b2a1 · May 29, 2026 · 3 filesMessage 80 · StrongInformational 15Details
Commit message · M1nd3r
chore: drop `oldNixpkgs` from `shell.nix`
Importing `oldNixpkgs` is no longer needed, as SDL2 was replaced by SDL3, and they are not used anywhere else.
[no changelog]
80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
AI analysis · Informational 15/100
This is a routine build-environment cleanup. The developers removed an old copy of Nix packages (oldNixpkgs) used only for older SDL2 graphics libraries, because the project now uses SDL3. They added a compatibility package so old test emulators still work, and updated documentation and a download script to use a fuller Nix shell environment. There is no indication this changes any wallet, firmware, or cryptographic code, and no security relevance is claimed.
AI review queuedfix(core): skip bitcoin.signtx_decred test on BTC_ONLYby M1nd3r · 80c9f204 · May 29, 2026 · 1 fileMessage 72 · AdequateInformational 15Details
Commit message · M1nd3r
fix(core): skip bitcoin.signtx_decred test on BTC_ONLY
[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 or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit only changes a test file so that a Decred signing test is skipped when the firmware is built in Bitcoin-only mode. It does not modify any production firmware code, wallet behavior, or cryptographic logic. There is no security issue here.
chore(python): chunkify printed fingerprints of fw-images
- Chunkifies fingerprints of fw-images, secmon, bootloader... - Makes visual comparison of expected and actual fingerprints easier.
[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
update trustboot or update path
AI analysis · Informational 15/100
This commit only changes how long hexadecimal fingerprints and hashes are displayed to users. It inserts spaces every four characters so the strings are easier to read and compare visually. There is no change to security logic, cryptography, or how firmware is verified.
AI review queuedtest(solana): sign message with many cosignersby Jakub Janků · e0b2dea3 · May 29, 2026 · 2 filesMessage 67 · AdequateInformational 15Details
Commit message · Jakub Janků
test(solana): sign message with many cosigners
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
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit only adds a new test case for Solana message signing with 64 cosigners and updates the expected screen snapshots used by automated testing. It does not change any firmware code that runs on the Trezor device, so it cannot introduce a security vulnerability or fix one.
AI review queuedchore(ci): skip `auto-assign` job for botsby M1nd3r · c784179b · May 29, 2026 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · M1nd3r
chore(ci): skip `auto-assign` job for bots
[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 pathdocumentation-only discountsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This is a minor GitHub workflow change that stops an automatic pull-request assignment job from running when the pull request is opened by a bot account (whose username ends with '[bot]'). It has no security relevance to the Trezor firmware itself or to user funds.
Security candidatechore(docs): fix and improve doc-linksby M1nd3r · 92a46a45 · May 29, 2026 · 8 filesMessage 80 · StrongInformational 15Details
Commit message · M1nd3r
chore(docs): fix and improve doc-links
- Fixed broken links to non-existent places. - Replaced doc links to github by relative links to the md files. This allows to use `docs.trezor.io` without being needlessly transported to the github version of the linked doc page. - Replaced "links to symlinks" by" links to the original files". Symlinks do not work nicely on github - user must manually find the symlinked doc page.
[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 pathdocumentation-only discount
AI analysis · Informational 15/100
This commit only updates documentation links inside the Trezor firmware repository. It fixes broken links, replaces GitHub URLs with relative links, and removes a symlinked documentation file. There are no code changes and no security impact.
Lower-prioritychore(translations): add [no changelog] to Crowdin pull commit messageby Michal Kazda · 355c2a50 · May 29, 2026 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Michal Kazda
chore(translations): add [no changelog] to Crowdin pull commit message
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 only changes the automated message used when the project pulls translation updates from Crowdin. It adds a '[no changelog]' tag so the translation sync does not appear in release notes. There is no security relevance.
Lower-prioritychore(translations): PR will be draft, no longer limited to only main [no changelog]by Michal Kazda · 9340c66e · May 29, 2026 · 1 fileMessage 77 · AdequateInformational 15Details
Commit message · Michal Kazda
chore(translations): PR will be draft, no longer limited to only main [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
documentation-only discount
AI analysis · Informational 15/100
This is a routine GitHub workflow change for managing translation updates. It removes a restriction that previously allowed the workflow to run only from the 'main' branch, and it makes the automatically created pull request a draft instead of a normal one. There is no security relevance in the code change itself.
AI review queuedfix(core): fix device_menu crash on devices without a serial numberby cepetr · 51790381 · May 29, 2026 · 1 fileMessage 62 · AdequateInformational 20Details
Commit message · cepetr
fix(core): fix device_menu crash on devices without a serial number
[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 a crash in the Trezor device's on-screen 'About device' menu when the device lacks a serial number. It wraps the serial-number lookup in a try/except block and displays 'N/A' instead of crashing. There is no indication this can be exploited by an attacker; it is a robustness fix for a user-interface screen.
- Verify tap_to_wake is True by default on a freshly initialized device - Verify the checkbox toggles the stored value and a round-trip restores the original value - Verify tap_to_wake is absent from the device menu on uninitialized device - All three tests skip gracefully when USE_TOUCH_WAKEUP is not compiled in
[no changelog]
78/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
AI analysis · Informational 15/100
This commit only adds new automated tests for the 'tap to wake' feature on the T3W1 hardware model. It does not change any firmware code that runs on the device, so it cannot introduce a security vulnerability or fix one. The tests verify that the setting is on by default, that the checkbox toggles it, and that the option is hidden on an uninitialized device.
AI review queuedchore(core): adjust json vendorheadersby M1nd3r · 82f1c71b · May 27, 2026 · 14 filesMessage 57 · ThinInformational 15Details
Commit message · M1nd3r
chore(core): adjust json vendorheaders
[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 only renames a single JSON key from '_reserved' to 'reserved' across 14 vendor header configuration files. No security settings, permissions, or values are changed. It appears to be a routine cleanup to match a naming convention elsewhere in the build tooling.
chore(python): add sanity check to `Struct`'s parsing
[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
boot or update path
AI analysis · Low 35/100
This commit adds a new optional sanity-check layer to the Python code that parses Trezor firmware files. It verifies that re-encoding a parsed firmware image produces the exact same bytes, and that any 'reserved' fields are all zeroes. The change is defensive: it can help detect malformed or tampered firmware images during analysis, but the strict mode is off by default, so existing behavior is preserved unless a caller explicitly opts in.
Security candidaterefactor(core/delizia): remove `Header`-related helpers from `Frame`by Roman Zeyde · 9c42d459 · May 27, 2026 · 22 filesMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde
refactor(core/delizia): remove `Header`-related helpers from `Frame`
[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
boot or update pathauthentication path
AI analysis · Informational 15/100
This commit is a straightforward internal code cleanup in the Trezor firmware's user interface code. It moves header-related helper methods (like title alignment, cancel/menu buttons, and styling) from the Frame component to the Header component. All existing call sites are updated to build a Header first and then pass it into a Frame. There is no change to user-visible behavior or security logic.
Lower-priorityci: run python unit tests with btc-only FW (nightly)by M1nd3r · 404c3066 · May 27, 2026 · 1 fileMessage 72 · AdequateInformational 15Details
Commit message · M1nd3r
ci: run python unit tests with btc-only FW (nightly)
[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
documentation-only discount
AI analysis · Informational 15/100
This commit only changes the project's automated testing configuration. It adds a nightly CI job that runs Python unit tests against a Bitcoin-only firmware build in addition to the existing universal build. There is no change to the actual firmware code, no bug fix, and no security-relevant behavior change.
Lower-priorityci(core): increase device_tests timeout to 50m on T3W1by Roman Zeyde · 6c060115 · May 27, 2026 · 1 fileMessage 87 · StrongInformational 15Details
Commit message · Roman Zeyde
ci(core): increase device_tests timeout to 50m on T3W1
When running with 2 workers, T3W1 UI tests timeout after 40m.
[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
documentation-only discount
AI analysis · Informational 15/100
This commit only adjusts how long automated test jobs are allowed to run in GitHub Actions for one Trezor hardware model (T3W1). It extends a pytest session timeout from 40 minutes to 50 minutes and the overall CI job timeout from 50 minutes to 60 minutes, but only for the T3W1 model. There is no change to firmware code, cryptography, user data handling, or device behavior. It is purely a CI configuration tweak to prevent false timeout failures when running tests with two workers.
Lower-priorityfix(solana): break import cycle in types.pyby Jakub Janků · badee388 · May 26, 2026 · 1 fileMessage 80 · StrongInformational 15Details
Commit message · Jakub Janků
fix(solana): break import cycle in types.py
types.py: from .transaction.parse import parse_pubkey transaction/__init__.py: from ..types import AddressType
The cycle is broken by making the first import lazy.
80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
AI analysis · Informational 15/100
This is a routine code cleanup that fixes a Python import cycle (two files importing each other) by moving one import to happen only when a specific function runs. There is no security-relevant change visible in the diff.
AI review queuedrefactor(core): move is_printable_ascii to commonby Jakub Janků · e07fff7d · May 26, 2026 · 3 filesMessage 80 · StrongInformational 15Details
Commit message · Jakub Janků
refactor(core): move is_printable_ascii to common
This is a preparatory commit. The utility will be used in solana off-chain message parsing, so we move it from cardano/helpers/utils to common/signverify.
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
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit simply moves a small helper function that checks whether a string contains only normal printable characters from one file to another shared location. No behavior changes, no bug fixes, and no security implications are visible in the code change itself.
AI review queuedtest(solana): sign and verify off-chain messagesby Jakub Janků · 2124b6b3 · May 26, 2026 · 4 filesMessage 67 · AdequateInformational 15Details
Commit message · Jakub Janků
test(solana): sign and verify off-chain messages
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
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit only adds new automated tests for a Solana feature that signs and verifies off-chain messages. It does not change the actual wallet firmware code that users rely on, so it cannot by itself introduce a security vulnerability or fix one. The tests exercise parsing, signing, and verification logic and include checks for invalid inputs.
The implementation deviates from the spec in the supported message formats --- the spec says that HW wallets should clear- sign format 0 (ascii), blind-sign format 1 (short utf-8), and not support format 2 (long utf-8) messages. We try to clear-sign messages irrespective of the format. The rationale is that we __can__ display even longer messages incl. some non-ascii chars and thus the limitation is arbitrary. Moreover, this restriction is removed in the newer v1 spec, see:
✓ 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 boundarysigning or wallet path
AI analysis · Low 36/100
This commit adds a new Trezor feature: signing and verifying Solana off-chain messages. It is a feature implementation, not a fix for a known vulnerability. The code introduces new message types, parsing logic, user confirmation screens, and client helpers. There is no direct evidence in the commit of a security bug, but any new signing path increases the attack surface and must be reviewed carefully for parsing mistakes or user-interface spoofing.