ultralytics-actions 0.0.34__tar.gz → 0.0.37__tar.gz

{ultralytics_actions-0.0.34 → ultralytics_actions-0.0.37}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: ultralytics-actions
-Version: 0.0.34
+Version: 0.0.37
 Summary: Ultralytics Actions for GitHub automation and PR management.
 Author-email: Glenn Jocher <glenn.jocher@ultralytics.com>
 Maintainer-email: Ultralytics <hello@ultralytics.com>
@@ -28,12 +28,11 @@ Classifier: Operating System :: OS Independent
 Requires-Python: >=3.8
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: requests>=2.26.0
-Requires-Dist: ruff>=0.1.6
+Requires-Dist: requests>=2.32.3
+Requires-Dist: ruff>=0.8.4
 Requires-Dist: docformatter>=1.7.5
 Provides-Extra: dev
 Requires-Dist: pytest; extra == "dev"
-Requires-Dist: black; extra == "dev"
 
 <a href="https://www.ultralytics.com/" target="_blank"><img src="https://raw.githubusercontent.com/ultralytics/assets/main/logo/Ultralytics_Logotype_Original.svg" width="320" alt="Ultralytics logo"></a>
 

{ultralytics_actions-0.0.34 → ultralytics_actions-0.0.37}/actions/__init__.py

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

{ultralytics_actions-0.0.34 → ultralytics_actions-0.0.37}/actions/utils/common_utils.py

@@ -1,10 +1,11 @@
 # Ultralytics Actions 🚀, AGPL-3.0 license https://ultralytics.com/license
 
 import re
-import socket
 import time
-import urllib
 from concurrent.futures import ThreadPoolExecutor
+from urllib import parse
+
+import requests
 
 
 def remove_html_comments(body: str) -> str:
@@ -37,6 +38,10 @@ def is_url(url, check=True, max_attempts=3, timeout=2):
         "github.com",  # ignore GitHub links that may be private repos
         "kaggle.com",  # blocks automated header requests
         "reddit.com",  # blocks automated header requests
+        "linkedin.com",
+        "twitter.com",
+        "x.com",
+        "storage.googleapis.com",  # private GCS buckets
     )
     try:
         # Check allow list
@@ -44,24 +49,23 @@ def is_url(url, check=True, max_attempts=3, timeout=2):
             return True
 
         # Check structure
-        result = urllib.parse.urlparse(url)
-        if not all([result.scheme, result.netloc]):
+        result = parse.urlparse(url)
+        partition = result.netloc.partition(".")  # i.e. netloc = "github.com" -> ("github", ".", "com")
+        if not result.scheme or not partition[0] or not partition[2]:
             return False
 
         # Check response
         if check:
             for attempt in range(max_attempts):
                 try:
-                    req = urllib.request.Request(
-                        url,
-                        method="HEAD",
-                        headers={
-                            "User-Agent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120.0.0.0 Safari/537.36",
-                        },
-                    )
-                    with urllib.request.urlopen(req, timeout=timeout) as response:
-                        return response.getcode() < 400
-                except (urllib.error.URLError, socket.timeout):
+                    headers = {
+                        "User-Agent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) Chrome/120.0.0.0",
+                        "Accept": "*",
+                        "Accept-Language": "*",
+                        "Accept-Encoding": "*",
+                    }
+                    return requests.head(url, headers=headers, timeout=timeout, allow_redirects=True).status_code < 400
+                except Exception:
                     if attempt == max_attempts - 1:  # last attempt
                         return False
                     time.sleep(2**attempt)  # exponential backoff
@@ -79,18 +83,20 @@ def check_links_in_string(text, verbose=True, return_bad=False):
         r"("  # Start capturing group for plaintext URLs
         r"(?:https?://)?"  # Optional http:// or https://
         r"(?:www\.)?"  # Optional www.
-        r"[\w.-]+"  # Domain name and subdomains
+        r"(?:[\w.-]+)?"  # Optional domain name and subdomains
         r"\.[a-zA-Z]{2,}"  # TLD
         r"(?:/[^\s\"')\]]*)?"  # Optional path
         r")"
     )
+    # all_urls.extend([url for url in match if url and parse.urlparse(url).scheme])
     all_urls = []
     for md_text, md_url, plain_url in re.findall(pattern, text):
         url = md_url or plain_url
-        if url and urllib.parse.urlparse(url).scheme:
+        if url and parse.urlparse(url).scheme:
             all_urls.append(url)
 
     urls = set(map(clean_url, all_urls))  # remove extra characters and make unique
+    # bad_urls = [x for x in urls if not is_url(x, check=True)]  # single-thread
     with ThreadPoolExecutor(max_workers=16) as executor:  # multi-thread
         bad_urls = [url for url, valid in zip(urls, executor.map(lambda x: not is_url(x, check=True), urls)) if valid]
 

{ultralytics_actions-0.0.34 → ultralytics_actions-0.0.37}/pyproject.toml

@@ -65,15 +65,14 @@ classifiers = [
 ]
 
 dependencies = [
-    "requests>=2.26.0",
-    "ruff>=0.1.6",
+    "requests>=2.32.3",
+    "ruff>=0.8.4",
     "docformatter>=1.7.5",
 ]
 
 [project.optional-dependencies]
 dev = [
     "pytest",
-    "black",
 ]
 
 [project.urls]
@@ -96,6 +95,10 @@ packages = { find = { where = ["."], include = ["actions", "actions.*"] } }
 [tool.setuptools.dynamic]
 version = { attr = "actions.__version__" }
 
+[tool.pytest.ini_options]
+addopts = "--doctest-modules --durations=30 --color=yes"
+norecursedirs = [".git", "dist", "build"]
+
 [tool.ruff]
 line-length = 120
 

ultralytics_actions-0.0.37/tests/test_urls.py

@@ -0,0 +1,98 @@
+# Ultralytics Actions 🚀, AGPL-3.0 license
+# Continuous Integration (CI) GitHub Actions tests
+
+import pytest
+
+from actions.utils.common_utils import check_links_in_string, is_url
+
+URLS = [
+    "https://docs.ultralytics.com/help/CLA/",
+    "https://docs.ultralytics.com/help/contributing",
+    "https://docs.ultralytics.com",
+    "https://ultralytics.com",
+    "https://github.com/ultralytics/ultralytics",
+]
+
+
+@pytest.fixture
+def verbose():
+    """Fixture that provides a verbose logging utility for detailed output during testing and debugging."""
+    return False  # Set False to suppress print statements during tests
+
+
+def test_is_url():
+    """Test each URL using is_url function."""
+    for url in URLS:
+        assert is_url(url), f"URL check failed: {url}"
+
+
+def test_html_links(verbose):
+    """Tests the validity of URLs within HTML anchor tags and returns any invalid URLs found."""
+    text = "Visit <a href='https://err.com'>our site</a> or <a href=\"http://test.org\">test site</a>"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.com", "http://test.org"}
+
+
+def test_markdown_links(verbose):
+    """Validates URLs in markdown links within a given text using check_links_in_string."""
+    text = "Check [Example](https://err.com) or [Test](http://test.org)"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.com", "http://test.org"}
+
+
+def test_mixed_formats(verbose):
+    """Tests URL detection in mixed text formats (HTML, Markdown, plain text) using check_links_in_string."""
+    text = "A <a href='https://1.com'>link</a> and [markdown](https://2.org) and https://3.net"
+    result, urls = check_links_in_string(text, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://1.com", "https://2.org", "https://3.net"}
+
+
+def test_duplicate_urls(verbose):
+    """Tests detection of duplicate URLs in various text formats using the check_links_in_string function."""
+    text = "Same URL: https://err.com and <a href='https://err.com'>link</a>"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.com"}
+
+
+def test_no_urls(verbose):
+    """Tests that a string with no URLs returns True when checked using the check_links_in_string function."""
+    text = "This text contains no URLs."
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is True
+    assert not set(urls)
+
+
+def test_invalid_urls(verbose):
+    """Test invalid URLs."""
+    text = "Invalid URL: http://.com"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"http://.com"}
+
+
+def test_urls_with_paths_and_queries(verbose):
+    """Test URLs with paths and query parameters to ensure they are correctly identified and validated."""
+    text = "Complex URL: https://err.com/path?query=value#fragment"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.com/path?query=value#fragment"}
+
+
+def test_urls_with_different_tlds(verbose):
+    """Test URLs with various top-level domains (TLDs) to ensure correct identification and handling."""
+    text = "Different TLDs: https://err.ml https://err.org https://err.net https://err.io https://err.ai"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.ml", "https://err.org", "https://err.net", "https://err.io", "https://err.ai"}
+
+
+def test_case_sensitivity(verbose):
+    """Tests URL case sensitivity by verifying that URLs with different cases are correctly identified and handled."""
+    text = "Case test: HTTPS://err.com and https://err.com"
+    result, urls = check_links_in_string(text, verbose, return_bad=True)
+    assert result is False
+    assert set(urls) == {"https://err.com"}

{ultralytics_actions-0.0.34 → ultralytics_actions-0.0.37}/ultralytics_actions.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: ultralytics-actions
-Version: 0.0.34
+Version: 0.0.37
 Summary: Ultralytics Actions for GitHub automation and PR management.
 Author-email: Glenn Jocher <glenn.jocher@ultralytics.com>
 Maintainer-email: Ultralytics <hello@ultralytics.com>
@@ -28,12 +28,11 @@ Classifier: Operating System :: OS Independent
 Requires-Python: >=3.8
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: requests>=2.26.0
-Requires-Dist: ruff>=0.1.6
+Requires-Dist: requests>=2.32.3
+Requires-Dist: ruff>=0.8.4
 Requires-Dist: docformatter>=1.7.5
 Provides-Extra: dev
 Requires-Dist: pytest; extra == "dev"
-Requires-Dist: black; extra == "dev"
 
 <a href="https://www.ultralytics.com/" target="_blank"><img src="https://raw.githubusercontent.com/ultralytics/assets/main/logo/Ultralytics_Logotype_Original.svg" width="320" alt="Ultralytics logo"></a>
 

{ultralytics_actions-0.0.34 → ultralytics_actions-0.0.37}/ultralytics_actions.egg-info/SOURCES.txt

@@ -10,6 +10,7 @@ actions/utils/__init__.py
 actions/utils/common_utils.py
 actions/utils/github_utils.py
 actions/utils/openai_utils.py
+tests/test_urls.py
 ultralytics_actions.egg-info/PKG-INFO
 ultralytics_actions.egg-info/SOURCES.txt
 ultralytics_actions.egg-info/dependency_links.txt

ultralytics_actions-0.0.37/ultralytics_actions.egg-info/requires.txt

@@ -0,0 +1,6 @@
+requests>=2.32.3
+ruff>=0.8.4
+docformatter>=1.7.5
+
+[dev]
+pytest

ultralytics_actions-0.0.34/ultralytics_actions.egg-info/requires.txt

@@ -1,7 +0,0 @@
-requests>=2.26.0
-ruff>=0.1.6
-docformatter>=1.7.5
-
-[dev]
-pytest
-black