fix(deps): update opentelemetry

.gitea/workflows/pre-commit.yaml

@@ -19,7 +19,7 @@ jobs:
           go-version: stable
       - uses: actions/setup-python@v7
         with:
-          python-version: '3.15'
+          python-version: '3.14'
       - name: Install goimports
         run: go install golang.org/x/tools/cmd/goimports@latest
       - uses: pre-commit/action@v3.0.1

.version

@@ -1,3 +1,3 @@
 {
-  "version": "v0.7.0"
+  "version": "v0.6.1"
 }

CHANGELOG.md

@@ -2,27 +2,6 @@
 
 All notable changes to this project will be documented in this file.
 
-## [0.7.0] - 2026-10-16
-
-### 🚀 Features
-
-- Add gqlgen Recover that logs panics as one structured record (#206)
-
-### 🐛 Bug Fixes
-
-- *(deps)* Update module codeberg.org/eventsourced/pg/v2 to v2.2.0 (#186)
-- *(deps)* Update module codeberg.org/eventsourced/pg/v2 to v2.2.1 (#188)
-- *(deps)* Update module codeberg.org/eventsourced/pg/v2 to v2.3.0 (#200)
-- *(deps)* Update opentelemetry (#198)
-- *(deps)* Update golang.org/x/net to v0.60.0 and golang.org/x/tools to v0.51.0 (#203)
-
-### ⚙️ Miscellaneous Tasks
-
-- *(deps)* Update pre-commit hook alessandrojcm/commitlint-pre-commit-hook to v9.27.0 (#190)
-- *(deps)* Update pre-commit hook golangci/golangci-lint to v2.14.0 (#192)
-- *(renovate)* Group OpenTelemetry, otelsetup and logging updates (#196)
-- *(deps)* Update dependency python to 3.15 (#204)
-
 ## [0.6.1] - 2026-09-19
 
 ### 🐛 Bug Fixes

CLAUDE.md

@@ -28,12 +28,6 @@ defer shutdown(ctx)
 
 // Wrap HTTP handlers with tracing
 http.Handle("/", otelsetup.Handler(myHandler))
-
-// gqlgen server hooks
-srv.AroundOperations(otelsetup.AroundOperations)
-srv.AroundRootFields(otelsetup.AroundRootFields)
-srv.AroundFields(otelsetup.AroundFields)
-srv.SetRecoverFunc(otelsetup.Recover)
 ```
 
 ### Features
@@ -43,8 +37,6 @@ srv.SetRecoverFunc(otelsetup.Recover)
 - **Logging** - Stdout log exporter
 - **Propagation** - TraceContext and Baggage propagation
 - **HTTP Handler** - Middleware for automatic span creation
-- **gqlgen hooks** - Spans per operation, root field and resolver field
-- **gqlgen recover** - `Recover` logs a resolver panic as one ERROR record via `slog.Default()` (attributes `panic`, `stack`, `path`), records it on the active span with status Error, and returns the generic "internal system error" to the client (gqlgen's default prints the stack to stderr line by line)
 
 ### Environment Variables
 

go.mod

@@ -6,11 +6,11 @@ require (
 	codeberg.org/eventsourced/eventsourced v1.24.0
 	codeberg.org/eventsourced/pg/v2 v2.3.0
 	github.com/99designs/gqlgen v0.17.95
-	github.com/vektah/gqlparser/v2 v2.5.37
 	go.opentelemetry.io/otel v1.47.0
 	go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp v1.47.0
 	go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.47.0
 	go.opentelemetry.io/otel/exporters/stdout/stdoutlog v0.23.0
+	go.opentelemetry.io/otel/log v1.47.0
 	go.opentelemetry.io/otel/metric v1.47.0
 	go.opentelemetry.io/otel/sdk v1.47.0
 	go.opentelemetry.io/otel/sdk/log v1.47.0
@@ -31,12 +31,12 @@ require (
 	github.com/pressly/goose/v3 v3.28.0 // indirect
 	github.com/sethvargo/go-retry v0.4.0 // indirect
 	github.com/sosodev/duration v1.4.0 // indirect
+	github.com/vektah/gqlparser/v2 v2.5.37 // indirect
 	go.opentelemetry.io/auto/sdk v1.2.1 // indirect
 	go.opentelemetry.io/otel/exporters/otlp/otlptrace v1.47.0 // indirect
-	go.opentelemetry.io/otel/log v1.47.0 // indirect
 	go.opentelemetry.io/proto/otlp v1.11.0 // indirect
 	go.uber.org/multierr v1.11.0 // indirect
-	golang.org/x/net v0.60.0 // indirect
+	golang.org/x/net v0.59.0 // indirect
 	golang.org/x/sync v0.23.0 // indirect
 	golang.org/x/sys v0.48.0 // indirect
 	golang.org/x/text v0.42.0 // indirect

go.sum

@@ -96,8 +96,8 @@ go.uber.org/multierr v1.11.0 h1:blXXJkSxSSfBVBlC76pxqeO+LN3aDfLQo+309xJstO0=
 go.uber.org/multierr v1.11.0/go.mod h1:20+QtiLqy0Nd6FdQB9TLXag12DsQkrbs3htMFfDN80Y=
 go.yaml.in/yaml/v3 v3.0.5 h1:N6y/pJk8buWs9NY5ERU2HSMfm+IuD/OtfdAnq6kESPw=
 go.yaml.in/yaml/v3 v3.0.5/go.mod h1:HVTZu1O7/Vkt2N+BFy8Zza+lnLsABggaTM2ZpNIGuKg=
-golang.org/x/net v0.60.0 h1:79p50tfZlm0J9YfoDsSi639qSXNGVwEzOPLCxM2FsYU=
+golang.org/x/net v0.59.0 h1:5zfYln+w5XCxwrnMMJPufRgNoXEaGxl0wo5GqPXyues=
-golang.org/x/net v0.60.0/go.mod h1:2DA/G1UfVbCpQPeWTmMPGY7Cs2PkBkwu743bVX5PIVg=
+golang.org/x/net v0.59.0/go.mod h1:2DA/G1UfVbCpQPeWTmMPGY7Cs2PkBkwu743bVX5PIVg=
 golang.org/x/sync v0.23.0 h1:KameEIfc1IkluZyXWLn39Wd4tURc6GbCiISGiZm2bQk=
 golang.org/x/sync v0.23.0/go.mod h1:sUUOizhqBxiL6pEWpqNLUiaJn1ShEbZ6BBqskPbjZm0=
 golang.org/x/sys v0.48.0 h1:bbX/i/6MgT9BVLM9RT1thmxL04yeTAhbEz4SyadbXoo=

otel.go

@@ -13,6 +13,7 @@ import (
 	"go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp"
 	"go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp"
 	"go.opentelemetry.io/otel/exporters/stdout/stdoutlog"
+	"go.opentelemetry.io/otel/log/global"
 	"go.opentelemetry.io/otel/propagation"
 	"go.opentelemetry.io/otel/sdk/log"
 	"go.opentelemetry.io/otel/sdk/metric"
@@ -85,7 +86,7 @@ func SetupOTelSDK(ctx context.Context, enabled bool, serviceName, buildVersion,
 	processor := log.NewSimpleProcessor(logExporter)
 	logProvider := log.NewLoggerProvider(log.WithProcessor(processor))
 
-	otel.SetLoggerProvider(logProvider)
+	global.SetLoggerProvider(logProvider)
 	shutdownFuncs = append(shutdownFuncs, logProvider.Shutdown)
 
 	exp, err := otlpmetrichttp.New(ctx, otlpmetrichttp.WithHTTPClient(otlpHTTPClient()))

recover.go

@@ -1,41 +0,0 @@
-package otelsetup
-
-import (
-	"context"
-	"fmt"
-	"log/slog"
-	"runtime/debug"
-
-	"github.com/99designs/gqlgen/graphql"
-	"github.com/vektah/gqlparser/v2/gqlerror"
-	"go.opentelemetry.io/otel/codes"
-	"go.opentelemetry.io/otel/trace"
-)
-
-// Recover is a gqlgen recover function, installed with srv.SetRecoverFunc(otelsetup.Recover).
-// Unlike graphql.DefaultRecover, which prints the stack to stderr line by line, it logs the
-// panic as a single structured ERROR record (via slog.Default) with the stack as an attribute,
-// records it on the active span and marks the span as failed. The client gets the same generic
-// "internal system error" as with the default, without the panic text.
-func Recover(ctx context.Context, err any) error {
-	stack := string(debug.Stack())
-	panicErr, ok := err.(error)
-	if !ok {
-		panicErr = fmt.Errorf("%v", err)
-	}
-
-	attrs := []any{
-		slog.String("panic", panicErr.Error()),
-		slog.String("stack", stack),
-	}
-	if fc := graphql.GetFieldContext(ctx); fc != nil {
-		attrs = append(attrs, slog.String("path", fc.Path().String()))
-	}
-	slog.Default().ErrorContext(ctx, "recovered panic in graphql resolver", attrs...)
-
-	span := trace.SpanFromContext(ctx)
-	span.RecordError(panicErr, trace.WithStackTrace(true))
-	span.SetStatus(codes.Error, "panic: "+panicErr.Error())
-
-	return gqlerror.Errorf("internal system error")
-}

recover_test.go

@@ -1,117 +0,0 @@
-package otelsetup
-
-import (
-	"context"
-	"errors"
-	"log/slog"
-	"strings"
-	"sync"
-	"testing"
-
-	"github.com/99designs/gqlgen/graphql"
-	"github.com/vektah/gqlparser/v2/gqlerror"
-	"go.opentelemetry.io/otel/codes"
-	sdktrace "go.opentelemetry.io/otel/sdk/trace"
-	"go.opentelemetry.io/otel/sdk/trace/tracetest"
-)
-
-type recordingHandler struct {
-	mu      sync.Mutex
-	records []slog.Record
-}
-
-func (h *recordingHandler) Enabled(context.Context, slog.Level) bool { return true }
-
-func (h *recordingHandler) Handle(_ context.Context, r slog.Record) error {
-	h.mu.Lock()
-	defer h.mu.Unlock()
-	h.records = append(h.records, r.Clone())
-	return nil
-}
-
-func (h *recordingHandler) WithAttrs([]slog.Attr) slog.Handler { return h }
-func (h *recordingHandler) WithGroup(string) slog.Handler      { return h }
-
-func TestRecover(t *testing.T) {
-	var _ graphql.RecoverFunc = Recover
-
-	tests := []struct {
-		name      string
-		panicVal  any
-		wantPanic string
-	}{
-		{name: "error value", panicVal: errors.New("runtime error: invalid memory address or nil pointer dereference"), wantPanic: "runtime error: invalid memory address or nil pointer dereference"},
-		{name: "string value", panicVal: "boom", wantPanic: "boom"},
-	}
-	for _, tt := range tests {
-		t.Run(tt.name, func(t *testing.T) {
-			h := &recordingHandler{}
-			prev := slog.Default()
-			slog.SetDefault(slog.New(h))
-			t.Cleanup(func() { slog.SetDefault(prev) })
-
-			sr := tracetest.NewSpanRecorder()
-			tp := sdktrace.NewTracerProvider(sdktrace.WithSpanProcessor(sr))
-			ctx, span := tp.Tracer("test").Start(context.Background(), "resolver")
-
-			err := Recover(ctx, tt.panicVal)
-			span.End()
-
-			var gqlErr *gqlerror.Error
-			if !errors.As(err, &gqlErr) || gqlErr.Message != "internal system error" {
-				t.Fatalf("Recover() error = %v, want gqlerror \"internal system error\"", err)
-			}
-			if strings.Contains(err.Error(), tt.wantPanic) {
-				t.Errorf("Recover() error leaks panic text: %v", err)
-			}
-
-			if len(h.records) != 1 {
-				t.Fatalf("got %d log records, want exactly 1", len(h.records))
-			}
-			rec := h.records[0]
-			if rec.Level != slog.LevelError {
-				t.Errorf("log level = %v, want ERROR", rec.Level)
-			}
-			got := map[string]string{}
-			rec.Attrs(func(a slog.Attr) bool {
-				got[a.Key] = a.Value.String()
-				return true
-			})
-			if got["panic"] != tt.wantPanic {
-				t.Errorf("panic attr = %q, want %q", got["panic"], tt.wantPanic)
-			}
-			if !strings.Contains(got["stack"], "goroutine") {
-				t.Errorf("stack attr missing stack trace: %q", got["stack"])
-			}
-
-			spans := sr.Ended()
-			if len(spans) != 1 {
-				t.Fatalf("got %d spans, want 1", len(spans))
-			}
-			s := spans[0]
-			if s.Status().Code != codes.Error {
-				t.Errorf("span status = %v, want Error", s.Status().Code)
-			}
-			var exception bool
-			for _, ev := range s.Events() {
-				if ev.Name != "exception" {
-					continue
-				}
-				exception = true
-				attrs := map[string]string{}
-				for _, kv := range ev.Attributes {
-					attrs[string(kv.Key)] = kv.Value.String()
-				}
-				if attrs["exception.message"] != tt.wantPanic {
-					t.Errorf("exception.message = %q, want %q", attrs["exception.message"], tt.wantPanic)
-				}
-				if attrs["exception.stacktrace"] == "" {
-					t.Error("exception event has no stacktrace")
-				}
-			}
-			if !exception {
-				t.Error("span has no exception event")
-			}
-		})
-	}
-}

renovate.json

@@ -2,5 +2,15 @@
   "$schema": "https://docs.renovatebot.com/renovate-schema.json",
   "extends": [
     "config:recommended"
+  ],
+  "packageRules": [
+    {
+      "groupName": "OpenTelemetry",
+      "matchPackageNames": [
+        "go.opentelemetry.io/**",
+        "gitea.unbound.se/shiny/otelsetup",
+        "gitea.unbound.se/shiny/logging"
+      ]
+    }
   ]
 }