[PR #631] [MERGED] fix(checkpoint-sqlite): populate pending_sends and pendingWrites #897

Closed
opened 2026-02-15 19:16:19 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langgraphjs/pull/631
Author: @benjamincburns
Created: 10/24/2024
Status: Merged
Merged: 10/29/2024
Merged by: @jacoblee93

Base: mainHead: fix-sqlitesaver-pending-writes


📝 Commits (1)

  • 3562068 fix(checkpoint-sqlite): populate pending_sends and pendingWrites

📊 Changes

2 files changed (+182 additions, -61 deletions)

View changed files

📝 libs/checkpoint-sqlite/src/index.ts (+180 -56)
📝 libs/checkpoint-validation/src/spec/list.ts (+2 -5)

📄 Description

Per the title, fixes up put and getTuple on SqliteSaver to correctly populate the pending_sends and pendingWrites fields. Also adds a couple of input validation checks.

Finally, this change also modifies checkpoint-validation accordingly to unignore tests for SqliteSaver.

Fixes #590.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/langgraphjs/pull/631 **Author:** [@benjamincburns](https://github.com/benjamincburns) **Created:** 10/24/2024 **Status:** ✅ Merged **Merged:** 10/29/2024 **Merged by:** [@jacoblee93](https://github.com/jacoblee93) **Base:** `main` ← **Head:** `fix-sqlitesaver-pending-writes` --- ### 📝 Commits (1) - [`3562068`](https://github.com/langchain-ai/langgraphjs/commit/356206893f8716dcac76d9aa07a09aff90f1463c) fix(checkpoint-sqlite): populate pending_sends and pendingWrites ### 📊 Changes **2 files changed** (+182 additions, -61 deletions) <details> <summary>View changed files</summary> 📝 `libs/checkpoint-sqlite/src/index.ts` (+180 -56) 📝 `libs/checkpoint-validation/src/spec/list.ts` (+2 -5) </details> ### 📄 Description Per the title, fixes up `put` and `getTuple` on `SqliteSaver` to correctly populate the `pending_sends` and `pendingWrites` fields. Also adds a couple of input validation checks. Finally, this change also modifies `checkpoint-validation` accordingly to unignore tests for `SqliteSaver`. Fixes #590. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-15 19:16:19 -05:00
yindo closed this issue 2026-02-15 19:16:20 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/langgraphjs#897