Refactor constants initialization

src/constants.py

@@ -1,10 +1,7 @@
 # constants.py
 
-import os
 import logging
-import sys
 from pathlib import Path
-import traceback
 
 # Instantiate the logger
 logger = logging.getLogger(__name__)
@@ -15,38 +12,32 @@ logger = logging.getLogger(__name__)
 MAX_RETRIES = 3  # Maximum number of retries for relay connections
 RETRY_DELAY = 5  # Seconds to wait before retrying a failed connection
 
-try:
-    # -----------------------------------
-    # Application Directory and Paths
-    # -----------------------------------
-    APP_DIR = Path.home() / ".seedpass"
-    APP_DIR.mkdir(exist_ok=True, parents=True)  # Ensure the directory exists
-    if logger.isEnabledFor(logging.DEBUG):
-        logger.info(f"Application directory created at {APP_DIR}")
-except Exception as e:
-    if logger.isEnabledFor(logging.DEBUG):
-        logger.error(f"Failed to create application directory: {e}", exc_info=True)
-
-try:
-    PARENT_SEED_FILE = APP_DIR / "parent_seed.enc"  # Encrypted parent seed
-    if logger.isEnabledFor(logging.DEBUG):
-        logger.info(f"Parent seed file path set to {PARENT_SEED_FILE}")
-except Exception as e:
-    if logger.isEnabledFor(logging.DEBUG):
-        logger.error(f"Error setting file paths: {e}", exc_info=True)
+# -----------------------------------
+# Application Directory and Paths
+# -----------------------------------
+APP_DIR = Path.home() / ".seedpass"
+PARENT_SEED_FILE = APP_DIR / "parent_seed.enc"  # Encrypted parent seed
 
 # -----------------------------------
 # Checksum Files for Integrity
 # -----------------------------------
-try:
-    SCRIPT_CHECKSUM_FILE = (
-        APP_DIR / "seedpass_script_checksum.txt"
-    )  # Checksum for main script
-    if logger.isEnabledFor(logging.DEBUG):
-        logger.info(f"Checksum file path set: Script {SCRIPT_CHECKSUM_FILE}")
-except Exception as e:
-    if logger.isEnabledFor(logging.DEBUG):
-        logger.error(f"Error setting checksum file paths: {e}", exc_info=True)
+SCRIPT_CHECKSUM_FILE = (
+    APP_DIR / "seedpass_script_checksum.txt"
+)  # Checksum for main script
+
+
+def initialize_app() -> None:
+    """Ensure the application directory exists."""
+    try:
+        APP_DIR.mkdir(exist_ok=True, parents=True)
+        if logger.isEnabledFor(logging.DEBUG):
+            logger.info(f"Application directory created at {APP_DIR}")
+    except Exception as exc:
+        if logger.isEnabledFor(logging.DEBUG):
+            logger.error(
+                f"Failed to create application directory: {exc}", exc_info=True
+            )
+
 
 # -----------------------------------
 # Password Generation Constants

src/main.py

@@ -16,7 +16,7 @@ import traceback
 
 from password_manager.manager import PasswordManager
 from nostr.client import NostrClient
-from constants import INACTIVITY_TIMEOUT
+from constants import INACTIVITY_TIMEOUT, initialize_app
 from utils.password_prompt import PasswordPromptError
 from local_bip85.bip85 import Bip85Error
 
@@ -613,6 +613,7 @@ def display_menu(
 if __name__ == "__main__":
     # Configure logging with both file and console handlers
     configure_logging()
+    initialize_app()
     logger = logging.getLogger(__name__)
     logger.info("Starting SeedPass Password Manager")
 

src/password_manager/manager.py

@@ -47,6 +47,7 @@ from constants import (
     DEFAULT_PASSWORD_LENGTH,
     INACTIVITY_TIMEOUT,
     DEFAULT_SEED_BACKUP_FILENAME,
+    initialize_app,
 )
 
 import traceback
@@ -80,6 +81,7 @@ class PasswordManager:
 
     def __init__(self) -> None:
         """Initialize the PasswordManager."""
+        initialize_app()
         self.encryption_mode: EncryptionMode = EncryptionMode.SEED_ONLY
         self.encryption_manager: Optional[EncryptionManager] = None
         self.entry_manager: Optional[EntryManager] = None