> "Reviewer barely scratches the surface, and misses huge issues"
Code reviewers sometimes miss huge issues. Automated testing, and load testing are designed to catch these issues, and even then sometimes things go live and turn into the flaming shit show that we have all lived through.
Review is only part of a full lifecycle.
- Formal usability: This is the big reset. A day or more of sitting behind a one way mirror with an impartal moderator taking users through your app. Your going to find that 7 people will magically bitch about the same thing. The thing you didn't notice, or thought was a bad idea.
- Informal usability: if your doing something risky, or questionable, don't be afraid to bring a few users into your office and run testing yourself. It isn't as clean as formal, but your going to get a lot of the same insights at a much lower cost.
- Testing and behavior tools: find some and integrate them into your site. User testing.com, Crazyegg and good old google analytics funnels are you friends. These are the "automated tests" of UI design, and you need to take time to do good analysis of the results.
You have to build up the tools an infrastructure around reviews, have the hard conversations, where legitimate critiques are taken seriously. It isn't just one process your putting in place its a whole culture change.