Rework of SideOnly inspections
MinecraftDev's SideOnly inspections gave an error when you tried to access client-only code from common code, helping to prevent a common cause of crashes on the dedicated server.
While they were very useful in the right circumstances, the old SideOnly inspections have a number of flaws which ultimately meant they didn't see the widespread use that was originally hoped for. The problems I'm about to outline led to them being disabled by default, so not everyone even knew they existed. While I was maintaining a fork of MinecraftDev with Fabric support (now merged upstream), I randomly came across them when scrolling through my inspections list, hacked together a fix so it would work with Fabric's
@Environment annotation, and enabled them by default in my fork. This was a partial success, but still shared some of the issues as before.
So, what are these issues? They can be broken down into 5 key problems with the old inspections:
- Litter. In order to use