CobblemonOptimizer

Ghost Pokemon Fix

We identified three causes of ghost (phantom) Pokemon entities:

1. Redundant entity.discard() in AdaptiveSpawnManager — When blocking a spawn, the code called both event.cancel() and entity.discard(). The cancel alone is sufficient; the extra discard created a race condition where the entity tracker could send spawn packets to clients before the entity was removed server-side, leaving a visible but non-interactive ghost on the client.
2. Missing velocity sync in MobEntityAiMixin — When AIFreeze zeroed a Pokemon's horizontal velocity, it never flagged the entity for network sync. The server had the Pokemon stopped, but the client kept rendering it with the old velocity, causing visual drift/desync. Fixed by setting velocityDirty = true (Fabric) / hasImpulse = true (NeoForge) after zeroing velocity, which forces the entity tracker to send the updated velocity to all clients.
3. Silent exception swallowing in GhostEntityHandler — The early delegate.tick() calls that prevent ghost entities were catching all exceptions silently. Now they log at DEBUG level with species name and position, making it possible to diagnose any remaining ghost issues.

ServerCore Compatibility

ServerCore and CobbleOptimizer both inject into the same method (tickNewAi / customServerAiStep) to skip entity AI ticks. When both mods are loaded, they independently cancel the AI tick — causing Pokemon to never get their AI processed, leading to unresponsive ghost-like entities.

Solution: Created a ServerCoreCompat module that:

• Auto-detects ServerCore at startup via the mod loader API
• When detected, suppresses TickThrottle and AIFreeze (the two features that conflict with ServerCore's entity activation ranges)
• Keeps AdaptiveSpawn, BatchDespawn, and Emergency active (these use Cobblemon events and periodic sweeps, not AI tick injection)
• Logs a clear warning at startup explaining what was disabled and why
• Adds a config option compat.auto_disable_with_servercore (default true) so users can override the behavior if needed, reloadable at runtime