fix[190]: pass ddb record keys through pipe instead of entire new record

aws-samples / bedrock-claude-chat

AWS-native chatbot using Bedrock + Claude (+Mistral)

MIT No Attribution

828 stars 304 forks source link

fix[190]: pass ddb record keys through pipe instead of entire new record #233

Closed JoshuaToth closed 5 months ago

JoshuaToth commented 5 months ago

Issue #, if available: 190

Description of changes: Pass in the DDB keys object using the Eventbridge pipe, then in the main.py of the ECS task, load the DDB record. This will prevent the 8k character limit for the ECS task override which is currently causing syncing issues with large context windows or many files.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

JoshuaToth commented 5 months ago

testing merged changes, will fix lint error when tests are all good.

statefb commented 5 months ago

LGTM, thank you for your contribution! closes #190