Skip to content

docs: update useChat reference to match actual behavior#7588

Merged
lgrammel merged 1 commit intovercel:mainfrom
bytaesu:docs/use-chat-api-reference
Jul 28, 2025
Merged

docs: update useChat reference to match actual behavior#7588
lgrammel merged 1 commit intovercel:mainfrom
bytaesu:docs/use-chat-api-reference

Conversation

@bytaesu
Copy link
Contributor

@bytaesu bytaesu commented Jul 27, 2025

Background

Important reference for useChat in v5 is missing.
Without the prepareReconnectToStreamRequest option,
resumeStream doesn't work when it doesn't match the default setups.

Summary

Fixed prepareSendMessagesRequest reference and added prepareReconnectToStreamRequest reference.
No changes other than documentation.

Tasks

  • Tests have been added / updated (for bug fixes / features)
  • Documentation has been added / updated (for bug fixes / features)
  • A patch changeset for relevant packages has been added (for bug fixes / features - run pnpm changeset in the project root)
  • Formatting issues have been fixed (run pnpm prettier-fix in the project root)

Future Work

I noticed that many parts of the actual behavior didn’t match the documentation.
Let me know specifically which parts need documentation updates, and I’ll be happy to help.
Feel free to leave a comment if needed.

@lgrammel lgrammel changed the title docs: update useChat reference to match actual behavior (v5) docs: update useChat reference to match actual behavior Jul 28, 2025
@lgrammel lgrammel merged commit dffbb9f into vercel:main Jul 28, 2025
8 of 9 checks passed
@lgrammel
Copy link
Collaborator

any prs with docs fixes are much appreciated!

Comment on lines +114 to +125
}
{
name: 'body',
type: 'Record<string, any> | undefined',
description: 'The request body',
}
{
name: 'credentials',
type: 'RequestCredentials | undefined',
description: 'The request credentials',
}
{
Copy link
Collaborator

@nicoalbanese nicoalbanese Jul 28, 2025

Choose a reason for hiding this comment

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

@bytaesu for future reference, please remember to add , between properties here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Got it! I ran Prettier but missed this. Would’ve been easier if I had access to the docs app😅

nicoalbanese added a commit that referenced this pull request Jul 28, 2025
@bytaesu
Copy link
Contributor Author

bytaesu commented Jul 28, 2025

any prs with docs fixes are much appreciated!

Alright, I’ll fix anything I spot.

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