docs: Bedrock example.

[nrfulton]

Misc PR

Type of PR

• Bug Fix
• New Feature
• Documentation
• Other

Description

• Link to Issue:

This PR adds a Bedrock example. Tests are not added because testing requires an AWS Bedrock key.

Testing

• Tests added to the respective file if code was changed
• New code has 100% coverage if code as added
• Ensure existing tests and github automation passes (a maintainer will kick off the github automation when the rest of the PR is populated)

[github-actions]

The PR description has been updated. Please fill out the template for your PR to be reviewed.

[mergify]

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

• title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert|release)(?:\(.+\))?:

[jakelorocco]

lgtm; only concern is if we want to be continuously adding new packages / groups to our dependencies? Do we want to have dependency groups for these well-known use cases like litellm for aws? and if so, we should have the aws optional dependency depend on the mellea[litellm] optional dependency.

[nrfulton]

Okay, I decided to move aws into the litellm optional.

Background/explanation:

I initially added boto3 to the litellm optional (hence the stray , in the litellm portion of that diff). But I then thought better of it and created [aws] for two reasons:

1. you can use bedrock with other client libraries. For example see the gpt-oss 120b example in the boto3 docs.
2. obviously, litellm works with other inference providers, so boto3 is usually not needed, so we're forcing people to download some code they don't need.

But (1) is not a valid reason for breaking out a separate aws group because the OpenAI backend does not require boto3.

And (2) is valid, but mellea is already heavy and we already have a slimming-down of mellea-core in our backlog. So for now there's no harm in adding one more dependency to an already heavy list.

So, given that more optionals is worse, ceteris paribus, and that boto3 is not always needed for bedrock, I think the best thing to do here is to move boto3 to litellm as @jakelorocco suggests.

That change is in and I also added an openai example. @jakelorocco would you mind re-reviewing and double-checking the examples given that we're not running cicd on them?

Also, @jakelorocco , do we still have tests that go through and run all of the examples? If so we should mark these as ignores. But I don't see that in the tests/ dir anymore.