[Guru] Use List to improve performance & Minor changes

Author: belaoud001

src/GuruPR.Application/Interfaces/Application/IAgentService.cs

@@ -5,7 +5,7 @@ namespace GuruPR.Application.Interfaces.Application;
 
 public interface IAgentService
 {
-    Task<IEnumerable<Agent>> GetAllAgentsAsync(string? userId = null);
+    Task<List<Agent>> GetAllAgentsAsync(string? userId = null);
 
     Task<Agent> GetAgentByIdAsync(string agentId);
 

src/GuruPR.Application/Interfaces/Application/IConversationService.cs

@@ -6,7 +6,7 @@ namespace GuruPR.Application.Interfaces.Application;
 
 public interface IConversationService
 {
-    Task<IEnumerable<Conversation>> GetAllConversationsByUserIdAsync(string userId);
+    Task<List<Conversation>> GetAllConversationsByUserIdAsync(string userId);
 
     Task<Conversation> GetConversationByIdAsync(string conversationId, string userId);
 

src/GuruPR.Application/Interfaces/Application/IProviderConnectionService.cs

@@ -6,7 +6,7 @@ namespace GuruPR.Application.Interfaces.Application;
 
 public interface IProviderConnectionService
 {
-    Task<IEnumerable<ProviderConnection>> GetConnectionsByProviderIdAsync(string providerId);
+    Task<List<ProviderConnection>> GetConnectionsByProviderIdAsync(string providerId);
 
     Task<ProviderConnection> GetProviderConnectionByIdAsync(string providerId, string providerConnectionId);
 

src/GuruPR.Application/Interfaces/Application/IProviderService.cs

@@ -8,7 +8,7 @@ public interface IProviderService
 {
     Task<Provider> CreateProviderAsync(CreateProviderRequest createProviderRequest);
 
-    Task<IEnumerable<Provider>> GetAllProvidersAsync();
+    Task<List<Provider>> GetAllProvidersAsync();
 
     Task<Provider> GetProviderByIdAsync(string providerId);
 

src/GuruPR.Application/Interfaces/Persistence/IAgentRepository.cs

@@ -4,5 +4,5 @@ namespace GuruPR.Application.Interfaces.Persistence;
 
 public interface IAgentRepository : IGenericRepository<Agent>
 {
-
+    Task<List<Agent>> GetAllAgentsAsync(string userId);
 }

src/GuruPR.Application/Interfaces/Persistence/IConversationRepository.cs

@@ -4,5 +4,5 @@ namespace GuruPR.Application.Interfaces.Persistence;
 
 public interface IConversationRepository : IGenericRepository<Conversation>
 {
-    Task<IEnumerable<Conversation>> GetAllByUserIdAsync(string userId);
+    Task<List<Conversation>> GetAllByUserIdAsync(string userId);
 }

src/GuruPR.Application/Interfaces/Persistence/IGenericRepository.cs

@@ -4,7 +4,7 @@ public interface IGenericRepository<T> where T : class
 {
     Task<T?> GetByIdAsync(string id);
 
-    Task<IEnumerable<T>> GetAllAsync();
+    Task<List<T>> GetAllAsync();
 
     Task<T> AddAsync(T entity);
 

src/GuruPR.Application/Services/AgentService.cs

@@ -34,7 +34,7 @@ public AgentService(ILogger<AgentService> logger,
 
     #region Public Methods
 
-    public async Task<IEnumerable<Agent>> GetAllAgentsAsync(string? userId = null)
+    public async Task<List<Agent>> GetAllAgentsAsync(string? userId = null)
     {
         if (string.IsNullOrEmpty(userId))
         {

src/GuruPR.Application/Services/ConversationService.cs

@@ -36,7 +36,7 @@ public ConversationService(ILogger<ConversationService> logger,
 
     #region Public Methods
 
-    public async Task<IEnumerable<Conversation>> GetAllConversationsByUserIdAsync(string userId)
+    public async Task<List<Conversation>> GetAllConversationsByUserIdAsync(string userId)
     {
         var conversations = await _unitOfWork.Conversations.GetAllByUserIdAsync(userId);
 

src/GuruPR.Application/Services/OAuth/ProviderConnectionService.cs

@@ -22,7 +22,7 @@ public ProviderConnectionService(IMapper mapper, IUnitOfWork unitOfWork)
 
     #region Public Methods
 
-    public async Task<IEnumerable<ProviderConnection>> GetConnectionsByProviderIdAsync(string providerId)
+    public async Task<List<ProviderConnection>> GetConnectionsByProviderIdAsync(string providerId)
     {
         var provider = await GetProviderByIdOrThrowExceptionAsync(providerId);