fix: handle owner/repo#issue format in github handler (#56)

Fixes #56

lua/gx/git.lua

@@ -16,7 +16,7 @@ local function parse_git_output(result)
   end
 end
 
-function M.get_remote_url(remotes, push, owner)
+function M.get_remote_url(remotes, push, owner, repo)
   local notifier = require("gx.notifier")
 
   local url = nil
@@ -41,6 +41,9 @@ function M.get_remote_url(remotes, push, owner)
   end
   if type(owner) == "string" and owner ~= "" then
     local domain, repository = url:match("^https?://([^/]+)/[^/]+/([^/]*)")
+    if repo ~= "" then
+      repository = repo
+    end
     url = string.format("https://%s/%s/%s", domain, owner, repository)
   end
 

lua/gx/handlers/github.lua

@@ -9,14 +9,17 @@ local M = {
 
 -- navigate to neovim github plugin url
 function M.handle(mode, line, handler_options)
-  local match = helper.find(line, mode, "([%w-_]+#%d+)")
+  local match = helper.find(line, mode, "([%w-_]+/[%w-_]+#%d+)")
+  if not match then
+    match = helper.find(line, mode, "([%w-_]+#%d+)")
+  end
   if not match then
     match = helper.find(line, mode, "(#%d+)")
   end
   if not match then
     return
   end
-  local owner, issue = match:match("(.*)#(.+)")
+  local owner, repo, issue = match:match("([^/#]*)/?([^#]*)#(.+)")
 
   local remotes = handler_options.git_remotes
   if type(remotes) == "function" then
@@ -28,7 +31,7 @@ function M.handle(mode, line, handler_options)
     push = push(vim.fn.expand("%:p"))
   end
 
-  local git_url = require("gx.git").get_remote_url(remotes, push, owner)
+  local git_url = require("gx.git").get_remote_url(remotes, push, owner, repo)
   if not git_url then
     return
   end

test/spec/gx/handlers/github_handler_spec.lua

@@ -40,4 +40,10 @@ describe("github_handler_does_work", function()
       handler.handle("v", "See foouser#42", handler_options)
     )
   end)
+  it("parses owner/repo#issue format", function()
+    assert.equals(
+      "https://github.com/neovim/neovim/issues/23943",
+      handler.handle("v", "Waiting on upstream neovim/neovim#23943", handler_options)
+    )
+  end)
 end)