@openhands/extensions 0.1.0 → 0.2.0

package/plugins/issue-duplicate-checker/scripts/auto_close_duplicate_issues.py

@@ -0,0 +1,569 @@
+#!/usr/bin/env python3
+from __future__ import annotations
+
+import argparse
+import json
+import os
+import re
+import sys
+import urllib.error
+import urllib.parse
+import urllib.request
+from datetime import UTC, datetime, timedelta
+from typing import Any
+
+
+GITHUB_API_BASE_URL = os.environ.get("GITHUB_API_BASE_URL", "https://api.github.com")
+MAX_PAGES = 100
+DUPLICATE_CANDIDATE_LABEL = "duplicate-candidate"
+DUPLICATE_VETO_MARKER = "<!-- openhands-duplicate-veto -->"
+AUTOMATION_BOT_LOGINS = {"all-hands-bot"}
+REPOSITORY_PATTERN = re.compile(r"^[a-zA-Z0-9_.-]+/[a-zA-Z0-9_.-]+$")
+DUPLICATE_MARKER_RE = re.compile(
+    r"<!-- openhands-duplicate-check canonical=(?P<canonical>\d+) "
+    r"auto-close=(?P<auto_close>true|false) -->"
+)
+
+
+class HTTPError(RuntimeError):
+    def __init__(self, method: str, path: str, status_code: int, body: str) -> None:
+        self.method = method
+        self.path = path
+        self.status_code = status_code
+        self.body = body
+        super().__init__(f"{method} {path} failed with HTTP {status_code}: {body}")
+
+
+def parse_args() -> argparse.Namespace:
+    parser = argparse.ArgumentParser(
+        description="Auto-close issues previously flagged as duplicate candidates."
+    )
+    parser.add_argument("--repository", required=True)
+    parser.add_argument("--close-after-days", type=int, default=3)
+    parser.add_argument("--dry-run", action="store_true")
+    args = parser.parse_args()
+    if not REPOSITORY_PATTERN.fullmatch(args.repository):
+        parser.error(f"Invalid repository format: {args.repository}")
+    return args
+
+
+def github_headers() -> dict[str, str]:
+    token = os.environ.get("GITHUB_TOKEN")
+    if not token:
+        raise RuntimeError("GITHUB_TOKEN environment variable is required")
+    return {
+        "Authorization": f"Bearer {token}",
+        "Accept": "application/vnd.github+json",
+        "User-Agent": "openhands-duplicate-auto-close",
+        "X-GitHub-Api-Version": "2022-11-28",
+    }
+
+
+def request_json(
+    path: str,
+    *,
+    method: str = "GET",
+    body: dict[str, Any] | None = None,
+) -> Any:
+    request_body = None
+    headers = github_headers()
+    if body is not None:
+        request_body = json.dumps(body).encode("utf-8")
+        headers["Content-Type"] = "application/json"
+
+    request = urllib.request.Request(
+        f"{GITHUB_API_BASE_URL}{path}",
+        data=request_body,
+        headers=headers,
+        method=method,
+    )
+    try:
+        with urllib.request.urlopen(request, timeout=60) as response:
+            payload = response.read().decode("utf-8")
+    except urllib.error.HTTPError as exc:
+        error_body = exc.read().decode("utf-8", errors="replace")
+        raise HTTPError(method, path, exc.code, error_body) from exc
+    except urllib.error.URLError as exc:
+        raise RuntimeError(f"{method} {path} failed: {exc}") from exc
+
+    if not payload:
+        return None
+    try:
+        return json.loads(payload)
+    except json.JSONDecodeError as exc:
+        raise RuntimeError(f"Failed to parse JSON from {path}: {exc}") from exc
+
+
+def parse_timestamp(value: str) -> datetime:
+    try:
+        return datetime.fromisoformat(value.replace("Z", "+00:00"))
+    except ValueError as exc:
+        raise ValueError(f"Failed to parse timestamp {value!r}: {exc}") from exc
+
+
+def ensure_page_limit(page: int, resource_name: str) -> None:
+    if page > MAX_PAGES:
+        raise RuntimeError(f"Exceeded pagination limit while listing {resource_name}")
+
+
+def list_open_issues(repository: str) -> list[dict[str, Any]]:
+    issues: list[dict[str, Any]] = []
+    page = 1
+    label_query = urllib.parse.quote(DUPLICATE_CANDIDATE_LABEL)
+    while True:
+        ensure_page_limit(page, f"open issues for {repository}")
+        payload = request_json(
+            f"/repos/{repository}/issues?state=open&labels={label_query}&per_page=100&page={page}"
+        )
+        if not isinstance(payload, list):
+            raise RuntimeError(
+                f"Expected list response while listing open issues for {repository}, "
+                f"got {type(payload).__name__}"
+            )
+        if not payload:
+            return issues
+        for issue in payload:
+            if issue.get("pull_request"):
+                continue
+            issues.append(issue)
+        page += 1
+
+
+def list_issue_comments(repository: str, issue_number: int) -> list[dict[str, Any]]:
+    comments: list[dict[str, Any]] = []
+    page = 1
+    while True:
+        ensure_page_limit(page, f"comments for issue #{issue_number}")
+        payload = request_json(
+            f"/repos/{repository}/issues/{issue_number}/comments?per_page=100&page={page}"
+        )
+        if not isinstance(payload, list):
+            raise RuntimeError(
+                "Expected list response while listing comments for issue "
+                f"#{issue_number}, got {type(payload).__name__}"
+            )
+        if not payload:
+            return comments
+        comments.extend(payload)
+        page += 1
+
+
+def list_comment_reactions(repository: str, comment_id: int) -> list[dict[str, Any]]:
+    reactions: list[dict[str, Any]] = []
+    page = 1
+    while True:
+        ensure_page_limit(page, f"reactions for comment {comment_id}")
+        payload = request_json(
+            f"/repos/{repository}/issues/comments/{comment_id}/reactions?per_page=100&page={page}"
+        )
+        if not isinstance(payload, list):
+            raise RuntimeError(
+                "Expected list response while listing reactions for comment "
+                f"{comment_id}, got {type(payload).__name__}"
+            )
+        if not payload:
+            return reactions
+        reactions.extend(payload)
+        page += 1
+
+
+def extract_duplicate_metadata(comment_body: str) -> tuple[int | None, bool]:
+    match = DUPLICATE_MARKER_RE.search(comment_body)
+    if not match:
+        return None, False
+    return int(match.group("canonical")), match.group("auto_close") == "true"
+
+
+def find_latest_auto_close_comment(
+    comments: list[dict[str, Any]],
+) -> tuple[dict[str, Any] | None, int | None]:
+    latest_comment: dict[str, Any] | None = None
+    latest_canonical_issue: int | None = None
+    latest_created_at: str | None = None
+    for comment in comments:
+        canonical_issue, auto_close = extract_duplicate_metadata(
+            comment.get("body") or ""
+        )
+        if canonical_issue is None or not auto_close:
+            continue
+        comment_created_at = comment.get("created_at")
+        if not isinstance(comment_created_at, str):
+            comment_created_at = None
+        if latest_comment is None:
+            latest_comment = comment
+            latest_canonical_issue = canonical_issue
+            latest_created_at = comment_created_at
+            continue
+        if comment_created_at is None:
+            continue
+        if latest_created_at is not None:
+            try:
+                if parse_timestamp(comment_created_at) < parse_timestamp(
+                    latest_created_at
+                ):
+                    continue
+            except ValueError:
+                continue
+        latest_comment = comment
+        latest_canonical_issue = canonical_issue
+        latest_created_at = comment_created_at
+    return latest_comment, latest_canonical_issue
+
+
+def issue_has_label(issue: dict[str, Any], label_name: str) -> bool:
+    labels = issue.get("labels") or []
+    for label in labels:
+        if label == label_name:
+            return True
+        if isinstance(label, dict) and label.get("name") == label_name:
+            return True
+    return False
+
+
+def user_id_from_item(item: dict[str, Any]) -> int | None:
+    user = item.get("user")
+    if not isinstance(user, dict):
+        return None
+    user_id = user.get("id")
+    return user_id if isinstance(user_id, int) else None
+
+
+def has_reaction_from_user(
+    reactions: list[dict[str, Any]], user_id: int | None, content: str
+) -> bool:
+    if user_id is None:
+        return False
+    return any(
+        user_id_from_item(reaction) == user_id and reaction.get("content") == content
+        for reaction in reactions
+    )
+
+
+def has_veto_note(comments: list[dict[str, Any]]) -> bool:
+    return any(
+        DUPLICATE_VETO_MARKER in (comment.get("body") or "") for comment in comments
+    )
+
+
+def is_non_bot_comment(comment: dict[str, Any]) -> bool:
+    if user_id_from_item(comment) is None:
+        return False
+    user = comment.get("user")
+    if not isinstance(user, dict):
+        return False
+    login = user.get("login")
+    if not isinstance(login, str):
+        return False
+    login = login.lower()
+    return (
+        user.get("type") != "Bot"
+        and not login.endswith("[bot]")
+        and login not in AUTOMATION_BOT_LOGINS
+    )
+
+
+def remove_candidate_label(
+    repository: str, issue_number: int, *, dry_run: bool
+) -> bool:
+    if dry_run:
+        return True
+    try:
+        request_json(
+            f"/repos/{repository}/issues/{issue_number}/labels/{DUPLICATE_CANDIDATE_LABEL}",
+            method="DELETE",
+        )
+    except HTTPError as exc:
+        if exc.status_code == 404:
+            return False
+        raise
+    return True
+
+
+def post_veto_note(repository: str, issue_number: int, *, dry_run: bool) -> bool:
+    if dry_run:
+        return True
+    request_json(
+        f"/repos/{repository}/issues/{issue_number}/comments",
+        method="POST",
+        body={
+            "body": (
+                "Thanks — leaving this open and removing the "
+                f"{DUPLICATE_CANDIDATE_LABEL} label.\n\n"
+                f"{DUPLICATE_VETO_MARKER}\n"
+                "_This comment was created by an AI assistant "
+                "(OpenHands) on behalf of the repository maintainer._"
+            )
+        },
+    )
+    return True
+
+
+def post_newer_activity_note(repository: str, issue_number: int, *, dry_run: bool) -> bool:
+    if dry_run:
+        return True
+    request_json(
+        f"/repos/{repository}/issues/{issue_number}/comments",
+        method="POST",
+        body={
+            "body": (
+                "Further activity was detected after the duplicate notice, so "
+                "this issue is being left open and the "
+                f"{DUPLICATE_CANDIDATE_LABEL} label is being removed.\n\n"
+                "_This comment was created by an AI assistant "
+                "(OpenHands) on behalf of the repository maintainer._"
+            )
+        },
+    )
+    return True
+
+
+def fetch_issue(repository: str, issue_number: int) -> dict[str, Any] | None:
+    try:
+        payload = request_json(f"/repos/{repository}/issues/{issue_number}")
+    except HTTPError as exc:
+        if exc.status_code == 404:
+            return None
+        raise
+    if not isinstance(payload, dict):
+        raise RuntimeError(
+            f"Expected issue object while fetching #{issue_number}, "
+            f"got {type(payload).__name__}"
+        )
+    return payload
+
+
+def is_open_canonical_issue(repository: str, issue_number: int) -> bool:
+    issue = fetch_issue(repository, issue_number)
+    if issue is None or issue.get("pull_request"):
+        return False
+    return issue.get("state") == "open" and not issue.get("locked")
+
+
+def close_issue_as_duplicate(
+    repository: str,
+    issue_number: int,
+    canonical_issue_number: int,
+    *,
+    dry_run: bool,
+) -> None:
+    if dry_run:
+        return
+
+    request_json(
+        f"/repos/{repository}/issues/{issue_number}/comments",
+        method="POST",
+        body={
+            "body": (
+                "This issue is being closed as a duplicate of "
+                f"#{canonical_issue_number}.\n\n"
+                "If this is incorrect, please add a comment and it can be "
+                "reopened.\n\n"
+                "_This comment was created by an AI assistant "
+                "(OpenHands) on behalf of the repository maintainer._"
+            )
+        },
+    )
+    request_json(
+        f"/repos/{repository}/issues/{issue_number}",
+        method="PATCH",
+        body={"state": "closed", "state_reason": "duplicate"},
+    )
+    remove_candidate_label(repository, issue_number, dry_run=dry_run)
+
+
+def keep_open_due_to_newer_comments(
+    repository: str,
+    issue: dict[str, Any],
+    issue_number: int,
+    *,
+    dry_run: bool,
+) -> dict[str, Any]:
+    label_removed = False
+    if issue_has_label(issue, DUPLICATE_CANDIDATE_LABEL):
+        label_removed = remove_candidate_label(
+            repository,
+            issue_number,
+            dry_run=dry_run,
+        )
+    activity_note_posted = post_newer_activity_note(
+        repository,
+        issue_number,
+        dry_run=dry_run,
+    )
+    return {
+        "issue_number": issue_number,
+        "action": "kept-open",
+        "reason": "newer-comment-after-duplicate-notice",
+        "label_removed": label_removed,
+        "activity_note_posted": activity_note_posted,
+    }
+
+
+def main() -> int:
+    args = parse_args()
+    now = datetime.now(UTC)
+    cutoff = now - timedelta(days=args.close_after_days)
+
+    summary: list[dict[str, Any]] = []
+    for issue in list_open_issues(args.repository):
+        issue_number = issue.get("number")
+        if issue_number is None:
+            continue
+        try:
+            issue_number = int(issue_number)
+        except (TypeError, ValueError):
+            continue
+
+        try:
+            comments = list_issue_comments(args.repository, issue_number)
+            latest_comment, canonical_issue_number = find_latest_auto_close_comment(
+                comments
+            )
+            if latest_comment is None or canonical_issue_number is None:
+                continue
+
+            comment_created_at_str = latest_comment.get("created_at")
+            comment_id = latest_comment.get("id")
+            if not comment_created_at_str or comment_id is None:
+                continue
+            try:
+                comment_id = int(comment_id)
+            except (TypeError, ValueError):
+                continue
+            try:
+                comment_created_at = parse_timestamp(comment_created_at_str)
+            except ValueError as exc:
+                print(
+                    "Warning: Skipping issue "
+                    f"#{issue_number} due to invalid duplicate-comment timestamp: "
+                    f"{exc}",
+                    file=sys.stderr,
+                )
+                continue
+            if comment_created_at > cutoff:
+                continue
+
+            author_id = user_id_from_item(issue)
+            reactions = list_comment_reactions(args.repository, comment_id)
+            author_thumbs_down = has_reaction_from_user(reactions, author_id, "-1")
+            author_thumbs_up = has_reaction_from_user(reactions, author_id, "+1")
+            if author_thumbs_down:
+                label_removed = False
+                if issue_has_label(issue, DUPLICATE_CANDIDATE_LABEL):
+                    label_removed = remove_candidate_label(
+                        args.repository,
+                        issue_number,
+                        dry_run=args.dry_run,
+                    )
+                veto_note_posted = False
+                if not has_veto_note(comments):
+                    veto_note_posted = post_veto_note(
+                        args.repository,
+                        issue_number,
+                        dry_run=args.dry_run,
+                    )
+                summary.append(
+                    {
+                        "issue_number": issue_number,
+                        "action": "kept-open",
+                        "reason": "author-thumbed-down-duplicate-comment",
+                        "label_removed": label_removed,
+                        "veto_note_posted": veto_note_posted,
+                        "author_thumbs_up": author_thumbs_up,
+                    }
+                )
+                continue
+
+            newer_comments = []
+            for comment in comments:
+                created_at = comment.get("created_at")
+                if not created_at or not is_non_bot_comment(comment):
+                    continue
+                try:
+                    newer_comment_created_at = parse_timestamp(created_at)
+                except ValueError as exc:
+                    print(
+                        "Warning: Ignoring newer comment with invalid timestamp on "
+                        f"issue #{issue_number}: {exc}",
+                        file=sys.stderr,
+                    )
+                    continue
+                if newer_comment_created_at > comment_created_at:
+                    newer_comments.append(comment)
+            if newer_comments:
+                summary.append(
+                    keep_open_due_to_newer_comments(
+                        args.repository,
+                        issue,
+                        issue_number,
+                        dry_run=args.dry_run,
+                    )
+                )
+                continue
+
+            if canonical_issue_number == issue_number:
+                summary.append(
+                    {
+                        "issue_number": issue_number,
+                        "action": "kept-open",
+                        "reason": "self-duplicate-marker",
+                        "canonical_issue_number": canonical_issue_number,
+                    }
+                )
+                continue
+
+            if not is_open_canonical_issue(args.repository, canonical_issue_number):
+                label_removed = False
+                if issue_has_label(issue, DUPLICATE_CANDIDATE_LABEL):
+                    label_removed = remove_candidate_label(
+                        args.repository,
+                        issue_number,
+                        dry_run=args.dry_run,
+                    )
+                summary.append(
+                    {
+                        "issue_number": issue_number,
+                        "action": "kept-open",
+                        "reason": "canonical-issue-not-open",
+                        "canonical_issue_number": canonical_issue_number,
+                        "label_removed": label_removed,
+                    }
+                )
+                continue
+
+            close_issue_as_duplicate(
+                args.repository,
+                issue_number,
+                canonical_issue_number,
+                dry_run=args.dry_run,
+            )
+            summary.append(
+                {
+                    "issue_number": issue_number,
+                    "action": "closed-as-duplicate"
+                    if not args.dry_run
+                    else "would-close-as-duplicate",
+                    "canonical_issue_number": canonical_issue_number,
+                    "author_thumbs_up": author_thumbs_up,
+                }
+            )
+        except RuntimeError as exc:
+            print(f"Error processing issue #{issue_number}: {exc}", file=sys.stderr)
+            summary.append(
+                {
+                    "issue_number": issue_number,
+                    "action": "failed",
+                    "error": str(exc),
+                }
+            )
+
+    print(json.dumps({"repository": args.repository, "results": summary}, indent=2))
+    return 0
+
+
+if __name__ == "__main__":
+    try:
+        raise SystemExit(main())
+    except Exception as exc:  # noqa: BLE001
+        print(f"error: {exc}", file=sys.stderr)
+        raise