test & fix ** pattern with directories

Author: comatory

lua/gh-co/co.lua

@@ -9,9 +9,22 @@ end
 local function buildEscapedPattern(rawPattern)
   -- Escape Lua pattern special characters except *
   local escaped = string.gsub(rawPattern, "([%-%+%?%(%)])", "%%%1")
-  -- Convert * to match any character except /
+  
+  -- Handle ** first (before single *) - use placeholder to avoid conflicts
+  escaped = string.gsub(escaped, "%*%*", "__DOUBLESTAR__")
+  
+  -- Convert remaining * to match any character except /
   escaped = string.gsub(escaped, "%*", "[^/]*")
 
+  -- Replace placeholder with pattern that matches any path including /
+  escaped = string.gsub(escaped, "__DOUBLESTAR__", ".*")
+  
+  -- Special handling for **/name patterns - they should match directories
+  if string.match(rawPattern, "%*%*/[^/]+$") then
+    -- **/logs should match files within logs directories
+    escaped = escaped .. "/"
+  end
+  
   -- Handle trailing slash - directory patterns should match everything within
   if string.match(escaped, "/$") then
     -- Remove trailing slash and match anything that starts with this path

lua/gh-co/co.test.lua

@@ -256,4 +256,19 @@ function TestCO:testRootScriptsDirectoryPattern() -- luacheck: ignore 212
   lu.assertEquals(result, {"@doctocat", "@octocat"})
 end
 
+function TestCO:testDoubleStarLogsPattern() -- luacheck: ignore 212
+  -- Test **/logs pattern matches any logs directory at any depth
+  self.FS.openCodeownersFileAsLines = function()
+    local lines = {"**/logs @octocat"}
+    local i = 0
+    return function()
+      i = i + 1
+      return lines[i]
+    end
+  end
+  
+  local result = CO.matchFilesToCodeowner({"build/logs/error.log", "app/server/logs/access.log", "logs/debug.log"})
+  lu.assertEquals(result, {"@octocat"})
+end
+
 os.exit(lu.LuaUnit.run())