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

Update StreamUI to include finishReason, usage, and any other relevant data #1845

Closed
Godrules500 opened this issue Jun 5, 2024 · 2 comments · Fixed by #1920
Closed

Update StreamUI to include finishReason, usage, and any other relevant data #1845

Godrules500 opened this issue Jun 5, 2024 · 2 comments · Fixed by #1920
Labels
ai/rsc enhancement New feature or request

Comments

@Godrules500
Copy link

Feature Description

Currently streamUI does not include finishReason and usage. I'm requesting that it is updated to include those since I am having to revert to streamText to do as I need.

Use Case

  1. With google vertex, if it fails for safety violation, there is no way for me to handle it.
  2. When the max token limit is reached, I am unable to detect and tell the user that they need to "continue response"

Additional context

No response

@lgrammel lgrammel added ai/rsc enhancement New feature or request labels Jun 5, 2024
@Godrules500
Copy link
Author

#1904 Tagging this here as it would solve this.

@gclark-eightfold
Copy link
Contributor

gclark-eightfold commented Jun 11, 2024

I'll see about pulling-out the onFinish callback into a standalone PR since the other changes in that PR may take a bit longer to review & document.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ai/rsc enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants