review

type ConfidenceBreakdown struct {
	High   []Finding `json:"high"`
	Medium []Finding `json:"medium"`
	Low    []Finding `json:"low"`
}

type Finding struct {
	Concern    string             `json:"concern"`
	Severity   contracts.Severity `json:"severity"`
	File       string             `json:"file"`
	Line       int                `json:"line"`
	EndLine    int                `json:"end_line,omitempty"`
	Message    string             `json:"message"`
	Fix        string             `json:"fix,omitempty"`
	Reasoning  string             `json:"reasoning,omitempty"`
	CWE        string             `json:"cwe,omitempty"`
	Confidence float64            `json:"confidence"`
	SASTSource bool               `json:"sast_source,omitempty"`
}

type InlineComment struct {
	Path       string `json:"path"`
	StartLine  int    `json:"start_line"`
	EndLine    int    `json:"end_line,omitempty"`
	Body       string `json:"body"`
	Suggestion string `json:"suggestion,omitempty"`
}

type Result struct {
	Findings []Finding          `json:"findings"`
	Comments []InlineComment    `json:"comments"`
	Stats    Stats              `json:"stats"`
	Report   string             `json:"report"`
	FailOn   contracts.Severity `json:"fail_on"`
	// FailOnSet reports whether FailOn was explicitly configured via
	// SetFailOn. When it is false, Failed() treats SeverityCritical as the
	// effective threshold: an unset FailOn must not fail the review on
	// informational findings just because SeverityInfo is the zero value.
	FailOnSet           bool                 `json:"fail_on_set,omitempty"`
	SASTFusion          *SASTFusionResult    `json:"sast_fusion,omitempty"`
	ConfidenceBreakdown *ConfidenceBreakdown `json:"confidence_breakdown,omitempty"`
}

type SASTFusionResult struct {
	Confirmed   []Finding `json:"confirmed"`
	Dismissed   []Finding `json:"dismissed"`
	Unaddressed []Finding `json:"unaddressed"`
}

type Stats struct {
	FilesReviewed       int                        `json:"files_reviewed"`
	HunksAnalyzed       int                        `json:"hunks_analyzed"`
	FindingsTotal       int                        `json:"findings_total"`
	BySeverity          map[contracts.Severity]int `json:"by_severity"`
	ByConcern           map[string]int             `json:"by_concern"`
	TokensUsed          int                        `json:"tokens_used"`
	DurationPerConcern  map[string]time.Duration   `json:"duration_per_concern"`
	AverageConfidence   float64                    `json:"average_confidence"`
	HighConfidenceCount int                        `json:"high_confidence_count"`
	LowConfidenceCount  int                        `json:"low_confidence_count"`
	// LLMErrors records non-fatal provider errors encountered during
	// analysis; findings may be partial when it is non-empty.
	LLMErrors []string `json:"llm_errors,omitempty"`
}