Skip to content

Conversation

anuraaga
Copy link
Contributor

This should be the last feature for Converse API.

/cc @codefromthecrypt

@anuraaga anuraaga requested a review from a team as a code owner March 13, 2025 02:56
Copy link

@codefromthecrypt codefromthecrypt left a comment

Choose a reason for hiding this comment

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

looks good. this is important as without the tool support you added, you can't really understand agentic apps. Also, most I've seen set streaming enabled.

}

private void handleEvent(ConverseStreamOutput event) {
if (captureMessageContent && event instanceof MessageStartEvent) {

Choose a reason for hiding this comment

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

bet you wish you could do a type switch ;)

Comment on lines 467 to 468
new software.amazon.awssdk.protocols.json.internal.unmarshall
.document.DocumentUnmarshaller());
Copy link
Contributor

Choose a reason for hiding this comment

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

as far as I can tell don't need the fully qualified name as it is already imported

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops, thanks

@trask trask merged commit 2f80171 into open-telemetry:main Mar 18, 2025
86 checks passed
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