TZ
← All projectsTrezor

Trezor firmware

Firmware monorepo for Trezor One, Model T, and Safe devices.

BitcoinHardware walletsNormal
Repository coverage

2673 commits in the local evidence base

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.

923security candidates311second-pass queue2662AI analyses
323commits · 30 days
552commits · 60 days
1337commits · 180 days
2644commits · 365 days
Backfill bands
Aug 5 → Feb 61298 seen115 candidatesComplete
Feb 6 → Jun 6775 seen58 candidatesComplete
Jun 6 → Jul 6217 seen13 candidatesComplete
Jul 6 → Aug 5360 seen54 candidatesComplete
Commit communication

Does the history explain itself?

Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.

66/100 average clarity
482Strong · 80–100
1323Adequate · 60–79
863Thin · 40–59
5Opaque · 0–39
Read the scoring rubric →
Developer activity

Who is changing the project?

Public Git author strings; identities are not independently verified.

DeveloperCommitsCandidatesAnalyzedHigh riskMessage avg.
tychovrahe292104292461
Roman Zeyde579182576272
PrisionMike945994273
Andrew Kozlik712470265
obrusvit21376213164
Jakub Janků22919182
Martin Pastyřík24823173
cepetr19682196059
Ioan Bizău23076230059
M1nd3r20772207071
Lukas Bielesch846784067
Martin Milata17446174063
Analysis record

Published AI watches

Last scanned 46 minutes ago

Informational 15 AI analysisMessage 72 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

test(core/ethereum): EIP-7702 tuple nonce must be non-zero

This commit only changes test data and expected screen fingerprints for Trezor's Ethereum EIP-7702 signing tests. It replaces one test case that used a nonce of 0 with the same scenario using a nonce of 67, because the real-world protocol …

Test-only change with no production code modificationEIP-7702 tuple nonce constraint reflected in test dataExpected UI test hashes regenerated for renamed test vector
76cd2ed4by Roman Zeyde+28−282 files
No security note in commit
Informational 15 AI analysisMessage 85 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

build(core): propagate emulator feature from projects

This commit is a build-system cleanup for the Trezor firmware's Rust code. It changes how the 'emulator' feature flag is passed between different software components. The firmware project no longer supports the emulator feature (which make…

5642074cby matejcik+5−44 files
No security note in commit
Low 27 AI analysisMessage 93 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(core): digest EIP-1559/EIP-7702 magic without RLP encoding

This commit is a small internal cleanup in Trezor's Ethereum signing code. It changes how two special 'magic' bytes (the transaction type markers for newer Ethereum transaction formats) are fed into the cryptographic hash. Previously the c…

Refactor of cryptographic hashing path for Ethereum transaction signingRemoval of implicit assumption that RLP encodes small integers as single bytesChange affects EIP-1559 and EIP-7702 transaction type markers
2362bc6fby Roman Zeyde+6−32 files
No security note in commit
Informational 15 AI analysisMessage 98 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

test(core): test RLP self-encoding bytestrings

This commit only adds a new unit test to the Trezor firmware codebase. It verifies that very short Ethereum RLP-encoded byte strings and small positive integers encode to themselves, which is a known property of the RLP specification. No p…

41538733by Roman Zeyde+16−01 file
No security note in commit
Informational 18 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

chore(core): remove sha1 from regular FW

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
16d15774by M1nd3r+0−2006 files
No security note in commit
Low 27 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(crypto): pass static public key to `noise_xxpsk3_*_init()`

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
1b3128a4by Ondřej Vejpustek+20−92 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(crypto): change order of parameters

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.

33a119eeby Ondřej Vejpustek+15−151 file
No security note in commit
Informational 15 AI analysisMessage 47 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(crypto): prefix enums

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 …

f1c74046by Ondřej Vejpustek+26−222 files
No security note in commit
Low 37 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

fix(crypto): prevent calling `memzero(NULL, ...)`

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
0394e934by Ondřej Vejpustek+11−51 file
No security note in commit
Informational 18 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(crypto): return remote static key from `noise_xxpsk3_*_handle_*()`

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
3ead1aa6by Ondřej Vejpustek+79−423 files
No security note in commit
Informational 15 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): rename ProjectProfile to ProjectConfig

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…

581b37bcby cepetr+18−182 files
No security note in commit
Informational 15 AI analysisMessage 47 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): introduce ResolvedBuildArgs

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…

c24d1be8by cepetr+99−295 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): make build args tristate

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…

271e249aby cepetr+57−561 file
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): introduce build options

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…

75085e2aby cepetr+275−2007 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

feat(xtask): introduce build presets

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…

eefe8f7fby cepetr+397−419 files
No security note in commit
Informational 15 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

feat(xtask): upgrade manifest.yaml for tf-tools 0.3.0

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…

57b28b89by cepetr+86−1351 file
No security note in commit
Informational 15 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

feat(core): add tf-tools 0.3.1+ configuration file

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.…

e554f618by cepetr+8−01 file
No security note in commit
Informational 15 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): rename ResolvedBuild to ResolvedBuildFeatures

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…

aaa2780bby cepetr+5−52 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

fix(core): fix up clippy warnings in storage.rs

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…

62354461by cepetr+2−21 file
No security note in commit
Informational 15 AI analysisMessage 47 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): simplify dependencies

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…

6268349fby cepetr+9−193 files
No security note in commit
Repository ledger

Explore captured commits

Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.

Security candidatefix(core): adjust random part of BLE name during pairingby tychovrahe · dc7deec3 · Oct 17, 2025 · 4 filesMessage 67 · AdequateInformational 23Details
Commit message · tychovrahe

fix(core): adjust random part of BLE name during pairing

[no changelog]

67/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Names security-relevant behavior explicitly! No meaningful explanatory body
Why it was queued
boot or update path
AI analysis · Informational 23/100

This commit changes how Trezor T3W1 devices generate their temporary Bluetooth (BLE) name during pairing. Previously the name contained three random characters from a pool of 26 letters plus 10 digits. Now it uses a digit-letter-digit pattern. This reduces the number of possible names from 36×36×36 (about 46,656) to 10×26×10 (2,600), making it easier for a nearby attacker to guess or collide with the advertised name during pairing. The change appears intentional to make the displayed name easier to read or type, but it weakens the randomness that helps distinguish devices.

Security candidatechore: bump versionsby obrusvit · 93173133 · Oct 16, 2025 · 13 filesMessage 40 · ThinInformational 15Details
Commit message · obrusvit

chore: bump versions

[no changelog]

40/100 · ThinMessage clarity
✓ Subject identifies a change✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathboot or update path
AI analysis · Informational 15/100

This commit only updates version numbers and translation metadata across Trezor firmware components. It does not change any executable code, fix any behavior, or alter security logic. There is no indication this is a security patch.

Lower-priorityfix(core): use dummy Version in About screenby obrusvit · 44acdd2d · Oct 16, 2025 · 2 filesMessage 90 · StrongInformational 15Details
Commit message · obrusvit

fix(core): use dummy Version in About screen

- we traverse the menu in click tests, this commit ensures that we do
not have to update fixtures after every change of firmware (or in the
future also emu-ble) version

[no changelog]

90/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Mentions testing or verification
AI analysis · Informational 15/100

This change only affects what version numbers are shown on the device's 'About' screen during automated user-interface tests. When running in the software emulator, real version numbers are replaced with placeholder '0.0.0.0' values so that test snapshots don't have to be updated every time the firmware version changes. There is no security issue here.

Security candidatechore(core): rename `ble__waiting_for_host` to `words__waiting_for_host`by Roman Zeyde · f4424b65 · Oct 16, 2025 · 13 filesMessage 77 · AdequateInformational 15Details
Commit message · Roman Zeyde

chore(core): rename `ble__waiting_for_host` to `words__waiting_for_host`

It will be also used for warning in case `ButtonRequest` ACK is delayed.

[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 pathboot or update path
AI analysis · Informational 15/100

This commit is a simple internal rename of a user-interface text label from 'ble__waiting_for_host' to 'words__waiting_for_host'. It does not change what the device does, what data it handles, or how it protects that data. It only reorganizes where a translation string is stored so it can be reused for a future on-screen message.

Lower-priorityfix(core): set high BLE speed for translations uploadby tychovrahe · 6159cfd9 · Oct 16, 2025 · 4 filesMessage 62 · AdequateInformational 18Details
Commit message · tychovrahe

fix(core): set high BLE speed for translations upload

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 18/100

This commit is a small cleanup and bug fix for the T3W1 hardware wallet's Bluetooth translation upload. It makes sure the Bluetooth connection switches to a faster mode when uploading language translations, just like it already did when uploading custom homescreens. The change also wraps the speed-up logic in a reusable helper so it is harder to accidentally leave high-speed mode on. There is no direct security vulnerability here; it is a reliability/performance improvement.

AI review queuedchore: let br_code be Warningby Ioan Bizău · cb65178a · Oct 16, 2025 · 2 filesMessage 57 · ThinInformational 15Details
Commit message · Ioan Bizău

chore: let br_code be Warning

[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 is a minor internal cleanup change in the Trezor hardware wallet firmware. It removes explicit button-request type labels from two warning screens in the Ripple and Stellar cryptocurrency apps, letting them default to a generic 'Warning' category instead of 'ConfirmOutput'. There is no visible security bug or vulnerability being fixed.

AI review queuedchore: warning for Ripple destination tag missingby Ioan Bizău · 99b3f9fb · Oct 16, 2025 · 9 filesMessage 57 · ThinInformational 20Details
Commit message · Ioan Bizău

chore: warning for Ripple destination tag missing

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 20/100

This commit adds a user-facing warning in Trezor hardware wallets when signing a Ripple transaction that does not include a destination tag. Destination tags are often required by exchanges to credit the correct account, so missing one can lead to lost funds. The change only shows an extra warning screen; it does not alter how transactions are validated or signed.

AI review queuedchore(tests): long values in payment requestsby Ioan Bizău · ba67a2de · Oct 16, 2025 · 2 filesMessage 67 · AdequateInformational 12Details
Commit message · Ioan Bizău

chore(tests): long values in payment requests

[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 12/100

This commit only adds new automated tests. It checks that Trezor devices correctly reject payment-request amounts encoded with the wrong number of bytes (for Bitcoin) or values that are too large (for Ethereum). There is no change to the actual firmware or production code, and the commit message explicitly marks itself as a test-only chore with no changelog.

AI review queuedfeat: 32 bytes amount in ETH payment requestsby Ioan Bizău · abb2816c · Oct 16, 2025 · 15 filesMessage 57 · ThinLow 36Details
Commit message · Ioan Bizău

feat: 32 bytes amount in ETH payment requests

[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 · Low 36/100

This commit changes how payment-request amounts are encoded in Trezor firmware. Previously amounts were 8-byte numbers; now they can be 8 or 32 bytes, with 32 bytes used for Ethereum/EVM transactions. The change is a feature update to support larger EVM token amounts, not a clearly disclosed security fix. It does add length checks and an overflow guard, which reduces risk, but the change touches the trusted payment-request verification path, so any bug there could affect what amount the user is asked to approve.

Lower-prioritychore(core): update fixturesby Ioan Bizău · 9d4a8a13 · Oct 16, 2025 · 1 fileMessage 47 · ThinInformational 15Details
Commit message · Ioan Bizău

chore(core): update fixtures

[no changelog]

47/100 · ThinMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 15/100

This commit only updates the expected visual test snapshots (called 'fixtures') used by Trezor's automated UI tests. It adds new test cases and refreshes image hashes because the underlying tests changed, for example replacing the Ethereum 'Holesky' testnet with the newer 'Hoodi' testnet. No actual device firmware code was changed, so this cannot introduce a security vulnerability on its own.

Lower-priorityfix(protob): remove wrong message annotationby Ioan Bizău · 786ba49f · Oct 16, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · Ioan Bizău

fix(protob): remove wrong message 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 removes a single incorrect documentation comment ('@next TxRequest') from a protocol buffer definition file. It is a documentation-only cleanup with no change to executable code, message structure, or device behavior. There is no security relevance.

Lower-priorityfix(core): let pm stabilize when resuming from suspend.by kopecdav · 42140928 · Oct 16, 2025 · 1 fileMessage 62 · AdequateLow 29Details
Commit message · kopecdav

fix(core): let pm stabilize when resuming from suspend.

[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 · Low 29/100

This is a one-line firmware change for the Trezor hardware wallet's power manager on STM32U5 chips. When the device wakes from a low-power suspend state, the code now marks the power-manager state machine as 'not yet stabilized.' This is a defensive fix: without it, the firmware might act on stale or transient power-state readings right after wakeup. The commit itself does not explain any security impact, and no public references were provided.

Lower-priorityrefactor(core): the display driver functions' API has been reviewed. The data types have been adapted to fit the actual data passed/used/returned. Since the backlight driver has undergone the similar change, the display driver code has been adapted to respect the changes.by bleska · e9805643 · Oct 16, 2025 · 11 filesMessage 62 · AdequateInformational 15Details
Commit message · bleska

refactor(core): the display driver functions' API has been reviewed. The data types have been adapted to fit the actual data passed/used/returned. Since the backlight driver has undergone the similar change, the display driver code has been adapted to respect the changes.

[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 straightforward code cleanup (refactor) in the Trezor firmware's display driver. It changes the data types used for backlight levels from signed integers (int) to unsigned 8-bit values (uint8_t), and changes the display_set_backlight function to return true/false (bool) instead of the brightness level. There is no indication this fixes a security bug or introduces a vulnerability.

Security candidaterefactor(core): the display driver's usage from RUST code has been adapted to the modified C driver code. The "backlight" function has been divided into "set_backlight" and "get_backlight" ftuncions. The data types respect the ones from C code. The "set_backlight" function parameter "value" (level by bleska · f01dfdeb · Oct 16, 2025 · 7 filesMessage 85 · StrongInformational 22Details
Commit message · bleska

refactor(core): the display driver's usage from RUST code has been adapted to the modified C driver code. The "backlight" function has been divided into "set_backlight" and "get_backlight" ftuncions. The data types respect the ones from C code. The "set_backlight" function parameter "value" (level of backlight) has data type uint8 (u8). Some function call, however, use the u16 data type for this argument. It's necessary to review whether there isn't any problem with this.

[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
boot or update path
AI analysis · Informational 22/100

This commit refactors how the Trezor hardware wallet's screen brightness is controlled from Rust code. The old single 'backlight' function is split into separate 'set_backlight' and 'get_backlight' functions, and the brightness value type is narrowed from a wider signed integer to an unsigned 8-bit value (0-255). The commit message and code comments explicitly flag that some callers still pass a 16-bit value, and the patch adds defensive clamping to 255 in those cases. There is no direct evidence of an exploitable security bug, but the change is clearly defensive hardening against potential type-conversion issues.

Security candidaterefactor(core): refactoring the function calls of display backlight driver in RUST so that the u16 data types have been converted to u8 ones. The code has been reviewed whether it's safe - no problems detected.by bleska · 99d21526 · Oct 16, 2025 · 4 filesMessage 74 · AdequateInformational 18Details
Commit message · bleska

refactor(core): refactoring the function calls of display backlight driver in RUST so that the u16 data types have been converted to u8 ones. The code has been reviewed whether it's safe - no problems detected.

[no changelog]

74/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Explains rationale or failure mode! No meaningful explanatory body
Why it was queued
boot or update path
AI analysis · Informational 18/100

This is a code cleanup change in the Trezor hardware wallet's screen-brightness controls. It changes internal number types from 16-bit to 8-bit to match the display driver, and removes temporary workarounds that converted the values. The commit message says the code was reviewed and no safety problems were found. There is no indication this fixes an active security bug; it is a defensive type-safety refactor.

Security candidaterefactor(core): the backlight driver functions 'API has been reviewed. The data types have been adapted to fit the actual data passed/used/returned.by bleska · 4f5ff591 · Oct 16, 2025 · 4 filesMessage 62 · AdequateInformational 17Details
Commit message · bleska

refactor(core): the backlight driver functions 'API has been reviewed. The data types have been adapted to fit the actual data passed/used/returned.

[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
authentication path
AI analysis · Informational 17/100

This commit is a routine code cleanup of the screen backlight driver in Trezor hardware wallets. It changes function return types from plain integers to booleans and from signed to unsigned 8-bit values, and removes redundant range checks. There is no indication this fixes a security vulnerability; it appears to be a refactoring for code clarity and type correctness.

Lower-priorityfix: use profile dir for Tropic model outputby Ioan Bizău · 0eb83eab · Oct 16, 2025 · 1 fileMessage 57 · ThinInformational 16Details
Commit message · Ioan Bizău

fix: use profile dir for Tropic model output

[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 16/100

This is a small fix in Trezor's internal emulator tooling. It changes where a helper program (the 'Tropic model') writes its output file, so the file goes into a per-profile directory instead of an unspecified default location. The change only affects internal test/development infrastructure and does not appear to fix a vulnerability in the actual Trezor hardware wallet or its firmware.

Lower-priorityfix(core): allow BLE pairing mode entry without setting device nameby tychovrahe · 61241a58 · Oct 16, 2025 · 2 filesMessage 62 · AdequateInformational 18Details
Commit message · tychovrahe

fix(core): allow BLE pairing mode entry without setting device name

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 18/100

This commit fixes a bug in the Trezor hardware wallet's Bluetooth code. Previously, entering Bluetooth pairing mode required providing a device advertising name; if no name was supplied, the function immediately failed. The change now allows pairing mode to start without a name, only rejecting names that are too long. The changelog describes this as fixing 'ble-adv-start without args' in the production-test (factory/testing) project. This is a functional bug fix rather than a clear security vulnerability.

Lower-priorityfix(core): fix fw hash calculationby cepetr · b717fd15 · Oct 16, 2025 · 1 fileMessage 57 · ThinModerate 57Details
Commit message · cepetr

fix(core): fix fw hash calculation

[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 · Moderate 57/100

This commit fixes an off-by-one error in the STM32U5 flash memory layout definition for Trezor's hardware wallet firmware. The number of sectors covering the kernel/firmware area was calculated as one too few, meaning the last sector was excluded from the area used for firmware hash verification. This could cause the device to compute or verify a firmware hash that does not cover the entire installed firmware image, potentially weakening integrity checks.

AI review queuedchore(core): Crowdin partial syncby obrusvit · 8da475da · Oct 15, 2025 · 10 filesMessage 72 · AdequateInformational 15Details
Commit message · obrusvit

chore(core): Crowdin partial sync

- including gen and fixtures

[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 pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit is a routine translation and localization update for the Trezor firmware. It updates translated user-facing strings in several languages, fixes a typo ('minumum' to 'minimum'), adjusts some strings to be device-layout-specific, and refreshes the corresponding UI test fixture hashes. There is no code behavior change and no security impact.

Security candidatechore(ethereum): upgrading from Holesky to Hoodiby Jiří Čermák · 58c0e94b · Oct 15, 2025 · 14 filesMessage 80 · StrongInformational 20Details
Commit message · Jiří Čermák

chore(ethereum): upgrading from Holesky to Hoodi

- Ethereum testnet upgrade. Updated literal references, and fixtures with the new name and chain ID.
- SLIP44 base indices updated from 1 to 60.
- Small change in eth/keychain required to work with older 1 index based
testnet addresses.

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 pathboot or update path
AI analysis · Informational 20/100

This commit replaces the deprecated Ethereum Holesky testnet with the new Hoodi testnet across Trezor firmware. It updates network names, chain IDs, SLIP44 derivation indices, staking contract address labels, and test fixtures. There is no indication of a security vulnerability being fixed; it is a routine network-maintenance change.

Lower-prioritychore(core): fix update_option of Crowdinby obrusvit · 95ed71c6 · Oct 15, 2025 · 1 fileMessage 80 · StrongInformational 15Details
Commit message · obrusvit

chore(core): fix update_option of Crowdin

- this makes sure that updating a string in sources (en_Bolt.json, etc.)
does not delete translations of the string

[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
translation-only discount
AI analysis · Informational 15/100

This commit changes a Crowdin translation-service configuration file so that when an English source string is updated, existing translations in other languages are kept but marked as needing re-approval. It is a workflow/translation-management fix with no effect on device firmware, cryptography, user funds, or security boundaries.

Lower-prioritychore(core): allow missing translations on Crowdinby obrusvit · df2a3553 · Oct 15, 2025 · 1 fileMessage 85 · StrongInformational 15Details
Commit message · obrusvit

chore(core): allow missing translations on Crowdin

- from now on, the Crowdin sources will not have the empty values, if
the keys are not used for a particular layout. This change supports this
workflow by modifying the `merge`, `split` functions to handle missing
keys.

[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
translation-only discount
AI analysis · Informational 15/100

This commit changes an internal Python helper script used to prepare translation files for the Crowdin localization service. It makes the tool tolerate translation files where some languages or screen layouts do not have every translation key. There is no change to the Trezor firmware that runs on the device, no user-facing behavior change, and no security relevance.

Lower-priorityfix(python): connect once during `trezorctl get-session`by Roman Zeyde · 1b2cd155 · Oct 15, 2025 · 2 filesMessage 62 · AdequateInformational 16Details
Commit message · Roman Zeyde

fix(python): connect once during `trezorctl get-session`

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 16/100

This is a small bug fix in the Trezor command-line tool `trezorctl`. The `get-session` command accidentally connected to the Trezor device twice because it used an existing connection object (`obj`) that already had an open session, then called `get_session()` again. The fix reuses the already-open `client` connection instead. This is a reliability/usability fix, not a security vulnerability. There is no indication it could be exploited to steal funds, keys, or bypass protections.

Lower-prioritychore(storage): add error string to static_assertby matejcik · 6b3098b8 · Oct 15, 2025 · 1 fileMessage 84 · StrongInformational 15Details
Commit message · matejcik

chore(storage): add error string to static_assert

otherwise clang 21 complains that not using the string is a C23
extension

84/100 · StrongMessage clarity
✓ 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 trivial build-compatibility change. It adds human-readable error messages to two compile-time size checks so that newer versions of the Clang compiler accept the code without warnings. The actual security logic is unchanged.