Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add support for streaming responses in conversation routes #379

Merged
merged 23 commits into from
Nov 5, 2024

Conversation

atierian
Copy link
Member

@atierian atierian commented Oct 30, 2024

Issue #, if available:
N/A

Description of changes:
Adds support for response streams in conversation routes.

  • Removes conversation.onMessage()
  • Adds conversation.onStreamEvent()
  • Adds ConversationStreamEvent type.

Related PRs

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Copy link

changeset-bot bot commented Oct 30, 2024

🦋 Changeset detected

Latest commit: 36a7c35

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@aws-amplify/data-schema Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Comment on lines +233 to +244
const ConversationMessageStreamEvent = `type ConversationMessageStreamPart @aws_cognito_user_pools {
id: ID!
owner: String
conversationId: ID!
associatedUserMessageId: ID!
contentBlockIndex: Int!
contentBlockText: String
contentBlockDeltaIndex: Int
contentBlockToolUse: ToolUseBlock
contentBlockDoneAtIndex: Int
stopReason: String
}`;
Copy link
Member Author

Choose a reason for hiding this comment

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

I have a WIP change that will consolidate all of these GraphQL supporting types for conversation routes in the conversation transformer, allowing us to remove these completely from data-schema. That change involves a few moving parts though, so keeping it separate.

@atierian atierian marked this pull request as ready for review October 31, 2024 16:53
@atierian atierian requested review from a team as code owners October 31, 2024 16:53
@atierian atierian marked this pull request as draft October 31, 2024 20:14
@atierian
Copy link
Member Author

Converting this to draft while waiting on the related construct and codegen changes to be merged and released.

@atierian atierian changed the base branch from main to ai.staging-streaming-and-errors November 5, 2024 16:20
@atierian atierian marked this pull request as ready for review November 5, 2024 16:20
@atierian atierian merged commit 77296be into ai.staging-streaming-and-errors Nov 5, 2024
9 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.

3 participants