🚀 feat: Add save time tracking to prevent frequent contract saves

[mkmccarty]

No description provided.

[Copilot]

Pull request overview

This PR implements a throttling mechanism to prevent frequent contract saves by tracking the last save time and enforcing a minimum 15-second interval between saves.

• Added LastSaveTime field to the Contract struct to track when contracts were last persisted
• Implemented save throttling logic in saveData() that checks if 15 seconds have passed since the last save
• Added nil check for contract before attempting to access its fields

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
src/boost/boost.go	Added LastSaveTime field to Contract struct to track save timestamps
src/boost/boost_datastore.go	Implemented save throttling logic with 15-second minimum interval and added nil contract check

[Copilot]

The LastSaveTime field is not protected by the contract's mutex, which could lead to race conditions in concurrent access scenarios. Since the Contract struct has a mutex sync.Mutex field (line 325), and LastSaveTime is being read and written in saveData(), these operations should be protected with mutex locks.

Consider wrapping the time check and update in a mutex lock:

contract.mutex.Lock()
shouldSave := time.Since(contract.LastSaveTime) >= 15*time.Second
if shouldSave {
    contract.LastSaveTime = time.Now()
}
contract.mutex.Unlock()

if shouldSave {
    saveSqliteData(contract)
}

Suggested change

	if contract.LastSaveTime.Before(time.Now().Add(-15 * time.Second)) {
	contract.LastSaveTime = time.Now()
	contract.mutex.Lock()
	shouldSave := contract.LastSaveTime.Before(time.Now().Add(-15 * time.Second))
	if shouldSave {
	contract.LastSaveTime = time.Now()
	}
	contract.mutex.Unlock()
	if shouldSave {