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 41 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 candidatechore(core): remove assets dirby obrusvit · fc95bb93 · Oct 14, 2025 · 84 filesMessage 72 · AdequateInformational 15Details
Commit message · obrusvit

chore(core): remove assets dir

- move png files which were used to generate their corresponding toifs
to the appropriate folders

[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
cryptography-sensitive pathsigning or wallet path
AI analysis · Informational 15/100

This commit is a simple housekeeping change: it deletes a top-level 'assets' folder and moves the same PNG image files into the resource folders used by each user-interface layout. No code was changed, no functionality was altered, and there is no security relevance.

AI review queuedchore(core): sign translationsby obrusvit · 09907258 · Oct 14, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · obrusvit

chore(core): sign translations

- note: translations are signed from branch
`release/translations/25.10.21`

[no changelog]

62/100 · AdequateMessage clarity
✓ Descriptive subject✓ 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 simply adds a new cryptographic signature to a list that verifies official language translation files for the Trezor hardware wallet. It does not change any code, fix any bug, or introduce any security weakness. It is a routine administrative update so devices can trust the latest translation pack.

Lower-prioritychore: start Tropic model from multicore testsby Ioan Bizău · 063ddf37 · Oct 14, 2025 · 8 filesMessage 67 · AdequateInformational 15Details
Commit message · Ioan Bizău

chore: start Tropic model from multicore tests

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

This commit is a testing infrastructure change. It makes the software emulator automatically start a separate 'Tropic' hardware-security model on a configurable TCP port during automated tests, so multiple test workers can run in parallel without colliding on the same port. There is no indication this fixes or introduces a security vulnerability in the device firmware itself.

Lower-prioritychore: add script to provision Tropic modelby Ioan Bizău · c9af1421 · Oct 14, 2025 · 7 filesMessage 57 · ThinInformational 13Details
Commit message · Ioan Bizău

chore: add script to provision Tropic model

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

This commit adds a build/test helper script and sample configuration files for simulating a Tropic security chip during automated testing. It does not change any firmware code that runs on real Trezor devices, and the private keys it includes are explicitly test-only artifacts copied from a public vendor example. There is no indication this is a security fix or that it introduces a vulnerability.

Lower-prioritychore(tests): start Tropic model in CI testsby Ioan Bizău · 56d803f1 · Oct 14, 2025 · 1 fileMessage 67 · AdequateInformational 15Details
Commit message · Ioan Bizău

chore(tests): start Tropic model in CI tests

[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 only changes the project's automated testing configuration. It adds steps to start a 'Tropic' simulation model during certain CI test jobs for the T3W1 hardware model. There is no change to the firmware code that runs on user devices, no fix for a security flaw, and no indication this is a security-related patch.

Lower-prioritychore: update fixturesby Ioan Bizău · 77d3264b · Oct 14, 2025 · 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 expected test result hashes (called 'fixtures') in a single test data file. It does not change any firmware, application, or security code. The changes appear to be routine test snapshot updates, likely because tests were renamed or their expected screen outputs changed. There is no indication of a security fix or vulnerability.

AI review queuedchore(tests): generate certificate chain for Tropicby Ioan Bizău · 46b5510e · Oct 14, 2025 · 4 filesMessage 72 · AdequateInformational 15Details
Commit message · Ioan Bizău

chore(tests): generate certificate chain for Tropic

[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
second-pass: broader security terminology
AI analysis · Informational 15/100

This commit only adds four test-only certificate and private-key files under the tests/tropic_model directory. They are clearly labeled as test fixtures, not used in production firmware, and the commit message explicitly says '[no changelog]'. There is no code change and no indication of a security fix or vulnerability.

Security candidatefeat(tests): check Tropic signature and cert chainby Ioan Bizău · 65154d90 · Oct 14, 2025 · 1 fileMessage 77 · AdequateInformational 15Details
Commit message · Ioan Bizău

feat(tests): check Tropic signature and cert chain

[no changelog]

77/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification✓ Names security-relevant behavior explicitly! No meaningful explanatory body
Why it was queued
signing boundaryauthentication path
AI analysis · Informational 15/100

This commit only adds and updates automated tests for Trezor's device-authentication feature. It does not change the firmware code that runs on the device, so it cannot introduce a security vulnerability in the product itself. The tests verify that certificate chains and signatures from two hardware security chips (Optiga and Tropic) are accepted correctly.

Lower-priorityci(core): show slowest click testsby Roman Zeyde · 39da228d · Oct 14, 2025 · 1 fileMessage 67 · AdequateInformational 15Details
Commit message · Roman Zeyde

ci(core): show slowest click tests

[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 only adds a CI (continuous integration) option to display the 10 slowest automated user-interface tests during GitHub Actions runs. It does not change any firmware code, cryptography, device behavior, or test logic. There is no security relevance.

Security candidatechore(core): update T3W1 embedded binariesby tychovrahe · 0bd923c7 · Oct 13, 2025 · 4 filesMessage 72 · AdequateInformational 0Details
Commit message · tychovrahe

chore(core): update T3W1 embedded binaries

[no changelog]

(cherry picked from commit 846bdd3a16a89c349f446790ca4e7f40ce1923ac)

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

This commit updates three pre-built binary files for the T3W1 hardware model (bootloader, security monitor, and Bluetooth firmware) plus the matching bootloader hash header. The actual code inside the binary files is not shown in the diff, and the commit message only describes it as a routine binary update with no changelog. There is no visible security fix or vulnerability indicator in the materials provided.

Security candidatechore(core): update T3W1 embedded binariesby tychovrahe · 846bdd3a · Oct 13, 2025 · 4 filesMessage 57 · ThinInformational 11Details
Commit message · tychovrahe

chore(core): update T3W1 embedded binaries

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

This commit updates three pre-built binary files for the Trezor T3W1 hardware wallet model: the bootloader, a security monitor, and a Bluetooth Low Energy firmware image. It also updates the corresponding bootloader hash header so the firmware can verify the new bootloader. The commit message calls this a routine chore with no changelog entry. There is no description of any security fix, bug, or functional change in the commit itself, and the binary differences cannot be inspected as source code.

AI review queuedfix(core): fix crash when setting wipe codeby tychovrahe · 66b0a65e · Oct 12, 2025 · 1 fileMessage 72 · AdequateLow 29Details
Commit message · tychovrahe

fix(core): fix crash when setting wipe code

[no changelog]

(cherry picked from commit da76bd501dd1bfefba472a969f213253ac6fc7cf)

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
second-pass: broader security terminology
AI analysis · Low 29/100

This commit fixes a crash that could occur when setting or changing the Trezor device's wipe code. The fix moves a memory-protection reconfiguration call to happen before user-interface progress is initialized. The crash appears to be a reliability bug rather than a security vulnerability that an attacker could exploit to steal funds or bypass protections.

AI review queuedfix(core): fix crash when setting wipe codeby tychovrahe · da76bd50 · Oct 10, 2025 · 1 fileMessage 57 · ThinLow 40Details
Commit message · tychovrahe

fix(core): fix crash when setting wipe code

[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
second-pass: broader security terminology
AI analysis · Low 40/100

This commit fixes a crash that could occur when setting or changing the Trezor device's wipe code (a special PIN that wipes the device). The fix moves a memory-protection reconfiguration call earlier, before user-interface progress initialization. The crash appears to be a reliability bug rather than a security vulnerability that would let an attacker steal funds, but a crash during security-sensitive wipe-code setup could disrupt the user experience or potentially be used in a denial-of-service scenario.

Lower-priorityfix(core): lazy tropic secure session startby tychovrahe · 167c5c9e · Oct 10, 2025 · 4 filesMessage 72 · AdequateLow 41Details
Commit message · tychovrahe

fix(core): lazy tropic secure session start

[no changelog]

(cherry picked from commit f598e385c4f0a680c4446f4dd3c24e01094956b7)

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
AI analysis · Low 41/100

This commit restructures how the Trezor firmware initializes and starts secure sessions with the Tropic secure chip. Previously, session startup happened eagerly during device initialization. Now it is deferred until the first actual use of the Tropic chip, and a separate wait-for-ready step is added. The change is framed as a fix for a lazy session-start issue. It is not obviously exploitable by itself, but it touches security-critical pairing/session logic and removes a duplicate tropic_init() call in production-test code, which could matter if the duplicate call had side effects.

Lower-priorityfix(core): remove premature backlight fade-in after device wakeupby tychovrahe · 618c3b95 · Oct 10, 2025 · 1 fileMessage 77 · AdequateInformational 17Details
Commit message · tychovrahe

fix(core): remove premature backlight fade-in after device wakeup

[no changelog]

(cherry picked from commit b66802932c5d40dc9ad0bf1d060e7f61270af5b0)

77/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
AI analysis · Informational 17/100

This commit removes one line of code that automatically brightened the screen when a Trezor device woke from sleep. The change is described as fixing a 'premature backlight fade-in' after wakeup. There is no indication in the commit that this was a security issue; it appears to be a user-experience or display bug fix.

Lower-priorityfix(core): lazy tropic secure session startby tychovrahe · f598e385 · Oct 10, 2025 · 4 filesMessage 57 · ThinLow 41Details
Commit message · tychovrahe

fix(core): lazy tropic secure session start

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

This commit restructures how the Trezor device starts a secure session with its Tropic security chip. Previously, the device tried to initialize and immediately start a secure session during boot. Now it initializes the chip early but waits until the chip reports ready, and only starts the secure session when a function actually needs it. This is a defensive hardening change that reduces the chance of session-start commands failing or being issued while the chip is still booting. It also removes a duplicate initialization call in production-test code.

Lower-priorityfix(core): remove premature backlight fade-in after device wakeupby tychovrahe · b6680293 · Oct 10, 2025 · 1 fileMessage 62 · AdequateInformational 17Details
Commit message · tychovrahe

fix(core): remove premature backlight fade-in after device wakeup

[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 removes a single line that automatically brightened the screen when a Trezor device woke up from suspend. The change is described as fixing a 'premature' backlight fade-in. There is no direct evidence in the commit that this is a security fix; it appears to be a user-experience or visual polish change. The security relevance, if any, would be minor: a brief screen illumination could theoretically reveal that the device had woken up or what screen was displayed, but no sensitive data or controls are exposed.

Lower-priorityrefactor(core): unify `ButtonRequest` handlersby Roman Zeyde · 2ddca8ec · Oct 10, 2025 · 2 filesMessage 80 · StrongInformational 12Details
Commit message · Roman Zeyde

refactor(core): unify `ButtonRequest` handlers

Handle initial `ButtonRequest` by `Layout.button_request_task()`,
allowing us to remove `common._button_request()` function.

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

This is a small internal code cleanup in the Trezor firmware's user-interface layer. It moves responsibility for sending on-screen confirmation prompts (ButtonRequest messages) into a single place inside the Layout class, and removes an older helper function. There is no indication this change fixes a security bug or introduces a vulnerability; it is a refactoring to simplify the code.

Lower-priorityfix(core): raise "already pending" `FirmwareError` only on a new `ButtonRequest`by Roman Zeyde · 51e533f4 · Oct 10, 2025 · 1 fileMessage 85 · StrongLow 25Details
Commit message · Roman Zeyde

fix(core): raise "already pending" `FirmwareError` only on a new `ButtonRequest`

The layout may return `None`, so it should be OK to continue waiting
for the previous ButtonRequest to be processed.

[no changelog]

85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
AI analysis · Low 25/100

This is a small defensive fix inside the Trezor hardware wallet's user-interface code. It moves a debug-only sanity check so that it no longer fires when a screen layout simply returns nothing (None). Previously, the check could incorrectly raise a 'FirmwareError' before the code had a chance to notice there was no new button request. The change only affects debug builds and is unlikely to be directly exploitable by an attacker.

Lower-priorityrefactor(core): avoid calling `workflow.close_others()` twice on `interact`by Roman Zeyde · 59f5236c · Oct 10, 2025 · 1 fileMessage 62 · AdequateInformational 11Details
Commit message · Roman Zeyde

refactor(core): avoid calling `workflow.close_others()` twice on `interact`

[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 is a small internal cleanup in the Trezor firmware's user-interface code. It removes one of two calls to a function that shuts down other active workflows when a button request is sent. The change is labeled as a refactor with no changelog entry, and there is no direct evidence in the commit that it fixes a security bug.

Lower-priorityrefactor(core): remove unneeded try-except blockby Roman Zeyde · e65aa5a7 · Oct 10, 2025 · 1 fileMessage 57 · ThinInformational 12Details
Commit message · Roman Zeyde

refactor(core): remove unneeded try-except block

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

This commit removes a try-except block that simply caught all exceptions and immediately re-raised them. It is a code cleanup with no functional change to how errors are handled.

Lower-priorityfix(core): add missing `if __debug__` before logging exceptionsby Roman Zeyde · ae19db0f · Oct 9, 2025 · 2 filesMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde

fix(core): add missing `if __debug__` before logging exceptions

[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 wraps two debug-only exception log statements with an `if __debug__:` guard. In Python, `__debug__` is true unless the interpreter is run with the `-O` (optimize) flag. The change prevents exception details from being logged in optimized/production builds. The patch is defensive and reduces information leakage, but the actual security impact is minor because the logged data is not shown to untrusted parties by default and the exceptions themselves are already being caught and suppressed.

Lower-priorityci(core): always upload coverage reportby Roman Zeyde · dc3ee829 · Oct 9, 2025 · 1 fileMessage 67 · AdequateInformational 15Details
Commit message · Roman Zeyde

ci(core): always upload coverage report

[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 is a minor GitHub Actions workflow change. It ensures that a code-coverage report is uploaded even if earlier test steps failed. It does not change the Trezor firmware code, cryptography, or device behavior, and has no direct security relevance.

Lower-priorityci(core): reduce coverage threshold for T3B1by Roman Zeyde · 732cdc42 · Oct 9, 2025 · 1 fileMessage 67 · AdequateInformational 15Details
Commit message · Roman Zeyde

ci(core): reduce coverage threshold for T3B1

[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 only changes a CI (continuous integration) test coverage threshold for one hardware model (T3B1) from 85% to 84%, with a TODO comment linking to an internal issue. It does not modify any firmware code, cryptography, or device behavior. There is no security-relevant change to the product itself.

Lower-prioritytest(python): use `nist256p1` in payment device tests [no changelog]by M1nd3r · 14e2064c · Oct 9, 2025 · 1 fileMessage 87 · StrongInformational 15Details
Commit message · M1nd3r

test(python): use `nist256p1` in payment device tests
[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
AI analysis · Informational 15/100

This commit only changes a test file used during automated testing. It swaps the cryptographic curve and private key used to generate fake payment-request signatures in tests, from Bitcoin's secp256k1 curve to the NIST P-256 curve. There is no change to the actual Trezor firmware or to how real payment requests are validated. It is a testing-only adjustment with no security relevance to end users.