From 69502757529bf1525dd57f4891911df03dd9abf9 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Wed, 1 Jul 2026 14:35:30 +0000
Subject: [PATCH] =?UTF-8?q?=F0=9F=A7=B9=20[code=20health]=20Remove=20unuse?=
 =?UTF-8?q?d=20function=20cloneEvents?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

🎯 What: Removed unused `cloneEvents` function from `internal/core/tokenaccounting/streamusage/reconstructor.go`.
💡 Why: It's an unexported function that is not referenced anywhere in the source file or package. Removing it improves maintainability and readability by reducing dead code.
✅ Verification: Verified by running `go fmt ./...` and `go test ./...` to ensure no functionality was affected.
✨ Result: A cleaner codebase without unnecessary dead code.

Co-authored-by: matdev83 <211248003+matdev83@users.noreply.github.com>
---
 .../core/tokenaccounting/streamusage/reconstructor.go     | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/internal/core/tokenaccounting/streamusage/reconstructor.go b/internal/core/tokenaccounting/streamusage/reconstructor.go
index 12c5a16d..e184283f 100644
--- a/internal/core/tokenaccounting/streamusage/reconstructor.go
+++ b/internal/core/tokenaccounting/streamusage/reconstructor.go
@@ -222,14 +222,6 @@ func warningForCountError(code WarningCode, message string, err error) Warning {
 	return Warning{Code: code, Message: fmt.Sprintf("%s: %v", message, err), Err: err}
 }
 
-func cloneEvents(events []lipapi.Event) []lipapi.Event {
-	out := make([]lipapi.Event, len(events))
-	for i, ev := range events {
-		out[i] = cloneEvent(ev)
-	}
-	return out
-}
-
 func nonUsageEvents(events []lipapi.Event) []lipapi.Event {
 	out := []lipapi.Event{}
 	for _, ev := range events {