Skip to content

Input parameters for tool #124

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Input parameters for tool #124

wants to merge 3 commits into from

Conversation

sankeyraut
Copy link
Contributor

Amazon Bedrock Agent Samples Pull Request

Instructions

  • Do not erase any parts of this template that are not applicable to your pull request.
  • If a section is not applicable, explicitly state the reason.
    • Tick the checkboxes for the items you have completed.
  • These are mandatory requirements, not mere suggestions.

Describe your changes

  • Concise description of the PR
Changes to mcp.py to allow configurable maximum parameters limit, because the current implementation has a hardcoded limit of 5 parameters which may not be sufficient for all use cases.

Issue ticket number and link


All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Are you uploading a dataset? (No)
  • Have you added contributions to RELEASE_NOTES? Yes

New Example Submissions:

Not applicable - this is a modification to an existing utility file.

  • Have you tested your code, and made sure the functionality runs successfully? Provide screenshots.

  • Have you linted your Python code with black?

  • Does this implementation use the shared tools src/utils/*. List them here:

  • Does this implementation use the helper functions src/utils/*. List them here:

No

src/utils Submissions:

Changes to the utils folder won't be accepted. Instead, open a new issue.

This is a modification to the InlineAgent/src/InlineAgent/tool/mcp.py file which enhances functionality by:

  1. Modifying the set_available_tools method to accept a max_parameters parameter with a default value of 5
  2. Updating both create methods in the MCPStdio and MCPHttp classes to accept and pass along this parameter
  3. Updating the error message to use the dynamic parameter value

These changes allow for more flexibility when working with tools that require more than 5 parameters.

Supported Quota : https://console.aws.amazon.com/servicequotas/home/services/bedrock/quotas/L-4B4330A0


src/shared tool Submissions:

Changes to existing tools won't be accepted. Instead, open a new issue.

  • Business justification for including a new tool
Not Applicable
  • How is this tool implemented?
      • AWS CDK
      • AWS CloudFormation (recommended)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant