refactor: use errors.New instead of fmt.Errorf (#3112)

Author: alexandear

.golangci.yml

@@ -8,6 +8,12 @@ linters-settings:
       - (io.Writer).Write
       - io.Copy
       - io.WriteString
+  perfsprint:
+    int-conversion: false
+    err-error: false
+    errorf: true
+    sprintf1: false
+    strconcat: false
   revive:
     enable-all-rules: false
     rules:
@@ -45,6 +51,7 @@ linters:
     - ineffassign
     - misspell
     - nakedret
+    - perfsprint
     - prealloc
     - revive
     - staticcheck

_examples/dataloader/resolvers.go

@@ -6,6 +6,7 @@ import (
 	"context"
 	"fmt"
 	"math/rand"
+	"strconv"
 	"time"
 )
 
@@ -70,7 +71,7 @@ func (r *queryResolver) Customers(ctx context.Context) ([]*Customer, error) {
 func (r *queryResolver) Torture1d(ctx context.Context, customerIds []int) ([]*Customer, error) {
 	result := make([]*Customer, len(customerIds))
 	for i, id := range customerIds {
-		result[i] = &Customer{ID: id, Name: fmt.Sprintf("%d", i), AddressID: rand.Int() % 10}
+		result[i] = &Customer{ID: id, Name: strconv.Itoa(i), AddressID: rand.Int() % 10}
 	}
 	return result, nil
 }

_examples/scalars/model/model.go

@@ -56,13 +56,13 @@ type Point struct {
 func (p *Point) UnmarshalGQL(v any) error {
 	pointStr, ok := v.(string)
 	if !ok {
-		return fmt.Errorf("points must be strings")
+		return errors.New("points must be strings")
 	}
 
 	parts := strings.Split(pointStr, ",")
 
 	if len(parts) != 2 {
-		return fmt.Errorf("points must have 2 parts")
+		return errors.New("points must have 2 parts")
 	}
 
 	var err error
@@ -108,7 +108,7 @@ func MarshalID(id external.ObjectID) graphql.Marshaler {
 func UnmarshalID(v any) (external.ObjectID, error) {
 	str, ok := v.(string)
 	if !ok {
-		return 0, fmt.Errorf("ids must be strings")
+		return 0, errors.New("ids must be strings")
 	}
 	i, err := strconv.Atoi(str[1 : len(str)-1])
 	return external.ObjectID(i), err
@@ -166,7 +166,7 @@ func (e Tier) String() string {
 func (e *Tier) UnmarshalGQL(v any) error {
 	str, ok := v.(string)
 	if !ok {
-		return fmt.Errorf("enums must be strings")
+		return errors.New("enums must be strings")
 	}
 
 	var err error

_examples/todo/todo.go

@@ -5,7 +5,6 @@ package todo
 import (
 	"context"
 	"errors"
-	"fmt"
 	"time"
 
 	"github.com/mitchellh/mapstructure"
@@ -47,11 +46,11 @@ func New() Config {
 		case RoleOwner:
 			ownable, isOwnable := obj.(Ownable)
 			if !isOwnable {
-				return nil, fmt.Errorf("obj cant be owned")
+				return nil, errors.New("obj cant be owned")
 			}
 
 			if ownable.Owner().ID != getUserId(ctx) {
-				return nil, fmt.Errorf("you don't own that")
+				return nil, errors.New("you don't own that")
 			}
 		}
 

client/websocket.go

@@ -2,6 +2,7 @@ package client
 
 import (
 	"encoding/json"
+	"errors"
 	"fmt"
 	"io"
 	"net/http/httptest"
@@ -127,7 +128,7 @@ func (p *Client) WebsocketWithPayload(query string, initPayload map[string]any,
 				case connectionKaMsg:
 					continue
 				case errorMsg:
-					return fmt.Errorf(string(op.Payload))
+					return errors.New(string(op.Payload))
 				default:
 					return fmt.Errorf("expected data message, got %#v", op)
 				}

codegen/config/binder.go

@@ -99,7 +99,7 @@ var (
 func (b *Binder) DefaultUserObject(name string) (types.Type, error) {
 	models := b.cfg.Models[name].Model
 	if len(models) == 0 {
-		return nil, fmt.Errorf(name + " not found in typemap")
+		return nil, fmt.Errorf("%s not found in typemap", name)
 	}
 
 	if models[0] == "map[string]interface{}" {
@@ -125,7 +125,7 @@ func (b *Binder) DefaultUserObject(name string) (types.Type, error) {
 
 func (b *Binder) FindObject(pkgName string, typeName string) (types.Object, error) {
 	if pkgName == "" {
-		return nil, fmt.Errorf("package cannot be nil")
+		return nil, errors.New("package cannot be nil")
 	}
 
 	pkg := b.pkgs.LoadWithTypes(pkgName)

codegen/config/config.go

@@ -2,6 +2,7 @@ package config
 
 import (
 	"bytes"
+	"errors"
 	"fmt"
 	"go/types"
 	"io"
@@ -564,11 +565,11 @@ func (c *Config) check() error {
 			Declaree: "federation",
 		})
 		if c.Federation.ImportPath() != c.Exec.ImportPath() {
-			return fmt.Errorf("federation and exec must be in the same package")
+			return errors.New("federation and exec must be in the same package")
 		}
 	}
 	if c.Federated {
-		return fmt.Errorf("federated has been removed, instead use\nfederation:\n    filename: path/to/federated.go")
+		return errors.New("federated has been removed, instead use\nfederation:\n    filename: path/to/federated.go")
 	}
 
 	for importPath, pkg := range fileList {

codegen/config/exec.go

@@ -1,6 +1,7 @@
 package config
 
 import (
+	"errors"
 	"fmt"
 	"go/types"
 	"path/filepath"
@@ -38,23 +39,23 @@ func (r *ExecConfig) Check() error {
 	switch r.Layout {
 	case ExecLayoutSingleFile:
 		if r.Filename == "" {
-			return fmt.Errorf("filename must be specified when using single-file layout")
+			return errors.New("filename must be specified when using single-file layout")
 		}
 		if !strings.HasSuffix(r.Filename, ".go") {
-			return fmt.Errorf("filename should be path to a go source file when using single-file layout")
+			return errors.New("filename should be path to a go source file when using single-file layout")
 		}
 		r.Filename = abs(r.Filename)
 	case ExecLayoutFollowSchema:
 		if r.DirName == "" {
-			return fmt.Errorf("dir must be specified when using follow-schema layout")
+			return errors.New("dir must be specified when using follow-schema layout")
 		}
 		r.DirName = abs(r.DirName)
 	default:
 		return fmt.Errorf("invalid layout %s", r.Layout)
 	}
 
 	if strings.ContainsAny(r.Package, "./\\") {
-		return fmt.Errorf("package should be the output package name only, do not include the output filename")
+		return errors.New("package should be the output package name only, do not include the output filename")
 	}
 
 	if r.Package == "" && r.Dir() != "" {

codegen/config/package.go

@@ -1,7 +1,7 @@
 package config
 
 import (
-	"fmt"
+	"errors"
 	"go/types"
 	"path/filepath"
 	"strings"
@@ -44,13 +44,13 @@ func (c *PackageConfig) IsDefined() bool {
 
 func (c *PackageConfig) Check() error {
 	if strings.ContainsAny(c.Package, "./\\") {
-		return fmt.Errorf("package should be the output package name only, do not include the output filename")
+		return errors.New("package should be the output package name only, do not include the output filename")
 	}
 	if c.Filename == "" {
-		return fmt.Errorf("filename must be specified")
+		return errors.New("filename must be specified")
 	}
 	if !strings.HasSuffix(c.Filename, ".go") {
-		return fmt.Errorf("filename should be path to a go source file")
+		return errors.New("filename should be path to a go source file")
 	}
 
 	c.Filename = abs(c.Filename)

codegen/config/resolver.go

@@ -1,6 +1,7 @@
 package config
 
 import (
+	"errors"
 	"fmt"
 	"go/types"
 	"path/filepath"
@@ -59,7 +60,7 @@ func (r *ResolverConfig) Check() error {
 	}
 
 	if strings.ContainsAny(r.Package, "./\\") {
-		return fmt.Errorf("package should be the output package name only, do not include the output filename")
+		return errors.New("package should be the output package name only, do not include the output filename")
 	}
 
 	if r.Package == "" && r.Dir() != "" {