fix(trace): avoid the forbidden test error pattern

The lint rule interprets the trace parameter's Error field as a forbidden test assertion. Name the parameter explicitly so production trace handling does not match that pattern.

Assisted-by: Codex:gpt-5.6 [Codex]

Author: localai-org-maint-bot

core/trace/backend_trace.go

@@ -200,7 +200,7 @@ func advanceBackendTraceID(id string) {
 	}
 }
 
-func RecordBackendTrace(t BackendTrace) {
+func RecordBackendTrace(trace BackendTrace) {
 	backendMu.Lock()
 	maxBody := backendMaxBodyBytes
 	store := backendTraceStore
@@ -210,22 +210,22 @@ func RecordBackendTrace(t BackendTrace) {
 	// (Inf/NaN) that encoding/json cannot marshal. A single such value — e.g. a
 	// -Inf dBFS audio metric from a silent clip — would otherwise fail the whole
 	// /api/backend-traces response and blank the Traces UI.
-	if t.Data != nil {
-		t.Data = sanitizeData(t.Data, maxBody)
+	if trace.Data != nil {
+		trace.Data = sanitizeData(trace.Data, maxBody)
 	}
-	if t.ID == "" {
-		t.ID = strconv.FormatUint(backendTraceIDSeq.Add(1), 10)
+	if trace.ID == "" {
+		trace.ID = strconv.FormatUint(backendTraceIDSeq.Add(1), 10)
 	}
-	if t.Error != "" {
-		t.Status = BackendTraceFailed
+	if trace.Error != "" {
+		trace.Status = BackendTraceFailed
 	} else {
-		t.Status = BackendTraceCompleted
+		trace.Status = BackendTraceCompleted
 	}
 	backendMu.Lock()
-	delete(backendInFlight, t.ID)
+	delete(backendInFlight, trace.ID)
 	backendMu.Unlock()
 	select {
-	case backendLogChan <- backendTraceCommand{trace: &t, store: store}:
+	case backendLogChan <- backendTraceCommand{trace: &trace, store: store}:
 	default:
 		xlog.Warn("Backend trace channel full, dropping trace")
 	}