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

Lower-prioritychore(core): improve unit_properties module commentsby tychovrahe · 1da30b93 · Oct 1, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · tychovrahe

chore(core): improve unit_properties module comments

[no changelog]

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

This commit only rewrites code comments in a header file to use a more formal documentation style. No actual program logic, function behavior, or security checks were changed. It is not a security fix and does not introduce any security issue.

Lower-priorityfix(core/prodtest): switch tropic silicon revision to ACABby tychovrahe · e9649663 · Oct 1, 2025 · 2 filesMessage 62 · AdequateInformational 16Details
Commit message · tychovrahe

fix(core/prodtest): switch tropic silicon revision to ACAB

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 commit updates the production-test configuration for a Trezor hardware model (T3W1 revC) to use a newer revision of an external secure-element firmware file and a matching compile-time identifier. It is a build-configuration change, not a patch to cryptographic or wallet code. There is no direct evidence in the commit that this fixes a security vulnerability; it appears to be a routine hardware bring-up or compatibility update.

Security candidatefeat(core/eckhart): LED effect in HoldToConfirmby obrusvit · c123574c · Oct 1, 2025 · 4 filesMessage 57 · ThinInformational 17Details
Commit message · obrusvit

feat(core/eckhart): LED effect in HoldToConfirm

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

This commit adds a visual LED effect and a short 'finalized' animation to the hold-to-confirm action on Trezor's newer Eckhart-style user interface. It changes when the device reports a confirmation: instead of confirming the instant the user releases the button, the device now waits for a brief 500 ms animation to finish. During that animation it ignores further button input and lights the RGB LED in a color matching the on-screen confirmation. There is no indication this fixes a security bug; it appears to be a user-experience and visual-feedback feature.

AI review queuedfeat(core): notify host about channel replacement [no changelog]by M1nd3r · 8892eee0 · Oct 1, 2025 · 2 filesMessage 77 · AdequateLow 27Details
Commit message · M1nd3r

feat(core): notify host about channel replacement
[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
parser or protocol pathsecond-pass: security-sensitive path
AI analysis · Low 27/100

This commit adds a new device state flag so the Trezor hardware wallet can tell a connected computer: 'this session replaced an existing channel, so the user does not need to confirm again.' The host-side library is updated to recognize the new state. It is a feature change, not an obvious vulnerability fix, but it touches the authentication/authorization boundary between device and host.

AI review queuedchore(core): shorten french stringby obrusvit · bde7f27e · Oct 1, 2025 · 3 filesMessage 57 · ThinInformational 15Details
Commit message · obrusvit

chore(core): shorten french string

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit simply shortens a French translation string used on the device screen. It changes 'Effacer les données de l'appareil' to 'Effacer l'appareil' and updates the corresponding translation signature file and UI test snapshots. There is no code behavior change and no security relevance.

Lower-priorityfix(core): wait for `_handle_button_requests` only on THP devicesby Roman Zeyde · 431148f4 · Sep 30, 2025 · 1 fileMessage 77 · AdequateLow 34Details
Commit message · Roman Zeyde

fix(core): wait for `_handle_button_requests` only on THP devices

Restore previous behaviour on non-THP devices.

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

This commit fixes a regression in the Trezor hardware wallet's user-interface code. A previous change had started waiting for a synchronization signal (called a 'mailbox') during every button request, but that wait was only needed for newer THP-style communication. On older non-THP devices, the extra wait could cause the device to hang or behave incorrectly when confirming actions on screen. The patch restores the old behavior for non-THP devices while keeping the new behavior for THP devices.

AI review queuedfix(prodtest): Hotfix for invalid Infineon certificate format.by Andrew Kozlik · fda38e88 · Sep 30, 2025 · 1 fileMessage 62 · AdequateInformational 20Details
Commit message · Andrew Kozlik

fix(prodtest): Hotfix for invalid Infineon certificate format.

[no changelog]

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
second-pass: broader security terminology
AI analysis · Informational 20/100

This commit is a small hotfix in Trezor's factory production-test tool (prodtest), which is used during manufacturing, not in normal end-user wallets. It changes how a certificate read from the Infineon OPTIGA secure chip is parsed. Previously, the code carefully checked a multi-certificate chain format; now it simply skips 9 bytes and assumes there is only one certificate. The change removes safety checks and could make the tool accept malformed or unexpected certificate data, but it only affects an internal manufacturing diagnostic command, not the wallet firmware users rely on for funds.

Lower-prioritychore(core): fix type hints for logging functions [no changelog]by M1nd3r · aaaf43c1 · Sep 30, 2025 · 1 fileMessage 77 · AdequateInformational 15Details
Commit message · M1nd3r

chore(core): fix type hints for logging functions
[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
AI analysis · Informational 15/100

This is a tiny type-hint cleanup in a logging helper. It changes the signature of an internal no-op function so that static type checkers better match how it is actually used. There is no runtime behavior change, no security fix, and no vulnerability.

Security candidatefix(core): cache host and app name also at credential-based connectionby Roman Zeyde · 4e0467bc · Sep 30, 2025 · 4 filesMessage 77 · AdequateLow 35Details
Commit message · Roman Zeyde

fix(core): cache host and app name also at credential-based connection

If the first pairing is done over USB, there won't be a MAC address so
the cache won't be populated.

[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
credential or privilege state
AI analysis · Low 35/100

This commit fixes a bug in Trezor's T3W1 hardware wallet where, if a user first paired their device over USB instead of Bluetooth, the device failed to remember the computer/app name for later use. The fix makes sure the host and app name are saved during credential-based pairing even when there is no Bluetooth MAC address. There is no direct evidence this is a security vulnerability, but missing pairing metadata could weaken future security checks or user trust.

Lower-prioritychore(tests): update fixturesby Ioan Bizău · ed9fa7cf · Sep 30, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · Ioan Bizău

chore(tests): update fixtures

[no changelog]

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

This commit only updates expected test result fingerprints (hashes) in a single test fixtures file. It does not change any firmware, application, or cryptographic code. There is no direct security relevance visible in the commit itself.

Security candidatefix(core): exit pairing screen when an already bonded device connectsby tychovrahe · c156ea10 · Sep 30, 2025 · 8 filesMessage 62 · AdequateLow 35Details
Commit message · tychovrahe

fix(core): exit pairing screen when an already bonded device connects

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
boot or update path
AI analysis · Low 35/100

This update fixes a Bluetooth pairing screen bug in Trezor hardware wallets. Previously, if an already-paired (bonded) host device connected while the wallet was showing a pairing screen, the wallet did not automatically exit that screen. The fix adds a new internal event, 'pairing not needed,' so the user interface can close the pairing screen in that situation. The change is a user-experience and minor reliability fix; the commit itself does not describe a security vulnerability.

Security candidatefix(core): fix integer overflow in storageby tychovrahe · 56755da9 · Sep 30, 2025 · 1 fileMessage 62 · AdequateModerate 58Details
Commit message · tychovrahe

fix(core): fix integer overflow in storage

[no changelog]

62/100 · AdequateMessage clarity
✓ 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
memory safety
AI analysis · Moderate 58/100

This commit fixes a potential integer overflow in the Trezor hardware wallet's storage code. A variable that holds a flash memory offset was declared as a 16-bit unsigned integer (uint16_t), which could overflow if the sector offset plus the amount of data already written exceeds 65,535. The fix widens that variable to a 32-bit unsigned integer (uint32_t), giving it enough room for larger flash addresses. If exploited, this kind of overflow could corrupt where data is written in the device's storage, potentially leading to data loss or unexpected behavior.

Lower-prioritychore(core): add clean_prodtest_emu makefile target, add it to cleanby tychovrahe · 353afc77 · Sep 29, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · tychovrahe

chore(core): add clean_prodtest_emu makefile target, add it to clean

[no changelog]

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

This commit only adds a missing cleanup step in the project's build Makefile. It ensures that when developers run 'make clean', a newly introduced emulator build directory for the production-test tool is also deleted. There is no change to device firmware, no security fix, and no vulnerability.

Lower-prioritychore(core): log reassembled THP messagesby Roman Zeyde · 269560fb · Sep 29, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · Roman Zeyde

chore(core): log reassembled THP messages

[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 adds a debug-only log message that records when a fragmented THP (Trezor Host Protocol) message has been successfully put back together. It does not change how messages are processed, validated, or secured. The logging is wrapped in `__debug__`, meaning it is normally stripped out of production firmware builds. There is no security-relevant change visible in the diff.

Lower-prioritychore(core): log THP channel errorsby Roman Zeyde · 53355f3c · Sep 29, 2025 · 1 fileMessage 57 · ThinInformational 14Details
Commit message · Roman Zeyde

chore(core): log THP channel errors

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

This commit only adds a debug-only error log message when a Trezor hardware wallet's THP (Trezor Host Protocol) channel reports an error. It does not change how errors are handled, what data is sent, or any security behavior. The logging is wrapped in `if __debug__:` so it only runs in debug builds and is stripped from production firmware.

Lower-priorityfix(core): clear THP channel after `DEVICE_LOCKED` errorby Roman Zeyde · e1e05c1e · Sep 29, 2025 · 1 fileMessage 93 · StrongLow 37Details
Commit message · Roman Zeyde

fix(core): clear THP channel after `DEVICE_LOCKED` error

It should fix the issue described here:
https://github.com/trezor/trezor-firmware/issues/5885#issuecomment-3345712654
(causing alternating `ThpTransportBusy` and `ThpDeviceLocked` errors)

Currently the device allocates THP buffers to the first channel
receiving a message after an event loop restart.

This is a simple workaround for the above issue, since `ThpDeviceLocked`
happens only during THP handshake, and if it happens, the host will
create and use a new channel.

[no changelog]

93/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
AI analysis · Low 37/100

This is a small bug fix in the Trezor hardware wallet firmware. It clears a temporary communication channel when the device reports it is locked during a specific handshake process. Without the fix, the device could get stuck alternating between two error states, potentially disrupting normal USB communication and requiring a reconnect or restart. There is no direct evidence this can be used to steal funds, but any disruption to a security device's communication channel is treated cautiously.

Lower-priorityrefactor(core): remove special handling THP ACKs from `Reassembler`by Roman Zeyde · 3cb4083b · Sep 29, 2025 · 1 fileMessage 97 · StrongLow 26Details
Commit message · Roman Zeyde

refactor(core): remove special handling THP ACKs from `Reassembler`

Once a message is retrieved over a cached channel, the THP buffers
will be owned by this channel until an event loop restart.

Thus, the `if control_byte.is_ack(ctrl_byte)` branch is not helping
to avoid buffer allocation - and should be removed to simplify the flow.

[no changelog]

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

This commit removes a small optimization in the Trezor firmware's THP (Trezor Host Protocol) message reassembly code. Previously, small ACK (acknowledgment) packets used a slice of the existing packet buffer instead of allocating a separate buffer. Now all packets, including ACKs, use the same buffer allocation path. The change is described by the developer as a simplification that does not affect security because the buffers are already owned by the channel. There is no direct evidence of a security vulnerability being fixed.

Lower-prioritychore(core): remove unused functionsby Roman Zeyde · 6592ade0 · Sep 29, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · Roman Zeyde

chore(core): remove unused functions

[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 simply deletes two helper functions that were no longer being used anywhere in the code. It is a routine cleanup with no visible security effect.

Lower-prioritychore(core): remove unused `cid` argument to THP `Reassembler`by Roman Zeyde · 3b4f27fe · Sep 29, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde

chore(core): remove unused `cid` argument to THP `Reassembler`

[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 minor code cleanup: a developer removed an unused channel ID argument from an internal helper class called Reassembler. There is no user-visible change, no bug fix, and no security relevance in the commit itself.

AI review queuedfix: add progress when requesting extra databy Ioan Bizău · f7239a7f · Sep 29, 2025 · 6 filesMessage 57 · ThinInformational 15Details
Commit message · Ioan Bizău

fix: add progress when requesting extra data

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit only adds on-screen progress indicators to several cryptocurrency signing flows (EOS, Ethereum, Stellar). It does not change any security logic, cryptography, or transaction validation. The large diff is mostly updated test screenshot fingerprints (fixtures.json) because the new progress screens change the device's displayed images during automated UI tests.

Security candidate chore(core/eckhart): redesign connected itemby Lukas Bielesch · 2dd79de7 · Sep 29, 2025 · 10 filesMessage 57 · ThinInformational 15Details
Commit message · Lukas Bielesch

chore(core/eckhart): redesign connected item

- add app name to host info screen

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 user-interface redesign for the Trezor hardware wallet (specifically the T3W1/Eckhart layout). It moves the name of the connected app from the Bluetooth connection button onto a separate 'Host Info' screen, and adjusts how long text is clipped in buttons. There is no security-relevant change here.

Lower-prioritychore(core/eckhart): use marquee only for the button subtext renderingby Lukas Bielesch · a6610b41 · Sep 29, 2025 · 2 filesMessage 62 · AdequateInformational 15Details
Commit message · Lukas Bielesch

chore(core/eckhart): use marquee only for the button subtext rendering

[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 routine user-interface cleanup for the Trezor hardware wallet's new 'Eckhart' layout. It simplifies how long button labels scroll (marquee) so that only the smaller 'subtext' line uses the scrolling animation, rather than having two separate rendering paths. The change only affects on-screen text display and updates the corresponding screen-shot test hashes. There is no indication it fixes or introduces any security vulnerability.

AI review queuedchore(core): sign translationsby obrusvit · 2f9f2100 · Sep 29, 2025 · 1 fileMessage 47 · ThinInformational 15Details
Commit message · obrusvit

chore(core): sign translations

[no changelog]

47/100 · ThinMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathtranslation-only discountsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit adds a new cryptographic signature entry to a JSON file that records approved translation bundles for the Trezor hardware wallet. There is no code change and no visible security defect. It appears to be a routine administrative step to authorize a new set of translated strings.

Security candidatechore: update embedded binaries for T3W1by obrusvit · ff1c4a31 · Sep 26, 2025 · 5 filesMessage 57 · ThinInformational 3Details
Commit message · obrusvit

chore: update embedded binaries for T3W1

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
boot or update path
AI analysis · Informational 3/100

This commit swaps out several pre-built binary files for the T3W1 hardware model: a newer bootloader version, a security monitor binary, and a Bluetooth firmware binary. The change is described as a routine chore with no changelog entry. Because the files are binary blobs, the actual code changes inside them cannot be inspected from the diff alone, and there is no stated security relevance.

Lower-prioritychore: update releases.jsonby obrusvit · eec40945 · Sep 26, 2025 · 1 fileMessage 47 · ThinInformational 15Details
Commit message · obrusvit

chore: update releases.json

[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 simply adds a new firmware version number (2.9.2) for a new hardware model (T3W1) to a release tracking file. It does not change any code, cryptographic logic, or security behavior. There is no security issue here.