Restore whitespace alignment formatting

SeedSigner/SeedSigner

b447edd6 View source ↗

Public commit record

What the developer wrote

Authored by kdmukai

35/100 · Opaque

Restore whitespace alignment formatting

✓ Descriptive subject! No meaningful explanatory body

The short version

What changed, and why it matters

This commit only adjusts whitespace in a test screenshot generator file so that the opening parentheses line up visually. No code behavior, logic, or security-sensitive content changed.

Recommended action

No security action needed; this is a cosmetic formatting commit.

Security signals we found

No strong security signals were identified.

Technical analysis

Evidence from the diff

The diff is a pure formatting change inside tests/screenshot_generator/generator.py. It adds extra spaces between ScreenshotConfig constructor arguments to align the dict(…) columns. No functional code, cryptographic handling, user input parsing, or runtime behavior is modified.

Changed components

tests/screenshot_generator/generator.py

Inspect captured patch +7 / −7

diff --git a/tests/screenshot_generator/generator.py b/tests/screenshot_generator/generator.py
index d5345ba..97229b7 100644
--- a/tests/screenshot_generator/generator.py
+++ b/tests/screenshot_generator/generator.py
@@ -351,13 +351,13 @@ def generate_screenshots(locale):
                 
                 ScreenshotConfig(seed_views.SeedOptionsView, dict(seed_num=0)),
                 ScreenshotConfig(seed_views.SeedBackupView,  dict(seed_num=0)),
-                ScreenshotConfig(seed_views.SeedExportXpubSigTypeView,    dict(seed_num=0)),
-                ScreenshotConfig(seed_views.SeedExportXpubScriptTypeView, dict(seed_num=0, sig_type="msig")),
-                ScreenshotConfig(seed_views.SeedExportXpubCustomDerivationView, dict(seed_num=0, sig_type="ss", script_type="")),
-                ScreenshotConfig(seed_views.SeedExportXpubQRFormatView, dict(seed_num=0, sig_type="ss", script_type="nat")),
-                ScreenshotConfig(seed_views.SeedExportXpubWarningView, dict(seed_num=0, sig_type="msig", script_type="nes", xpub_qr_format="urca", custom_derivation="")),
-                ScreenshotConfig(seed_views.SeedExportXpubDetailsView, dict(seed_num=0, sig_type="ss", script_type="nat", xpub_qr_format="urca", custom_derivation="")),
-                ScreenshotConfig(SeedExportXpubQR_ScreenBrightnessView, dict(seed_num=0, xpub_qr_format="urca", derivation_path="m/84'/0'/0'")),
+                ScreenshotConfig(seed_views.SeedExportXpubSigTypeView,          dict(seed_num=0)),
+                ScreenshotConfig(seed_views.SeedExportXpubScriptTypeView,       dict(seed_num=0, sig_type="msig")),
+                ScreenshotConfig(seed_views.SeedExportXpubCustomDerivationView, dict(seed_num=0, sig_type="ss",   script_type="")),
+                ScreenshotConfig(seed_views.SeedExportXpubQRFormatView,         dict(seed_num=0, sig_type="ss",   script_type="nat")),
+                ScreenshotConfig(seed_views.SeedExportXpubWarningView,          dict(seed_num=0, sig_type="msig", script_type="nes", xpub_qr_format="urca", custom_derivation="")),
+                ScreenshotConfig(seed_views.SeedExportXpubDetailsView,          dict(seed_num=0, sig_type="ss",   script_type="nat", xpub_qr_format="urca", custom_derivation="")),
+                ScreenshotConfig(SeedExportXpubQR_ScreenBrightnessView,         dict(seed_num=0, xpub_qr_format="urca", derivation_path="m/84'/0'/0'")),
 
                 ScreenshotConfig(seed_views.SeedWordsWarningView, dict(seed_num=0)),
                 ScreenshotConfig(seed_views.SeedWordsView, dict(seed_num=0)),

Risk score

Why this scored 15/100

Our methodology →

Potential impact 0/30

Exploitability 0/25

Stealth signal 0/15

Affected reach 0/15

Confidence 10/10

Evidence quality 5/5

Human-validated context

Community notes

Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.

No validated notes yet.

The AI analysis stands alone for now. Submit a note if you can add evidence or important context.

What the developer wrote

What changed, and why it matters

Security signals we found

Evidence from the diff

Changed components

Why this scored 15/100

Community notes

Related watches

Merge pull request #722 from Chaitanya-Keyal/psbt-huge-fee-warning

Settle is_high_fee once in parse()

Merge pull request #1032 from kdmukai/psbt_output_ownership