fix(api): Capture images on error even when exception is interrupted

[CaseyBatten]

Overview

Covers RQA-4842

Ensures we execute error image capture even if the original exception that prompted image capture gets interrupted by another exception.

Test Plan and Hands on Testing

• On an OT-2 trigger a smoothie error via collision and ensure an image is still captured.
• Test with Flex Estop
• Test with protocol cancelation

Changelog

Changed called for image capture via error to a finally case.

Review requests

Should this cover us, or should this be relocated away from the CommandExecutor entirely incase external errors interrupt us? Would we even want a photo from one of those?

Risk assessment

Low - reinforces new behavior.

[sfoster1]

This should be fine; do me a favor and test it with a flex estop and some cancels, those are the only ones i worry about