Browse Source

Refactor WorkspaceConfiguration property initialization

Changed ChatClient and Kernel properties in WorkspaceConfiguration to be non-nullable and initialized by default. Removed null checks and redundant instantiation logic in configuration methods for improved code clarity and safety.
pull/23533/head
enisn 1 year ago
parent
commit
7800e95641
No known key found for this signature in database GPG Key ID: A052619F04155D1C
  1. 14
      framework/src/Volo.Abp.AI/Volo/Abp/AI/AbpAIOptions.cs

14
framework/src/Volo.Abp.AI/Volo/Abp/AI/AbpAIOptions.cs

@ -44,8 +44,8 @@ public class WorkspaceConfigurationDictionary : Dictionary<string, WorkspaceConf
public class WorkspaceConfiguration public class WorkspaceConfiguration
{ {
public string Name { get; } public string Name { get; }
public ChatClientConfiguration? ChatClient { get; set; } public ChatClientConfiguration ChatClient { get; } = new();
public KernelConfiguration? Kernel { get; set; } public KernelConfiguration Kernel { get; } = new();
public WorkspaceConfiguration(string name) public WorkspaceConfiguration(string name)
{ {
@ -54,11 +54,6 @@ public class WorkspaceConfiguration
public WorkspaceConfiguration ConfigureChatClient(Action<ChatClientConfiguration> configureAction) public WorkspaceConfiguration ConfigureChatClient(Action<ChatClientConfiguration> configureAction)
{ {
if (ChatClient is null)
{
ChatClient = new ChatClientConfiguration();
}
configureAction(ChatClient); configureAction(ChatClient);
return this; return this;
} }
@ -66,11 +61,6 @@ public class WorkspaceConfiguration
public WorkspaceConfiguration ConfigureKernel(Action<KernelConfiguration> configureAction) public WorkspaceConfiguration ConfigureKernel(Action<KernelConfiguration> configureAction)
{ {
if (Kernel is null)
{
Kernel = new KernelConfiguration();
}
configureAction(Kernel); configureAction(Kernel);
return this; return this;
} }

Loading…
Cancel
Save