fix(keymaps): improve git fetch keymap to use fidget and show errors

Refactored the <leader>gpf keymap to use utils.lazy_cmd_with_fidget for better feedback and error reporting. Updated group label for clarity. Also enhanced utils to notify on stderr output.

Author: dcai

lua/dcai/keymaps/git.lua

@@ -105,7 +105,7 @@ local git_keymap = {
   },
   {
     '<leader>gp',
-    group = 'pull and push',
+    group = 'pull/fetch and push',
   },
   {
     '<leader>gpr',
@@ -126,7 +126,10 @@ local git_keymap = {
   },
   {
     '<leader>gpf',
-    lazy_git_cmd({ args = { 'fetch', '--tags', '--all', '--verbose' } }),
+    utils.lazy_cmd_with_fidget(
+      'git',
+      { 'fetch', '--tags', '--all', '--verbose' }
+    ),
     desc = 'git fetch',
   },
 }

lua/dcai/keymaps/utils.lua

@@ -19,27 +19,25 @@ M.lazy_cmd = function(command, args, opts)
     args = args or {}
     local cwd = opts.cwd or vim.fn.getcwd()
 
-    Job
-      :new({
-        command = command,
-        args = args,
-        cwd = cwd,
-        on_exit = vim.schedule_wrap(function(job, ret)
-          local stderr = table.concat(job:stderr_result(), vim.g.nl)
-          local stdout = table.concat(job:result(), vim.g.nl)
+    Job:new({
+      command = command,
+      args = args,
+      cwd = cwd,
+      on_exit = vim.schedule_wrap(function(job, ret)
+        local stderr = table.concat(job:stderr_result(), vim.g.nl)
+        local stdout = table.concat(job:result(), vim.g.nl)
 
-          if ret == 0 then
-            if opts.on_success then
-              opts.on_success(stdout, stderr)
-            end
-          else
-            if opts.on_error then
-              opts.on_error(stdout, stderr, ret)
-            end
+        if ret == 0 then
+          if opts.on_success then
+            opts.on_success(stdout, stderr)
           end
-        end),
-      })
-      :start()
+        else
+          if opts.on_error then
+            opts.on_error(stdout, stderr, ret)
+          end
+        end
+      end),
+    }):start()
   end
 end
 
@@ -73,11 +71,7 @@ local function lazy_cmd_with_window(command, opts, desc)
           popupwin.append(stderr)
         else
           vim.notify(
-            command
-              .. ''
-              .. table.concat(args, ',')
-              .. ' failed: '
-              .. stderr
+            command .. '' .. table.concat(args, ',') .. ' failed: ' .. stderr
           )
         end
       end,
@@ -89,7 +83,8 @@ M.lazy_cmd_with_window = lazy_cmd_with_window
 M.lazy_cmd_with_fidget = function(command, args, opts)
   return function()
     opts = opts or {}
-    local command_full = string.format('%s %s', command, table.concat(args, ' '))
+    local command_full =
+      string.format('%s %s', command, table.concat(args, ' '))
 
     -- Create fidget progress handle
     local fidget_progress = require('fidget.progress')
@@ -106,6 +101,9 @@ M.lazy_cmd_with_fidget = function(command, args, opts)
         if stdout ~= '' then
           vim.notify(stdout, vim.log.levels.INFO)
         end
+        if stderr ~= '' then
+          vim.notify(stderr, vim.log.levels.ERROR)
+        end
         handle:finish()
       end,
       on_error = function(stdout, stderr, ret)