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
2642commits · 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 33 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-priorityRevert "fix(tests): translate strings"by Ioan Bizău · 439950a7 · Sep 3, 2025 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · Ioan Bizău

Revert "fix(tests): translate strings"

This reverts commit baae34149b9a26af36855de8ba518b642185b6eb.

70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification
AI analysis · Informational 15/100

This commit is a simple revert of a previous test-only change. It modifies one Python test file to go back to an older way of looking up translated strings during automated device tests. There is no change to the actual Trezor firmware code that runs on the hardware, so this cannot affect real users or device security.

Lower-priorityfix(tests): translate stringsby Ioan Bizău · 2d27c4db · Sep 3, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · Ioan Bizău

fix(tests): translate strings

[no changelog]

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

This commit only updates a test file so that automated tests can correctly look up translated button labels for different device screen layouts. It does not change the actual Trezor firmware or any user-facing behavior, and it is not a security fix.

Security candidatefix(tests): visit multi-page messageby Ioan Bizău · a4f80401 · Sep 3, 2025 · 1 fileMessage 67 · AdequateInformational 13Details
Commit message · Ioan Bizău

fix(tests): visit multi-page message

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

This commit only changes a test file. It adds a helper function to scroll through multi-page on-screen messages during automated PIN and wipe-code setup tests. There is no change to the actual Trezor firmware or wallet behavior, so it does not create or fix a security vulnerability in the product users rely on.

AI review queuedfix(translations): shorten some stringsby Ioan Bizău · 50c8a7bf · Sep 3, 2025 · 4 filesMessage 57 · ThinInformational 15Details
Commit message · Ioan Bizău

fix(translations): shorten some 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 pathtranslation-only discountsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit only shortens and updates translated user-interface text in Spanish, French, and Portuguese translation files, plus refreshes the translation signature metadata. There are no code, logic, cryptographic, or security changes.

Lower-prioritychore(core): remove unused translation keysby obrusvit · cc82fd1d · Sep 3, 2025 · 6 filesMessage 72 · AdequateInformational 15Details
Commit message · obrusvit

chore(core): remove unused translation keys

- these keys stayed in in foreign lang JSONs by mistake, they are not
present in en.json

[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
translation-only discount
AI analysis · Informational 15/100

This commit simply removes a handful of unused translation text strings from non-English language files. There is no code change, no security fix, and no user-facing behavior change. It is a routine cleanup.

Lower-priorityfix(tests): ignore spaces on paginated contentby Ioan Bizău · 81f58e7c · Sep 3, 2025 · 1 fileMessage 67 · AdequateInformational 15Details
Commit message · Ioan Bizău

fix(tests): ignore spaces on paginated content

[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 is a tiny change to a single test file. It makes a test more tolerant of how spaces appear in on-screen text by ignoring spaces when comparing expected and actual text. There is no change to the actual Trezor firmware or wallet behavior, and no security issue is present.

Lower-prioritychore: update fixturesby obrusvit · bb3fc738 · Sep 3, 2025 · 1 fileMessage 40 · ThinInformational 15Details
Commit message · obrusvit

chore: update fixtures

[no changelog]

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

This commit only updates the expected visual test snapshots (called 'fixtures') used by Trezor's automated UI testing suite. It does not change any firmware, application, or cryptographic code. The hash values in the fixtures file changed because the on-screen text, layout, or rendering was updated in a previous code change, and now the test expectations are being refreshed to match. There is no direct security relevance in this commit itself.

Lower-priorityfix(tests): translate stringsby obrusvit · e1bfe039 · Sep 3, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · obrusvit

fix(tests): translate strings

[no changelog]

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

This commit only changes a test file. It updates helper functions in the language-related device tests to use a new shared translation lookup helper instead of reading language JSON files directly. There is no change to the actual Trezor firmware, wallet application, or any code that runs on the device. It is purely a test-code cleanup and does not affect user security.

AI review queuedchore(core): pull Crowdin translationsby obrusvit · 3d267500 · Sep 3, 2025 · 10 filesMessage 57 · ThinInformational 15Details
Commit message · obrusvit

chore(core): pull Crowdin translations

[no changelog]

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

This commit is a routine update of user-facing text translations for the Trezor firmware. It changes wording in multiple languages, adds new translation keys for upcoming features (such as Bluetooth and new device screens), and replaces a non-breaking space with a regular space in a few English strings. There is no code logic change and no security-relevant behavior is altered.

Lower-priorityfix(common): require fields on SLIP-24 Text Details memoby matejcik · aea4e4a2 · Sep 3, 2025 · 4 filesMessage 62 · AdequateLow 27Details
Commit message · matejcik

fix(common): require fields on SLIP-24 Text Details memo

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 tightens the rules for a Trezor message type called TextDetailsMemo, which is part of the SLIP-24 payment-request memo format. Previously the title and text fields were optional and defaulted to empty strings; now both are required. The change is defensive: it prevents a device from accepting a payment-request memo that omits the title or body text. Without this, a malicious or buggy host could send a payment request with an empty/incomplete memo and still have it treated as a valid TextDetailsMemo, potentially misleading the user about what they are approving.

Lower-prioritybuild: add T3W1 model to build-docker.shby obrusvit · 525b9ec3 · Sep 2, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · obrusvit

build: add T3W1 model to build-docker.sh

[no changelog]

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

This commit simply adds a new hardware model code (T3W1) to the list of default models in a build helper script. It changes two lines: the help text and the default model array. There is no security-relevant change visible in the diff.

Lower-priorityfix(nordic): update mcuboot with a fix to PRODUCTION=1 buildby matejcik · 723a4d29 · Sep 2, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · matejcik

fix(nordic): update mcuboot with a fix to PRODUCTION=1 build

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 simply updates the version (revision) of an embedded bootloader component called MCUboot used in Trezor's Nordic-based products. The change is a one-line pointer update in a project manifest file, moving from one tagged internal revision to another. There is no description of a security problem, no patch to Trezor's own code, and no supplied references indicating this is a security fix.

Security candidatefeat(core): implement fingerprint logic for BootloaderV2 imageby matejcik · 1249dbf6 · Sep 2, 2025 · 1 fileMessage 62 · AdequateInformational 17Details
Commit message · matejcik

feat(core): implement fingerprint logic for BootloaderV2 image

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

This commit updates a small Python helper script used to display a fingerprint (a kind of checksum) for Trezor firmware files. It adds support for a newer bootloader image format (BootloaderV2) so the tool can handle both old and new formats. There is no direct evidence this change fixes a security vulnerability; it appears to be a feature/robustness improvement for a developer tool.

Security candidatebuild(core): firmware_fingerprint.py prints fingerprint of secmon in prodtest caseby matejcik · 5af0f0e4 · Sep 2, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · matejcik

build(core): firmware_fingerprint.py prints fingerprint of secmon in prodtest case

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

This is a small build-tool change that makes a helper script print the correct cryptographic fingerprint for a special 'prodtest' firmware image. It does not change any code that runs on the Trezor device itself, does not introduce any security vulnerability, and is not described by the vendor as a security fix.

Security candidatechore(core): derive `Debug` only for testsby Roman Zeyde · 4c34bb22 · Sep 2, 2025 · 7 filesMessage 67 · AdequateInformational 15Details
Commit message · Roman Zeyde

chore(core): derive `Debug` only for 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
cryptography-sensitive path
AI analysis · Informational 15/100

This commit removes automatic Debug printing support from several Rust data structures in the Trezor firmware's production builds, keeping it only for unit tests. It is a code hygiene or size-reduction change with no visible security relevance.

Lower-prioritychore(core): remove unused import in `base64.rs`by Roman Zeyde · 356dc984 · Sep 2, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · Roman Zeyde

chore(core): remove unused import in `base64.rs`

[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 removes an unused import (`use std::vec::Vec;`) from a Rust test module in the Trezor firmware codebase. It is a routine code cleanup with no functional or security effect.

Lower-priorityrefactor(core/prodtest): Expose prodtest_tropic_factory_session_start().by Andrew Kozlik · 371a057e · Sep 2, 2025 · 2 filesMessage 62 · AdequateInformational 12Details
Commit message · Andrew Kozlik

refactor(core/prodtest): Expose prodtest_tropic_factory_session_start().

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

This commit is a code cleanup in Trezor's internal production-test tooling. It moves an existing factory-session startup routine into its own reusable function and exposes it through a new header file. No new behavior is added, no new keys are introduced, and no user-facing or wallet security feature is changed. The production factory pairing key remains compiled into the prodtest binary, which is unchanged from before this commit.

Lower-priorityfeat(core): Use randomness from Tropic to generate secrets.by Andrew Kozlik · ecf8cde0 · Sep 2, 2025 · 1 fileMessage 67 · AdequateLow 32Details
Commit message · Andrew Kozlik

feat(core): Use randomness from Tropic to generate secrets.

[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
AI analysis · Low 32/100

This commit changes how secret keys are generated during Trezor device production testing. Previously, randomness came from the device's own RNG and (on some models) an Infineon OPTIGA chip. The patch adds support for also mixing in randomness from a new 'Tropic' secure chip, and ensures a factory secure session with Tropic is started before secrets are generated. This is a feature addition, not a vulnerability fix. It improves entropy quality but introduces a new hardware dependency and a TODO-removal that should be reviewed for correctness.

Lower-priorityfeat(core/prodtest): Allocate larger stack for ML-DSA.by Andrew Kozlik · 71d0d534 · Sep 2, 2025 · 1 fileMessage 62 · AdequateLow 26Details
Commit message · Andrew Kozlik

feat(core/prodtest): Allocate larger stack for ML-DSA.

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

This commit increases the memory reserved for the production-test firmware's call stack from 24 KB to 128 KB and moves it from MAIN_RAM to AUX1_RAM. The stated reason is to support ML-DSA (a post-quantum digital-signature algorithm) operations that need more stack space. A stack that is too small can trigger a UsageFault crash; the change prevents such crashes during production testing. There is no direct evidence in the commit that this fixes an exploitable security vulnerability, but running with an undersized stack can in some cases lead to memory corruption or unstable behavior.

Security candidatefeat(core): Use ML-DSA-44 for MCU device attestation key.by Andrew Kozlik · ba51fa46 · Sep 2, 2025 · 12 filesMessage 62 · AdequateLow 25Details
Commit message · Andrew Kozlik

feat(core): Use ML-DSA-44 for MCU device attestation key.

[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 trust
AI analysis · Low 25/100

This commit swaps the cryptographic algorithm used for one of the Trezor hardware wallet's internal device-attestation keys from Ed25519 to ML-DSA-44, a post-quantum NIST-standardized signature scheme. It also enlarges the certificate storage area and doubles the production-test command-line input buffer. There is no direct evidence in the commit of a security vulnerability being fixed; it reads as a planned cryptographic modernization change.

Security candidatefeat(core/prodtest): set HSM public keysby Ondřej Vejpustek · 58245dd5 · Sep 2, 2025 · 3 filesMessage 57 · ThinLow 25Details
Commit message · Ondřej Vejpustek

feat(core/prodtest): set HSM public keys

[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
cryptography-sensitive path
AI analysis · Low 25/100

This commit updates Trezor's production-test firmware to accept multiple trusted public keys from secure hardware modules (HSMs) instead of just one. It also adds a helper function in the Noise crypto library that tries each trusted key in turn during a secure handshake. The change itself is a feature addition, not an obvious bug fix. However, the helper function restores a sensitive temporary key from a backup on every retry, which is good practice, and it wipes the backup when done. There is no direct evidence in the commit that this fixes a security vulnerability; it looks like operational hardening for factory testing.

Lower-prioritybuild: add nRF build to build-docker.shby obrusvit · b39ddff4 · Sep 2, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · obrusvit

build: add nRF build to build-docker.sh

[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 is a build-script change that adds an optional way to compile firmware for a new Trezor Bluetooth chip (nRF) inside the existing Docker build tool. It does not change any wallet code, cryptography, or device behavior. There is no security issue visible in the diff.

Lower-prioritybuild: adjust nix and poetry dependencies for nRFby obrusvit · a39f4ade · Sep 2, 2025 · 3 filesMessage 80 · StrongInformational 18Details
Commit message · obrusvit

build: adjust nix and poetry dependencies for nRF

- exception is noiseprotocol which was in the poetry.lock but not in
pyproject.toml, seems like a mistake from before?

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

This commit updates the project's build dependency files (poetry.lock, pyproject.toml, shell.nix) to add tools needed for nRF microcontroller work. It is a routine build-environment change, not a fix for a security vulnerability. The only security-relevant note is that it pins several new Python packages, which is good practice, and it removes one unused tool (nrf-command-line-tools) from the development shell.

AI review queuedbuild: detect env in nRF build scriptby obrusvit · f8a1787f · Sep 2, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · obrusvit

build: detect env in nRF build script

[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 changes a build script for Trezor's Nordic (nRF) hardware so it can automatically detect whether it is running inside a Docker/Nix build environment or a local developer's machine, and then run the build command the appropriate way. It is a build-system convenience change with no visible security relevance.

Lower-priorityrefactor(core): simplify `_get_least_recently_used_item()` typingby Roman Zeyde · 3643722d · Sep 2, 2025 · 1 fileMessage 97 · StrongInformational 15Details
Commit message · Roman Zeyde

refactor(core): simplify `_get_least_recently_used_item()` typing

Since `from typing import` are "commented out" by `generate_frozen_module()`,
we need to add a "trailing" `pass` - otherwise the resulting Python code will
be invalid, and frozen builds will fail.

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

This is a minor code cleanup that changes how a helper function's input type is described. It makes no functional change to what the code does and does not affect security.