fix: add (*statz.statusError).Format() for stacktrace

Author: ginokent

errors/fmt.go

@@ -176,7 +176,7 @@ loop:
 			e.Format(s, verb)
 			break loop
 		default:
-			fmt.Fprint(s, e)
+			fmt.Fprintf(s, fmt.FormatString(s, verb), e)
 			break loop
 		}
 		if err == nil {

grpc/status/status.go

@@ -64,7 +64,8 @@ var (
 	DefaultLogger Logger = func(_ context.Context, level Level, stat *status.Status, err error) {
 		log.Printf("level=%s code=%s message=%q details=%s error=%q stacktrace=%q", level, stat.Code(), stat.Message(), stat.Details(), fmt.Sprintf("%v", err), fmt.Sprintf("%+v", err))
 	}
-	errorf = errorz.NewErrorf(errorz.WithCallerSkip(1))
+	_      interface{ GRPCStatus() *status.Status } = (*statusError)(nil)
+	errorf                                          = errorz.NewErrorf(errorz.WithCallerSkip(1))
 )
 
 func New(ctx context.Context, level Level, code codes.Code, msg string, err error, opts ...ErrorOption) error {
@@ -78,7 +79,16 @@ func New(ctx context.Context, level Level, code codes.Code, msg string, err erro
 		opt.apply(e)
 	}
 	e.logger(ctx, e.level, e.s, err)
-	return errorf("%s: %s", msg, err)
+	return e
+}
+
+func (e *statusError) Format(s fmt.State, verb rune) {
+	if formatter, ok := e.error.(fmt.Formatter); ok { //nolint:errorlint
+		formatter.Format(s, verb)
+		return
+	}
+
+	_, _ = fmt.Fprintf(s, fmt.FormatString(s, verb), e.error)
 }
 
 func (e *statusError) GRPCStatus() *status.Status {

grpc/status/status_test.go

@@ -1,15 +1,16 @@
-package statz_test
+package statz
 
 import (
 	"context"
+	"errors"
 	"io"
 	"testing"
 
 	"google.golang.org/genproto/googleapis/rpc/errdetails"
 	"google.golang.org/grpc/codes"
+	"google.golang.org/grpc/status"
 
 	errorz "github.com/kunitsucom/util.go/errors"
-	statz "github.com/kunitsucom/util.go/grpc/status"
 )
 
 func TestNew(t *testing.T) {
@@ -20,9 +21,21 @@ func TestNew(t *testing.T) {
 
 		ctx := context.Background()
 		before := errorz.Errorf("err: %w", io.ErrUnexpectedEOF)
-		err := statz.New(ctx, statz.ErrorLevel, codes.Internal, "my error message", before, statz.WithDetails(&errdetails.DebugInfo{StackEntries: []string{"stack1", "stack2"}}), statz.WithDetails(nil), statz.WithLogger(statz.DefaultLogger))
+		err := New(ctx, ErrorLevel, codes.Internal, "my error message", before, WithDetails(&errdetails.DebugInfo{StackEntries: []string{"stack1", "stack2"}}), WithDetails(nil), WithLogger(DefaultLogger))
 		t.Logf("🪲: [%%v]:\n%v", err)
 		t.Logf("🪲: [%%+v]:\n%+v", err)
-		_ = statz.New(ctx, statz.ErrorLevel, codes.Internal, "my error message", before, statz.WithLogger(statz.DiscardLogger))
+
+		var e *statusError
+		if !errors.As(err, &e) {
+			t.Errorf("❌: something wrong")
+		}
+		e.error = io.ErrUnexpectedEOF
+		t.Logf("🪲: [%%v]:\n%v", e)
+
+		s, ok := New(ctx, ErrorLevel, codes.Internal, "my error message", io.ErrUnexpectedEOF, WithLogger(DiscardLogger)).(interface{ GRPCStatus() *status.Status }) //nolint:errorlint
+		if !ok {
+			t.Errorf("❌: New: not implement interface{ GRPCStatus() *status.Status }")
+		}
+		t.Logf("🪲: [%%v]:\n%v", s.GRPCStatus())
 	})
 }