shorter error messages

andrewmbenton · andrewmbenton · commit 37eb9aeb6610 · 2025-03-09T11:41:19.000-07:00
diff --git a/internal/codegen/golang/templates/pgx/queryCode.tmpl b/internal/codegen/golang/templates/pgx/queryCode.tmpl
@@ -39,7 +39,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{.Arg.Pair}}) ({{.Ret.De
 	err := row.Scan({{.Ret.Scan}})
 	{{- if $.WrapErrors}}
 	if err != nil {
-		err = fmt.Errorf("error executing query {{.MethodName}}: %w", err)
+		err = fmt.Errorf("query {{.MethodName}}: %w", err)
 	}
 	{{- end}}
 	return {{.Ret.ReturnName}}, err
@@ -57,7 +57,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{.Arg.Pair}}) ([]{{.Ret.
 	rows, err := q.db.Query(ctx, {{.ConstantName}}, {{.Arg.Params}})
 {{- end}}
 	if err != nil {
-		return nil, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}
+		return nil, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}
 	}
 	defer rows.Close()
 	{{- if $.EmitEmptySlices}}
@@ -68,12 +68,12 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{.Arg.Pair}}) ([]{{.Ret.
 	for rows.Next() {
 		var {{.Ret.Name}} {{.Ret.Type}}
 		if err := rows.Scan({{.Ret.Scan}}); err != nil {
-			return nil, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}
+			return nil, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}
 		}
 		items = append(items, {{.Ret.ReturnName}})
 	}
 	if err := rows.Err(); err != nil {
-		return nil, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}
+		return nil, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}
 	}
 	return items, nil
 }
@@ -89,7 +89,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, db DBTX, {{.Arg.Pair}}) e
 func (q *Queries) {{.MethodName}}(ctx context.Context, {{.Arg.Pair}}) error {
 	_, err := q.db.Exec(ctx, {{.ConstantName}}, {{.Arg.Params}})
 {{- end}}
-	return {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}
+	return {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}
 }
 {{end}}
 
@@ -104,7 +104,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{.Arg.Pair}}) (int64, er
 	result, err := q.db.Exec(ctx, {{.ConstantName}}, {{.Arg.Params}})
 {{- end}}
 	if err != nil {
-		return 0, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}
+		return 0, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}
 	}
 	return result.RowsAffected(), nil
 }
@@ -122,7 +122,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{.Arg.Pair}}) (pgconn.Co
 {{- end}}
 	{{- if $.WrapErrors}}
 	if err != nil {
-		err = fmt.Errorf("error executing query {{.MethodName}}: %w", err)
+		err = fmt.Errorf("query {{.MethodName}}: %w", err)
 	}
 	return result, err
 	{{- end}}
diff --git a/internal/codegen/golang/templates/stdlib/queryCode.tmpl b/internal/codegen/golang/templates/stdlib/queryCode.tmpl
@@ -30,7 +30,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}
 	err := row.Scan({{.Ret.Scan}})
 	{{- if $.WrapErrors}}
 	if err != nil {
-		err = fmt.Errorf("error executing query {{.MethodName}}: %w", err)
+		err = fmt.Errorf("query {{.MethodName}}: %w", err)
 	}
 	{{- end}}
 	return {{.Ret.ReturnName}}, err
@@ -43,7 +43,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}
 func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}) ([]{{.Ret.DefineType}}, error) {
     {{- template "queryCodeStdExec" . }}
     if err != nil {
-        return nil, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}
+        return nil, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}
     }
     defer rows.Close()
     {{- if $.EmitEmptySlices}}
@@ -54,15 +54,15 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}
     for rows.Next() {
         var {{.Ret.Name}} {{.Ret.Type}}
         if err := rows.Scan({{.Ret.Scan}}); err != nil {
-            return nil, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}
+            return nil, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}
         }
         items = append(items, {{.Ret.ReturnName}})
     }
     if err := rows.Close(); err != nil {
-        return nil, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}
+        return nil, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}
     }
     if err := rows.Err(); err != nil {
-        return nil, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}
+        return nil, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}
     }
     return items, nil
 }
@@ -75,7 +75,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}
     {{- template "queryCodeStdExec" . }}
     {{- if $.WrapErrors}}
     if err != nil {
-        err = fmt.Errorf("error executing query {{.MethodName}}: %w", err)
+        err = fmt.Errorf("query {{.MethodName}}: %w", err)
     }
     {{- end}}
     return err
@@ -88,7 +88,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}
 func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}) (int64, error) {
     {{- template "queryCodeStdExec" . }}
     if err != nil {
-        return 0, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}
+        return 0, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}
     }
     return result.RowsAffected()
 }
@@ -100,7 +100,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}
 func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}) (int64, error) {
     {{- template "queryCodeStdExec" . }}
     if err != nil {
-        return 0, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}
+        return 0, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}
     }
     return result.LastInsertId()
 }
@@ -113,7 +113,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}
     {{- template "queryCodeStdExec" . }}
     {{- if $.WrapErrors}}
     if err != nil {
-        err = fmt.Errorf("error executing query {{.MethodName}}: %w", err)
+        err = fmt.Errorf("query {{.MethodName}}: %w", err)
     }
     return result, err
     {{- end}}
diff --git a/internal/endtoend/testdata/wrap_errors/postgresql/pgx/db/query.sql.go b/internal/endtoend/testdata/wrap_errors/postgresql/pgx/db/query.sql.go

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{.Arg.Pair}}) ({{.Ret.De`
`39`	`39`	`err := row.Scan({{.Ret.Scan}})`
`40`	`40`	`{{- if $.WrapErrors}}`
`41`	`41`	`if err != nil {`
`42`		`- err = fmt.Errorf("error executing query {{.MethodName}}: %w", err)`
	`42`	`+ err = fmt.Errorf("query {{.MethodName}}: %w", err)`
`43`	`43`	`}`
`44`	`44`	`{{- end}}`
`45`	`45`	`return {{.Ret.ReturnName}}, err`
`@@ -57,7 +57,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{.Arg.Pair}}) ([]{{.Ret.`
`57`	`57`	`rows, err := q.db.Query(ctx, {{.ConstantName}}, {{.Arg.Params}})`
`58`	`58`	`{{- end}}`
`59`	`59`	`if err != nil {`
`60`		`- return nil, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}`
	`60`	`+ return nil, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}`
`61`	`61`	`}`
`62`	`62`	`defer rows.Close()`
`63`	`63`	`{{- if $.EmitEmptySlices}}`
`@@ -68,12 +68,12 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{.Arg.Pair}}) ([]{{.Ret.`
`68`	`68`	`for rows.Next() {`
`69`	`69`	`var {{.Ret.Name}} {{.Ret.Type}}`
`70`	`70`	`if err := rows.Scan({{.Ret.Scan}}); err != nil {`
`71`		`- return nil, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}`
	`71`	`+ return nil, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}`
`72`	`72`	`}`
`73`	`73`	`items = append(items, {{.Ret.ReturnName}})`
`74`	`74`	`}`
`75`	`75`	`if err := rows.Err(); err != nil {`
`76`		`- return nil, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}`
	`76`	`+ return nil, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}`
`77`	`77`	`}`
`78`	`78`	`return items, nil`
`79`	`79`	`}`
`@@ -89,7 +89,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, db DBTX, {{.Arg.Pair}}) e`
`89`	`89`	`func (q *Queries) {{.MethodName}}(ctx context.Context, {{.Arg.Pair}}) error {`
`90`	`90`	`_, err := q.db.Exec(ctx, {{.ConstantName}}, {{.Arg.Params}})`
`91`	`91`	`{{- end}}`
`92`		`- return {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}`
	`92`	`+ return {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}`
`93`	`93`	`}`
`94`	`94`	`{{end}}`
`95`	`95`
`@@ -104,7 +104,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{.Arg.Pair}}) (int64, er`
`104`	`104`	`result, err := q.db.Exec(ctx, {{.ConstantName}}, {{.Arg.Params}})`
`105`	`105`	`{{- end}}`
`106`	`106`	`if err != nil {`
`107`		`- return 0, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}`
	`107`	`+ return 0, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}`
`108`	`108`	`}`
`109`	`109`	`return result.RowsAffected(), nil`
`110`	`110`	`}`
`@@ -122,7 +122,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{.Arg.Pair}}) (pgconn.Co`
`122`	`122`	`{{- end}}`
`123`	`123`	`{{- if $.WrapErrors}}`
`124`	`124`	`if err != nil {`
`125`		`- err = fmt.Errorf("error executing query {{.MethodName}}: %w", err)`
	`125`	`+ err = fmt.Errorf("query {{.MethodName}}: %w", err)`
`126`	`126`	`}`
`127`	`127`	`return result, err`
`128`	`128`	`{{- end}}`
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}`
`30`	`30`	`err := row.Scan({{.Ret.Scan}})`
`31`	`31`	`{{- if $.WrapErrors}}`
`32`	`32`	`if err != nil {`
`33`		`- err = fmt.Errorf("error executing query {{.MethodName}}: %w", err)`
	`33`	`+ err = fmt.Errorf("query {{.MethodName}}: %w", err)`
`34`	`34`	`}`
`35`	`35`	`{{- end}}`
`36`	`36`	`return {{.Ret.ReturnName}}, err`
`@@ -43,7 +43,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}`
`43`	`43`	`func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}) ([]{{.Ret.DefineType}}, error) {`
`44`	`44`	`{{- template "queryCodeStdExec" . }}`
`45`	`45`	`if err != nil {`
`46`		`- return nil, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}`
	`46`	`+ return nil, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}`
`47`	`47`	`}`
`48`	`48`	`defer rows.Close()`
`49`	`49`	`{{- if $.EmitEmptySlices}}`
`@@ -54,15 +54,15 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}`
`54`	`54`	`for rows.Next() {`
`55`	`55`	`var {{.Ret.Name}} {{.Ret.Type}}`
`56`	`56`	`if err := rows.Scan({{.Ret.Scan}}); err != nil {`
`57`		`- return nil, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}`
	`57`	`+ return nil, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}`
`58`	`58`	`}`
`59`	`59`	`items = append(items, {{.Ret.ReturnName}})`
`60`	`60`	`}`
`61`	`61`	`if err := rows.Close(); err != nil {`
`62`		`- return nil, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}`
	`62`	`+ return nil, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}`
`63`	`63`	`}`
`64`	`64`	`if err := rows.Err(); err != nil {`
`65`		`- return nil, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}`
	`65`	`+ return nil, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}`
`66`	`66`	`}`
`67`	`67`	`return items, nil`
`68`	`68`	`}`
`@@ -75,7 +75,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}`
`75`	`75`	`{{- template "queryCodeStdExec" . }}`
`76`	`76`	`{{- if $.WrapErrors}}`
`77`	`77`	`if err != nil {`
`78`		`- err = fmt.Errorf("error executing query {{.MethodName}}: %w", err)`
	`78`	`+ err = fmt.Errorf("query {{.MethodName}}: %w", err)`
`79`	`79`	`}`
`80`	`80`	`{{- end}}`
`81`	`81`	`return err`
`@@ -88,7 +88,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}`
`88`	`88`	`func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}) (int64, error) {`
`89`	`89`	`{{- template "queryCodeStdExec" . }}`
`90`	`90`	`if err != nil {`
`91`		`- return 0, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}`
	`91`	`+ return 0, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}`
`92`	`92`	`}`
`93`	`93`	`return result.RowsAffected()`
`94`	`94`	`}`
`@@ -100,7 +100,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}`
`100`	`100`	`func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}) (int64, error) {`
`101`	`101`	`{{- template "queryCodeStdExec" . }}`
`102`	`102`	`if err != nil {`
`103`		`- return 0, {{if $.WrapErrors}}fmt.Errorf("error executing query {{.MethodName}}: %w", err){{else}}err{{end}}`
	`103`	`+ return 0, {{if $.WrapErrors}}fmt.Errorf("query {{.MethodName}}: %w", err){{else}}err{{end}}`
`104`	`104`	`}`
`105`	`105`	`return result.LastInsertId()`
`106`	`106`	`}`
`@@ -113,7 +113,7 @@ func (q *Queries) {{.MethodName}}(ctx context.Context, {{ dbarg }} {{.Arg.Pair}}`
`113`	`113`	`{{- template "queryCodeStdExec" . }}`
`114`	`114`	`{{- if $.WrapErrors}}`
`115`	`115`	`if err != nil {`
`116`		`- err = fmt.Errorf("error executing query {{.MethodName}}: %w", err)`
	`116`	`+ err = fmt.Errorf("query {{.MethodName}}: %w", err)`
`117`	`117`	`}`
`118`	`118`	`return result, err`
`119`	`119`	`{{- end}}`