Hmm, this still doesn't handle this case properly:

const std = @import("std");
test {
    return error.SkipZigTest;
}
fn foo() !void {
    return error.DontShow;
}
test {
    foo() catch {
        foo() catch {};
        std.debug.dumpStackTrace(@errorReturnTrace().?.*);
    };
}

Maybe it'd be better to start off with a simplified version which always resets to zero since it handles all the common cases properly.

Hmm, this still doesn't handle this case properly:

Not sure if this is good news or bad, but I'm just finishing up an overlapping change that should handle that case correctly. It intends to implement the rest of #1923 (comment) (esp. supporting break)

Let me know how we can best coordinate the follow-up change so that I don't step on your toes here 🙂

If you have a more complete fix then let's go with that, I just want this annoyance fixed to get the full benefit of #12807

Closing in favor of the more complete solution.