agentjido / req_llm / 23859a93d75c59fc85903a424d21a19e3d198d43-PR-66

50%

main: 49%

Build Type

Pull #66

github

Committed by neilberkman

Commit Message

Rename Bedrock to AmazonBedrock and address all review feedback

- Rename all modules from Bedrock to AmazonBedrock per review request
- Use :amazon_bedrock provider ID to match upstream models.dev
- Move AWS Event Stream parser under AmazonBedrock namespace
- Remove debug.log file
- Revert changes to priv/models_dev/amazon-bedrock.json (use upstream version)
- Fix Context API usage in tests (use Context.user/system helpers)
- Fix StreamChunk type expectations (:content and :meta)
- Add @dialyzer :no_match suppression for DSL macro issues
- Fix system message extraction in Anthropic formatter
- Update all test files to handle provider renaming

Addresses all of Mike's review comments:
1. Removed debug.log
2. Moved AWS Event Stream under AmazonBedrock namespace
3. No modifications to models_dev JSON files
4. Renamed Bedrock to AmazonBedrock throughout

All 959 tests passing, live streaming verified with AWS credentials.

Pull Request Pull Request #66: Add AWS Bedrock provider with streaming support

Run Details

112 of 268 new or added lines in 5 files covered. (41.79%)

23 existing lines in 2 files now uncovered.

2085 of 3628 relevant lines covered (57.47%)

284.15 hits per line