You're totally right about ResetDurationStats() missing if you go via the keyholderIntro. The intent was to not clear the stats on release but on lock/relock. That way stats could potentially be viewed after the fact if need be. I'll add ResetDurationStats() to powerExchangeNegotiateIntro as I think it makes more sense for it to be there.
As for the _on_lookconfirm_pressed() thing and the missing tab, it does indeed look out of place and would result in that text being set at any stage when lookconfirm is pressed, as opposed to only when it's pressed on stage 7. Unsure if this makes any difference either way, will do some testing here.