|
| 1 | +import argparse |
| 2 | +import json |
| 3 | +import os |
| 4 | +import random |
| 5 | +import subprocess |
| 6 | +import sys |
| 7 | +from pathlib import Path |
| 8 | + |
| 9 | + |
| 10 | +def get_pr_changed_files(pr_number: str) -> list[str]: |
| 11 | + """Get files changed in PR using GitHub CLI (more reliable than git diff)""" |
| 12 | + result = subprocess.run( |
| 13 | + [ |
| 14 | + "gh", "pr", "view", pr_number, "--json", "files", "--jq", |
| 15 | + ".files[].path" |
| 16 | + ], |
| 17 | + capture_output=True, |
| 18 | + text=True, |
| 19 | + check=True, |
| 20 | + ) |
| 21 | + return [line.strip() for line in result.stdout.splitlines() if line.strip()] |
| 22 | + |
| 23 | + |
| 24 | +def get_existing_reviewers(pr_number: str) -> tuple[set[str], set[str]]: |
| 25 | + """Get currently assigned reviewers (users and teams) for a PR""" |
| 26 | + try: |
| 27 | + # Get user reviewers |
| 28 | + user_result = subprocess.run( |
| 29 | + [ |
| 30 | + "gh", "pr", "view", pr_number, "--json", "reviewRequests", |
| 31 | + "--jq", |
| 32 | + "(.reviewRequests // []) | .[] | select(.login) | .login" |
| 33 | + ], |
| 34 | + capture_output=True, |
| 35 | + text=True, |
| 36 | + check=True, |
| 37 | + ) |
| 38 | + user_reviewers = { |
| 39 | + line.strip() |
| 40 | + for line in user_result.stdout.splitlines() if line.strip() |
| 41 | + } |
| 42 | + |
| 43 | + # Get team reviewers |
| 44 | + team_result = subprocess.run( |
| 45 | + [ |
| 46 | + "gh", "pr", "view", pr_number, "--json", "reviewRequests", |
| 47 | + "--jq", "(.reviewRequests // []) | .[] | select(.name) | .name" |
| 48 | + ], |
| 49 | + capture_output=True, |
| 50 | + text=True, |
| 51 | + check=True, |
| 52 | + ) |
| 53 | + team_reviewers = { |
| 54 | + line.strip() |
| 55 | + for line in team_result.stdout.splitlines() if line.strip() |
| 56 | + } |
| 57 | + |
| 58 | + return user_reviewers, team_reviewers |
| 59 | + except subprocess.CalledProcessError as e: |
| 60 | + print(f"Warning: Could not fetch existing reviewers: {e}") |
| 61 | + return set(), set() |
| 62 | + |
| 63 | + |
| 64 | +def load_json(path: str): |
| 65 | + with open(path, "r", encoding="utf-8") as f: |
| 66 | + return json.load(f) |
| 67 | + |
| 68 | + |
| 69 | +def map_modules(changed_files: list[str], module_paths: dict[str, |
| 70 | + str]) -> set[str]: |
| 71 | + modules: set[str] = set() |
| 72 | + for file in changed_files: |
| 73 | + for prefix, module in module_paths.items(): |
| 74 | + if file.startswith(prefix): |
| 75 | + modules.add(module) |
| 76 | + break |
| 77 | + return modules |
| 78 | + |
| 79 | + |
| 80 | +def gather_reviewers(modules: set[str], |
| 81 | + module_owners: dict[str, list[str]], |
| 82 | + *, |
| 83 | + pr_author: str | None = None, |
| 84 | + existing_reviewers: set[str] | None = None) -> list[str]: |
| 85 | + reviewers: set[str] = set() |
| 86 | + for module in modules: |
| 87 | + reviewers.update(module_owners.get(module, [])) |
| 88 | + |
| 89 | + if pr_author: |
| 90 | + reviewers.discard(pr_author) |
| 91 | + |
| 92 | + # Remove existing reviewers to avoid duplicate assignments |
| 93 | + if existing_reviewers: |
| 94 | + reviewers -= existing_reviewers |
| 95 | + |
| 96 | + return sorted(reviewers) |
| 97 | + |
| 98 | + |
| 99 | +def main() -> None: |
| 100 | + parser = argparse.ArgumentParser( |
| 101 | + description="Assign reviewers based on changed modules") |
| 102 | + parser.add_argument("--dry-run", |
| 103 | + action="store_true", |
| 104 | + help="Print the gh command instead of executing") |
| 105 | + parser.add_argument( |
| 106 | + "--force-assign", |
| 107 | + action="store_true", |
| 108 | + help= |
| 109 | + "Assign reviewers even if some already exist (default: only assign if no reviewers)" |
| 110 | + ) |
| 111 | + args = parser.parse_args() |
| 112 | + |
| 113 | + pr_number = os.environ["PR_NUMBER"] |
| 114 | + reviewer_limit = int(os.environ.get("REVIEWER_LIMIT", "0")) |
| 115 | + pr_author = os.environ.get("PR_AUTHOR") |
| 116 | + |
| 117 | + print(f"Testing PR #{pr_number} with author: {pr_author}") |
| 118 | + |
| 119 | + # Check existing reviewers |
| 120 | + existing_user_reviewers, existing_team_reviewers = get_existing_reviewers( |
| 121 | + pr_number) |
| 122 | + total_existing = len(existing_user_reviewers) + len(existing_team_reviewers) |
| 123 | + |
| 124 | + print(f"Existing user reviewers: {sorted(existing_user_reviewers)}") |
| 125 | + print(f"Existing team reviewers: {sorted(existing_team_reviewers)}") |
| 126 | + |
| 127 | + # Skip assignment if reviewers already exist (unless forced) |
| 128 | + if total_existing > 0 and not args.force_assign: |
| 129 | + print( |
| 130 | + f"✅ PR already has {total_existing} reviewer(s) assigned. Skipping auto-assignment." |
| 131 | + ) |
| 132 | + print(" Use --force-assign to assign additional reviewers.") |
| 133 | + return |
| 134 | + |
| 135 | + try: |
| 136 | + changed_files = get_pr_changed_files(pr_number) |
| 137 | + print(f"Changed files: {changed_files}") |
| 138 | + |
| 139 | + module_paths = load_json(Path(".github") / "module-paths.json") |
| 140 | + module_owners = load_json( |
| 141 | + Path(".github/workflows") / "module-owners.json") |
| 142 | + |
| 143 | + modules = map_modules(changed_files, module_paths) |
| 144 | + reviewers = gather_reviewers( |
| 145 | + modules, |
| 146 | + module_owners, |
| 147 | + pr_author=pr_author, |
| 148 | + existing_reviewers= |
| 149 | + existing_user_reviewers # Avoid re-assigning existing users |
| 150 | + ) |
| 151 | + |
| 152 | + if reviewer_limit and len(reviewers) > reviewer_limit: |
| 153 | + reviewers = random.sample(reviewers, reviewer_limit) |
| 154 | + |
| 155 | + print(f"Changed modules: {sorted(modules)}") |
| 156 | + print(f"Potential reviewers: {reviewers}") |
| 157 | + |
| 158 | + if reviewers: |
| 159 | + cmd = ["gh", "pr", "edit", pr_number] |
| 160 | + for reviewer in reviewers: |
| 161 | + cmd.extend(["--add-reviewer", reviewer]) |
| 162 | + |
| 163 | + if args.dry_run: |
| 164 | + print(f"🔍 DRY RUN: {' '.join(cmd)}") |
| 165 | + else: |
| 166 | + try: |
| 167 | + subprocess.run(cmd, check=True) |
| 168 | + print( |
| 169 | + f"✅ Successfully assigned {len(reviewers)} new reviewer(s)" |
| 170 | + ) |
| 171 | + except subprocess.CalledProcessError as e: |
| 172 | + print(f"❌ Failed to add reviewers: {e}", file=sys.stderr) |
| 173 | + print( |
| 174 | + " This might be due to permissions or invalid usernames" |
| 175 | + ) |
| 176 | + sys.exit(1) |
| 177 | + else: |
| 178 | + print("✅ No new reviewers to assign") |
| 179 | + |
| 180 | + except subprocess.CalledProcessError as e: |
| 181 | + print(f"❌ Error processing PR: {e}", file=sys.stderr) |
| 182 | + sys.exit(1) |
| 183 | + |
| 184 | + |
| 185 | +if __name__ == "__main__": |
| 186 | + main() |
0 commit comments