Skip to content

Commit ff4552a

Browse files
[feature/openrouter-provider] Refactor finish reason handling and tool call logic in agent and OpenAI provider
- Simplified finish reason check in `agent.go` by removing redundant variable assignment. - Updated `openai.go` to override the finish reason to `FinishReasonToolUse` when tool calls are present. - Ensured consistent finish reason handling in both `send` and `stream` methods of the OpenAI provider.
1 parent c3568d6 commit ff4552a

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

internal/llm/provider/openai.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -280,7 +280,6 @@ func (o *openaiClient) stream(ctx context.Context, messages []message.Message, t
280280

281281
finishReason := o.finishReason(string(acc.ChatCompletion.Choices[0].FinishReason))
282282

283-
// Override finish reason to tool_use if there are tool calls
284283
if len(toolCalls) > 0 {
285284
finishReason = message.FinishReasonToolUse
286285
}

0 commit comments

Comments
 (0)