[PR #1046] [MERGED] fix(langgraph): handle cycles correctly on Command/Send deserialization #1199

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

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langgraphjs/pull/1046
Author: @benjamincburns
Created: 3/26/2025
Status: Merged
Merged: 3/26/2025
Merged by: @benjamincburns

Base: mainHead: ben/fix-command-cycle-error


📝 Commits (2)

  • 46ac172 fix(langgraph): handle cycles correctly on Command/Send deserialization
  • 7e1ded8 fix(langgraph): Handle Command/Send type narrowing w/o instanceof

📊 Changes

4 files changed (+264 additions, -58 deletions)

View changed files

📝 libs/langgraph/src/constants.ts (+68 -43)
libs/langgraph/src/tests/constants.test.ts (+176 -0)
📝 libs/langgraph/src/tests/python_port/graph_structure.test.ts (+18 -14)
📝 libs/langgraph/src/tests/remote.test.ts (+2 -1)

📄 Description

Fixes #1041


🔄 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/1046 **Author:** [@benjamincburns](https://github.com/benjamincburns) **Created:** 3/26/2025 **Status:** ✅ Merged **Merged:** 3/26/2025 **Merged by:** [@benjamincburns](https://github.com/benjamincburns) **Base:** `main` ← **Head:** `ben/fix-command-cycle-error` --- ### 📝 Commits (2) - [`46ac172`](https://github.com/langchain-ai/langgraphjs/commit/46ac1727cb7b70ad0397e0d134e46d19a9b40870) fix(langgraph): handle cycles correctly on Command/Send deserialization - [`7e1ded8`](https://github.com/langchain-ai/langgraphjs/commit/7e1ded8fc70bf539816b895b6d66d62422fa9a8d) fix(langgraph): Handle Command/Send type narrowing w/o instanceof ### 📊 Changes **4 files changed** (+264 additions, -58 deletions) <details> <summary>View changed files</summary> 📝 `libs/langgraph/src/constants.ts` (+68 -43) ➕ `libs/langgraph/src/tests/constants.test.ts` (+176 -0) 📝 `libs/langgraph/src/tests/python_port/graph_structure.test.ts` (+18 -14) 📝 `libs/langgraph/src/tests/remote.test.ts` (+2 -1) </details> ### 📄 Description Fixes #1041 --- <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:17:38 -05:00
yindo closed this issue 2026-02-15 19:17:38 -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#1199