🔧 fix: ensure farmer exists before accessing tokens

[mkmccarty]

No description provided.

[Copilot]

Pull request overview

This PR adds a nil check to the GetTokens function to prevent potential nil pointer dereference errors when accessing a farmer's tokens. The fix ensures that if a farmer doesn't exist for the given userID, a new farmer is created before attempting to access their token count.

Key Changes

• Added nil check and newFarmer() call in GetTokens function to ensure farmer exists before token access
• Aligns with the existing defensive programming pattern used in other getter/setter functions in the file

Comments suppressed due to low confidence (1)

src/farmerstate/farmerstate.go:235

• The nil check on lines 229-231 makes the subsequent check on line 232 redundant. After calling newFarmer(userID), farmerstate[userID] is guaranteed to be non-nil, so the ok check will always be true. Consider simplifying this to directly return farmerstate[userID].Tokens after the nil check, similar to the pattern used in GetMissionShipSecondary (lines 208-213).

	if farmerstate, ok := farmerstate[userID]; ok {
		return farmerstate.Tokens
	}
	return 0