One thing I didn’t ask about…What theme are you modifying. Not the most likely concern but it might matter. And the specifics on the product (one time charge I assume).
Also, the text you have that says “Do Not Change” on the first name is recursive which means it’s getting added repeatedly to the label (see image)
This usually means that it needs to be placed inside an ajaxLoad trigger.
Why is part of the code in a CDATA and the other not? It really shouldn’t need it regardless but I’m guessing there’s a reason just that part of code was wrapped in it?