History

Merge pull request #194 from PancakeZik/fix/assistant-content-parroting

Luis Pater committed 9 months agov6.8.1-1

Commit d182e892 parents 2e8d49a 98edcad1 file · +6 −2Browse files

Unified · Split

internal/translator/kiro/common/constants.go+6−2
@@ -31,11 +31,15 @@ const ( · 30 unmodified lines
3131
3232 // DefaultAssistantContentWithTools is the fallback content for assistant messages
3333 // that have tool_use but no text content. Kiro API requires non-empty content.
34− DefaultAssistantContentWithTools = "I'll help you with that."
34+ // IMPORTANT: Use a minimal neutral string that the model won't mimic in responses.
35+ // Previously "I'll help you with that." which caused the model to parrot it back.
36+ DefaultAssistantContentWithTools = "."
3537
3638 // DefaultAssistantContent is the fallback content for assistant messages
3739 // that have no content at all. Kiro API requires non-empty content.
38− DefaultAssistantContent = "I understand."
40+ // IMPORTANT: Use a minimal neutral string that the model won't mimic in responses.
41+ // Previously "I understand." which could leak into model behavior.
42+ DefaultAssistantContent = "."
3943
4044 // DefaultUserContentWithToolResults is the fallback content for user messages
4145 // that have only tool_result (no text). Kiro API requires non-empty content.