fix: comment

packages/shared/src/types/events/common.ts

@@ -58,7 +58,12 @@ export type ProtocolEvent<T extends ContractName, E extends ContractToEventName<
           : never;
 };
 
-//TODO: check if this is the best way
+/**
+ * TODO: This type is currently only used in the EventsFetcher and IndexerClient.
+ * In the future, we should evaluate if a more decoupled or generic type is needed
+ * to improve flexibility and reduce dependencies across different parts of the system.
+ * Consider creating separate event types for different contexts if necessary.
+ */
 export type AnyProtocolEvent = Omit<
     ProtocolEvent<ContractName, ContractToEventName<ContractName>>,
     "strategyId"