fix(gofmt): fix the issue where the gofmt stderr is not handled

internal/linters/go/gofmt/gofmt.go

@@ -1,6 +1,8 @@
 package gofmt
 
 import (
+	"bytes"
+	"errors"
 	"os/exec"
 	"regexp"
 	"strconv"
@@ -52,7 +54,7 @@ func gofmtHandler(log *xlog.Logger, a linters.Agent) error {
 type Gofmt struct {
 	dir     string
 	gofmt   string
-	execute func(dir, command string, args ...string) ([]byte, error)
+	execute func(dir, command string, args ...string) ([]byte, []byte, error)
 }
 
 func NewgofmtExecutor(dir string) (linters.Linter, error) {
@@ -64,24 +66,38 @@ func NewgofmtExecutor(dir string) (linters.Linter, error) {
 	return &Gofmt{
 		dir:   dir,
 		gofmt: g,
-		execute: func(dir, command string, args ...string) ([]byte, error) {
+		execute: func(dir, command string, args ...string) ([]byte, []byte, error) {
 			c := exec.Command(command, args...)
 			c.Dir = dir
 			log.Printf("final command:  %v \n", c)
-			return c.Output()
+			if c.Stdout != nil {
+				return nil, nil, errors.New("exec: Stdout already set")
+			}
+			if c.Stderr != nil {
+				return nil, nil, errors.New("exec: Stderr already set")
+			}
+			var stdoutBuffer bytes.Buffer
+			var stderrBuffer bytes.Buffer
+			c.Stdout = &stdoutBuffer
+			c.Stderr = &stderrBuffer
+			err := c.Run()
+			return stdoutBuffer.Bytes(), stderrBuffer.Bytes(), err
 		},
 	}, nil
 }
 
 func (g *Gofmt) Run(log *xlog.Logger, args ...string) ([]byte, error) {
-	b, err := g.execute(g.dir, g.gofmt, args...)
+	stdoutput, stderr, err := g.execute(g.dir, g.gofmt, args...)
 	if err != nil {
 		log.Errorf("gofmt run with status: %v, mark and continue", err)
-		return b, err
+		if stderr != nil {
+			log.Errorf("gofmt run cause stderr: %s, mark and continue", stderr)
+		}
+		return stdoutput, err
 	} else {
 		log.Infof("gofmt running succeeded")
 	}
-	return b, nil
+	return stdoutput, nil
 }
 
 func (g *Gofmt) Parse(log *xlog.Logger, output []byte) (map[string][]linters.LinterOutput, error) {