fix: anthropic documents in prompt

[milancermak]

Resolves #183

[milancermak]

Rebased with main

[0xMochan]

This form of document serialization sorta bypasses the general pattern that is used via the Display trait for all Documents, tbh, it might make sense to use anthropic's patterns as the way to serialize for all providers. I chose the current method arbitrarily.

Specifically for anthropic, it seems like you can (now?) pass documents as a type of content in a user message. It might be more suitable to use this method (albeit, contradicting with it's own user guides?) for passing documents to anthropic? Curious if you saw this and had thoughts?

[milancermak]

pass documents as a type of content in a user message

It seems that the API only supports PDFs in this case - the API docs only show application/pdf as the available option for messages.content.source.media_type

BTW I watched this video on prompting Claude yesterday. The guy mentions that Claude understand XML better (because apparently it has been trained on a lot of XML 🤷 ) and it doesn't necessarily matter what tags you use as long as they are meaningful and separate stuff in the prompt (so it's also a good idea to use sth like ). I'm writing all that just to say that the original you used isn't bad at all 😄

[joshua-mo-143]

It looks like the test failed because we're missing an API key.

cc @0xMochan

[0xMochan]

It looks like the test failed because we're missing an API key.

cc @0xMochan

This PR needs to rebase / merge w/ main for the CI/CD to update.