on main help sign in
fix: fix openai provider
commitb5cbb38c98f6ad58d803afca123604eb940496c1 authorKujtim Hoxha <kujtimii.h@gmail.com> dateWed 16 Jul 2025, 09:21 (1 year ago) parent94bf17a
1 file changed+7
internal/llm/provider/openai.go+7
internal/llm/provider/openai.go+7
@@ -77,13 +77,16 @@ func (o *openaiClient) convertMessages(messages []message.Message) (openaiMessag
7777 Role: "assistant",
7878 }
7979
80+ hasContent := false
8081 if msg.Content().String() != "" {
82+ hasContent = true
8183 assistantMsg.Content = openai.ChatCompletionAssistantMessageParamContentUnion{
8284 OfString: openai.String(msg.Content().String()),
8385 }
8486 }
8587
8688 if len(msg.ToolCalls()) > 0 {
89+ hasContent = true
8790 assistantMsg.ToolCalls = make([]openai.ChatCompletionMessageToolCallParam, len(msg.ToolCalls()))
8891 for i, call := range msg.ToolCalls() {
8992 assistantMsg.ToolCalls[i] = openai.ChatCompletionMessageToolCallParam{
@@ -96,6 +99,10 @@ func (o *openaiClient) convertMessages(messages []message.Message) (openaiMessag
9699 }
97100 }
98101 }
102+ if !hasContent {
103+ slog.Warn("There is a message without content, investigate, this should not happen")
104+ continue
105+ }
99106
100107 openaiMessages = append(openaiMessages, openai.ChatCompletionMessageParamUnion{
101108 OfAssistant: &assistantMsg,