fix: render error types as strings in human log sink

`formatValue` checked `reflect.Struct` before checking the `error`
interface, so struct-typed errors like `context.DeadlineExceeded`
were serialized via `json.Marshal` as `{}` instead of their error
string.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: dannykopping

go.mod

@@ -28,6 +28,7 @@ require (
 	github.com/mattn/go-runewidth v0.0.15 // indirect
 	github.com/muesli/reflow v0.3.0 // indirect
 	github.com/rivo/uniseg v0.4.4 // indirect
+	github.com/stretchr/testify v1.11.1 // indirect
 	go.opentelemetry.io/otel v1.16.0 // indirect
 	go.opentelemetry.io/otel/metric v1.16.0 // indirect
 	golang.org/x/net v0.12.0 // indirect

go.sum

@@ -38,7 +38,8 @@ github.com/rivo/uniseg v0.1.0/go.mod h1:J6wj4VEh+S6ZtnVlnTBMWIodfgj8LQOQFoIToxlJ
 github.com/rivo/uniseg v0.2.0/go.mod h1:J6wj4VEh+S6ZtnVlnTBMWIodfgj8LQOQFoIToxlJtxc=
 github.com/rivo/uniseg v0.4.4 h1:8TfxU8dW6PdqD27gjM8MVNuicgxIjxpm4K7x4jp8sis=
 github.com/rivo/uniseg v0.4.4/go.mod h1:FN3SvrM+Zdj16jyLfmOkMNblXMcoc8DfTHruCPUcx88=
-github.com/stretchr/testify v1.8.3 h1:RP3t2pwF7cMEbC1dqtB6poj3niw/9gnV4Cjg5oW5gtY=
+github.com/stretchr/testify v1.11.1 h1:7s2iGBzp5EwR7/aIZr8ao5+dra3wiQyKjjFuvgVKu7U=
+github.com/stretchr/testify v1.11.1/go.mod h1:wZwfW3scLgRK+23gO65QZefKpKQRnfz6sD981Nm4B6U=
 go.opentelemetry.io/otel v1.16.0 h1:Z7GVAX/UkAXPKsy94IU+i6thsQS4nb7LviLpnaNeW8s=
 go.opentelemetry.io/otel v1.16.0/go.mod h1:vl0h9NUa1D5s1nv3A5vZOYWn8av4K8Ml6JDeHrT/bx4=
 go.opentelemetry.io/otel/metric v1.16.0 h1:RbrpwVG1Hfv85LgnZ7+txXioPDoh6EdbZHo26Q3hqOo=

internal/entryhuman/entry.go

@@ -67,6 +67,10 @@ func formatValue(v interface{}) (string, error) {
 	if v == nil {
 		return "<nil>", nil
 	}
+	switch v.(type) {
+	case error, fmt.Stringer:
+		return quote(fmt.Sprintf("%+v", v)), nil
+	}
 	typ := reflect.TypeOf(v)
 	switch typ.Kind() {
 	case reflect.Struct, reflect.Map:

internal/entryhuman/entry_test.go

@@ -2,12 +2,14 @@ package entryhuman_test
 
 import (
 	"bytes"
+	"context"
 	"database/sql"
 	"flag"
 	"fmt"
 	"io"
 	"math"
 	"os"
+	"strings"
 	"testing"
 	"time"
 
@@ -284,6 +286,51 @@ func TestEntry(t *testing.T) {
 	})
 }
 
+func TestContextErrorFormatting(t *testing.T) {
+	t.Parallel()
+
+	tests := []struct {
+		name     string
+		err      error
+		expected string
+	}{
+		{
+			name:     "DeadlineExceeded",
+			err:      context.DeadlineExceeded,
+			expected: "context deadline exceeded",
+		},
+		{
+			name:     "Canceled",
+			err:      context.Canceled,
+			expected: "context canceled",
+		},
+		{
+			name:     "WrappedDeadlineExceeded",
+			err:      fmt.Errorf("request failed: %w", context.DeadlineExceeded),
+			expected: "request failed: context deadline exceeded",
+		},
+	}
+
+	for _, tc := range tests {
+		t.Run(tc.name, func(t *testing.T) {
+			t.Parallel()
+
+			var buf bytes.Buffer
+			entryhuman.Fmt(&buf, io.Discard, slog.SinkEntry{
+				Level:   slog.LevelError,
+				Message: "something failed",
+				Fields: slog.M(
+					slog.Error(tc.err),
+				),
+			})
+
+			got := buf.String()
+			assert.False(t, "error not empty object", strings.Contains(got, "error={}"))
+			assert.True(t, "error contains expected string", strings.Contains(got, tc.expected))
+		})
+	}
+}
+
 func BenchmarkFmt(b *testing.B) {
 	bench := func(b *testing.B, color bool) {
 		nfs := []int{1, 4, 16}

map_test.go

@@ -2,6 +2,7 @@ package slog_test
 
 import (
 	"bytes"
+	"context"
 	"encoding/json"
 	"io"
 	"runtime"
@@ -62,12 +63,12 @@ func TestMap(t *testing.T) {
 				{
 					"msg": "wrap1",
 					"fun": "cdr.dev/slog/v3_test.TestMap.func2",
-					"loc": "`+mapTestFile+`:41"
+					"loc": "`+mapTestFile+`:42"
 				},
 				{
 					"msg": "wrap2",
 					"fun": "cdr.dev/slog/v3_test.TestMap.func2",
-					"loc": "`+mapTestFile+`:42"
+					"loc": "`+mapTestFile+`:43"
 				},
 				"EOF"
 			],
@@ -220,6 +221,26 @@ func TestMap(t *testing.T) {
 			"val": "{meow:hi bar:23 far:600}"
 		}`)
 	})
+
+	t.Run("contextDeadlineExceeded", func(t *testing.T) {
+		t.Parallel()
+
+		test(t, slog.M(
+			slog.Error(context.DeadlineExceeded),
+		), `{
+			"error": "context deadline exceeded"
+		}`)
+	})
+
+	t.Run("contextCanceled", func(t *testing.T) {
+		t.Parallel()
+
+		test(t, slog.M(
+			slog.Error(context.Canceled),
+		), `{
+			"error": "context canceled"
+		}`)
+	})
 }
 
 func indentJSON(t *testing.T, j string) string {

sloggers/slogjson/slogjson_test.go

@@ -6,6 +6,7 @@ import (
 	"database/sql"
 	"fmt"
 	"runtime"
+	"strings"
 	"testing"
 
 	sdktrace "go.opentelemetry.io/otel/sdk/trace"
@@ -34,7 +35,7 @@ func TestMake(t *testing.T) {
 	l.Error(ctx, "line1\n\nline2", slog.F("wowow", "me\nyou"))
 
 	j := entryjson.Filter(b.String(), "ts")
-	exp := fmt.Sprintf(`{"level":"ERROR","msg":"line1\n\nline2","caller":"%v:34","func":"cdr.dev/slog/v3/sloggers/slogjson_test.TestMake","logger_names":["named"],"trace":"%v","span":"%v","fields":{"wowow":"me\nyou"}}
+	exp := fmt.Sprintf(`{"level":"ERROR","msg":"line1\n\nline2","caller":"%v:35","func":"cdr.dev/slog/v3/sloggers/slogjson_test.TestMake","logger_names":["named"],"trace":"%v","span":"%v","fields":{"wowow":"me\nyou"}}
 `, slogjsonTestFile, span.SpanContext().TraceID().String(), span.SpanContext().SpanID().String())
 	assert.Equal(t, "entry", exp, j)
 }
@@ -60,7 +61,44 @@ func TestNoDriverValue(t *testing.T) {
 	l.Error(bg, "error!", slog.F("inval", invalidField), slog.F("val", validField), slog.F("int", validInt))
 
 	j := entryjson.Filter(b.String(), "ts")
-	exp := fmt.Sprintf(`{"level":"ERROR","msg":"error!","caller":"%v:60","func":"cdr.dev/slog/v3/sloggers/slogjson_test.TestNoDriverValue","logger_names":["named"],"fields":{"inval":null,"val":"cat","int":42}}
+	exp := fmt.Sprintf(`{"level":"ERROR","msg":"error!","caller":"%v:61","func":"cdr.dev/slog/v3/sloggers/slogjson_test.TestNoDriverValue","logger_names":["named"],"fields":{"inval":null,"val":"cat","int":42}}
 `, slogjsonTestFile)
 	assert.Equal(t, "entry", exp, j)
 }
+
+func TestContextErrors(t *testing.T) {
+	t.Parallel()
+
+	t.Run("DeadlineExceeded", func(t *testing.T) {
+		t.Parallel()
+
+		b := &bytes.Buffer{}
+		l := slog.Make(slogjson.Sink(b))
+		l.Error(bg, "request failed", slog.Error(context.DeadlineExceeded))
+
+		j := entryjson.Filter(b.String(), "ts")
+		assert.True(t, "error contains deadline exceeded", strings.Contains(j, `"error":"context deadline exceeded"`))
+	})
+
+	t.Run("Canceled", func(t *testing.T) {
+		t.Parallel()
+
+		b := &bytes.Buffer{}
+		l := slog.Make(slogjson.Sink(b))
+		l.Error(bg, "request failed", slog.Error(context.Canceled))
+
+		j := entryjson.Filter(b.String(), "ts")
+		assert.True(t, "error contains context canceled", strings.Contains(j, `"error":"context canceled"`))
+	})
+
+	t.Run("WrappedDeadlineExceeded", func(t *testing.T) {
+		t.Parallel()
+
+		b := &bytes.Buffer{}
+		l := slog.Make(slogjson.Sink(b))
+		l.Error(bg, "request failed", slog.Error(fmt.Errorf("dial: %w", context.DeadlineExceeded)))
+
+		j := entryjson.Filter(b.String(), "ts")
+		assert.True(t, "error contains dial: context deadline exceeded", strings.Contains(j, `"error":"dial: context deadline exceeded"`))
+	})
+}