-
Notifications
You must be signed in to change notification settings - Fork 360
Labels
P3Nice to haves, rare edge casesNice to haves, rare edge casesbreaking changeThis requires a breaking API or behavior changeThis requires a breaking API or behavior changev2Planned to be addressed in the v2 releasePlanned to be addressed in the v2 release
Description
Following up on some discussion in #685: When fixing #199, I was overly cautious about handling typed nils: if the user provided their own output schema, we wouldn't fix typed nils for them, and so returning 'nil' would result in a missing structured output, rather than its zero value.
This was wrong: switching from an inferred schema to a user-provided schema shouldn't change the handling of nil, and structured output, if provided, must be an object.
PR incoming.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
P3Nice to haves, rare edge casesNice to haves, rare edge casesbreaking changeThis requires a breaking API or behavior changeThis requires a breaking API or behavior changev2Planned to be addressed in the v2 releasePlanned to be addressed in the v2 release