gitflow-analytics 3.3.0__py3-none-any.whl → 3.4.7__py3-none-any.whl

gitflow_analytics/security/extractors/secret_detector.py

@@ -3,7 +3,7 @@
 import logging
 import math
 import re
-from typing import Dict, List, Optional, Tuple
+from typing import Optional
 
 logger = logging.getLogger(__name__)
 
@@ -13,9 +13,9 @@ class SecretDetector:
 
     def __init__(
         self,
-        patterns: Dict[str, str],
+        patterns: dict[str, str],
         entropy_threshold: float = 4.5,
-        exclude_paths: List[str] = None,
+        exclude_paths: list[str] = None,
     ):
         """Initialize secret detector.
 
@@ -37,7 +37,7 @@ class SecretDetector:
             re.compile(r"xxx+|placeholder|your[_-]?api[_-]?key", re.IGNORECASE),
         ]
 
-    def scan_text(self, text: str, file_path: Optional[str] = None) -> List[Dict]:
+    def scan_text(self, text: str, file_path: Optional[str] = None) -> list[dict]:
         """Scan text for potential secrets.
 
         Args:
@@ -93,7 +93,7 @@ class SecretDetector:
 
         return findings
 
-    def scan_commit(self, commit_data: Dict) -> List[Dict]:
+    def scan_commit(self, commit_data: dict) -> list[dict]:
         """Scan a commit for secrets.
 
         Args:
@@ -118,17 +118,11 @@ class SecretDetector:
         """Check if file should be excluded from scanning."""
         from fnmatch import fnmatch
 
-        for pattern in self.exclude_paths:
-            if fnmatch(file_path, pattern):
-                return True
-        return False
+        return any(fnmatch(file_path, pattern) for pattern in self.exclude_paths)
 
     def _is_false_positive(self, value: str) -> bool:
         """Check if a detected secret is likely a false positive."""
-        for pattern in self.false_positive_patterns:
-            if pattern.search(value):
-                return True
-        return False
+        return any(pattern.search(value) for pattern in self.false_positive_patterns)
 
     def _get_severity(self, secret_type: str) -> str:
         """Determine severity based on secret type."""
@@ -164,7 +158,7 @@ class SecretDetector:
 
     def _find_high_entropy_strings(
         self, text: str, min_length: int = 20
-    ) -> List[Tuple[str, float]]:
+    ) -> list[tuple[str, float]]:
         """Find strings with high entropy (potential secrets).
 
         Args:

gitflow_analytics/security/extractors/vulnerability_scanner.py

@@ -8,7 +8,7 @@ import subprocess
 import tempfile
 from concurrent.futures import ThreadPoolExecutor
 from pathlib import Path
-from typing import Any, Dict, List, Optional
+from typing import Any, Optional
 
 logger = logging.getLogger(__name__)
 
@@ -26,7 +26,7 @@ class VulnerabilityScanner:
         # Check which tools are available
         self.available_tools = self._detect_available_tools()
 
-    def scan_files(self, files_changed: List[str], repo_path: Path) -> List[Dict]:
+    def scan_files(self, files_changed: list[str], repo_path: Path) -> list[dict]:
         """Scan changed files for vulnerabilities.
 
         Args:
@@ -49,7 +49,7 @@ class VulnerabilityScanner:
 
         return findings
 
-    def _detect_available_tools(self) -> Dict[str, bool]:
+    def _detect_available_tools(self) -> dict[str, bool]:
         """Detect which security tools are installed."""
         tools = {}
 
@@ -77,7 +77,7 @@ class VulnerabilityScanner:
         """Check if a tool is available in PATH."""
         return shutil.which(tool_name) is not None
 
-    def _scan_with_patterns(self, files_changed: List[str], repo_path: Path) -> List[Dict]:
+    def _scan_with_patterns(self, files_changed: list[str], repo_path: Path) -> list[dict]:
         """Quick pattern-based vulnerability detection."""
         findings = []
 
@@ -108,7 +108,7 @@ class VulnerabilityScanner:
 
         return findings
 
-    def _scan_with_tools(self, files_changed: List[str], repo_path: Path) -> List[Dict]:
+    def _scan_with_tools(self, files_changed: list[str], repo_path: Path) -> list[dict]:
         """Run security tools on changed files."""
         all_findings = []
 
@@ -143,7 +143,7 @@ class VulnerabilityScanner:
 
         return all_findings
 
-    def _group_files_by_language(self, files: List[str]) -> Dict[str, List[str]]:
+    def _group_files_by_language(self, files: list[str]) -> dict[str, list[str]]:
         """Group files by programming language."""
         groups = {}
 
@@ -166,7 +166,7 @@ class VulnerabilityScanner:
 
         return groups
 
-    def _run_semgrep(self, files: List[str], repo_path: Path) -> List[Dict]:
+    def _run_semgrep(self, files: list[str], repo_path: Path) -> list[dict]:
         """Run Semgrep security scanning."""
         findings = []
 
@@ -219,7 +219,7 @@ class VulnerabilityScanner:
 
         return findings
 
-    def _run_bandit(self, files: List[str], repo_path: Path) -> List[Dict]:
+    def _run_bandit(self, files: list[str], repo_path: Path) -> list[dict]:
         """Run Bandit for Python security scanning."""
         findings = []
 
@@ -258,7 +258,7 @@ class VulnerabilityScanner:
 
         return findings
 
-    def _run_gosec(self, files: List[str], repo_path: Path) -> List[Dict]:
+    def _run_gosec(self, files: list[str], repo_path: Path) -> list[dict]:
         """Run gosec for Go security scanning."""
         findings = []
 

gitflow_analytics/security/llm_analyzer.py

@@ -5,7 +5,7 @@ import logging
 import os
 from datetime import datetime, timedelta
 from pathlib import Path
-from typing import Any, Dict, List, Optional
+from typing import Any, Optional
 
 import httpx
 
@@ -33,7 +33,7 @@ class LLMSecurityAnalyzer:
         # Cache LLM responses for 7 days to save costs
         self.cache_ttl = timedelta(days=7)
 
-    def analyze_commit(self, commit_data: Dict) -> List[Dict]:
+    def analyze_commit(self, commit_data: dict) -> list[dict]:
         """Analyze a commit for security issues using LLM.
 
         Args:
@@ -72,7 +72,7 @@ class LLMSecurityAnalyzer:
 
         return findings
 
-    def _analyze_commit_message(self, commit_data: Dict) -> List[Dict]:
+    def _analyze_commit_message(self, commit_data: dict) -> list[dict]:
         """Analyze commit message for security implications."""
         prompt = self.config.commit_review_prompt.format(
             message=commit_data.get("message", ""),
@@ -83,7 +83,7 @@ class LLMSecurityAnalyzer:
         response = self._call_llm(prompt)
         return self._parse_llm_response(response, commit_data)
 
-    def _analyze_code_changes(self, commit_data: Dict) -> List[Dict]:
+    def _analyze_code_changes(self, commit_data: dict) -> list[dict]:
         """Analyze actual code changes for security issues."""
         # Limit the amount of code sent to LLM for cost control
         lines_added = commit_data.get("diff_content", "")
@@ -176,8 +176,8 @@ class LLMSecurityAnalyzer:
             return ""
 
     def _parse_llm_response(
-        self, response: str, commit_data: Dict, is_code_analysis: bool = False
-    ) -> List[Dict]:
+        self, response: str, commit_data: dict, is_code_analysis: bool = False
+    ) -> list[dict]:
         """Parse LLM response and extract security findings."""
         findings = []
 
@@ -273,7 +273,7 @@ class LLMSecurityAnalyzer:
 
         return "high" if len(response) > 100 else "medium"
 
-    def _get_cache_key(self, commit_data: Dict) -> str:
+    def _get_cache_key(self, commit_data: dict) -> str:
         """Generate cache key for commit data."""
         key_parts = [
             commit_data.get("commit_hash", ""),
@@ -286,7 +286,7 @@ class LLMSecurityAnalyzer:
 
         return hashlib.sha256(key_str.encode()).hexdigest()[:16]
 
-    def _get_cached_result(self, cache_key: str) -> Optional[List[Dict]]:
+    def _get_cached_result(self, cache_key: str) -> Optional[list[dict]]:
         """Get cached result if it exists and is not expired."""
         cache_file = self.cache_dir / f"{cache_key}.json"
         if not cache_file.exists():
@@ -305,7 +305,7 @@ class LLMSecurityAnalyzer:
             logger.debug(f"Error reading cache: {e}")
             return None
 
-    def _cache_result(self, cache_key: str, result: List[Dict]) -> None:
+    def _cache_result(self, cache_key: str, result: list[dict]) -> None:
         """Cache the analysis result."""
         cache_file = self.cache_dir / f"{cache_key}.json"
         try:
@@ -314,7 +314,7 @@ class LLMSecurityAnalyzer:
         except Exception as e:
             logger.debug(f"Error writing cache: {e}")
 
-    def generate_security_insights(self, all_findings: List[Dict]) -> str:
+    def generate_security_insights(self, all_findings: list[dict]) -> str:
         """Generate high-level security insights from all findings."""
         if not all_findings:
             return "No security issues detected in the analyzed period."

gitflow_analytics/security/security_analyzer.py

@@ -5,7 +5,7 @@ from concurrent.futures import ThreadPoolExecutor
 from dataclasses import dataclass
 from datetime import datetime
 from pathlib import Path
-from typing import Dict, List, Optional
+from typing import Optional
 
 from .config import SecurityConfig
 from .extractors import DependencyChecker, SecretDetector, VulnerabilityScanner
@@ -20,13 +20,13 @@ class SecurityAnalysis:
 
     commit_hash: str
     timestamp: datetime
-    files_changed: List[str]
+    files_changed: list[str]
 
     # Findings by type
-    secrets: List[Dict]
-    vulnerabilities: List[Dict]
-    dependency_issues: List[Dict]
-    llm_findings: List[Dict]
+    secrets: list[dict]
+    vulnerabilities: list[dict]
+    dependency_issues: list[dict]
+    llm_findings: list[dict]
 
     # Summary metrics
     total_findings: int
@@ -83,7 +83,7 @@ class SecurityAnalyzer:
         else:
             self.llm_analyzer = None
 
-    def analyze_commit(self, commit_data: Dict) -> SecurityAnalysis:
+    def analyze_commit(self, commit_data: dict) -> SecurityAnalysis:
         """Analyze a single commit for security issues.
 
         Args:
@@ -153,7 +153,7 @@ class SecurityAnalyzer:
             risk_score=risk_score,
         )
 
-    def analyze_batch(self, commits: List[Dict]) -> List[SecurityAnalysis]:
+    def analyze_batch(self, commits: list[dict]) -> list[SecurityAnalysis]:
         """Analyze multiple commits for security issues.
 
         Args:
@@ -184,7 +184,7 @@ class SecurityAnalyzer:
 
         return results
 
-    def _run_secret_detection(self, commit_data: Dict) -> List[Dict]:
+    def _run_secret_detection(self, commit_data: dict) -> list[dict]:
         """Run secret detection on commit."""
         try:
             return self.secret_detector.scan_commit(commit_data)
@@ -192,7 +192,7 @@ class SecurityAnalyzer:
             logger.warning(f"Secret detection error: {e}")
             return []
 
-    def _run_vulnerability_scan(self, commit_data: Dict) -> List[Dict]:
+    def _run_vulnerability_scan(self, commit_data: dict) -> list[dict]:
         """Run vulnerability scanning on changed files."""
         try:
             files_changed = commit_data.get("files_changed", [])
@@ -201,7 +201,7 @@ class SecurityAnalyzer:
             logger.warning(f"Vulnerability scanning error: {e}")
             return []
 
-    def _run_dependency_check(self, commit_data: Dict) -> List[Dict]:
+    def _run_dependency_check(self, commit_data: dict) -> list[dict]:
         """Check for vulnerable dependencies."""
         try:
             files_changed = commit_data.get("files_changed", [])
@@ -210,7 +210,7 @@ class SecurityAnalyzer:
             logger.warning(f"Dependency checking error: {e}")
             return []
 
-    def _run_llm_analysis(self, commit_data: Dict) -> List[Dict]:
+    def _run_llm_analysis(self, commit_data: dict) -> list[dict]:
         """Run LLM-based security analysis."""
         try:
             return self.llm_analyzer.analyze_commit(commit_data)
@@ -218,7 +218,7 @@ class SecurityAnalyzer:
             logger.warning(f"LLM analysis error: {e}")
             return []
 
-    def _count_severities(self, findings: List[Dict]) -> Dict[str, int]:
+    def _count_severities(self, findings: list[dict]) -> dict[str, int]:
         """Count findings by severity level."""
         counts = {"critical": 0, "high": 0, "medium": 0, "low": 0}
 
@@ -229,7 +229,7 @@ class SecurityAnalyzer:
 
         return counts
 
-    def _calculate_risk_score(self, severity_counts: Dict[str, int]) -> float:
+    def _calculate_risk_score(self, severity_counts: dict[str, int]) -> float:
         """Calculate overall risk score (0-100).
 
         Weighted formula:
@@ -249,7 +249,7 @@ class SecurityAnalyzer:
 
         return min(100.0, float(score))
 
-    def generate_summary_report(self, analyses: List[SecurityAnalysis]) -> Dict:
+    def generate_summary_report(self, analyses: list[SecurityAnalysis]) -> dict:
         """Generate summary report from multiple analyses.
 
         Args:
@@ -324,7 +324,7 @@ class SecurityAnalyzer:
 
         return summary
 
-    def _identify_top_issues(self, analyses: List[SecurityAnalysis]) -> List[Dict]:
+    def _identify_top_issues(self, analyses: list[SecurityAnalysis]) -> list[dict]:
         """Identify the most common/critical issues."""
         issue_counts = {}
 
@@ -374,7 +374,7 @@ class SecurityAnalyzer:
 
         return top_issues
 
-    def _generate_recommendations(self, analyses: List[SecurityAnalysis]) -> List[str]:
+    def _generate_recommendations(self, analyses: list[SecurityAnalysis]) -> list[str]:
         """Generate actionable security recommendations."""
         recommendations = []
 

gitflow_analytics/tui/screens/analysis_progress_screen.py

@@ -697,7 +697,7 @@ class AnalysisProgressScreen(Screen):
 
                     # Convert daily_commits to flat commits list
                     commits = []
-                    for date_str, day_commits in daily_commits.items():
+                    for _date_str, day_commits in daily_commits.items():
                         commits.extend(day_commits)
 
                     # Add flattened commits to result for compatibility

gitflow_analytics/ui/progress_display.py

@@ -11,7 +11,7 @@ from contextlib import contextmanager
 from dataclasses import dataclass
 from datetime import datetime, timedelta
 from enum import Enum
-from typing import Any, Dict, Optional
+from typing import Any, Optional
 
 # Try to import psutil, but make it optional
 try:
@@ -23,10 +23,7 @@ except ImportError:
 
 try:
     from rich import box
-    from rich.align import Align
-    from rich.columns import Columns
     from rich.console import Console, Group
-    from rich.layout import Layout
     from rich.live import Live
     from rich.panel import Panel
     from rich.progress import (
@@ -35,7 +32,6 @@ try:
         Progress,
         SpinnerColumn,
         TextColumn,
-        TimeElapsedColumn,
         TimeRemainingColumn,
     )
     from rich.table import Table
@@ -160,7 +156,7 @@ class RichProgressDisplay:
         )
 
         # Data tracking
-        self.repositories: Dict[str, RepositoryInfo] = {}
+        self.repositories: dict[str, RepositoryInfo] = {}
         self.statistics = ProgressStatistics()
         self.current_repo: Optional[str] = None
 
@@ -414,8 +410,20 @@ class RichProgressDisplay:
                 try:
                     self.statistics.memory_usage = self._process.memory_info().rss / 1024 / 1024
                     self.statistics.cpu_percent = self._process.cpu_percent()
-                except:
+                except (AttributeError, OSError):
+                    # Process might have terminated or psutil unavailable
+                    # This is non-critical for analysis, so just skip the update
                     pass
+                except Exception as e:
+                    # Log unexpected errors but don't fail progress display
+                    # Only log once to avoid spam
+                    if not hasattr(self, "_stats_error_logged"):
+                        import logging
+
+                        logging.getLogger(__name__).debug(
+                            f"Could not update process statistics: {e}"
+                        )
+                        self._stats_error_logged = True
 
         stats_items = []
 
@@ -483,26 +491,26 @@ class RichProgressDisplay:
 
         # Estimate total time if possible
         eta_text = ""
-        if self.statistics.processed_repositories > 0 and self.statistics.total_repositories > 0:
-            if self.statistics.processed_repositories < self.statistics.total_repositories:
-                elapsed_seconds = (
-                    (datetime.now() - self.statistics.start_time).total_seconds()
-                    if self.statistics.start_time
+        if (
+            self.statistics.processed_repositories > 0
+            and self.statistics.total_repositories > 0
+            and self.statistics.processed_repositories < self.statistics.total_repositories
+        ):
+            elapsed_seconds = (
+                (datetime.now() - self.statistics.start_time).total_seconds()
+                if self.statistics.start_time
+                else 0
+            )
+            if elapsed_seconds > 0:
+                rate = self.statistics.processed_repositories / elapsed_seconds
+                remaining = (
+                    (self.statistics.total_repositories - self.statistics.processed_repositories)
+                    / rate
+                    if rate > 0
                     else 0
                 )
-                if elapsed_seconds > 0:
-                    rate = self.statistics.processed_repositories / elapsed_seconds
-                    remaining = (
-                        (
-                            self.statistics.total_repositories
-                            - self.statistics.processed_repositories
-                        )
-                        / rate
-                        if rate > 0
-                        else 0
-                    )
-                    if remaining > 0:
-                        eta_text = f" • ETA: {timedelta(seconds=int(remaining))}"
+                if remaining > 0:
+                    eta_text = f" • ETA: {timedelta(seconds=int(remaining))}"
 
         stats_items.append(f"{phase_text} • {elapsed_text}{eta_text}")
 
@@ -766,7 +774,7 @@ class RichProgressDisplay:
         """
         with self._lock:
             # Pre-populate all repositories with their status
-            for idx, repo in enumerate(repository_list):
+            for _idx, repo in enumerate(repository_list):
                 repo_name = repo.get("name", "Unknown")
                 status_str = repo.get("status", "pending")
 
@@ -870,9 +878,8 @@ class RichProgressDisplay:
             # Update overall progress (handle both "repos" and "main" for compatibility)
             if description:
                 self.update_overall(completed or 0, description)
-            elif advance:
-                if self.overall_task_id is not None:
-                    self.overall_progress.advance(self.overall_task_id, advance)
+            elif advance and self.overall_task_id is not None:
+                self.overall_progress.advance(self.overall_task_id, advance)
         elif hasattr(self, "_task_ids") and task_id in self._task_ids:
             # Update specific task
             update_kwargs = {}

gitflow_analytics/verify_activity.py

@@ -8,7 +8,7 @@ import logging
 from collections import defaultdict
 from datetime import datetime, timedelta, timezone
 from pathlib import Path
-from typing import Any, Dict, List, Optional
+from typing import Any, Optional
 
 import click
 import git
@@ -48,7 +48,7 @@ class ActivityVerifier:
         if config.github and config.github.token:
             self.github_client = Github(config.github.token)
 
-    def verify_all_projects(self) -> Dict[str, Any]:
+    def verify_all_projects(self) -> dict[str, Any]:
         """Verify activity for all configured projects.
 
         Returns:
@@ -115,7 +115,7 @@ class ActivityVerifier:
 
         return results
 
-    def _get_repositories(self) -> List[Dict[str, Any]]:
+    def _get_repositories(self) -> list[dict[str, Any]]:
         """Get list of repositories to analyze.
 
         Returns:
@@ -209,23 +209,24 @@ class ActivityVerifier:
             try:
                 org = self.github_client.get_organization(self.config.github.organization)
                 for repo in org.get_repos(type="all"):
-                    if not repo.archived:
-                        # Check if not already added
-                        if not any(r["name"] == repo.full_name for r in repositories):
-                            repositories.append(
-                                {
-                                    "name": repo.full_name,
-                                    "path": None,
-                                    "is_local": False,
-                                    "github_name": repo.full_name,
-                                }
-                            )
+                    # Check if not archived and not already added
+                    if not repo.archived and not any(
+                        r["name"] == repo.full_name for r in repositories
+                    ):
+                        repositories.append(
+                            {
+                                "name": repo.full_name,
+                                "path": None,
+                                "is_local": False,
+                                "github_name": repo.full_name,
+                            }
+                        )
             except GithubException as e:
                 logger.error(f"Error fetching organization repos: {e}")
 
         return repositories
 
-    def _verify_project_activity(self, repo_info: Dict[str, Any]) -> Dict[str, Any]:
+    def _verify_project_activity(self, repo_info: dict[str, Any]) -> dict[str, Any]:
         """Verify activity for a single project.
 
         Args:
@@ -241,7 +242,7 @@ class ActivityVerifier:
         else:
             raise ValueError(f"No valid path or GitHub name for repository {repo_info['name']}")
 
-    def _verify_github_activity(self, repo_name: str) -> Dict[str, Any]:
+    def _verify_github_activity(self, repo_name: str) -> dict[str, Any]:
         """Verify activity for a GitHub repository using API.
 
         Args:
@@ -327,7 +328,7 @@ class ActivityVerifier:
 
         return result
 
-    def _verify_local_activity(self, repo_path: Path) -> Dict[str, Any]:
+    def _verify_local_activity(self, repo_path: Path) -> dict[str, Any]:
         """Verify activity for a local Git repository.
 
         Args:
@@ -451,7 +452,7 @@ class ActivityVerifier:
 
         return result
 
-    def _initialize_daily_matrix(self) -> Dict[str, Dict[str, int]]:
+    def _initialize_daily_matrix(self) -> dict[str, dict[str, int]]:
         """Initialize the daily activity matrix structure.
 
         Returns:
@@ -468,7 +469,7 @@ class ActivityVerifier:
         return matrix
 
     def _update_daily_matrix(
-        self, matrix: Dict[str, Dict[str, int]], project_name: str, daily_commits: Dict[str, int]
+        self, matrix: dict[str, dict[str, int]], project_name: str, daily_commits: dict[str, int]
     ) -> None:
         """Update the daily matrix with project commit data.
 
@@ -481,7 +482,7 @@ class ActivityVerifier:
             if date_str in matrix:
                 matrix[date_str][project_name] = count
 
-    def format_report(self, results: Dict[str, Any]) -> str:
+    def format_report(self, results: dict[str, Any]) -> str:
         """Format the verification results as a readable report.
 
         Args:
@@ -524,11 +525,7 @@ class ActivityVerifier:
         matrix_data = []
         dates = sorted(results["daily_matrix"].keys())
         projects = sorted(
-            set(
-                project
-                for date_data in results["daily_matrix"].values()
-                for project in date_data.keys()
-            )
+            set(project for date_data in results["daily_matrix"].values() for project in date_data)
         )
 
         if projects and dates:
@@ -691,7 +688,7 @@ def verify_activity_command(
             click.echo(f"   - {project}")
 
     zero_days = len(
-        [d for d in results["daily_matrix"].keys() if sum(results["daily_matrix"][d].values()) == 0]
+        [d for d in results["daily_matrix"] if sum(results["daily_matrix"][d].values()) == 0]
     )
     if zero_days > 0:
         click.echo(f"\n⚠️  WARNING: Found {zero_days} days with zero activity across all projects!")

{gitflow_analytics-3.3.0.dist-info → gitflow_analytics-3.4.7.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gitflow-analytics
-Version: 3.3.0
+Version: 3.4.7
 Summary: Analyze Git repositories for developer productivity insights
 Author-email: Bob Matyas <bobmatnyc@gmail.com>
 License: MIT