From bc3195c8d8d0052f0cc7d2f09914aff9344f2aa3 Mon Sep 17 00:00:00 2001
From: Luis Pater <webmaster@idotorg.org>
Date: Sat, 10 Jan 2026 14:46:59 +0800
Subject: [PATCH] refactor(logger): remove unnecessary request details limit
 logic

---
 internal/usage/logger_plugin.go | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/internal/usage/logger_plugin.go b/internal/usage/logger_plugin.go
index 38177d7d..e4371e8d 100644
--- a/internal/usage/logger_plugin.go
+++ b/internal/usage/logger_plugin.go
@@ -87,10 +87,6 @@ type modelStats struct {
 	Details       []RequestDetail
 }
 
-// maxDetailsPerModel limits the number of request details retained per model
-// to prevent unbounded memory growth. Oldest entries are dropped when exceeded.
-const maxDetailsPerModel = 1000
-
 // RequestDetail stores the timestamp and token usage for a single request.
 type RequestDetail struct {
 	Timestamp time.Time  `json:"timestamp"`
@@ -225,11 +221,6 @@ func (s *RequestStatistics) updateAPIStats(stats *apiStats, model string, detail
 	modelStatsValue.TotalRequests++
 	modelStatsValue.TotalTokens += detail.Tokens.TotalTokens
 	modelStatsValue.Details = append(modelStatsValue.Details, detail)
-	// Prevent unbounded growth by dropping oldest entries when limit exceeded
-	if len(modelStatsValue.Details) > maxDetailsPerModel {
-		excess := len(modelStatsValue.Details) - maxDetailsPerModel
-		modelStatsValue.Details = modelStatsValue.Details[excess:]
-	}
 }
 
 // Snapshot returns a copy of the aggregated metrics for external consumption.