[PR #41] [MERGED] Replace Tool by StructuredTool in tool_executor.ts #440

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

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langgraphjs/pull/41
Author: @zeropaper
Created: 2/1/2024
Status: Merged
Merged: 2/10/2024
Merged by: @bracesproul

Base: mainHead: main


📝 Commits (2)

  • a6478d5 replace Tool by StructuredTool
  • cc90256 Merge branch 'main' into main

📊 Changes

2 files changed (+8 additions, -8 deletions)

View changed files

📝 langgraph/src/prebuilt/chat_agent_executor.ts (+3 -3)
📝 langgraph/src/prebuilt/tool_executor.ts (+5 -5)

📄 Description

Because the Tool abstract class extends the StructuredTool abstract class, using a DynamicStructuredTool would not be usable in the ToolExecutor.

https://api.js.langchain.com/classes/langchain_core_tools.StructuredTool.html

Fixes #40

Note: I did not clone the repo locally or tested that.


🔄 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/41 **Author:** [@zeropaper](https://github.com/zeropaper) **Created:** 2/1/2024 **Status:** ✅ Merged **Merged:** 2/10/2024 **Merged by:** [@bracesproul](https://github.com/bracesproul) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (2) - [`a6478d5`](https://github.com/langchain-ai/langgraphjs/commit/a6478d5074a64dac8bb1338c82526ff2ae3a5f66) replace Tool by StructuredTool - [`cc90256`](https://github.com/langchain-ai/langgraphjs/commit/cc902566708a091f5bea7db52efcb21dad313ed6) Merge branch 'main' into main ### 📊 Changes **2 files changed** (+8 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `langgraph/src/prebuilt/chat_agent_executor.ts` (+3 -3) 📝 `langgraph/src/prebuilt/tool_executor.ts` (+5 -5) </details> ### 📄 Description <!-- 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! --> Because the `Tool` abstract class extends the `StructuredTool` abstract class, using a `DynamicStructuredTool` would not be usable in the `ToolExecutor`. https://api.js.langchain.com/classes/langchain_core_tools.StructuredTool.html Fixes #40 ~~**Note:** I did not clone the repo locally or tested that.~~ --- <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:59 -05:00
yindo closed this issue 2026-02-15 18:16:59 -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#440