feat: Add support for partial literals and enums

[lucasastorian]

---

	⚠️ This PR description was created by Ellipsis for commit 23a3abb.

Summary:

This PR modifies the '_make_field_optional' function in 'partial.py' to convert Literal and Enum fields to Optional[str] to prevent streaming failures with OpenAI.

Key points:

• Added checks in '_make_field_optional' function to convert Literal and Enum fields to Optional[str].
• This change is made to prevent failures when streaming with OpenAI.

---

Generated with ❤️ by ellipsis.dev

[ellipsis-dev]

Looks good to me!

---

Something look wrong? You can customize Ellipsis' behavior when reviewing code by editing the config file for this repository.

Generated with ❤️ by ellipsis.dev

[jxnl]

beautiful! Can you:

• add a test
• making a note of this in the docs?

[jxnl]

Looking back, I feel like things should be... validated with and without strict mode. Something we should think harder about.

[lucasastorian]

Sorry - haven't gotten to this yet. Will take a look tomorrow.

I've been using Partial to stream incomplete tool completions from OpenAI (I added support for tool choice myself). The issue was that (obviously), the Pydantic schema would fail mid-stream, when OpenAI didn't stream the full Literal/Enum value at once. One way to do it, is to validate completed attributes in strict mode, and incomplete attributes ones in unstrict mode. That might be a little overkill though - it's just as easy to validate in unstrict mode during the stream & then in strict mode once complete.

[jxnl]

Yeah, let's just do unstrict, doing partial streaming in strict.

[jxnl]

are you able to add tests?