Cleanup chat history before sending it to LLM in V2 agent

changed milestone to %17.4

added Category:Duo Chat backend bugfunctional devopsai-powered featureaddition groupduo chat sectiondata-science typebug labels

assigned to @brytannia

requested review from @lesley-r

removed featureaddition label

added pipelinetier-1 label

@lesley-r could you please review this MR? it's a port of the functionality introduced by you. I wasn't able to just copy past your code as in this module messages are AiMessage objects and not hash, so I needed to adjust the approach. Let me know what you think!

1 Warning

The master pipeline status page reported failures in

If these jobs fail in your merge request with the same errors, then they are not caused by your changes.
Please check for any on-going incidents in the incident issue tracker or in the #master-broken Slack channel.

1 Message

CHANGELOG missing:

If this merge request needs a changelog entry, add the Changelog trailer to the commit message you want to add to the changelog.

If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.

Reviewer roulette

Category	Reviewer	Maintainer
backend	`@bucoleary` (UTC-4, 6 hours behind author)	`@vshushlin` (UTC+2, same timezone as author)

Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.

If needed, you can retry the danger-review job that generated this comment.

Generated by Danger

mentioned in epic gitlab-org#13533 (closed)

approved this merge request

@brytannia Looks good to me!

@seanarnold I was wondering if you can do the backend maintainer review. Thank you!

requested review from @seanarnold

removed review request for @seanarnold

added pipeline:mr-approved label

added pipelinetier-2 label and removed pipelinetier-1 label

Before you set this MR to auto-merge

This merge request will progress on pipeline tiers until it reaches the last tier: pipelinetier-3. We will trigger a new pipeline for each transition to a higher tier.

Before you set this MR to auto-merge, please check the following:

You are the last maintainer of this merge request
The latest pipeline for this merge request is pipelinetier-3 (You can find which tier it is in the pipeline name)
This pipeline is recent enough (created in the last 8 hours)

If all the criteria above apply, please set auto-merge for this merge request.

See pipeline tiers and merging a merge request for more details.

requested review from @seanarnold

removed review request for @lesley-r

E2E Test Result Summary

allure-report-publisher generated test report!

e2e-test-on-gdk: test report for 426ed285

expand test summary

+------------------------------------------------------------------+
|                          suites summary                          |
+-------------+--------+--------+---------+-------+-------+--------+
|             | passed | failed | skipped | flaky | total | result |
+-------------+--------+--------+---------+-------+-------+--------+
| Verify      | 44     | 0      | 2       | 0     | 46    | ✅     |
| Plan        | 74     | 0      | 1       | 0     | 75    | ✅     |
| Data Stores | 31     | 0      | 1       | 0     | 32    | ✅     |
| Create      | 128    | 0      | 16      | 0     | 144   | ✅     |
| Package     | 20     | 0      | 12      | 0     | 32    | ✅     |
| Govern      | 72     | 0      | 0       | 0     | 72    | ✅     |
| Secure      | 4      | 0      | 0       | 0     | 4     | ✅     |
| Fulfillment | 2      | 0      | 0       | 0     | 2     | ✅     |
| Release     | 5      | 0      | 0       | 0     | 5     | ✅     |
| Monitor     | 8      | 0      | 0       | 0     | 8     | ✅     |
| Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
| Manage      | 1      | 0      | 1       | 0     | 2     | ✅     |
+-------------+--------+--------+---------+-------+-------+--------+
| Total       | 391    | 0      | 33      | 0     | 424   | ✅     |
+-------------+--------+--------+---------+-------+-------+--------+