Always fall back to offline UUID when inspect lookup misses

- Resolve identifiers to a UUID in all cases by using OfflinePlayer fallback after Mojang miss
- Remove `onlineMode` branching from `selectResolvedUuid`
- Update tests to cover Mojang-miss fallback behavior and non-null resolution

Author: rossnoah

src/main/java/dev/noah/perplayerkit/commands/inspect/InspectCommandUtil.java

@@ -50,10 +50,11 @@ private InspectCommandUtil() {
     /**
      * Attempts to resolve a player identifier (name or UUID) to a UUID asynchronously.
      * This method first tries to parse as UUID, then checks online players synchronously,
-     * and finally searches cached offline players and Mojang asynchronously.
+     * and finally searches cached offline players and Mojang asynchronously before
+     * falling back to the deterministic offline-mode UUID.
      *
      * @param identifier Player name or UUID string
-     * @return CompletableFuture containing UUID if found, null otherwise
+     * @return CompletableFuture containing the resolved UUID
      */
     public static CompletableFuture<UUID> resolvePlayerIdentifierAsync(String identifier) {
         // First try to parse as UUID
@@ -73,7 +74,7 @@ public static CompletableFuture<UUID> resolvePlayerIdentifierAsync(String identi
         return CompletableFuture.supplyAsync(() -> {
             UUID cachedOfflinePlayer = findCachedOfflinePlayerUuid(identifier);
             return selectResolvedUuid(identifier, cachedOfflinePlayer,
-                    () -> lookupPlayerUuidFromMojang(identifier), Bukkit.getOnlineMode());
+                    () -> lookupPlayerUuidFromMojang(identifier));
         });
     }
 
@@ -159,21 +160,17 @@ public static void showUsage(@NotNull Player player, @NotNull String commandName
         }
     }
 
-    static @Nullable UUID selectResolvedUuid(@NotNull String identifier, @Nullable UUID cachedOfflinePlayer,
-                                             @NotNull Supplier<UUID> mojangLookup, boolean onlineMode) {
+    static @NotNull UUID selectResolvedUuid(@NotNull String identifier, @Nullable UUID cachedOfflinePlayer,
+                                            @NotNull Supplier<UUID> mojangLookup) {
         if (cachedOfflinePlayer != null) {
             return cachedOfflinePlayer;
         }
 
-        if (!onlineMode) {
-            return UUID.nameUUIDFromBytes(("OfflinePlayer:" + identifier).getBytes(StandardCharsets.UTF_8));
-        }
-
         UUID mojangUuid = mojangLookup.get();
         if (mojangUuid != null) {
             return mojangUuid;
         }
 
-        return null;
+        return UUID.nameUUIDFromBytes(("OfflinePlayer:" + identifier).getBytes(StandardCharsets.UTF_8));
     }
 }

src/test/java/dev/noah/perplayerkit/commands/inspect/InspectCommandUtilTest.java

@@ -24,7 +24,7 @@ void selectResolvedUuidPrefersCachedOfflinePlayer() {
         UUID resolvedUuid = InspectCommandUtil.selectResolvedUuid("TargetPlayer", cachedUuid,
                 () -> {
                     throw new AssertionError("Mojang lookup should not run when cached player exists");
-                }, false);
+                });
 
         assertEquals(cachedUuid, resolvedUuid);
     }
@@ -33,30 +33,21 @@ void selectResolvedUuidPrefersCachedOfflinePlayer() {
     void selectResolvedUuidUsesMojangResultWhenCacheMisses() {
         UUID mojangUuid = UUID.randomUUID();
 
-        UUID resolvedUuid = InspectCommandUtil.selectResolvedUuid("TargetPlayer", null, () -> mojangUuid, true);
+        UUID resolvedUuid = InspectCommandUtil.selectResolvedUuid("TargetPlayer", null, () -> mojangUuid);
 
         assertEquals(mojangUuid, resolvedUuid);
     }
 
     @Test
-    void selectResolvedUuidUsesOfflineFallbackWhenServerIsOfflineMode() {
+    void selectResolvedUuidUsesOfflineFallbackWhenMojangLookupMisses() {
         String identifier = "TargetPlayer";
 
-        UUID resolvedUuid = InspectCommandUtil.selectResolvedUuid(identifier, null, () -> {
-            throw new RuntimeException("mojang should not be consulted");
-        }, false);
+        UUID resolvedUuid = InspectCommandUtil.selectResolvedUuid(identifier, null, () -> null);
 
         assertEquals(UUID.nameUUIDFromBytes(("OfflinePlayer:" + identifier).getBytes(StandardCharsets.UTF_8)),
                 resolvedUuid);
     }
 
-    @Test
-    void selectResolvedUuidReturnsNullWhenServerIsOnlineModeAndNoMatchExists() {
-        UUID resolvedUuid = InspectCommandUtil.selectResolvedUuid("TargetPlayer", null, () -> null, true);
-
-        assertNull(resolvedUuid);
-    }
-
     @Test
     void findCachedOfflinePlayerUuidDoesNotScanOfflinePlayers() throws Exception {
         Method method = InspectCommandUtil.class.getDeclaredMethod("findCachedOfflinePlayerUuid", String.class);