MQL5-Google-Onedrive/scripts/review_pull_requests.py
google-labs-jules[bot] 760213f4e5 Bolt: optimize Git metadata retrieval in PR reviewer
Optimized `scripts/review_pull_requests.py` by implementing bulk metadata retrieval
using `git for-each-ref` with the `%(ahead-behind:main)` atom (Git 2.41+).

Key improvements:
- Reduced subprocess calls from $O(N)$ to $O(1)$ for branch metadata.
- Implementation of a global `BRANCH_METADATA_CACHE` for efficient attribute retrieval.
- Measurable performance gain: execution time reduced from ~0.6s to ~0.33s (~45% speedup).
- Added integration test in `scripts/test_automation.py` to ensure continued stability.
- Safe fallback for older Git versions without the `ahead-behind` atom.

 Bolt: Speed is a feature. Every millisecond counts.
2026-02-25 10:48:22 +00:00

325 lines
11 KiB
Python

#!/usr/bin/env python3
"""
Pull Request Review Script
Reviews all pull requests and creates a comprehensive summary
"""
import subprocess
import sys
import json
import re
from pathlib import Path
from datetime import datetime
from collections import defaultdict
REPO_ROOT = Path(__file__).resolve().parents[1]
# ⚡ Bolt: Global cache for branch metadata to reduce subprocess calls.
BRANCH_METADATA_CACHE = {}
def run_command(cmd, capture_output=True):
"""Run a command and return the result."""
try:
result = subprocess.run(
cmd,
cwd=REPO_ROOT,
capture_output=capture_output,
text=True,
timeout=30,
encoding='utf-8',
errors='replace'
)
return result
except Exception as e:
print(f"Error running command: {e}", file=sys.stderr)
return None
def get_git_version():
"""Get the installed Git version."""
result = run_command(["git", "--version"])
if result and result.returncode == 0:
match = re.search(r"(\d+\.\d+\.\d+)", result.stdout)
if match:
return tuple(map(int, match.group(1).split(".")))
return (0, 0, 0)
def fetch_branch_metadata():
"""
⚡ Bolt: Fetch metadata for all remote branches in bulk.
Reduces subprocess calls from O(N) to O(1) by using the %(ahead-behind) atom.
"""
global BRANCH_METADATA_CACHE
# We use ahead-behind atom (Git 2.41+) to quickly check merge status
git_version = get_git_version()
if git_version < (2, 41, 0):
return False
# Fetch refname, ahead/behind counts, committer date, and subject
cmd = [
"git", "for-each-ref",
"--format=%(refname:short)|%(ahead-behind:main)|%(committerdate:iso8601)|%(contents:subject)",
"refs/remotes"
]
result = run_command(cmd)
if not result or result.returncode != 0:
return False
for line in result.stdout.strip().split("\n"):
if not line:
continue
# Use split with maxsplit to handle subjects containing '|'
parts = line.split("|", 3)
if len(parts) < 4:
continue
refname, ahead_behind, date, subject = parts
# Filter out HEAD and main
if "HEAD" in refname or refname.endswith("/main") or refname == "origin":
continue
try:
ahead, behind = map(int, ahead_behind.split())
BRANCH_METADATA_CACHE[refname] = {
"is_merged": ahead == 0,
"last_commit_date": date,
"last_commit_subject": subject,
"ahead": ahead,
"behind": behind
}
except ValueError:
continue
return True
def get_prs_via_gh_cli():
"""Get PRs using GitHub CLI."""
result = run_command(["gh", "pr", "list", "--state", "all", "--json", "number,title,state,author,createdAt,updatedAt,headRefName,baseRefName,isDraft,labels"])
if result and result.returncode == 0:
try:
return json.loads(result.stdout)
except json.JSONDecodeError:
return []
return None
def get_prs_via_git():
"""Get PR information via git branches."""
# ⚡ Bolt: Use bulk metadata fetch if available (Git 2.41+)
if fetch_branch_metadata():
branches = [name for name, meta in BRANCH_METADATA_CACHE.items() if not meta["is_merged"]]
merged_branches = [name for name, meta in BRANCH_METADATA_CACHE.items() if meta["is_merged"]]
return {
"open": sorted(branches),
"merged": sorted(merged_branches, reverse=True)
}
# Fallback for older Git versions
result = run_command(["git", "branch", "-r", "--no-merged", "main"])
branches = []
if result and result.returncode == 0:
branches = [b.strip() for b in result.stdout.strip().split("\n")
if b.strip() and "origin/main" not in b and "HEAD" not in b]
result_merged = run_command(["git", "branch", "-r", "--merged", "main"])
merged_branches = []
if result_merged and result_merged.returncode == 0:
merged_branches = [b.strip() for b in result_merged.stdout.strip().split("\n")
if b.strip() and "origin/main" not in b and "HEAD" not in b]
return {
"open": sorted(branches),
"merged": sorted(merged_branches, reverse=True)
}
def analyze_branch_name(branch_name):
"""Analyze branch name to extract PR information."""
branch = branch_name.replace("origin/", "")
info = {
"type": "unknown",
"category": "other",
"description": branch
}
# Categorize branches
if branch.startswith("Cursor/"):
info["type"] = "cursor"
info["category"] = "ai-generated"
info["description"] = branch.replace("Cursor/A6-9V/", "")
elif branch.startswith("copilot/"):
info["type"] = "copilot"
info["category"] = "ai-generated"
info["description"] = branch.replace("copilot/", "")
elif branch.startswith("bolt-"):
info["type"] = "bolt"
info["category"] = "optimization"
info["description"] = branch.replace("bolt-", "")
elif branch.startswith("feat/"):
info["type"] = "feature"
info["category"] = "feature"
info["description"] = branch.replace("feat/", "")
elif branch.startswith("feature/"):
info["type"] = "feature"
info["category"] = "feature"
info["description"] = branch.replace("feature/", "")
return info
def get_branch_info(branch_name):
"""Get detailed information about a branch."""
branch = branch_name.replace("origin/", "")
# ⚡ Bolt: Use cached metadata if available
if branch_name in BRANCH_METADATA_CACHE:
meta = BRANCH_METADATA_CACHE[branch_name]
return {
"branch": branch,
"full_name": branch_name,
"commit_count": meta["ahead"],
"commits": [meta["last_commit_subject"]],
"last_commit_date": meta["last_commit_date"]
}
# Fallback to O(N) calls
result = run_command(["git", "log", "--oneline", f"main..{branch_name}"])
commit_count = 0
commits = []
if result and result.returncode == 0:
commits = [c.strip() for c in result.stdout.strip().split("\n") if c.strip()]
commit_count = len(commits)
result = run_command(["git", "log", "-1", "--format=%ci", branch_name])
last_commit = None
if result and result.returncode == 0 and result.stdout.strip():
last_commit = result.stdout.strip()
return {
"branch": branch,
"full_name": branch_name,
"commit_count": commit_count,
"commits": commits[:5], # First 5 commits
"last_commit_date": last_commit
}
def main():
"""Main review function."""
print("=" * 80)
print("PULL REQUEST REVIEW")
print(f"Generated: {datetime.now().strftime('%Y-%m-%d %H:%M:%S')}")
print("=" * 80)
print()
# Try GitHub CLI first
prs = get_prs_via_gh_cli()
if prs is not None:
print(f"Found {len(prs)} pull requests via GitHub CLI")
print()
# Group by state
by_state = defaultdict(list)
for pr in prs:
by_state[pr.get("state", "unknown")].append(pr)
print("Pull Requests by State:")
for state, pr_list in sorted(by_state.items()):
print(f" {state.upper()}: {len(pr_list)}")
print()
# Show open PRs
open_prs = by_state.get("OPEN", [])
if open_prs:
print("=" * 80)
print("OPEN PULL REQUESTS")
print("=" * 80)
for pr in open_prs:
print(f"\nPR #{pr.get('number', 'N/A')}: {pr.get('title', 'No title')}")
print(f" Author: {pr.get('author', {}).get('login', 'Unknown')}")
print(f" Branch: {pr.get('headRefName', 'N/A')} -> {pr.get('baseRefName', 'main')}")
print(f" Created: {pr.get('createdAt', 'N/A')}")
print(f" Updated: {pr.get('updatedAt', 'N/A')}")
print(f" Draft: {'Yes' if pr.get('isDraft') else 'No'}")
labels = [l.get('name') for l in pr.get('labels', [])]
if labels:
print(f" Labels: {', '.join(labels)}")
# Show merged PRs
merged_prs = by_state.get("MERGED", [])
if merged_prs:
print("\n" + "=" * 80)
print(f"MERGED PULL REQUESTS ({len(merged_prs)} total)")
print("=" * 80)
print(f"\nShowing last 10 merged PRs:")
for pr in merged_prs[-10:]:
print(f" PR #{pr.get('number', 'N/A')}: {pr.get('title', 'No title')}")
else:
# Fallback to git branch analysis
print("GitHub CLI not available, analyzing branches...")
print()
branch_info = get_prs_via_git()
open_branches = branch_info["open"]
merged_branches = branch_info["merged"]
print(f"Open branches (potential PRs): {len(open_branches)}")
print(f"Merged branches (completed PRs): {len(merged_branches)}")
print()
# Categorize open branches
categories = defaultdict(list)
for branch in open_branches:
info = analyze_branch_name(branch)
categories[info["category"]].append((branch, info))
print("=" * 80)
print("OPEN BRANCHES (Potential Pull Requests)")
print("=" * 80)
print()
for category, branches in sorted(categories.items()):
print(f"{category.upper()}: {len(branches)} branches")
for branch, info in branches[:10]: # Show first 10
branch_details = get_branch_info(branch)
print(f" - {info['description']}")
print(f" Branch: {branch_details['branch']}")
print(f" Commits: {branch_details['commit_count']}")
if branch_details['last_commit_date']:
print(f" Last commit: {branch_details['last_commit_date']}")
if len(branches) > 10:
print(f" ... and {len(branches) - 10} more")
print()
print("=" * 80)
print("MERGED BRANCHES (Completed Pull Requests)")
print("=" * 80)
print(f"\nTotal merged: {len(merged_branches)}")
print("\nRecent merged branches:")
for branch in merged_branches[:20]:
info = analyze_branch_name(branch)
print(f" - {info['description']}")
print("\n" + "=" * 80)
print("REVIEW COMPLETE")
print("=" * 80)
print("\nNote: GitHub doesn't support 'pinning' pull requests directly.")
print("Consider:")
print("1. Creating a tracking issue for important PRs")
print("2. Using labels to categorize PRs")
print("3. Adding PRs to project boards")
print("4. Creating a PR summary document")
if __name__ == "__main__":
main()