ultralytics-actions 0.1.5__py3-none-any.whl → 0.1.6__py3-none-any.whl

actions/__init__.py

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

actions/review_pr.py

@@ -8,6 +8,7 @@ import re
 from .utils import GITHUB_API_URL, MAX_PROMPT_CHARS, Action, get_completion, remove_html_comments
 
 REVIEW_MARKER = "🔍 PR Review"
+ERROR_MARKER = "⚠️ Review generation encountered an error"
 EMOJI_MAP = {"CRITICAL": "❗", "HIGH": "⚠️", "MEDIUM": "💡", "LOW": "📝", "SUGGESTION": "💭"}
 SKIP_PATTERNS = [
     r"\.lock$",  # Lock files
@@ -179,7 +180,7 @@ def generate_pr_review(repository: str, diff_text: str, pr_title: str, pr_descri
         error_details = traceback.format_exc()
         print(f"Review generation failed: {e}\n{error_details}")
         summary = (
-            f"⚠️ Review generation encountered an error: `{type(e).__name__}`\n\n"
+            f"{ERROR_MARKER}: `{type(e).__name__}`\n\n"
             f"<details><summary>Debug Info</summary>\n\n```\n{error_details}\n```\n</details>"
         )
         return {"comments": [], "summary": summary}
@@ -219,7 +220,29 @@ 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 = "COMMENT" if any(c.get("severity") not in ["LOW", "SUGGESTION", None] for c in comments) else "APPROVE"
+    summary = review_data.get("summary") or ""
+
+    # Don't approve if error occurred or if there are critical/high severity issues
+    has_error = not summary or ERROR_MARKER in summary
+    has_issues = any(c.get("severity") not in ["LOW", "SUGGESTION", None] for c in comments)
+    requests_changes = any(
+        phrase in summary.lower()
+        for phrase in [
+            "please",
+            "should",
+            "must",
+            "need to",
+            "needs to",
+            "before merging",
+            "fix",
+            "error",
+            "issue",
+            "problem",
+            "warning",
+            "concern",
+        ]
+    )
+    event_type = "COMMENT" if (has_error or has_issues or requests_changes) else "APPROVE"
 
     body = (
         f"## {review_title}\n\n"

actions/utils/github_utils.py

@@ -125,21 +125,22 @@ class Action:
 
     def _request(self, method: str, url: str, headers=None, expected_status=None, hard=False, **kwargs):
         """Unified request handler with error checking."""
-        response = getattr(requests, method)(url, headers=headers or self.headers, **kwargs)
+        r = getattr(requests, method)(url, headers=headers or self.headers, **kwargs)
         expected = expected_status or self._default_status[method]
-        success = response.status_code in expected
+        success = r.status_code in expected
 
         if self.verbose:
-            print(f"{'✓' if success else '✗'} {method.upper()} {url} → {response.status_code}")
+            elapsed = r.elapsed.total_seconds()
+            print(f"{'✓' if success else '✗'} {method.upper()} {url} → {r.status_code} ({elapsed:.1f}s)")
             if not success:
                 try:
-                    print(f"  ❌ Error: {response.json().get('message', 'Unknown error')}")
+                    print(f"  ❌ Error: {r.json().get('message', 'Unknown error')}")
                 except Exception:
-                    print(f"  ❌ Error: {response.text[:200]}")
+                    print(f"  ❌ Error: {r.text[:200]}")
 
         if not success and hard:
-            response.raise_for_status()
-        return response
+            r.raise_for_status()
+        return r
 
     def get(self, url, **kwargs):
         """Performs GET request with error handling."""

actions/utils/openai_utils.py

@@ -125,6 +125,8 @@ def get_completion(
 
         try:
             r = requests.post(url, json=data, headers=headers, timeout=600)
+            success = r.status_code == 200
+            print(f"{'✓' if success else '✗'} POST {url} → {r.status_code} ({r.elapsed.total_seconds():.1f}s)")
             r.raise_for_status()
 
             # Parse response

{ultralytics_actions-0.1.5.dist-info → ultralytics_actions-0.1.6.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ultralytics-actions
-Version: 0.1.5
+Version: 0.1.6
 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.5.dist-info → ultralytics_actions-0.1.6.dist-info}/RECORD

@@ -1,19 +1,19 @@
-actions/__init__.py,sha256=KUWLDPR8kn8FVsWMHHQd4WLv3RUhNUMbEhW9NeYzxNY,772
+actions/__init__.py,sha256=owzYNIk3pmwSP7At9iZuEgZyKtiaKSazh22WkbEyfFo,772
 actions/dispatch_actions.py,sha256=i81UeHrYudAsOUFUfN71u6X-1cmZaZaiiTj6p2rvz8A,4217
 actions/first_interaction.py,sha256=QxPsLjd-m2G-QYOcQb2hQfIB_alupzeZzSHTk-jw0bg,9856
-actions/review_pr.py,sha256=b9QJ380XPU90odJ7cnIOOfMIJUcuoxLZYRpeuu91SmU,14405
+actions/review_pr.py,sha256=_x-HwX2iqR1xhQkJBDW5CPVq-4Cba7pZZBz63hN9e1Y,15060
 actions/summarize_pr.py,sha256=3nFotiZX42dz-mzDQ9wcoUILJKkcaxrC5EeyxvuvY60,5775
 actions/summarize_release.py,sha256=iCXa9a1DcOrDVe8pMWEsYKgDxuIOhIgMsYymElOLK6o,9083
 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=unjXYIFNFeHrdC8LooDFVWlj6fAdGhssUgASo5229zY,1073
 actions/utils/common_utils.py,sha256=2DRvcyCgmn507w3T4FJcQSZNI9KC1gVUb8CnJqPapD0,11943
-actions/utils/github_utils.py,sha256=_JCasKaWwxwk5DAFr-ljdkphGRMfvHClfJTkemSrc2I,19686
-actions/utils/openai_utils.py,sha256=lRRLtI0FuOJLAu7S28Ol86aX_diWi5-deiHOZtcnHhY,10497
+actions/utils/github_utils.py,sha256=cBgEDJBpImTJbGBoZTteVSmCqXPuzEb51np7gRhqPeM,19702
+actions/utils/openai_utils.py,sha256=xI_DZpsEBzXyqQDozMLEtmjwuNlOpNL9n2b-gA6xL5Y,10658
 actions/utils/version_utils.py,sha256=EIbm3iZVNyNl3dh8aNz_9ITeTC93ZxfyUzIRkO3tSXw,3242
-ultralytics_actions-0.1.5.dist-info/licenses/LICENSE,sha256=hIahDEOTzuHCU5J2nd07LWwkLW7Hko4UFO__ffsvB-8,34523
-ultralytics_actions-0.1.5.dist-info/METADATA,sha256=aVXhKequ2mloiCNMVmrg01JPSQ3c9k4jCwltIu4mqhs,12368
-ultralytics_actions-0.1.5.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-ultralytics_actions-0.1.5.dist-info/entry_points.txt,sha256=n_VbDs3Xj33daaeN_2D72UTEuyeH8hVc6-CPH55ymkY,496
-ultralytics_actions-0.1.5.dist-info/top_level.txt,sha256=5apM5x80QlJcGbACn1v3fkmIuL1-XQCKcItJre7w7Tw,8
-ultralytics_actions-0.1.5.dist-info/RECORD,,
+ultralytics_actions-0.1.6.dist-info/licenses/LICENSE,sha256=hIahDEOTzuHCU5J2nd07LWwkLW7Hko4UFO__ffsvB-8,34523
+ultralytics_actions-0.1.6.dist-info/METADATA,sha256=7Zj67iXEIw61XyAKY8Myu0vZYqsO7F_08XvcDstqhWI,12368
+ultralytics_actions-0.1.6.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+ultralytics_actions-0.1.6.dist-info/entry_points.txt,sha256=n_VbDs3Xj33daaeN_2D72UTEuyeH8hVc6-CPH55ymkY,496
+ultralytics_actions-0.1.6.dist-info/top_level.txt,sha256=5apM5x80QlJcGbACn1v3fkmIuL1-XQCKcItJre7w7Tw,8
+ultralytics_actions-0.1.6.dist-info/RECORD,,