Skip to content

Treat a missing fixed_code block as a false positive

Darby Frey requested to merge missing-fixed-code-handler into master

What does this MR do and why?

In the process of investing this error #497194, I found that there were cases where the LLM does not include the <fixed_code></fixed_code> section in the response. This case was being handled in a way that was causing a no _dump_data is defined for class Proc exception when attempting to cache the response.

The cases I found all looked to be false_positive responses from the LLM, so this change updates the behavior to treat a missing code block as a false positive, similar to how the empty code block cases is treated.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Merge request reports