ultralytics-actions 0.1.5__tar.gz → 0.1.6__tar.gz

{ultralytics_actions-0.1.5 → ultralytics_actions-0.1.6}/PKG-INFO

@@ -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 → ultralytics_actions-0.1.6}/actions/__init__.py

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

{ultralytics_actions-0.1.5 → ultralytics_actions-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"

{ultralytics_actions-0.1.5 → ultralytics_actions-0.1.6}/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."""

{ultralytics_actions-0.1.5 → ultralytics_actions-0.1.6}/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 → ultralytics_actions-0.1.6}/tests/test_github_utils.py

@@ -15,6 +15,7 @@ def test_check_pypi_version():
         with patch("requests.get") as mock_get:
             mock_response = MagicMock()
             mock_response.status_code = 200
+            mock_response.elapsed.total_seconds.return_value = 0.5
             mock_response.json.return_value = {"info": {"version": "0.9.0"}}
             mock_get.return_value = mock_response
 
@@ -43,6 +44,7 @@ def test_action_request_methods():
     with patch("requests.get") as mock_get:
         mock_response = MagicMock()
         mock_response.status_code = 200
+        mock_response.elapsed.total_seconds.return_value = 0.3
         mock_get.return_value = mock_response
 
         action = Action(token="test-token")

{ultralytics_actions-0.1.5 → ultralytics_actions-0.1.6}/tests/test_openai_utils.py

@@ -28,6 +28,7 @@ def test_get_completion(mock_post):
     # Setup mock response with Responses API structure
     mock_response = MagicMock()
     mock_response.status_code = 200
+    mock_response.elapsed.total_seconds.return_value = 1.5
     mock_response.json.return_value = {
         "output": [
             {
@@ -57,6 +58,7 @@ def test_get_completion_with_link_check(mock_check_links, mock_post):
     # Setup mocks with Responses API structure
     mock_response = MagicMock()
     mock_response.status_code = 200
+    mock_response.elapsed.total_seconds.return_value = 2.0
     mock_response.json.return_value = {
         "output": [
             {

{ultralytics_actions-0.1.5 → ultralytics_actions-0.1.6}/ultralytics_actions.egg-info/PKG-INFO

@@ -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>