Skip to content

Added request_id to the model response. #1547

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

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

peterHoburg
Copy link

I could not find an easy way to get the AWS Bedrock generated request_id in the response.

Is this something other people would want to use? If so I can update the docs, fix the tests, and update the other models to set request_id (if they support it).

…be updated. Only changed the bedrock model to support the new field.
Copy link
Contributor

hyperlint-ai bot commented Apr 19, 2025

PR Change Summary

Added request_id to the model response to facilitate tracking of AWS Bedrock requests.

  • Introduced request_id in the model response for better request tracking.
  • Updated multiple documentation files to include the new request_id parameter.

Modified Files

  • docs/agents.md
  • docs/message-history.md
  • docs/tools.md

How can I customize these reviews?

Check out the Hyperlint AI Reviewer docs for more information on how to customize the review.

If you just want to ignore it on this PR, you can add the hyperlint-ignore label to the PR. Future changes won't trigger a Hyperlint review.

Note specifically for link checks, we only check the first 30 links in a file and we cache the results for several hours (for instance, if you just added a page, you might experience this). Our recommendation is to add hyperlint-ignore to the PR to ignore the link check for this PR.

@DouweM
Copy link
Contributor

DouweM commented Apr 24, 2025

@peterHoburg Is this equivalent to OpenAI's chat.id at https://platform.openai.com/docs/api-reference/chat/object? What do you think about calling it vendor_id and covering other providers as well?

@peterHoburg
Copy link
Author

peterHoburg commented Apr 25, 2025

@peterHoburg Is this equivalent to OpenAI's chat.id at https://platform.openai.com/docs/api-reference/chat/object? What do you think about calling it vendor_id and covering other providers as well?

@DouweM Yes it is the same, and that sounds great! I will make the changes tonight and mark the PR as ready to review.

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

Successfully merging this pull request may close these issues.

2 participants