Small bit of work on crosspost cog

cogs/crosspostcog.py

@@ -1,3 +1,4 @@
+from rscollections import BoundList
 from discord import Guild, Message
 from cogs.basecog import BaseCog
 
@@ -5,6 +6,7 @@ class CrossPostCog(BaseCog):
 	class SpamContext:
 		def __init__(self, member):
 			self.member = member
+			self.age = timedate.now()
 			self.is_warned = False
 
 	STATE_KEY_RECENT_MESSAGES = "crosspost_recent_messages"
@@ -13,14 +15,14 @@ class CrossPostCog(BaseCog):
 	def __init__(self, bot):
 		super().__init__(bot)
 		self.max_recent_messages = 20
+		self.max_spam_contexts = 12
 		self.messages_per_user = 3
 		self.min_message_length = 10
 
 	def __record_message(self, message: Message) -> None:
-		recent_messages = Storage.get_state_value(message.guild, self.STATE_KEY_RECENT_MESSAGES) or []
+		recent_messages = Storage.get_state_value(message.guild, self.STATE_KEY_RECENT_MESSAGES) \
+			or BoundList(self.max_recent_messages, lambda index, message : message.created_at)
 		recent_messages.append(message)
-		while len(recent_messages) > self.max_recent_messages:
-			recent_messages.pop(0)
 		Storage.set_state_value(message.guild, self.STATE_KEY_RECENT_MESSAGES, recent_messages)
 
 	def __check_for_spam(self, guild: Guild) -> None:
@@ -40,11 +42,18 @@ class CrossPostCog(BaseCog):
 				spamming_members.add(message.author)
 		for member in spamming_members:
 			context = self.__spam_context_for_user(member, True)
+			if not context.is_warned:
+				self.__on_new_spam(context)
+
+	def __on_new_spam(self, context: SpamContext):
+		context.is_warned = True
+		# TODO
 
 	def __spam_context_for_user(self,
 			member: Member,
 			create_if_missing: bool = False) -> SpamContext:
-		spam_lookup = Storage.get_state_value(member.guild, self.STATE_KEY_SPAM_CONTEXT) or {}
+		spam_lookup = Storage.get_state_value(member.guild, self.STATE_KEY_SPAM_CONTEXT) \
+			or BoundDict(self.max_spam_contexts, lambda key, context : context.age)
 		context = spam_lookup.get(member.id)
 		if context:
 			return context