As far as data fixers are concerned, I think they are fine.
entity, etc packages, the prefix is justifiedhmm maybe. Not sure I agree though. I'd suggest make an entity package then.
I don't agree that the prefix is justified and am siding with @ProfessorProspector on this.
PlayerServerEntity -> ServerPlayerEntity
Actually, I agree the prefix is justified. Looking into it myself now.
Possibly also
->BlockEntityFactory`?A few more:
PlacementContextItem -> ItemPlacementContextnet.minecraft.server.command.OutputCommand -> CommandOutputnet.minecraft.server.command.SourceCommand -> CommandSourceEntityProviderBlock -> BlockEntityProvider. Maybe BlockEntityProviderBlock, but probably not?馃挴
Most helpful comment
A few more:
PlacementContextItem->ItemPlacementContextnet.minecraft.server.command.OutputCommand->CommandOutputnet.minecraft.server.command.SourceCommand->CommandSourceEntityProviderBlock->BlockEntityProvider. MaybeBlockEntityProviderBlock, but probably not?馃挴