Skip to content

Refactor review skill as judge with report templates#1773

Closed
preinlein wants to merge 1 commit intomainfrom
paul.reinlein/review-skill-as-judge
Closed

Refactor review skill as judge with report templates#1773
preinlein wants to merge 1 commit intomainfrom
paul.reinlein/review-skill-as-judge

Conversation

@preinlein
Copy link
Contributor

@preinlein preinlein commented Feb 17, 2026

What does this PR do?

Refactors the review skill to act as the "judge" — it runs post-change benchmarks, performs 5-persona review, and returns structured YAML reports via templates. Removes review's own db.yaml (review no longer records results; hunt does).

Motivation

Clearer separation of responsibilities between hunt (coordinator/recorder) and review (judge). Previously both skills recorded results independently, leading to duplicated data and unclear ownership.

Related issues

N/A

Additional Notes

  • Adds 4 report templates: approved, rejected, duplicate, bug-found
  • Removes assets/db.yaml and assets/db/ from review (data consolidated into hunt's db)
  • Review now has benchmark tool permissions (cargo, hyperfine, etc.) since it runs post-change benchmarks

@preinlein preinlein requested a review from a team as a code owner February 17, 2026 21:53
Copy link
Contributor Author

preinlein commented Feb 17, 2026

@preinlein preinlein changed the title Refactor review as judge with templates Refactor review skill as judge with report templates Feb 17, 2026
@preinlein preinlein closed this Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant