feat: session restore for streamable HTTP transport #152
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Added StreamableSessionRepository for managing streamable HTTP session persistence, including creation, retrieval, updating access, and deletion of sessions. - Introduced StreamableSessionData interface to define the structure of streamable session data. - Enhanced ExpressServer to utilize StreamableSessionRepository for session handling, allowing restoration of sessions from persistent storage. - Updated streamable HTTP routes to support session restoration and management, ensuring seamless user experience across server restarts. - Added unit tests for StreamableSessionRepository to validate session operations and data integrity.
- Enhanced FileStorageService to support optional subdirectories for session storage, allowing better organization of session files. - Updated SDKOAuthClientProvider and OAuthStorageService to utilize the new subdirectory feature. - Implemented migration logic to move existing session files into appropriate subdirectories based on their prefixes. - Added unit tests to verify correct behavior of subdirectory creation and file migration. - Adjusted related tests to reflect changes in file path handling for session storage.
…n constants - Added migration flag creation logic to the FileStorageService, ensuring proper handling of old file structures. - Implemented methods to utilize FILE_PREFIX_MAPPING and STORAGE_SUBDIRS for improved file organization during migration. - Added unit tests to verify migration behavior, including scenarios for existing flags and error handling. - Refactored ID validation logic to use a new helper method for better maintainability. - Updated tests to reflect changes in migration and configuration integration.
xizhibei
changed the title
- Updated migration logic in FileStorageService to handle independent migrations for server and client session files. - Introduced migration flags for both server and client directories to track migration status. - Enhanced unit tests to verify migration behavior, including scenarios for independent migrations and the handling of transport files. - Refactored methods to improve clarity and maintainability of the migration process.
- Updated session restoration logic to retrieve session data more clearly and set the transport as initialized to allow restored sessions to function correctly. - Enhanced unit tests for streamable HTTP routes to include new initialization checks and ensure proper setup of async orchestrator. - Improved clarity in test assertions regarding transport creation and connection verification.
xizhibei
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Implements session restoration for streamable HTTP transport, allowing clients to reconnect and maintain state across connections.
Key Features
Changes
StreamableSessionRepositoryandStreamableSessionDatainterfaceBreaking Changes
None - maintains backward compatibility with automatic migration.