Skip to content

fix: session state persistence in sequential agent pipelines#196

Closed
l33tdawg wants to merge 10 commits into
google:mainfrom
l33tdawg:fix-session-state-persistence
Closed

fix: session state persistence in sequential agent pipelines#196
l33tdawg wants to merge 10 commits into
google:mainfrom
l33tdawg:fix-session-state-persistence

Conversation

@l33tdawg

Copy link
Copy Markdown

This PR addresses a critical issue in the ADK where session state isn't properly persisted between agent transitions in sequential pipelines. It introduces an EnhancedStateDict implementation with global cache synchronization to ensure critical state values persist even when session objects are copied or recreated.

@hangfei

hangfei commented May 30, 2025

Copy link
Copy Markdown
Collaborator

@DeanChensj to help take a look to see if this aligns with your design.

@hangfei hangfei requested a review from DeanChensj May 30, 2025 17:46
@seanzhou1023

Copy link
Copy Markdown
Collaborator

@l33tdawg thank you so much for the PR :) Sorry for the late reply. could you please resolve conflicts ?

@l33tdawg

Copy link
Copy Markdown
Author

Sure man - let me take a look this weekend

@l33tdawg l33tdawg changed the title Fix session state persistence fix: session state persistence in sequential agent pipelines Jul 12, 2025
@l33tdawg

Copy link
Copy Markdown
Author

Done @seanzhou1023

@rohityan rohityan self-assigned this Oct 2, 2025
@rohityan

rohityan commented Oct 2, 2025

Copy link
Copy Markdown
Collaborator

Hi @l33tdawg and thanks to everyone who participated in this thread.

We're currently doing some housekeeping on our pull request queue. Given the rapid pace of development and the number of updates since this was last active, there's a good chance this pull request needs to be caught up to the latest version. To help us keep our backlog focused on current pull requests, we are closing this as stale.

If you're still interested in contributing to this change, could you please update to the very latest version of the library and create a new pull request? Our team will be glad to help with the refreshed PRs and answer any questions!

Thanks for your contribution and understanding!”

@rohityan rohityan closed this Oct 2, 2025
@l33tdawg

l33tdawg commented Oct 3, 2025 via email

Copy link
Copy Markdown
Author

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.

4 participants