perf: optimize exception capture

Author: insistence

dash_tailwindcss_plugin/cli.py

@@ -124,14 +124,11 @@ def init_tailwindcss(self, input_css_path: str, config_js_path: str):
         Returns:
             None
         """
-        try:
-            self.tailwind_command.init().install()
-            logger.info('📝 Next steps:')
-            logger.info('1. Customize your config file if needed')
-            logger.info('2. Build CSS with:')
-            logger.info('dash-tailwindcss-plugin build')
-        except Exception as e:
-            logger.error(f'Error initializing Tailwind CSS: {e}')
+        self.tailwind_command.init().install()
+        logger.info('📝 Next steps:')
+        logger.info('1. Customize your config file if needed')
+        logger.info('2. Build CSS with:')
+        logger.info('dash-tailwindcss-plugin build')
 
     def build_tailwindcss(self, clean_after: bool):
         """
@@ -143,13 +140,10 @@ def build_tailwindcss(self, clean_after: bool):
         Returns:
             None
         """
-        try:
-            built = self.tailwind_command.init().install().build()
-            # Clean up if requested
-            if clean_after:
-                built.clean()
-        except Exception as e:
-            logger.error(f'Error: {e}')
+        built = self.tailwind_command.init().install().build()
+        # Clean up if requested
+        if clean_after:
+            built.clean()
 
     def watch_tailwindcss(self):
         """
@@ -158,11 +152,7 @@ def watch_tailwindcss(self):
         Returns:
             None
         """
-        try:
-            self.tailwind_command.init().install().watch()
-
-        except Exception as e:
-            logger.error(f'Error watching Tailwind CSS: {e}')
+        self.tailwind_command.init().install().watch()
 
     def clean_tailwindcss(self):
         """
@@ -171,10 +161,7 @@ def clean_tailwindcss(self):
         Returns:
             None
         """
-        try:
-            self.tailwind_command.clean()
-        except Exception as e:
-            logger.error(f'Error cleaning Tailwind CSS: {e}')
+        self.tailwind_command.clean()
 
 
 def main():

dash_tailwindcss_plugin/plugin.py

@@ -195,12 +195,9 @@ def _build_tailwindcss(self):
         Returns:
             None
         """
-        try:
-            built = self.tailwind_command.init().install().build()
-            if self.clean_after:
-                built.clean()
-        except Exception as e:
-            logger.error(f'❌ Failed to build Tailwind CSS: {e}')
+        built = self.tailwind_command.init().install().build()
+        if self.clean_after:
+            built.clean()
 
 
 def setup_tailwindcss_plugin(

dash_tailwindcss_plugin/utils.py

@@ -472,9 +472,9 @@ def _check_tailwindcss(self) -> bool:
         Returns:
             bool: True if Tailwind CSS is installed, False otherwise
         """
-        cmd = [self.npx_path, f'{self._tailwind_cli} --help']
+        check_cmd = [self.npx_path, f'{self._tailwind_cli} --help']
 
-        result = subprocess.run(cmd, capture_output=True, text=True, env=self.node_env)
+        result = subprocess.run(check_cmd, capture_output=True, text=True, env=self.node_env)
         return result.returncode == 0
 
     def init(self) -> Self:
@@ -518,13 +518,13 @@ def init(self) -> Self:
                 init_cmd = [self.npm_path, 'init', '-y']
                 result = subprocess.run(init_cmd, capture_output=True, text=True, env=self.node_env)
                 if result.returncode != 0:
-                    logger.error(f'❌ Error initializing Tailwind CSS: {result.stderr}')
-                    return self
+                    raise RuntimeError(result.stderr)
 
             logger.info('✅ Tailwind CSS initialized successfully!')
 
         except Exception as e:
             logger.error(f'❌ Error initializing Tailwind CSS: {e}')
+            raise e
 
         return self
 
@@ -546,13 +546,13 @@ def install(self) -> Self:
                 ]
                 result = subprocess.run(install_cmd, capture_output=True, text=True, env=self.node_env)
                 if result.returncode != 0:
-                    logger.error(f'❌ Error installing Tailwind CSS: {result.stderr}')
-                    return self
+                    raise RuntimeError(result.stderr)
 
             logger.info('✅ Tailwind CSS installed successfully!')
 
         except Exception as e:
             logger.error(f'❌ Error installing Tailwind CSS: {e}')
+            raise e
 
         return self
 
@@ -579,14 +579,14 @@ def build(self) -> Self:
             result = subprocess.run(build_cmd, capture_output=True, text=True, env=self.node_env)
 
             if result.returncode != 0:
-                logger.error(f'❌ Error building Tailwind CSS: {result.stderr}')
-                return self
+                raise RuntimeError(result.stderr)
 
             logger.info('✅ Build completed successfully!')
             logger.info(f'🎨 Tailwind CSS built successfully to {self.output_css_path}')
 
         except Exception as e:
             logger.error(f'❌ Error building Tailwind CSS: {e}')
+            raise e
 
         return self
 
@@ -617,6 +617,7 @@ def watch(self) -> Self:
 
         except Exception as e:
             logger.error(f'❌ Error watching for changes: {e}')
+            raise e
 
         return self
 
@@ -628,35 +629,40 @@ def clean(self) -> Self:
             Self: The TailwindCommand instance
         """
         logger.info('🧹 Cleaning up generated files...')
-        files_to_remove = [
-            self.config_js_path,
-            'package.json',
-            'package-lock.json',
-            self.input_css_path,
-        ]
-
-        directories_to_remove = ['node_modules']
-
-        # Remove files
-        for file_path in files_to_remove:
-            if os.path.exists(file_path):
-                try:
-                    os.remove(file_path)
-                    if self.is_cli:
-                        logger.info(f'🗑️ Removed {file_path}')
-                except Exception as e:
-                    logger.warning(f'⚠️ Warning: Could not remove {file_path}: {e}')
-
-        # Remove directories
-        for dir_path in directories_to_remove:
-            if os.path.exists(dir_path):
-                try:
-                    shutil.rmtree(dir_path)
-                    if self.is_cli:
-                        logger.info(f'🗑️ Removed {dir_path}')
-                except Exception as e:
-                    logger.warning(f'⚠️ Warning: Could not remove {dir_path}: {e}')
-
-        logger.info('✅ Cleanup completed.')
+        try:
+            files_to_remove = [
+                self.config_js_path,
+                'package.json',
+                'package-lock.json',
+                self.input_css_path,
+            ]
+
+            directories_to_remove = ['node_modules']
+
+            # Remove files
+            for file_path in files_to_remove:
+                if os.path.exists(file_path):
+                    try:
+                        os.remove(file_path)
+                        if self.is_cli:
+                            logger.info(f'🗑️ Removed {file_path}')
+                    except Exception as e:
+                        logger.warning(f'⚠️ Warning: Could not remove {file_path}: {e}')
+
+            # Remove directories
+            for dir_path in directories_to_remove:
+                if os.path.exists(dir_path):
+                    try:
+                        shutil.rmtree(dir_path)
+                        if self.is_cli:
+                            logger.info(f'🗑️ Removed {dir_path}')
+                    except Exception as e:
+                        logger.warning(f'⚠️ Warning: Could not remove {dir_path}: {e}')
+
+            logger.info('✅ Cleanup completed.')
+
+        except Exception as e:
+            logger.error(f'❌ Error cleaning up: {e}')
+            raise e
 
         return self