fix: resolve CI failures — lint and Go 1.21 incompatibility

- cmd/gosqlx-mcp/main.go: extract run() helper so os.Exit is not called
  after defer stop() — fixes gocritic exitAfterDefer lint error
- .github/workflows/test.yml: bump minimum Go from 1.21 to 1.23 since
  mark3labs/mcp-go requires go 1.23.0
- CLAUDE.md: document new minimum Go version requirement

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: 2 people

.github/workflows/test.yml

@@ -11,7 +11,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        go-version: ['1.21', '1.24']
+        go-version: ['1.23', '1.24']
 
     steps:
     - uses: actions/checkout@v4
@@ -34,7 +34,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu-latest, macos-latest, windows-latest]
-        go: ['1.21', '1.24']
+        go: ['1.23', '1.24']
     env:
       # Prevent Go from auto-downloading toolchain which conflicts with setup-go cache
       GOTOOLCHAIN: local

CLAUDE.md

@@ -6,7 +6,7 @@ This file provides guidance to Claude Code (claude.ai/code) when working with co
 
 GoSQLX is a **production-ready**, **race-free**, high-performance SQL parsing SDK for Go that provides lexing, parsing, and AST generation with zero-copy optimizations. The library is designed for enterprise use with comprehensive object pooling for memory efficiency.
 
-**Requirements**: Go 1.21+
+**Requirements**: Go 1.23+ (upgraded from 1.21 when MCP server was added; `mark3labs/mcp-go` requires 1.23)
 
 **Production Status**: ✅ Validated for production deployment (v1.6.0+, current: v1.9.0)
 - Thread-safe with zero race conditions (20,000+ concurrent operations tested)

cmd/gosqlx-mcp/main.go

@@ -41,18 +41,20 @@ import (
 )
 
 func main() {
+	if err := run(); err != nil {
+		fmt.Fprintf(os.Stderr, "gosqlx-mcp: %v\n", err)
+		os.Exit(1)
+	}
+}
+
+func run() error {
 	cfg, err := gosqlxmcp.LoadConfig()
 	if err != nil {
-		fmt.Fprintf(os.Stderr, "gosqlx-mcp: configuration error: %v\n", err)
-		os.Exit(1)
+		return fmt.Errorf("configuration error: %w", err)
 	}
 
 	ctx, stop := signal.NotifyContext(context.Background(), os.Interrupt, syscall.SIGTERM)
 	defer stop()
 
-	srv := gosqlxmcp.New(cfg)
-	if err := srv.Start(ctx); err != nil {
-		fmt.Fprintf(os.Stderr, "gosqlx-mcp: %v\n", err)
-		os.Exit(1)
-	}
+	return gosqlxmcp.New(cfg).Start(ctx)
 }