ultralytics-actions 0.0.85__tar.gz → 0.0.87__tar.gz

{ultralytics_actions-0.0.85 → ultralytics_actions-0.0.87}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ultralytics-actions
-Version: 0.0.85
+Version: 0.0.87
 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.85 → ultralytics_actions-0.0.87}/actions/__init__.py

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

{ultralytics_actions-0.0.85 → ultralytics_actions-0.0.87}/actions/summarize_release.py

@@ -25,7 +25,15 @@ def get_prs_between_tags(event, previous_tag: str, latest_tag: str) -> list:
     url = f"{GITHUB_API_URL}/repos/{event.repository}/compare/{previous_tag}...{latest_tag}"
     r = event.get(url)
 
+    if r.status_code != 200:
+        print(f"Failed to get comparison between {previous_tag} and {latest_tag}, status: {r.status_code}")
+        return []
+
     data = r.json()
+    if "commits" not in data:
+        print(f"No commits found in comparison between {previous_tag} and {latest_tag}")
+        return []
+
     pr_numbers = set()
     for commit in data["commits"]:
         pr_matches = re.findall(r"#(\d+)", commit["commit"]["message"])
@@ -139,9 +147,9 @@ def create_github_release(event, tag_name: str, name: str, body: str):
     event.post(url, json=data)
 
 
-def get_previous_tag() -> str:
-    """Retrieves the previous Git tag, excluding the current tag, using the git describe command."""
-    cmd = ["git", "describe", "--tags", "--abbrev=0", "--exclude", CURRENT_TAG]
+def get_actual_previous_tag(current_tag: str) -> str:
+    """Gets the actual previous tag using git, excluding the current tag."""
+    cmd = ["git", "describe", "--tags", "--abbrev=0", "--exclude", current_tag]
     try:
         return subprocess.run(cmd, check=True, text=True, capture_output=True).stdout.strip()
     except subprocess.CalledProcessError:
@@ -156,7 +164,14 @@ def main(*args, **kwargs):
     if not all([event.token, CURRENT_TAG]):
         raise ValueError("One or more required environment variables are missing.")
 
-    previous_tag = PREVIOUS_TAG or get_previous_tag()
+    # Try PREVIOUS_TAG first, fall back to actual previous tag if comparison fails
+    previous_tag = PREVIOUS_TAG or get_actual_previous_tag(CURRENT_TAG)
+
+    # Test if the previous tag works for comparison
+    test_url = f"{GITHUB_API_URL}/repos/{event.repository}/compare/{previous_tag}...{CURRENT_TAG}"
+    if event.get(test_url).status_code != 200:
+        previous_tag = get_actual_previous_tag(CURRENT_TAG)
+        print(f"Using actual previous tag: {previous_tag}")
 
     # Get the diff between the tags
     diff = get_release_diff(event, previous_tag, CURRENT_TAG)

{ultralytics_actions-0.0.85 → ultralytics_actions-0.0.87}/actions/utils/common_utils.py

@@ -189,7 +189,8 @@ def is_url(url, session=None, check=True, max_attempts=3, timeout=3, return_url=
                     # Try HEAD first, then GET if needed
                     for method in (requester.head, requester.get):
                         response = method(url, stream=method == requester.get, **kwargs)
-                        if redirect and allow_redirect(start=url, end=response.url):
+                        # Only update URL if there were actual HTTP redirects (indicated by response.history)
+                        if redirect and response.history and allow_redirect(start=url, end=response.url):
                             url = response.url
                         if response.status_code not in BAD_HTTP_CODES:
                             return (True, url) if return_url else True

{ultralytics_actions-0.0.85 → ultralytics_actions-0.0.87}/tests/test_file_headers.py

@@ -5,8 +5,9 @@
 
 from pathlib import Path
 from tempfile import TemporaryDirectory
+from unittest.mock import patch
 
-from actions.update_file_headers import COMMENT_MAP, IGNORE_PATHS, update_file
+from actions.update_file_headers import COMMENT_MAP, IGNORE_PATHS, main, update_file
 
 
 def test_update_file_python():
@@ -112,3 +113,23 @@ def test_ignore_paths():
     assert isinstance(IGNORE_PATHS, set)
     assert ".git" in IGNORE_PATHS
     assert "__pycache__" in IGNORE_PATHS
+
+
+def test_main_real_files():
+    """Test main function on actual repository files."""
+    # Mock update_file to prevent actual file modifications during testing
+    with patch("actions.update_file_headers.update_file", return_value=False) as mock_update:
+        main()
+        # Verify that update_file was called (indicates file processing occurred)
+        assert mock_update.call_count > 0
+
+
+def test_main_with_custom_header():
+    """Test main function with custom header environment variable."""
+    with patch("actions.update_file_headers.update_file", return_value=False):
+        with patch("actions.update_file_headers.HEADER", "Custom Test Header"):
+            with patch("actions.update_file_headers.Action") as mock_action:
+                mock_event = mock_action.return_value
+                mock_event.repository = "test/repo"
+                main()
+                mock_action.assert_called_once()

{ultralytics_actions-0.0.85 → ultralytics_actions-0.0.87}/tests/test_summarize_release.py

@@ -35,45 +35,51 @@ def test_get_prs_between_tags():
     mock_event = MagicMock()
     mock_event.repository = "test/repo"
 
-    # Mock compare response
-    mock_compare_response = MagicMock()
-    mock_compare_response.json.return_value = {
-        "commits": [{"commit": {"message": "Fix bug #123"}}, {"commit": {"message": "Add feature #456"}}]
-    }
-
-    # Mock PR responses
-    mock_pr1_response = MagicMock()
-    mock_pr1_response.status_code = 200
-    mock_pr1_response.json.return_value = {
-        "number": 123,
-        "title": "Fix bug",
-        "body": "Fixes a bug",
-        "user": {"login": "user1"},
-        "html_url": "https://github.com/test/repo/pull/123",
-        "merged_at": "2023-01-01T12:00:00Z",
-    }
-
-    mock_pr2_response = MagicMock()
-    mock_pr2_response.status_code = 200
-    mock_pr2_response.json.return_value = {
-        "number": 456,
-        "title": "Add feature",
-        "body": "Adds a new feature",
-        "user": {"login": "user2"},
-        "html_url": "https://github.com/test/repo/pull/456",
-        "merged_at": "2023-01-02T12:00:00Z",
-    }
-
-    # Set up get method to return different responses
-    mock_event.get.side_effect = [mock_compare_response, mock_pr1_response, mock_pr2_response]
-
-    # Use patch to skip the sleep
+    mock_event.get.side_effect = [
+        MagicMock(status_code=200, json=lambda: {"commits": [{"commit": {"message": "Fix bug #123"}}]}),
+        MagicMock(
+            status_code=200,
+            json=lambda: {
+                "number": 123,
+                "title": "Fix bug",
+                "body": "Fix",
+                "user": {"login": "user1"},
+                "html_url": "url",
+                "merged_at": "2023-01-01T12:00:00Z",
+            },
+        ),
+    ]
+
     with patch("time.sleep"):
         prs = get_prs_between_tags(mock_event, "v1.0.0", "v1.1.0")
 
-    assert len(prs) == 2
+    assert len(prs) == 1
     assert prs[0]["number"] == 123
-    assert prs[1]["number"] == 456
+
+
+def test_get_prs_between_tags_api_failure():
+    """Test API failure returns empty list."""
+    mock_event = MagicMock()
+    mock_event.get.return_value = MagicMock(status_code=404)
+
+    assert get_prs_between_tags(mock_event, "v1.0.0", "v1.1.0") == []
+
+
+def test_get_prs_between_tags_missing_commits():
+    """Test missing commits data returns empty list."""
+    mock_event = MagicMock()
+    mock_event.get.return_value = MagicMock(status_code=200, json=lambda: {"message": "No commits"})
+
+    assert get_prs_between_tags(mock_event, "v1.0.0", "v1.1.0") == []
+
+
+def test_get_prs_between_tags_empty_commits():
+    """Test empty commits list returns empty list."""
+    mock_event = MagicMock()
+    mock_event.get.return_value = MagicMock(status_code=200, json=lambda: {"commits": []})
+
+    with patch("time.sleep"):
+        assert get_prs_between_tags(mock_event, "v1.0.0", "v1.1.0") == []
 
 
 @patch("actions.summarize_release.get_completion")

{ultralytics_actions-0.0.85 → ultralytics_actions-0.0.87}/tests/test_update_markdown_codeblocks.py

@@ -7,6 +7,7 @@ from actions.update_markdown_code_blocks import (
     add_indentation,
     extract_code_blocks,
     generate_temp_filename,
+    main,
     process_markdown_file,
     remove_indentation,
 )
@@ -104,3 +105,10 @@ def test():
     assert len(temp_files) == 1
     assert temp_files[0][1] == "def test():\n    return True"
     mock_file.assert_called_once()
+
+
+def test_main_real_files():
+    """Test main function on actual repository markdown files."""
+    # Run main on current directory which contains README.md and other markdown files
+    # This provides real-world test coverage of the entire pipeline
+    main(process_python=True, process_bash=True, verbose=False)

{ultralytics_actions-0.0.85 → ultralytics_actions-0.0.87}/ultralytics_actions.egg-info/PKG-INFO

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