Return IReadOnlyCollection instead of IEnumerable in Catalogs (#175)

Author: MikeAlhayek

src/Abstractions/CrestApps.OrchardCore.AI.Abstractions/AIToolSourceType.cs

@@ -1,4 +1,4 @@
-namespace CrestApps.OrchardCore.AI;
+namespace CrestApps.OrchardCore.AI;
 
 public enum AIToolSourceType
 {

src/Abstractions/CrestApps.OrchardCore.AI.Abstractions/IAIDataSourceStore.cs

@@ -46,13 +46,13 @@ public interface IAIDataSourceStore : IReadCatalog<AIDataSource>
     /// </summary>
     /// <param name="providerName"></param>
     /// <returns></returns>
-    ValueTask<IEnumerable<AIDataSource>> GetAsync(string providerName);
+    ValueTask<IReadOnlyCollection<AIDataSource>> GetAsync(string providerName);
 
     /// <summary>
     /// Asynchronously finds data sources by the given provider-name and type.
     /// </summary>
     /// <param name="providerName"></param>
     /// <param name="type"></param>
     /// <returns></returns>
-    ValueTask<IEnumerable<AIDataSource>> GetAsync(string providerName, string type);
+    ValueTask<IReadOnlyCollection<AIDataSource>> GetAsync(string providerName, string type);
 }

src/Abstractions/CrestApps.OrchardCore.AI.Abstractions/Models/AIChatProfileSettings.cs

@@ -1,4 +1,4 @@
-namespace CrestApps.OrchardCore.AI.Models;
+namespace CrestApps.OrchardCore.AI.Models;
 
 public class AIChatProfileSettings
 {

src/Abstractions/CrestApps.OrchardCore.AI.Abstractions/Models/AIDataSource.cs

@@ -1,9 +1,10 @@
 using System.Text.Json.Nodes;
 using CrestApps.OrchardCore.Models;
+using CrestApps.OrchardCore.Services;
 
 namespace CrestApps.OrchardCore.AI.Models;
 
-public sealed class AIDataSource : CatalogEntry, IDisplayTextAwareModel
+public sealed class AIDataSource : CatalogEntry, IDisplayTextAwareModel, ICloneable<AIDataSource>
 {
     public string ProfileSource { get; set; }
 

src/Abstractions/CrestApps.OrchardCore.AI.Abstractions/Models/AIDeployment.cs

@@ -1,10 +1,11 @@
 using System.Text.Json.Nodes;
 using System.Text.Json.Serialization;
 using CrestApps.OrchardCore.Models;
+using CrestApps.OrchardCore.Services;
 
 namespace CrestApps.OrchardCore.AI.Models;
 
-public class AIDeployment : SourceCatalogEntry, INameAwareModel, ISourceAwareModel
+public class AIDeployment : SourceCatalogEntry, INameAwareModel, ISourceAwareModel, ICloneable<AIDeployment>
 {
     [JsonIgnore]
     public string ProviderName

src/Abstractions/CrestApps.OrchardCore.AI.Abstractions/Models/AIProfile.cs

@@ -1,9 +1,10 @@
 using System.Text.Json.Nodes;
 using CrestApps.OrchardCore.Models;
+using CrestApps.OrchardCore.Services;
 
 namespace CrestApps.OrchardCore.AI.Models;
 
-public sealed class AIProfile : SourceCatalogEntry, INameAwareModel, IDisplayTextAwareModel
+public sealed class AIProfile : SourceCatalogEntry, INameAwareModel, IDisplayTextAwareModel, ICloneable<AIProfile>
 {
     /// <summary>
     /// Gets or sets the technical name of the profile.

src/Abstractions/CrestApps.OrchardCore.AI.Abstractions/Models/AIProviderConnection.cs

@@ -1,10 +1,11 @@
 using System.Text.Json.Nodes;
 using System.Text.Json.Serialization;
 using CrestApps.OrchardCore.Models;
+using CrestApps.OrchardCore.Services;
 
 namespace CrestApps.OrchardCore.AI.Models;
 
-public sealed class AIProviderConnection : SourceCatalogEntry, INameAwareModel, IDisplayTextAwareModel
+public sealed class AIProviderConnection : SourceCatalogEntry, INameAwareModel, IDisplayTextAwareModel, ICloneable<AIProviderConnection>
 {
     public string Name { get; set; }
 

src/Abstractions/CrestApps.OrchardCore.AI.Abstractions/Models/AIToolInstance.cs

@@ -1,8 +1,9 @@
 using CrestApps.OrchardCore.Models;
+using CrestApps.OrchardCore.Services;
 
 namespace CrestApps.OrchardCore.AI.Models;
 
-public sealed class AIToolInstance : SourceCatalogEntry, IDisplayTextAwareModel
+public sealed class AIToolInstance : SourceCatalogEntry, IDisplayTextAwareModel, ICloneable<AIToolInstance>
 {
     public string DisplayText { get; set; }
 

src/Abstractions/CrestApps.OrchardCore.Abstractions/ISourceAwareModel.cs

@@ -1,4 +1,4 @@
-namespace CrestApps.OrchardCore;
+namespace CrestApps.OrchardCore;
 
 public interface ISourceAwareModel
 {

src/Abstractions/CrestApps.OrchardCore.Abstractions/Models/PageResult.cs

@@ -4,5 +4,5 @@ public class PageResult<T>
 {
     public int Count { get; set; }
 
-    public IEnumerable<T> Entries { get; set; }
+    public IReadOnlyCollection<T> Entries { get; set; }
 }