From acc4664409d7a736cf00adaf489647314bb3b3a2 Mon Sep 17 00:00:00 2001
From: Misaka_Company <pengqiangqiang0@gmail.com>
Date: Fri, 27 Mar 2026 12:45:04 +0800
Subject: [PATCH] refactor: remove close_session() function

- Remove close_session() wrapper function entirely
- Callers now directly manage browser/context lifecycle
- Reduces module to pure authentication functions only

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 utils/auth.py | 18 ------------------
 1 file changed, 18 deletions(-)

diff --git a/utils/auth.py b/utils/auth.py
index c5e0efd..14f8ee2 100644
--- a/utils/auth.py
+++ b/utils/auth.py
@@ -124,21 +124,3 @@ def logout(main_frame: Frame, verbose: bool = True) -> None:
             print("Confirmation dialog not found, may have auto-logged out")
 
     time.sleep(2)  # Wait for logout to complete
-
-
-def close_session(browser: Browser, context: BrowserContext) -> None:
-    """
-    Close browser session
-
-    Args:
-        browser: Browser instance
-        context: Browser context
-    """
-    # Check if auto-close browser is enabled
-    auto_close = os.getenv("ERP_AUTO_CLOSE_BROWSER", "true").lower() in ("true", "1", "yes")
-
-    if auto_close:
-        context.close()
-        browser.close()
-    else:
-        print("Note: Browser not auto-closed (ERP_AUTO_CLOSE_BROWSER=false)")