refine update agent in file repository

Author: Jicheng Lu

src/Infrastructure/BotSharp.Core/Agents/Services/AgentService.CreateAgent.cs

@@ -84,7 +84,7 @@ private Agent FetchAgentFileByName(string agentName, string filePath)
 
     private string FetchInstructionFromFile(string fileDir)
     {
-        var file = Path.Combine(fileDir, "instruction.liquid");
+        var file = Path.Combine(fileDir, $"instruction.{_agentSettings.TemplateFormat}");
         if (!File.Exists(file)) return null;
 
         var instruction = File.ReadAllText(file);
@@ -100,7 +100,7 @@ private List<AgentTemplate> FetchTemplatesFromFile(string fileDir)
             var splits = fileName.ToLower().Split('.');
             var name = splits[0];
             var extension = splits[1];
-            if (name != "instruction" && extension == "liquid")
+            if (name != "instruction" && extension == _agentSettings.TemplateFormat)
             {
                 var content = File.ReadAllText(file);
                 templates.Add(new AgentTemplate(name, content));

src/Infrastructure/BotSharp.Core/Agents/Services/AgentService.cs

@@ -9,20 +9,20 @@ public partial class AgentService : IAgentService
     private readonly IBotSharpRepository _db;
     private readonly ILogger _logger;
     private readonly IUserIdentity _user;
-    private readonly AgentSettings _settings;
+    private readonly AgentSettings _agentSettings;
     private readonly JsonSerializerOptions _options;
 
     public AgentService(IServiceProvider services,
         IBotSharpRepository db,
         ILogger<AgentService> logger, 
         IUserIdentity user, 
-        AgentSettings settings)
+        AgentSettings agentSettings)
     {
         _services = services;
         _db = db;
         _logger = logger;
         _user = user;
-        _settings = settings;
+        _agentSettings = agentSettings;
         _options = new JsonSerializerOptions
         {
             PropertyNameCaseInsensitive = true,
@@ -40,7 +40,7 @@ public string GetDataDir()
     public string GetAgentDataDir(string agentId)
     {
         var dbSettings = _services.GetRequiredService<BotSharpDatabaseSettings>();
-        var dir = Path.Combine(dbSettings.FileRepository, _settings.DataDir, agentId);
+        var dir = Path.Combine(dbSettings.FileRepository, _agentSettings.DataDir, agentId);
         if (!Directory.Exists(dir))
         {
             Directory.CreateDirectory(dir);

src/Infrastructure/BotSharp.Core/Repository/BotSharpDbContext.cs

@@ -1,4 +1,3 @@
-using BotSharp.Abstraction.Agents.Enums;
 using BotSharp.Abstraction.Agents.Models;
 using BotSharp.Abstraction.Repositories;
 using BotSharp.Abstraction.Routing.Models;
@@ -72,89 +71,99 @@ public int Transaction<TTableInterface>(Action action)
     }
 
 
-
-    public void CreateNewConversation(Conversation conversation)
+    #region Agent
+    public Agent GetAgent(string agentId)
     {
         throw new NotImplementedException();
     }
 
-    public List<RoutingItem> CreateRoutingItems(List<RoutingItem> routingItems)
+    public void UpdateAgent(Agent agent, AgentField field)
     {
         throw new NotImplementedException();
     }
 
-    public List<RoutingProfile> CreateRoutingProfiles(List<RoutingProfile> profiles)
+    public string GetAgentTemplate(string agentId, string templateName)
     {
         throw new NotImplementedException();
     }
 
-    public void CreateUser(User user)
+    public List<string> GetAgentResponses(string agentId, string prefix, string intent)
     {
         throw new NotImplementedException();
     }
+    #endregion
 
-    public void DeleteRoutingItems()
+
+    #region Conversation
+    public void CreateNewConversation(Conversation conversation)
     {
         throw new NotImplementedException();
     }
 
-    public void DeleteRoutingProfiles()
+    public Conversation GetConversation(string conversationId)
     {
         throw new NotImplementedException();
     }
 
-    public Agent GetAgent(string agentId)
+    public List<Conversation> GetConversations(string userId)
     {
         throw new NotImplementedException();
     }
 
-    public List<string> GetAgentResponses(string agentId, string prefix, string intent)
+    public string GetConversationDialog(string conversationId)
     {
         throw new NotImplementedException();
     }
 
-    public Conversation GetConversation(string conversationId)
+    public List<StateKeyValue> GetConversationStates(string conversationId)
     {
         throw new NotImplementedException();
     }
 
-    public string GetConversationDialog(string conversationId)
+    public void UpdateConversationDialog(string conversationId, string dialogs)
     {
         throw new NotImplementedException();
     }
 
-    public List<Conversation> GetConversations(string userId)
+    public void UpdateConversationStates(string conversationId, List<StateKeyValue> states)
     {
         throw new NotImplementedException();
     }
+    #endregion
 
-    public List<StateKeyValue> GetConversationStates(string conversationId)
+
+    #region User
+    public User GetUserByEmail(string email)
     {
         throw new NotImplementedException();
     }
 
-    public User GetUserByEmail(string email)
+    public void CreateUser(User user)
     {
         throw new NotImplementedException();
     }
+    #endregion
 
-    public void UpdateAgent(Agent agent, AgentField field)
+
+    #region Routing
+    public List<RoutingItem> CreateRoutingItems(List<RoutingItem> routingItems)
     {
         throw new NotImplementedException();
     }
 
-    public void UpdateConversationDialog(string conversationId, string dialogs)
+    public List<RoutingProfile> CreateRoutingProfiles(List<RoutingProfile> profiles)
     {
         throw new NotImplementedException();
     }
 
-    public void UpdateConversationStates(string conversationId, List<StateKeyValue> states)
+    public void DeleteRoutingItems()
     {
         throw new NotImplementedException();
     }
 
-    public string GetAgentTemplate(string agentId, string templateName)
+    public void DeleteRoutingProfiles()
     {
         throw new NotImplementedException();
     }
+    #endregion
 }