Skip to content

Fix AI review to not include meta lines as part of diff hunks

Patrick Bajao requested to merge 463753-ai-review-not-published into master

What does this MR do and why?

We previously include diff lines that are meta lines (lines with type as new-nonewline, old-nonewline, etc) as part of diff hunks we remove. It results to our worker creating a draft note on that line which doesn't work (we don't allow commenting on those lines even in the UI).

To fix this, we exclude them from the diff hunk we review so the draft note is created at the correct line.

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.

Related to #463753 (closed)

Merge request reports