From 96bda270a252e10ded4c80ddce8aa1f86625129a Mon Sep 17 00:00:00 2001
From: Vincent Prigent <vincent.prigent1@gmail.com>
Date: Fri, 22 Nov 2024 22:57:45 +1300
Subject: [PATCH] Add two edge cases to the url regex

---
 src/config/url.zig | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/config/url.zig b/src/config/url.zig
index d7c1ab2a2..1d0764736 100644
--- a/src/config/url.zig
+++ b/src/config/url.zig
@@ -24,7 +24,7 @@ const oni = @import("oniguruma");
 /// handling them well requires a non-regex approach.
 pub const regex =
     "(?:" ++ url_schemes ++
-    \\)(?:[\w\-.~:/?#\[\]@!$&*+,;=%]+(?:\(\w*\))?)+(?<!\.)
+    \\)(?:[\w\-.~:/?#@!$&*+,;=%]+(?:[\(\[]\w*[\)\]])?)+(?<![,.])
 ;
 const url_schemes =
     \\https?://|mailto:|ftp://|file:|ssh:|git://|ssh://|tel:|magnet:|ipfs://|ipns://|gemini://|gopher://|news:
@@ -70,6 +70,10 @@ test "url regex" {
             .input = "Link period https://example.com. More text.",
             .expect = "https://example.com",
         },
+        .{
+            .input = "Link trailing colon https://example.com, more text.",
+            .expect = "https://example.com",
+        },
         .{
             .input = "Link in double quotes \"https://example.com\" and more",
             .expect = "https://example.com",
@@ -112,6 +116,11 @@ test "url regex" {
             .input = "square brackets https://example.com/[foo] and more",
             .expect = "https://example.com/[foo]",
         },
+        // square bracket following url
+        .{
+            .input = "[13]:TooManyStatements: TempFile#assign_temp_file_to_entity has approx 7 statements [https://example.com/docs/Too-Many-Statements.md]",
+            .expect = "https://example.com/docs/Too-Many-Statements.md",
+        },
         // remaining URL schemes tests
         .{
             .input = "match ftp://example.com ftp links",