Skip to content

feat(event-normalization): Add gen_ai.input.messages and gen_ai.output.messages as distinct fields for SpanData#5797

Merged
constantinius merged 5 commits intomasterfrom
constantinius/feat/event-schema/gen-ai-input-output-messages
Apr 8, 2026
Merged

feat(event-normalization): Add gen_ai.input.messages and gen_ai.output.messages as distinct fields for SpanData#5797
constantinius merged 5 commits intomasterfrom
constantinius/feat/event-schema/gen-ai-input-output-messages

Conversation

@constantinius
Copy link
Copy Markdown
Contributor

…tput.messages` as distinct fields for SpanData
@constantinius constantinius requested a review from a team as a code owner April 8, 2026 14:09
@linear-code
Copy link
Copy Markdown

linear-code bot commented Apr 8, 2026

@constantinius constantinius requested a review from a team April 8, 2026 14:10
@constantinius constantinius changed the title feat(event-normalization): add gen_ai.input.messages and gen_ai.output.messages as distinct fields for SpanData feat(event-normalization): Add gen_ai.input.messages and gen_ai.output.messages as distinct fields for SpanData Apr 8, 2026
Comment on lines +703 to +709
/// The input messages to the model call.
#[metastructure(field = "gen_ai.input.messages", pii = "maybe")]
pub gen_ai_input_messages: Annotated<Value>,

/// The output messages from the model call.
#[metastructure(field = "gen_ai.output.messages", pii = "maybe")]
pub gen_ai_output_messages: Annotated<Value>,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You'll also want to add getters to impl Getter for SpanData below.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

JFYI it's similar to what was done in this PR: #5788

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, thanks!

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 28ec890. Configure here.

@constantinius constantinius enabled auto-merge April 8, 2026 14:37
@constantinius constantinius added this pull request to the merge queue Apr 8, 2026
Merged via the queue into master with commit 9dcd339 Apr 8, 2026
29 of 30 checks passed
@constantinius constantinius deleted the constantinius/feat/event-schema/gen-ai-input-output-messages branch April 8, 2026 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants