feat: add log_show_mode parameter to control log display (#3)

* feat: add `log_show_mode` parameter to control log display

* refactor: refactor test cases

* chore: update the ruff configuration file

* style: fix lint and format

Author: insistence

py_node_manager/__init__.py

@@ -1,7 +1,6 @@
 from .logger import get_logger
 from .manager import NodeManager
 
-
 __version__ = '0.1.1'
 
 

py_node_manager/manager.py

@@ -5,9 +5,9 @@
 import tarfile
 import urllib.request
 import zipfile
-from typing import Dict, Optional, Tuple
-from .logger import get_logger
+from typing import Dict, Literal, Optional, Tuple
 
+from .logger import get_logger
 
 logger = get_logger(logging.getLogger(__name__))
 
@@ -17,7 +17,13 @@ class NodeManager:
     Node.js manager class
     """
 
-    def __init__(self, download_node: bool, node_version: str, is_cli: bool = False):
+    def __init__(
+        self,
+        download_node: bool,
+        node_version: str,
+        is_cli: bool = False,
+        log_show_mode: Literal['all', 'slim', 'hide'] = 'all',
+    ):
         """
         Node.js manager class
 
@@ -29,6 +35,7 @@ def __init__(self, download_node: bool, node_version: str, is_cli: bool = False)
         self.download_node = download_node
         self.node_version = node_version
         self.is_cli = is_cli
+        self.log_show_mode = log_show_mode
         self.node_path = self._node_path()
         self.node_env = self._node_env()
         self.npm_path = self._npm_path()
@@ -86,7 +93,7 @@ def download_nodejs(self) -> str:
 
         # Create directory for downloaded Node.js within the package directory
         # Use the package directory instead of current working directory
-        # Get the directory of this utils.py file
+        # Get the directory of this file
         package_dir = os.path.dirname(os.path.abspath(__file__))
         node_dir_path = os.path.join(package_dir, '.nodejs_cache')
         if not os.path.exists(node_dir_path):
@@ -100,18 +107,20 @@ def download_nodejs(self) -> str:
 
         # If Node.js already exists, return the path without downloading
         if os.path.exists(node_executable):
-            logger.info(f'📦 Using cached Node.js from {node_executable}')
+            if self.log_show_mode in {'all', 'slim'}:
+                logger.info(f'📦 Using cached Node.js from {node_executable}')
             return node_executable
 
         # Download Node.js
         node_archive = os.path.join(node_dir_path, os.path.basename(node_url))
-        logger.info('🌐 Node.js not found in PATH. Downloading Node.js...')
-        if self.is_cli:
+        if self.log_show_mode in {'all', 'slim'}:
+            logger.info('🌐 Node.js not found in PATH. Downloading Node.js...')
+        if self.log_show_mode == 'all' and self.is_cli:
             logger.info(f'📥 Downloading Node.js from {node_url}...')
         urllib.request.urlretrieve(node_url, node_archive)
 
         # Extract Node.js
-        if self.is_cli:
+        if self.log_show_mode == 'all' and self.is_cli:
             logger.info('🔧 Extracting Node.js...')
         if node_archive.endswith('.tar.gz'):
             with tarfile.open(node_archive, 'r:gz') as tar:
@@ -130,7 +139,8 @@ def download_nodejs(self) -> str:
         if system != 'windows':
             os.chmod(node_executable, 0o755)
 
-        logger.info(f'✅ Node.js downloaded and extracted to {node_executable}')
+        if self.log_show_mode in {'all', 'slim'}:
+            logger.info(f'✅ Node.js downloaded and extracted to {node_executable}')
         return node_executable
 
     def check_or_download_nodejs(self) -> Optional[str]:
@@ -143,7 +153,8 @@ def check_or_download_nodejs(self) -> Optional[str]:
         # First check if Node.js is available in PATH
         is_available, version = self.check_nodejs_available()
         if is_available:
-            logger.info(f'💻 Using System Default Node.js {version}')
+            if self.log_show_mode in {'all', 'slim'}:
+                logger.info(f'💻 Using System Default Node.js {version}')
 
             return None  # Use system Node.js
 

ruff.toml

@@ -1,6 +1,19 @@
+line-length = 120
+show-fixes = true
 target-version = "py38"
 
-line-length = 120
+[lint]
+select = [
+    "I",      # isort
+    "E",      # pycodestyle errors
+    "W",      # pycodestyle warnings
+    "F",      # pyflakes
+]
+ignore = [
+    "E501",   # line too long
+    "W191",   # indentation contains tabs
+]
 
 [format]
+docstring-code-format = true
 quote-style = "single"