Skip to content

feat (ai): restructure chat transports #6746

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

Merged
merged 39 commits into from
Jun 17, 2025
Merged

feat (ai): restructure chat transports #6746

merged 39 commits into from
Jun 17, 2025

Conversation

lgrammel
Copy link
Collaborator

@lgrammel lgrammel commented Jun 14, 2025

Background

Chat resumption on reload needs a separate endpoint to connect to.

Summary

  • restructure chat transports
  • move default route for stream reconnection to /chat/[id]/stream
  • rename prepareRequest to `prepareSubmitMessagesRequest``

Verification

  • test next example

@lgrammel lgrammel changed the title Lg/g zm vk3 bo feat (ai): restructure chat transports Jun 17, 2025
@lgrammel lgrammel marked this pull request as ready for review June 17, 2025 07:51
@lgrammel lgrammel merged commit 60e2c56 into v5 Jun 17, 2025
8 checks passed
@lgrammel lgrammel deleted the lg/gZmVk3BO branch June 17, 2025 07:56
lgrammel pushed a commit that referenced this pull request Jun 18, 2025
## Background

Fixes a bug where `fetch` was not used and it always called the global
`fetch`. This was introduced in #6746

## Summary

Call `this.fetch()` instead of `fetch()`

## Verification

I verified in a patched package locally.
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.

2 participants