fix: Address linting issues in web search implementation

- Fix errcheck issues by properly handling error returns in defer statements
- Fix staticcheck issue by removing unnecessary fmt.Sprintf usage

Co-authored-by: Eden Reich <edenreich@users.noreply.github.com>

Author: claude[bot]

config/websearch_test.go

@@ -44,7 +44,7 @@ func TestLoadConfig_WebSearch(t *testing.T) {
 	if err != nil {
 		t.Fatalf("Failed to create temp dir: %v", err)
 	}
-	defer os.RemoveAll(tempDir)
+	defer func() { _ = os.RemoveAll(tempDir) }()
 
 	configPath := filepath.Join(tempDir, "config.yaml")
 	configContent := `
@@ -138,7 +138,7 @@ func TestSaveConfig_WebSearch(t *testing.T) {
 	if err != nil {
 		t.Fatalf("Failed to create temp dir: %v", err)
 	}
-	defer os.RemoveAll(tempDir)
+	defer func() { _ = os.RemoveAll(tempDir) }()
 
 	configPath := filepath.Join(tempDir, "config.yaml")
 
@@ -189,7 +189,7 @@ func TestLoadConfig_MissingWebSearchSection(t *testing.T) {
 	if err != nil {
 		t.Fatalf("Failed to create temp dir: %v", err)
 	}
-	defer os.RemoveAll(tempDir)
+	defer func() { _ = os.RemoveAll(tempDir) }()
 
 	configPath := filepath.Join(tempDir, "config.yaml")
 	configContent := `

internal/services/websearch.go

@@ -104,7 +104,7 @@ func (s *WebSearchService) performGoogleSearch(ctx context.Context, query string
 	if err != nil {
 		return nil, fmt.Errorf("search request failed: %w", err)
 	}
-	defer resp.Body.Close()
+	defer func() { _ = resp.Body.Close() }()
 
 	if resp.StatusCode != http.StatusOK {
 		// For non-200 responses, return mock results instead of failing
@@ -132,7 +132,7 @@ func (s *WebSearchService) performDuckDuckGoSearch(ctx context.Context, query st
 	if err != nil {
 		return nil, fmt.Errorf("search request failed: %w", err)
 	}
-	defer resp.Body.Close()
+	defer func() { _ = resp.Body.Close() }()
 
 	if resp.StatusCode != http.StatusOK {
 		// For non-200 responses, return mock results instead of failing
@@ -199,7 +199,7 @@ func (s *WebSearchService) parseDuckDuckGoResponse(response map[string]interface
 		if abstract, ok := response["Abstract"].(string); ok && abstract != "" {
 			if abstractURL, ok := response["AbstractURL"].(string); ok && abstractURL != "" {
 				results = append(results, domain.WebSearchResult{
-					Title:   fmt.Sprintf("DuckDuckGo Result"),
+					Title:   "DuckDuckGo Result",
 					URL:     abstractURL,
 					Snippet: abstract,
 				})