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 18 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 candidatefeat(cardano): stop hashing in message signingby Peter Jaško · 0e71c8da · Aug 25, 2025 · 12 filesMessage 57 · ThinLow 32Details
Commit message · Peter Jaško

feat(cardano): stop hashing in message signing

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 boundarysigning or wallet path
AI analysis · Low 32/100

This commit removes the option to sign a hash of a Cardano message instead of the message itself. The device now always signs the actual message payload directly, up to a size limit. This is a feature/API change rather than a fix for an active security flaw, but it does eliminate a mode where users might have signed data they could not fully inspect.

Security candidatefeat(cardano): Message signingby Peter Jaško · 4dcfaf18 · Aug 25, 2025 · 25 filesMessage 47 · ThinLow 28Details
Commit message · Peter Jaško

feat(cardano): Message signing

47/100 · ThinMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing boundaryseed or entropy pathsigning or wallet path
AI analysis · Low 28/100

This commit adds a new feature to Trezor hardware wallets: the ability to sign arbitrary Cardano messages. It introduces new protocol messages, device firmware code to handle signing, and client libraries to use the feature. The signing process requires user confirmation on the device screen and includes safety checks, such as limiting unhashed payload length and validating ASCII display options. There is no direct evidence in the commit of a security vulnerability, but any new signing feature carries inherent risks if not fully reviewed.

Lower-prioritychore(tests): update fixturesby Ioan Bizău · 2168449c · Aug 25, 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 the expected test result hashes (called 'fixtures') for the user-interface test suite. It does not change any firmware, application, or cryptographic code. The changes are test-only data, so they do not create or fix any security vulnerability on their own.

Lower-priorityfix: changelog formattingby Peter Jaško · 91f8c85a · Aug 25, 2025 · 2 filesMessage 47 · ThinInformational 15Details
Commit message · Peter Jaško

fix: changelog formatting

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

This commit only adds a missing period to two changelog entries describing a new Cardano feature. There are no code changes and no security relevance.

AI review queuedfix(cardano): Return pubkeyby Peter Jaško · 389e1eba · Aug 25, 2025 · 7 filesMessage 62 · AdequateInformational 20Details
Commit message · Peter Jaško

fix(cardano): Return pubkey

This ends up being useful for software wallets.

62/100 · AdequateMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 20/100

This commit adds the public key to the Cardano message-signing response. It is a feature enhancement for software wallets, not a fix for a vulnerability. There is no security defect in the change itself.

AI review queuedchore(cardano): add missing translationsby David Misiak · 7fcd3ec8 · Aug 25, 2025 · 8 filesMessage 57 · ThinInformational 15Details
Commit message · David Misiak

chore(cardano): add missing translations

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 or updates translated text strings for the Cardano cryptocurrency feature. It replaces hard-coded English labels like 'Confirm message' and 'Message text' with references to the device's translation system, and updates the translation files, generated code, and a test marker. There is no change to security logic, cryptography, or how user approvals work.

AI review queuedfeat(cardano): replace chunking with length+offsetby Peter Jaško · adb9d194 · Aug 25, 2025 · 15 filesMessage 62 · AdequateLow 29Details
Commit message · Peter Jaško

feat(cardano): replace chunking with length+offset

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
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 29/100

This commit rewrites how the Trezor device fetches the message payload during Cardano message signing. Instead of the host pushing fixed-size chunks to the device, the device now asks for a specific slice of data by sending a length and offset, and the host replies with exactly that slice. The change also removes the previous 1024-byte maximum payload size limit, so larger messages can now be signed. The code adds bounds checks on both sides, but because the device now requests the entire payload in one go, this could stress limited device RAM. There is no direct evidence in the commit that this fixes a reported security bug; it reads as a feature/refactoring change.

AI review queuedfeat(cardano): reject probable hashesby Peter Jaško · dde4ec8a · Aug 25, 2025 · 3 filesMessage 57 · ThinLow 36Details
Commit message · Peter Jaško

feat(cardano): reject probable hashes

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

This update makes Trezor's Cardano message-signing feature refuse to sign 28-byte payloads that look like cryptographic hashes. The goal is to prevent users from accidentally signing a hash of an unknown message, which could let an attacker trick them into authorizing something they did not read. The change is a simple heuristic: if the payload is exactly 28 bytes and is not made of ordinary printable text, the device now rejects it with an error.

AI review queuedchore(cardano): update UI fixturesby David Misiak · aff9f150 · Aug 25, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · David Misiak

chore(cardano): update UI fixtures

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 changes a single test marker so that a Cardano message-signing test runs on Trezor Model T/Core devices instead of being skipped on the older T1B1 model. It is a test-suite maintenance change with no effect on the actual firmware code that users rely on.

AI review queuedfix(cardano): stop trimming long messagesby Ioan Bizău · 88af9542 · Aug 25, 2025 · 2 filesMessage 57 · ThinLow 34Details
Commit message · Ioan Bizău

fix(cardano): stop trimming long 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
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 34/100

This commit fixes a display issue in Trezor's Cardano message signing feature. Previously, very long messages were visually trimmed on the device screen, so the user could not see the full content before approving a signature. The patch removes the trimming limit and adds a test case for a long payload. The risk is that a user might unknowingly sign a message whose full content they did not review, but the signature itself is still produced over the complete message (not a truncated one).

AI review queuedchore(cardano): rename payloadby Ioan Bizău · bfd66dbb · Aug 25, 2025 · 2 filesMessage 47 · ThinInformational 15Details
Commit message · Ioan Bizău

chore(cardano): rename payload

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

This commit is a simple internal code cleanup in the Cardano app of Trezor firmware. It renames a function parameter from 'payload_first_chunk' to 'payload' and switches from keyword arguments to positional arguments in one call. There is no change to user-visible behavior, security logic, or data handling.

AI review queuedchore(cardano): changes after rebaseby Peter Jaško · 2427d232 · Aug 25, 2025 · 5 filesMessage 57 · ThinInformational 16Details
Commit message · Peter Jaško

chore(cardano): changes after rebase

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

This commit is a routine cleanup after merging (rebasing) other Cardano code changes. It updates the Cardano message-signing flow to use newer internal APIs (sessions instead of raw clients, and a separate SLIP-21 keychain argument) and refreshes automated test snapshots. There is no indication it fixes or introduces a security vulnerability.

Lower-priorityfix(core): allow cross-sign with ETH mainnetby obrusvit · 0cb0650c · Aug 25, 2025 · 2 filesMessage 80 · StrongLow 39Details
Commit message · obrusvit

fix(core): allow cross-sign with ETH mainnet

- Previously only non-standard networks (not SLIP-44 60 or 1) could cross-sign
with Ethereum mainnet. Now any network that isn't Ethereum mainnet can use
Ethereum derivation paths
- This is a workaround enabling access to networks like Hyperliquid that
have conflicting chain IDs and can't claim official SLIP-44 registration.

80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
AI analysis · Low 39/100

This commit changes how Trezor hardware wallets pick which private key to use when signing Ethereum-style transactions on non-Ethereum networks. Previously, only testnet-like or alt-networks could fall back to the standard Ethereum key (the one derived from path m/44'/60'/...). Now every network except Ethereum mainnet itself is allowed to use that same Ethereum key. The stated reason is to support newer networks such as Hyperliquid that share a chain ID with an already-registered network and therefore cannot get their own SLIP-44 coin type. The change is described as a workaround, not as a security fix, and it slightly relaxes the boundary between alt-network keys and the main Ethereum key.

Security candidatetest(core/ethereum): allow signing testnets on ETH mainnetby Tomas Martykan · aa2417a6 · Aug 25, 2025 · 2 filesMessage 72 · AdequateInformational 15Details
Commit message · Tomas Martykan

test(core/ethereum): allow signing testnets on ETH mainnet

72/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
signing boundarysigning or wallet path
AI analysis · Informational 15/100

This commit only adds new test cases for the Ethereum signing feature. It does not change any production firmware code, so it cannot introduce a security vulnerability by itself. The tests verify that the Holesky Ethereum testnet can be signed using both a testnet-style and a mainnet-style derivation path.

Lower-prioritychore(core): move `show_pairing_dialog()` to `trezor.wire.thp.ui` moduleby Roman Zeyde · ff18b00e · Aug 25, 2025 · 3 filesMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde

chore(core): move `show_pairing_dialog()` to `trezor.wire.thp.ui` module

[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 simple code cleanup: it moves the function that displays the on-screen pairing confirmation dialog from one internal file to another. The actual user-facing behavior and security checks remain exactly the same.

Lower-priorityfix(core): correct `ticks_diff` arguments' orderby Roman Zeyde · dc82cd1b · Aug 25, 2025 · 1 fileMessage 57 · ThinInformational 18Details
Commit message · Roman Zeyde

fix(core): correct `ticks_diff` arguments' order

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

This commit fixes the order of arguments in a mock type stub for a timing function. It only changes a Python interface declaration used for testing/type-checking, not the actual device firmware code. The real security risk is minimal, but if the stub had misled developers, it could theoretically contribute to subtle timing bugs in firmware.

Lower-prioritychore(core): exclude logging on non-debug buildsby Roman Zeyde · c95f5b50 · Aug 25, 2025 · 1 fileMessage 57 · ThinInformational 18Details
Commit message · Roman Zeyde

chore(core): exclude logging on non-debug builds

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

This commit removes leftover debug logging from production firmware builds. In non-debug builds, two log messages will no longer be emitted. This is a routine code-quality change with no direct security impact visible in the diff.

Lower-priorityrefactor(core): inline THP `PairingContext` confirmation methodsby Roman Zeyde · af2e7871 · Aug 25, 2025 · 2 filesMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde

refactor(core): inline THP `PairingContext` confirmation methods

[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 simple code cleanup: it removes two small helper methods from a class and replaces their calls with direct calls to the underlying user-interface functions. There is no change in behavior, no bug fix, and no security-related change.

Lower-prioritychore(core): remove `thp.pairing._skip_pairing_dialog()`by Roman Zeyde · d680f9ed · Aug 25, 2025 · 2 filesMessage 77 · AdequateInformational 15Details
Commit message · Roman Zeyde

chore(core): remove `thp.pairing._skip_pairing_dialog()`

It is currently unused, since `should_show_pairing_dialog` is always `True`.

[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 removes a debug-only shortcut that could skip the user-visible pairing dialog on a Trezor device. The shortcut was already unused because the flag that enabled it was always set to True, meaning the dialog was always shown. The change is a cleanup with no functional effect on normal or debug builds.

AI review queuedrefactor(python): refactor THP-related exception typesby Roman Zeyde · c693963a · Aug 25, 2025 · 6 filesMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde

refactor(python): refactor THP-related exception types

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

This commit is a routine code cleanup that renames and reorganizes error/exception types used in Trezor's Python library, particularly for the new THP (Trezor Host Protocol) transport. It does not change security behavior; it only makes error handling more specific and easier to maintain. For example, a generic 'device locked' exception is renamed and moved under a new THP error family, and THP error codes now raise distinct exception classes instead of generic strings.

Security candidatechore(core): add LED enable/disable functionalityby Lukas Bielesch · eecd1e83 · Aug 25, 2025 · 26 filesMessage 72 · AdequateInformational 15Details
Commit message · Lukas Bielesch

chore(core): add LED enable/disable functionality

- align LED colors with firmware
[no changelog]

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
signing or wallet pathboot or update path
AI analysis · Informational 15/100

This commit adds a user-facing setting to turn the device's RGB LED on or off. It is a routine feature addition (a 'chore') with no apparent security relevance. There are no changes that introduce memory corruption, bypass authentication, leak secrets, or alter cryptographic behavior.

Security candidatechore(core): add secmon keys to trezorlibby tychovrahe · a4844203 · Aug 25, 2025 · 2 filesMessage 57 · ThinInformational 18Details
Commit message · tychovrahe

chore(core): add secmon keys to trezorlib

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

This commit updates Trezor's Python library so it can recognize and verify digital signatures for a new 'secure monitor' (secmont) firmware component. It fixes a mismatch where the secmon image verifier was accidentally returning bootloader keys instead of secmon keys, and adds the correct public keys and signature-count rules for two development hardware models. There is no direct evidence this is a security vulnerability fix; it reads as a routine key-registry update for a new feature.

Security candidatechore(core): add new menu lengthby Lukas Bielesch · 6e4b343d · Aug 25, 2025 · 3 filesMessage 57 · ThinInformational 15Details
Commit message · Lukas Bielesch

chore(core): add new menu length

[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 UI code cleanup. It introduces a new medium-length menu size (10 items) and updates the device menu screen to use it instead of the previous short size (5 items). There is no security-relevant change here.

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): update device menu about screenby Lukas Bielesch · 18b20052 · Aug 25, 2025 · 16 filesMessage 62 · AdequateInformational 15Details
Commit message · Lukas Bielesch

chore(core/eckhart): update device menu about screen

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

This commit is a routine user-interface update for the Trezor hardware wallet. It changes the 'About' screen in the device menu to show more information—firmware version, firmware type (Bitcoin-only or Universal), and a Bluetooth version number—instead of only the firmware version. There is no security-relevant change; it is purely a display/layout improvement.