feat(aws): add Bedrock AgentCore Code Interpreter connector#6741
Conversation
7ce0db4 to
11dda4f
Compare
ee9858e to
3f95c4d
Compare
nikonovd
left a comment
There was a problem hiding this comment.
Generally looks good to me.
We should definitely sync with @gbetances089 regarding E2E test setup.
I added some minor comments. Please take a look ⛱️
nikonovd
left a comment
There was a problem hiding this comment.
Looks good to me so far, good work! 🚀
Please check the two open questions of mine regarding UX, that we could potentially improve here, otherwise it's looking good from my side
|
@ztefanie I already adapted it in #6780, but right now I think we have a general issue with code owners as as |
…e retrieval Drive the real InvokeCodeInterpreterResponseHandler with synthesized events to cover stdout/stderr/exitCode accumulation, file listing + diff, maxFiles and maxTotalFileSize limits, read-failure recovery, and resource content extraction. Replaces a handful of direct helper tests; getBlockSize and convertToDocuments are now private again.
|
🎉 This pull request has been included in release Thank you for your contribution! 🚀 |
Description
Implement custom connecter for AWS Bedrock AgentCore Code Interpreter
Related issues
closes #6855
Checklist
release, as this branch will be rebased onto main before the next release. Example backport labels:
backport stable/8.8: for changes that should be included in the next 8.8.x release.backport release-8.8.7: for changes that should be included in the specific release 8.8.7, and thisrelease has already been created. The release branch will be merged back into stable/8.8 later, so the change
will be included in future 8.8.x releases as well.