TZ
← All projectsTrezor

Trezor firmware

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

BitcoinHardware walletsNormal
Repository coverage

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

926security candidates314second-pass queue2668AI analyses
336commits · 30 days
553commits · 60 days
1345commits · 180 days
2618commits · 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
488Strong · 80–100
1330Adequate · 60–79
864Thin · 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 Zeyde581183578272
PrisionMike945994273
Andrew Kozlik742572266
obrusvit21376213164
Jakub Janků22919182
Martin Pastyřík24823173
cepetr19682196059
Ioan Bizău23076230059
M1nd3r21173208071
Lukas Bielesch846784067
Martin Milata17946175063
Analysis record

Published AI watches

Last scanned 8 minutes ago

Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

chore: remove libtropic's pin to develop branch

This commit removes a single line from the project's Git submodule configuration. It stops pinning the libtropic submodule to the 'develop' branch, meaning the project will now use the submodule's default branch (likely 'main' or 'master')…

8290eeefby M1nd3r+0−11 file
No security note in commit
Low 37 AI analysisMessage 70 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

chore(core/embed): clear temporary buffer across Optiga/Tropic calls

This is a hardening change for the random number generator inside Trezor hardware wallets. It makes sure that temporary memory holding secret random values from the Optiga and Tropic security chips is wiped immediately after use, rather th…

Sensitive intermediate buffer not cleared promptlyPotential stale-buffer reuse across RNG source boundariesDefense-in-depth memory sanitization for secure-element entropy
01ebda93by Roman Zeyde+2−21 file
No security note in commit
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 72 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

test(core): drop an unused parameter of `make_payment_request()`

This commit only cleans up test code by removing an unused 'session' argument from a helper function used in automated tests. It does not change the actual Trezor firmware or any code that runs on the device, so it has no security impact f…

6dd19fdcby Roman Zeyde+0−188 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
Informational 15 AI analysisMessage 85 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

fix(build-docker): stop swallowing pub-bin copy errors

This change fixes a build script that was ignoring copy errors. Previously, the script would silently ignore any failure when copying public binary files, which could hide real problems. Now it only skips the copy when the file genuinely d…

Build script silently ignored copy failures before this fixFix removes unconditional `|| true` error suppressionKnown-benign missing files are now explicitly skipped with a file existence check
d829eda8by Andrew Kozlik+4−11 file
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
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/eckhart): device menu translationsby Lukas Bielesch · 97396ff6 · Aug 25, 2025 · 8 filesMessage 57 · ThinInformational 15Details
Commit message · Lukas Bielesch

chore(core/eckhart): device menu translations

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

This commit is a routine UI translation update for the Trezor hardware wallet's device menu. It replaces hard-coded English text strings with references to the device's translation system so the menu can be displayed in multiple languages. There are no security fixes or functional behavior changes.

Security candidatechore(core/eckhart): enable menu separators by defaultby Lukas Bielesch · 73f3a6d6 · Aug 23, 2025 · 4 filesMessage 62 · AdequateInformational 15Details
Commit message · Lukas Bielesch

chore(core/eckhart): enable menu separators by default

[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 user-interface cleanup for the upcoming Trezor 'Eckhart' device layout. It makes menu divider lines always appear instead of requiring each screen to opt in. There is no security change, no bug fix, and no handling of secrets or sensitive data.

Security candidatechore(core/eckhart): extend header buttonsby Lukas Bielesch · d8cff7ac · Aug 23, 2025 · 7 filesMessage 72 · AdequateInformational 20Details
Commit message · Lukas Bielesch

chore(core/eckhart): extend header buttons

- use header builder functions where possible
- update 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
boot or update path
AI analysis · Informational 20/100

This is a routine user-interface cleanup for the upcoming Trezor Safe 5 (Eckhart layout). It makes header buttons slightly larger and easier to tap, and switches several screens to use a shared 'close button' helper. The only functional change visible in the code is a larger touch target and a small visual radius on header icon buttons; there is no evidence this affects security-sensitive behavior such as transaction confirmation, PIN entry, or seed handling. The huge list of changed test hashes is simply because every screen screenshot changed by a few pixels.

Security candidatechore(core/eckhart): update apply settings label flowby Lukas Bielesch · 9ac1dbde · Aug 23, 2025 · 12 filesMessage 77 · AdequateInformational 15Details
Commit message · Lukas Bielesch

chore(core/eckhart): update apply settings label flow

- change button label and gradient
- add success screen after changing device name
[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
cryptography-sensitive pathsigning or wallet pathboot or update path
AI analysis · Informational 15/100

This commit is a routine user-interface polish change for the Trezor hardware wallet. It updates the on-screen button label and color when changing the device's name, and adds a short 'Device name changed' confirmation screen afterward. There is no security fix or vulnerability here.

Security candidatechore(core/eckhart): make paginate buttons equally sizedby Lukas Bielesch · 62b26ca4 · Aug 23, 2025 · 3 filesMessage 62 · AdequateInformational 15Details
Commit message · Lukas Bielesch

chore(core/eckhart): make paginate buttons equally sized

[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 cosmetic user-interface tweak for the Trezor hardware wallet's new 'Eckhart' layout. It makes the up/down pagination buttons the same size and updates the expected screen snapshots used in automated tests. There is no indication it fixes or introduces any security vulnerability.

Security candidatechore(core/eckhart): invert QR screen colorsby Lukas Bielesch · 031247b0 · Aug 23, 2025 · 4 filesMessage 57 · ThinInformational 15Details
Commit message · Lukas Bielesch

chore(core/eckhart): invert QR screen colors

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

This commit is a routine user-interface styling change for the Trezor hardware wallet's QR-code display screen. It swaps the foreground and background colors (inverts them) and adjusts the layout code accordingly. There is no indication this change affects security, cryptography, private keys, or transaction signing. The large number of changed test hashes is simply because the on-screen appearance changed, which is expected for a visual update.

Security candidatechore(core): update T3W1 embedded binariesby obrusvit · bd31cc41 · Aug 22, 2025 · 2 filesMessage 80 · StrongInformational 3Details
Commit message · obrusvit

chore(core): update T3W1 embedded binaries

- build command: make build_bootloader TREZOR_MODEL=T3W1
BOOTLOADER_DEVEL=1 BOOTLOADER_QA=1
- then: make bootloader_hashes

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

This commit replaces a pre-built bootloader binary file for the T3W1 hardware model and updates the matching cryptographic hashes in a header file. The change itself is a routine binary refresh with no source-code diff, no changelog entry, and no stated security relevance. There is no visible evidence of a vulnerability or malicious change, but the opaque binary blob cannot be independently verified from the commit alone.

Lower-prioritychore(core): fix log formatting in `trezor/wire/thp/channel.py`by Roman Zeyde · 326d381f · Aug 22, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde

chore(core): fix log formatting in `trezor/wire/thp/channel.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
AI analysis · Informational 15/100

This is a one-character logging fix in a Trezor firmware debug message. It adds a colon and space to a log string so the formatting reads correctly. There is no security relevance: no behavior changes, no crash, no data exposure, and no vulnerability is introduced or fixed.

Lower-priorityfix(core): avoid infinite THP error loopby Roman Zeyde · 6335c00e · Aug 22, 2025 · 1 fileMessage 57 · ThinLow 42Details
Commit message · Roman Zeyde

fix(core): avoid infinite THP error loop

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

This small patch fixes a bug in the Trezor hardware wallet's message-handling code where certain errors could cause the device to loop forever instead of stopping. Before the fix, if a specific kind of error happened while processing a message, the code would just log it and continue running, potentially retrying the same failing work endlessly. The patch makes the handler return after logging, breaking the loop. This could prevent device lock-ups or unresponsiveness, but the change alone does not clearly enable remote theft of funds or private keys.

Security candidatefix(core): increase `MAX_SUBSCREENS` a bitby Roman Zeyde · d86a50d1 · Aug 22, 2025 · 1 fileMessage 84 · StrongInformational 24Details
Commit message · Roman Zeyde

fix(core): increase `MAX_SUBSCREENS` a bit

Otherwise, entering the menu when there is
a connected BLE device results in a RSOD.

[no changelog]

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
Why it was queued
boot or update path
AI analysis · Informational 24/100

This commit fixes a bug where opening the device menu on a Trezor hardware wallet would crash with a red screen of death (RSOD) if a Bluetooth device was connected. The fix simply increases the maximum number of subscreens the menu system can handle from 8 to 10, because the Bluetooth menu entries push the count over the previous limit. There is no evidence this is exploitable by an attacker.

Lower-priorityfix: re-lock Poetry env using the version from Nixby obrusvit · ff68e6bc · Aug 22, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · obrusvit

fix: re-lock Poetry env using the version from Nix

- Nix env has 2.1.3

[no changelog]

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

This commit is a routine maintenance update to the project's Python dependency lock file (poetry.lock). It was generated because the team upgraded their Poetry tool version from 1.8.5 to 2.1.3, matching the version available in their Nix development environment. The file changes are purely metadata format differences produced by the newer Poetry version, plus one local package version bump (tvl from 0.10.1 to 2.2). There is no indication of a security fix, vulnerability patch, or code behavior change.

Lower-priorityfix(python): correct CLI error message on uninitialized deviceby Roman Zeyde · b4f857bd · Aug 22, 2025 · 1 fileMessage 62 · AdequateInformational 19Details
Commit message · Roman Zeyde

fix(python): correct CLI error message on uninitialized device

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

This is a tiny user-interface fix in the Trezor Python command-line library. When a user runs a command on a brand-new, uninitialised Trezor, the tool now prints 'Device is not initialized.' instead of falling through to a generic 'Failed to find a Trezor device.' message. It does not change device firmware, crypto handling, or any security boundary; it only makes the error text clearer.

Lower-priorityfeat(core): firmware suspend by power buttonby obrusvit · dbda8d0d · Aug 21, 2025 · 3 filesMessage 72 · AdequateInformational 18Details
Commit message · obrusvit

feat(core): firmware suspend by power button

- power button event is handled before the event is passed to rust
layout

[no changelog]

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

This commit adds a new power-saving feature: pressing the physical power button on a Trezor device can now suspend the firmware. If the device has a PIN and is currently unlocked, the button press first locks the device (ending any active workflow) and then triggers a shutdown. Otherwise, the device is suspended and later woken up. The change is a feature addition, not a fix for a known security bug, and there is no evidence in the commit or supplied references that it addresses a vulnerability.

Lower-priorityfix(core): increase prodtest stack sizeby cepetr · a6f6dbb3 · Aug 21, 2025 · 1 fileMessage 57 · ThinLow 28Details
Commit message · cepetr

fix(core): increase prodtest stack size

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

This commit doubles the stack memory size for the Trezor hardware wallet's production-test firmware from 12 KB to 24 KB. A stack that is too small can cause crashes (the commit notes 'Overflow causes UsageFault'). There is no direct evidence in the commit that this is exploitable as a security vulnerability, and the change appears to be a stability fix for a special manufacturing/QA mode rather than the normal user-facing wallet firmware.

AI review queuedbuild(core): fix dependency versions in trezor-core-toolsby matejcik · b14e3432 · Aug 21, 2025 · 3 filesMessage 97 · StrongInformational 15Details
Commit message · matejcik

build(core): fix dependency versions in trezor-core-tools

otherwise it's not possible to `poetry lock` when one of trezor or
trezor-core-tools is touched, because their version specs clash

97/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
second-pass: broader security terminology
AI analysis · Informational 15/100

This commit is a routine build/dependency housekeeping change. It updates Python version constraints and the internal 'trezor' package version requirement so that the project's Poetry lock file can be regenerated without conflicts. There is no indication this fixes a security vulnerability or changes runtime behavior of the Trezor firmware or wallet software.

Security candidatefeat(core/bootloader): stronger haptic effect on power onby tychovrahe · f947c4c6 · Aug 21, 2025 · 3 filesMessage 62 · AdequateInformational 15Details
Commit message · tychovrahe

feat(core/bootloader): stronger haptic effect on power on

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

This commit changes the vibration pattern that the Trezor hardware wallet produces when it powers on. It adds a new dedicated 'power on' haptic effect and uses it instead of reusing the existing button-press vibration. There is no security relevance in this change.

Lower-priorityfeat(core): reimplement THP retransmissionsby Roman Zeyde · 3ca95252 · Aug 21, 2025 · 2 filesMessage 57 · ThinLow 34Details
Commit message · Roman Zeyde

feat(core): reimplement THP retransmissions

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

This commit finishes a previously stubbed-out feature in Trezor's core firmware: automatic retransmission of lost USB packets during a secure protocol called THP. Before this change, the code had a TODO comment where retransmissions should have been. After this change, the device will resend a packet up to 50 times with an increasing timeout if it does not receive an acknowledgment. This is a normal reliability improvement, but because it touches low-level packet handling and timeout logic, it could introduce subtle bugs such as the device hanging, retrying forever in edge cases, or misinterpreting a delayed packet as a fresh one.

Lower-prioritytest(core): wait for debuglink task before restarting THP event loopby Roman Zeyde · 5bb49f1e · Aug 21, 2025 · 1 fileMessage 72 · AdequateInformational 17Details
Commit message · Roman Zeyde

test(core): wait for debuglink task before restarting THP event loop

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

This is a test-only change for the Trezor hardware wallet firmware. It adds a small cleanup step in the debug/test code path that waits for a debug-link helper task to finish before restarting the event loop used by the new THP (Trezor Host Protocol) code. It does not change normal user-facing behavior and is not a security fix for a live vulnerability.

Lower-prioritytest(core): wait for device availability to avoid UI tests flakinessby Roman Zeyde · 12c045d4 · Aug 21, 2025 · 1 fileMessage 84 · StrongInformational 15Details
Commit message · Roman Zeyde

test(core): wait for device availability to avoid UI tests flakiness

[no changelog]

84/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Explains rationale or failure mode✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 15/100

This is a test-only change that replaces a fixed 0.1-second sleep with an explicit device-readiness check before starting screen recordings in UI tests. It does not touch firmware, wallet logic, cryptography, or any production code path, so it has no direct security relevance for end users.

Security candidatefeat(core): reimplement THP event loop restartsby Roman Zeyde · c3200208 · Aug 21, 2025 · 21 filesMessage 57 · ThinLow 38Details
Commit message · Roman Zeyde

feat(core): reimplement THP event loop restarts

[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 pathparser or protocol path
AI analysis · Low 38/100

This commit is a large internal refactor of the Trezor firmware's THP (Trezor Host Protocol) event loop and message handling. It removes an old global buffer-locking scheme and a separate retransmission task, replacing them with per-channel buffer objects and a simpler synchronous send-and-wait-for-ACK flow. The change also removes the old `loop.clear()` restart-on-error behavior in favor of explicitly joining workflows before restarting. The diff itself is a feature rewrite with no explicit security claims, but it touches sensitive code paths (encrypted channel setup, session scheduling, buffer management) and removes some safety checks, so it should be reviewed carefully for regressions.

Lower-priorityfix(core/prodtest): fix conditional usage of tropic testing keysby tychovrahe · 8e51245b · Aug 21, 2025 · 1 fileMessage 72 · AdequateLow 32Details
Commit message · tychovrahe

fix(core/prodtest): fix conditional usage of tropic testing keys

[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
AI analysis · Low 32/100

This is a one-line fix in Trezor's factory production-test code for the Tropic secure chip. It changes a preprocessor condition from an apparently wrong/placeholder macro name 'ABAB' to the intended 'TROPIC_TESTING_KEYS'. The effect is to ensure that testing-only cryptographic keys are compiled in only when the proper configuration flag is set, rather than under an unrelated or accidental flag. This reduces the risk that production devices could be built with test keys instead of real factory keys.

Lower-priorityfix(prodtest): fix secrets-certdev-write arg count checkby cepetr · 755f6ab2 · Aug 21, 2025 · 2 filesMessage 62 · AdequateLow 37Details
Commit message · cepetr

fix(prodtest): fix secrets-certdev-write arg count check

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

This commit fixes a command in Trezor's production-test firmware that checks how many arguments a user passed. The old check rejected commands with more than zero arguments, but accidentally allowed commands with zero arguments through. The fix requires exactly one argument. The affected command is part of a low-level factory/testing tool, not the normal wallet firmware users run every day.

Lower-priorityfix(prodtest): fix wpc-info command outputby cepetr · ac426c6b · Aug 21, 2025 · 3 filesMessage 57 · ThinInformational 15Details
Commit message · cepetr

fix(prodtest): fix wpc-info command output

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 fixes the formatting of a debug/production-test command output for a wireless charging chip. It changes how numbers are printed (decimal to hexadecimal) and moves the data values from a 'PROGRESS' line to the 'OK' line. There is no security issue here—it's a cosmetic/output-formatting fix in a factory testing tool.

Lower-priorityfix(prodtest): remove extra crlf in response in non-interactive modeby cepetr · 441283c1 · Aug 21, 2025 · 2 filesMessage 62 · AdequateInformational 15Details
Commit message · cepetr

fix(prodtest): remove extra crlf in response in non-interactive mode

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 fixes a minor formatting bug in the Trezor production-test command-line tool. Previously, every response line in non-interactive mode had an extra blank line (CRLF) appended. The change moves that blank line so it only appears in interactive mode, where it serves as a prompt separator. There is no security relevance in the commit or diff.

Lower-prioritychore(core): remove forgotten `log.info()` in `interface_context.py`by Roman Zeyde · e451adc9 · Aug 21, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde

chore(core): remove forgotten `log.info()` in `interface_context.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
AI analysis · Informational 15/100

This commit removes a single leftover debug logging line that printed the contents of a channel allocation packet. It is a cleanup change with no security relevance: the log line was informational, not an error, and removing it does not fix a vulnerability or change any behavior an attacker could exploit.