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.
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.
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.
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.
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.
Security candidatefix(solana): support multisig transfer instructionby Jakub Janků · ad2a4443 · May 26, 2026 · 6 filesMessage 100 · StrongLow 37Details
Commit message · Jakub Janků
fix(solana): support multisig transfer instruction
The test vector is based on the trezorctl command in the GH issue. The signing account was changed to 14CCvQzQzHCVgZM3j9soPnXuJXh1RmCfwLVUcdfbZVBS (address derived from m/44'/501'/0'/0').
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
Why it was queued
signing boundaryfuzzing or regression evidencesigning or wallet path
AI analysis · Low 37/100
This commit updates the Trezor firmware's Solana support so that the standard 'Transfer' instruction can be used with multi-signature accounts. Previously, the firmware treated this instruction as non-multisig, which likely caused it to reject or mis-handle legitimate Solana transactions that required multiple signers. The change is a feature/fix to broaden supported transaction shapes, not a patch for an active vulnerability.
AI review queuedchore(translations): minor string update [no changelog]by Michal Kazda · b2d0adf4 · May 26, 2026 · 2 filesMessage 77 · AdequateInformational 15Details
Commit message · Michal Kazda
chore(translations): minor string update [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
signing or wallet pathtranslation-only discountsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit changes one German translation string to add the word 'ist' (is) for consistency, and updates the corresponding translation signature metadata. There is no code or security change.
Lower-prioritychore(translations): update fixtures [no changelog]by Michal Kazda · 111b748d · May 26, 2026 · 1 fileMessage 77 · AdequateInformational 15Details
✓ 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 updates expected test result hashes (called 'fixtures') in a single test data file. It does not change any firmware, application, or cryptographic code. The hash changes are consistent with updated screen snapshots or translated text strings used by the test suite. There is no indication of a security fix or vulnerability.
AI review queuedchore(translations): sync Crowdin translations [no changelog]by Michal Kazda · c565e45c · May 26, 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 for the Trezor hardware wallet firmware. It changes wording for things like Ethereum transaction prompts, Bluetooth device labels, backup/recovery instructions, and passphrase prompts. There are no code logic changes, no new features, and no security fixes.
✓ 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 commit adds new regulatory certification screens for Brazil, Ghana, and Nigeria to the Trezor hardware wallet's user interface. It is purely a compliance/labeling change: new image files are included and a list of regulatory regions is updated. There is no indication of any security bug, vulnerability, or behavior change affecting funds or device security.
AI review queuedchore(core/eckhart): update About screenby obrusvit · 41ff96ea · May 26, 2026 · 10 filesMessage 57 · ThinInformational 15Details
Commit message · obrusvit
chore(core/eckhart): update About screen
[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 is a routine user-interface update for the Trezor hardware wallet's 'About' screen. It adds a 'Made in Ostrava, Czechia' label and, on devices that use serial numbers, displays the serial number. There is no security-relevant change.
AI review queuedchore(tests,ethereum): tests from Suiteby Ioan Bizău · fc87316d · May 26, 2026 · 1 fileMessage 67 · AdequateInformational 15Details
Commit message · Ioan Bizău
chore(tests,ethereum): tests from Suite
[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
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit only adds new test cases for Ethereum transaction signing. It does not change any production code, so it cannot introduce a security vulnerability by itself. The test data includes sample transactions and their expected signatures, used to verify the device signs complex DeFi transactions correctly.
Lower-prioritychore: update fixturesby Ioan Bizău · 6756ee77 · May 26, 2026 · 1 fileMessage 40 · ThinInformational 15Details
Commit message · Ioan Bizău
chore: update fixtures
[no changelog]
40/100 · ThinMessage clarity
✓ Subject identifies a change✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 15/100
This commit only updates the expected test result hashes (called 'fixtures') for the user-interface test suite. It adds new test cases for LiFi token-swap/bridge transactions and refreshes a few existing pagination test hashes. There is no change to the actual Trezor firmware code that runs on the device, so this cannot introduce a security vulnerability by itself.
AI review queuedchore(ethereum): preload 6kb of databy Ioan Bizău · a36d79b7 · May 26, 2026 · 2 filesMessage 57 · ThinInformational 18Details
Commit message · Ioan Bizău
chore(ethereum): preload 6kb of data
[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 18/100
This is a small internal tuning change for Trezor's Ethereum app. It increases the amount of transaction data the device preloads from 4 KB to 6 KB before asking the user to confirm. The only visible effect is fewer back-and-forth data requests for Ethereum transactions with large data payloads. There is no direct security fix or vulnerability being patched.
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Moderate 61/100
This patch fixes a bug in how Trezor's Ethereum app checked whether a transaction was meant for the claim.xyz rewards distributor. Previously it compared a lowercase hex string (like '0xabc...') against raw address bytes, which would never match. The fix compares raw bytes to raw bytes. The practical effect is that the special claim.xyz user interface and safety checks now actually trigger for the intended contract, instead of silently being skipped for all transactions.
Security candidatechore(python): minor adjustments to trezorlib/firmwareby M1nd3r · 6124c57b · May 25, 2026 · 2 filesMessage 62 · AdequateInformational 15Details
Commit message · M1nd3r
chore(python): minor adjustments to trezorlib/firmware
[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 · Informational 15/100
This is a routine Python code change in Trezor's firmware handling library. It adds support for a new Nordic (NRF) firmware header type and helper methods to insert cryptographic signatures into firmware image structures, plus a minor formatting tweak. There is no indication of a security vulnerability or fix.
chore(trezorlib): fix `BootloaderV2Image` handling in `firmware_headers.py`
[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
update trustboot or update path
AI analysis · Low 32/100
This commit fixes how Trezor's Python library validates signatures for a new bootloader image format (BootloaderV2). Before the fix, the code ignored the signature bitmask and simply checked every signature against every public key in order. This could let an invalid or mismatched signature slip through, or cause validation to fail incorrectly. The patch makes the library read the bitmask, count signatures correctly, and only verify the signatures the image actually claims to have signed with. It also adds a missing signature-status line to the printed header. The change is in developer/CLI tooling, not the device firmware itself, so a direct attack on a real device is unlikely, but the library is used to inspect and validate firmware files.
✓ 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 simply reserves an unused number (27) in a list of device capabilities so it cannot accidentally be reused later. It does not change any running code, fix a bug, or introduce a security issue.
Security candidatechore(core/eckhart): drop warning header if title is emptyby Roman Zeyde · af8bfeb3 · May 25, 2026 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde
chore(core/eckhart): drop warning header if title is empty
[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 · Informational 15/100
This is a minor user-interface cleanup for the Trezor hardware wallet's Eckhart layout. It removes the warning header from a screen when the title text is empty, so users don't see a blank header. There is no security-relevant change.