Bug fix: #6

using deepcopy with the discord.Message object causes an unforseen
crash where serialization of the object with pickle by deepcopy causes
the method call to crash, which leads to a halt in the async job.

This is fixed by moving the only temporary stripping of characters for
identifying a callback bind to the method "matches" in the class
Callback itself.

It resolves the previous bug (5) as well as this resulting bug.

Author: dotchetter

baseclasses/baseclasses.py

@@ -10,7 +10,6 @@
 from datetime import datetime, timedelta, time
 from enum import Enum, auto
 from abc import ABC, abstractmethod
-from copy import deepcopy
 
 from ..core.callback import Callback
 from ..core.internals import _cim
@@ -161,11 +160,8 @@ def get_callback(self, message: Message) -> Callback:
         should be overloaded if a different return behavior 
         in a no-match-found scenario is desired.
         """
-        message_copy = deepcopy(message)
-        message_copy.content = [i.strip(FeatureCommandParserBase.IGNORED_CHARS) for i in message_copy.content]
-        
         for cb in self._callbacks:
-            match = cb.matches(message_copy)
+            match = cb.matches(Message)
             if match: return cb
         return None
 

core/callback.py

@@ -38,6 +38,7 @@ class Callback:
 		(bool) whether the order of items in the 
 		lead or trail property is trivial or not 
 	"""
+	IGNORED_CHARS = '?=)(/&%¤#"!,.-;:_^*`´><|'
 
 	__slots__ = ('_lead', '_trail', '_func', '_bindings', 
 				 '_interactive', '_ordered', '_intact_lead', 
@@ -90,7 +91,7 @@ def matches(self, message: Message) -> bool:
 			Bool, True if self matches command
 		"""
 		match_trail = False
-		lowered = [i.lower() for i in message.content]
+		lowered = [i.lower().strip(Callback.IGNORED_CHARS) for i in message.content]
 
 		if not (match_lead := [i for i in self._lead if i in lowered]):
 			return None