Fix parenthesis in broken link checks (#380)

Signed-off-by: Glenn Jocher <[email protected]>

actions/__init__.py

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

actions/utils/common_utils.py

@@ -46,14 +46,14 @@
 )
 
 URL_PATTERN = re.compile(
-    r"\[([^]]+)]\(([^)]+)\)"  # Matches Markdown links [text](url)
+    r"\[([^]]+)]\((.*?)(?=\)(?:\s|$))\)"  # Markdown links with lookahead for space/end
     r"|"
     r"("  # Start capturing group for plaintext URLs
     r"(?:https?://)?"  # Optional http:// or https://
     r"(?:www\.)?"  # Optional www.
     r"(?:[\w.-]+)?"  # Optional domain name and subdomains
     r"\.[a-zA-Z]{2,}"  # TLD
-    r"(?:/[^\s\"')\]]*)?"  # Optional path
+    r"(?:/[^\s\"'\]]*)?"  # Optional path
     r")"
 )
 
@@ -128,4 +128,8 @@ def check_links_in_string(text, verbose=True, return_bad=False):
 
 
 if __name__ == "__main__":
-    print(is_url("https://ultralytics.com/images/bus.jpg"))
+    url = "https://ultralytics.com/images/bus.jpg"
+    string = f"This is a string with a [Markdown link]({url}) inside it."
+
+    print(f"is_url(): {is_url(url)}")
+    print(f"check_links_in_string(): {check_links_in_string(string)}")

tests/test_urls.py

@@ -23,6 +23,7 @@
     "https://www.reddit.com/r/Ultralytics/comments/1fw3605/release_megathread/",
     "https://www.kaggle.com/models/ultralytics/yolo11",
     "https://apps.apple.com/xk/app/ultralytics/id1583935240",
+    "https://en.wikipedia.org/wiki/Active_learning_(machine_learning)",  # parentheses in link
 ]
 
 
@@ -38,6 +39,21 @@ def test_is_url():
         assert is_url(url), f"URL check failed: {url}"
 
 
+def test_links_in_string_func():
+    """Test URLs in strings function."""
+    assert check_links_in_string(" abc ".join(url for url in URLS))
+
+
+def test_markdown_links_in_string_func():
+    """Test Markdown links in strings function."""
+    assert check_links_in_string(" abc ".join(f"[text]({url})" for url in URLS))
+
+
+def test_html_links_in_string_func():
+    """Test HTML links in strings function."""
+    assert check_links_in_string(" abc ".join(f'<a href="{url}">text</a>' for url in URLS))
+
+
 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>"