- Amended linting directives.

Author: general-kroll-4-life

.golangci.yml

@@ -174,7 +174,17 @@ linters:
         path: internal\/test\/.*\.go
       - linters:
           - mnd
+          - revive
+          - nolintlint
+          - lll
+          - gochecknoglobals
+          - unused
         path: pkg\/mcp_server\/.*\.go
+      - linters:
+          - gochecknoglobals
+          - lll
+          - revive
+        path: mcp_client\/cmd\/.*\.go
       - linters: 
           - staticcheck
         path: ast_format_postgres\.go

internal/stackql/cmd/mcp.go

@@ -27,6 +27,7 @@ import (
 	"github.com/stackql/stackql/pkg/mcp_server"
 )
 
+//nolint:gochecknoglobals // cobra pattern
 var (
 	mcpServerType string // overwritten by flag
 	mcpConfig     string // overwritten by flag

mcp_client/cmd/exec.go

@@ -48,6 +48,7 @@ var execCmd = &cobra.Command{
 		if outPutErr != nil {
 			panic(fmt.Sprintf("error marshaling output: %v", outPutErr))
 		}
+		//nolint:forbidigo // legacy
 		fmt.Println(string(output))
 	},
 }

pkg/mcp_server/client.go

@@ -93,14 +93,14 @@ func (c *httpMCPClient) InspectTools() ([]map[string]any, error) {
 	c.logger.Println("Getting time for each city...")
 	for _, city := range cities {
 		// Call the tool.
-		result, err := session.CallTool(ctx, &mcp.CallToolParams{
+		result, resultErr := session.CallTool(ctx, &mcp.CallToolParams{
 			Name: "cityTime",
 			Arguments: map[string]any{
 				"city": city,
 			},
 		})
-		if err != nil {
-			c.logger.Infof("Failed to get time for %s: %v\n", city, err)
+		if resultErr != nil {
+			c.logger.Infof("Failed to get time for %s: %v\n", city, resultErr)
 			continue
 		}
 

pkg/mcp_server/server.go

@@ -55,6 +55,7 @@ func (s *simpleMCPServer) runHTTPServer(server *mcp.Server, address string) erro
 	s.logger.Debugf("Available tool: cityTime (cities: nyc, sf, boston)")
 
 	// Start the HTTP server with logging handler.
+	//nolint:gosec // TODO: find viable alternative to http.ListenAndServe
 	if err := http.ListenAndServe(address, handlerWithLogging); err != nil {
 		s.logger.Errorf("Server failed: %v", err)
 		return err
@@ -76,6 +77,8 @@ func NewExampleBackendServer(config *Config, logger *logrus.Logger) (MCPServer,
 // }
 
 // NewMCPServer creates a new MCP server with the provided configuration and backend.
+//
+//nolint:gocognit,funlen // ok
 func newMCPServer(config *Config, backend Backend, logger *logrus.Logger) (MCPServer, error) {
 	if config == nil {
 		config = DefaultConfig()