code_style: reduce token usage by remove repo parameter from tool call

Signed-off-by: leo <longshuang@msn.cn>

Author: love-linger

src/AI/Agent.cs

@@ -95,7 +95,7 @@ public async Task GenerateCommitMessageAsync(string repo, string currentBranch,
 
                             foreach (var call in completion.ToolCalls)
                             {
-                                var result = await ChatTools.ProcessAsync(call, onUpdate, amendParent);
+                                var result = await ChatTools.ProcessAsync(call, repo, amendParent, onUpdate);
                                 messages.Add(result);
                             }
 

src/AI/ChatTools.cs

@@ -15,10 +15,6 @@ public static class ChatTools
             {
                 "type": "object",
                 "properties": {
-                    "repo": {
-                        "type": "string",
-                        "description": "The path to the repository."
-                    },
                     "file": {
                         "type": "string",
                         "description": "The path to the file."
@@ -28,28 +24,25 @@ public static class ChatTools
                         "description": "The path to the original file when it has been renamed or copied."
                     }
                  },
-                 "required": ["repo", "file"]
+                 "required": ["file"]
             }
             """)), false);
 
-        public static async Task<ToolChatMessage> ProcessAsync(ChatToolCall call, Action<string> output, string amendParent)
+        public static async Task<ToolChatMessage> ProcessAsync(ChatToolCall call, string repo, string amendParent, Action<string> output)
         {
             using var doc = JsonDocument.Parse(call.FunctionArguments);
 
             if (call.FunctionName.Equals(GetDetailChangesInFile.FunctionName))
             {
-                var hasRepo = doc.RootElement.TryGetProperty("repo", out var repoPath);
                 var hasFile = doc.RootElement.TryGetProperty("file", out var filePath);
                 var hasOriginalFile = doc.RootElement.TryGetProperty("originalFile", out var originalFilePath);
-                if (!hasRepo)
-                    throw new ArgumentException("repo", "The repo argument is required");
                 if (!hasFile)
                     throw new ArgumentException("file", "The file argument is required");
 
                 output?.Invoke($"Read changes in file: {filePath.GetString()}");
 
                 var orgFilePath = hasOriginalFile ? originalFilePath.GetString() : string.Empty;
-                var rs = await new Commands.GetFileChangeForAI(repoPath.GetString(), filePath.GetString(), orgFilePath, amendParent).ReadAsync();
+                var rs = await new Commands.GetFileChangeForAI(repo, filePath.GetString(), orgFilePath, amendParent).ReadAsync();
                 var message = rs.IsSuccess ? rs.StdOut : string.Empty;
                 return new ToolChatMessage(call.Id, message);
             }

src/ViewModels/AIAssistant.cs

@@ -50,7 +50,8 @@ public AIAssistant(Repository repo, AI.Service service, List<Models.Change> chan
                 SerializeChange(c, builder);
             _changeList = builder.ToString();
 
-            _amendParent = changes.Count > 0 ? changes[0].DataForAmend?.ParentSHA : null;
+            if (changes.Count > 0 && changes[0].DataForAmend is { } amend)
+                _amendParent = amend.ParentSHA;
         }
 
         public async Task GenAsync()