When I have something like a player or a pellet that a pacman is supposed to eat or a weapon pickup, do I add networkIdentities to those? Does it hurt to do so if it makes my code cleaner? I'm just curious about what the best practices are

The Unity documentation for NetworkIdentity says that every object you want to talk about on the network, should have an identity. So it sounds like you should.