fix: linting

Author: michiosw

internal/web/handlers/api.go

@@ -195,7 +195,9 @@ func (h *APIHandlers) AddMCPServer(w http.ResponseWriter, r *http.Request) {
 		h.logger.Error().Err(err).Msg("Failed to parse form data")
 		w.WriteHeader(http.StatusBadRequest)
 		// Use components package directly
-		components.AddServerErrorFeedback("Form parse error").Render(r.Context(), w)
+		if renderErr := components.AddServerErrorFeedback("Form parse error").Render(r.Context(), w); renderErr != nil {
+			h.logger.Error().Err(renderErr).Msg("Failed to render error feedback component")
+		}
 		return
 	}
 
@@ -205,7 +207,9 @@ func (h *APIHandlers) AddMCPServer(w http.ResponseWriter, r *http.Request) {
 	if name == "" || url == "" {
 		w.WriteHeader(http.StatusBadRequest)
 		// Use components package directly
-		components.AddServerErrorFeedback("Missing name or url").Render(r.Context(), w)
+		if renderErr := components.AddServerErrorFeedback("Missing name or url").Render(r.Context(), w); renderErr != nil {
+			h.logger.Error().Err(renderErr).Msg("Failed to render error feedback component")
+		}
 		return
 	}
 
@@ -220,7 +224,9 @@ func (h *APIHandlers) AddMCPServer(w http.ResponseWriter, r *http.Request) {
 		}
 		w.WriteHeader(statusCode)
 		// Use components package directly
-		components.AddServerErrorFeedback(errorMsg).Render(r.Context(), w)
+		if renderErr := components.AddServerErrorFeedback(errorMsg).Render(r.Context(), w); renderErr != nil {
+			h.logger.Error().Err(renderErr).Msg("Failed to render error feedback component")
+		}
 		return
 	}
 
@@ -229,7 +235,10 @@ func (h *APIHandlers) AddMCPServer(w http.ResponseWriter, r *http.Request) {
 	// Send HX-Trigger to reset form after successful request (Optional - requires JS listener)
 	// w.Header().Set("HX-Trigger", "resetAddForm")
 	// Use components package directly
-	components.AddServerSuccessFeedback(server.Name).Render(r.Context(), w)
+	// Also check error on success feedback rendering
+	if renderErr := components.AddServerSuccessFeedback(server.Name).Render(r.Context(), w); renderErr != nil {
+		h.logger.Error().Err(renderErr).Msg("Failed to render success feedback component")
+	}
 	// SSE refresh will handle list update separately
 }
 
@@ -254,15 +263,19 @@ func (h *APIHandlers) ListMCPServersFragment(w http.ResponseWriter, r *http.Requ
 	if err != nil {
 		h.logger.Error().Err(err).Msg("Error listing MCP servers for fragment")
 		w.WriteHeader(http.StatusInternalServerError)
-		// Use components package directly for error feedback
-		components.AddServerErrorFeedback("Failed to load server list").Render(r.Context(), w)
+		// Use components package directly for error feedback, and check error
+		if renderErr := components.AddServerErrorFeedback("Failed to load server list").Render(r.Context(), w); renderErr != nil {
+			h.logger.Error().Err(renderErr).Msg("Failed to render error feedback in list fragment")
+		}
 		return
 	}
 
 	// Set content type to HTML
 	w.Header().Set("Content-Type", "text/html; charset=utf-8")
-	// Render the list body component directly using parent templates package
-	templates.ServerListBodyComponent(servers).Render(r.Context(), w)
+	// Render the list body component directly using parent templates package, and check error
+	if renderErr := templates.ServerListBodyComponent(servers).Render(r.Context(), w); renderErr != nil {
+		h.logger.Error().Err(renderErr).Msg("Failed to render server list body component")
+	}
 }
 
 // GetMCPServer handles GET /api/mcp/servers/:id