Refactor MetricsService to use getSyncExecutor for async snapshot collection

Author: renvins

common/src/main/java/it/renvins/serverpulse/common/MetricsService.java

@@ -57,7 +57,7 @@ public void collectAndSendMetrics() {
             ServerPulseProvider.get().getDatabaseService().startRetryTaskIfNeeded();
             return;
         }
-        CompletableFuture.supplyAsync(this::collectSnapshot, scheduler::runSync)
+        CompletableFuture.supplyAsync(this::collectSnapshot, scheduler.getSyncExecutor())
                 .thenApplyAsync(snapshot -> {
                     if (snapshot == null) {
                         logger.warning("Snapshot is null. Skipping metrics send.");

common/src/main/java/it/renvins/serverpulse/common/scheduler/TaskScheduler.java

@@ -1,5 +1,7 @@
 package it.renvins.serverpulse.common.scheduler;
 
+import java.util.concurrent.Executor;
+
 public interface TaskScheduler {
 
     /**
@@ -54,4 +56,12 @@ default Task runTaskLater(Runnable task, long delayTicks) {
      * @return A Task object representing the task
      */
     Task runTaskLaterAsync(Runnable task, long delayTicks);
+
+    default Executor getSyncExecutor() {
+        try {
+            return this::runSync;
+        } catch (UnsupportedOperationException e) {
+            return this::runAsync;
+        }
+    }
 }