refactor(gitcommit): improve readability of async handlers

- expand multi-statement anonymous functions for clarity
- format error handling and data appending in uv.read_start callbacks
- split long return statements for better maintainability
- enhance error message formatting in git merge conflict handling
- reformat long table returns for staged changes check

Author: jinzhongjia

lua/codecompanion/_extensions/gitcommit/generator.lua

@@ -117,7 +117,7 @@ function Generator._handle_response(err, data, _adapter, callback)
     local error_msg = "Error generating commit message: " .. (err.stderr or err.message or "Unknown error")
     return callback(nil, error_msg)
   end
-  
+
   -- Check for empty or invalid data
   if not data then
     return callback(nil, "No response received from LLM")

lua/codecompanion/_extensions/gitcommit/init.lua

@@ -169,13 +169,21 @@ local function setup_slash_commands(opts)
     end)
 
     vim.uv.read_start(stdout, function(err, data)
-      if err then return end
-      if data then output = output .. data end
+      if err then
+        return
+      end
+      if data then
+        output = output .. data
+      end
     end)
 
     vim.uv.read_start(stderr, function(err, data)
-      if err then return end
-      if data then error_output = error_output .. data end
+      if err then
+        return
+      end
+      if data then
+        error_output = error_output .. data
+      end
     end)
   end
 
@@ -219,19 +227,31 @@ local function setup_slash_commands(opts)
           end
           select_commit(chat, items)
         else
-          chat:add_reference({ role = "user", content = "Error: Failed to get git log\n" .. error_output }, "git", "<git_error>")
+          chat:add_reference(
+            { role = "user", content = "Error: Failed to get git log\n" .. error_output },
+            "git",
+            "<git_error>"
+          )
         end
       end)
     end)
 
     vim.uv.read_start(stdout, function(err, data)
-      if err then return end
-      if data then output = output .. data end
+      if err then
+        return
+      end
+      if data then
+        output = output .. data
+      end
     end)
 
     vim.uv.read_start(stderr, function(err, data)
-      if err then return end
-      if data then error_output = error_output .. data end
+      if err then
+        return
+      end
+      if data then
+        error_output = error_output .. data
+      end
     end)
   end
 

lua/codecompanion/_extensions/gitcommit/tools/git.lua

@@ -540,7 +540,8 @@ function GitTool.merge(branch)
     return true, output
   else
     if output:match("CONFLICT") then
-      return false, "Merge conflict detected. Please resolve the conflicts manually. You can use 'git merge --abort' to cancel."
+      return false,
+        "Merge conflict detected. Please resolve the conflicts manually. You can use 'git merge --abort' to cancel."
     else
       return false, output
     end

lua/codecompanion/_extensions/gitcommit/tools/git_edit.lua

@@ -211,9 +211,12 @@ Available write-access Git operations:
         -- Check if there are staged changes
         local diff_success, diff_output = GitTool.get_diff(true) -- staged changes
         if not diff_success or not diff_output or vim.trim(diff_output) == "" then
-          return { status = "error", data = "No staged changes found for commit. Please stage your changes first using the stage operation." }
+          return {
+            status = "error",
+            data = "No staged changes found for commit. Please stage your changes first using the stage operation.",
+          }
         end
-        
+
         -- Return success with instruction for AI to use the diff tool
         return {
           status = "success",