Bump mcp-go 0.29.0 -> 0.32.0

wendorf · wendorf · commit dd3adaf5a252 · 2025-06-23T11:02:27.000-07:00
diff --git a/go.mod b/go.mod
@@ -4,7 +4,7 @@ go 1.24.1
 
 require (
 	github.com/jackc/pgx/v5 v5.7.4
-	github.com/mark3labs/mcp-go v0.29.0
+	github.com/mark3labs/mcp-go v0.32.0
 	github.com/oapi-codegen/runtime v1.1.1
 	github.com/stretchr/testify v1.10.0
 	gopkg.in/yaml.v3 v3.0.1
diff --git a/go.sum b/go.sum
@@ -24,8 +24,8 @@ github.com/kr/pretty v0.3.1 h1:flRD4NNwYAUpkphVc1HcthR4KEIFJ65n8Mw5qdRn3LE=
 github.com/kr/pretty v0.3.1/go.mod h1:hoEshYVHaxMs3cyo3Yncou5ZscifuDolrwPKZanG3xk=
 github.com/kr/text v0.2.0 h1:5Nx0Ya0ZqY2ygV366QzturHI13Jq95ApcVaJBhpS+AY=
 github.com/kr/text v0.2.0/go.mod h1:eLer722TekiGuMkidMxC/pM04lWEeraHUUmBw8l2grE=
-github.com/mark3labs/mcp-go v0.29.0 h1:sH1NBcumKskhxqYzhXfGc201D7P76TVXiT0fGVhabeI=
-github.com/mark3labs/mcp-go v0.29.0/go.mod h1:rXqOudj/djTORU/ThxYx8fqEVj/5pvTuuebQ2RC7uk4=
+github.com/mark3labs/mcp-go v0.32.0 h1:fgwmbfL2gbd67obg57OfV2Dnrhs1HtSdlY/i5fn7MU8=
+github.com/mark3labs/mcp-go v0.32.0/go.mod h1:rXqOudj/djTORU/ThxYx8fqEVj/5pvTuuebQ2RC7uk4=
 github.com/maxbrunsfeld/counterfeiter/v6 v6.11.2 h1:yVCLo4+ACVroOEr4iFU1iH46Ldlzz2rTuu18Ra7M8sU=
 github.com/maxbrunsfeld/counterfeiter/v6 v6.11.2/go.mod h1:VzB2VoMh1Y32/QqDfg9ZJYHj99oM4LiGtqPZydTiQSQ=
 github.com/oapi-codegen/runtime v1.1.1 h1:EXLHh0DXIJnWhdRPN2w4MXAzFyE4CskzhNLUmtpMYro=
diff --git a/pkg/postgres/tools.go b/pkg/postgres/tools.go
@@ -232,18 +232,18 @@ func queryPostgres(postgresRepo *Repo) server.ServerTool {
 
 			config, err := pgx.ParseConfig(connectionInfo.ExternalConnectionString)
 			if err != nil {
-				return mcp.NewToolResultError(fmt.Sprintf("Error parsing connection string: %s", err.Error())), nil
+				return mcp.NewToolResultErrorf("Error parsing connection string: %s", err.Error()), nil
 			}
 			conn, err := pgx.ConnectConfig(ctx, config)
 			if err != nil {
-				return mcp.NewToolResultError(fmt.Sprintf("Error connecting to database: %s", err.Error())), nil
+				return mcp.NewToolResultErrorf("Error connecting to database: %s", err.Error()), nil
 			}
 			defer conn.Close(ctx)
 
 			// Wrap all queries in a READ ONLY transaction
 			tx, err := conn.BeginTx(ctx, pgx.TxOptions{AccessMode: pgx.ReadOnly})
 			if err != nil {
-				return mcp.NewToolResultError(fmt.Sprintf("Error beginning transaction: %s", err.Error())), nil
+				return mcp.NewToolResultErrorf("Error beginning transaction: %s", err.Error()), nil
 			}
 
 			// Make sure we roll back the transaction if it's not committed successfully
@@ -253,7 +253,7 @@ func queryPostgres(postgresRepo *Repo) server.ServerTool {
 
 			rows, err := tx.Query(ctx, sqlQuery)
 			if err != nil {
-				return mcp.NewToolResultError(fmt.Sprintf("Error executing query: %s", err.Error())), nil
+				return mcp.NewToolResultErrorf("Error executing query: %s", err.Error()), nil
 			}
 			defer rows.Close()
 
@@ -269,7 +269,7 @@ func queryPostgres(postgresRepo *Repo) server.ServerTool {
 			for rows.Next() {
 				values, err := rows.Values()
 				if err != nil {
-					return mcp.NewToolResultError(fmt.Sprintf("Error reading row values: %s", err.Error())), nil
+					return mcp.NewToolResultErrorf("Error reading row values: %s", err.Error()), nil
 				}
 
 				rowMap := make(map[string]interface{})
@@ -290,12 +290,12 @@ func queryPostgres(postgresRepo *Repo) server.ServerTool {
 
 			// Check for any errors encountered during iteration
 			if err := rows.Err(); err != nil {
-				return mcp.NewToolResultError(fmt.Sprintf("Error iterating rows: %s", err.Error())), nil
+				return mcp.NewToolResultErrorf("Error iterating rows: %s", err.Error()), nil
 			}
 
 			respJSON, err := json.Marshal(results)
 			if err != nil {
-				return mcp.NewToolResultError(fmt.Sprintf("Error marshaling results: %s", err.Error())), nil
+				return mcp.NewToolResultErrorf("Error marshaling results: %s", err.Error()), nil
 			}
 
 			return mcp.NewToolResultText(string(respJSON)), nil
