Skip to content

AgentChat messages should have an id field. #6317

@ekzhu

Description

@ekzhu

Confirmation

  • I confirm that I am a maintainer and so can use this template. If I am not, I understand this issue will be closed and I will be asked to use a different template.

Issue body

Message id is useful to correlate ModelClientStreamingChunkEvent with the eventual completed message being produced.

This should be added to BaseChatMessage and BaseAgentEvent as an id: str field. For ModelClientStreamingChunkEvent, it should have a separate optional field full_message_id: str | None = None to reference the complete message that may come after the chunks. So the consumer of the stream don't need to double-print it.

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions