chore: demote some no longer urgent TODOs

Author: thlorenz

magicblock-api/src/magic_validator.rs

@@ -174,7 +174,7 @@ impl MagicValidator {
         config: MagicValidatorConfig,
         identity_keypair: Keypair,
     ) -> ApiResult<Self> {
-        // TODO(thlorenz): @@ this will need to be recreated on each start
+        // TODO(thlorenz): this will need to be recreated on each start
         let token = CancellationToken::new();
 
         let (geyser_manager, geyser_rpc_service) =
@@ -650,7 +650,7 @@ impl MagicValidator {
     }
 
     async fn ensure_validator_funded_on_chain(&self) -> ApiResult<()> {
-        // TODO: @@@ configurable?
+        // TODO(thlorenz) make this configurable in the future
         const MIN_BALANCE_SOL: u64 = 5;
         // TODO: @@ duplicate code getting remote_rpc_config
         let accounts_config = try_convert_accounts_config(

magicblock-geyser-plugin/src/grpc.rs

@@ -144,7 +144,7 @@ impl GrpcService {
         block_fail_action: ConfigBlockFailAction,
     ) {
         const PROCESSED_MESSAGES_MAX: usize = 31;
-        // TODO(thlorenz): @@@ This could become a bottleneck affecting latency
+        // TODO(thlorenz): This could become a bottleneck affecting latency
         const PROCESSED_MESSAGES_SLEEP: Duration = Duration::from_millis(10);
 
         let mut messages: BTreeMap<u64, SlotMessages> = Default::default();

magicblock-rpc/src/transaction.rs

@@ -213,7 +213,7 @@ pub(crate) async fn send_transaction(
 
 /// Verifies only the transaction signature and is used when sending a
 /// transaction to avoid the extra overhead of [sig_verify_transaction_and_check_precompiles]
-/// TODO(thlorenz): @@ sigverify takes upwards of 90µs which is 30%+ of
+/// TODO(thlorenz): sigverify takes upwards of 90µs which is 30%+ of
 /// the entire time it takes to execute a transaction.
 /// Therefore this an intermediate solution and we need to investigate verifying the
 /// wire_transaction instead (solana sigverify implementation is packet based)