[PR #266] [MERGED] Adds automatic error handling for ToolNode, RemoveMessage support #619

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

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langgraphjs/pull/266
Author: @jacoblee93
Created: 7/17/2024
Status: Merged
Merged: 7/18/2024
Merged by: @jacoblee93

Base: mainHead: jacob/tool_node


📝 Commits (3)

  • 6cb2306 Adds automatic error handling for ToolNode
  • 11100b3 Adds support for RemoveMessage, add tool calling error guide
  • b17e3b5 Fix typos

📊 Changes

12 files changed (+1050 additions, -135 deletions)

View changed files

📝 docs/docs/how-tos/index.md (+1 -0)
📝 docs/mkdocs.yml (+1 -0)
examples/how-tos/tool-calling-errors.ipynb (+808 -0)
📝 langgraph/package.json (+4 -4)
📝 langgraph/src/graph/index.ts (+1 -1)
📝 langgraph/src/graph/message.ts (+55 -5)
📝 langgraph/src/prebuilt/tool_node.ts (+37 -16)
📝 langgraph/src/tests/prebuilt.test.ts (+57 -41)
📝 langgraph/src/tests/pregel.test.ts (+33 -29)
📝 langgraph/src/web.ts (+1 -0)
📝 package.json (+1 -1)
📝 yarn.lock (+51 -38)

📄 Description

This PR also changes the params of the ToolNode constructor, but the other params were undocumented and unlikely to be used. I think worth fixing now.


🔄 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/266 **Author:** [@jacoblee93](https://github.com/jacoblee93) **Created:** 7/17/2024 **Status:** ✅ Merged **Merged:** 7/18/2024 **Merged by:** [@jacoblee93](https://github.com/jacoblee93) **Base:** `main` ← **Head:** `jacob/tool_node` --- ### 📝 Commits (3) - [`6cb2306`](https://github.com/langchain-ai/langgraphjs/commit/6cb23060c31f164fefd51fd5538dd70def506e37) Adds automatic error handling for ToolNode - [`11100b3`](https://github.com/langchain-ai/langgraphjs/commit/11100b3b6234efded1c11e990abf600435561475) Adds support for RemoveMessage, add tool calling error guide - [`b17e3b5`](https://github.com/langchain-ai/langgraphjs/commit/b17e3b5daac22ca0805425b9704e7b4fead933ae) Fix typos ### 📊 Changes **12 files changed** (+1050 additions, -135 deletions) <details> <summary>View changed files</summary> 📝 `docs/docs/how-tos/index.md` (+1 -0) 📝 `docs/mkdocs.yml` (+1 -0) ➕ `examples/how-tos/tool-calling-errors.ipynb` (+808 -0) 📝 `langgraph/package.json` (+4 -4) 📝 `langgraph/src/graph/index.ts` (+1 -1) 📝 `langgraph/src/graph/message.ts` (+55 -5) 📝 `langgraph/src/prebuilt/tool_node.ts` (+37 -16) 📝 `langgraph/src/tests/prebuilt.test.ts` (+57 -41) 📝 `langgraph/src/tests/pregel.test.ts` (+33 -29) 📝 `langgraph/src/web.ts` (+1 -0) 📝 `package.json` (+1 -1) 📝 `yarn.lock` (+51 -38) </details> ### 📄 Description This PR also changes the params of the `ToolNode` constructor, but the other params were undocumented and unlikely to be used. I think worth fixing now. --- <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:14 -05:00
yindo closed this issue 2026-02-15 19:15:14 -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#619