[PR #35] [MERGED] [patch]: Pass generics through to Channel #438

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

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langgraphjs/pull/35
Author: @bracesproul
Created: 1/29/2024
Status: Merged
Merged: 1/29/2024
Merged by: @bracesproul

Base: mainHead: brace/pass-generics


📝 Commits (9)

  • 776224e [patch]: Pass generics through to Channel
  • 7043c58 revert notebook changes
  • 3a0465e drop console log
  • 3e3d116 drop .only
  • 37465dc Merge branch 'main' into brace/pass-generics
  • 7eaae83 chore: lint files
  • 6007ced Update multi-agent-collaboration.ipynb
  • 39385fd cr
  • 9c010f8 Merge branch 'main' into brace/pass-generics

📊 Changes

10 files changed (+72 additions, -47 deletions)

View changed files

📝 langgraph/src/checkpoint/base.ts (+1 -1)
📝 langgraph/src/graph/state.ts (+15 -13)
📝 langgraph/src/prebuilt/agent_executor.ts (+13 -10)
📝 langgraph/src/prebuilt/chat_agent_executor.ts (+7 -5)
📝 langgraph/src/prebuilt/index.ts (+8 -2)
📝 langgraph/src/pregel/index.ts (+3 -3)
📝 langgraph/src/pregel/read.ts (+1 -1)
📝 langgraph/src/pregel/reserved.ts (+3 -3)
📝 langgraph/src/pregel/validate.ts (+2 -2)
📝 langgraph/src/tests/pregel.test.ts (+19 -7)

📄 Description

Fix generic typing

I had to switch to ReservedChannelsMap from ReservedChannels (enum) because not doing const enum caused a build error, and even if we could do const enum I got a TS error saying const enum's are removed and not available at runtime.


🔄 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/35 **Author:** [@bracesproul](https://github.com/bracesproul) **Created:** 1/29/2024 **Status:** ✅ Merged **Merged:** 1/29/2024 **Merged by:** [@bracesproul](https://github.com/bracesproul) **Base:** `main` ← **Head:** `brace/pass-generics` --- ### 📝 Commits (9) - [`776224e`](https://github.com/langchain-ai/langgraphjs/commit/776224e1bf99756aa6ec21db447fa0b40369d2e1) [patch]: Pass generics through to Channel - [`7043c58`](https://github.com/langchain-ai/langgraphjs/commit/7043c582747bd582f4bfbf7a608592f3097bc7c3) revert notebook changes - [`3a0465e`](https://github.com/langchain-ai/langgraphjs/commit/3a0465ecb3ebcf9c8394628f4a0e367c937e7887) drop console log - [`3e3d116`](https://github.com/langchain-ai/langgraphjs/commit/3e3d116fe23cb7b4114a2d9948c2eb52755502ee) drop .only - [`37465dc`](https://github.com/langchain-ai/langgraphjs/commit/37465dc64090d800b7a59a31dd61c5a1ee608d7a) Merge branch 'main' into brace/pass-generics - [`7eaae83`](https://github.com/langchain-ai/langgraphjs/commit/7eaae83f6196a30e5943b7704821d0d333609861) chore: lint files - [`6007ced`](https://github.com/langchain-ai/langgraphjs/commit/6007ced30fd1e1d3cb717b073530adfa92215532) Update multi-agent-collaboration.ipynb - [`39385fd`](https://github.com/langchain-ai/langgraphjs/commit/39385fd56a024a401d92c70c954de1e016e372e3) cr - [`9c010f8`](https://github.com/langchain-ai/langgraphjs/commit/9c010f8542b5b1cca19c974b292db42adabada22) Merge branch 'main' into brace/pass-generics ### 📊 Changes **10 files changed** (+72 additions, -47 deletions) <details> <summary>View changed files</summary> 📝 `langgraph/src/checkpoint/base.ts` (+1 -1) 📝 `langgraph/src/graph/state.ts` (+15 -13) 📝 `langgraph/src/prebuilt/agent_executor.ts` (+13 -10) 📝 `langgraph/src/prebuilt/chat_agent_executor.ts` (+7 -5) 📝 `langgraph/src/prebuilt/index.ts` (+8 -2) 📝 `langgraph/src/pregel/index.ts` (+3 -3) 📝 `langgraph/src/pregel/read.ts` (+1 -1) 📝 `langgraph/src/pregel/reserved.ts` (+3 -3) 📝 `langgraph/src/pregel/validate.ts` (+2 -2) 📝 `langgraph/src/tests/pregel.test.ts` (+19 -7) </details> ### 📄 Description Fix generic typing I had to switch to `ReservedChannelsMap` from `ReservedChannels` (enum) because not doing `const enum` caused a build error, and even if we could do `const enum` I got a TS error saying `const enum`'s are removed and not available at runtime. --- <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:16:58 -05:00
yindo closed this issue 2026-02-15 18:16:58 -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#438