fix: task ID and persistence discrepancy (#48)

* fix: align task names and handler_ids

* fix: .contract_address is correct ContractLog attr

* fix: bad prop ref for event_name on ContractLog

* docs: added minor TODO for future us

silverback/middlewares.py

@@ -18,10 +18,11 @@ def resolve_task(message: TaskiqMessage) -> Tuple[str, Optional[int], Optional[i
     if task_id == "block":
         block_number = message.args[0].number
         task_id = handler_id_block(block_number)
-    elif task_id == "event":
+    elif "event" in task_id:
         block_number = message.args[0].block_number
         log_index = message.args[0].log_index
-        task_id = handler_id_event(message.args[0].address, message.args[0].abi.name)
+        # TODO: Should standardize on event signature here instead of name in case of overloading
+        task_id = handler_id_event(message.args[0].contract_address, message.args[0].event_name)
 
     return task_id, block_number, log_index
 

silverback/types.py

@@ -39,4 +39,4 @@ def handler_id_block(block_number: Optional[int]) -> str:
 def handler_id_event(contract_address: Optional[str], event_signature: str) -> str:
     """Return a unique handler ID string for an event"""
     # TODO: Under what circumstance can address be None?
-    return f"event/{contract_address or 'unknown'}/{event_signature}"
+    return f"{contract_address or 'unknown'}/event/{event_signature}"