diff --git a/src/main/java/me/unurled/sacredrealms/sr/commands/CommandManager.java b/src/main/java/me/unurled/sacredrealms/sr/commands/CommandManager.java
index ef9ee08..a6a1fe7 100644
--- a/src/main/java/me/unurled/sacredrealms/sr/commands/CommandManager.java
+++ b/src/main/java/me/unurled/sacredrealms/sr/commands/CommandManager.java
@@ -4,6 +4,7 @@ import me.unurled.sacredrealms.sr.SR;
 import me.unurled.sacredrealms.sr.commands.admin.AttributeCommand;
 import me.unurled.sacredrealms.sr.commands.player.ResetAdventureCommand;
 import me.unurled.sacredrealms.sr.managers.Manager;
+import org.bukkit.command.PluginCommand;
 
 public class CommandManager extends Manager {
 
@@ -11,10 +12,17 @@ public class CommandManager extends Manager {
   @Override
   public void load() {
     super.load();
-    SR.getInstance().getCommand("attributes").setExecutor(new AttributeCommand());
-    SR.getInstance().getCommand("attributes").setTabCompleter(new AttributeCommand());
+    SR instance = SR.getInstance();
+    PluginCommand attributes = instance.getCommand("attributes");
+    if (attributes != null) {
+      attributes.setExecutor(new AttributeCommand());
+      attributes.setTabCompleter(new AttributeCommand());
+    }
 
-    SR.getInstance().getCommand("resetadventure").setExecutor(new ResetAdventureCommand());
-    SR.getInstance().getCommand("resetadventure").setTabCompleter(new ResetAdventureCommand());
+    PluginCommand resetadventure = instance.getCommand("resetadventure");
+    if (resetadventure != null) {
+      resetadventure.setExecutor(new ResetAdventureCommand());
+      resetadventure.setTabCompleter(new ResetAdventureCommand());
+    }
   }
 }
diff --git a/src/main/java/me/unurled/sacredrealms/sr/commands/player/ResetAdventureCommand.java b/src/main/java/me/unurled/sacredrealms/sr/commands/player/ResetAdventureCommand.java
index e99d611..216864b 100644
--- a/src/main/java/me/unurled/sacredrealms/sr/commands/player/ResetAdventureCommand.java
+++ b/src/main/java/me/unurled/sacredrealms/sr/commands/player/ResetAdventureCommand.java
@@ -133,12 +133,19 @@ public class ResetAdventureCommand implements TabExecutor {
       if (args[0].isBlank()) {
         return Arrays.stream(Bukkit.getOfflinePlayers())
             .map(OfflinePlayer::getName)
-            .toList().stream().limit(10).toList();
+            .toList()
+            .stream()
+            .limit(10)
+            .toList();
       }
       return Arrays.stream(Bukkit.getOfflinePlayers())
-        .map(OfflinePlayer::getName).filter(Objects::nonNull)
+          .map(OfflinePlayer::getName)
+          .filter(Objects::nonNull)
           .filter(name -> name.toLowerCase().startsWith(args[0].toLowerCase()))
-          .toList().stream().limit(10).toList();
+          .toList()
+          .stream()
+          .limit(10)
+          .toList();
     }
     return null;
   }
diff --git a/src/main/java/me/unurled/sacredrealms/sr/components/attributes/Attribute.java b/src/main/java/me/unurled/sacredrealms/sr/components/attributes/Attribute.java
index 3b04d07..65b3af5 100644
--- a/src/main/java/me/unurled/sacredrealms/sr/components/attributes/Attribute.java
+++ b/src/main/java/me/unurled/sacredrealms/sr/components/attributes/Attribute.java
@@ -50,6 +50,7 @@ public enum Attribute {
     return minValue;
   }
 
+  @SuppressWarnings("SameReturnValue")
   public PersistentDataType<Double, Double> getType() {
     return PersistentDataType.DOUBLE;
   }
diff --git a/src/main/java/me/unurled/sacredrealms/sr/components/combat/CombatManager.java b/src/main/java/me/unurled/sacredrealms/sr/components/combat/CombatManager.java
index 92051ba..ca64642 100644
--- a/src/main/java/me/unurled/sacredrealms/sr/components/combat/CombatManager.java
+++ b/src/main/java/me/unurled/sacredrealms/sr/components/combat/CombatManager.java
@@ -17,11 +17,13 @@ import org.bukkit.event.entity.EntityDamageByBlockEvent;
 import org.bukkit.event.entity.EntityDamageByEntityEvent;
 import org.bukkit.event.entity.EntityDamageEvent;
 import org.bukkit.inventory.EntityEquipment;
+import org.jetbrains.annotations.NotNull;
 
+@SuppressWarnings("EmptyMethod")
 public class CombatManager extends Manager {
 
   @EventHandler
-  public void onDamage(EntityDamageByBlockEvent e) {
+  public void onDamage(@NotNull EntityDamageByBlockEvent e) {
     // TODO: Implement
     // blast enchant :shrug:
     // fall damage, fire damage, etc
@@ -31,7 +33,7 @@ public class CombatManager extends Manager {
 
   // TODO: Finish this
   @EventHandler
-  public void onDamage(EntityDamageByEntityEvent e) {
+  public void onDamage(@NotNull EntityDamageByEntityEvent e) {
     e.setDamage(0.0);
 
     if (!(e.getDamager() instanceof LivingEntity damager)
diff --git a/src/main/java/me/unurled/sacredrealms/sr/components/entity/EntityManager.java b/src/main/java/me/unurled/sacredrealms/sr/components/entity/EntityManager.java
index ea31908..d037abc 100644
--- a/src/main/java/me/unurled/sacredrealms/sr/components/entity/EntityManager.java
+++ b/src/main/java/me/unurled/sacredrealms/sr/components/entity/EntityManager.java
@@ -38,7 +38,7 @@ public class EntityManager extends Manager {
   }
 
   @EventHandler
-  public void onEntitySpawn(EntitySpawnEvent e) {
+  public void onEntitySpawn(@NotNull EntitySpawnEvent e) {
     if (e.getEntity() instanceof Player) return;
 
     if (isSREntity(e.getEntity().getUniqueId())) return;
diff --git a/src/main/java/me/unurled/sacredrealms/sr/components/item/ItemManager.java b/src/main/java/me/unurled/sacredrealms/sr/components/item/ItemManager.java
index 211bd9e..09c47e4 100644
--- a/src/main/java/me/unurled/sacredrealms/sr/components/item/ItemManager.java
+++ b/src/main/java/me/unurled/sacredrealms/sr/components/item/ItemManager.java
@@ -4,6 +4,7 @@ import java.util.HashMap;
 import me.unurled.sacredrealms.sr.managers.Manager;
 import org.bukkit.NamespacedKey;
 
+@SuppressWarnings("EmptyMethod")
 public class ItemManager extends Manager {
   public static final NamespacedKey ID = new NamespacedKey("sr", "id");
 
diff --git a/src/main/java/me/unurled/sacredrealms/sr/components/item/abilities/Ability.java b/src/main/java/me/unurled/sacredrealms/sr/components/item/abilities/Ability.java
index f48e79f..d0250f6 100644
--- a/src/main/java/me/unurled/sacredrealms/sr/components/item/abilities/Ability.java
+++ b/src/main/java/me/unurled/sacredrealms/sr/components/item/abilities/Ability.java
@@ -1,6 +1,7 @@
 package me.unurled.sacredrealms.sr.components.item.abilities;
 
 /** Represents an ability that an item can have. TODO: Implement this class */
+@SuppressWarnings("unused")
 public class Ability {
 
   private final String name;
diff --git a/src/main/java/me/unurled/sacredrealms/sr/components/player/PlayerManager.java b/src/main/java/me/unurled/sacredrealms/sr/components/player/PlayerManager.java
index 7bdf286..cf9cc4e 100644
--- a/src/main/java/me/unurled/sacredrealms/sr/components/player/PlayerManager.java
+++ b/src/main/java/me/unurled/sacredrealms/sr/components/player/PlayerManager.java
@@ -40,11 +40,19 @@ public class PlayerManager extends Manager {
       return;
     }
     DataHandler dh = dm.getDataHandler();
-  }
+    // save all players
+    Gson gson =
+        new GsonBuilder()
+            .excludeFieldsWithoutExposeAnnotation()
+            .registerTypeAdapter(Inventory.class, new InventorySerializer())
+            .registerTypeAdapter(PotionEffect.class, new PotionEffectSerializer())
+            .create();
+    for (SRPlayer player : players.values()) {
+      dh.set(PLAYER_KEY + player.getUuid(), gson.toJson(player));
+    }
 
-  /** Load the data */
-  /*@Override
-  public void loadData() {}*/
+    players.clear();
+  }
 
   @Nullable
   public SRPlayer getPlayer(UUID uuid) {
@@ -89,7 +97,7 @@ public class PlayerManager extends Manager {
   }
 
   @EventHandler
-  public void onPlayerQuit(PlayerQuitEvent e) {
+  public void onPlayerQuit(@NotNull PlayerQuitEvent e) {
     SRPlayer player = getPlayer(e.getPlayer().getUniqueId());
     if (player == null) {
       return;
diff --git a/src/main/java/me/unurled/sacredrealms/sr/data/DataHandler.java b/src/main/java/me/unurled/sacredrealms/sr/data/DataHandler.java
index bb561de..a6ff112 100644
--- a/src/main/java/me/unurled/sacredrealms/sr/data/DataHandler.java
+++ b/src/main/java/me/unurled/sacredrealms/sr/data/DataHandler.java
@@ -1,6 +1,5 @@
 package me.unurled.sacredrealms.sr.data;
 
-import com.google.gson.Gson;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
@@ -14,14 +13,6 @@ public interface DataHandler {
    */
   String get(@NotNull String key);
 
-  /**
-   * Get a value from the data source
-   *
-   * @param key The key to get the value from
-   * @return The value of the key
-   */
-  Object getGson(@NotNull String key);
-
   /**
    * Set a value in the data source
    *
@@ -30,23 +21,6 @@ public interface DataHandler {
    */
   void set(@NotNull String key, @Nullable String value);
 
-  /**
-   * Set a value in the data source
-   *
-   * @param key The key to set the value to
-   * @param value The value to set
-   */
-  void set(@NotNull String key, Gson value);
-
-  /**
-   * Append a value to the data source
-   *
-   * @param key The key to append the value to
-   * @param path The path to append the value to
-   * @param json The json to append
-   */
-  void jsonAppend(@NotNull String key, @NotNull String path, @NotNull Object... json);
-
   /**
    * Remove a value from the data source
    *
@@ -56,6 +30,7 @@ public interface DataHandler {
 
   /**
    * Check if a key exists in the data source
+   *
    * @param key The key to check
    * @return If the key exists
    */
diff --git a/src/main/java/me/unurled/sacredrealms/sr/data/Redis.java b/src/main/java/me/unurled/sacredrealms/sr/data/Redis.java
index e902917..fbb6eb0 100644
--- a/src/main/java/me/unurled/sacredrealms/sr/data/Redis.java
+++ b/src/main/java/me/unurled/sacredrealms/sr/data/Redis.java
@@ -2,11 +2,9 @@ package me.unurled.sacredrealms.sr.data;
 
 import static me.unurled.sacredrealms.sr.utils.Logger.error;
 
-import com.google.gson.Gson;
 import me.unurled.sacredrealms.sr.SR;
 import org.jetbrains.annotations.NotNull;
 import redis.clients.jedis.JedisPooled;
-import redis.clients.jedis.json.Path2;
 
 public class Redis implements DataHandler {
 
@@ -41,17 +39,6 @@ public class Redis implements DataHandler {
     return client.get(key);
   }
 
-  /**
-   * Get a value from the data source
-   *
-   * @param key The key to get the value from
-   * @return The value of the key
-   */
-  @Override
-  public Object getGson(@NotNull String key) {
-    return client.jsonGet(key);
-  }
-
   /**
    * Set a value in the data source
    *
@@ -63,28 +50,6 @@ public class Redis implements DataHandler {
     client.set(key, value);
   }
 
-  /**
-   * Set a value in the data source
-   *
-   * @param key The key to set the value to
-   * @param value The value to set
-   */
-  public void set(@NotNull String key, Gson value) {
-    client.jsonSet(key, value);
-  }
-
-  /**
-   * Append a value to the data source
-   *
-   * @param key The key to append the value to
-   * @param path The path to append the value to
-   * @param json The json to append
-   */
-  @Override
-  public void jsonAppend(@NotNull String key, @NotNull String path, @NotNull Object... json) {
-    client.jsonArrAppend(key, Path2.of(path), json);
-  }
-
   /**
    * Remove a value from the data source
    *
diff --git a/src/main/java/me/unurled/sacredrealms/sr/utils/Items.java b/src/main/java/me/unurled/sacredrealms/sr/utils/Items.java
index 1771d36..90d6426 100644
--- a/src/main/java/me/unurled/sacredrealms/sr/utils/Items.java
+++ b/src/main/java/me/unurled/sacredrealms/sr/utils/Items.java
@@ -17,6 +17,7 @@ import org.bukkit.persistence.PersistentDataContainer;
 import org.bukkit.persistence.PersistentDataType;
 import org.jetbrains.annotations.NotNull;
 
+@SuppressWarnings("UnusedReturnValue")
 public class Items {
   public static @NotNull Double getTotalAttribute(Player player, Attribute attribute) {
     return getTotalAttribute(player.getEquipment(), attribute);
@@ -63,7 +64,6 @@ public class Items {
    * <dark_purple>Enchantments: <blue>➤ Enchantment <if max level gold>1 <blue>➤ Enchantment <if max
    * level gold>2
    *
-   * @param item the item
    * @param stack the itemstack
    * @return the itemstack with lore
    */