This feature had never worked at all because the code incorrectly used <commit2> for both the "old" and "new", so the diff was always empty! Now we correctly use <commit1> where we should and it works fine.
Actually the old code might have been working fine. Please hold off on reviewing.
Yeah, I was incorrect. The feature worked fine. I don't know what I was thinking.