Apply suggestions from code review

Co-authored-by: Copilot <[email protected]>

Author: jongalloway

src/NLWebNet/Extensions/ServiceCollectionExtensions.cs

@@ -40,7 +40,10 @@ public static IServiceCollection AddNLWebNet(this IServiceCollection services, A
         services.AddScoped<IToolSelector, ToolSelector>();
 
         // Register Advanced Tool System
-        services.AddAdvancedToolSystem();
+        if (!services.Any(service => service.ServiceType == typeof(IAdvancedToolService)))
+        {
+            services.AddAdvancedToolSystem();
+        }
 
         // Register MCP services
         services.AddScoped<IMcpService, McpService>();        // Register default data backend (can be overridden)

src/NLWebNet/Services/DetailsToolHandler.cs

@@ -44,7 +44,6 @@ public override async Task<NLWebResponse> ExecuteAsync(NLWebRequest request, Can
 
             // Create details-focused query
             var detailsQuery = $"{subject} overview definition explanation details";
-            var processedQuery = await QueryProcessor.ProcessQueryAsync(request, cancellationToken);
 
             // Generate detailed results
             var searchResults = await ResultGenerator.GenerateListAsync(detailsQuery, request.Site, cancellationToken);

src/NLWebNet/Services/NLWebService.cs

@@ -117,32 +117,10 @@ public async Task<NLWebResponse> ProcessRequestAsync(NLWebRequest request, Cance
             _logger.LogDebug("ProcessQueryAsync complete for QueryId={QueryId}", queryId);
 
             // Check if tool execution is available and enabled
-            if (_toolSelector != null && _toolExecutor != null && _options.ToolSelectionEnabled)
+            var toolResponse = await HandleToolSelectionAsync(request, queryId, cancellationToken);
+            if (toolResponse != null)
             {
-                _logger.LogDebug("Tool execution enabled, checking if tool selection is needed for QueryId={QueryId}", queryId);
-
-                if (_toolSelector.ShouldSelectTool(request))
-                {
-                    _logger.LogDebug("Tool selection needed for QueryId={QueryId}", queryId);
-
-                    var selectedTool = await _toolSelector.SelectToolAsync(request, cancellationToken);
-                    if (!string.IsNullOrEmpty(selectedTool))
-                    {
-                        _logger.LogInformation("Tool '{Tool}' selected for QueryId={QueryId}, executing tool", selectedTool, queryId);
-
-                        try
-                        {
-                            var toolResponse = await _toolExecutor.ExecuteToolAsync(request, selectedTool, cancellationToken);
-                            _logger.LogInformation("[END] Tool execution completed for QueryId={QueryId} with tool '{Tool}'", queryId, selectedTool);
-                            return toolResponse;
-                        }
-                        catch (Exception toolEx)
-                        {
-                            _logger.LogError(toolEx, "Tool execution failed for QueryId={QueryId} with tool '{Tool}', falling back to standard processing", queryId, selectedTool);
-                            // Fall through to standard processing
-                        }
-                    }
-                }
+                return toolResponse;
             }
 
             _logger.LogDebug("Using standard processing pipeline for QueryId={QueryId}", queryId);