TZ
← All projectsTrezor

Trezor firmware

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

BitcoinHardware walletsNormal
Repository coverage

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

916security candidates311second-pass queue1812AI analyses
336commits · 30 days
588commits · 60 days
1363commits · 180 days
2648commits · 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
473Strong · 80–100
1321Adequate · 60–79
862Thin · 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.
Roman Zeyde572177367271
tychovrahe292104171261
PrisionMike945990273
Andrew Kozlik712449265
Jakub Janků22919182
Martin Pastyřík23816173
cepetr19682133059
Ioan Bizău23076162059
obrusvit21376141064
M1nd3r20671145071
Lukas Bielesch846771067
Martin Milata17446146063
Analysis record

Published AI watches

Last scanned 15 minutes ago

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
Informational 15 AI analysisMessage 47 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

docs: add xtask documentation

This commit only adds documentation. It updates the table of contents and adds a new markdown page plus a diagram explaining the project's 'xtask' build helper. No code, build scripts, or firmware logic was changed, so it cannot introduce …

4879dababy cepetr+296−03 files
No security note in commit
Informational 17 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

feat(xtask): introduce project-toml build-options

This commit is a build-system refactoring for Trezor firmware. It moves the mapping of command-line build options to Rust/cargo features out of hard-coded Rust logic and into per-project TOML files. It also adds a small change so debug bui…

Build-option mapping now lives in project.toml files, increasing the attack surface for supply-chain/build-configuration tamperingMakefile change automatically enables debug-link for PYOPT=0 debug buildsValidation added to reject storage_insecure_testing_mode in production builds
d44de007by cepetr+681−40817 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

ci(core): fix nightly ARM emulator filename

This commit fixes a filename-renaming step in the project's automated nightly build pipeline. It ensures ARM64 emulator files get the correct name expected by another testing tool. There is no change to the actual Trezor firmware code, dev…

ab95f08fby Martin Milata+3−01 file
No security note in commit
Informational 12 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

chore(vendor): avoid micropython submodule noise in git status

This is a routine housekeeping change to reduce clutter in Git's status output for a third-party component (the MicroPython submodule). There is no indication it changes any code that runs on the Trezor device or affects security.

06cab32bby Martin Milata+1−11 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 candidatefix(solana): support multisig transfer instructionby Jakub Janků · ad2a4443 · May 26, 2026 · 6 filesMessage 100 · StrongLow 37Details
Commit message · Jakub Janků

fix(solana): support multisig transfer instruction

The test vector is based on the trezorctl command in the GH issue.
The signing account was changed to 14CCvQzQzHCVgZM3j9soPnXuJXh1RmCfwLVUcdfbZVBS
(address derived from m/44'/501'/0'/0').

Closes: https://github.com/trezor/trezor-firmware/issues/6843

100/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
Why it was queued
signing boundaryfuzzing or regression evidencesigning or wallet path
AI analysis · Low 37/100

This commit updates the Trezor firmware's Solana support so that the standard 'Transfer' instruction can be used with multi-signature accounts. Previously, the firmware treated this instruction as non-multisig, which likely caused it to reject or mis-handle legitimate Solana transactions that required multiple signers. The change is a feature/fix to broaden supported transaction shapes, not a patch for an active vulnerability.

Lower-prioritychore(translations): update fixtures [no changelog]by Michal Kazda · 111b748d · May 26, 2026 · 1 fileMessage 77 · AdequateInformational 15Details
Commit message · Michal Kazda

chore(translations): update fixtures
[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 commit only updates expected test result hashes (called 'fixtures') in a single test data file. It does not change any firmware, application, or cryptographic code. The hash changes are consistent with updated screen snapshots or translated text strings used by the test suite. There is no indication of a security fix or vulnerability.

AI review queuedchore(translations): sync Crowdin translations [no changelog]by Michal Kazda · c565e45c · May 26, 2026 · 6 filesMessage 77 · AdequateInformational 15Details
Commit message · Michal Kazda

chore(translations): sync Crowdin translations
[no changelog]

77/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
signing or wallet pathtranslation-only discountsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit only updates translated text strings in Czech, German, Spanish, French, and Portuguese for the Trezor hardware wallet firmware. It changes wording for things like Ethereum transaction prompts, Bluetooth device labels, backup/recovery instructions, and passphrase prompts. There are no code logic changes, no new features, and no security fixes.

AI review queuedchore(translations): minor string update [no changelog]by Michal Kazda · b2d0adf4 · May 26, 2026 · 2 filesMessage 77 · AdequateInformational 15Details
Commit message · Michal Kazda

chore(translations): minor string update
[no changelog]

77/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
signing or wallet pathtranslation-only discountsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit changes one German translation string to add the word 'ist' (is) for consistency, and updates the corresponding translation signature metadata. There is no code or security change.

AI review queuedchore(core/eckhart): update About screenby obrusvit · 41ff96ea · May 26, 2026 · 10 filesMessage 57 · ThinInformational 15Details
Commit message · obrusvit

chore(core/eckhart): update About screen

[no changelog]

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

This commit is a routine user-interface update for the Trezor hardware wallet's 'About' screen. It adds a 'Made in Ostrava, Czechia' label and, on devices that use serial numbers, displays the serial number. There is no security-relevant change.

Security candidatechore(core/eckhart): regulatory updateby obrusvit · 2ad1502c · May 26, 2026 · 11 filesMessage 72 · AdequateInformational 15Details
Commit message · obrusvit

chore(core/eckhart): regulatory update

- Brasil
- Ghana
- Nigeria

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

This commit adds new regulatory certification screens for Brazil, Ghana, and Nigeria to the Trezor hardware wallet's user interface. It is purely a compliance/labeling change: new image files are included and a list of regulatory regions is updated. There is no indication of any security bug, vulnerability, or behavior change affecting funds or device security.

AI review queuedchore(ethereum): preload 6kb of databy Ioan Bizău · a36d79b7 · May 26, 2026 · 2 filesMessage 57 · ThinInformational 18Details
Commit message · Ioan Bizău

chore(ethereum): preload 6kb of data

[no changelog]

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

This is a small internal tuning change for Trezor's Ethereum app. It increases the amount of transaction data the device preloads from 4 KB to 6 KB before asking the user to confirm. The only visible effect is fewer back-and-forth data requests for Ethereum transactions with large data payloads. There is no direct security fix or vulnerability being patched.

AI review queuedchore(tests,ethereum): tests from Suiteby Ioan Bizău · fc87316d · May 26, 2026 · 1 fileMessage 67 · AdequateInformational 15Details
Commit message · Ioan Bizău

chore(tests,ethereum): tests from Suite

[no changelog]

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

This commit only adds new test cases for Ethereum transaction signing. It does not change any production code, so it cannot introduce a security vulnerability by itself. The test data includes sample transactions and their expected signatures, used to verify the device signs complex DeFi transactions correctly.

Lower-prioritychore: update fixturesby Ioan Bizău · 6756ee77 · May 26, 2026 · 1 fileMessage 40 · ThinInformational 15Details
Commit message · Ioan Bizău

chore: update fixtures

[no changelog]

40/100 · ThinMessage clarity
✓ Subject identifies a change✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 15/100

This commit only updates the expected test result hashes (called 'fixtures') for the user-interface test suite. It adds new test cases for LiFi token-swap/bridge transactions and refreshes a few existing pagination test hashes. There is no change to the actual Trezor firmware code that runs on the device, so this cannot introduce a security vulnerability by itself.

Lower-priorityfix(ethereum): claim.xyz address byte comparision.by PrisionMike · 79cac314 · May 26, 2026 · 1 fileMessage 62 · AdequateModerate 61Details
Commit message · PrisionMike

fix(ethereum): claim.xyz address byte comparision.

instead of strings.
[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 · Moderate 61/100

This patch fixes a bug in how Trezor's Ethereum app checked whether a transaction was meant for the claim.xyz rewards distributor. Previously it compared a lowercase hex string (like '0xabc...') against raw address bytes, which would never match. The fix compares raw bytes to raw bytes. The practical effect is that the special claim.xyz user interface and safety checks now actually trigger for the intended contract, instead of silently being skipped for all transactions.

Security candidatechore(trezorlib): fix `BootloaderV2Image` handling in `firmware_headers.py`by M1nd3r · ac308d2e · May 25, 2026 · 1 fileMessage 62 · AdequateLow 32Details
Commit message · M1nd3r

chore(trezorlib): fix `BootloaderV2Image` handling in `firmware_headers.py`

[no changelog]

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

This commit fixes how Trezor's Python library validates signatures for a new bootloader image format (BootloaderV2). Before the fix, the code ignored the signature bitmask and simply checked every signature against every public key in order. This could let an invalid or mismatched signature slip through, or cause validation to fail incorrectly. The patch makes the library read the bitmask, count signatures correctly, and only verify the signatures the image actually claims to have signed with. It also adds a missing signature-status line to the printed header. The change is in developer/CLI tooling, not the device firmware itself, so a direct attack on a real device is unlikely, but the library is used to inspect and validate firmware files.

Security candidatechore(python): minor adjustments to trezorlib/firmwareby M1nd3r · 6124c57b · May 25, 2026 · 2 filesMessage 62 · AdequateInformational 15Details
Commit message · M1nd3r

chore(python): minor adjustments to trezorlib/firmware

[no changelog]

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

This is a routine Python code change in Trezor's firmware handling library. It adds support for a new Nordic (NRF) firmware header type and helper methods to insert cryptographic signatures into firmware image structures, plus a minor formatting tweak. There is no indication of a security vulnerability or fix.

Lower-prioritychore(common): reserve capability fieldby M1nd3r · 1abc05d3 · May 25, 2026 · 2 filesMessage 57 · ThinInformational 15Details
Commit message · M1nd3r

chore(common): reserve capability field

[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 reserves an unused number (27) in a list of device capabilities so it cannot accidentally be reused later. It does not change any running code, fix a bug, or introduce a security issue.

Security candidatechore(core/eckhart): drop warning header if title is emptyby Roman Zeyde · af8bfeb3 · May 25, 2026 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde

chore(core/eckhart): drop warning header if title is empty

[no changelog]

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

This is a minor user-interface cleanup for the Trezor hardware wallet's Eckhart layout. It removes the warning header from a screen when the title text is empty, so users don't see a blank header. There is no security-relevant change.

Security candidateRevert "chore(core): don't send MCU attestation"by Roman Zeyde · b6c93a4e · May 25, 2026 · 2 filesMessage 60 · AdequateInformational 19Details
Commit message · Roman Zeyde

Revert "chore(core): don't send MCU attestation"

This reverts commit 8bc6d1b28b130a881119dd7e48ae99166ff9870b.

60/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
Why it was queued
update trustauthentication path
AI analysis · Informational 19/100

This commit re-enables a security feature called 'MCU device attestation' for the T3W1 hardware wallet model. It is not a vulnerability fix; it restores a previously removed capability that lets the device cryptographically prove its microcontroller is genuine using a modern quantum-resistant signature scheme (ML-DSA-44).

Security candidatefeat(core): allow streaming authenticity proofsby Roman Zeyde · 97848653 · May 25, 2026 · 2 filesMessage 80 · StrongInformational 21Details
Commit message · Roman Zeyde

feat(core): allow streaming authenticity proofs

Since MCU attestation data is quite large, it may fail to be sent
successfully in case of transport packet loss (e.g. over BLE).

- A new boolean `stream` field is added to `AuthenticateDevice` message.
- If it's set, the FW prepares the authenticity proofs and responds with a new `AuthenticityProofSizes` (containing the sizes of the proof-related blobs).
- The host can request any blob chunk by using the new `GetAuthenticityProofChunk` message, allowing to limit the amount of data sent from the device.
- The `AuthenticityProof` message is reconstructed by the host from the received chunks.
- We keep the previous implementation for backwards compatibility.

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

This commit adds a new optional 'streaming' mode for retrieving device authenticity proofs on Trezor hardware wallets. Instead of sending all attestation data (certificates and signatures) in one large message, the device can now send it in smaller chunks. This is intended to improve reliability over lossy transports like Bluetooth Low Energy. The old one-shot mode is kept for backward compatibility. There is no direct evidence in the commit that this fixes a security vulnerability; it reads as a reliability/feature improvement.

Security candidatechore(core): send MCU attestation only when streaming is supportedby Roman Zeyde · cefdddc6 · May 25, 2026 · 2 filesMessage 62 · AdequateLow 30Details
Commit message · Roman Zeyde

chore(core): send MCU attestation only when streaming is supported

[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 trustauthentication path
AI analysis · Low 30/100

This commit changes the Trezor hardware wallet so that a large secondary device-attestation proof (the MCU attestation) is only sent when the caller explicitly requests streaming support. The goal is to avoid sending one oversized response. It is a defensive hardening change rather than a fix for an active vulnerability, and the commit message does not claim it resolves a security issue.

Security candidatefeat(common): allow streaming authenticity proofsby Roman Zeyde · 66df4dda · May 25, 2026 · 12 filesMessage 57 · ThinInformational 21Details
Commit message · Roman Zeyde

feat(common): allow streaming authenticity proofs

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

This commit only changes the protocol definitions (the 'language' the Trezor device and apps use to talk to each other). It adds a new optional streaming mode for requesting device-authenticity proofs, so large proofs can be sent in smaller chunks instead of one big message. There is no actual device firmware logic shown here—only generated message bindings—so it is not a security fix or vulnerability by itself. It is a feature addition that could, in theory, introduce new attack surface once the corresponding firmware code is implemented, but that code is not present in this commit.

Security candidatetest(core): test both `AuthenticateDevice` flowsby Roman Zeyde · 3d9da61d · May 25, 2026 · 2 filesMessage 82 · StrongInformational 15Details
Commit message · Roman Zeyde

test(core): test both `AuthenticateDevice` flows

Also, de-duplicate `challenge` parametrization using a fixture.

[no changelog]

82/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Mentions testing or verification
Why it was queued
authentication path
AI analysis · Informational 15/100

This commit only adds and reorganizes automated tests for the device-authentication feature on Trezor hardware wallets. It does not change the firmware code that runs on the device, so it cannot introduce a security vulnerability or fix one in the shipped product. The changes make the test suite exercise both the Optiga and Tropic authentication flows, test chunking sizes, and verify that invalid requests are rejected with proper error messages.

Lower-prioritytest(core): add `wait` keyword argument to `InputFunc`by Roman Zeyde · 46dca1cb · May 24, 2026 · 1 fileMessage 72 · AdequateInformational 15Details
Commit message · Roman Zeyde

test(core): add `wait` keyword argument to `InputFunc`

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

This is a tiny test-only change that adds a new optional keyword argument named 'wait' to a type definition used in Trezor's debug/testing tooling. It does not change any runtime behavior, device firmware, or security logic. There is no security issue here.

Lower-priorityrefactor(core): use `async` function instead of a generatorby Roman Zeyde · a678d4a8 · May 24, 2026 · 1 fileMessage 93 · StrongInformational 15Details
Commit message · Roman Zeyde

refactor(core): use `async` function instead of a generator

Following https://satoshilabs.slack.com/archives/C054XGU77QV/p1777387713801909.

`yield from` and `await` both allocate, so let's use `await` for better readability.

Checked `core/build/firmware/frozen_mpy.c`, the same code is generated:
```
// child of trezor_wire_thp_channel_Channel_write_encrypted_payload
// frozen bytecode for file trezor/wire/thp/channel.py, scope trezor_wire_thp_channel_Channel_write_encrypted_payload__wait_for_ack
static const byte fun_data_trezor_wire_thp_channel_Channel_write_encrypted_payload__wait_for_ack[39] = {
0xa1,0x40,0x08, // prelude
0x81,0x1b,0x81,0x2d, // names: _wait_for_ack, *
// code info
0x42,0x4f, // JUMP 15
0x25,0x00, // LOAD_DEREF 0
0x14,0x74, // LOAD_METHOD 'recv_payload'
0x10,0x81,0x1c, // LOAD_CONST_STRING 'expected_ctrl_byte'
0x51, // LOAD_CONST_NONE
0x36,0x82,0x00, // CALL_METHOD 256
0x5e, // GET_ITER
0x51, // LOAD_CONST_NONE
0x68, // YIELD_FROM
0x59, // POP_TOP
0x12,0x81,0x1d, // LOAD_GLOBAL 'ABP'
0x14,0x7b, // LOAD_METHOD 'is_sending_allowed'
0x25,0x00, // LOAD_DEREF 0
0x13,0x45, // LOAD_ATTR 'channel_cache'
0x36,0x01, // CALL_METHOD 1
0x44,0x24, // POP_JUMP_IF_FALSE -28
0x51, // LOAD_CONST_NONE
0x63, // RETURN_VALUE
};
```

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

This commit is a pure code cleanup: it rewrites two small internal helper functions from Python generator style (`yield from`) to modern async/await style (`await`). The author verified that the compiled firmware bytecode is identical, meaning the device behaves exactly the same as before. There is no security-relevant change.

Lower-priorityrefactor(core): exclude THP-related confirmation from `THP=0` buildsby Roman Zeyde · db312019 · May 24, 2026 · 1 fileMessage 62 · AdequateInformational 18Details
Commit message · Roman Zeyde

refactor(core): exclude THP-related confirmation from `THP=0` builds

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

This commit is a build-system cleanup: it wraps a Trezor-Host Protocol (THP) credential confirmation screen so it is only compiled into firmware builds that actually have THP enabled. For non-THP builds, only the simpler non-THP confirmation is kept. There is no direct evidence in the commit that this fixes an active security bug; it appears to be a refactor to avoid including unused THP code in builds where the feature is disabled.

AI review queuedchore(python): re-add `pytest-retry` dependency for `pytest.mark.flaky`by Roman Zeyde · c081328b · May 23, 2026 · 2 filesMessage 77 · AdequateInformational 15Details
Commit message · Roman Zeyde

chore(python): re-add `pytest-retry` dependency for `pytest.mark.flaky`

It was removed in 108c87a776 by mistake.

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

This commit simply restores a missing Python test dependency called pytest-retry. It only affects the project's testing environment and does not change any code that runs on the Trezor device or any production software. There is no security issue here.

Lower-prioritytest(core): ignore `DeprecationWarning` from `dominate` during UI testsby Roman Zeyde · 6fd2ba37 · May 23, 2026 · 1 fileMessage 72 · AdequateInformational 15Details
Commit message · Roman Zeyde

test(core): ignore `DeprecationWarning` from `dominate` during UI tests

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

This change only adjusts a test configuration file to silence a harmless deprecation warning coming from a third-party UI library called dominate. It does not modify any firmware, wallet, or security-related code, and there is no indication it fixes or hides a real security problem.

Lower-prioritychore(ci): differentiate `UI_FAILED` and `UI_MISSING`by M1nd3r · a7bd0a91 · May 23, 2026 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · M1nd3r

chore(ci): differentiate `UI_FAILED` and `UI_MISSING`

[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 changes the wording of a test-result message in the project's internal testing tools. It splits one message into two so that missing expected test snapshots are labeled 'UI_MISSING' rather than 'UI_FAILED'. There is no change to the firmware, wallet behavior, cryptography, or any user-facing security feature.