feat: add leeway.package.name attribute to phase and test spans

Co-authored-by: Ona <no-reply@ona.com>

Author: WVerlaek

pkg/leeway/build.go

@@ -3170,7 +3170,7 @@ func runGoTestWithTracing(buildctx *buildContext, p *Package, env []string, cwd,
 	}
 
 	// Create tracer and parse output
-	goTracer := NewGoTestTracer(tracer, parentCtx)
+	goTracer := NewGoTestTracer(tracer, parentCtx, p.FullName())
 	outputWriter := &reporterStream{R: buildctx.Reporter, P: p, IsErr: false}
 
 	if err := goTracer.parseJSONOutput(stdout, outputWriter); err != nil {

pkg/leeway/gotest_trace.go

@@ -35,19 +35,21 @@ type testSpanData struct {
 
 // GoTestTracer handles parsing Go test JSON output and creating OpenTelemetry spans
 type GoTestTracer struct {
-	tracer    trace.Tracer
-	parentCtx context.Context
+	tracer        trace.Tracer
+	parentCtx     context.Context
+	leewayPkgName string
 
 	mu    sync.Mutex
 	spans map[string]*testSpanData // key: "package/testname" or just "package" for package-level
 }
 
 // NewGoTestTracer creates a new GoTestTracer that will create spans as children of the given context
-func NewGoTestTracer(tracer trace.Tracer, parentCtx context.Context) *GoTestTracer {
+func NewGoTestTracer(tracer trace.Tracer, parentCtx context.Context, leewayPkgName string) *GoTestTracer {
 	return &GoTestTracer{
-		tracer:    tracer,
-		parentCtx: parentCtx,
-		spans:     make(map[string]*testSpanData),
+		tracer:        tracer,
+		parentCtx:     parentCtx,
+		leewayPkgName: leewayPkgName,
+		spans:         make(map[string]*testSpanData),
 	}
 }
 
@@ -163,6 +165,7 @@ func (t *GoTestTracer) handleRun(event *goTestEvent) {
 	)
 
 	span.SetAttributes(
+		attribute.String("leeway.package.name", t.leewayPkgName),
 		attribute.String("test.name", event.Test),
 		attribute.String("test.package", event.Package),
 		attribute.String("test.framework", "go"),
@@ -193,6 +196,7 @@ func (t *GoTestTracer) handlePackageStart(event *goTestEvent) {
 	)
 
 	span.SetAttributes(
+		attribute.String("leeway.package.name", t.leewayPkgName),
 		attribute.String("test.package", event.Package),
 		attribute.String("test.framework", "go"),
 		attribute.String("test.scope", "package"),

pkg/leeway/gotest_trace_test.go

@@ -24,7 +24,7 @@ func TestGoTestTracer_ParseJSONOutput(t *testing.T) {
 	ctx, parentSpan := tracer.Start(context.Background(), "parent")
 	defer parentSpan.End()
 
-	goTracer := NewGoTestTracer(tracer, ctx)
+	goTracer := NewGoTestTracer(tracer, ctx, "test:pkg")
 
 	// Simulate go test -json output
 	jsonOutput := `{"Time":"2024-01-01T10:00:00Z","Action":"start","Package":"example.com/pkg"}
@@ -123,7 +123,7 @@ func TestGoTestTracer_ParallelTests(t *testing.T) {
 	ctx, parentSpan := tracer.Start(context.Background(), "parent")
 	defer parentSpan.End()
 
-	goTracer := NewGoTestTracer(tracer, ctx)
+	goTracer := NewGoTestTracer(tracer, ctx, "test:pkg")
 
 	// Simulate parallel test execution with pause/cont events
 	jsonOutput := `{"Time":"2024-01-01T10:00:00Z","Action":"run","Package":"example.com/pkg","Test":"TestParallel"}
@@ -167,7 +167,7 @@ func TestGoTestTracer_ParallelTests(t *testing.T) {
 
 func TestGoTestTracer_NoTracer(t *testing.T) {
 	// Test that nil tracer doesn't panic
-	goTracer := NewGoTestTracer(nil, context.Background())
+	goTracer := NewGoTestTracer(nil, context.Background(), "test:pkg")
 
 	jsonOutput := `{"Time":"2024-01-01T10:00:00Z","Action":"run","Package":"example.com/pkg","Test":"TestOne"}
 {"Time":"2024-01-01T10:00:00.100Z","Action":"pass","Package":"example.com/pkg","Test":"TestOne","Elapsed":0.1}

pkg/leeway/reporter.go

@@ -949,6 +949,7 @@ func (r *OTelReporter) PackageBuildPhaseStarted(pkg *Package, phase PackageBuild
 
 	// Add phase attributes
 	span.SetAttributes(
+		attribute.String("leeway.package.name", pkgName),
 		attribute.String("leeway.phase.name", string(phase)),
 	)