TZ
← All projectsTrezor

Trezor firmware

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

BitcoinHardware walletsNormal
Repository coverage

2661 commits in the local evidence base

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

916security candidates311second-pass queue2399AI analyses
336commits · 30 days
588commits · 60 days
1363commits · 180 days
2648commits · 365 days
Backfill bands
Aug 5 → Feb 61298 seen115 candidatesComplete
Feb 6 → Jun 6775 seen58 candidatesComplete
Jun 6 → Jul 6217 seen13 candidatesComplete
Jul 6 → Aug 5360 seen54 candidatesComplete
Commit communication

Does the history explain itself?

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

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

Who is changing the project?

Public Git author strings; identities are not independently verified.

DeveloperCommitsCandidatesAnalyzedHigh riskMessage avg.
tychovrahe292104253461
Roman Zeyde572177487271
PrisionMike945994273
Andrew Kozlik712464265
obrusvit21376186164
Jakub Janků22919182
Martin Pastyřík23823173
cepetr19682188059
Ioan Bizău23076217059
M1nd3r20671193071
Lukas Bielesch846776067
Martin Milata17446167063
Analysis record

Published AI watches

Last scanned 5 minutes ago

Informational 18 AI analysisMessage 57 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

chore(core): remove sha1 from regular FW

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

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

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

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

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

refactor(crypto): change order of parameters

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

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

refactor(crypto): prefix enums

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

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

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

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

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

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

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

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

refactor(xtask): rename ProjectProfile to ProjectConfig

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

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

refactor(xtask): introduce ResolvedBuildArgs

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

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

refactor(xtask): make build args tristate

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

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

refactor(xtask): introduce build options

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

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

feat(xtask): introduce build presets

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

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

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

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

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

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

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

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

refactor(xtask): rename ResolvedBuild to ResolvedBuildFeatures

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

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

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

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

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

refactor(xtask): simplify dependencies

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

6268349fby cepetr+9−193 files
No security note in commit
Informational 15 AI analysisMessage 47 · Thin
TZ TrezorTrezor firmware BitcoinHardware wallets

docs: add xtask documentation

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

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

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

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

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

ci(core): fix nightly ARM emulator filename

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

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

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

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

06cab32bby Martin Milata+1−11 file
No security note in commit
Repository ledger

Explore captured commits

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

AI review queuedchore(translations): sync Crowdin translations [no changelog]by Michal Kazda · 0e969ceb · Apr 28, 2026 · 6 filesMessage 77 · AdequateInformational 15Details
Commit message · Michal Kazda

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

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

This commit is a routine synchronization of translated user-interface text strings for the Trezor hardware wallet firmware. It updates wording in Czech, German, Spanish, French, Portuguese, and the translation signature file. There are no code, logic, cryptographic, or security behavior changes.

Lower-prioritychore(translations): preserve NBSP in front of : as well when merging [no changelog]by Michal Kazda · b444e8a2 · Apr 28, 2026 · 1 fileMessage 77 · AdequateInformational 15Details
Commit message · Michal Kazda

chore(translations): preserve NBSP in front of : as well when merging
[no changelog]

77/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
translation-only discount
AI analysis · Informational 15/100

This commit is a minor translation-processing fix. It changes a script that cleans translated text so it keeps a special non-breaking space before colons (':') in addition to question marks and exclamation points. This is purely a localization/punctuation formatting tweak and has no security relevance.

AI review queuedchore(translations): shorten `fr` stringby Ioan Bizău · 866e0ed6 · Apr 28, 2026 · 3 filesMessage 57 · ThinInformational 15Details
Commit message · Ioan Bizău

chore(translations): shorten `fr` string

[no changelog]

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

This commit is a routine translation update for the French language. It changes the capitalization of one word ('fragment' to 'Fragment') and removes the word 'n°' (number) from two French text strings shown on the device during SLIP39 backup/recovery flows. The accompanying files are just updated translation signatures and UI test fixture hashes. There is no code logic change and no security relevance.

Security candidatechore(translations): adding ° glyph to French fonts [no changelog]by Michal Kazda · 27d3c904 · Apr 28, 2026 · 23 filesMessage 77 · AdequateInformational 15Details
Commit message · Michal Kazda

chore(translations): adding ° glyph to French fonts
[no changelog]

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

This commit adds the degree symbol (°) to French font files used on Trezor hardware wallets. It is a routine translation/localization update with no security relevance. There is no code change that could be exploited or affect device security.

AI review queuedchore(translations): sync second batch of Crowdin translations [no changelog]by Michal Kazda · 1f6513c7 · Apr 28, 2026 · 7 filesMessage 77 · AdequateInformational 15Details
Commit message · Michal Kazda

chore(translations): sync second batch of Crowdin translations
[no changelog]

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

This commit is a routine synchronization of translated user-interface text for the Trezor hardware wallet. It updates Czech, German, Spanish, French, and Portuguese translation files, refreshes the cryptographic signature that protects those translations, and updates the expected screen hashes used by automated UI tests. There is no change to security logic, cryptography, or how the device handles private keys or transactions.

Lower-priorityfeat(core): force longer auto-lock dur if neededby obrusvit · 15e9b2c0 · Apr 28, 2026 · 5 filesMessage 80 · StrongInformational 18Details
Commit message · obrusvit

feat(core): force longer auto-lock dur if needed

- we want to use longer auto-lock (rather auto-suspend) time in critical
onboarding flows, i.e. creating a backup, and recovering a wallet, when
the device is powered by a battery
- this involves only battery-powered device (currently T3W1)
- the auto-lock is set to 2 minutes, a device default is 40 seconds

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

This commit changes the Trezor hardware wallet's screen-lock/suspend behavior so that during two sensitive setup tasks—creating a wallet backup and recovering a wallet—the device stays awake for at least 2 minutes instead of using the normal 40-second battery timeout. This only affects the battery-powered T3W1 model. It is a usability/usability-safety feature, not a fix for a known exploitable vulnerability.

Security candidatefix(core): handle stack overflow on stm32f4 properlyby cepetr · 32fa8e16 · Apr 28, 2026 · 1 fileMessage 82 · StrongModerate 59Details
Commit message · cepetr

fix(core): handle stack overflow on stm32f4 properly

[no changelog]

Co-authored-by: Copilot <copilot@github.com>

82/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Names security-relevant behavior explicitly
Why it was queued
memory safety
AI analysis · Moderate 59/100

This commit fixes a bug in the Trezor hardware wallet's low-level crash handler for STM32F4 devices. The handler decides whether a memory fault is a stack overflow by checking the wrong stack pointer due to an inverted branch condition. Before the fix, the device could fail to recognize or recover from a stack overflow on the main stack, potentially leaving it in a crashed or unresponsive state instead of resetting safely.

Lower-priorityfix(core): increase stack size on F4 modelsby tychovrahe · 0c49e9e8 · Apr 27, 2026 · 7 filesMessage 57 · ThinLow 32Details
Commit message · tychovrahe

fix(core): increase stack size on F4 models

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

This commit increases the kernel's main stack size from 8 KB to 10 KB on several Trezor hardware wallet models that use STM32F4 microcontrollers. To make room, it slightly shrinks a separate auxiliary RAM region (AUX2). The change is a defensive hardening fix: a stack that is too small can overflow, which on these devices triggers a memory-protection fault and may crash or reset the device. The commit itself does not describe a specific attack, but insufficient stack space is a known reliability and security concern in embedded systems.

Security candidatefeat(ethereum): Add erc-4626 claim flow.by PrisionMike · ad421a7c · Apr 27, 2026 · 18 filesMessage 72 · AdequateLow 37Details
Commit message · PrisionMike

feat(ethereum): Add erc-4626 claim flow.

- Changelog previously added.

[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
cryptography-sensitive pathsigning or wallet path
AI analysis · Low 37/100

This commit adds support for ERC-4626 vault reward claims and changes how Trezor handles Ethereum transactions with extra trailing bytes (calldata suffix). Previously, vault transactions with any trailing bytes were rejected as errors. Now, the device accepts them but shows the extra bytes to the user as a 'Calldata suffix' screen. The change also introduces a new 'claim' function path, but the actual claim UI is stubbed out and returns None (disabled) for now. The commit is a feature addition, not a documented security fix, but it touches transaction parsing and user confirmation flows.

AI review queuedfix(l10n): various en strings improvements [no changelog]by Michal Kazda · 7c014e24 · Apr 27, 2026 · 12 filesMessage 77 · AdequateInformational 15Details
Commit message · Michal Kazda

fix(l10n): various en strings improvements
[no changelog]

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

This commit is a routine language and translation update for the Trezor hardware wallet firmware. It changes English wording (for example, 'Input data' becomes 'Data', 'Access passphrase wallet' becomes 'Access Passphrase wallet'), removes some trailing colons from prompts, and updates the generated translation tables and test fixtures accordingly. There is no code behavior change, no security fix, and no vulnerability.

Security candidaterefactor(core): startup args api improvementby cepetr · 43abfa54 · Apr 27, 2026 · 3 filesMessage 57 · ThinInformational 12Details
Commit message · cepetr

refactor(core): startup args api improvement

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

This commit is a code cleanup in the Trezor firmware's bootloader startup system. It replaces a fixed-size on-stack certificate buffer with a new 'reserve/commit/discard' API that lets the bootloader allocate space for the MCU attestation certificate directly in the shared startup-arguments buffer. There is no indication in the commit or supplied references that this fixes a security bug; it appears to be a defensive refactor to avoid large stack buffers and make the API more flexible.

Security candidatechore(core): add `{0} / {1} bytes` to translation stringsby Roman Zeyde · e37d8e7b · Apr 26, 2026 · 16 filesMessage 77 · AdequateInformational 15Details
Commit message · Roman Zeyde

chore(core): add `{0} / {1} bytes` to translation strings

Also, move streaming blob confirmation into universal FW.

[no changelog]

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

This commit is a routine user-interface cleanup. It adds new translation strings for showing Ethereum transaction data size (for example, "Input data: 12 / 100 bytes") and moves the code that displays streaming data chunks so it can be shared across all Trezor firmware variants. There is no security-relevant change visible in the code.

Lower-priorityrefactor(core): allow using custom `Layout` for menu flowsby Roman Zeyde · 1cda41a6 · Apr 24, 2026 · 1 fileMessage 77 · AdequateInformational 15Details
Commit message · Roman Zeyde

refactor(core): allow using custom `Layout` for menu flows

Will be used for N4W1 layouts.

[no changelog]

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

This is a small, safe code cleanup in the user-interface layer of the Trezor firmware. It lets menu screens use a different visual layout type in the future, but does not change any existing behavior or fix a security problem. There is no indication of a vulnerability.

Lower-priorityfeat(core): allow waiting for N4W1 emulator "connection"by Roman Zeyde · ffb3293a · Apr 24, 2026 · 1 fileMessage 85 · StrongInformational 21Details
Commit message · Roman Zeyde

feat(core): allow waiting for N4W1 emulator "connection"

"Hold the tag" layout should wait for tap event.

Data read/write should be done later (using a different "progress" layout).

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

This commit changes a debug-only mock file for the N4W1 hardware feature in Trezor firmware. It adds a user-facing 'hold the tag' screen that waits for a simulated tap/connection event before continuing. The change is confined to a debugging helper and does not appear to alter normal wallet security logic.

Lower-priorityrefactor(core): move `_DisplayHandler` to `recovery_device.layout`by Roman Zeyde · f5108df9 · Apr 24, 2026 · 2 filesMessage 77 · AdequateInformational 15Details
Commit message · Roman Zeyde

refactor(core): move `_DisplayHandler` to `recovery_device.layout`

Similar to `reset_device.layout` containing wordlist-related layouts.

[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 is a straightforward internal code reorganization. It moves the `_DisplayHandler` class and related helper functions from one file to another within the recovery device feature, without changing what the code actually does. There is no security-relevant change visible in the diff.

Lower-priorityrefactor(core): split per-share loop into a helper `_DisplayBackup` methodby Roman Zeyde · f34b2dbc · Apr 24, 2026 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Roman Zeyde

refactor(core): split per-share loop into a helper `_DisplayBackup` method

[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 takes an existing loop that walks the user through backing up each recovery-share and moves that loop body into a new helper method named `_backup_share`. There is no change to what the code does, what data it handles, or how it interacts with the user. It is purely a structural refactor.

Security candidaterefactor(core): enforce layout scoping for `flow_get_address`by Roman Zeyde · 45b9fac8 · Apr 24, 2026 · 4 filesMessage 62 · AdequateInformational 11Details
Commit message · Roman Zeyde

refactor(core): enforce layout scoping for `flow_get_address`

[no changelog]

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

This is a small internal code cleanup in the Trezor firmware's user-interface layer. It changes how the 'show address' screen is wrapped so that the layout object is managed within a scoped context manager rather than passed directly to a helper function. There is no indication this fixes a security bug or changes user-visible behavior.

Security candidaterefactor(core): enforce layout scoping for keyboards and select_menuby Roman Zeyde · 49194736 · Apr 24, 2026 · 6 filesMessage 97 · StrongInformational 23Details
Commit message · Roman Zeyde

refactor(core): enforce layout scoping for keyboards and select_menu

Introduce `LayoutContext` type, which will be returned by some Rust
layout factory functions. In order to get a `LayoutObj`, it has to
be used as a context manager - thus enforcing proper layout scoping.

The change will be gradually introduced, since some UI flows require
more refactoring.

[no changelog]

97/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
cryptography-sensitive pathboot or update path
AI analysis · Informational 23/100

This is a code cleanup change in the Trezor hardware wallet's user-interface layer. It introduces a new 'LayoutContext' wrapper so that on-screen keyboards and menus are always used inside a 'with' block, ensuring their memory is released promptly. The commit itself does not claim to fix a security bug, but it references an internal issue (#6811) about lifetime management, suggesting it is a defensive step against possible UI object lifetime problems.

AI review queuedchore(deps): bump protobuf from 6.32 to 6.33.5by dependabot[bot] · a15a0f63 · Apr 24, 2026 · 2 filesMessage 88 · StrongInformational 15Details
Commit message · dependabot[bot]

chore(deps): bump protobuf from 6.32 to 6.33.5

Bumps [protobuf](https://github.com/protocolbuffers/protobuf) from 6.32 to 6.33.5.
- [Release notes](https://github.com/protocolbuffers/protobuf/releases)
- [Commits](https://github.com/protocolbuffers/protobuf/commits)

---
updated-dependencies:
- dependency-name: protobuf
dependency-version: 6.33.5
dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

88/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
Why it was queued
automated dependency-update discountsecond-pass: broader security terminology
AI analysis · Informational 15/100

This is a routine automated dependency update by Dependabot, bumping the pinned Python protobuf package from version 6.32 to 6.33.5. The change only edits two version pins in project configuration files and contains no code changes. There is no indication in the commit that this fixes or introduces a security issue.

Lower-priorityrefactor(core): Add secret_mcu_device_cert_*() methods.by Andrew Kozlik · bffee487 · Apr 24, 2026 · 3 filesMessage 62 · AdequateInformational 18Details
Commit message · Andrew Kozlik

refactor(core): Add secret_mcu_device_cert_*() 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 18/100

This commit is a code cleanup (refactor) that moves the handling of a device certificate stored in secure memory into dedicated helper functions. It does not change what data is stored or how it is protected; it only makes the code more maintainable. There is no indication this fixes a security bug.

Lower-priorityrefactor: Move HSM keys to common/hsm_keys.jsonby Andrew Kozlik · ebdb3d0b · Apr 24, 2026 · 7 filesMessage 57 · ThinInformational 18Details
Commit message · Andrew Kozlik

refactor: Move HSM keys to common/hsm_keys.json

[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 is a code-maintenance refactor, not a security fix. It moves public HSM (Hardware Security Module) and device-authenticity root keys from a C header file into a shared JSON file, then auto-generates both the C header and a Python file from that JSON. The actual key values do not change. A few related C files are updated to use a new feature flag (USE_MCU_ATTESTATION) and consistent size constants for the ML-DSA algorithm. There is no evidence in the commit of a vulnerability being patched.

Lower-priorityfeat(core): Add emulator ML-DSA certificates.by Andrew Kozlik · 1bce9735 · Apr 24, 2026 · 5 filesMessage 57 · ThinInformational 18Details
Commit message · Andrew Kozlik

feat(core): Add emulator ML-DSA certificates.

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 adds hardcoded test-only cryptographic credentials used by the Trezor emulator (software simulator) for a new ML-DSA device-attestation feature. It does not change real hardware firmware. The credentials are clearly emulator-only placeholders and are not intended for production devices.

Lower-prioritychore: Update mldsa-native submodule to v1.0.0-betaby Andrew Kozlik · 4f229cad · Apr 24, 2026 · 8 filesMessage 62 · AdequateLow 27Details
Commit message · Andrew Kozlik

chore: Update mldsa-native submodule to v1.0.0-beta

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

This commit updates the mldsa-native submodule to a new beta version and adjusts the build system and source code to use the new API names and file paths. ML-DSA is a post-quantum digital signature algorithm used for device attestation in production testing. The change itself is a routine dependency/API migration, but because it touches cryptographic verification and key generation code, it warrants a security-aware review. There is no direct evidence in the commit of a vulnerability being fixed.

Security candidatefix(core): Fix Optiga signature masking.by Andrew Kozlik · ee55f006 · Apr 24, 2026 · 1 fileMessage 62 · AdequateModerate 59Details
Commit message · Andrew Kozlik

fix(core): Fix Optiga signature masking.

62/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Names security-relevant behavior explicitly! No meaningful explanatory body
Why it was queued
signing boundary
AI analysis · Moderate 59/100

This commit fixes a bug in how the Trezor hardware wallet's secure chip (Optiga) handles masked digital signatures. The fix changes two things: it now uses the actual returned signature length rather than the maximum buffer size when parsing the signature, and it explicitly uses the NIST P-256 curve instead of a variable curve parameter when removing the masking. The bug could potentially cause signature parsing to fail or behave incorrectly, especially if the returned signature was shorter than the maximum expected size. There is no public statement linking this to an active security vulnerability or attack.

Security candidatefeat(core): Add MCU signature to AuthenticityProofby Andrew Kozlik · db430e92 · Apr 24, 2026 · 4 filesMessage 67 · AdequateInformational 19Details
Commit message · Andrew Kozlik

feat(core): Add MCU signature to AuthenticityProof

[no changelog]

67/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Names security-relevant behavior explicitly! No meaningful explanatory body
Why it was queued
signing boundary
AI analysis · Informational 19/100

This commit only adds two new optional data fields to an existing device-authentication proof message used in Trezor hardware wallets. It extends the protocol definition and the matching generated code in Python, Rust, and the core firmware so that future firmware can include an MCU (microcontroller) certificate and signature alongside the already-present Optiga and Tropic proofs. The change is additive and does not by itself alter any verification logic, cryptographic checks, or access controls visible in the diff.