From e34c63487961185839a683c0611994458ba69bec Mon Sep 17 00:00:00 2001 From: kite Date: Sat, 6 Jun 2026 20:31:07 +0800 Subject: [PATCH] fix(output): skip subtask_error in final warning summary to avoid duplicate logging Subtask errors are already printed in real-time when they occur in agent.go. The final outputTextWithWarnings was printing them again after comments were displayed. --- cmd/opencodereview/output.go | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/cmd/opencodereview/output.go b/cmd/opencodereview/output.go index a2ccc7b..9bbd980 100644 --- a/cmd/opencodereview/output.go +++ b/cmd/opencodereview/output.go @@ -43,10 +43,11 @@ func outputTextWithWarnings(comments []model.LlmComment, warnings []agent.AgentW renderComment(c) } } - if len(warnings) > 0 { - for _, w := range warnings { - fmt.Fprintf(os.Stderr, "[ocr] WARNING [%s] %s: %s\n", w.Type, w.File, w.Message) + for _, w := range warnings { + if w.Type == "subtask_error" { + continue } + fmt.Fprintf(os.Stderr, "[ocr] WARNING [%s] %s: %s\n", w.Type, w.File, w.Message) } }