Support Thread and Message for Assistant v2

[ZhihanWei]

Is your feature request related to a problem? Please describe.
OpenAI has released their new version of assistant(v2) and the old version of assistant(v1) will be deprecated by the end of 2024. There are some incompatibilities between the two version in terms of API request body.
For example, MessageRequest in v1

type MessageRequest struct {
	Role     string         `json:"role"`
	Content  string         `json:"content"`
	FileIds  []string       `json:"file_ids,omitempty"` //nolint:revive // backwards-compatibility
	Metadata map[string]any `json:"metadata,omitempty"`
}

MessageRequest in v2, which introduced attachments

type MessageRequest struct {
	Role        string         `json:"role"`
	Content     string         `json:"content"`
	Attachments []Attachment   `json:"attachments,omitempty"`
	Metadata    map[string]any `json:"metadata,omitempty"`
}

We let the user configure the assistant version in the client configuration but the new assistant version will cause trouble at runtime when making calls to Thread and Message.

Describe the solution you'd like
Support both assistant v1 and v2.

Additional context
There are different implementations I can think of that could make the enhancement, but most likely the compatibility will be checked at runtime. Would be good to see an implementation to support both assistant versions and check errors at compile time while maintaining the configured assistantVersion.

[Trojan295]

There are more changes, because the Content can be now a string, but also an array of objects. https://platform.openai.com/docs/api-reference/messages/createMessage

[ZhihanWei]

In the new version, seems like assistant v1 cannot usegpt-4o.

[nadeemc]

A workaround for now is to use a custom config to set the version:

config := openai.DefaultConfig(key)
config.AssistantVersion = "v2"

return openai.NewClientWithConfig(config)

[dezchai]

Was working on a pull request for this but don't know how to handle content being either a string or array in Go.

[dezchai]

@sashabaranov thoughts?

[sashabaranov]

https://github.com/sashabaranov/go-openai/releases/tag/v1.26.0 contains assistant API v2 by default

[dezchai]

@sashabaranov It's still missing attachments and content being a string or object.
dezchai@f4149b4
See: https://platform.openai.com/docs/api-reference/messages/object

[sashabaranov]

@dezchai we're open for your contributions to add those! 🥰

[dezchai]

@sashabaranov Sounds good, but I'm new to Go and am not sure how content being a string or array* in https://platform.openai.com/docs/api-reference/runs/createThreadAndRun

[sashabaranov]

@dezchai there are a couple of ways we can go here:

1. Use any as a type and document that it's either a string or array (slice in Go's terms). Then, it would be possible to use a type switch with it https://go.dev/tour/methods/16
2. Use json.RawMessage for it, see it's usage for errors
3. Add a custom type with json.Unmarshaler implemented for it (i.e. having a UnmarshalJSON method) errors examples applies here as well

UPD: note that with the first approach you'll get []any by default, not []string — see example https://go.dev/play/p/WTFANwhyJBT