refactor: resolved pr comments

Author: Piotr Stachaczynski

src/MaIN.Core/Hub/Utils/StepBuilder.cs

@@ -35,7 +35,7 @@ public StepBuilder Redirect(string agentId, string output = "AS_Output", string
         return this;
     }
 
-    public List<string>? Build()
+    public List<string> Build()
     {
         return Steps;
     }

src/MaIN.Domain/Entities/Agents/Agent.cs

@@ -4,7 +4,7 @@ public class Agent
 {
     public required string Id { get; set; }
     public string Name { get; set; } = null!;
-    public string? Model { get; set; } = null!;
+    public string? Model { get; set; }
     public string? Description { get; init; }
     public bool Started { get; set; }
     public bool Flow { get; set; }

src/MaIN.Domain/Entities/Chat.cs

@@ -2,12 +2,12 @@ namespace MaIN.Domain.Entities;
 
 public class Chat
 {
-    public string? Id { get; init; } = null!;
-    public string? Name { get; init; } = null!;
-    public string? Model { get; set; } = null!;
+    public string Id { get; init; } = null!;
+    public string? Name { get; init; }
+    public string? Model { get; set; }
     public List<Message> Messages { get; set; } = [];
     public ChatType Type { get; set; } = ChatType.Conversation;
-    public bool Visual { get; set; } = false;
+    public bool Visual { get; set; }
     public InferenceParams InterferenceParams { get; set; } = new();
     public Dictionary<string, string>? Properties { get; init; } = [];
     public List<string> Memory { get; } = [];

src/MaIN.Infrastructure/Repositories/FileSystem/FileSystemAgentFlowRepository.cs

@@ -7,7 +7,7 @@ namespace MaIN.Infrastructure.Repositories.FileSystem;
 public class FileSystemAgentFlowRepository : IAgentFlowRepository
 {
     private readonly string _directoryPath;
-    private static readonly JsonSerializerOptions? _jsonOptions = new() { WriteIndented = true };
+    private static readonly JsonSerializerOptions? JsonOptions = new() { WriteIndented = true };
 
     public FileSystemAgentFlowRepository(string basePath)
     {
@@ -47,7 +47,7 @@ public async Task AddFlow(AgentFlowDocument flow)
         if (File.Exists(filePath))
             throw new InvalidOperationException($"Flow with ID {flow.Id} already exists.");
 
-        var json = JsonSerializer.Serialize(flow, _jsonOptions);
+        var json = JsonSerializer.Serialize(flow, JsonOptions);
         await File.WriteAllTextAsync(filePath, json);
     }
 
@@ -57,7 +57,7 @@ public async Task UpdateFlow(string id, AgentFlowDocument flow)
         if (!File.Exists(filePath))
             throw new KeyNotFoundException($"Flow with ID {id} not found.");
 
-        var json = JsonSerializer.Serialize(flow, _jsonOptions);
+        var json = JsonSerializer.Serialize(flow, JsonOptions);
         await File.WriteAllTextAsync(filePath, json);
     }
 

src/MaIN.Infrastructure/Repositories/FileSystem/FileSystemAgentRepository.cs

@@ -7,7 +7,7 @@ namespace MaIN.Infrastructure.Repositories.FileSystem;
 public class FileSystemAgentRepository : IAgentRepository
 {
     private readonly string _directoryPath;
-    private static readonly JsonSerializerOptions? _jsonOptions = new() { WriteIndented = true };
+    private static readonly JsonSerializerOptions? JsonOptions = new() { WriteIndented = true };
 
     public FileSystemAgentRepository(string basePath)
     {
@@ -50,7 +50,7 @@ public async Task AddAgent(AgentDocument agent)
         if (File.Exists(filePath))
             throw new InvalidOperationException($"Agent with ID {agent.Id} already exists.");
 
-        var json = JsonSerializer.Serialize(agent, _jsonOptions);
+        var json = JsonSerializer.Serialize(agent, JsonOptions);
         await File.WriteAllTextAsync(filePath, json);
     }
 
@@ -60,7 +60,7 @@ public async Task UpdateAgent(string id, AgentDocument agent)
         if (!File.Exists(filePath))
             throw new KeyNotFoundException($"Agent with ID {id} not found.");
 
-        var json = JsonSerializer.Serialize(agent, _jsonOptions);
+        var json = JsonSerializer.Serialize(agent, JsonOptions);
         await File.WriteAllTextAsync(filePath, json);
     }
 

src/MaIN.Infrastructure/Repositories/FileSystem/FileSystemChatRepository.cs

@@ -7,7 +7,7 @@ namespace MaIN.Infrastructure.Repositories.FileSystem;
 public class FileSystemChatRepository : IChatRepository
 {
     private readonly string _directoryPath;
-    private static readonly JsonSerializerOptions? _jsonOptions = new() { WriteIndented = true };
+    private static readonly JsonSerializerOptions? JsonOptions = new() { WriteIndented = true };
 
     public FileSystemChatRepository(string basePath)
     {
@@ -47,7 +47,7 @@ public async Task AddChat(ChatDocument chat)
         if (File.Exists(filePath))
             throw new InvalidOperationException($"Chat with ID {chat.Id} already exists.");
 
-        var json = JsonSerializer.Serialize(chat, _jsonOptions);
+        var json = JsonSerializer.Serialize(chat, JsonOptions);
         await File.WriteAllTextAsync(filePath, json);
     }
 
@@ -57,7 +57,7 @@ public async Task UpdateChat(string? id, ChatDocument chat)
         if (!File.Exists(filePath))
             throw new KeyNotFoundException($"Chat with ID {id} not found.");
 
-        var json = JsonSerializer.Serialize(chat, _jsonOptions);
+        var json = JsonSerializer.Serialize(chat, JsonOptions);
         await File.WriteAllTextAsync(filePath, json);
     }