chore: rename logItem

Author: giortzisg

interfaces.go

@@ -728,13 +728,13 @@ func (l *Log) ToEnvelopeItem() (*protocol.EnvelopeItem, error) {
 	}, nil
 }
 
-// ToLogPayload converts the Log to a protocol.LogPayload for batching.
-func (l *Log) ToLogPayload() protocol.LogPayload {
+// ToLogPayload converts the Log to a protocol.LogItem for batching.
+func (l *Log) ToLogPayload() protocol.LogItem {
 	attrs := make(map[string]protocol.LogAttribute, len(l.Attributes))
 	for k, v := range l.Attributes {
 		attrs[k] = protocol.LogAttribute{Value: v.Value, Type: string(v.Type)}
 	}
-	return protocol.LogPayload{
+	return protocol.LogItem{
 		Timestamp:  l.Timestamp,
 		TraceID:    l.TraceID.String(),
 		Level:      string(l.Level),

internal/protocol/log_batch.go

@@ -13,9 +13,9 @@ type LogAttribute struct {
 	Type  string `json:"type"`
 }
 
-// LogPayload represents the serialized shape of a single log record inside a batched
+// LogItem represents the serialized shape of a single log record inside a batched
 // log envelope item. Keep in sync with sentry.Log fields that are meant to be emitted.
-type LogPayload struct {
+type LogItem struct {
 	Timestamp  time.Time               `json:"timestamp,omitempty"`
 	TraceID    string                  `json:"trace_id,omitempty"`
 	Level      string                  `json:"level"`
@@ -24,13 +24,13 @@ type LogPayload struct {
 	Attributes map[string]LogAttribute `json:"attributes,omitempty"`
 }
 
-// LogPayloader is implemented by items that can convert to a LogPayload for batching.
+// LogPayloader is implemented by items that can convert to a LogItem for batching.
 type LogPayloader interface {
-	ToLogPayload() LogPayload
+	ToLogPayload() LogItem
 }
 
 // MarshalJSON encodes timestamp as seconds since epoch per Sentry logs spec.
-func (lp LogPayload) MarshalJSON() ([]byte, error) {
+func (lp LogItem) MarshalJSON() ([]byte, error) {
 	// Convert time.Time to seconds float if set
 	var ts *float64
 	if !lp.Timestamp.IsZero() {
@@ -56,13 +56,13 @@ func (lp LogPayload) MarshalJSON() ([]byte, error) {
 	return json.Marshal(out)
 }
 
-// Logs is a container for multiple LogPayload items which knows how to convert
+// Logs is a container for multiple LogItem items which knows how to convert
 // itself into a single batched log envelope item.
-type Logs []LogPayload
+type Logs []LogItem
 
 func (ls Logs) ToEnvelopeItem() (*EnvelopeItem, error) {
 	wrapper := struct {
-		Items []LogPayload `json:"items"`
+		Items []LogItem `json:"items"`
 	}{Items: ls}
 
 	payload, err := json.Marshal(wrapper)

internal/telemetry/scheduler.go

@@ -209,9 +209,9 @@ func (s *Scheduler) processItems(buffer BufferInterface[protocol.EnvelopeItemCon
 
 	switch category {
 	case ratelimit.CategoryLog:
-		envItems := make([]protocol.LogPayload, 0, len(items))
+		envItems := make([]protocol.LogItem, 0, len(items))
 		for _, it := range items {
-			if lp, ok := any(it).(interface{ ToLogPayload() protocol.LogPayload }); ok {
+			if lp, ok := any(it).(interface{ ToLogPayload() protocol.LogItem }); ok {
 				envItems = append(envItems, lp.ToLogPayload())
 			} else {
 				debuglog.Printf("Invalid envelope item; cannot convert to log: %v", it)
@@ -243,7 +243,6 @@ func (s *Scheduler) processItems(buffer BufferInterface[protocol.EnvelopeItemCon
 			s.sendItems(it)
 		}
 	}
-
 }
 
 func (s *Scheduler) sendItems(item protocol.EnvelopeItemConvertible) {

internal/telemetry/scheduler_test.go

@@ -53,6 +53,13 @@ func (t *testTelemetryItem) GetDynamicSamplingContext() map[string]string {
 	return nil
 }
 
+func (t *testTelemetryItem) ToLogPayload() protocol.LogItem {
+	return protocol.LogItem{
+		Level: "info",
+		Body:  t.data,
+	}
+}
+
 func TestNewTelemetryScheduler(t *testing.T) {
 	transport := &testutils.MockTelemetryTransport{}
 	dsn := &protocol.Dsn{}
@@ -148,7 +155,7 @@ func TestTelemetrySchedulerFlush(t *testing.T) {
 					i++
 				}
 			},
-			expectedCount: 4,
+			expectedCount: 3,
 		},
 		{
 			name: "priority ordering - error and log",