|
1 | | -# reviewbot/utils.py |
| 1 | +import os |
| 2 | +import requests |
| 3 | +import json |
| 4 | + |
| 5 | +def get_pr_number(): |
| 6 | + event_path = os.getenv("GITHUB_EVENT_PATH") |
| 7 | + if not event_path: |
| 8 | + raise Exception("GITHUB_EVENT_PATH not set") |
| 9 | + |
| 10 | + with open(event_path, 'r') as f: |
| 11 | + event = json.load(f) |
| 12 | + |
| 13 | + return event["pull_request"]["number"] |
2 | 14 |
|
3 | 15 | def get_diff(): |
4 | | - # In real setup, parse `git diff` or PR files |
5 | | - return "def hello(name):\n print('Hello ' + name)" |
| 16 | + repo = os.environ.get("GITHUB_REPOSITORY") # e.g. "owner/repo" |
| 17 | + pr_number = get_pr_number() |
| 18 | + |
| 19 | + token = os.environ.get("GITHUB_TOKEN") |
| 20 | + if not token: |
| 21 | + raise Exception("Missing GITHUB_TOKEN environment variable.") |
| 22 | + |
| 23 | + headers = { |
| 24 | + "Authorization": f"Bearer {token}", |
| 25 | + "Accept": "application/vnd.github.v3+json", |
| 26 | + } |
| 27 | + |
| 28 | + url = f"https://api.github.com/repos/{repo}/pulls/{pr_number}/files" |
| 29 | + |
| 30 | + diffs = [] |
| 31 | + page = 1 |
| 32 | + while True: |
| 33 | + response = requests.get(url, headers=headers, params={"page": page, "per_page": 100}) |
| 34 | + response.raise_for_status() |
| 35 | + files = response.json() |
| 36 | + if not files: |
| 37 | + break |
| 38 | + |
| 39 | + for f in files: |
| 40 | + patch = f.get("patch") |
| 41 | + if patch: |
| 42 | + diffs.append(f"File: {f['filename']}\n{patch}") |
| 43 | + |
| 44 | + page += 1 |
| 45 | + |
| 46 | + return "\n\n".join(diffs) |
| 47 | + |
| 48 | +def post_pr_comment(body: str): |
| 49 | + repo = os.getenv("GITHUB_REPOSITORY") |
| 50 | + token = os.getenv("GITHUB_TOKEN") |
| 51 | + pr_number = get_pr_number() |
| 52 | + |
| 53 | + url = f"https://api.github.com/repos/{repo}/issues/{pr_number}/comments" |
| 54 | + headers = { |
| 55 | + "Authorization": f"Bearer {token}", |
| 56 | + "Accept": "application/vnd.github+json" |
| 57 | + } |
| 58 | + payload = { |
| 59 | + "body": body |
| 60 | + } |
6 | 61 |
|
7 | | -def fake_llm_review(diff): |
8 | | - if "print" in diff: |
9 | | - return "Consider using logging instead of print for better production code quality." |
10 | | - return "No issues found." |
| 62 | + response = requests.post(url, headers=headers, json=payload) |
| 63 | + print("GitHub API response:", response.status_code, response.text) |
| 64 | + response.raise_for_status() |
0 commit comments