getsentry / getsentry/sentry-python

Ensure that, in multi-turn conversations, we include the full conversation history, not just the last message

Open
#5,976 1 comment 0 reactions 0 assignees View on GitHub
Improvement Python Refactoring
Dominant language
Python
Stars
2.2k
Forks
669
Avg merge
1d 40m
Merged PRs (30d)
212

Description

Currently, we "pop" the last message in a conversation and only add that as the `gen_ai.request.messages` value in order to manage the size of the span (we don't want to accidentally exceed it).

The ability to include the full conversation history is blocked by the "span first" feature. Once it's released, we need to revisit this to ensure that we're retaining the entire conversation history passed in `contents` in `generate_content` calls on the `gen_ai.request.messages` attribute on spans.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.