Very nice. On the home brew jobs I think 1n4001s are going to be better than 1n4148s or other small signal diodes because they're horrific imd generators in certain instances such as nearby ambc stas, and they might not live after a pop or two from a tstorm. The 1n4001s are similar to a PIN diode in their junction, thus almost impervious to generating imd, and they're rated to handle much greater power levels than small sig diodes.
In some of my hf rigs that don't feature built in gdt protection, I place a gdt across the uhf socket(s) and a 20v mov across the dc input.
I suppose one could get away with a simple ne2 bulb across a decent inductance to ground, mounted inside a metal box and call it good. Pretty much what the polyphaser protector inline with the rig in front of me has.