TZ
← All projectsTrezor

Trezor firmware

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

BitcoinHardware walletsNormal
Repository coverage

2673 commits in the local evidence base

Every captured commit receives deterministic security triage and a separate communication-quality score. Security candidates and broader second-pass signals receive full-patch Ollama analysis.

923security candidates311second-pass queue2662AI analyses
323commits · 30 days
552commits · 60 days
1337commits · 180 days
2644commits · 365 days
Backfill bands
Aug 5 → Feb 61298 seen115 candidatesComplete
Feb 6 → Jun 6775 seen58 candidatesComplete
Jun 6 → Jul 6217 seen13 candidatesComplete
Jul 6 → Aug 5360 seen54 candidatesComplete
Commit communication

Does the history explain itself?

Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.

66/100 average clarity
482Strong · 80–100
1323Adequate · 60–79
863Thin · 40–59
5Opaque · 0–39
Read the scoring rubric →
Developer activity

Who is changing the project?

Public Git author strings; identities are not independently verified.

DeveloperCommitsCandidatesAnalyzedHigh riskMessage avg.
tychovrahe292104292461
Roman Zeyde579182576272
PrisionMike945994273
Andrew Kozlik712470265
obrusvit21376213164
Jakub Janků22919182
Martin Pastyřík24823173
cepetr19682196059
Ioan Bizău23076230059
M1nd3r20772207071
Lukas Bielesch846784067
Martin Milata17446174063
Analysis record

Published AI watches

Last scanned 49 minutes ago

Informational 15 AI analysisMessage 72 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

test(core/ethereum): EIP-7702 tuple nonce must be non-zero

This commit only changes test data and expected screen fingerprints for Trezor's Ethereum EIP-7702 signing tests. It replaces one test case that used a nonce of 0 with the same scenario using a nonce of 67, because the real-world protocol …

Test-only change with no production code modificationEIP-7702 tuple nonce constraint reflected in test dataExpected UI test hashes regenerated for renamed test vector
76cd2ed4by Roman Zeyde+28−282 files
No security note in commit
Informational 15 AI analysisMessage 85 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

build(core): propagate emulator feature from projects

This commit is a build-system cleanup for the Trezor firmware's Rust code. It changes how the 'emulator' feature flag is passed between different software components. The firmware project no longer supports the emulator feature (which make…

5642074cby matejcik+5−44 files
No security note in commit
Low 27 AI analysisMessage 93 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(core): digest EIP-1559/EIP-7702 magic without RLP encoding

This commit is a small internal cleanup in Trezor's Ethereum signing code. It changes how two special 'magic' bytes (the transaction type markers for newer Ethereum transaction formats) are fed into the cryptographic hash. Previously the c…

Refactor of cryptographic hashing path for Ethereum transaction signingRemoval of implicit assumption that RLP encodes small integers as single bytesChange affects EIP-1559 and EIP-7702 transaction type markers
2362bc6fby Roman Zeyde+6−32 files
No security note in commit
Informational 15 AI analysisMessage 98 · Strong
TZ TrezorTrezor firmware BitcoinHardware wallets

test(core): test RLP self-encoding bytestrings

This commit only adds a new unit test to the Trezor firmware codebase. It verifies that very short Ethereum RLP-encoded byte strings and small positive integers encode to themselves, which is a known property of the RLP specification. No p…

41538733by Roman Zeyde+16−01 file
No security note in commit
Informational 18 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

chore(core): remove sha1 from regular FW

This commit removes the SHA-1 hashing function from the regular Trezor firmware. SHA-1 is an old, weak hash algorithm that is no longer considered secure for sensitive uses. The change deletes the code that exposes SHA-1 to apps running on…

Removal of a deprecated cryptographic primitive (SHA-1) from the firmware API surfaceReduction of attack surface and prevention of future misuse of a collision-vulnerable hashNo direct vulnerability patch or memory-safety bug is present in the diff
16d15774by M1nd3r+0−2006 files
No security note in commit
Low 27 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(crypto): pass static public key to `noise_xxpsk3_*_init()`

This commit changes how a cryptographic library sets up secure connections. Previously, the code automatically calculated the public key from the private key. Now, the caller must provide the public key directly. This is a code-quality ref…

Cryptographic key handling changed: public key is now supplied rather than derivedPotential reduction of key-mismatch risk if caller provides correct public keyNew null-pointer check added for static_public_key
1b3128a4by Ondřej Vejpustek+20−92 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(crypto): change order of parameters

This commit simply reorders the arguments of an internal function called dh() and updates every place that calls it. The actual math and security behavior are unchanged; it is a code cleanup with no security effect.

33a119eeby Ondřej Vejpustek+15−151 file
No security note in commit
Informational 15 AI analysisMessage 47 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(crypto): prefix enums

This commit simply renames two groups of internal status labels (called enums) in the Trezor firmware's cryptographic code. The old names like WAITING_FOR_REQUEST1 were shared between two different parts of the code, so the developer gave …

f1c74046by Ondřej Vejpustek+26−222 files
No security note in commit
Low 37 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

fix(crypto): prevent calling `memzero(NULL, ...)`

This commit fixes three places in the Trezor firmware's cryptographic code where a memory-clearing function could be called with a NULL pointer. In practice, passing NULL to memzero is harmless on Trezor's platform (it does nothing), but i…

NULL pointer passed to memory-zeroing helper in cryptographic codeUndefined behavior in C standard library contractDefensive hardening in Noise protocol implementation
0394e934by Ondřej Vejpustek+11−51 file
No security note in commit
Informational 18 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(crypto): return remote static key from `noise_xxpsk3_*_handle_*()`

This commit is a code cleanup (refactor) for the cryptographic handshake code used in Trezor devices. It changes how the other party's long-term public key is returned to the caller: instead of storing it inside an internal state structure…

Removal of long-term public key storage from internal handshake stateCaller-supplied output buffer for remote static public key reduces internal secret retentionError-path memzero of returned key material on failure
3ead1aa6by Ondřej Vejpustek+79−423 files
No security note in commit
Informational 15 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): rename ProjectProfile to ProjectConfig

This commit is a simple rename from 'ProjectProfile' to 'ProjectConfig' in two Rust source files. It only changes variable names, struct names, and error messages. There is no change to program logic, security behavior, or how data is hand…

581b37bcby cepetr+18−182 files
No security note in commit
Informational 15 AI analysisMessage 47 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): introduce ResolvedBuildArgs

This commit is a straightforward internal code refactor in Trezor's build tooling. It introduces a new ResolvedBuildArgs structure that centralizes default values for command-line build options before they are used. There is no change to u…

c24d1be8by cepetr+99−295 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): make build args tristate

This commit is a straightforward internal refactor of the Trezor firmware build tool's command-line argument handling. It changes many on/off flags from plain booleans to optional booleans so they can explicitly be set to true, false, or l…

271e249aby cepetr+57−561 file
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): introduce build options

This commit is a straightforward internal code reorganization in Trezor's build tooling. It moves build-option definitions from one Rust source file to a new module and adds the ability to load those options from a configuration file. Ther…

75085e2aby cepetr+275−2007 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

feat(xtask): introduce build presets

This commit is a developer tooling change for the Trezor firmware build system. It introduces 'build presets'—named configuration bundles stored in TOML files—so developers can select common build settings with a single command-line flag i…

eefe8f7fby cepetr+397−419 files
No security note in commit
Informational 15 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

feat(xtask): upgrade manifest.yaml for tf-tools 0.3.0

This commit is a routine update to a build-system manifest file used by Trezor's internal tooling. It reorganizes and re-labels feature flags (for example, moving options between 'Features', 'Debugging', and 'Signing' groups and simplifyin…

57b28b89by cepetr+86−1351 file
No security note in commit
Informational 15 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

feat(core): add tf-tools 0.3.1+ configuration file

This commit adds a single configuration file for a Trezor developer tool (a VS Code extension). It only points the tool to existing folders in the repository and contains no executable code, no secrets, and no changes to firmware behavior.…

e554f618by cepetr+8−01 file
No security note in commit
Informational 15 AI analysisMessage 62 · Adequate
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): rename ResolvedBuild to ResolvedBuildFeatures

This commit is a simple internal code cleanup: it renames a Rust data structure called ResolvedBuild to ResolvedBuildFeatures and updates the places that use it. There are no functional changes, no security fixes, and no changes to how the…

aaa2780bby cepetr+5−52 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

fix(core): fix up clippy warnings in storage.rs

This is a tiny code cleanup commit that replaces two calls to `ptr::null()` with Rust's `unwrap_or_default()` to silence automated Clippy lint warnings. It does not change what value is produced (still a null pointer when no salt is provid…

62354461by cepetr+2−21 file
No security note in commit
Informational 15 AI analysisMessage 47 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

refactor(xtask): simplify dependencies

This commit is a straightforward internal code cleanup in Trezor's build tooling. It moves a data structure and two helper methods from one Rust source file to another and updates callers accordingly. There is no change to user-facing beha…

6268349fby cepetr+9−193 files
No security note in commit
Repository ledger

Explore captured commits

Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.

Lower-priorityfix(core): fix mpu settings in secmonby cepetr · 7a97dc53 · Sep 18, 2025 · 1 fileMessage 57 · ThinModerate 57Details
Commit message · cepetr

fix(core): fix mpu settings in secmon

[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 · Moderate 57/100

This commit fixes a memory-protection configuration bug in the Trezor hardware wallet's secure monitor (secmon) for STM32U5 devices. The code was using the wrong linker symbol (_codelen, which measures the entire firmware) to set the size of the protected secure-monitor memory region. It now uses the correct symbol (_secmon_size, which measures only the secure monitor). The effect is that the MPU (Memory Protection Unit) guard band around the secure monitor was likely oversized or misaligned, potentially weakening isolation between the secure monitor and the rest of the firmware.

Lower-prioritychore(nordic): bump trezor-ble version before releaseby tychovrahe · a16dd648 · Sep 17, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · tychovrahe

chore(nordic): bump trezor-ble version before release

[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 a version number for a Bluetooth Low Energy component from 0.1.1 to 1.0.0. There is no code change, no bug fix, and no security-related content visible in the diff or commit message.

Lower-priorityfeat(core): recover battery_critical flag on soc threshold.by kopecdav · 3ecb6a35 · Sep 17, 2025 · 3 filesMessage 62 · AdequateLow 29Details
Commit message · kopecdav

feat(core): recover battery_critical flag on soc threshold.

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

This commit changes how a Trezor hardware wallet decides whether its battery is dangerously low and when it is safe to turn on again. Previously the device used raw battery voltage thresholds and a smoothed voltage estimate. Now it relies mainly on the fuel gauge's reported state-of-charge (SOC) percentage and whether USB power is connected. The change removes some voltage-based recovery thresholds and a smoothing variable. It is a power-management refinement, not an obvious security fix, but it could affect device availability and safe shutdown behavior.

Lower-priorityfix(core): fix battery_critical flag recovering from backup memory.by kopecdav · 2b3a9d82 · Sep 17, 2025 · 1 fileMessage 62 · AdequateLow 31Details
Commit message · kopecdav

fix(core): fix battery_critical flag recovering from backup memory.

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

This commit fixes a bug in the Trezor hardware wallet's power manager for STM32U5 devices. When the device resumed from a low-power state using saved 'backup memory,' it forgot whether the battery had been flagged as critically low. The fix restores that flag so the device correctly remembers a critical battery state across power events. The bug could theoretically allow a device to power on or continue operating when it should instead treat the battery as critically low, but the practical security impact is limited because other battery checks remain active.

Lower-priorityfix(core): remove backup RAM forced soc reset in power manager at battery critical state.by kopecdav · 0db89f9e · Sep 17, 2025 · 1 fileMessage 62 · AdequateLow 30Details
Commit message · kopecdav

fix(core): remove backup RAM forced soc reset in power manager at battery critical state.

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

This commit removes a safety override in the Trezor hardware wallet's power manager. Previously, when the battery was at a critically low level, the device forced the reported battery percentage (state of charge, or 'soc') to 0 before saving it to backup memory. Now it saves the actual fuel gauge reading instead. This is likely a bug fix for incorrect battery reporting, but the security implications are unclear from the diff alone.

Lower-priorityfix(core): allow Evolu msgs in BTC onlyby obrusvit · 06c82aff · Sep 17, 2025 · 5 filesMessage 57 · ThinLow 29Details
Commit message · obrusvit

fix(core): allow Evolu msgs in BTC only

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

This commit changes which Trezor firmware variant supports the 'Evolu' messages. Previously, these messages were only available in the full (non-Bitcoin-only) firmware. The patch marks them as available in the Bitcoin-only firmware instead. It is a feature-availability fix rather than a direct security patch, but misclassification of message availability could theoretically affect the attack surface of the Bitcoin-only build.

Security candidatefeat(core/bootloader): stop charging indication when the battery is fully chargedby tychovrahe · 3d6c4354 · Sep 17, 2025 · 2 filesMessage 62 · AdequateInformational 15Details
Commit message · tychovrahe

feat(core/bootloader): stop charging indication when the battery is fully charged

[no changelog]

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

This commit changes the bootloader's battery-charging LED behavior. When the battery is fully charged, the charging LED effect now turns off instead of continuing to show a charging animation. It also adds a small helper function for the Unix emulator build so it can compile. There is no security issue visible in this change.

AI review queuedchore(core): Crowdin sync for wipe code cancelby obrusvit · 3f4ac8b0 · Sep 17, 2025 · 7 filesMessage 57 · ThinInformational 15Details
Commit message · obrusvit

chore(core): Crowdin sync for wipe code cancel

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

This commit only updates translated text strings for the Trezor hardware wallet's user interface. It adds a new message about canceling the setup of a 'wipe code' (a feature that erases the device if a specific code is entered) and updates the digital signature that verifies the translation files. There is no code that handles secrets, cryptography, device communication, or security logic being changed.

Security candidatefix(core): differentiate wipe code from pin set upby Ioan Bizău · 08ea80c9 · Sep 17, 2025 · 23 filesMessage 62 · AdequateLow 28Details
Commit message · Ioan Bizău

fix(core): differentiate wipe code from pin set up

[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
access controlcryptography-sensitive pathsigning or wallet pathboot or update pathauthentication path
AI analysis · Low 28/100

This commit fixes a user-interface labeling bug where setting up a 'wipe code' (a special code that erases the device) was incorrectly shown using the same on-screen text and flow as setting up a regular PIN. The change renames the confirmation flow, adds a flag to distinguish wipe-code setup from PIN setup, and shows wipe-code-specific cancellation text on newer Trezor layouts. It is a UI clarity fix, not a cryptographic or access-control flaw.

Lower-priorityfeat(core): show nRF version in About menuby tychovrahe · 375ee785 · Sep 17, 2025 · 4 filesMessage 57 · ThinInformational 15Details
Commit message · tychovrahe

feat(core): show nRF version in About menu

[no changelog]

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

This commit adds a new feature that displays the version of the nRF Bluetooth chip firmware in the device's About menu. It replaces a previously hardcoded placeholder version number with a real value read from the hardware. There is no security issue visible in this change.

AI review queuedchore(core): additional Crowdin sync for Eckhartby obrusvit · e70903f5 · Sep 17, 2025 · 8 filesMessage 57 · ThinInformational 15Details
Commit message · obrusvit

chore(core): additional Crowdin sync for Eckhart

[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 translation update for the Trezor hardware wallet firmware. It adds and updates translated text strings for a new device model called 'Eckhart' across several language files (Czech, German, Spanish, French, Italian, Portuguese). It also updates the digital signature that verifies the translation files and refreshes the expected visual test snapshots because the on-screen text has changed. There is no code that handles secrets, cryptography, or device security logic in this change.

Lower-priorityfix(core): disable secret access after lockingby tychovrahe · 9bfda179 · Sep 17, 2025 · 1 fileMessage 57 · ThinHigh 71Details
Commit message · tychovrahe

fix(core): disable secret access after locking

[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 · High 71/100

This update fixes a security gap in how the Trezor hardware wallet protects its most sensitive storage area. Previously, after the device 'locked' its secret memory region, that region might still have remained accessible to running code. Now the code explicitly disables all access to the secret area as soon as it is locked, making it harder for an attacker who has already broken into the device to read protected keys or data.

Lower-priorityfix(core/prodtest): improve nrf re-pairing protectionby tychovrahe · c067b9c5 · Sep 17, 2025 · 2 filesMessage 62 · AdequateModerate 59Details
Commit message · tychovrahe

fix(core/prodtest): improve nrf re-pairing protection

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

This commit tightens the rules for pairing a Trezor hardware wallet with its Nordic (nRF) wireless chip during factory production testing. Previously, the production-test tool could try to re-derive the pairing secret even after the device's secure secret storage was locked, which could have allowed unauthorized re-pairing or leaking of the pairing secret. Now the tool first checks whether the devices are already paired, and the secret-derivation function refuses to run if the secrets sector is locked. This is a defensive hardening fix in a manufacturing diagnostic tool, not a user-facing wallet bug.

Lower-priorityfix(core/eckhart): button text splittingby obrusvit · 0cd9af20 · Sep 17, 2025 · 1 fileMessage 80 · StrongInformational 17Details
Commit message · obrusvit

fix(core/eckhart): button text splitting

- ActionBar::Double variant uses negative x offset for the right button
so we must use absolute value of the offset to calculate available width
for the content

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

This is a tiny UI layout fix for the Trezor hardware wallet's 'Eckhart' design. When a button with two text labels is drawn, the code now uses the absolute value of a negative horizontal offset to correctly figure out how much room the text has. Without the fix, the available width could be calculated as larger than it really is, which might cause long button labels to be split awkwardly or overflow visually. There is no direct security signal here—this is a cosmetic/layout bug fix.

AI review queuedchore(core): Crowdin sync for new Eckhart stringsby obrusvit · e6f18cea · Sep 17, 2025 · 8 filesMessage 57 · ThinInformational 15Details
Commit message · obrusvit

chore(core): Crowdin sync for new Eckhart strings

[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 synchronization of translated user-interface text strings for the new 'Eckhart' Trezor device model. It adds and updates translations in Czech, German, Spanish, French, Italian, and Portuguese, plus updates translation signatures and UI test snapshot hashes. There are no code logic changes and no security relevance.

Lower-priorityfeat(core/prodtest): Improve state handling around Tropic locking.by Andrew Kozlik · 2d74a9e7 · Sep 16, 2025 · 6 filesMessage 62 · AdequateLow 42Details
Commit message · Andrew Kozlik

feat(core/prodtest): Improve state handling around Tropic locking.

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

This commit tightens the order in which factory-setup commands must run for Trezor devices that use a Tropic secure chip. It prevents secrets from being initialized or the Tropic chip from being locked if earlier pairing steps were skipped or failed. The changes reduce the chance that a device leaves the factory in an inconsistent or partially-secured state, but they are defensive hardening rather than a fix for a known active attack.

Lower-priorityfeat(core/prodtest): Require Tropic to be locked before otp-variant-write.by Andrew Kozlik · ee395e25 · Sep 16, 2025 · 3 filesMessage 62 · AdequateLow 42Details
Commit message · Andrew Kozlik

feat(core/prodtest): Require Tropic to be locked before otp-variant-write.

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

This commit tightens a factory/production testing tool in Trezor firmware so that it refuses to permanently write a device variant code into one-time programmable memory unless the Tropic secure chip is already locked. Previously, the variant could be written before the chip was locked, which could let a misconfigured or tampered device pass through production with an incorrect or attacker-chosen variant setting. The change also improves the lock-check command so it can return an explicit error state instead of silently answering 'NO'.

Security candidatechore(core/eckhart): make wipe code dependent on pin codeby Lukas Bielesch · 00fce6c9 · Sep 16, 2025 · 1 fileMessage 62 · AdequateLow 26Details
Commit message · Lukas Bielesch

chore(core/eckhart): make wipe code dependent on pin code

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

This small change adjusts when a Trezor device shows that a 'wipe code' is set. Previously, the device menu would display the wipe-code status if the device was initialized, regardless of whether a PIN was set. After the change, the wipe-code status is only shown when both the device is initialized and a PIN is set. This is a UI/logic hardening change: it prevents a user from being misled into thinking a wipe code is active and usable when no PIN is set, because the wipe code feature depends on the PIN entry flow to trigger.

Lower-prioritychore(core/eckhart): enable going back in the 1st recovery wordby Lukas Bielesch · ed4c675a · Sep 16, 2025 · 9 filesMessage 77 · AdequateInformational 18Details
Commit message · Lukas Bielesch

chore(core/eckhart): enable going back in the 1st recovery word

- The flow goes back to the word number selection

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

This commit is a user-interface convenience change for the Trezor hardware wallet. It lets users on the new 'Eckhart' model go back from the very first recovery-word screen to re-pick the number of words, instead of being forced to continue or cancel entirely. The code change is small and appears to be a normal UX improvement, not a security fix or vulnerability.

Security candidatechore(core): ensure dependency between PIN and wipe codeby Lukas Bielesch · 9e4c86c1 · Sep 16, 2025 · 18 filesMessage 62 · AdequateLow 47Details
Commit message · Lukas Bielesch

chore(core): ensure dependency between PIN and wipe code

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
access controlcryptography-sensitive pathsigning or wallet pathauthentication path
AI analysis · Low 47/100

This commit tightens the rules between a Trezor device's PIN and its 'wipe code' (a special code that erases the device). Previously, it was possible to end up in an inconsistent state: for example, you could set a wipe code even when no PIN was set, or remove the PIN while the wipe code was still active. The patch blocks both situations and shows explanatory on-screen messages. It also replaces some internal 'assert False' statements with explicit error raises, which is a defensive cleanup. The change is a hardening fix rather than a remote exploit.

Security candidatechore(core/eckhart): Update regulatory screenby Lukas Bielesch · cfab1dac · Sep 16, 2025 · 4 filesMessage 57 · ThinInformational 15Details
Commit message · Lukas Bielesch

chore(core/eckhart): Update regulatory 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
boot or update path
AI analysis · Informational 15/100

This commit updates the legal/regulatory information screen shown on a Trezor hardware wallet. It adds Japan to the list of regions, includes a new Japan icon, and adds the standard FCC compliance disclaimer text. There is no security-relevant change.

Security candidatefeat(core): introduce lock_managerby matejcik · 37595a63 · Sep 16, 2025 · 11 filesMessage 70 · AdequateLow 27Details
Commit message · matejcik

feat(core): introduce lock_manager

it eats some functions from apps.base, and introduces an asynchronous
notification mechanism for suspending via `notify_suspend()`

70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
Why it was queued
authentication path
AI analysis · Low 27/100

This commit is a code reorganization: it moves device locking, unlocking, homescreen selection, and suspend-related logic out of apps/base.py into a new module called apps/common/lock_manager.py. It also changes how the power button triggers suspend, using a new asynchronous notification mechanism (notify_suspend) instead of immediately locking the device. There is no direct evidence in the diff that this fixes a security vulnerability; it appears to be a feature/refactoring change.

Security candidaterefactor(core): move power management modulesby obrusvit · cb6da02d · Sep 16, 2025 · 7 filesMessage 72 · AdequateInformational 15Details
Commit message · obrusvit

refactor(core): move power management modules

- better code organization

[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 simply moves two small power-management files from one folder to another and updates the build scripts and import paths accordingly. The actual code behavior is unchanged. There is no security fix or vulnerability here.

Security candidatefeat(core): fade backlight even in case a background process is still runningby matejcik · 0aedc0c6 · Sep 16, 2025 · 1 fileMessage 85 · StrongInformational 23Details
Commit message · matejcik

feat(core): fade backlight even in case a background process is still running

when autolock_interrupts_workflow is False and we need to wait for e.g.
transaction signing to finish before we can go to suspend

85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
Why it was queued
signing boundary
AI analysis · Informational 23/100

This small change moves where the screen backlight fade happens in the Trezor hardware wallet's lock/suspend logic. Previously, the screen dimmed only inside the suspend function. Now it also dims earlier when the device decides it should suspend but is still waiting for a background task (like signing a transaction) to finish. The goal is to avoid leaving the screen bright while the user is waiting for a long operation before the device can lock. This is a user-experience and minor privacy improvement, not a fix for a serious security flaw.

AI review queuedstyle: pystyle & cstyleby matejcik · 55f15d0e · Sep 16, 2025 · 3 filesMessage 28 · OpaqueInformational 15Details
Commit message · matejcik

style: pystyle & cstyle

28/100 · OpaqueMessage clarity
✓ Subject identifies a change! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100

This commit is purely a code-style and formatting cleanup. It re-indents a large embedded image byte array, adds a missing import and two function calls related to screen dimming in the lock manager, and wraps a debug log line to fit style guidelines. There is no security-relevant change.