ultralytics-actions 0.1.1__py3-none-any.whl → 0.1.2__py3-none-any.whl

actions/__init__.py

@@ -23,4 +23,4 @@
 #     ├── test_summarize_pr.py
 #     └── ...
 
-__version__ = "0.1.1"
+__version__ = "0.1.2"

actions/first_interaction.py

@@ -176,6 +176,9 @@ YOUR {issue_type.upper()} RESPONSE:
 def main(*args, **kwargs):
     """Executes auto-labeling and custom response generation for new GitHub issues, PRs, and discussions."""
     event = Action(*args, **kwargs)
+    if event.should_skip_openai():
+        return
+
     number, node_id, title, body, username, issue_type, action = get_event_content(event)
     available_labels = event.get_repo_data("labels")
     label_descriptions = {label["name"]: label.get("description", "") for label in available_labels}

actions/review_pr.py

@@ -90,7 +90,7 @@ def generate_pr_review(repository: str, diff_text: str, pr_title: str, pr_descri
     content = (
         "You are an expert code reviewer for Ultralytics. Provide detailed inline comments on specific code changes.\n\n"
         "Focus on: Code quality, style, best practices, bugs, edge cases, error handling, performance, security, documentation, test coverage\n\n"
-        "FORMATTING: Use backticks for code, file names, branch names, function names, variable names, packages\n\n"
+        "FORMATTING: Use backticks for all summary and suggestion code, files, branches, functions, variables, packages, e.g. `x=3`\n\n"
         "CRITICAL RULES:\n"
         "1. Quality over quantity: Zero comments is fine for clean code - only flag truly important issues\n"
         f"2. {comment_guidance} - these are maximums, not targets\n"
@@ -113,7 +113,7 @@ def generate_pr_review(repository: str, diff_text: str, pr_title: str, pr_descri
         "- Suggestions replace ONLY the single line at 'line' - for multi-line fixes, describe the change in 'message' instead\n"
         "- Do NOT provide 'start_line' when including a 'suggestion' - suggestions are always single-line only\n"
         "- Suggestion content must match the exact indentation of the original line\n"
-        "- Never include triple backticks (```) in suggestions as they break markdown formatting\n"
+        "- Avoid triple backticks (```) in suggestions as they break markdown formatting\n"
         "- It's better to flag an issue without a suggestion than provide a wrong or uncertain fix\n\n"
         "Return JSON: "
         '{"comments": [{"file": "exact/path", "line": N, "severity": "HIGH", "message": "...", "suggestion": "..."}], "summary": "..."}\n\n'
@@ -236,9 +236,7 @@ def post_review_summary(event: Action, review_data: dict, review_number: int) ->
 
     review_title = f"{REVIEW_MARKER} {review_number}" if review_number > 1 else REVIEW_MARKER
     comments = review_data.get("comments", [])
-    event_type = (
-        "REQUEST_CHANGES" if any(c.get("severity") not in ["LOW", "SUGGESTION", None] for c in comments) else "APPROVE"
-    )
+    event_type = "COMMENT" if any(c.get("severity") not in ["LOW", "SUGGESTION", None] for c in comments) else "APPROVE"
 
     body = (
         f"## {review_title}\n\n"
@@ -313,7 +311,8 @@ def main(*args, **kwargs):
     review_number = dismiss_previous_reviews(event)
 
     diff = event.get_pr_diff()
-    review = generate_pr_review(event.repository, diff, event.pr.get("title", ""), event.pr.get("body", ""))
+    pr_description = event._pr_summary_cache or event.pr.get("body", "")
+    review = generate_pr_review(event.repository, diff, event.pr.get("title", ""), pr_description)
 
     post_review_summary(event, review, review_number)
     print("PR review completed")

actions/summarize_pr.py

@@ -96,10 +96,11 @@ def main(*args, **kwargs):
     """Summarize a pull request and update its description with a summary."""
     event = Action(*args, **kwargs)
     action = event.event_data.get("action")
-
     if action == "opened":
         print("Skipping PR open - handled by first_interaction.py with unified API call")
         return
+    if event.should_skip_openai():
+        return
 
     print(f"Retrieving diff for PR {event.pr['number']}")
     diff = event.get_pr_diff()

actions/summarize_release.py

@@ -19,7 +19,7 @@ def get_release_diff(event, previous_tag: str, latest_tag: str) -> str:
     """Retrieves the differences between two specified Git tags in a GitHub repository."""
     url = f"{GITHUB_API_URL}/repos/{event.repository}/compare/{previous_tag}...{latest_tag}"
     r = event.get(url, headers=event.headers_diff)
-    return r.text if r.status_code == 200 else f"Failed to get diff: {r.content}"
+    return r.text if r.status_code == 200 else f"Failed to get diff: {r.text}"
 
 
 def get_prs_between_tags(event, previous_tag: str, latest_tag: str) -> list:
@@ -42,8 +42,8 @@ def get_prs_between_tags(event, previous_tag: str, latest_tag: str) -> list:
         pr_numbers.update(pr_matches)
 
     prs = []
-    time.sleep(10)  # Allow final PR summary to update on merge
     for pr_number in sorted(pr_numbers):  # earliest to latest
+        time.sleep(1)  # Rate limit: GitHub search API has strict limits
         pr_url = f"{GITHUB_API_URL}/repos/{event.repository}/pulls/{pr_number}"
         pr_response = event.get(pr_url)
         if pr_response.status_code == 200:
@@ -52,7 +52,7 @@ def get_prs_between_tags(event, previous_tag: str, latest_tag: str) -> list:
                 {
                     "number": pr_data["number"],
                     "title": pr_data["title"],
-                    "body": remove_html_comments(pr_data["body"]),
+                    "body": remove_html_comments(pr_data.get("body", "")),
                     "author": pr_data["user"]["login"],
                     "html_url": pr_data["html_url"],
                     "merged_at": pr_data["merged_at"],
@@ -140,7 +140,7 @@ def generate_release_summary(
         },
         {
             "role": "user",
-            "content": f"Summarize the updates made in the '{latest_tag}' tag, focusing on major model or features changes, their purpose, and potential impact. Keep the summary clear and suitable for a broad audience. Add emojis to enliven the summary. Prioritize changes from the current PR (the first in the list), which is usually the most important in the release. Reply directly with a summary along these example guidelines, though feel free to adjust as appropriate:\n\n"
+            "content": f"Summarize the updates made in the '{latest_tag}' tag, focusing on major model or features changes, their purpose, and potential impact. Keep the summary clear and suitable for a broad audience. Add emojis to enliven the summary. Prioritize changes from the current PR (the last in the list), which is usually the most important in the release. Reply directly with a summary along these example guidelines, though feel free to adjust as appropriate:\n\n"
             f"## 🌟 Summary (single-line synopsis)\n"
             f"## 📊 Key Changes (bullet points highlighting any major changes)\n"
             f"## 🎯 Purpose & Impact (bullet points explaining any benefits and potential impact to users)\n\n\n"

actions/utils/github_utils.py

@@ -114,6 +114,7 @@ class Action:
         self.verbose = verbose
         self.eyes_reaction_id = None
         self._pr_diff_cache = None
+        self._pr_summary_cache = None
         self._username_cache = None
         self._default_status = {
             "get": [200],
@@ -187,6 +188,22 @@ class Action:
         """Checks if a user is a member of the organization."""
         return self.get(f"{GITHUB_API_URL}/orgs/{self.repository.split('/')[0]}/members/{username}").status_code == 204
 
+    def is_fork_pr(self) -> bool:
+        """Checks if PR is from a fork (different repo than base)."""
+        if not self.pr:
+            return False
+        head_repo = self.pr.get("head", {}).get("repo", {}).get("full_name")
+        return bool(head_repo) and head_repo != self.repository
+
+    def should_skip_openai(self) -> bool:
+        """Check if OpenAI operations should be skipped."""
+        from actions.utils.openai_utils import OPENAI_API_KEY
+
+        if not OPENAI_API_KEY:
+            print("⚠️ Skipping LLM operations (OPENAI_API_KEY not found)")
+            return True
+        return False
+
     def get_pr_diff(self) -> str:
         """Retrieves the diff content for a specified pull request with caching."""
         if self._pr_diff_cache:
@@ -256,6 +273,7 @@ class Action:
             updated_description = description + "\n\n" + new_summary
 
         self.patch(url, json={"body": updated_description})
+        self._pr_summary_cache = new_summary
 
     def get_label_ids(self, labels: list[str]) -> list[str]:
         """Retrieves GitHub label IDs for a list of label names using the GraphQL API."""

{ultralytics_actions-0.1.1.dist-info → ultralytics_actions-0.1.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ultralytics-actions
-Version: 0.1.1
+Version: 0.1.2
 Summary: Ultralytics Actions for GitHub automation and PR management.
 Author-email: Glenn Jocher <glenn.jocher@ultralytics.com>
 Maintainer-email: Ultralytics <hello@ultralytics.com>

ultralytics_actions-0.1.2.dist-info/RECORD

@@ -0,0 +1,19 @@
+actions/__init__.py,sha256=TJd-NrNbVN3UJfNKfPjHUjsNhC7dbGDea80Ig0A1IVk,772
+actions/dispatch_actions.py,sha256=8jaaVkA_LSlpUQ4tuzmQtf2kw3G09uVRD_LmJyXYKNE,4215
+actions/first_interaction.py,sha256=X0eUAov3HcdNPyNa6tpl3ttbfiBhJZndlyy_LGDmsqU,9244
+actions/review_pr.py,sha256=P9ONfDxIE3w7PcTcgBhatTfxb2L_ghqxQjk6UEUipHs,15485
+actions/summarize_pr.py,sha256=XLYsNTf4J6VPyyecZcuiJaSBDgjDSWFj37v5vb1ATCA,5773
+actions/summarize_release.py,sha256=_067Q5AP-Zdnt_qzhHaCuGCr7T4MXSB5_N-M5GX6qgQ,9081
+actions/update_file_headers.py,sha256=E5fKYLdeW16-BHCcuqxohGpGZqgEh-WX4ZmCQJw2R90,6684
+actions/update_markdown_code_blocks.py,sha256=w3DTRltg2Rmr4-qrNawv_S2vJbheKE0tne1iz79FzXg,8692
+actions/utils/__init__.py,sha256=sKNx6o5jcAraEdGFph0o-YC7dMMY-dg_FprIBa6Jydw,1027
+actions/utils/common_utils.py,sha256=8ZmgaXZU3J2sg-HSaldp3hHYq7bI3akcJHdIXPmcNAo,11908
+actions/utils/github_utils.py,sha256=voWQbn6eeqvhDceVfokHypSydlmmNPsUOuisAqGJ1CQ,18963
+actions/utils/openai_utils.py,sha256=91ohkSlldVZt-icYcK7ZLfgQfnMbf4mTWT3nRnQ1QG4,10486
+actions/utils/version_utils.py,sha256=EIbm3iZVNyNl3dh8aNz_9ITeTC93ZxfyUzIRkO3tSXw,3242
+ultralytics_actions-0.1.2.dist-info/licenses/LICENSE,sha256=hIahDEOTzuHCU5J2nd07LWwkLW7Hko4UFO__ffsvB-8,34523
+ultralytics_actions-0.1.2.dist-info/METADATA,sha256=uZQWWi4EX-5Cpu_H8VdWxyzFzG7MmearTFZsVTVQjz4,12389
+ultralytics_actions-0.1.2.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+ultralytics_actions-0.1.2.dist-info/entry_points.txt,sha256=n_VbDs3Xj33daaeN_2D72UTEuyeH8hVc6-CPH55ymkY,496
+ultralytics_actions-0.1.2.dist-info/top_level.txt,sha256=5apM5x80QlJcGbACn1v3fkmIuL1-XQCKcItJre7w7Tw,8
+ultralytics_actions-0.1.2.dist-info/RECORD,,

ultralytics_actions-0.1.1.dist-info/RECORD

@@ -1,19 +0,0 @@
-actions/__init__.py,sha256=Sxxh83BO8x10c7LkJcNoSTK6sE8e4KtnqLV9qZsp_pA,772
-actions/dispatch_actions.py,sha256=8jaaVkA_LSlpUQ4tuzmQtf2kw3G09uVRD_LmJyXYKNE,4215
-actions/first_interaction.py,sha256=BkiMKppSeZ3VOP9jK0Q7FwqUYivFM4LIjJia9KgqNco,9193
-actions/review_pr.py,sha256=O9EfC8qNbfwq1FgFkVVFse1YTUKYyLIbo9d3Hf9luu4,15434
-actions/summarize_pr.py,sha256=K7o6RQQxUjsWpP4FgH7Nrgp4Qbvnprk8C01PasX_2vE,5724
-actions/summarize_release.py,sha256=bIAqBbjq4eCDBAV3MWb8yq1hSK93MEyazoN29kT1-js,9068
-actions/update_file_headers.py,sha256=E5fKYLdeW16-BHCcuqxohGpGZqgEh-WX4ZmCQJw2R90,6684
-actions/update_markdown_code_blocks.py,sha256=w3DTRltg2Rmr4-qrNawv_S2vJbheKE0tne1iz79FzXg,8692
-actions/utils/__init__.py,sha256=sKNx6o5jcAraEdGFph0o-YC7dMMY-dg_FprIBa6Jydw,1027
-actions/utils/common_utils.py,sha256=8ZmgaXZU3J2sg-HSaldp3hHYq7bI3akcJHdIXPmcNAo,11908
-actions/utils/github_utils.py,sha256=WpvVQIWScIB_mZxWSZ_zGun-oW90bI15eKqVCDLusJs,18264
-actions/utils/openai_utils.py,sha256=91ohkSlldVZt-icYcK7ZLfgQfnMbf4mTWT3nRnQ1QG4,10486
-actions/utils/version_utils.py,sha256=EIbm3iZVNyNl3dh8aNz_9ITeTC93ZxfyUzIRkO3tSXw,3242
-ultralytics_actions-0.1.1.dist-info/licenses/LICENSE,sha256=hIahDEOTzuHCU5J2nd07LWwkLW7Hko4UFO__ffsvB-8,34523
-ultralytics_actions-0.1.1.dist-info/METADATA,sha256=iDfUTlwy7koo7KAtTHwkrvYMSRRqJHzr67RJEBI4nus,12389
-ultralytics_actions-0.1.1.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-ultralytics_actions-0.1.1.dist-info/entry_points.txt,sha256=n_VbDs3Xj33daaeN_2D72UTEuyeH8hVc6-CPH55ymkY,496
-ultralytics_actions-0.1.1.dist-info/top_level.txt,sha256=5apM5x80QlJcGbACn1v3fkmIuL1-XQCKcItJre7w7Tw,8
-ultralytics_actions-0.1.1.dist-info/RECORD,,