[PR #374] [MERGED] Handle explicit undefined values passed into serde #705

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

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langgraphjs/pull/374
Author: @jacoblee93
Created: 8/25/2024
Status: Merged
Merged: 8/25/2024
Merged by: @jacoblee93

Base: mainHead: jacob/serde


📝 Commits (3)

  • 3e9860d Handle explicit undefined values passed in an update with a warning
  • 4ddea7e Serialize undefined as a special value
  • aa968cf Format

📊 Changes

7 files changed (+273 additions, -56 deletions)

View changed files

📝 libs/checkpoint/src/memory.ts (+33 -34)
📝 libs/checkpoint/src/serde/jsonplus.ts (+10 -3)
📝 libs/checkpoint/src/serde/tests/jsonplus.test.ts (+23 -7)
📝 libs/langgraph/src/pregel/index.ts (+0 -1)
📝 libs/langgraph/src/pregel/loop.ts (+5 -2)
📝 libs/langgraph/src/pregel/types.ts (+1 -1)
📝 libs/langgraph/src/tests/pregel.test.ts (+201 -8)

📄 Description

Cleans up instances where we do it internally, can also occur if a user passes an undefined value into a node update step.


🔄 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/374 **Author:** [@jacoblee93](https://github.com/jacoblee93) **Created:** 8/25/2024 **Status:** ✅ Merged **Merged:** 8/25/2024 **Merged by:** [@jacoblee93](https://github.com/jacoblee93) **Base:** `main` ← **Head:** `jacob/serde` --- ### 📝 Commits (3) - [`3e9860d`](https://github.com/langchain-ai/langgraphjs/commit/3e9860d79250edc4883b488ddd3a08d0f3ef93cd) Handle explicit undefined values passed in an update with a warning - [`4ddea7e`](https://github.com/langchain-ai/langgraphjs/commit/4ddea7e1ae5c9e804126789a6b54140749f2b86c) Serialize undefined as a special value - [`aa968cf`](https://github.com/langchain-ai/langgraphjs/commit/aa968cf6586bb7c0f4d09390b14856af3702bb02) Format ### 📊 Changes **7 files changed** (+273 additions, -56 deletions) <details> <summary>View changed files</summary> 📝 `libs/checkpoint/src/memory.ts` (+33 -34) 📝 `libs/checkpoint/src/serde/jsonplus.ts` (+10 -3) 📝 `libs/checkpoint/src/serde/tests/jsonplus.test.ts` (+23 -7) 📝 `libs/langgraph/src/pregel/index.ts` (+0 -1) 📝 `libs/langgraph/src/pregel/loop.ts` (+5 -2) 📝 `libs/langgraph/src/pregel/types.ts` (+1 -1) 📝 `libs/langgraph/src/tests/pregel.test.ts` (+201 -8) </details> ### 📄 Description Cleans up instances where we do it internally, can also occur if a user passes an `undefined` value into a node update step. --- <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:15:35 -05:00
yindo closed this issue 2026-02-15 19:15:36 -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#705