Cheater_Detection: performance fix

Author: titaniummachine1

Cheater_Detection/Main.lua

@@ -384,6 +384,7 @@ local function OnCreateMove(cmd)
 
 	-- Pre-filter active players into a flat list (reuse module-level table)
 	Profiler.Begin("PlayerScan_Loop")
+	Profiler.Begin("PlayerScan_BuildActive")
 	for k = 1, #activePlayers do activePlayers[k] = nil end
 	for id, existingState in pairs(stateTable) do
 		local pdata = existingState.pdata
@@ -423,6 +424,7 @@ local function OnCreateMove(cmd)
 		activePlayers[#activePlayers + 1] = existingState
 		::continue::
 	end
+	Profiler.End("PlayerScan_BuildActive")
 
 	if enableSilent then
 		Profiler.Begin("History_SilentAim")
@@ -434,14 +436,15 @@ local function OnCreateMove(cmd)
 	if enableChoke or enableDoubleTap then
 		Profiler.Begin("History_Simtime")
 		for _, pState in ipairs(activePlayers) do
-			DetectionConfig.RecordHistory(pState.wrap, "Simtime")
+			HistoryManager.RecordSimtimeTick(pState.id, pState.pdata.simTime)
 		end
 		Profiler.End("History_Simtime")
 	end
 
 	-- ValveCheck runs via scheduler + DirtySystem "checks" (once per player per map)
 	-- Auto-disconnect enforcement still runs every tick for confirmed Valve employees
 	if enableValveCheck then
+		Profiler.Begin("ValveCheck")
 		for _, pState in ipairs(activePlayers) do
 			enforceValveAutoDisconnect(pState)
 			if sessionState.valveDisconnectTriggered then
@@ -450,6 +453,7 @@ local function OnCreateMove(cmd)
 				return
 			end
 		end
+		Profiler.End("ValveCheck")
 	end
 
 	if enableSilent then

Cheater_Detection/Utils/HistoryManager.lua

@@ -87,11 +87,22 @@ local function simtimeToTicks(deltaSec)
 	return math.floor(deltaSec / globals.TickInterval() + 0.5)
 end
 
+local function getPreviousRecord(history)
+	if not history or history._count < 2 then
+		return nil
+	end
+	local prevIdx = history._head - 1
+	if prevIdx < 1 then
+		prevIdx = prevIdx + maxRetentionTicks
+	end
+	return history[prevIdx]
+end
+
 -- Stored on each ring slot when it becomes head: gap to the previous simtime sample (ticks).
 local function writeSimDeltaTicks(history, record)
 	local simField = HistoryManager.Fields.SimulationTime
 	local simNew = record[simField]
-	local older = HistoryManager.GetRecordAt(history, 1)
+	local older = getPreviousRecord(history)
 	if not simNew or not older then
 		record._sim_delta_ticks = nil
 		return
@@ -285,6 +296,62 @@ function HistoryManager.RequestField(player, field)
 	return record[field]
 end
 
+--- Fast simtime ring write: uses pdata.simTime (no WrappedPlayer / second netprop read).
+--- Shares head advance + dedup with RequestField (SilentAim angles on same tick stay in one slot).
+function HistoryManager.RecordSimtimeTick(steamID, simTime)
+	if not initialized or not activeFields[HistoryManager.Fields.SimulationTime] then
+		return
+	end
+	if not steamID or simTime == nil then
+		return
+	end
+
+	local id = tostring(steamID)
+	local simField = HistoryManager.Fields.SimulationTime
+	local curTick = globals.TickCount()
+
+	local stored = _storedFieldsByPlayer[id]
+	if stored and stored[simField] == curTick then
+		return
+	end
+
+	local history = playerHistories[id]
+	if not history then
+		history = { _head = 0, _count = 0, _lastTick = -1 }
+		playerHistories[id] = history
+	end
+
+	if history._lastTick ~= curTick then
+		history._head = (history._head % maxRetentionTicks) + 1
+		if history._count < maxRetentionTicks then
+			history._count = history._count + 1
+		end
+		history._lastTick = curTick
+		if stored then
+			for f in pairs(stored) do
+				stored[f] = nil
+			end
+		end
+	end
+
+	local head = history._head
+	local record = history[head]
+	if type(record) ~= "table" then
+		record = {}
+		history[head] = record
+	end
+
+	record[simField] = simTime
+	writeSimDeltaTicks(history, record)
+	record._tick = curTick
+
+	if not stored then
+		stored = {}
+		_storedFieldsByPlayer[id] = stored
+	end
+	stored[simField] = curTick
+end
+
 function HistoryManager.RequestFields(player, fields)
 	if not fields then return end
 	for i = 1, #fields do