[FF cleanup] remove resolve_vulnerability_send_finding_description flag

What does this MR do and why?

Context

We had stopped sending the description of a vulnerability finding when this feature was first launched, in order to save on tokens.

With newer models, this token constraint is no longer a worry, so we want to start sending the finding description in the LLM prompt again.

This change

We added a derisk FF just to be really really sure the description wouldn't result in LLM errors.

The flag has been globally enabled, and we see no error rate difference in the dashboard.

MR acceptance checklist

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


closes: #537615 (closed)
EE: true
Changelog: removed

Edited by Michael Becker

Merge request reports

Loading