Skip to content

Reject empty content from chat history

Shinya Maeda requested to merge reject-empty-content-from-chat-history into master

This is a high priority MR for Switch to Chat Agent V2 (gitlab-org#13533). 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.

Edited by Shinya Maeda

Merge request reports