Skip to content

Commit 789346f

Browse files
committed
chore: update service collection extensions and NLWebService for improved maintainability
1 parent 89b88b4 commit 789346f

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

src/NLWebNet/Extensions/ServiceCollectionExtensions.cs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -39,11 +39,8 @@ public static IServiceCollection AddNLWebNet(this IServiceCollection services, A
3939
services.AddScoped<IResultGenerator, ResultGenerator>();
4040
services.AddScoped<IToolSelector, ToolSelector>();
4141

42-
// Register Advanced Tool System
43-
if (!services.Any(service => service.ServiceType == typeof(IAdvancedToolService)))
44-
{
45-
services.AddAdvancedToolSystem();
46-
}
42+
// Register Advanced Tool System (idempotent)
43+
services.AddAdvancedToolSystem();
4744

4845
// Register MCP services
4946
services.AddScoped<IMcpService, McpService>(); // Register default data backend (can be overridden)

src/NLWebNet/Services/NLWebService.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ public async Task<NLWebResponse> ProcessRequestAsync(NLWebRequest request, Cance
117117
_logger.LogDebug("ProcessQueryAsync complete for QueryId={QueryId}", queryId);
118118

119119
// Check if tool execution is available and enabled
120-
var toolResponse = await HandleToolSelectionAsync(request, queryId, cancellationToken);
120+
var toolResponse = await TryExecuteToolAsync(request, queryId, cancellationToken);
121121
if (toolResponse != null)
122122
{
123123
return toolResponse;

0 commit comments

Comments
 (0)