Refactor user validation and enhance debugging in PlayerManager

- Removed commented dependency from pom.xml.
- Improved debug logging in PlayerManager for user validation checks.
- Added detailed debug statements to track user existence and Bedrock
player checks.
- Updated BedrockNameResolver documentation for clarity.

Author: BenCodez

AdvancedCore/pom.xml

@@ -207,9 +207,6 @@
             <version>2.12.2</version>
             <scope>provided</scope>
         </dependency>
-        <!-- <dependency> <groupId>com.github.Ben12345rocks</groupId>
-		<artifactId>ChatComponentAPI</artifactId> 
-			<version>master-SNAPSHOT</version> <scope>compile</scope> </dependency> -->
         <dependency>
             <groupId>org.slf4j</groupId>
             <artifactId>slf4j-simple</artifactId>

AdvancedCore/src/main/java/com/bencodez/advancedcore/api/bedrock/BedrockNameResolver.java

@@ -301,6 +301,7 @@ public Result resolve(String incomingName) {
 		// implementation)
 		try {
 			AdvancedCoreUser user = userManager.getUser(incomingName);
+
 			if (user == null) {
 				String canonical = ciIndex.get(incomingName.toLowerCase(Locale.ROOT));
 				if (canonical != null)
@@ -486,7 +487,8 @@ private Player findOnlineByNameOrStripped(String name) {
 	}
 
 	/**
-	 * Result of name resolution containing the final name, Bedrock status, and rationale.
+	 * Result of name resolution containing the final name, Bedrock status, and
+	 * rationale.
 	 */
 	public static final class Result {
 		/** The final resolved name. */

AdvancedCore/src/main/java/com/bencodez/advancedcore/api/misc/PlayerManager.java

@@ -237,33 +237,66 @@ public boolean isValidUser(String name) {
 
 	@SuppressWarnings("deprecation")
 	public boolean isValidUser(String name, boolean checkServer) {
+		plugin.extraDebug("isValidUser START: name=" + name + ", checkServer=" + checkServer);
+
+		if (name == null) {
+			plugin.extraDebug("isValidUser: name is null -> false");
+			return false;
+		}
+
+		name = name.trim();
+
 		Player player = Bukkit.getPlayerExact(name);
 		if (player != null) {
+			plugin.extraDebug("isValidUser: matched ONLINE player -> true (" + player.getName() + ")");
 			return true;
 		}
 
 		boolean userExist = plugin.getUserManager().userExist(name);
+		plugin.extraDebug("isValidUser: userExist(" + name + ")=" + userExist);
 		if (userExist) {
-			return userExist;
+			plugin.extraDebug("isValidUser: returning true from userExist");
+			return true;
 		}
 
 		if (name.isEmpty()) {
+			plugin.extraDebug("isValidUser: empty name -> false");
 			return false;
 		}
 
-		if (plugin.getBedrockHandle().isBedrock(name)) {
-			plugin.extraDebug("Player " + name + " is a bedrock player, skipping offline check");
+		boolean isBedrock = plugin.getBedrockHandle().isBedrock(name);
+		plugin.extraDebug("isValidUser: isBedrock(" + name + ")=" + isBedrock);
+		if (isBedrock) {
+			plugin.extraDebug("isValidUser: bedrock match -> true (skipping offline check)");
 			return true;
 		}
 
-		if (checkServer && !name.startsWith(plugin.getOptions().getBedrockPlayerPrefix())) {
-			OfflinePlayer p = Bukkit.getOfflinePlayer(name);
-			if (p.hasPlayedBefore() || p.isOnline() || p.getLastPlayed() != 0) {
-				return true;
+		if (checkServer) {
+			String prefix = plugin.getOptions().getBedrockPlayerPrefix();
+			plugin.extraDebug("isValidUser: checkServer enabled, bedrockPrefix=" + prefix);
+
+			if (!name.startsWith(prefix)) {
+				OfflinePlayer p = Bukkit.getOfflinePlayer(name);
+
+				boolean hasPlayed = p.hasPlayedBefore();
+				boolean isOnline = p.isOnline();
+				long lastPlayed = p.getLastPlayed();
+
+				plugin.extraDebug("isValidUser: OfflinePlayer check for " + name + " -> hasPlayedBefore=" + hasPlayed
+						+ ", isOnline=" + isOnline + ", lastPlayed=" + lastPlayed);
+
+				if (hasPlayed || isOnline || lastPlayed != 0) {
+					plugin.extraDebug("isValidUser: checkServer match -> true");
+					return true;
+				}
+			} else {
+				plugin.extraDebug("isValidUser: skipping checkServer due to bedrock prefix match");
 			}
+		} else {
+			plugin.extraDebug("isValidUser: checkServer disabled");
 		}
 
-		plugin.extraDebug("Player " + name + " does not exist");
+		plugin.extraDebug("isValidUser: FINAL -> false (no checks passed for " + name + ")");
 		return false;
 	}
 }

AdvancedCore/src/main/java/com/bencodez/advancedcore/api/user/usercache/UserDataManager.java

@@ -17,6 +17,7 @@
 import com.bencodez.advancedcore.api.user.usercache.keys.UserDataKeyBoolean;
 import com.bencodez.advancedcore.api.user.usercache.keys.UserDataKeyInt;
 import com.bencodez.advancedcore.api.user.usercache.keys.UserDataKeyString;
+import com.bencodez.simpleapi.debug.DebugLevel;
 
 import lombok.Getter;
 
@@ -73,6 +74,13 @@ public void addKey(UserDataKey userDataKey) {
 	@Deprecated
 	public void cacheUser(UUID uuid) {
 		plugin.devDebug("Caching " + uuid.toString());
+		if (plugin.getOptions().getDebug().isDebug(DebugLevel.DEV)) {
+			try {
+				throw new Exception("caching here: " + uuid.toString());
+			} catch (Exception e) {
+				e.printStackTrace();
+			}
+		}
 		if (userDataCache.containsKey(uuid)) {
 			UserDataCache data = userDataCache.get(uuid);
 			data.clearChanges();
@@ -83,6 +91,7 @@ public void cacheUser(UUID uuid) {
 				userDataCache.put(uuid, data);
 			}
 		}
+
 	}
 
 	public void cacheUser(UUID uuid, String playerName) {