My design thoughts;
I say toss the 9v battery and use 2x AAs, and a boost converter, so many reasons to do this;
AAs are cheaper and more energy dense than 9v batteries
Rechargeable AAs are very reliable and chargers are common compared to 9vs
You can use 2x 1x AA battery holders, and put you components down the middle, so they act as protection
The cost difference between a 9v battery and the components for the boost converter are about the same - except the boost converter components will last forever.
Other things;
Use a slide switch, i mean, you don't need the massive current handling of the toggle switch, plus it will be harder to accidentally turn off and be less awkward to hold, also easier to manufacture and possibly cheaper.
No idea what op amp you're using but I hope it's one with decent current output, people might stick massive headphones on this thing, they will probably want them driven and if it's used as a "tool" potentially a small speaker would be wanted to be driven.
Check out this oneIf you used a 2+mm thick PCB, and used full SMD you could get away without a back cover, instead just using the PCB.
I know this is all a bit late now but eh.