Skip to content

Fix exception and minor formatting issue in bedrock_agent_helper.py when tracing and formatting references #135

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 2 commits into
base: main
Choose a base branch
from

Conversation

rostcheck
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 bedrock_agent_helper.py to 1. fix an exception when tracing references (needed datetime custom formatter) and 2. fix minor reference formatting issue that sometimes clipped last character of description

Issue ticket number and link

  • Issue # (if applicable) (internal) V1679408150

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?
  • [NO] Are you uploading a dataset?
  • Have you added contributions to RELEASE NOTES?

New Example Submissions:

  • Have you tested your code, and made sure the functionality runs successfully? Provide screenshots.
  • Have you linted your Python code with black?
  • [NA] Does this implementation use the shared tools src/utils/*. List them here:
1. web_search
2. ...
  • Does this implementation use the helper functions src/utils/*. List them here:
1. bedrock_agent_helper.py
2. ...

src/utils Submissions:

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


src/shared tool Submissions:

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

  • [NA] Business justification for including a new tool
This tool is necessary because ...
  • How is this tool implemented?
      • [NA] AWS CDK
      • [NA] AWS CloudFormation (recommended)

rostcheck added 2 commits May 19, 2025 19:02
…hich sometimes includes datetime objects not pre-formatted to JSON, and fixed minor issue that sometimes clipped a character off a block of text when a reference was being inserted.
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