1.9.4 is the final version of Minecraft for 1.9, it also does contain bug fixes, and it should be downwards compatible if it doesn't mess with changes made mainly bugfixes.
You are only killing it client side though because world.isRemote you should be using !world.isRemote.
You are killing the entity anyways it is at the end of the method and always gets called. Try adding return; at the end of the if entityHit != null if statement.
That is what is supposed to happen, you need to override onDescriptionPacket and getDescriptionPacket (not sure if i got those names right) in your TE's class.
*Edit that is for syncing client and server.