[PR #127] [MERGED] Pregel/Graph: Make channel creation explicit, make Pregel constructor type safe #508

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

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langgraphjs/pull/127
Author: @nfcampos
Created: 5/9/2024
Status: Merged
Merged: 5/10/2024
Merged by: @nfcampos

Base: mainHead: nc/9may/pregel-rm-defaults


📝 Commits (3)

  • 3ce2ff8 Pregel/Graph: Make channel creation explicit, make Pregel constructor type safe
  • 6ff6646 Make type args const
  • a255da0 Add stricter typings for graph/stategraph/messagegraph constructors (#128)

📊 Changes

22 files changed (+1062 additions, -1257 deletions)

View changed files

📝 langgraph/src/channels/any_value.ts (+2 -2)
📝 langgraph/src/channels/base.ts (+5 -7)
📝 langgraph/src/channels/binop.ts (+2 -2)
📝 langgraph/src/channels/ephemeral_value.ts (+2 -2)
📝 langgraph/src/channels/last_value.ts (+2 -2)
📝 langgraph/src/channels/named_barrier_value.ts (+2 -2)
📝 langgraph/src/channels/topic.ts (+2 -2)
📝 langgraph/src/graph/graph.ts (+50 -23)
📝 langgraph/src/graph/message.ts (+7 -6)
📝 langgraph/src/graph/state.ts (+78 -56)
📝 langgraph/src/prebuilt/agent_executor.ts (+48 -79)
📝 langgraph/src/prebuilt/chat_agent_executor.ts (+31 -35)
📝 langgraph/src/pregel/index.ts (+118 -125)
📝 langgraph/src/pregel/io.ts (+24 -22)
📝 langgraph/src/pregel/validate.ts (+47 -34)
📝 langgraph/src/tests/chatbot.int.test.ts (+35 -48)
📝 langgraph/src/tests/graph.test.ts (+6 -4)
📝 langgraph/src/tests/prebuilt.int.test.ts (+4 -162)
📝 langgraph/src/tests/pregel.io.test.ts (+3 -3)
📝 langgraph/src/tests/pregel.test.ts (+397 -407)

...and 2 more files

📄 Description

  • still need to migrate StateGraph

Fixes # (issue)


🔄 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/127 **Author:** [@nfcampos](https://github.com/nfcampos) **Created:** 5/9/2024 **Status:** ✅ Merged **Merged:** 5/10/2024 **Merged by:** [@nfcampos](https://github.com/nfcampos) **Base:** `main` ← **Head:** `nc/9may/pregel-rm-defaults` --- ### 📝 Commits (3) - [`3ce2ff8`](https://github.com/langchain-ai/langgraphjs/commit/3ce2ff85742654ef1964a140fb0e88ace706b9d0) Pregel/Graph: Make channel creation explicit, make Pregel constructor type safe - [`6ff6646`](https://github.com/langchain-ai/langgraphjs/commit/6ff66465381673a5b26887dddbbed0cf14790eec) Make type args const - [`a255da0`](https://github.com/langchain-ai/langgraphjs/commit/a255da05f1e07b56b9f9df741a3254565adca54a) Add stricter typings for graph/stategraph/messagegraph constructors (#128) ### 📊 Changes **22 files changed** (+1062 additions, -1257 deletions) <details> <summary>View changed files</summary> 📝 `langgraph/src/channels/any_value.ts` (+2 -2) 📝 `langgraph/src/channels/base.ts` (+5 -7) 📝 `langgraph/src/channels/binop.ts` (+2 -2) 📝 `langgraph/src/channels/ephemeral_value.ts` (+2 -2) 📝 `langgraph/src/channels/last_value.ts` (+2 -2) 📝 `langgraph/src/channels/named_barrier_value.ts` (+2 -2) 📝 `langgraph/src/channels/topic.ts` (+2 -2) 📝 `langgraph/src/graph/graph.ts` (+50 -23) 📝 `langgraph/src/graph/message.ts` (+7 -6) 📝 `langgraph/src/graph/state.ts` (+78 -56) 📝 `langgraph/src/prebuilt/agent_executor.ts` (+48 -79) 📝 `langgraph/src/prebuilt/chat_agent_executor.ts` (+31 -35) 📝 `langgraph/src/pregel/index.ts` (+118 -125) 📝 `langgraph/src/pregel/io.ts` (+24 -22) 📝 `langgraph/src/pregel/validate.ts` (+47 -34) 📝 `langgraph/src/tests/chatbot.int.test.ts` (+35 -48) 📝 `langgraph/src/tests/graph.test.ts` (+6 -4) 📝 `langgraph/src/tests/prebuilt.int.test.ts` (+4 -162) 📝 `langgraph/src/tests/pregel.io.test.ts` (+3 -3) 📝 `langgraph/src/tests/pregel.test.ts` (+397 -407) _...and 2 more files_ </details> ### 📄 Description - still need to migrate StateGraph <!-- Thank you for contributing to LangGraph.js! Your PR will appear in our next release under the title you set above. Please make sure it highlights your valuable contribution. To help streamline the review process, please make sure you read our contribution guidelines: https://github.com/langchain-ai/langgraphjs/blob/main/CONTRIBUTING.md Replace this block with a description of the change, the issue it fixes (if applicable), and relevant context. Finally, we'd love to show appreciation for your contribution - if you'd like us to shout you out on Twitter, please also include your handle below! --> <!-- Remove if not applicable --> Fixes # (issue) --- <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 18:17:13 -05:00
yindo closed this issue 2026-02-15 18:17:13 -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#508