It's a bad behavior, it causes off-hand items to be accidentally activated. If you can't tell if using an item will do something on clientside, use EnumActionResult.SUCCESS instead. See ItemMonsterPlacer for example

Makes sense i will check this out