change code execution response

Author: iceljc

src/Infrastructure/BotSharp.Abstraction/Coding/Models/CodeExecutionResponseModel.cs

@@ -1,3 +1,5 @@
+using BotSharp.Abstraction.Coding.Responses;
+
 namespace BotSharp.Abstraction.Coding.Models;
 
 public class CodeExecutionResponseModel
@@ -6,5 +8,5 @@ public class CodeExecutionResponseModel
     public AgentCodeScript CodeScript { get; set; }
     public IDictionary<string, string>? Arguments { get; set; }
     public string Text { get; set; } = default!;
-    public string ExecutionResult { get; set; } = default!;
+    public CodeInterpretResponse? ExecutionResult { get; set; }
 }

src/Infrastructure/BotSharp.Core.Rules/Engines/RuleEngine.cs

@@ -153,7 +153,7 @@ private async Task<bool> TriggerCodeScript(Agent agent, string triggerName, Rule
                 CodeProcessor = processor.Provider,
                 CodeScript = codeScript,
                 Arguments = arguments.DistinctBy(x => x.Key).ToDictionary(x => x.Key, x => x.Value ?? string.Empty),
-                ExecutionResult = response?.ToString() ?? string.Empty
+                ExecutionResult = response
             };
 
             foreach (var hook in hooks)

src/Infrastructure/BotSharp.Core/Instructs/Services/InstructService.Execute.cs

@@ -169,11 +169,11 @@ await hook.OnResponseGenerated(new InstructResponseModel
         string templateName,
         CodeInstructOptions? codeOptions)
     {
-        InstructResult? response = null;
+        InstructResult? instructResult = null;
 
         if (agent == null)
         {
-            return response;
+            return instructResult;
         }
 
         var agentService = _services.GetRequiredService<IAgentService>();
@@ -192,7 +192,7 @@ await hook.OnResponseGenerated(new InstructResponseModel
 #if DEBUG
             _logger.LogWarning($"No code processor found. (Agent: {agent.Id}, Code processor: {codeProvider})");
 #endif
-            return response;
+            return instructResult;
         }
 
         // Get code script name
@@ -211,7 +211,7 @@ await hook.OnResponseGenerated(new InstructResponseModel
 #if DEBUG
             _logger.LogWarning($"Empty code script name. (Agent: {agent.Id}, {scriptName})");
 #endif
-            return response;
+            return instructResult;
         }
 
         // Get code script
@@ -222,7 +222,7 @@ await hook.OnResponseGenerated(new InstructResponseModel
 #if DEBUG
             _logger.LogWarning($"Empty code script. (Agent: {agent.Id}, {scriptName})");
 #endif
-            return response;
+            return instructResult;
         }
 
         // Get code arguments
@@ -268,7 +268,7 @@ await hook.OnResponseGenerated(new InstructResponseModel
 
         if (codeResponse?.Success == true)
         {
-            response = new InstructResult
+            instructResult = new InstructResult
             {
                 MessageId = message.MessageId,
                 Template = context.CodeScript?.Name,
@@ -280,19 +280,24 @@ await hook.OnResponseGenerated(new InstructResponseModel
         {
             CodeProcessor = codeProcessor.Provider,
             CodeScript = context.CodeScript,
-            ExecutionResult = codeResponse?.ToString() ?? string.Empty,
+            ExecutionResult = codeResponse,
             Text = message.Content,
             Arguments = context.Arguments?.DistinctBy(x => x.Key).ToDictionary(x => x.Key, x => x.Value ?? string.Empty)
         };
 
         // After code execution
         foreach (var hook in hooks)
         {
-            await hook.AfterCompletion(agent, response ?? new());
+            await hook.AfterCompletion(agent, instructResult ?? new());
             await hook.AfterCodeExecution(agent, codeExeResponse);
         }
 
-        return response;
+        if (instructResult != null)
+        {
+            instructResult.Text = codeExeResponse?.ExecutionResult?.ToString() ?? string.Empty;
+        }
+
+        return instructResult;
     }
 
     private async Task<string> GetTextCompletion(

src/Infrastructure/BotSharp.Logger/Hooks/InstructionLogHook.cs

@@ -82,7 +82,7 @@ public override async Task AfterCodeExecution(Agent agent, CodeExecutionResponse
                 TemplateName = response.CodeScript?.Name,
                 UserMessage = response.Text,
                 SystemInstruction = $"Code script name: {response.CodeScript}, Version: {codeScriptVersion.ToString("o")}",
-                CompletionText = response.ExecutionResult,
+                CompletionText = response.ExecutionResult?.ToString() ?? string.Empty,
                 States = response.Arguments?.ToDictionary() ?? [],
                 UserId = user?.Id
             }