build: bump version to 1.2.7 and fix metrics storage

Author: Arakiss

CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [1.2.7] - 2025-03-05
+### 🛠️ Fixes
+- Fix JSON handling in metrics storage
+- Make statistics display more robust against malformed data
+- Add safety checks for repository and model usage data
+
+
+
 ## [1.2.6] - 2025-03-05
 ### 🛠️ Fixes
 - Connect Auto Release and Package Publishing workflows

commitloom/__init__.py

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

commitloom/cli/cli_handler.py

@@ -349,51 +349,56 @@ def stats_command(self) -> None:
 
         # Display basic stats
         console.console.print("\n[bold cyan]Basic Statistics:[/bold cyan]")
-        console.console.print(f"  • Total commits generated: {stats['total_commits']:,}")
-        console.console.print(f"  • Total tokens used: {stats['total_tokens']:,}")
-        console.console.print(f"  • Total cost: €{stats['total_cost_in_eur']:.4f}")
-        console.console.print(f"  • Total files processed: {stats['total_files_processed']:,}")
+        console.console.print(f"  • Total commits generated: {stats.get('total_commits', 0):,}")
+        console.console.print(f"  • Total tokens used: {stats.get('total_tokens', 0):,}")
+        console.console.print(f"  • Total cost: €{stats.get('total_cost_in_eur', 0.0):.4f}")
+        console.console.print(f"  • Total files processed: {stats.get('total_files_processed', 0):,}")
 
         # Display time saved if available
         if 'time_saved_formatted' in stats:
             console.console.print(f"  • Total time saved: {stats['time_saved_formatted']}")
 
         # Display activity period if available
         if 'first_used_at' in stats and stats['first_used_at'] and 'days_active' in stats:
-            console.console.print(f"  • Active since: {stats['first_used_at'].split('T')[0]}")
+            first_used = stats['first_used_at']
+            date_part = first_used.split('T')[0] if isinstance(first_used, str) and 'T' in first_used else first_used
+            console.console.print(f"  • Active since: {date_part}")
             console.console.print(f"  • Days active: {stats['days_active']}")
 
             if 'avg_commits_per_day' in stats:
                 console.console.print(f"  • Average commits per day: {stats['avg_commits_per_day']:.2f}")
                 console.console.print(f"  • Average cost per day: €{stats['avg_cost_per_day']:.4f}")
 
         # Display repository stats if available
-        if stats['repositories']:
+        repositories = stats.get('repositories', {})
+        if repositories and isinstance(repositories, dict):
             console.console.print("\n[bold cyan]Repository Activity:[/bold cyan]")
-            console.console.print(f"  • Most active repository: {stats['most_active_repository']}")
-            console.console.print(f"  • Repositories used: {len(stats['repositories'])}")
+            if 'most_active_repository' in stats and stats['most_active_repository']:
+                console.console.print(f"  • Most active repository: {stats['most_active_repository']}")
+            console.console.print(f"  • Repositories used: {len(repositories)}")
 
         # Display model usage if available
-        if stats['model_usage']:
+        model_usage = stats.get('model_usage', {})
+        if model_usage and isinstance(model_usage, dict):
             console.console.print("\n[bold cyan]Model Usage:[/bold cyan]")
-            for model, count in stats['model_usage'].items():
+            for model, count in model_usage.items():
                 console.console.print(f"  • {model}: {count} commits")
 
         # Display batch vs single commits
         console.console.print("\n[bold cyan]Processing Methods:[/bold cyan]")
-        console.console.print(f"  • Batch commits: {stats['batch_commits']}")
-        console.console.print(f"  • Single commits: {stats['single_commits']}")
+        console.console.print(f"  • Batch commits: {stats.get('batch_commits', 0)}")
+        console.console.print(f"  • Single commits: {stats.get('single_commits', 0)}")
 
         # Get more detailed stats if commits exist
-        if stats['total_commits'] > 0:
+        if stats.get('total_commits', 0) > 0:
             model_stats = metrics_manager.get_model_usage_stats()
             if model_stats:
                 console.console.print("\n[bold cyan]Detailed Model Stats:[/bold cyan]")
                 for model, model_data in model_stats.items():
                     console.console.print(f"  • {model}:")
-                    console.console.print(f"    - Total tokens: {model_data['tokens']:,}")
-                    console.console.print(f"    - Total cost: €{model_data['cost']:.4f}")
-                    console.console.print(f"    - Avg tokens per commit: {model_data['avg_tokens_per_commit']:.1f}")
+                    console.console.print(f"    - Total tokens: {model_data.get('tokens', 0):,}")
+                    console.console.print(f"    - Total cost: €{model_data.get('cost', 0.0):.4f}")
+                    console.console.print(f"    - Avg tokens per commit: {model_data.get('avg_tokens_per_commit', 0.0):.1f}")
 
     def run(
         self, auto_commit: bool = False, combine_commits: bool = False, debug: bool = False

commitloom/services/metrics.py

@@ -84,6 +84,10 @@ def _load_statistics(self) -> UsageStatistics:
         try:
             with open(self._stats_file) as f:
                 data = json.load(f)
+                # Ensure valid JSON structure
+                if not isinstance(data, dict):
+                    logger.warning("Invalid statistics file format, creating new file")
+                    return UsageStatistics()
                 stats = UsageStatistics(**data)
                 return stats
         except (json.JSONDecodeError, FileNotFoundError, KeyError) as e:
@@ -93,8 +97,19 @@ def _load_statistics(self) -> UsageStatistics:
     def _save_statistics(self) -> None:
         """Save usage statistics to file."""
         try:
+            # Ensure valid data structure before saving
+            stats_dict = asdict(self._statistics)
+            
+            # Fix any potential problematic values
+            if 'repositories' in stats_dict and not isinstance(stats_dict['repositories'], dict):
+                stats_dict['repositories'] = {}
+                
+            if 'model_usage' in stats_dict and not isinstance(stats_dict['model_usage'], dict):
+                stats_dict['model_usage'] = {}
+            
             with open(self._stats_file, "w") as f:
-                json.dump(asdict(self._statistics), f, indent=2)
+                json.dump(stats_dict, f, indent=2)
+                
         except (OSError, TypeError) as e:
             logger.warning(f"Failed to save statistics: {str(e)}")
 

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "commitloom"
-version = "1.2.6"
+version = "1.2.7"
 description = "Weave perfect git commits with AI-powered intelligence"
 authors = ["Petru Arakiss <petruarakiss@gmail.com>"]
 readme = "README.md"