review

type Category string

type Event struct {
	Kind EventKind
	Text string
}

type EventKind int

type Finding struct {
	ID         string       `json:"id"`
	File       string       `json:"file,omitempty"`     // new path, repo-relative; empty on MR-level manual comments
	OldFile    string       `json:"old_file,omitempty"` // as reported by the model for renames; advisory only
	Line       LineRef      `json:"line,omitzero"`
	Severity   Severity     `json:"severity,omitempty"`
	Category   Category     `json:"category,omitempty"`
	Title      string       `json:"title,omitempty"`
	Body       string       `json:"body"`                 // markdown, user-editable
	Suggestion string       `json:"suggestion,omitempty"` // optional replacement for the flagged line
	State      FindingState `json:"state"`
	// Manual marks a comment written by the reviewer in the TUI rather than
	// produced by the model: it publishes verbatim, without the body
	// template or the attribution footer.
	Manual bool `json:"manual,omitempty"`
}

type FindingState int

type LineRef struct {
	OldLine *int `json:"old_line,omitempty"`
	NewLine *int `json:"new_line,omitempty"`
}

type Request struct {
	// RepoPath is the checkout the review runs in (the subprocess cwd).
	RepoPath string
	// MR carries metadata shown to the model (title, description, branches).
	MR gitlabx.MRDetail
	// Diffs is the bounded, pre-filtered set of file diffs to review.
	Diffs []gitlabx.FileDiff
	// Commits are the MR's commits, shown to the model as context (and for
	// commit-message hygiene checks driven via Instructions).
	Commits []gitlabx.Commit
	// Template is the project's default MR description template, shown to the
	// model so instructions can drive a description-vs-template hygiene
	// check. Empty when the project has no template.
	Template string
	// Truncated lists files that were excluded or cut by the diff budget.
	Truncated []string
	// Instructions is extra prompt text: global then per-project.
	Instructions string
	// Categories to report on.
	Categories []Category

	Model        string
	Timeout      time.Duration
	MaxBudgetUSD float64
}

type Result struct {
	Summary   string
	Findings  []Finding
	Warnings  []string // dropped findings, truncation notes
	SessionID string
	CostUSD   float64
	Raw       []byte // raw output for drift debugging; persisted by the caller
}

type Reviewer interface {
	Name() string
	// CheckAvailable verifies the backend can run (binary present, version
	// supported) and returns a user-actionable error otherwise.
	CheckAvailable(ctx context.Context) error
	Review(ctx context.Context, req Request, onEvent func(Event)) (*Result, error)
}

type Severity string