TZ
← All projectsTrezor

Trezor firmware

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

BitcoinHardware walletsNormal
Repository coverage

2691 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 candidates315second-pass queue2669AI analyses
340commits · 30 days
557commits · 60 days
1349commits · 180 days
2621commits · 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
1332Adequate · 60–79
866Thin · 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 Zeyde582183578272
PrisionMike945994273
Andrew Kozlik742572266
obrusvit21376213164
Jakub Janků22919182
Martin Pastyřík24823173
cepetr19782196059
Ioan Bizău23076230059
M1nd3r21173208071
Lukas Bielesch846784067
Martin Milata18146176063
Analysis record

Published AI watches

Last scanned 48 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 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

docs: fix links outside book root

This commit only fixes broken or awkward documentation links inside the project's user guide. It replaces relative links that point outside the documentation folder with internal placeholder files or direct GitHub URLs. There is no change …

0d8d4b1bby Martin Milata+14−810 files
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
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: start Tropic model from multicore testsby Ioan Bizău · 063ddf37 · Oct 14, 2025 · 8 filesMessage 67 · AdequateInformational 15Details
Commit message · Ioan Bizău

chore: start Tropic model from multicore tests

[no changelog]

67/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 15/100

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

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

feat(tests): check Tropic signature and cert chain

[no changelog]

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

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

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

chore: add script to provision Tropic model

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 13/100

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

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

chore(tests): start Tropic model in CI tests

[no changelog]

67/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

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

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

ci(core): show slowest click tests

[no changelog]

67/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

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

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

chore(core): update T3W1 embedded binaries

[no changelog]

(cherry picked from commit 846bdd3a16a89c349f446790ca4e7f40ce1923ac)

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

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

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

chore(core): update T3W1 embedded binaries

[no changelog]

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

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

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

fix(core): fix crash when setting wipe code

[no changelog]

(cherry picked from commit da76bd501dd1bfefba472a969f213253ac6fc7cf)

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
second-pass: broader security terminology
AI analysis · Low 29/100

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

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

fix(core): fix crash when setting wipe code

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
second-pass: broader security terminology
AI analysis · Low 40/100

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

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

fix(core): lazy tropic secure session start

[no changelog]

(cherry picked from commit f598e385c4f0a680c4446f4dd3c24e01094956b7)

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

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

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

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

[no changelog]

(cherry picked from commit b66802932c5d40dc9ad0bf1d060e7f61270af5b0)

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

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

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

fix(core): lazy tropic secure session start

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Low 41/100

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

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

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

[no changelog]

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

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

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

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

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

[no changelog]

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

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

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

refactor(core): unify `ButtonRequest` handlers

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

[no changelog]

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

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

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

refactor(core): remove unneeded try-except block

[no changelog]

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 12/100

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

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

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

[no changelog]

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

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

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

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

[no changelog]

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

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

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

ci(core): reduce coverage threshold for T3B1

[no changelog]

67/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

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

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

ci(core): always upload coverage report

[no changelog]

67/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

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

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

test(python): use `nist256p1` in payment device tests
[no changelog]

87/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Mentions testing or verification
AI analysis · Informational 15/100

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

Lower-prioritychore(core): use `nist256p1` instead of `secp256k1` for payment request verification [no changelog]by M1nd3r · 5dca9304 · Oct 9, 2025 · 1 fileMessage 87 · StrongInformational 18Details
Commit message · M1nd3r

chore(core): use `nist256p1` instead of `secp256k1` for payment request verification
[no changelog]

87/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Mentions testing or verification
Why it was queued
defensive validation
AI analysis · Informational 18/100

This commit changes the cryptographic curve used to verify payment requests in Trezor firmware from secp256k1 (used by Bitcoin) to nist256p1 (also known as secp256r1, a standard government/enterprise curve). It also updates the built-in test public key to match the new curve. The change itself is a normal engineering update, but it touches code that protects payment request signatures, so it has minor security relevance.

Lower-priorityfix(core): don't send disconnect notification on pairing mode entryby tychovrahe · ef47b214 · Oct 9, 2025 · 1 fileMessage 77 · AdequateLow 34Details
Commit message · tychovrahe

fix(core): don't send disconnect notification on pairing mode entry

[no changelog]

(cherry picked from commit 8caad2a02f6a121faef66d52e6a2f221779410a6)

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 removes a step in the Trezor hardware wallet's Bluetooth pairing flow. Previously, when entering pairing mode, the device forcibly sent a disconnect notification, waited 300 milliseconds, and then disconnected any active Bluetooth connection. The patch skips that forced disconnect. The change could matter for security because sending a disconnect notification before pairing might have leaked that the device was about to pair, or the forced disconnect could have disrupted a secure session at a sensitive moment. However, the commit message gives no security explanation, so this is mostly a functional or UX fix.

Lower-prioritychore: reserve message type range [no changelog]by M1nd3r · 249db066 · Oct 9, 2025 · 2 filesMessage 72 · AdequateInformational 15Details
Commit message · M1nd3r

chore: reserve message type range
[no changelog]

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

This commit simply marks protobuf message type numbers 123 through 149 as reserved for future internal use. It is a routine development housekeeping change with no functional code changes and no security relevance.

Lower-priorityfix(core): don't send disconnect notification on pairing mode entryby tychovrahe · 8caad2a0 · Oct 9, 2025 · 1 fileMessage 62 · AdequateLow 33Details
Commit message · tychovrahe

fix(core): don't send disconnect notification on pairing mode entry

[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 · Low 33/100

This commit removes code that forcibly disconnected an active Bluetooth connection and sent a disconnect notification whenever the user entered Bluetooth pairing mode on a Trezor device. The change likely fixes a user-experience or minor security issue where entering pairing mode unnecessarily dropped an existing trusted connection, which could be disruptive or potentially exploitable in a social-engineering or pairing-confusion scenario. The patch is small and removes a deliberate disconnect step, but the commit message gives no explicit security context.