Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance Bedrock Agents support to accommodate Messages, ToolNode #261

Open
3coins opened this issue Oct 29, 2024 · 0 comments
Open

Enhance Bedrock Agents support to accommodate Messages, ToolNode #261

3coins opened this issue Oct 29, 2024 · 0 comments
Assignees
Labels
bedrock agents enhancement New feature or request

Comments

@3coins
Copy link
Collaborator

3coins commented Oct 29, 2024

The original PR that introduced Bedrock Agents support had a few shortcomings w.r.t. use of deprecated components and no message support which is central to the way LangChain components work.

  1. Even though there is no LLM present here (agent manages the LLM calls), a lot of the LangChain components work around messages which is central to it's working. We should follow up with a PR to support messages.
  2. The LangGraph integration test is using the ToolExecutor, which has been deprecated in favor of ToolNode, which works only with messages. To future proof this work, we should move to use ToolNode in LangGraph samples and tests.

Originally posted by @3coins in #152 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bedrock agents enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants