Reject empty content from chat history
This is a high priority MR for Switch to Chat Agent V2 (gitlab-org#13533 - closed). Please prioritize the review and merge.
What does this MR do and why?
This MR rejects empty content from chat history (e.g. ""
). I suspect that this is causing all messages must have non-empty content except for the optional final assistant message
exception.
Closes Exclude a pair of user and assistant role messa... (#479460 - closed) Seeing A1002 Duo Chat error message (#494004 - closed) and Handle Anthropic invalid_request_error in Duo Chat (#493681 - closed)
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.