A patriot could also have decide d to sabotage the war effort to end the war faster, to get rid of Hitler or to somewhat save the reputation of Germany.
He didn't openly defect but that doesn't mean much for a spy.
Yes, I agree it is quite a feat of mental acrobatics. My impression was that he somehow seperated Nazis and the German nation. And that the war was a German and not really a Nazi thing. Maybe he just didn't want to see that Germany and the Nazis were the same thing at the time, maybe he also wanted a round two after WW1 or maybe he wasn't able to shake decades of upbringing and training.
Either way, he was one of the few "good" Germans, even if not on Schindler levels, and definitely a very interesting person. Just look up his WW1 adventures.
Notable, so, is that even in WW1 and after he was not necessarily a trained spy intel guy, AFAIK.
Considering wars of aggression acceptable wasn't all that unusual either.
Sabotaging the war effort would have meant helping the Allies fight Germany. Sabotaging war crimes and the Holocaust meant trying to stop Germany from something evil and stupid (at least if he considered German Jews German). While there were reasons for a patriot to sabotage the war effort, only sabotaging the crimes was also a consistent position.
[1] Especially when it isn't democratically elected. The last multi-party elections in 1933 weren't free. The communist Reichstag members were jailed, many others were intimidated to make them support the enabling act.