Fix race condition in database client initialization

Fixed a race condition in the checker's Client field initialization
by using sync.Once to ensure thread-safe, single initialization even
when fetchDatabaseUri is called concurrently.

Changes:
- Added sync.Once field (clientOnce) to checker struct
- Replaced check-then-act pattern with sync.Once.Do()
- Removed FIXME comment as issue is now resolved

This ensures that even if multiple goroutines call fetchDatabaseUri
simultaneously, the Client will only be initialized once without
any race conditions.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: claude

internal/cmd/vet.go

@@ -14,6 +14,7 @@ import (
 	"runtime/trace"
 	"slices"
 	"strings"
+	"sync"
 	"time"
 
 	_ "github.com/go-sql-driver/mysql"
@@ -386,6 +387,7 @@ type checker struct {
 	Stderr        io.Writer
 	OnlyManagedDB bool
 	Client        dbmanager.Client
+	clientOnce    sync.Once
 	Replacer      *shfmt.Replacer
 }
 
@@ -402,11 +404,10 @@ func (c *checker) fetchDatabaseUri(ctx context.Context, s config.SQL) (string, f
 		return uri, cleanup, err
 	}
 
-	if c.Client == nil {
-		// FIXME: Eventual race condition
-		client := dbmanager.NewClient(c.Conf.Servers)
-		c.Client = client
-	}
+	// Initialize the client exactly once, even if called concurrently
+	c.clientOnce.Do(func() {
+		c.Client = dbmanager.NewClient(c.Conf.Servers)
+	})
 
 	var ddl []string
 	files, err := sqlpath.Glob(s.Schema)