fix more linter errors

mish15 · mish15 · commit 4d36a27c9e45 · 2025-04-30T13:25:50.000+10:00
diff --git a/pkg/abtesting/stop_abtest.go b/pkg/abtesting/stop_abtest.go
@@ -11,7 +11,7 @@ import (
 	"github.com/mark3labs/mcp-go/server"
 )
 
-// RegisterStopABTest registers the stop_abtest tool with the MCP server
+// RegisterStopABTest registers the stop_abtest tool with the MCP server.
 func RegisterStopABTest(mcps *server.MCPServer) {
 	stopABTestTool := mcp.NewTool(
 		"abtesting_stop_abtest",
diff --git a/pkg/analytics/get_click_through_rate.go b/pkg/analytics/get_click_through_rate.go
@@ -12,7 +12,7 @@ import (
 	"github.com/mark3labs/mcp-go/server"
 )
 
-// RegisterGetClickThroughRate registers the get_click_through_rate tool with the MCP server
+// RegisterGetClickThroughRate registers the get_click_through_rate tool with the MCP server.
 func RegisterGetClickThroughRate(mcps *server.MCPServer) {
 	getClickThroughRateTool := mcp.NewTool(
 		"analytics_get_click_through_rate",
diff --git a/pkg/analytics/get_no_results_rate.go b/pkg/analytics/get_no_results_rate.go
@@ -12,7 +12,7 @@ import (
 	"github.com/mark3labs/mcp-go/server"
 )
 
-// RegisterGetNoResultsRate registers the get_no_results_rate tool with the MCP server
+// RegisterGetNoResultsRate registers the get_no_results_rate tool with the MCP server.
 func RegisterGetNoResultsRate(mcps *server.MCPServer) {
 	getNoResultsRateTool := mcp.NewTool(
 		"analytics_get_no_results_rate",
diff --git a/pkg/collections/upsert_collection.go b/pkg/collections/upsert_collection.go
@@ -13,7 +13,7 @@ import (
 	"github.com/mark3labs/mcp-go/server"
 )
 
-// RegisterUpsertCollection registers the upsert_collection tool with the MCP server
+// RegisterUpsertCollection registers the upsert_collection tool with the MCP server.
 func RegisterUpsertCollection(mcps *server.MCPServer) {
 	upsertCollectionTool := mcp.NewTool(
 		"collections_upsert_collection",
@@ -50,7 +50,7 @@ func RegisterUpsertCollection(mcps *server.MCPServer) {
 		),
 	)
 
-	mcps.AddTool(upsertCollectionTool, func(ctx context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
+	mcps.AddTool(upsertCollectionTool, func(_ context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
 		appID := os.Getenv("ALGOLIA_APP_ID")
 		apiKey := os.Getenv("ALGOLIA_WRITE_API_KEY") // Note: Using write API key for creating/updating collections
 		if appID == "" || apiKey == "" {
@@ -119,7 +119,7 @@ func RegisterUpsertCollection(mcps *server.MCPServer) {
 		// Create HTTP client and request
 		client := &http.Client{}
 		url := "https://experiences.algolia.com/1/collections"
-		httpReq, err := http.NewRequest("POST", url, bytes.NewBuffer(jsonBody))
+		httpReq, err := http.NewRequest(http.MethodPost, url, bytes.NewBuffer(jsonBody))
 		if err != nil {
 			return nil, fmt.Errorf("failed to create request: %w", err)
 		}
diff --git a/pkg/monitoring/get_cluster_incidents.go b/pkg/monitoring/get_cluster_incidents.go
@@ -11,7 +11,7 @@ import (
 	"github.com/mark3labs/mcp-go/server"
 )
 
-// RegisterGetClusterIncidents registers the get_cluster_incidents tool with the MCP server
+// RegisterGetClusterIncidents registers the get_cluster_incidents tool with the MCP server.
 func RegisterGetClusterIncidents(mcps *server.MCPServer) {
 	getClusterIncidentsTool := mcp.NewTool(
 		"monitoring_get_cluster_incidents",
@@ -33,7 +33,7 @@ func RegisterGetClusterIncidents(mcps *server.MCPServer) {
 		// Create HTTP client and request
 		client := &http.Client{}
 		url := fmt.Sprintf("https://status.algolia.com/1/incidents/%s", clusters)
-		httpReq, err := http.NewRequest("GET", url, nil)
+		httpReq, err := http.NewRequest(http.MethodGet, url, nil)
 		if err != nil {
 			return nil, fmt.Errorf("failed to create request: %w", err)
 		}
diff --git a/pkg/monitoring/get_cluster_status.go b/pkg/monitoring/get_cluster_status.go
@@ -11,7 +11,7 @@ import (
 	"github.com/mark3labs/mcp-go/server"
 )
 
-// RegisterGetClusterStatus registers the get_cluster_status tool with the MCP server
+// RegisterGetClusterStatus registers the get_cluster_status tool with the MCP server.
 func RegisterGetClusterStatus(mcps *server.MCPServer) {
 	getClusterStatusTool := mcp.NewTool(
 		"monitoring_get_cluster_status",
@@ -23,7 +23,7 @@ func RegisterGetClusterStatus(mcps *server.MCPServer) {
 		),
 	)
 
-	mcps.AddTool(getClusterStatusTool, func(ctx context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
+	mcps.AddTool(getClusterStatusTool, func(_ context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
 		// Extract parameters
 		clusters, _ := req.Params.Arguments["clusters"].(string)
 		if clusters == "" {
@@ -33,7 +33,7 @@ func RegisterGetClusterStatus(mcps *server.MCPServer) {
 		// Create HTTP client and request
 		client := &http.Client{}
 		url := fmt.Sprintf("https://status.algolia.com/1/status/%s", clusters)
-		httpReq, err := http.NewRequest("GET", url, nil)
+		httpReq, err := http.NewRequest(http.MethodGet, url, nil)
 		if err != nil {
 			return nil, fmt.Errorf("failed to create request: %w", err)
 		}
diff --git a/pkg/monitoring/get_clusters_status.go b/pkg/monitoring/get_clusters_status.go
@@ -11,18 +11,18 @@ import (
 	"github.com/mark3labs/mcp-go/server"
 )
 
-// RegisterGetClustersStatus registers the get_clusters_status tool with the MCP server
+// RegisterGetClustersStatus registers the get_clusters_status tool with the MCP server.
 func RegisterGetClustersStatus(mcps *server.MCPServer) {
 	getClustersStatusTool := mcp.NewTool(
 		"monitoring_get_clusters_status",
 		mcp.WithDescription("Retrieves the status of all Algolia clusters and instances"),
 	)
 
-	mcps.AddTool(getClustersStatusTool, func(ctx context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
+	mcps.AddTool(getClustersStatusTool, func(_ context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
 		// Create HTTP client and request
 		client := &http.Client{}
 		url := "https://status.algolia.com/1/status"
-		httpReq, err := http.NewRequest("GET", url, nil)
+		httpReq, err := http.NewRequest(http.MethodGet, url, nil)
 		if err != nil {
 			return nil, fmt.Errorf("failed to create request: %w", err)
 		}
diff --git a/pkg/usage/get_daily_metrics.go b/pkg/usage/get_daily_metrics.go
@@ -14,7 +14,7 @@ import (
 	"github.com/mark3labs/mcp-go/server"
 )
 
-// RegisterGetDailyMetrics registers the get_daily_metrics tool with the MCP server
+// RegisterGetDailyMetrics registers the get_daily_metrics tool with the MCP server.
 func RegisterGetDailyMetrics(mcps *server.MCPServer) {
 	getDailyMetricsTool := mcp.NewTool(
 		"usage_get_daily_metrics",
@@ -40,7 +40,7 @@ func RegisterGetDailyMetrics(mcps *server.MCPServer) {
 		),
 	)
 
-	mcps.AddTool(getDailyMetricsTool, func(ctx context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
+	mcps.AddTool(getDailyMetricsTool, func(_ context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
 		appID := os.Getenv("ALGOLIA_APP_ID")
 		apiKey := os.Getenv("ALGOLIA_API_KEY")
 		if appID == "" || apiKey == "" {
@@ -93,7 +93,7 @@ func RegisterGetDailyMetrics(mcps *server.MCPServer) {
 		}
 		url := fmt.Sprintf("%s?%s", baseURL, params.Encode())
 
-		httpReq, err := http.NewRequest("GET", url, nil)
+		httpReq, err := http.NewRequest(http.MethodGet, url, nil)
 		if err != nil {
 			return nil, fmt.Errorf("failed to create request: %w", err)
 		}
diff --git a/pkg/usage/get_hourly_metrics.go b/pkg/usage/get_hourly_metrics.go
@@ -14,7 +14,7 @@ import (
 	"github.com/mark3labs/mcp-go/server"
 )
 
-// RegisterGetHourlyMetrics registers the get_hourly_metrics tool with the MCP server
+// RegisterGetHourlyMetrics registers the get_hourly_metrics tool with the MCP server.
 func RegisterGetHourlyMetrics(mcps *server.MCPServer) {
 	getHourlyMetricsTool := mcp.NewTool(
 		"usage_get_hourly_metrics",
@@ -40,7 +40,7 @@ func RegisterGetHourlyMetrics(mcps *server.MCPServer) {
 		),
 	)
 
-	mcps.AddTool(getHourlyMetricsTool, func(ctx context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
+	mcps.AddTool(getHourlyMetricsTool, func(_ context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
 		appID := os.Getenv("ALGOLIA_APP_ID")
 		apiKey := os.Getenv("ALGOLIA_API_KEY")
 		if appID == "" || apiKey == "" {
@@ -85,7 +85,7 @@ func RegisterGetHourlyMetrics(mcps *server.MCPServer) {
 		}
 		url := fmt.Sprintf("%s?%s", baseURL, params.Encode())
 
-		httpReq, err := http.NewRequest("GET", url, nil)
+		httpReq, err := http.NewRequest(http.MethodGet, url, nil)
 		if err != nil {
 			return nil, fmt.Errorf("failed to create request: %w", err)
 		}
diff --git a/pkg/usage/get_metrics_registry.go b/pkg/usage/get_metrics_registry.go
@@ -14,7 +14,7 @@ import (
 	"github.com/mark3labs/mcp-go/server"
 )
 
-// RegisterGetMetricsRegistry registers the get_metrics_registry tool with the MCP server
+// RegisterGetMetricsRegistry registers the get_metrics_registry tool with the MCP server.
 func RegisterGetMetricsRegistry(mcps *server.MCPServer) {
 	getMetricsRegistryTool := mcp.NewTool(
 		"usage_get_metrics_registry",
@@ -26,7 +26,7 @@ func RegisterGetMetricsRegistry(mcps *server.MCPServer) {
 		),
 	)
 
-	mcps.AddTool(getMetricsRegistryTool, func(ctx context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
+	mcps.AddTool(getMetricsRegistryTool, func(_ context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
 		appID := os.Getenv("ALGOLIA_APP_ID")
 		apiKey := os.Getenv("ALGOLIA_API_KEY")
 		if appID == "" || apiKey == "" {
@@ -56,7 +56,7 @@ func RegisterGetMetricsRegistry(mcps *server.MCPServer) {
 		}
 		url := fmt.Sprintf("%s?%s", baseURL, params.Encode())
 
-		httpReq, err := http.NewRequest("GET", url, nil)
+		httpReq, err := http.NewRequest(http.MethodGet, url, nil)
 		if err != nil {
 			return nil, fmt.Errorf("failed to create request: %w", err)
 		}

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ import (`
`11`	`11`	`"github.com/mark3labs/mcp-go/server"`
`12`	`12`	`)`
`13`	`13`
`14`		`-// RegisterStopABTest registers the stop_abtest tool with the MCP server`
	`14`	`+// RegisterStopABTest registers the stop_abtest tool with the MCP server.`
`15`	`15`	`func RegisterStopABTest(mcps *server.MCPServer) {`
`16`	`16`	`stopABTestTool := mcp.NewTool(`
`17`	`17`	`"abtesting_stop_abtest",`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@ import (`
`12`	`12`	`"github.com/mark3labs/mcp-go/server"`
`13`	`13`	`)`
`14`	`14`
`15`		`-// RegisterGetClickThroughRate registers the get_click_through_rate tool with the MCP server`
	`15`	`+// RegisterGetClickThroughRate registers the get_click_through_rate tool with the MCP server.`
`16`	`16`	`func RegisterGetClickThroughRate(mcps *server.MCPServer) {`
`17`	`17`	`getClickThroughRateTool := mcp.NewTool(`
`18`	`18`	`"analytics_get_click_through_rate",`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@ import (`
`14`	`14`	`"github.com/mark3labs/mcp-go/server"`
`15`	`15`	`)`
`16`	`16`
`17`		`-// RegisterGetDailyMetrics registers the get_daily_metrics tool with the MCP server`
	`17`	`+// RegisterGetDailyMetrics registers the get_daily_metrics tool with the MCP server.`
`18`	`18`	`func RegisterGetDailyMetrics(mcps *server.MCPServer) {`
`19`	`19`	`getDailyMetricsTool := mcp.NewTool(`
`20`	`20`	`"usage_get_daily_metrics",`
`@@ -40,7 +40,7 @@ func RegisterGetDailyMetrics(mcps *server.MCPServer) {`
`40`	`40`	`),`
`41`	`41`	`)`
`42`	`42`
`43`		`- mcps.AddTool(getDailyMetricsTool, func(ctx context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {`
	`43`	`+ mcps.AddTool(getDailyMetricsTool, func(_ context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {`
`44`	`44`	`appID := os.Getenv("ALGOLIA_APP_ID")`
`45`	`45`	`apiKey := os.Getenv("ALGOLIA_API_KEY")`
`46`	`46`	`if appID == "" \|\| apiKey == "" {`
`@@ -93,7 +93,7 @@ func RegisterGetDailyMetrics(mcps *server.MCPServer) {`
`93`	`93`	`}`
`94`	`94`	`url := fmt.Sprintf("%s?%s", baseURL, params.Encode())`
`95`	`95`
`96`		`- httpReq, err := http.NewRequest("GET", url, nil)`
	`96`	`+ httpReq, err := http.NewRequest(http.MethodGet, url, nil)`
`97`	`97`	`if err != nil {`
`98`	`98`	`return nil, fmt.Errorf("failed to create request: %w", err)`
`99`	`99`	`}`