Toss on a MMSZ5231B and you're fine.
I wouldn't actually recommend clamp diodes, because there's nowhere for the current to go -- likely you don't have much load on the 5V supply, so it will be pulled up, and cause problems. The solution to that, is a TVS across the supply. Which isn't a bad idea, and usually comes for free in TVS array chips, which is not a bad idea here.
You may also want to replace Q1 with a protected switch device. This is a MOSFET with an internal control circuit to handle level shifting (may not need Q2 and resistors), to protect against excessive voltage, current and temperature (but not all three at once, obviously(?)).
As-is, you still may want to protect Q1's gate with a zener, 5 to 15V rating is fine. The zener would be wired from gate to source, in parallel with R5. Q2 should then have a resistor added in series with its collector, maybe 1-10k, so it doesn't draw excessive current through the zener. This protects against excessive input voltage (automotive load dump can swell up to 40V+, well outside of Q1's Vgs(max) rating).
Tim