feat: add client-side security and API notations to .internal contents

Author: superstrellaa

src/main/java/es/superstrellaa/storagemanager/StorageManagerAPI.java

@@ -1,9 +1,9 @@
 package es.superstrellaa.storagemanager;
 
-import es.superstrellaa.storagemanager.internal.SQLiteBackend;
-import es.superstrellaa.storagemanager.internal.cache.WriteCache;
-import es.superstrellaa.storagemanager.internal.lifecycle.ShutdownHook;
+import es.superstrellaa.storagemanager.internal.lifecycle.ServerBootstrap;
+import net.fabricmc.api.EnvType;
 import net.fabricmc.api.ModInitializer;
+import net.fabricmc.loader.api.FabricLoader;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -12,14 +12,28 @@ public class StorageManagerAPI implements ModInitializer {
 	public static final String MOD_ID = "storagemanager-api";
 	public static final Logger LOGGER = LoggerFactory.getLogger(MOD_ID);
 
-	//TODO General: Optimizar algo más, separar clases grandes en varias pequeñas y nuevas funcionalidades(añadir sistema de usar otro tipo de Backend aparte de SQLite, como texto plano(json, xml...) o PostgreSQL/MySQL)
+	/**
+	 * TODO:
+	 * * - Ver cómo hacerlo en mundos singleplayer
+	 *
+	 * * - Optimizar algo más el rendimiento, especialmente en la parte de escritura, para reducir la latencia y mejorar la eficiencia general.
+	 * * - Separar clases grandes en varias más pequeñas para mejorar la mantenibilidad y la claridad del código.
+	 * * - Añadir nuevas funcionalidades, como la capacidad de usar diferentes tipos de backend aparte de SQLite, como texto plano (JSON, XML...)
+	 * * o bases de datos relacionales(PostgreSQL/MySQL) para ofrecer más flexibilidad y cosas raras
+	 *
+	 * * - Hacer el readme de los huevos y la wiki (importante en el readme agregar que .internal no se toca)
+	 * * - Ah y el modrinth/curseforge
+	 * */
 
 	@Override
 	public void onInitialize() {
-		SQLiteBackend.init();
-		WriteCache.getInstance().start();
-		ShutdownHook.register();
+		if (FabricLoader.getInstance().getEnvironmentType() == EnvType.SERVER) {
+			ServerBootstrap.register();
+		} else {
+			LOGGER.warn("StorageManager API is running in client-side enviroment.");
+			LOGGER.warn("This is normal, but keep in mind that all API operations will be no-ops and won't have any effect.");
+		}
 
-		LOGGER.info("StorageManager API initialized with write-behind cache");
+		LOGGER.info("StorageManager API initialized (common)");
 	}
 }

src/main/java/es/superstrellaa/storagemanager/internal/SQLiteBackend.java

@@ -1,13 +1,15 @@
 package es.superstrellaa.storagemanager.internal;
 
 import es.superstrellaa.storagemanager.StorageManagerAPI;
+import org.jetbrains.annotations.ApiStatus;
 
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.sql.Connection;
 import java.sql.DriverManager;
 import java.sql.Statement;
 
+@ApiStatus.Internal
 public final class SQLiteBackend {
 
     private static Connection connection;

src/main/java/es/superstrellaa/storagemanager/internal/StoragePaths.java

@@ -1,9 +1,11 @@
 package es.superstrellaa.storagemanager.internal;
 
 import net.fabricmc.loader.api.FabricLoader;
+import org.jetbrains.annotations.ApiStatus;
 
 import java.nio.file.Path;
 
+@ApiStatus.Internal
 public final class StoragePaths {
 
     private static final Path BASE_DIR =

src/main/java/es/superstrellaa/storagemanager/internal/TableExecutor.java

@@ -5,13 +5,15 @@
 import es.superstrellaa.storagemanager.api.schema.Column;
 import es.superstrellaa.storagemanager.api.schema.TableSchema;
 import es.superstrellaa.storagemanager.internal.cache.WriteCache;
+import org.jetbrains.annotations.ApiStatus;
 
 import java.sql.*;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
 import java.util.StringJoiner;
 
+@ApiStatus.Internal
 public final class TableExecutor {
 
     /**

src/main/java/es/superstrellaa/storagemanager/internal/cache/WriteCache.java

@@ -3,6 +3,7 @@
 import es.superstrellaa.storagemanager.StorageManagerAPI;
 import es.superstrellaa.storagemanager.api.data.RowData;
 import es.superstrellaa.storagemanager.internal.SQLiteBackend;
+import org.jetbrains.annotations.ApiStatus;
 
 import java.sql.PreparedStatement;
 import java.sql.SQLException;
@@ -12,6 +13,7 @@
 import java.util.concurrent.ScheduledExecutorService;
 import java.util.concurrent.TimeUnit;
 
+@ApiStatus.Internal
 public final class WriteCache {
 
     private static final long FLUSH_INTERVAL_MS = 5000;

src/main/java/es/superstrellaa/storagemanager/internal/lifecycle/ServerBootstrap.java

@@ -0,0 +1,29 @@
+package es.superstrellaa.storagemanager.internal.lifecycle;
+
+import es.superstrellaa.storagemanager.StorageManagerAPI;
+import es.superstrellaa.storagemanager.internal.SQLiteBackend;
+import es.superstrellaa.storagemanager.internal.cache.WriteCache;
+import es.superstrellaa.storagemanager.internal.middleware.ServerGuard;
+import net.fabricmc.fabric.api.event.lifecycle.v1.ServerLifecycleEvents;
+
+public final class ServerBootstrap {
+
+    private static boolean initialized = false;
+
+    public static void register() {
+        ServerLifecycleEvents.SERVER_STARTED.register(server -> {
+            if (initialized) return;
+
+            SQLiteBackend.init();
+            WriteCache.getInstance().start();
+            ShutdownHook.register();
+
+            ServerGuard.markServerReady();
+
+            initialized = true;
+            StorageManagerAPI.LOGGER.info("StorageManager API initialized (server-side)");
+        });
+    }
+
+    private ServerBootstrap() {}
+}

src/main/java/es/superstrellaa/storagemanager/internal/lifecycle/ShutdownHook.java

@@ -3,7 +3,9 @@
 import es.superstrellaa.storagemanager.internal.SQLiteBackend;
 import es.superstrellaa.storagemanager.internal.cache.WriteCache;
 import net.fabricmc.fabric.api.event.lifecycle.v1.ServerLifecycleEvents;
+import org.jetbrains.annotations.ApiStatus;
 
+@ApiStatus.Internal
 public final class ShutdownHook {
 
     public static void register() {

src/main/java/es/superstrellaa/storagemanager/internal/middleware/ServerGuard.java

@@ -0,0 +1,28 @@
+package es.superstrellaa.storagemanager.internal.middleware;
+
+import es.superstrellaa.storagemanager.StorageManagerAPI;
+import net.fabricmc.api.EnvType;
+import net.fabricmc.loader.api.FabricLoader;
+import org.jetbrains.annotations.ApiStatus;
+
+@ApiStatus.Internal
+public final class ServerGuard {
+
+    private static volatile boolean serverReady = false;
+
+    public static void markServerReady() {
+        serverReady = true;
+    }
+
+    public static void requireServer() {
+        if (!serverReady) {
+            if (FabricLoader.getInstance().getEnvironmentType() == EnvType.SERVER) {
+                StorageManagerAPI.LOGGER.error("StorageManager API is not ready yet! This likely means that you are trying to use it during mod initialization, which is too early. Please wait until the server has started.");
+            } else {
+                StorageManagerAPI.LOGGER.warn("StorageManager API isn't meant for here! This likely means that you are trying to use it on the client side, which is not supported. Please only use the API on the server side.");
+            }
+        }
+    }
+
+    private ServerGuard() {}
+}