Update src/services/utils.py

src/services/utils.py

@@ -13,7 +13,7 @@ from ..state import get_user_temp_dir
 # --- Shared Utilities ---
 
 def create_session(retries: int = 3, backoff_factor: float = 0.5, status_forcelist=(429, 500, 502, 503, 504)) -> requests.Session:
-    """Configures a requests session with automatic retry logic for resilience."""
+    # Initialize requests session with exponential backoff and retry logic
     session = requests.Session()
     retry = Retry(
         total=retries,
@@ -29,7 +29,7 @@ def create_session(retries: int = 3, backoff_factor: float = 0.5, status_forceli
 SESSION = create_session()
 
 def short_num(n: float | int) -> str:
-    """Formats large numbers into readable strings (e.g., 1.5B, 200M)."""
+    # Scale large integers into abbreviated strings (K, M, B)
     try:
         n = float(n)
     except Exception:
@@ -43,34 +43,29 @@ def short_num(n: float | int) -> str:
     return str(round(n))
 
 def now_str(fmt: str = "%d-%m-%Y %H:%M:%S") -> str:
+    # Return current local system time in specified format
     return datetime.datetime.now().strftime(fmt)
 
 # --- PDF Generation ---
 
 def convert_html_to_pdf(html_content: str, user_id: str) -> Optional[Path]:
-    """
-    Uses Playwright (Local Chromium) to convert HTML to PDF.
-    Renders exactly as defined in CSS (Scale 1.0) on US Letter paper.
-    """
+    # Render HTML string to PDF file using headless Chromium
     print("\n   Converting to PDF (Using Playwright)...")
     timestamp = datetime.datetime.now().strftime("%Y-%m-%d_%H-%M-%S")
     pdf_name = f"{timestamp}-crypto-analysis.pdf"
     
-    # Use user isolated directory
     user_dir = get_user_temp_dir(user_id)
     pdf_path = user_dir / pdf_name
         
     try:
         with sync_playwright() as p:
-            # 1. Launch Browser
             browser = p.chromium.launch()
             page = browser.new_page()
             
-            # 2. Load HTML
-            # wait_until="load" is safer to ensure styles load
+            # Ensure styles/assets are fully loaded before printing
             page.set_content(html_content, wait_until="load")
             
-            # 3. Print to PDF (US Letter, No Auto-Scaling)
+            # Export to US Letter with full background rendering and 0 margins
             page.pdf(
                 path=pdf_path,
                 format="Letter",       
@@ -94,7 +89,7 @@ def convert_html_to_pdf(html_content: str, user_id: str) -> Optional[Path]:
 # --- File Cleanup ---
 
 def cleanup_after_analysis(spot_file: Optional[Path], futures_file: Optional[Path]) -> int:
-    """Removes source files (CSV/PDF) after successful analysis to keep the temp dir clean."""
+    # Delete temporary analysis files if they were modified on the current date
     files_cleaned = 0
     now_date = datetime.datetime.now().date()