Looks good to me.
It may just be how its drawn but there's a few wires which are permanent electrical connections to ground I would move around. The green neck lead and resistor leg connected together at the switch and the tone pot ground wire...
I would solder those three to the back of the volume pot instead of jumpering them to ground off other lugs. It reduces possible failure points. The killswitch in particular - those get hit hard and often - if it failed mid-show with the tone pot grounded through it, the tone could also stop working.
PS - it took me a moment to realize its the DiMarzio convention of Neck is 1 and Bridge is 5. This makes much more sense to me because like people have asked with pedals we read left to right so... Why the opposite Bridge is 1 and Neck is 5 became more common I don't know...