BinaryContent identifier should be optional #3107
Description
Initial Checks
- I confirm that I'm using the latest version of Pydantic AI
- I confirm that I searched for my issue in https://github.com/pydantic/pydantic-ai/issues before opening this issue
Description
As mentioned above, that PR erroneously switched BinaryContent.identifier to non-optional, assuming that the constructor will always set it if it is not provided. This however breaks e.g. ModelMessageAdapter.validate_json from previous versions which exported json objects with identifier=None
Example Code
Python, Pydantic AI & LLM client version
1.0.14