ultralytics-actions 0.0.47__py3-none-any.whl → 0.0.49__py3-none-any.whl

actions/__init__.py

@@ -22,4 +22,4 @@
 #     ├── test_summarize_pr.py
 #     └── ...
 
-__version__ = "0.0.47"
+__version__ = "0.0.49"

actions/summarize_pr.py

@@ -16,29 +16,29 @@ SUMMARY_START = (
 )
 
 
-def generate_merge_message(pr_summary=None, pr_credit=None):
-    """Generates a thank-you message for merged PR contributors."""
+def generate_merge_message(pr_summary=None, pr_credit=None, pr_url=None):
+    """Generates a motivating thank-you message for merged PR contributors."""
     messages = [
         {
             "role": "system",
-            "content": "You are an Ultralytics AI assistant. Generate meaningful, inspiring messages to GitHub users.",
+            "content": "You are an Ultralytics AI assistant. Generate inspiring, appreciative messages for GitHub contributors.",
         },
         {
             "role": "user",
-            "content": f"Write a friendly thank you for a merged GitHub PR by {pr_credit}. "
-            f"Context from PR:\n{pr_summary}\n\n"
-            f"Start with the exciting message that this PR is now merged, and weave in an inspiring but obscure quote "
-            f"from a historical figure in science, art, stoicism and philosophy. "
-            f"Keep the message concise yet relevant to the specific contributions in this PR. "
-            f"We want the contributors to feel their effort is appreciated and will make a difference in the world.",
+            "content": (
+                f"Write a warm thank-you comment for the merged PR {pr_url} by {pr_credit}."
+                f"Context:\n{pr_summary}\n\n"
+                f"Start with an enthusiastic note about the merge, incorporate a relevant inspirational quote from a historical "
+                f"figure, and connect it to the PR’s impact. Keep it concise yet meaningful, ensuring contributors feel valued."
+            ),
         },
     ]
     return get_completion(messages)
 
 
-def post_merge_message(pr_number, repository, summary, pr_credit, headers):
+def post_merge_message(pr_number, pr_url, repository, summary, pr_credit, headers):
     """Posts thank you message on PR after merge."""
-    message = generate_merge_message(summary, pr_credit)
+    message = generate_merge_message(summary, pr_credit, pr_url)
     comment_url = f"{GITHUB_API_URL}/repos/{repository}/issues/{pr_number}/comments"
     response = requests.post(comment_url, json={"body": message}, headers=headers)
     return response.status_code == 201
@@ -94,9 +94,8 @@ def generate_pr_summary(repository, diff_text):
     return SUMMARY_START + reply
 
 
-def update_pr_description(repository, pr_number, new_summary, headers, max_retries=2):
+def update_pr_description(pr_url, new_summary, headers, max_retries=2):
     """Updates PR description with new summary, retrying if description is None."""
-    pr_url = f"{GITHUB_API_URL}/repos/{repository}/pulls/{pr_number}"
     description = ""
     for i in range(max_retries + 1):
         description = requests.get(pr_url, headers=headers).json().get("body") or ""
@@ -210,9 +209,10 @@ def remove_todos_on_merge(pr_number, repository, headers):
 def main(*args, **kwargs):
     """Summarize a pull request and update its description with a summary."""
     action = Action(*args, **kwargs)
-    pr_number = action.pr["number"]
     headers = action.headers
     repository = action.repository
+    pr_number = action.pr["number"]
+    pr_url = f"{GITHUB_API_URL}/repos/{repository}/pulls/{pr_number}"
 
     print(f"Retrieving diff for PR {pr_number}")
     diff = action.get_pr_diff()
@@ -223,7 +223,7 @@ def main(*args, **kwargs):
 
     # Update PR description
     print("Updating PR description...")
-    status_code = update_pr_description(repository, pr_number, summary, headers)
+    status_code = update_pr_description(pr_url, summary, headers)
     if status_code == 200:
         print("PR description updated successfully.")
     else:
@@ -237,7 +237,7 @@ def main(*args, **kwargs):
         remove_todos_on_merge(pr_number, repository, headers)
         if pr_credit:
             print("Posting PR author thank you message...")
-            post_merge_message(pr_number, repository, summary, pr_credit, headers)
+            post_merge_message(pr_number, pr_url, repository, summary, pr_credit, headers)
 
 
 if __name__ == "__main__":

actions/utils/common_utils.py

@@ -22,7 +22,18 @@ REQUESTS_HEADERS = {
     "Referer": "https://www.google.com/",
     "Origin": "https://www.google.com/",
 }
-
+BAD_HTTP_CODES = frozenset(
+    {
+        # 403,  # Forbidden - client lacks permission to access the resource (commented as works in browser typically)
+        404,  # Not Found - requested resource doesn't exist
+        405,  # Method Not Allowed - HTTP method not supported for this endpoint
+        410,  # Gone - resource permanently removed
+        500,  # Internal Server Error - server encountered an error
+        502,  # Bad Gateway - upstream server sent invalid response
+        503,  # Service Unavailable - server temporarily unable to handle request
+        504,  # Gateway Timeout - upstream server didn't respond in time
+    }
+)
 URL_IGNORE_LIST = frozenset(
     {
         "localhost",
@@ -44,7 +55,6 @@ URL_IGNORE_LIST = frozenset(
         "storage.googleapis.com",  # private GCS buckets
     }
 )
-
 URL_PATTERN = re.compile(
     r"\[([^]]+)]\(([^)]+)\)"  # Matches Markdown links [text](url)
     r"|"
@@ -85,7 +95,6 @@ def is_url(url, session=None, check=True, max_attempts=3, timeout=2):
 
         if check:
             requester = session or requests
-            bad_codes = {404, 410, 500, 502, 503, 504}
             kwargs = {"timeout": timeout, "allow_redirects": True}
             if not session:
                 kwargs["headers"] = REQUESTS_HEADERS
@@ -94,7 +103,7 @@ def is_url(url, session=None, check=True, max_attempts=3, timeout=2):
                 try:
                     # Try HEAD first, then GET if needed
                     for method in (requester.head, requester.get):
-                        if method(url, stream=method == requester.get, **kwargs).status_code not in bad_codes:
+                        if method(url, stream=method == requester.get, **kwargs).status_code not in BAD_HTTP_CODES:
                             return True
                     return False
                 except Exception:

{ultralytics_actions-0.0.47.dist-info → ultralytics_actions-0.0.49.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: ultralytics-actions
-Version: 0.0.47
+Version: 0.0.49
 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.0.47.dist-info → ultralytics_actions-0.0.49.dist-info}/RECORD

@@ -1,15 +1,15 @@
-actions/__init__.py,sha256=A3KLreATd7sSdPwF5hJy-yAjrcE0rtjVdaXz_g0_Yg0,742
+actions/__init__.py,sha256=YUeY18AoDEXKIMWCCr6xUi_stDUqF6_LCok9BxN8iQg,742
 actions/first_interaction.py,sha256=Oosw3GlXObF4RxYJG8haIjKXSV4MAx_uPXfMdRKY5qA,17851
-actions/summarize_pr.py,sha256=Pfcm6ArfZfcMme48FcCGNmMwrFFlsG0czifhXHDKAlY,11166
+actions/summarize_pr.py,sha256=dnjZcbGItvhpxq2yNiSWuiPBmZTHAW3ufu2upDgOT3I,11074
 actions/summarize_release.py,sha256=2D1IIeS4xrQNEJER3HItm5u9XvTL8hwGX_jWD2S8q1Y,8455
 actions/update_markdown_code_blocks.py,sha256=DN6rrDw2VHXUTetrrg1SHlYDco9iaGOfQBKFORqbCBI,6362
 actions/utils/__init__.py,sha256=WStdEAYROVnF0nubEOmrFLrejkRiMXIefA5O1ckfcFs,476
-actions/utils/common_utils.py,sha256=8Au56sY6toCv76WJ-8jbmW94BgE_2VXevQbt3XKDyVc,4826
+actions/utils/common_utils.py,sha256=o2dMSS7TxpwJLU58ybcZFgtaV-PM5lzQRP0B9cW-bBc,5420
 actions/utils/github_utils.py,sha256=0h0Hz2tgUta61Ymn9YggRXBZ7aZdF5krKnX7Tj9jqRU,7068
 actions/utils/openai_utils.py,sha256=U7DjxTdFGdhmWSE4_KIg1yPQdtrfG4GkbNZCgMw4_1Q,1822
-ultralytics_actions-0.0.47.dist-info/LICENSE,sha256=hIahDEOTzuHCU5J2nd07LWwkLW7Hko4UFO__ffsvB-8,34523
-ultralytics_actions-0.0.47.dist-info/METADATA,sha256=xcYxNklkUgrpYsvP5HkQRl21H_vIiQsQWmLW_YxRjeo,10561
-ultralytics_actions-0.0.47.dist-info/WHEEL,sha256=In9FTNxeP60KnTkGw7wk6mJPYd_dQSjEZmXdBdMCI-8,91
-ultralytics_actions-0.0.47.dist-info/entry_points.txt,sha256=GowvOFplj0C7JmsjbKcbpgLpdf2r921pcaOQkAHWZRA,378
-ultralytics_actions-0.0.47.dist-info/top_level.txt,sha256=5apM5x80QlJcGbACn1v3fkmIuL1-XQCKcItJre7w7Tw,8
-ultralytics_actions-0.0.47.dist-info/RECORD,,
+ultralytics_actions-0.0.49.dist-info/LICENSE,sha256=hIahDEOTzuHCU5J2nd07LWwkLW7Hko4UFO__ffsvB-8,34523
+ultralytics_actions-0.0.49.dist-info/METADATA,sha256=nw_Qne-ukAxDpCNRxbDRqN7jiTTIHNZqf5xxK8XxU3w,10561
+ultralytics_actions-0.0.49.dist-info/WHEEL,sha256=In9FTNxeP60KnTkGw7wk6mJPYd_dQSjEZmXdBdMCI-8,91
+ultralytics_actions-0.0.49.dist-info/entry_points.txt,sha256=GowvOFplj0C7JmsjbKcbpgLpdf2r921pcaOQkAHWZRA,378
+ultralytics_actions-0.0.49.dist-info/top_level.txt,sha256=5apM5x80QlJcGbACn1v3fkmIuL1-XQCKcItJre7w7Tw,8
+ultralytics_actions-0.0.49.dist-info/RECORD,,