[PR #1023] [MERGED] chore(langgraph): begin porting python tests, with bug fixes #1177

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

📋 Pull Request Information

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

Base: mainHead: ben/port-python-tests


📝 Commits (10+)

  • 450cdb9 chore(langgraph): port first two checkpoint tests
  • 27ed45e chore(langgraph): port next checkpoint test
  • 4da74d1 chore(langgraph): port next checkpoint test
  • fbd21e3 feat(langgraph): add logWarning option to RetryPolicy (default: true)
  • 28585ec chore(langgraph): port next checkpoint test
  • 14460a4 chore(langgraph): port next checkpoint test (fails, exposing bug)
  • 0f100f5 fix(langgraph): StateSnapshot fixes
  • e856de0 chore(langgraph): port next checkpoint test
  • 3d97bdd chore(langgraph): port next checkpoint test
  • 92a9dfd feat(checkpoint): allow add'l custom fields on CheckpointMetadata

📊 Changes

14 files changed (+5342 additions, -285 deletions)

View changed files

📝 libs/checkpoint/src/types.ts (+7 -2)
📝 libs/langgraph/src/constants.ts (+122 -10)
📝 libs/langgraph/src/graph/state.ts (+10 -6)
📝 libs/langgraph/src/pregel/algo.ts (+43 -3)
📝 libs/langgraph/src/pregel/debug.ts (+2 -1)
📝 libs/langgraph/src/pregel/index.ts (+108 -33)
📝 libs/langgraph/src/pregel/loop.ts (+69 -41)
📝 libs/langgraph/src/pregel/retry.ts (+7 -5)
📝 libs/langgraph/src/pregel/runner.ts (+43 -22)
📝 libs/langgraph/src/pregel/utils/index.ts (+3 -0)
📝 libs/langgraph/src/tests/func.test.ts (+4 -1)
📝 libs/langgraph/src/tests/pregel.test.ts (+156 -161)
libs/langgraph/src/tests/python_port/checkpoint.test.ts (+2126 -0)
libs/langgraph/src/tests/python_port/graph_structure.test.ts (+2642 -0)

📄 Description

There's a bunch more to come, but this is the first batch.


🔄 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/1023 **Author:** [@benjamincburns](https://github.com/benjamincburns) **Created:** 3/24/2025 **Status:** ✅ Merged **Merged:** 3/24/2025 **Merged by:** [@benjamincburns](https://github.com/benjamincburns) **Base:** `main` ← **Head:** `ben/port-python-tests` --- ### 📝 Commits (10+) - [`450cdb9`](https://github.com/langchain-ai/langgraphjs/commit/450cdb99fd1069376f02054f009dcee6503abeb2) chore(langgraph): port first two checkpoint tests - [`27ed45e`](https://github.com/langchain-ai/langgraphjs/commit/27ed45ec970ede37110e7244cedcf1958c0a390f) chore(langgraph): port next checkpoint test - [`4da74d1`](https://github.com/langchain-ai/langgraphjs/commit/4da74d1d4c88f97b863beb941e2dc21612694190) chore(langgraph): port next checkpoint test - [`fbd21e3`](https://github.com/langchain-ai/langgraphjs/commit/fbd21e3be5162194b034c49465bbdcbb04b235eb) feat(langgraph): add logWarning option to RetryPolicy (default: true) - [`28585ec`](https://github.com/langchain-ai/langgraphjs/commit/28585ec031be44ec779ce00812d6b913791d307c) chore(langgraph): port next checkpoint test - [`14460a4`](https://github.com/langchain-ai/langgraphjs/commit/14460a4acc90d3d3bb28e74aa30e77313444c40d) chore(langgraph): port next checkpoint test (fails, exposing bug) - [`0f100f5`](https://github.com/langchain-ai/langgraphjs/commit/0f100f551f2b3aeda35e0025dae7c5c5827ab02c) fix(langgraph): StateSnapshot fixes - [`e856de0`](https://github.com/langchain-ai/langgraphjs/commit/e856de035adbde8a410a1a08e0c08e138d40ce0b) chore(langgraph): port next checkpoint test - [`3d97bdd`](https://github.com/langchain-ai/langgraphjs/commit/3d97bdd397d65551b79e12536bf7273237427333) chore(langgraph): port next checkpoint test - [`92a9dfd`](https://github.com/langchain-ai/langgraphjs/commit/92a9dfdef2f4b3f784ae5665f2247d4c7d735bef) feat(checkpoint): allow add'l custom fields on CheckpointMetadata ### 📊 Changes **14 files changed** (+5342 additions, -285 deletions) <details> <summary>View changed files</summary> 📝 `libs/checkpoint/src/types.ts` (+7 -2) 📝 `libs/langgraph/src/constants.ts` (+122 -10) 📝 `libs/langgraph/src/graph/state.ts` (+10 -6) 📝 `libs/langgraph/src/pregel/algo.ts` (+43 -3) 📝 `libs/langgraph/src/pregel/debug.ts` (+2 -1) 📝 `libs/langgraph/src/pregel/index.ts` (+108 -33) 📝 `libs/langgraph/src/pregel/loop.ts` (+69 -41) 📝 `libs/langgraph/src/pregel/retry.ts` (+7 -5) 📝 `libs/langgraph/src/pregel/runner.ts` (+43 -22) 📝 `libs/langgraph/src/pregel/utils/index.ts` (+3 -0) 📝 `libs/langgraph/src/tests/func.test.ts` (+4 -1) 📝 `libs/langgraph/src/tests/pregel.test.ts` (+156 -161) ➕ `libs/langgraph/src/tests/python_port/checkpoint.test.ts` (+2126 -0) ➕ `libs/langgraph/src/tests/python_port/graph_structure.test.ts` (+2642 -0) </details> ### 📄 Description There's a bunch more to come, but this is the first batch. --- <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:33 -05:00
yindo closed this issue 2026-02-15 19:17:33 -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#1177