refactor: use constants for default assistant messages

Apply code review feedback from gemini-code-assist:
- Define default messages as local constants to improve maintainability
- Avoid magic strings in the empty content handling logic
This commit is contained in:
taetaetae
2026-02-03 07:10:38 +09:00
parent b45ede0b71
commit 1f7c58f7ce

View File

@@ -695,10 +695,13 @@ func buildAssistantMessageFromOpenAI(msg gjson.Result) KiroAssistantResponseMess
// This can happen with compaction requests or error recovery scenarios
finalContent := contentBuilder.String()
if strings.TrimSpace(finalContent) == "" {
const defaultAssistantContentWithTools = "I'll help you with that."
const defaultAssistantContent = "I understand."
if len(toolUses) > 0 {
finalContent = "I'll help you with that."
finalContent = defaultAssistantContentWithTools
} else {
finalContent = "I understand."
finalContent = defaultAssistantContent
}
log.Debugf("kiro-openai: assistant content was empty, using default: %s", finalContent)
}