From 7d87a58a731dde41b70e0fd01bf0c03793722ff3 Mon Sep 17 00:00:00 2001
From: Ben Kircher <bkircher@0xadd.de>
Date: Sat, 14 Feb 2026 09:40:35 +0100
Subject: [PATCH] url: fix trailing colon in path matches

Paths like `./.config/ghostty:` and `./Downloads:` were incorrectly
including the trailing colon. Add a `no_trailing_colon` lookbehind to
all path branches and prevent space segments from starting after a colon
so that `": text"` is not consumed as part of the path.
---
 src/config/url.zig | 20 ++++++++++++++++++--
 1 file changed, 18 insertions(+), 2 deletions(-)

diff --git a/src/config/url.zig b/src/config/url.zig
index fa902210f..da0892a91 100644
--- a/src/config/url.zig
+++ b/src/config/url.zig
@@ -48,6 +48,10 @@ const no_trailing_punctuation =
     \\(?<![,.])
 ;
 
+const no_trailing_colon =
+    \\(?<!:)
+;
+
 const trailing_spaces_at_eol =
     \\(?: +(?= *$))?
 ;
@@ -61,11 +65,11 @@ const non_dotted_path_lookahead =
 ;
 
 const dotted_path_space_segments =
-    \\(?: (?!\w+:\/\/)[\w\-.~:\/?#@!$&*+;=%]*[\/.])*
+    \\(?:(?<!:) (?!\w+:\/\/)[\w\-.~:\/?#@!$&*+;=%]*[\/.])*
 ;
 
 const any_path_space_segments =
-    \\(?: (?!\w+:\/\/)[\w\-.~:\/?#@!$&*+;=%]+)*
+    \\(?:(?<!:) (?!\w+:\/\/)[\w\-.~:\/?#@!$&*+;=%]+)*
 ;
 
 // Branch 1: URLs with explicit schemes (http, mailto, ftp, etc.).
@@ -87,11 +91,13 @@ const rooted_or_relative_path_branch =
     dotted_path_lookahead ++
     path_chars ++ "+" ++
     dotted_path_space_segments ++
+    no_trailing_colon ++
     trailing_spaces_at_eol ++
     "|" ++
     non_dotted_path_lookahead ++
     path_chars ++ "+" ++
     any_path_space_segments ++
+    no_trailing_colon ++
     trailing_spaces_at_eol ++
     ")";
 
@@ -105,6 +111,7 @@ const bare_relative_path_branch =
     bare_relative_path_prefix ++
     path_chars ++ "+" ++
     dotted_path_space_segments ++
+    no_trailing_colon ++
     trailing_spaces_at_eol;
 
 pub const regex =
@@ -447,6 +454,15 @@ test "url regex" {
             .input = "/tmp/foo bar,baz",
             .expect = "/tmp/foo bar",
         },
+        // trailing colon should not be part of the path
+        .{
+            .input = "./.config/ghostty: Needs upstream (main)",
+            .expect = "./.config/ghostty",
+        },
+        .{
+            .input = "./Downloads: Operation not permitted",
+            .expect = "./Downloads",
+        },
     };
 
     for (cases) |case| {