SeedShield 0.2.3__tar.gz → 0.2.4__tar.gz

{seedshield-0.2.3/SeedShield.egg-info → seedshield-0.2.4}/PKG-INFO

@@ -1,8 +1,9 @@
 Metadata-Version: 2.4
 Name: SeedShield
-Version: 0.2.3
+Version: 0.2.4
 Summary: Secure BIP39 word viewer with masking and reveal functionality
 Author-email: Barlog951 <barlog951@gmail.com>
+License: MIT
 Project-URL: Documentation, https://github.com/Barlog951/SeedShield/blob/main/README.md
 Project-URL: Source Code, https://github.com/Barlog951/SeedShield
 Project-URL: Bug Tracker, https://github.com/Barlog951/SeedShield/issues
@@ -12,15 +13,13 @@ Classifier: Intended Audience :: End Users/Desktop
 Classifier: License :: OSI Approved :: MIT License
 Classifier: Operating System :: OS Independent
 Classifier: Programming Language :: Python :: 3
-Classifier: Programming Language :: Python :: 3.6
-Classifier: Programming Language :: Python :: 3.7
-Classifier: Programming Language :: Python :: 3.8
-Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
+Classifier: Programming Language :: Python :: 3.13
+Classifier: Programming Language :: Python :: 3.14
 Classifier: Topic :: Security
-Requires-Python: >=3.6
+Requires-Python: >=3.10
 Description-Content-Type: text/markdown
 License-File: LICENSE
 Requires-Dist: pyperclip>=1.8.0
@@ -36,6 +35,7 @@ Requires-Dist: windows-curses; extra == "windows"
 Provides-Extra: dev
 Requires-Dist: build; extra == "dev"
 Requires-Dist: twine; extra == "dev"
+Requires-Dist: black; extra == "dev"
 Dynamic: license-file
 
 # SeedShield

{seedshield-0.2.3 → seedshield-0.2.4/SeedShield.egg-info}/PKG-INFO

@@ -1,8 +1,9 @@
 Metadata-Version: 2.4
 Name: SeedShield
-Version: 0.2.3
+Version: 0.2.4
 Summary: Secure BIP39 word viewer with masking and reveal functionality
 Author-email: Barlog951 <barlog951@gmail.com>
+License: MIT
 Project-URL: Documentation, https://github.com/Barlog951/SeedShield/blob/main/README.md
 Project-URL: Source Code, https://github.com/Barlog951/SeedShield
 Project-URL: Bug Tracker, https://github.com/Barlog951/SeedShield/issues
@@ -12,15 +13,13 @@ Classifier: Intended Audience :: End Users/Desktop
 Classifier: License :: OSI Approved :: MIT License
 Classifier: Operating System :: OS Independent
 Classifier: Programming Language :: Python :: 3
-Classifier: Programming Language :: Python :: 3.6
-Classifier: Programming Language :: Python :: 3.7
-Classifier: Programming Language :: Python :: 3.8
-Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
+Classifier: Programming Language :: Python :: 3.13
+Classifier: Programming Language :: Python :: 3.14
 Classifier: Topic :: Security
-Requires-Python: >=3.6
+Requires-Python: >=3.10
 Description-Content-Type: text/markdown
 License-File: LICENSE
 Requires-Dist: pyperclip>=1.8.0
@@ -36,6 +35,7 @@ Requires-Dist: windows-curses; extra == "windows"
 Provides-Extra: dev
 Requires-Dist: build; extra == "dev"
 Requires-Dist: twine; extra == "dev"
+Requires-Dist: black; extra == "dev"
 Dynamic: license-file
 
 # SeedShield

{seedshield-0.2.3 → seedshield-0.2.4}/SeedShield.egg-info/SOURCES.txt

@@ -22,10 +22,12 @@ seedshield/ui_manager.py
 seedshield/data/english.txt
 tests/__init__.py
 tests/conftest.py
+tests/test_config.py
 tests/test_display_handler.py
 tests/test_fixtures.py
 tests/test_input_handler.py
 tests/test_main.py
+tests/test_secure_memory.py
 tests/test_secure_word_interface.py
 tests/test_state_handler.py
 tests/test_ui_manager.py

{seedshield-0.2.3 → seedshield-0.2.4}/SeedShield.egg-info/requires.txt

@@ -3,6 +3,7 @@ pyperclip>=1.8.0
 [dev]
 build
 twine
+black
 
 [test]
 pytest>=6.0

{seedshield-0.2.3 → seedshield-0.2.4}/pyproject.toml

@@ -4,13 +4,14 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "SeedShield"
-version = "0.2.3"
+dynamic = ["version"]
 readme = { file = "README.md", content-type = "text/markdown" }
 description = "Secure BIP39 word viewer with masking and reveal functionality"
+license = {text = "MIT"}
 authors = [
     {name = "Barlog951", email = "barlog951@gmail.com"}
 ]
-requires-python = ">=3.6"
+requires-python = ">=3.10"
 dependencies = [
     "pyperclip>=1.8.0",
 ]
@@ -21,13 +22,11 @@ classifiers = [
     "License :: OSI Approved :: MIT License",
     "Operating System :: OS Independent",
     "Programming Language :: Python :: 3",
-    "Programming Language :: Python :: 3.6",
-    "Programming Language :: Python :: 3.7",
-    "Programming Language :: Python :: 3.8",
-    "Programming Language :: Python :: 3.9",
     "Programming Language :: Python :: 3.10",
     "Programming Language :: Python :: 3.11",
     "Programming Language :: Python :: 3.12",
+    "Programming Language :: Python :: 3.13",
+    "Programming Language :: Python :: 3.14",
     "Topic :: Security",
 ]
 
@@ -48,15 +47,23 @@ test = [
 windows = ["windows-curses"]
 dev = [
     "build",
-    "twine"
+    "twine",
+    "black"
 ]
 
 [project.scripts]
 seedshield = "seedshield.main:main"
 
+[tool.setuptools.dynamic]
+# Single source of truth for the version; semantic-release updates config.py
+version = {attr = "seedshield.config.VERSION"}
+
 [tool.pytest.ini_options]
 testpaths = ["tests"]
+pythonpath = ["."]
 python_files = ["test_*.py"]
+python_classes = ["Test*"]
+python_functions = ["test_*"]
 markers = [
     "timeout: mark test with timeout value in seconds"
 ]
@@ -67,12 +74,9 @@ disable = [
     "C0103",  # invalid-name
 ]
 
-[tool.flake8]
-max-line-length = 100
-exclude = [".git", "__pycache__", "build", "dist"]
-
 [tool.mypy]
-python_version = "3.8"
+# Type-check against the supported floor
+python_version = "3.10"
 warn_return_any = true
 warn_unused_configs = true
-disallow_untyped_defs = true
+disallow_untyped_defs = true

{seedshield-0.2.3 → seedshield-0.2.4}/seedshield/__init__.py

@@ -19,14 +19,14 @@ __version__ = VERSION
 
 # Define available components
 __all__ = [
-    'SecureWordInterface',
-    'InputHandler',
-    'DisplayHandler',
-    'StateHandler',
-    'UIManager',
-    'secure_clear_string',
-    'secure_clear_list',
-    'secure_clipboard_clear',
-    'setup_logging',
-    'logger'
+    "SecureWordInterface",
+    "InputHandler",
+    "DisplayHandler",
+    "StateHandler",
+    "UIManager",
+    "secure_clear_string",
+    "secure_clear_list",
+    "secure_clipboard_clear",
+    "setup_logging",
+    "logger",
 ]

{seedshield-0.2.3 → seedshield-0.2.4}/seedshield/config.py

@@ -9,10 +9,11 @@ import os
 import logging
 import logging.handlers
 import sys
+from typing import Optional
 
 # Application constants
 APP_NAME = "SeedShield"
-VERSION = "0.2.3"
+VERSION = "0.2.4"
 
 # Security settings
 REVEAL_TIMEOUT = 3  # Seconds before auto-hiding revealed words
@@ -32,16 +33,22 @@ SCROLL_INDICATOR_DOWN = "↓ More ↓"
 MENU_TEXT = {
     "standard": "'n' - new input, 's' - show one by one, 'q' - quit, ↑↓ - scroll",
     "with_reset": "'n' - input, 's' - show one by one, 'r' - reset, 'q' - quit, ↑↓ - scroll",
-    "mouse_help": "Mouse over to reveal word"
+    "mouse_help": "Mouse over to reveal word",
 }
 
 
-def setup_logging(log_level: int = logging.INFO) -> logging.Logger:
+def setup_logging(
+    log_level: int = logging.WARNING, log_file: Optional[str] = None
+) -> logging.Logger:
     """
     Set up application logging with proper security measures.
 
+    File logging is opt-in: no usage trail is written to disk unless a log
+    file is explicitly requested (e.g. via the --verbose flag).
+
     Args:
-        log_level: Desired logging level (default: INFO)
+        log_level: Desired logging level (default: WARNING)
+        log_file: Optional path for file logging; None disables it
 
     Returns:
         logging.Logger: Configured logger instance
@@ -49,35 +56,38 @@ def setup_logging(log_level: int = logging.INFO) -> logging.Logger:
     log = logging.getLogger(APP_NAME)
     log.setLevel(log_level)
 
+    # Reconfiguring must not stack duplicate handlers
+    for handler in log.handlers[:]:
+        log.removeHandler(handler)
+        handler.close()
+
     # Console handler for error messages only
     console_handler = logging.StreamHandler(sys.stderr)
     console_handler.setLevel(logging.ERROR)
-    console_formatter = logging.Formatter('%(levelname)s: %(message)s')
+    console_formatter = logging.Formatter("%(levelname)s: %(message)s")
     console_handler.setFormatter(console_formatter)
 
-    # File handler with rotation to prevent excessive log growth
-    try:
-        file_handler = logging.handlers.RotatingFileHandler(
-            DEFAULT_LOG_PATH,
-            maxBytes=LOG_MAX_SIZE,
-            backupCount=LOG_BACKUP_COUNT
-        )
-        file_handler.setLevel(log_level)
-        file_formatter = logging.Formatter(
-            '%(asctime)s - %(name)s - %(levelname)s - %(message)s',
-            datefmt='%Y-%m-%d %H:%M:%S'
-        )
-        file_handler.setFormatter(file_formatter)
-
-        log.addHandler(file_handler)
-    except (PermissionError, IOError, OSError):
-        # Fall back to console-only logging if file logging fails
-        pass
+    # File handler with rotation, only when explicitly requested
+    if log_file is not None:
+        try:
+            file_handler = logging.handlers.RotatingFileHandler(
+                log_file, maxBytes=LOG_MAX_SIZE, backupCount=LOG_BACKUP_COUNT
+            )
+            file_handler.setLevel(log_level)
+            file_formatter = logging.Formatter(
+                "%(asctime)s - %(name)s - %(levelname)s - %(message)s", datefmt="%Y-%m-%d %H:%M:%S"
+            )
+            file_handler.setFormatter(file_formatter)
+
+            log.addHandler(file_handler)
+        except (PermissionError, IOError, OSError):
+            # Fall back to console-only logging if file logging fails
+            pass
 
     log.addHandler(console_handler)
 
     return log
 
 
-# Create the application logger
+# Create the application logger (console-only; file logging is opt-in)
 logger = setup_logging()

{seedshield-0.2.3 → seedshield-0.2.4}/seedshield/display_handler.py

@@ -6,6 +6,7 @@ ensuring proper masking and interaction.
 """
 
 import curses
+from dataclasses import dataclass
 from typing import List, Optional, Any
 
 from .config import logger, MASK_CHARACTER, MASK_LENGTH
@@ -13,6 +14,35 @@ from .config import SCROLL_INDICATOR_UP, SCROLL_INDICATOR_DOWN, MENU_TEXT
 from .state_handler import StateHandler
 
 
+@dataclass(frozen=True)
+class Viewport:
+    """Terminal viewport for one render pass; start doubles as scroll position."""
+
+    height: int
+    width: int
+    start: int
+    end: int
+
+
+@dataclass(frozen=True)
+class WordRow:
+    """A single word row ready for rendering."""
+
+    word: str
+    display_num: int
+    y_pos: int
+    revealed: bool
+
+
+@dataclass(frozen=True)
+class DisplayState:
+    """View state for one display pass."""
+
+    scroll: int
+    cursor: Optional[int]
+    reached_last: bool
+
+
 class DisplayHandler:
     """
     Handles display and UI rendering for the secure word interface.
@@ -35,7 +65,6 @@ class DisplayHandler:
         self.words = words
         self.ui_manager = ui_manager
         self.mask = MASK_CHARACTER * MASK_LENGTH
-        self.legacy_mask = self.mask  # For backward compatibility with tests
         self.state_handler: Optional[StateHandler] = None  # Will be set later
 
         # Store the last state to optimize rendering
@@ -44,40 +73,41 @@ class DisplayHandler:
 
         logger.debug("DisplayHandler initialized with %s words", len(words))
 
-    def _add_scroll_indicators(
-            self, stdscr: Any, visible_start: int, visible_end: int,
-            positions: List[int], height: int, width: int
-    ) -> None:
+    def _add_scroll_indicators(self, stdscr: Any, viewport: Viewport, total: int) -> None:
         """
         Add scroll indicators to the display if needed.
 
         Args:
             stdscr: Curses window objec
-            visible_start: Index of first visible word
-            visible_end: Index of last visible word
-            positions: List of word positions
-            height: Terminal heigh
-            width: Terminal width
+            viewport: Current terminal viewport
+            total: Total number of positions in the list
         """
         # Show up indicator if there are hidden items above
-        if visible_start > 0:
+        if viewport.start > 0:
             try:
-                stdscr.addstr(0, max(0, width - len(SCROLL_INDICATOR_UP) - 1), SCROLL_INDICATOR_UP)
-                logger.debug("Added up scroll indicator at position 0,%s",
-                           width - len(SCROLL_INDICATOR_UP) - 1)
+                stdscr.addstr(
+                    0, max(0, viewport.width - len(SCROLL_INDICATOR_UP) - 1), SCROLL_INDICATOR_UP
+                )
+                logger.debug(
+                    "Added up scroll indicator at position 0,%s",
+                    viewport.width - len(SCROLL_INDICATOR_UP) - 1,
+                )
             except curses.error:
                 pass
 
         # Show down indicator if there are hidden items below
-        if visible_end < len(positions):
+        if viewport.end < total:
             try:
                 stdscr.addstr(
-                    height - 7,
-                    max(0, width - len(SCROLL_INDICATOR_DOWN) - 1),
-                    SCROLL_INDICATOR_DOWN
+                    viewport.height - 7,
+                    max(0, viewport.width - len(SCROLL_INDICATOR_DOWN) - 1),
+                    SCROLL_INDICATOR_DOWN,
+                )
+                logger.debug(
+                    "Added down scroll indicator at position %s,%s",
+                    viewport.height - 7,
+                    viewport.width - len(SCROLL_INDICATOR_DOWN) - 1,
                 )
-                logger.debug("Added down scroll indicator at position %s,%s",
-                           height - 7, width - len(SCROLL_INDICATOR_DOWN) - 1)
             except curses.error:
                 pass
 
@@ -132,30 +162,26 @@ class DisplayHandler:
             except curses.error:
                 pass
 
-    def _render_word(self, stdscr: Any, word: str, display_num: int,
-                     y_pos: int, is_revealed: bool, width: int) -> None:
+    def _render_word(self, stdscr: Any, row: WordRow, width: int) -> None:
         """
         Render a single word with proper masking and formatting.
 
         Args:
             stdscr: Curses window objec
-            word: The word to display
-            display_num: The display number
-            y_pos: The y position to render a
-            is_revealed: Whether the word should be revealed
+            row: The word row to render
             width: Terminal width
         """
-        display_text = f"{display_num:2d}. {word if is_revealed else self.mask}"
+        display_text = f"{row.display_num:2d}. {row.word if row.revealed else self.mask}"
 
         # Ensure text fits within terminal width
         if len(display_text) >= width:
-            display_text = display_text[:width - 1]
+            display_text = display_text[: width - 1]
 
         # Highlight revealed word
-        if is_revealed:
-            stdscr.addstr(y_pos, 0, display_text, curses.A_BOLD)
+        if row.revealed:
+            stdscr.addstr(row.y_pos, 0, display_text, curses.A_BOLD)
         else:
-            stdscr.addstr(y_pos, 0, display_text)
+            stdscr.addstr(row.y_pos, 0, display_text)
 
     def _get_word_for_position(self, pos: int) -> str:
         """
@@ -170,22 +196,18 @@ class DisplayHandler:
         if 0 <= pos - 1 < len(self.words):
             return self.words[pos - 1]
 
-        logger.warning("Invalid word position: %s", pos)
+        # Never log the position value: positions encode the seed
+        logger.warning("Invalid word position requested")
         return f"INVALID({pos})"
 
-    def display_words(
-            self, stdscr: Any, positions: List[int], scroll_position: int,
-            cursor_pos: Optional[int], is_last_reached: bool
-    ) -> int:
+    def display_words(self, stdscr: Any, positions: List[int], state: DisplayState) -> int:
         """
         Display words with masking in the terminal interface.
 
         Args:
             stdscr: Curses window objec
             positions: List of word positions to display
-            scroll_position: Current scroll position
-            cursor_pos: Position of cursor (revealed word)
-            is_last_reached: Whether last word has been reached
+            state: Current view state (scroll, cursor, reached_last)
 
         Returns:
             int: Number of visible words that could fit in the current view
@@ -197,34 +219,33 @@ class DisplayHandler:
 
         # Calculate display metrics
         max_display_lines = max(1, height - 7)  # Reserve space for menu and indicators
-        visible_start = scroll_position
-        visible_end = min(len(positions), scroll_position + max_display_lines // 2)
+        viewport = Viewport(
+            height=height,
+            width=width,
+            start=state.scroll,
+            end=min(len(positions), state.scroll + max_display_lines // 2),
+        )
 
-        logger.debug("Displaying words %s-%s of %s", visible_start, visible_end, len(positions))
+        logger.debug("Displaying words %s-%s of %s", viewport.start, viewport.end, len(positions))
 
         # Clear screen for fresh rendering
         stdscr.clear()
 
         # Display words with proper masking
-        self._display_visible_words(stdscr, positions, visible_start, visible_end,
-                                   scroll_position, cursor_pos, height, width)
+        self._display_visible_words(stdscr, positions, viewport, state.cursor)
 
         # Add scroll indicators if needed
-        self._add_scroll_indicators(stdscr, visible_start, visible_end, positions,
-                                  height, width)
+        self._add_scroll_indicators(stdscr, viewport, len(positions))
 
         # Add command menu
-        self._add_menu(stdscr, height, is_last_reached)
+        self._add_menu(stdscr, height, state.reached_last)
 
         # Return number of visible words
-        result: int = visible_end - visible_start
+        result: int = viewport.end - viewport.start
         return result
 
     def _display_visible_words(
-            self, stdscr: Any, positions: List[int],
-            visible_start: int, visible_end: int,
-            scroll_position: int, cursor_pos: Optional[int],
-            height: int, width: int
+        self, stdscr: Any, positions: List[int], viewport: Viewport, cursor_pos: Optional[int]
     ) -> None:
         """
         Display the visible words on the screen.
@@ -232,31 +253,28 @@ class DisplayHandler:
         Args:
             stdscr: Curses window objec
             positions: List of word positions
-            visible_start: Index of first visible word
-            visible_end: Index of last visible word
-            scroll_position: Current scroll position
+            viewport: Current terminal viewport
             cursor_pos: Position of cursor (revealed word)
-            height: Terminal heigh
-            width: Terminal width
         """
-        for i, pos in enumerate(positions[visible_start:visible_end], visible_start):
+        for i, pos in enumerate(positions[viewport.start : viewport.end], viewport.start):
             try:
                 # Get word for the position
                 word = self._get_word_for_position(pos)
 
                 # Calculate position and check if it's visible
-                y_pos = i * 2 - scroll_position * 2
+                y_pos = i * 2 - viewport.start * 2
 
                 # Only render if within displayable area
-                if 0 <= y_pos < height - 6:
-                    display_num = i + 1
-                    is_revealed = cursor_pos == i
-                    self._render_word(stdscr, word, display_num, y_pos, is_revealed, width)
+                if 0 <= y_pos < viewport.height - 6:
+                    row = WordRow(
+                        word=word, display_num=i + 1, y_pos=y_pos, revealed=cursor_pos == i
+                    )
+                    self._render_word(stdscr, row, viewport.width)
 
             except curses.error:
                 # Handle rendering errors
                 logger.debug("Error displaying word at position %s", i)
-            except Exception as e:
+            except Exception as e:  # pylint: disable=broad-exception-caught
                 # Handle unexpected errors
                 logger.error("Unexpected error displaying word at position %s: %s", i, str(e))
 
@@ -306,6 +324,3 @@ class DisplayHandler:
 
         # Cursor is already visible
         return scroll_pos
-
-    # For backward compatibility with tests
-    handle_scroll = handle_autoscroll