feat(agents): человекочитаемые вердикты аналитика, ревьюера, dev и постмортема в панель «Логи»
This commit is contained in:
@@ -186,6 +186,7 @@ func (w *Worker) postMortem(ctx context.Context, task *storage.Task) {
|
||||
output = "(постмортем-агент не вернул текст)"
|
||||
}
|
||||
}
|
||||
log.Printf("worker: task %d: постмортем: %s", task.ID, truncateTrace(output, 2000))
|
||||
w.finalizeTrace(ctx, traceID, status, output)
|
||||
|
||||
if res.RC == 0 && output != "" {
|
||||
|
||||
@@ -4,6 +4,7 @@ import (
|
||||
"context"
|
||||
"encoding/json"
|
||||
"fmt"
|
||||
"log"
|
||||
"path/filepath"
|
||||
"strings"
|
||||
|
||||
@@ -96,6 +97,10 @@ func (w *Worker) runReviewer(ctx context.Context, taskID int64, cwd, prompt stri
|
||||
}
|
||||
w.finalizeTrace(ctx, traceID, ts, output)
|
||||
|
||||
if v != nil {
|
||||
log.Printf("reviewer: вердикт: %s", formatReviewVerdict(v))
|
||||
}
|
||||
|
||||
if v == nil {
|
||||
// Невалидный/пустой вывод или RC!=0: не парсится — сигнал на retry.
|
||||
return nil, output, traceID, nil
|
||||
@@ -119,3 +124,35 @@ func parseReviewVerdict(out string) (*reviewVerdict, error) {
|
||||
}
|
||||
return &v, nil
|
||||
}
|
||||
|
||||
// formatReviewVerdict собирает человекочитаемое однострочное описание вердикта
|
||||
// ревьюера (без JSON-разметки) для панели «Логи».
|
||||
func formatReviewVerdict(v *reviewVerdict) string {
|
||||
var b strings.Builder
|
||||
b.WriteString("passed=")
|
||||
b.WriteString(boolElse(v.Passed, "yes", "no"))
|
||||
if len(v.CriticalIssues) > 0 {
|
||||
b.WriteString(", critical=[")
|
||||
b.WriteString(strings.Join(v.CriticalIssues, " | "))
|
||||
b.WriteString("]")
|
||||
}
|
||||
if len(v.SolidViolations) > 0 {
|
||||
b.WriteString(", solid=[")
|
||||
b.WriteString(strings.Join(v.SolidViolations, " | "))
|
||||
b.WriteString("]")
|
||||
}
|
||||
if len(v.Comments) > 0 {
|
||||
b.WriteString(", comments=[")
|
||||
b.WriteString(strings.Join(v.Comments, " | "))
|
||||
b.WriteString("]")
|
||||
}
|
||||
return b.String()
|
||||
}
|
||||
|
||||
// boolElse возвращает строку для логического значения (без вызова тернарника).
|
||||
func boolElse(cond bool, yes, no string) string {
|
||||
if cond {
|
||||
return yes
|
||||
}
|
||||
return no
|
||||
}
|
||||
|
||||
@@ -281,6 +281,8 @@ func (w *Worker) runTask(ctx context.Context, task *storage.Task) (err error) {
|
||||
|
||||
output := res.Stdout
|
||||
|
||||
log.Printf("worker: task %d: dev вердикт (хвост): %s", task.ID, truncateTrace(output, 2000))
|
||||
|
||||
// 5b. dev не завершился успешно (RC!=0) → фиксируем без ревью.
|
||||
switch res.RC {
|
||||
case 0:
|
||||
|
||||
@@ -1090,3 +1090,46 @@ func TestRenderPostMortemPrompt(t *testing.T) {
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// TestFormatReviewVerdict — человекочитаемое описание вердикта ревьюера.
|
||||
func TestFormatReviewVerdict(t *testing.T) {
|
||||
tests := []struct {
|
||||
name string
|
||||
v *reviewVerdict
|
||||
want []string
|
||||
}{
|
||||
{
|
||||
name: "passed with everything",
|
||||
v: &reviewVerdict{
|
||||
Passed: true,
|
||||
CriticalIssues: []string{"bug A"},
|
||||
SolidViolations: []string{"v1", "v2"},
|
||||
Comments: []string{"comment"},
|
||||
},
|
||||
want: []string{"passed=yes", "critical=[bug A]", "solid=[v1 | v2]", "comments=[comment]"},
|
||||
},
|
||||
{
|
||||
name: "failed with critical only",
|
||||
v: &reviewVerdict{
|
||||
Passed: false,
|
||||
CriticalIssues: []string{"bug A", "bug B"},
|
||||
},
|
||||
want: []string{"passed=no", "critical=[bug A | bug B]"},
|
||||
},
|
||||
{
|
||||
name: "empty verdict",
|
||||
v: &reviewVerdict{},
|
||||
want: []string{"passed=no"},
|
||||
},
|
||||
}
|
||||
for _, tt := range tests {
|
||||
t.Run(tt.name, func(t *testing.T) {
|
||||
got := formatReviewVerdict(tt.v)
|
||||
for _, w := range tt.want {
|
||||
if !strings.Contains(got, w) {
|
||||
t.Errorf("formatReviewVerdict = %q, want contain %q", got, w)
|
||||
}
|
||||
}
|
||||
})
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user