gitflow-analytics 3.6.1__py3-none-any.whl → 3.7.0__py3-none-any.whl

gitflow_analytics/__init__.py

@@ -5,20 +5,16 @@ from ._version import __version__, __version_info__
 __author__ = "Bob Matyas"
 __email__ = "bobmatnyc@gmail.com"
 
-from .core.analyzer import GitAnalyzer
-from .core.cache import GitAnalysisCache
-from .core.identity import DeveloperIdentityResolver
-from .extractors.story_points import StoryPointExtractor
-from .extractors.tickets import TicketExtractor
-from .reports.csv_writer import CSVReportGenerator
+# Heavy imports removed from package __init__ for CLI performance
+# Import these directly when needed in your code:
+#   from gitflow_analytics.core.analyzer import GitAnalyzer
+#   from gitflow_analytics.core.cache import GitAnalysisCache
+#   from gitflow_analytics.core.identity import DeveloperIdentityResolver
+#   from gitflow_analytics.extractors.story_points import StoryPointExtractor
+#   from gitflow_analytics.extractors.tickets import TicketExtractor
+#   from gitflow_analytics.reports.csv_writer import CSVReportGenerator
 
 __all__ = [
     "__version__",
     "__version_info__",
-    "GitAnalyzer",
-    "GitAnalysisCache",
-    "DeveloperIdentityResolver",
-    "StoryPointExtractor",
-    "TicketExtractor",
-    "CSVReportGenerator",
 ]

gitflow_analytics/_version.py

@@ -1,4 +1,4 @@
 """Version information for gitflow-analytics."""
 
-__version__ = "3.6.1"
+__version__ = "3.7.0"
 __version_info__ = tuple(int(x) for x in __version__.split("."))

gitflow_analytics/cli.py

@@ -14,25 +14,28 @@ from typing import Any, Optional, cast
 
 import click
 import git
-import pandas as pd
 import yaml
 
 from ._version import __version__
 from .config import ConfigLoader
-from .core.analyzer import GitAnalyzer
-from .core.cache import GitAnalysisCache
-from .core.git_auth import preflight_git_authentication
-from .core.identity import DeveloperIdentityResolver
-from .integrations.orchestrator import IntegrationOrchestrator
-from .metrics.dora import DORAMetricsCalculator
-from .reports.analytics_writer import AnalyticsReportGenerator
-from .reports.csv_writer import CSVReportGenerator
-from .reports.json_exporter import ComprehensiveJSONExporter
-from .reports.narrative_writer import NarrativeReportGenerator
-from .reports.weekly_trends_writer import WeeklyTrendsWriter
-from .training.pipeline import CommitClassificationTrainer
 from .ui.progress_display import create_progress_display
 
+# Heavy imports are lazy-loaded to improve CLI startup time
+# These imports add 1-2 seconds to startup but are only needed for actual analysis
+# from .core.analyzer import GitAnalyzer
+# from .core.cache import GitAnalysisCache
+# from .core.git_auth import preflight_git_authentication
+# from .core.identity import DeveloperIdentityResolver
+# from .integrations.orchestrator import IntegrationOrchestrator
+# from .metrics.dora import DORAMetricsCalculator
+# from .reports.analytics_writer import AnalyticsReportGenerator
+# from .reports.csv_writer import CSVReportGenerator
+# from .reports.json_exporter import ComprehensiveJSONExporter
+# from .reports.narrative_writer import NarrativeReportGenerator
+# from .reports.weekly_trends_writer import WeeklyTrendsWriter
+# from .training.pipeline import CommitClassificationTrainer
+# import pandas as pd  # Only used in one location, lazy loaded
+
 logger = logging.getLogger(__name__)
 
 
@@ -257,64 +260,33 @@ class ImprovedErrorHandler:
         click.echo("\nFor help: gitflow-analytics help", err=True)
 
 
-class TUIAsDefaultGroup(click.Group):
+class AnalyzeAsDefaultGroup(click.Group):
     """
-    Custom Click group that defaults to TUI when no explicit subcommand is provided.
-    This allows 'gitflow-analytics -c config.yaml' to launch the TUI by default.
-    For explicit CLI analysis, use 'gitflow-analytics analyze -c config.yaml'
+    Custom Click group that defaults to analyze when no explicit subcommand is provided.
+    This allows 'gitflow-analytics -c config.yaml' to run analysis by default.
     """
 
     def parse_args(self, ctx, args):
-        """Override parse_args to default to TUI unless explicit subcommand or CLI-only options."""
+        """Override parse_args to default to analyze unless explicit subcommand provided."""
         # Check if the first argument is a known subcommand
         if args and args[0] in self.list_commands(ctx):
             return super().parse_args(ctx, args)
 
-        # Check for global options that should NOT be routed to TUI
+        # Check for global options that should NOT be routed to analyze
         global_options = {"--version", "--help", "-h"}
         if args and args[0] in global_options:
             return super().parse_args(ctx, args)
 
-        # Check if we have arguments that indicate explicit CLI analysis request
-        cli_only_indicators = {
-            "--no-rich",
-            "--generate-csv",
-            "--validate-only",
-            "--warm-cache",
-            "--validate-cache",
-            "--qualitative-only",
-        }
-
-        # If user explicitly requests CLI-only features, route to analyze
-        if args and any(arg in cli_only_indicators for arg in args):
+        # For all other cases (including -c config.yaml), default to analyze
+        if args and args[0].startswith("-"):
             new_args = ["analyze"] + args
             return super().parse_args(ctx, new_args)
 
-        # For all other cases (including -c config.yaml), default to TUI
-        if args and args[0].startswith("-"):
-            # Check if TUI dependencies are available
-            try:
-                import importlib.util
-
-                textual_spec = importlib.util.find_spec("textual")
-                # TUI is available - route to TUI
-                if textual_spec is not None:
-                    new_args = ["tui"] + args
-                    return super().parse_args(ctx, new_args)
-                else:
-                    # TUI not available - fallback to analyze
-                    new_args = ["analyze"] + args
-                    return super().parse_args(ctx, new_args)
-            except (ImportError, ValueError):
-                # TUI not available - fallback to analyze
-                new_args = ["analyze"] + args
-                return super().parse_args(ctx, new_args)
-
         # Otherwise, use default behavior
         return super().parse_args(ctx, args)
 
 
-@click.group(cls=TUIAsDefaultGroup, invoke_without_command=True)
+@click.group(cls=AnalyzeAsDefaultGroup, invoke_without_command=True)
 @click.version_option(version=__version__, prog_name="GitFlow Analytics")
 @click.help_option("-h", "--help")
 @click.pass_context
@@ -374,119 +346,6 @@ def cli(ctx: click.Context) -> None:
         ctx.exit(0)
 
 
-# TUI command - Terminal User Interface
-@cli.command(name="tui")
-@click.option(
-    "--config",
-    "-c",
-    type=click.Path(exists=True, path_type=Path),
-    default=None,
-    help="Path to YAML configuration file (optional - can be loaded in TUI)",
-)
-@click.option(
-    "--weeks",
-    "-w",
-    type=int,
-    default=None,
-    help="Number of weeks to analyze (passed to TUI)",
-)
-@click.option(
-    "--clear-cache",
-    is_flag=True,
-    help="Clear cache before analysis (passed to TUI)",
-)
-@click.option(
-    "--output",
-    "-o",
-    type=click.Path(path_type=Path),
-    default=None,
-    help="Output directory for reports (passed to TUI)",
-)
-def tui_command(
-    config: Optional[Path],
-    weeks: Optional[int],
-    clear_cache: bool,
-    output: Optional[Path],
-) -> None:
-    """Launch the Terminal User Interface for GitFlow Analytics.
-
-    \b
-    The TUI provides an interactive interface for:
-    - Loading and editing configuration files
-    - Running analysis with real-time progress updates
-    - Viewing results in an organized, navigable format
-    - Exporting reports in various formats
-
-    \b
-    FEATURES:
-    • Full-screen terminal interface with keyboard navigation
-    • Real-time progress tracking during analysis
-    • Interactive configuration management
-    • Results browser with filtering and export options
-    • Built-in help system and keyboard shortcuts
-
-    \b
-    EXAMPLES:
-    # Launch TUI without pre-loading configuration
-    gitflow-analytics tui
-
-    # Launch TUI with a specific configuration file
-    gitflow-analytics tui -c config.yaml
-
-    \b
-    KEYBOARD SHORTCUTS:
-    • Ctrl+Q / Ctrl+C: Quit application
-    • F1: Show help
-    • Ctrl+D: Toggle dark/light mode
-    • Escape: Go back/cancel current action
-    """
-    try:
-        # Import TUI components only when needed
-        from .tui.app import GitFlowAnalyticsApp
-
-        # Create and run the TUI application
-        app = GitFlowAnalyticsApp()
-
-        # Pass CLI parameters to TUI
-        if weeks is not None:
-            app.default_weeks = weeks
-        if clear_cache:
-            app.clear_cache_on_start = True
-        if output:
-            app.default_output_dir = output
-
-        # If config path provided, try to load it
-        if config:
-            try:
-                from .config import ConfigLoader
-
-                loaded_config = ConfigLoader.load(config)
-                app.config = loaded_config
-                app.config_path = config
-                app.initialization_complete = True
-            except Exception as e:
-                # Don't fail - let TUI handle config loading
-                click.echo(f"⚠️  Could not pre-load config: {e}")
-                click.echo("   You can load the configuration within the TUI.")
-
-        # Run the TUI
-        app.run()
-
-    except ImportError as e:
-        click.echo("❌ TUI dependencies not installed.", err=True)
-        click.echo(f"   Error: {e}", err=True)
-        click.echo("\n💡 Install TUI dependencies:", err=True)
-        click.echo("   pip install 'gitflow-analytics[tui]'", err=True)
-        click.echo("   # or", err=True)
-        click.echo("   pip install textual>=0.41.0", err=True)
-        sys.exit(1)
-    except Exception as e:
-        click.echo(f"❌ Failed to launch TUI: {e}", err=True)
-        if "--debug" in sys.argv:
-            raise
-        sys.exit(1)
-
-
 @cli.command(name="analyze")
 @click.option(
     "--config",
@@ -534,7 +393,7 @@ def tui_command(
     "--no-rich",
     is_flag=True,
     default=True,
-    help="Disable rich terminal output (simple output is default to prevent TUI hanging)",
+    help="Disable rich terminal output (use simple text progress instead)",
 )
 @click.option(
     "--log",
@@ -697,8 +556,20 @@ def analyze(
 ) -> None:
     """Analyze Git repositories using configuration file."""
 
-    # Initialize progress service early with the correct style
+    # Lazy imports: Only load heavy dependencies when actually running analysis
+    # This improves CLI startup time from ~2s to <100ms for commands like --help
+    from .core.analyzer import GitAnalyzer
+    from .core.cache import GitAnalysisCache
+    from .core.git_auth import preflight_git_authentication
+    from .core.identity import DeveloperIdentityResolver
     from .core.progress import get_progress_service
+    from .integrations.orchestrator import IntegrationOrchestrator
+    from .metrics.dora import DORAMetricsCalculator
+    from .reports.analytics_writer import AnalyticsReportGenerator
+    from .reports.csv_writer import CSVReportGenerator
+    from .reports.json_exporter import ComprehensiveJSONExporter
+    from .reports.narrative_writer import NarrativeReportGenerator
+    from .reports.weekly_trends_writer import WeeklyTrendsWriter
 
     try:
         from ._version import __version__
@@ -710,7 +581,7 @@ def analyze(
     # Initialize progress service with user's preference
     progress = get_progress_service(display_style=progress_style, version=version)
 
-    # Initialize display - simple output by default to prevent TUI hanging
+    # Initialize display - simple output by default for better compatibility
     # Create display - only create if rich output is explicitly enabled (--no-rich=False)
     display = (
         create_progress_display(style="simple" if no_rich else "rich", version=__version__)
@@ -1448,6 +1319,11 @@ def analyze(
                 else:
                     click.echo("🔄 Force fetch enabled - analyzing all repositories")
 
+            # Initialize counters before fetching (used in validation even if skipped)
+            total_commits = 0
+            total_tickets = 0
+            total_developers = set()
+
             # Step 1: Fetch data only for repos that need analysis
             if repos_needing_analysis:
                 if display and display._live:
@@ -1552,11 +1428,6 @@ def analyze(
                         progress.initialize_repositories(all_repo_list)
                         progress.set_phase("Step 1: Data Fetching")
 
-                # Fetch data for repositories that need analysis
-                total_commits = 0
-                total_tickets = 0
-                total_developers = set()  # Track unique developers
-
                 # Create top-level progress for all repositories
                 with progress.progress(
                     total=len(repos_needing_analysis),
@@ -1886,6 +1757,102 @@ def analyze(
                             repo_path = Path(repo_config.path)
                             project_key = repo_config.project_key or repo_path.name
 
+                            # Check if repo exists, clone if needed (critical for organization mode)
+                            if not repo_path.exists():
+                                if repo_config.github_repo and cfg.github.organization:
+                                    if display:
+                                        display.print_status(
+                                            f"   📥 Cloning {repo_config.github_repo} from GitHub...",
+                                            "info",
+                                        )
+                                    else:
+                                        click.echo(f"   📥 Cloning {repo_config.github_repo} from GitHub...")
+                                    try:
+                                        # Ensure parent directory exists
+                                        repo_path.parent.mkdir(parents=True, exist_ok=True)
+
+                                        # Build clone URL with authentication
+                                        clone_url = f"https://github.com/{repo_config.github_repo}.git"
+                                        if cfg.github.token:
+                                            clone_url = f"https://{cfg.github.token}@github.com/{repo_config.github_repo}.git"
+
+                                        # Clone using subprocess for better control
+                                        env = os.environ.copy()
+                                        env["GIT_TERMINAL_PROMPT"] = "0"
+                                        env["GIT_ASKPASS"] = ""
+                                        env["GCM_INTERACTIVE"] = "never"
+
+                                        cmd = ["git", "clone", "--config", "credential.helper="]
+                                        if repo_config.branch:
+                                            cmd.extend(["-b", repo_config.branch])
+                                        cmd.extend([clone_url, str(repo_path)])
+
+                                        result = subprocess.run(
+                                            cmd,
+                                            env=env,
+                                            capture_output=True,
+                                            text=True,
+                                            timeout=30,
+                                        )
+
+                                        if result.returncode != 0:
+                                            error_msg = result.stderr or result.stdout
+                                            if any(
+                                                x in error_msg.lower()
+                                                for x in ["authentication", "permission denied", "401", "403"]
+                                            ):
+                                                if display:
+                                                    display.print_status(
+                                                        f"   ❌ Authentication failed for {repo_config.github_repo}",
+                                                        "error",
+                                                    )
+                                                else:
+                                                    click.echo(
+                                                        f"   ❌ Authentication failed for {repo_config.github_repo}"
+                                                    )
+                                                continue
+                                            else:
+                                                raise subprocess.CalledProcessError(
+                                                    result.returncode, cmd, result.stdout, result.stderr
+                                                )
+
+                                        if display:
+                                            display.print_status(
+                                                f"   ✅ Cloned {repo_config.github_repo}", "success"
+                                            )
+                                        else:
+                                            click.echo(f"   ✅ Cloned {repo_config.github_repo}")
+
+                                    except subprocess.TimeoutExpired:
+                                        if display:
+                                            display.print_status(
+                                                f"   ❌ Clone timeout for {repo_config.github_repo}",
+                                                "error",
+                                            )
+                                        else:
+                                            click.echo(f"   ❌ Clone timeout for {repo_config.github_repo}")
+                                        continue
+                                    except Exception as e:
+                                        if display:
+                                            display.print_status(
+                                                f"   ❌ Failed to clone {repo_config.github_repo}: {e}",
+                                                "error",
+                                            )
+                                        else:
+                                            click.echo(
+                                                f"   ❌ Failed to clone {repo_config.github_repo}: {e}"
+                                            )
+                                        continue
+                                else:
+                                    # No github_repo configured, can't clone
+                                    if display:
+                                        display.print_status(
+                                            f"   ❌ Repository not found: {repo_path}", "error"
+                                        )
+                                    else:
+                                        click.echo(f"   ❌ Repository not found: {repo_path}")
+                                    continue
+
                             # Progress callback for fetch
                             def progress_callback(message: str):
                                 if display:
@@ -3567,6 +3534,9 @@ def analyze(
                 logger.debug("Starting narrative report generation")
                 narrative_gen = NarrativeReportGenerator()
 
+                # Lazy import pandas - only needed for CSV reading in narrative generation
+                import pandas as pd
+
                 # Load activity distribution data
                 logger.debug("Loading activity distribution data")
                 activity_df = pd.read_csv(activity_report)
@@ -4111,7 +4081,7 @@ def analyze(
     "--no-rich",
     is_flag=True,
     default=True,
-    help="Disable rich terminal output (simple output is default to prevent TUI hanging)",
+    help="Disable rich terminal output (use simple text progress instead)",
 )
 def fetch(
     config: Path,
@@ -4161,7 +4131,7 @@ def fetch(
       - Use --clear-cache to force fresh fetch
     """
     # Initialize display
-    # Create display - simple output by default to prevent TUI hanging, rich only when explicitly enabled
+    # Create display - simple output by default for better compatibility, rich only when explicitly enabled
     display = (
         create_progress_display(style="simple" if no_rich else "rich", version=__version__)
         if not no_rich
@@ -4186,6 +4156,10 @@ def fetch(
         logger = logging.getLogger(__name__)
 
     try:
+        # Lazy imports
+        from .core.cache import GitAnalysisCache
+        from .integrations.orchestrator import IntegrationOrchestrator
+
         if display:
             display.show_header()
 
@@ -5604,6 +5578,9 @@ def train(
         # Initialize trainer
         click.echo("🧠 Initializing training pipeline...")
         try:
+            # Lazy import - only needed for train command
+            from .training.pipeline import CommitClassificationTrainer
+
             trainer = CommitClassificationTrainer(
                 config=cfg, cache=cache, orchestrator=orchestrator, training_config=training_config
             )
@@ -5939,6 +5916,10 @@ def training_statistics(config: Path) -> None:
     - Compare different model versions
     """
     try:
+        # Lazy imports
+        from .core.cache import GitAnalysisCache
+        from .training.pipeline import CommitClassificationTrainer
+
         cfg = ConfigLoader.load(config)
         cache = GitAnalysisCache(cfg.cache.directory)
 

gitflow_analytics/cli_wizards/install_wizard.py

@@ -117,7 +117,7 @@ class InstallWizard:
             return getpass.getpass(prompt)
         else:
             click.echo(f"⚠️  Non-interactive mode detected - {field_name} will be visible", err=True)
-            return click.prompt(prompt, hide_input=False)
+            return click.prompt(prompt, hide_input=False).strip()
 
     def _select_profile(self) -> dict:
         """Let user select installation profile."""
@@ -484,7 +484,7 @@ class InstallWizard:
                 if not click.confirm("Add anyway?", default=False):
                     continue
 
-            repo_name = click.prompt("Repository name", default=path_obj.name)
+            repo_name = click.prompt("Repository name", default=path_obj.name).strip()
 
             repositories.append({"name": repo_name, "path": str(path_obj)})
             click.echo(f"Added repository #{len(repositories)}\n")
@@ -850,7 +850,7 @@ class InstallWizard:
                 "Output directory for reports",
                 type=str,
                 default="./reports",
-            )
+            ).strip()
             output_path = self._validate_directory_path(output_dir, "Output directory")
             if output_path is not None:
                 output_dir = str(output_path)
@@ -863,7 +863,7 @@ class InstallWizard:
                 "Cache directory",
                 type=str,
                 default="./.gitflow-cache",
-            )
+            ).strip()
             cache_path = self._validate_directory_path(cache_dir, "Cache directory")
             if cache_path is not None:
                 cache_dir = str(cache_path)
@@ -928,7 +928,7 @@ class InstallWizard:
                 # Use existing file
                 aliases_path = click.prompt(
                     "Path to aliases.yaml (relative to config)", default="../shared/aliases.yaml"
-                )
+                ).strip()
 
                 # Ensure analysis.identity section exists
                 if "identity" not in self.config_data.get("analysis", {}):

gitflow_analytics/core/cache.py

@@ -5,7 +5,7 @@ import json
 import logging
 import os
 from contextlib import contextmanager
-from datetime import datetime, timedelta
+from datetime import datetime, timedelta, timezone
 from pathlib import Path
 from typing import Any, Optional, Union
 
@@ -990,7 +990,7 @@ class GitAnalysisCache:
                 "complexity_delta": commit_data.get("complexity_delta", 0.0),
                 "story_points": commit_data.get("story_points"),
                 "ticket_references": commit_data.get("ticket_references", []),
-                "cached_at": datetime.utcnow(),
+                "cached_at": datetime.now(timezone.utc),
             }
             mappings.append(mapping)
 
@@ -1101,7 +1101,7 @@ class GitAnalysisCache:
                         "complexity_delta": commit_data.get("complexity_delta"),
                         "story_points": commit_data.get("story_points"),
                         "ticket_references": commit_data.get("ticket_references"),
-                        "cached_at": datetime.utcnow(),
+                        "cached_at": datetime.now(timezone.utc),
                     }
 
                     # Only include non-None values in update