Skip to content

Commit 1b8458a

Browse files
Merge pull request #294 from ibuildthecloud/main
bug: allow sending empty messages to the LLM
2 parents b5ad549 + eb92e33 commit 1b8458a

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

pkg/engine/engine.go

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -335,15 +335,14 @@ func (e *Engine) Continue(ctx Context, state *State, results ...CallResult) (*Re
335335
}
336336

337337
for _, result := range results {
338-
if result.CallID != "" {
339-
state.Results[result.CallID] = result
340-
}
341-
if result.User != "" {
338+
if result.CallID == "" {
342339
added = true
343340
state.Completion.Messages = append(state.Completion.Messages, types.CompletionMessage{
344341
Role: types.CompletionMessageRoleTypeUser,
345342
Content: types.Text(result.User),
346343
})
344+
} else {
345+
state.Results[result.CallID] = result
347346
}
348347
}
349348

pkg/openai/client.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -304,7 +304,7 @@ func (c *Client) Call(ctx context.Context, messageRequest types.CompletionReques
304304
}
305305

306306
if len(msgs) == 0 {
307-
return nil, fmt.Errorf("invalid request, no messages to send to OpenAI")
307+
return nil, fmt.Errorf("invalid request, no messages to send to LLM")
308308
}
309309

310310
request := openai.ChatCompletionRequest{

0 commit comments

Comments
 (0)