feat(gitcommit): improve diff filtering and notification flow (#7)

* feat(gitcommit): improve diff filtering and notification flow

- add tracking of all and excluded files in diff filtering
- return original diff if all files are excluded
- move commit message notification to after language selection
- trim whitespace in contextual diff checks for accuracy
- enable auto submit on git tool success in config
- refactor generate_commit_message for clearer notification order

* feat(git-tool): 丰富所有工具函数的返回值，增加 user_msg 和 llm_msg 字段

- 为 status、log、diff、branch、remotes、show、blame、stash_list、diff_commits、contributors、search_commits、tags 等函数增加 user_msg 和 llm_msg 字段
- 便于工具链和 LLM 处理更丰富的上下文信息
- 保持原有接口兼容

* fix(gitcommit): improve .gitignore and git read tool error handling

- 增强 .gitignore 相关函数的错误提示和返回结构
- git_read 工具输出增加 user_msg/llm_msg，提升交互体验
- 修复部分判断和消息提示逻辑

* feat(git): improve error handling and robustness in git tools

- wrap all git operations in pcall for safer execution
- ensure consistent error messages and output formatting
- enhance commit, stage, unstage, and read tool reliability
- update user and LLM messages for clearer feedback
- refactor contextual diff and commit logic for edge cases
- add utility trim function and remove redundant code

* fix(gitcommit): 优化参数校验与响应格式

- 增加参数校验辅助函数，提升健壮性
- 统一各工具响应格式，便于维护和扩展
- 修复部分注释与边界条件判断
- 优化错误处理，确保一致性

* fix(generator): improve LLM response handling logic

- restructure response processing for clarity
- handle error status with specific error messages
- ensure callback is called for all response cases
- add missing handler comment for LLM response

Author: jinzhongjia

lua/codecompanion/_extensions/gitcommit/buffer.lua

@@ -88,8 +88,6 @@ end
 ---Generate commit message and insert into gitcommit buffer
 ---@param bufnr number Buffer number
 function Buffer._generate_and_insert_commit_message(bufnr)
-  vim.notify("Generating commit message...", vim.log.levels.INFO)
-
   -- Check if we're in a git repository
   if not Git.is_repository() then
     vim.notify("Not in a git repository", vim.log.levels.ERROR)
@@ -99,14 +97,18 @@ function Buffer._generate_and_insert_commit_message(bufnr)
   -- Get relevant changes (staged or amend)
   local diff, context = Git.get_contextual_diff()
   if not diff then
-    local msg = context == "no_changes"
-        and (Git.is_amending() and "No changes to amend" or "No staged changes found. Please stage your changes first.")
-      or "Failed to get git changes"
+    local msg
+    if context == "no_changes" then
+      msg = Git.is_amending() and "No changes to amend" or "No staged changes found. Please stage your changes first."
+    else
+      msg = "Failed to get git changes, context=" .. tostring(context)
+    end
     vim.notify(msg, vim.log.levels.ERROR)
     return
   end
 
   Langs.select_lang(function(lang)
+    vim.notify("Generating commit message...", vim.log.levels.INFO)
     -- Generate commit message using LLM
     Generator.generate_commit_message(diff, lang, function(result, error)
       if error then

lua/codecompanion/_extensions/gitcommit/config.lua

@@ -31,7 +31,7 @@ M.default_opts = {
   enable_git_bot = true, -- Only enabled if both enable_git_read and enable_git_edit are true
   add_git_commands = true,
   git_tool_auto_submit_errors = false,
-  git_tool_auto_submit_success = false,
+  git_tool_auto_submit_success = true,
   gitcommit_select_count = 100,
 }
 

lua/codecompanion/_extensions/gitcommit/generator.lua

@@ -59,8 +59,8 @@ function Generator.generate_commit_message(diff, lang, callback)
 
   -- Send request to LLM
   new_client:request(payload, {
-    callback = function(err, data, _adapter)
-      Generator._handle_response(err, data, _adapter, callback)
+    callback = function(err, data, adapter)
+      Generator._handle_response(err, data, adapter, callback)
     end,
   }, {
     silent = true,
@@ -109,9 +109,9 @@ end
 ---Handle LLM response
 ---@param err table|nil Error from request
 ---@param data table|nil Response data
----@param _adapter table The adapter used
+---@param adapter table The adapter used
 ---@param callback fun(result: string|nil, error: string|nil) Callback function
-function Generator._handle_response(err, data, _adapter, callback)
+function Generator._handle_response(err, data, adapter, callback)
   -- Handle request errors
   if err then
     local error_msg = "Error generating commit message: " .. (err.stderr or err.message or "Unknown error")
@@ -125,7 +125,7 @@ function Generator._handle_response(err, data, _adapter, callback)
 
   -- Process successful response
   if data then
-    local result = _adapter.handlers.chat_output(_adapter, data)
+    local result = adapter.handlers.chat_output(adapter, data)
     if result and result.status then
       if result.status == CONSTANTS.STATUS_SUCCESS then
         local content = result.output and result.output.content