Skip to content

feat: storage pubsub and resume - #25

Draft
simone-sanfratello wants to merge 36 commits into
mainfrom
feat/storage-pubsub
Draft

simone-sanfratello wants to merge 36 commits into
mainfrom
feat/storage-pubsub

Conversation

@simone-sanfratello

@simone-sanfratello simone-sanfratello commented Jul 25, 2025

Copy link
Copy Markdown
Contributor
  • detach streaming response passing through the storage, for following shared request
  • fix resume introducing resume method and eventId and streaming back prompts in chronological order
  • update ai-client to properly support resume
  • update documentation and examples
    • describe resumeEventId on ai-provider and ai-client
    • describe response type on ai-provider
    • describe resume behavior: it sends the response from requested event id
    • update architecture schema
  • run simulation, check ai-client, add error cases

@simone-sanfratello simone-sanfratello changed the title Feat/storage pubsub feat: storage pubsub and resume Jul 29, 2025
Base automatically changed from test/improve-coverage to main July 31, 2025 07:31

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

code loogs good so far

Comment thread .github/workflows/test.yml Outdated
@mikaelkaron

Copy link
Copy Markdown

would love to see this merged, keep up the good work!

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