Unknown parameter for CreateThreadAndRun method

[rasmusviben]

Describe the bug
When calling the CreateThreadAndRun method, the response is 400 due to unknown parameter in the ThreadRequest: thread.messages[0].file_ids

To Reproduce
Run follow code given assistant is created:

run, err := ss.openAiClient.CreateThreadAndRun(context.Background(), openai.CreateThreadAndRunRequest{
		RunRequest: openai.RunRequest{
			Model:        openai.GPT4o,
			AssistantID:  assistant.ID,
			Instructions: "Very cool instructions - test",
		},
		Thread: openai.ThreadRequest{
			Messages: []openai.ThreadMessage{
				{
					Role:    openai.ThreadMessageRoleUser,
					Content: "Do something nice with the file",
					FileIDs: []string{file.ID},
					Metadata: map[string]interface{}{
						"tools": []openai.AssistantTool{{Type: "file_search"}},
					},
				},
			},
		},
	})

Expected behavior
Expected that the request was succesfull and not throwing 400 bad request for that parameter.

Screenshots/Logs

2024/06/19 20:19:55 CreateThreadAndRun error: error, status code: 400, message: Unknown parameter: 'thread.messages[0].file_ids'.

Environment (please complete the following information):

• go-openai version: [e.g. v1.26.0]
• Go version: [1.22.3 windows/amd64]
• OS: Windows

[tforbus]

I think this PR is aiming to fix this issue.

@rasmusviben i'm seeing issues where the run states are never making it out of queued. Wondering if you'd seen this (maybe prior to v1.26.0)

[rasmusviben]

I think this PR is aiming to fix this issue.

@rasmusviben i'm seeing issues where the run states are never making it out of queued. Wondering if you'd seen this (maybe prior to v1.26.0)

Alright - do you know the ETA for getting that PR into a new release? :)

I tried downgrading to 1.24 and 1.25, but then new exception is thrown:

Invalid type for 'thread.messages[0].metadata.tools': expected a string, but got an array instead.

[Marreck]

Having the same issue. Downgrading to 1.24 worked for me though.

[ecoralic]

I assume that a problem here is outdated ThreadMessage struct. Even the mentioned PR above does not solve ThreadMessage struct, at least based on current OpenAI docs, where it states that ThreadMessage should have fields: Role, Content, Attachments and Metadata. Attachments is a list of files attached to the message, and the tools they should be added to.

[adamdecaf]

Need help getting a PR across the line? I'm seeing this error as well.

[adamdecaf]

#764 doesn't seem to fix the OP's issue (or mine) so I opened a PR which works, but has a breaking change. Do we have a policy for handling breaking changes better?

[peimanja]

Any updates on this?