fix: critical bug fixes for v1.6.1

Bug Fixes:
- Fixed duplicate logging output (messages appearing 2-3 times)
- Fixed metrics JSON parsing errors on corrupted files
- Fixed MyPy type errors with Response objects
- Reduced verbose output from smart grouping

Improvements:
- Cleaner, more concise console output
- Silent handling of first-run file creation
- Better error handling for API responses

Author: Arakiss

CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [1.6.1] - 2025-08-21
+
+### 🐛 Bug Fixes
+- **Fixed duplicate logging**: Removed redundant logger calls causing messages to appear 2-3 times
+- **Fixed metrics JSON parsing**: Better handling of corrupted or missing metrics files  
+- **Fixed MyPy type errors**: Added proper type checks for Response objects and type hints
+- **Reduced output verbosity**: Simplified smart grouping output to be more concise
+
+### 🚀 Improvements
+- Cleaner console output without debug noise
+- Silent handling of first-run file creation
+- More concise smart grouping summaries
+- Better error handling for API responses
+
 ## [1.6.0] - 2025-08-21
 
 ### ✨ Features

commitloom/__init__.py

@@ -5,7 +5,7 @@
 from .core.git import GitError, GitFile, GitOperations
 from .services.ai_service import AIService, CommitSuggestion, TokenUsage
 
-__version__ = "1.6.0"
+__version__ = "1.6.1"
 __author__ = "Petru Arakiss"
 __email__ = "petruarakiss@gmail.com"
 

commitloom/__main__.py

@@ -81,10 +81,6 @@ def commit(ctx, yes: bool, combine: bool, debug: bool, smart_grouping: bool, mod
 
         # Configure smart grouping
         loom.use_smart_grouping = smart_grouping
-        if smart_grouping:
-            console.print_info("Smart grouping: ENABLED (analyzing file relationships)")
-        else:
-            console.print_info("Smart grouping: DISABLED (using basic grouping)")
 
         # Validación personalizada para modelos OpenAI
         if model:

commitloom/cli/cli_handler.py

@@ -1,7 +1,6 @@
 #!/usr/bin/env python3
 """Main CLI handler module for CommitLoom."""
 
-import logging
 import os
 import subprocess
 import sys
@@ -19,14 +18,8 @@
 env_path = os.path.join(os.path.dirname(__file__), "..", "..", ".env")
 load_dotenv(dotenv_path=env_path)
 
-# Configure logging
-logging.basicConfig(
-    level=logging.INFO,
-    format="%(levelname)s: %(message)s",
-    handlers=[logging.StreamHandler(sys.stdout)],
-)
-
-logger = logging.getLogger(__name__)
+# Logging is configured by console module
+# logger = logging.getLogger(__name__)
 
 # Minimum number of files to activate batch processing
 BATCH_THRESHOLD = 3
@@ -281,23 +274,15 @@ def _create_basic_batches(self, valid_files: list[GitFile]) -> list[list[GitFile
 
     def _create_smart_batches(self, valid_files: list[GitFile]) -> list[list[GitFile]]:
         """Create intelligent batches using semantic analysis."""
-        console.print_info("Analyzing file relationships for intelligent grouping...")
-
         # Use the smart grouper to analyze files
         file_groups = self.smart_grouper.analyze_files(valid_files)
 
         if not file_groups:
             console.print_warning("Smart grouping produced no groups, falling back to basic grouping")
             return self._create_basic_batches(valid_files)
 
-        # Print group summary for user
-        console.print_info(f"Created {len(file_groups)} intelligent groups:")
-        for i, group in enumerate(file_groups, 1):
-            console.print_info(f"  Group {i}: {group.change_type.value} - {len(group.files)} files")
-            console.print_info(f"    Reason: {group.reason}")
-            console.print_info(f"    Confidence: {group.confidence:.0%}")
-            for file in group.files:
-                console.print_info(f"      - {file.path}")
+        # Print concise group summary
+        console.print_info(f"Smart grouping created {len(file_groups)} groups based on file relationships")
 
         # Convert FileGroup objects to lists of GitFile
         batches = [group.files for group in file_groups]

commitloom/cli/console.py

@@ -41,7 +41,10 @@ def set_auto_confirm(value: bool) -> None:
 def setup_logging(debug: bool = False):
     """Configure logging with optional debug mode."""
     level = logging.DEBUG if debug else logging.INFO
-
+    
+    # Clear existing handlers to avoid duplicates
+    logger.handlers.clear()
+    
     # Configure rich handler
     rich_handler = RichHandler(rich_tracebacks=True, markup=True, show_time=debug, show_path=debug)
     rich_handler.setLevel(level)
@@ -51,7 +54,7 @@ def setup_logging(debug: bool = False):
     logger.addHandler(rich_handler)
 
     if debug:
-        logger.debug("Debug mode enabled")
+        console.print("[dim]Debug mode enabled[/dim]")
 
 
 def print_debug(message: str, exc_info: bool = False) -> None:
@@ -66,25 +69,21 @@ def print_debug(message: str, exc_info: bool = False) -> None:
 
 def print_info(message: str) -> None:
     """Print info message."""
-    logger.info(f"ℹ️ {message}")
     console.print(f"\n[bold blue]ℹ️ {message}[/bold blue]")
 
 
 def print_warning(message: str) -> None:
     """Print warning message."""
-    logger.warning(f"⚠️ {message}")
     console.print(f"\n[bold yellow]⚠️ {message}[/bold yellow]")
 
 
 def print_error(message: str) -> None:
     """Print error message."""
-    logger.error(f"❌ {message}")
     console.print(f"\n[bold red]❌ {message}[/bold red]")
 
 
 def print_success(message: str) -> None:
     """Print success message."""
-    logger.info(f"✅ {message}")
     console.print(f"\n[bold green]✅ {message}[/bold green]")
 
 

commitloom/core/smart_grouping.py

@@ -419,7 +419,7 @@ def _extract_imports(self, file_path: str, language: str) -> list[str]:
         Returns:
             List of imported modules/files
         """
-        imports = []
+        imports: list[str] = []
 
         # For this implementation, we'll return empty list
         # In a real implementation, we would read the file and extract imports

commitloom/services/ai_service.py

@@ -251,6 +251,9 @@ def generate_commit_message(
                 error_message = str(last_exception)
             raise ValueError(f"API Request failed: {error_message}") from last_exception
 
+        if response is None:
+            raise ValueError("No response received from API")
+            
         if response.status_code == 400:
             error_data = response.json()
             error_message = error_data.get("error", {}).get("message", "Unknown error")

commitloom/services/metrics.py

@@ -86,12 +86,16 @@ def _load_statistics(self) -> UsageStatistics:
                 data = json.load(f)
                 # Ensure valid JSON structure
                 if not isinstance(data, dict):
-                    logger.warning("Invalid statistics file format, creating new file")
+                    logger.debug("Invalid statistics file format, resetting")
                     return UsageStatistics()
                 stats = UsageStatistics(**data)
                 return stats
-        except (json.JSONDecodeError, FileNotFoundError, KeyError) as e:
-            logger.warning(f"Failed to load statistics, creating new file: {str(e)}")
+        except FileNotFoundError:
+            # Normal for first run, don't log
+            return UsageStatistics()
+        except (json.JSONDecodeError, KeyError) as e:
+            # Only log for actual corruption
+            logger.debug(f"Statistics file corrupted, resetting: {str(e)}")
             return UsageStatistics()
 
     def _save_statistics(self) -> None:
@@ -123,10 +127,14 @@ def _save_metrics(self, metrics: CommitMetrics) -> None:
                 with open(self._metrics_file) as f:
                     metrics_list = json.load(f)
                 if not isinstance(metrics_list, list):
-                    logger.warning("Invalid metrics file format, creating new file")
+                    logger.debug("Invalid metrics file format, resetting")
                     metrics_list = []
-            except (json.JSONDecodeError, FileNotFoundError) as e:
-                logger.warning(f"Failed to load metrics, creating new file: {str(e)}")
+            except FileNotFoundError:
+                # Normal for first run, don't log
+                metrics_list = []
+            except json.JSONDecodeError as e:
+                # Only log for actual corruption
+                logger.debug(f"Metrics file corrupted, resetting: {str(e)}")
                 metrics_list = []
 
         # Add new metrics and save

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "commitloom"
-version = "1.6.0"
+version = "1.6.1"
 description = "Weave perfect git commits with AI-powered intelligence"
 authors = [
     { name = "Petru Arakiss", email = "petruarakiss@gmail.com" }