yak shave

the "|tee" was a no-op, the logic of the check was a bit mysterious, and I was worried it would be brittle in the face of changes to the error message. In scripts, program failure should be detected by exit status, not by ignoring the exit status and scrounging in their stderr. So I tried to simplify and standardize the logic.