should use fmt.Errorf(...) instead of errors.New(fmt.Sprintf(...))

[sanbei101]

rule S1028

S1028 - Simplify error construction with fmt.Errorf
Before:

errors.New(fmt.Sprintf(...))

After:

fmt.Errorf(...)

[Copilot]

Pull Request Overview

This PR simplifies error construction by replacing errors.New(fmt.Sprintf(...)) with the more idiomatic fmt.Errorf(...) pattern, following staticcheck rule S1028.

• Replaces verbose error construction with the standard Go idiom for formatted errors

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[jinzhu]

@copilot fix the test