Skip to content

Conversation

@yaron2
Copy link
Member

@yaron2 yaron2 commented Nov 7, 2025

This PR registers a new extension that allows LangGraph to use Dapr as a persistent backend for saving the graph/execution state.

@yaron2 yaron2 requested review from a team as code owners November 7, 2025 23:33
@yaron2 yaron2 changed the title Add dapr checkpointer for langgraph Add dapr checkpointer for LangGraph Nov 7, 2025
Signed-off-by: yaron2 <[email protected]>
@codecov
Copy link

codecov bot commented Nov 7, 2025

Codecov Report

❌ Patch coverage is 94.73684% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.31%. Comparing base (bffb749) to head (4a1a258).
⚠️ Report is 49 commits behind head on main.

Files with missing lines Patch % Lines
...-langgraph/dapr/ext/langgraph/dapr_checkpointer.py 91.35% 7 Missing ⚠️
ext/dapr-ext-langgraph/tests/test_checkpointer.py 98.55% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #862      +/-   ##
==========================================
+ Coverage   86.63%   87.31%   +0.68%     
==========================================
  Files          84       96      +12     
  Lines        4473     6315    +1842     
==========================================
+ Hits         3875     5514    +1639     
- Misses        598      801     +203     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@cicoyle cicoyle left a comment

Choose a reason for hiding this comment

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

tiny typo :)

Co-authored-by: Cassie Coyle <[email protected]>
Signed-off-by: Yaron Schneider <[email protected]>
@acroca
Copy link
Member

acroca commented Nov 10, 2025

It's also missing tests

yaron2 and others added 10 commits November 13, 2025 21:33
Co-authored-by: Albert Callarisa <[email protected]>
Signed-off-by: Yaron Schneider <[email protected]>
Co-authored-by: Albert Callarisa <[email protected]>
Signed-off-by: Yaron Schneider <[email protected]>
Co-authored-by: Albert Callarisa <[email protected]>
Signed-off-by: Yaron Schneider <[email protected]>
Signed-off-by: yaron2 <[email protected]>
Signed-off-by: yaron2 <[email protected]>
Signed-off-by: yaron2 <[email protected]>
Signed-off-by: yaron2 <[email protected]>
Signed-off-by: yaron2 <[email protected]>
@yaron2
Copy link
Member Author

yaron2 commented Nov 19, 2025

It's also missing tests

Added

Signed-off-by: Albert Callarisa <[email protected]>
@acroca acroca added this pull request to the merge queue Nov 21, 2025
Merged via the queue into dapr:main with commit cf1e5d8 Nov 21, 2025
13 checks passed
@yaron2
Copy link
Member Author

yaron2 commented Nov 21, 2025

@acroca Casper will follow up with a PR that improves serialization

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