Add session key support for AWS Bedrock#12
Conversation
|
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for AWS session tokens (temporary credentials) in the Bedrock model configuration and fixes an execution retry issue. The changes enable the use of AWS temporary credentials that require a session token, which is common when using AWS STS or IAM roles.
- Adds AWS session token support for Bedrock credentials
- Updates credential validation and configuration messages to include session token
- Fixes isComplete flag behavior in execution retry logic
Reviewed Changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| packages/factifai-agent/src/core/models/models.ts | Updates Bedrock model credentials to conditionally include AWS session token |
| packages/factifai-agent/src/cli.ts | Adds session token display and setup instructions in configuration commands |
| packages/factifai-agent/src/core/nodes/execution/execution-and-verification-node.ts | Changes isComplete flag from true to false on maximum retries |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Pull Request
📝 Description
🔍 Related Issues
🔄 Type of Change
📋 Changeset
🧪 Testing
📸 Screenshots
✅ Checklist
📣 Additional Context