gitflow-analytics 3.13.5__tar.gz → 3.13.6__tar.gz

{gitflow_analytics-3.13.5 → gitflow_analytics-3.13.6}/PKG-INFO

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

{gitflow_analytics-3.13.5 → gitflow_analytics-3.13.6}/pyproject.toml

@@ -96,6 +96,9 @@ target-version = "py39"
 select = ["E", "F", "UP", "B", "SIM", "I"]
 ignore = ["E501"]  # Line too long - handled by formatter
 
+[tool.ruff.lint.per-file-ignores]
+"tests/**/*.py" = ["E402", "E722", "F401", "SIM102"]  # Allow test-specific patterns
+
 [tool.black]
 line-length = 100
 target-version = ['py39']

{gitflow_analytics-3.13.5 → gitflow_analytics-3.13.6}/src/gitflow_analytics/_version.py

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

{gitflow_analytics-3.13.5 → gitflow_analytics-3.13.6}/src/gitflow_analytics/cli.py

@@ -1590,6 +1590,68 @@ def analyze(
                             total_commits += result["stats"]["total_commits"]
                             total_tickets += result["stats"]["unique_tickets"]
 
+                            # Fetch and enrich with GitHub PRs after data collection
+                            if repo_config.github_repo:
+                                try:
+                                    if display:
+                                        display.print_status(
+                                            "   📥 Fetching pull requests from GitHub...",
+                                            "info",
+                                        )
+
+                                    # Load commits that were just fetched from cache
+                                    with cache.get_session() as session:
+                                        from gitflow_analytics.models.database import CachedCommit
+
+                                        cached_commits = (
+                                            session.query(CachedCommit)
+                                            .filter(
+                                                CachedCommit.repo_path == str(repo_path),
+                                                CachedCommit.timestamp >= start_date,
+                                                CachedCommit.timestamp <= end_date,
+                                            )
+                                            .all()
+                                        )
+
+                                        # Convert to dict format for enrichment
+                                        commits_for_enrichment = []
+                                        for cached_commit in cached_commits:
+                                            commit_dict = {
+                                                "hash": cached_commit.commit_hash,
+                                                "author_name": cached_commit.author_name,
+                                                "author_email": cached_commit.author_email,
+                                                "date": cached_commit.timestamp,
+                                                "message": cached_commit.message,
+                                            }
+                                            commits_for_enrichment.append(commit_dict)
+
+                                    # Enrich with GitHub PR data
+                                    enrichment = orchestrator.enrich_repository_data(
+                                        repo_config, commits_for_enrichment, start_date
+                                    )
+
+                                    if enrichment["prs"]:
+                                        pr_count = len(enrichment["prs"])
+                                        if display:
+                                            display.print_status(
+                                                f"   ✅ Found {pr_count} pull requests",
+                                                "success",
+                                            )
+                                        else:
+                                            click.echo(f"   ✅ Found {pr_count} pull requests")
+
+                                except Exception as e:
+                                    logger.warning(
+                                        f"Failed to fetch PRs for {repo_config.github_repo}: {e}"
+                                    )
+                                    if display:
+                                        display.print_status(
+                                            f"   ⚠️  Could not fetch PRs: {e}",
+                                            "warning",
+                                        )
+                                    else:
+                                        click.echo(f"   ⚠️  Could not fetch PRs: {e}")
+
                             # Collect unique developers if available
                             if "developers" in result["stats"]:
                                 total_developers.update(result["stats"]["developers"])
@@ -2088,6 +2150,68 @@ def analyze(
                             total_commits += result["stats"]["total_commits"]
                             total_tickets += result["stats"]["unique_tickets"]
 
+                            # Fetch and enrich with GitHub PRs after data collection
+                            if repo_config.github_repo:
+                                try:
+                                    if display:
+                                        display.print_status(
+                                            "   📥 Fetching pull requests from GitHub...",
+                                            "info",
+                                        )
+
+                                    # Load commits that were just fetched from cache
+                                    with cache.get_session() as session:
+                                        from gitflow_analytics.models.database import CachedCommit
+
+                                        cached_commits = (
+                                            session.query(CachedCommit)
+                                            .filter(
+                                                CachedCommit.repo_path == str(repo_path),
+                                                CachedCommit.timestamp >= start_date,
+                                                CachedCommit.timestamp <= end_date,
+                                            )
+                                            .all()
+                                        )
+
+                                        # Convert to dict format for enrichment
+                                        commits_for_enrichment = []
+                                        for cached_commit in cached_commits:
+                                            commit_dict = {
+                                                "hash": cached_commit.commit_hash,
+                                                "author_name": cached_commit.author_name,
+                                                "author_email": cached_commit.author_email,
+                                                "date": cached_commit.timestamp,
+                                                "message": cached_commit.message,
+                                            }
+                                            commits_for_enrichment.append(commit_dict)
+
+                                    # Enrich with GitHub PR data
+                                    enrichment = orchestrator.enrich_repository_data(
+                                        repo_config, commits_for_enrichment, start_date
+                                    )
+
+                                    if enrichment["prs"]:
+                                        pr_count = len(enrichment["prs"])
+                                        if display:
+                                            display.print_status(
+                                                f"   ✅ Found {pr_count} pull requests",
+                                                "success",
+                                            )
+                                        else:
+                                            click.echo(f"   ✅ Found {pr_count} pull requests")
+
+                                except Exception as e:
+                                    logger.warning(
+                                        f"Failed to fetch PRs for {repo_config.github_repo}: {e}"
+                                    )
+                                    if display:
+                                        display.print_status(
+                                            f"   ⚠️  Could not fetch PRs: {e}",
+                                            "warning",
+                                        )
+                                    else:
+                                        click.echo(f"   ⚠️  Could not fetch PRs: {e}")
+
                             # Collect unique developers if available
                             if "developers" in result["stats"]:
                                 total_developers.update(result["stats"]["developers"])

{gitflow_analytics-3.13.5 → gitflow_analytics-3.13.6}/src/gitflow_analytics/core/git_auth.py

@@ -112,6 +112,80 @@ def setup_git_credentials(token: str, username: str = "git") -> bool:
         return False
 
 
+def ensure_remote_url_has_token(repo_path: Path, token: str) -> bool:
+    """Embed GitHub token in remote URL for HTTPS authentication.
+
+    This is needed because subprocess git operations may not have access
+    to the credential helper store due to environment variable restrictions
+    (GIT_CREDENTIAL_HELPER="" and GIT_ASKPASS="/bin/echo" in git_timeout_wrapper).
+
+    Args:
+        repo_path: Path to the git repository
+        token: GitHub personal access token
+
+    Returns:
+        True if URL was updated with token, False if already has token,
+        not applicable (SSH URL), or operation failed
+    """
+    if not token:
+        logger.debug("No token provided, skipping remote URL update")
+        return False
+
+    try:
+        # Get current origin remote URL
+        result = subprocess.run(
+            ["git", "remote", "get-url", "origin"],
+            cwd=repo_path,
+            capture_output=True,
+            text=True,
+            check=True,
+        )
+        current_url = result.stdout.strip()
+
+        if not current_url:
+            logger.debug(f"No origin remote found for {repo_path}")
+            return False
+
+        # Check if it's an HTTPS GitHub URL without embedded token
+        if current_url.startswith("https://github.com/"):
+            # URL format: https://github.com/org/repo.git
+            # New format: https://git:TOKEN@github.com/org/repo.git
+            new_url = current_url.replace("https://github.com/", f"https://git:{token}@github.com/")
+
+            # Update the remote URL
+            subprocess.run(
+                ["git", "remote", "set-url", "origin", new_url],
+                cwd=repo_path,
+                capture_output=True,
+                text=True,
+                check=True,
+            )
+            logger.debug(f"Updated remote URL with embedded token for {repo_path.name}")
+            return True
+
+        elif "@github.com" in current_url:
+            # Already has authentication embedded (either token or SSH)
+            logger.debug(f"Remote URL already has authentication for {repo_path.name}")
+            return False
+
+        elif current_url.startswith("git@github.com:"):
+            # SSH URL, no need to modify
+            logger.debug(f"Using SSH authentication for {repo_path.name}")
+            return False
+
+        else:
+            # Unknown URL format
+            logger.debug(f"Unknown URL format for {repo_path.name}: {current_url}")
+            return False
+
+    except subprocess.CalledProcessError as e:
+        logger.warning(f"Could not update remote URL for {repo_path.name}: {e.stderr}")
+        return False
+    except Exception as e:
+        logger.warning(f"Unexpected error updating remote URL for {repo_path.name}: {e}")
+        return False
+
+
 def preflight_git_authentication(config: dict) -> bool:
     """Run pre-flight checks for git authentication and setup credentials.
 

{gitflow_analytics-3.13.5 → gitflow_analytics-3.13.6}/src/gitflow_analytics/core/git_timeout_wrapper.py

@@ -14,6 +14,7 @@ from pathlib import Path
 from typing import Callable, Optional, TypeVar
 
 from ..constants import Timeouts
+from .git_auth import ensure_remote_url_has_token
 
 logger = logging.getLogger(__name__)
 
@@ -208,6 +209,13 @@ class GitTimeoutWrapper:
             True if fetch succeeded, False otherwise
         """
         try:
+            # Embed GitHub token in remote URL if available
+            # This is necessary because git operations run with GIT_CREDENTIAL_HELPER=""
+            # and GIT_ASKPASS="/bin/echo", which disable credential helpers
+            token = os.environ.get("GITHUB_TOKEN")
+            if token:
+                ensure_remote_url_has_token(repo_path, token)
+
             self.run_git_command(
                 ["git", "fetch", "--all"], cwd=repo_path, timeout=timeout, check=True
             )

{gitflow_analytics-3.13.5 → gitflow_analytics-3.13.6}/src/gitflow_analytics.egg-info/PKG-INFO

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

{gitflow_analytics-3.13.5 → gitflow_analytics-3.13.6}/tests/test_reports.py

@@ -259,7 +259,6 @@ class TestNarrativeReportGenerator:
 
         pr_metrics = {"avg_pr_size": 150, "total_prs": 10}
 
-
         result_path = generator.generate_narrative_report(
             commits,
             prs,