Refactor Exceptions: remove ScreenshotException. add OperationFailedExceptionWithScreenshot.

[jw098]

Unfortunately, this is a massive PR.

I think the main things to review are all the files within Exception/. And the ProgramSessions, where the exceptions are caught.

[Mysticial]

If this pattern appears 35 times, then that's a lot of places to copy-paste inline it. Can you add back a helper function somewhere that does this?

[jw098]

So, basically, you want to go from this:

send_program_fatal_error_notification(env, m_option.instance().NOTIFICATION_ERROR_FATAL, e.message(), *e.screenshot());
if (e.error_report_mode() == ErrorReport::SEND_ERROR_REPORT){
    PokemonAutomation::report_error(
        &env.logger(),
        env.program_info(),
        "Fatal: OperationFailedExceptionWithScreenshot",
        {{"Message:", e.message()}},
        *e.screenshot(),
        &e.video_stream()->history()
    );
}

to this?

  send_program_fatal_error_notification_and_telemetry_report(
      env, &env.logger(), env.program_info(), 
      m_option.instance().NOTIFICATION_ERROR_FATAL, 
      e.error_report_mode(),
      e.message(),
      "OperationFailedExceptionWithScreenshot",
      *e.screenshot(),
      &e.video_stream()->history()
  );

[Mysticial]

No I mean back to a one-line function call like what it was before. I don't necessarily mean revert it, but make a helper function that does that.

Basically copying the same 10 lines 35 times when it's a clean 1 line each is not great.