Agreed on the phone length, but lets check to make sure length is at least greater than 0.

We do have checks for this everywhere in the code -- and that's good, but I would still like to have this safety net just in case there is one code path we have overlooked where things like "" for first name or primary phone might happen.
I don't have a strong opinion on this one, so I'm fine with whatever you decide next.