[Platform] Prepend ImageNormalizer in OpenAIContract

[melkamar]

Fixes serialization of Image messages as Image
rather than its parent File.

Q	A
Bug fix?	yes
New feature?	no
Docs?	no
Issues	Fix #488
License	MIT

[chr-hertel]

Not sure, maybe the FileNormalizer is just to abstract and should rather be a DocumentNormalizer - does that work out?

[melkamar]

Narrowing down the FileNormalizer makes this work for the Image case. But I can't comment on whether the normalizer was really intended only for Document.

There is a unit test in the commit introducing it which asserts that the normalizer can also handle File instances.

[chr-hertel]

Narrowing down the FileNormalizer makes this work for the Image case. But I can't comment on whether the normalizer was really intended only for Document.

There is a unit test in the commit introducing it which asserts that the normalizer can also handle File instances.

@glengemann can you comment? what was intended there?

[glengemann]

Actually, the original intention was only to support Document.

[chr-hertel]

Thanks @glengemann for the fast feedback!

Let's go with limiting the FileNormalizer to be a DocumentNormalizer for a fix than. If that works out at least.

[OskarStark]

And renameing FileNormalizer to DocumentNormalizer, right?

[glengemann]

Oskar, yes, that would be fine. We named it file to match the JSON key ("type": "file") in the request. However, now it seems more appropriate to call it Document.

{
    "type": "file",
    "file": {
        "file_id": "file-6F2ksmvXxt4VdoqmHRw6kL"
    }
},

[OskarStark]

I renamed the class in
#493

[chr-hertel]

Closed in favor of #496