What the developer wrote
Authored by kdmukai
Remove dead code
What changed, and why it matters
This commit removes unused code from a single screen file. One line that set a scroll position variable was never referenced anywhere, and an entire 'Powering Off' screen class was defined but apparently not used. There is no security-relevant change here—just cleanup.
No action needed. This is a routine code-cleanup commit with no security implications.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff deletes one unused assignment to self.cur_scroll_y in ButtonListScreen and removes the PowerOffScreen dataclass entirely. No functional logic, input handling, cryptography, or network code is modified. The removed code appears to be genuinely dead/unreferenced.
Changed components
src/seedsigner/gui/screens/screen.pyInspect captured patch +0 / −16
diff --git a/src/seedsigner/gui/screens/screen.py b/src/seedsigner/gui/screens/screen.py
index 0a159a3..4278cb9 100644
--- a/src/seedsigner/gui/screens/screen.py
+++ b/src/seedsigner/gui/screens/screen.py
@@ -371,7 +371,6 @@ class ButtonListScreen(BaseTopNavScreen):
if self.has_scroll_arrows:
self.arrow_half_width = 10
- self.cur_scroll_y = self.scroll_y_initial_offset if self.scroll_y_initial_offset is not None else 0
self.up_arrow_img = Image.new("RGBA", size=(2 * self.arrow_half_width, 8), color="black")
self.up_arrow_img_y = self.top_nav.height - 12
arrow_draw = ImageDraw.Draw(self.up_arrow_img)
@@ -1072,21 +1071,6 @@ class ResetScreen(BaseTopNavScreen):
-@dataclass
-class PowerOffScreen(BaseTopNavScreen):
- def __post_init__(self):
- self.title = _("Powering Off")
- self.show_back_button = False
- super().__post_init__()
-
- self.components.append(TextArea(
- text=_("Please wait about 30 seconds before disconnecting power."),
- screen_y=self.top_nav.height,
- height=self.canvas_height - self.top_nav.height,
- ))
-
-
-
@dataclass
class PowerOffNotRequiredScreen(BaseTopNavScreen):
def __post_init__(self):
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.