vrr/open-code-review
2
0
Fork 0
mirror of https://github.com/alibaba/open-code-review.git synced 2026-07-09 17:28:58 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

fix(output): skip subtask_error in final warning summary to avoid duplicate logging

Browse source 
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.
This commit is contained in:
kite 2026-06-06 20:31:07 +08:00
parent 5466baa16e
commit e34c634879
1 changed files with 4 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

7
cmd/opencodereview/output.go
View file

@ -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)
}
}