Skip to content

Add /review-workflow skill#2

Open
hefgi wants to merge 4 commits into
mainfrom
feat/review-workflow-skill
Open

Add /review-workflow skill#2
hefgi wants to merge 4 commits into
mainfrom
feat/review-workflow-skill

Conversation

@hefgi

@hefgi hefgi commented Jul 3, 2026

Copy link
Copy Markdown
Owner

What

Adds a new project-agnostic skill /review-workflow to the collection: an iterative review → fix → commit → re-review loop that runs until a code review comes back clean.

How it works (two actors)

A Claude Code workflow can orchestrate review subagents but cannot edit the working tree or commit. So the loop is split:

  • Bundled workflow (review-workflow/references/review-loop.mjs) — the per-round review engine: scope resolution → parallel dimension reviewers → adversarial per-finding verification → structured, severity-graded findings + counts. Runs from the skill's bundled path (nothing is copied into the user's repo).
  • Skill (SKILL.md) — the main agent drives the outer loop: call the workflow, apply fixes for confirmed critical/major/minor findings (nits opportunistically), commit one Address code review feedback (round N) per round, then re-review. Stops when the review is clean, on a no-progress guard, or after 8 rounds.

Configurable model ladder (cost-efficient)

  • Mechanical work (scope, file listing) runs on Haiku.
  • The review ladder is chosen at invocation — default Sonnet → Opus (cheap first pass, smart final sign-off); also sonnet, opus, or an explicit models=… list.
  • Each round's verify phase matches its review tier; the no-progress guard resets per tier.

Reviewer agent

Prefers the code-reviewer agent (shipped by Claude Code's official review plugins) and falls back to the default workflow subagent when unavailable, so it runs anywhere.

Registration

  • New review-workflow/ skill folder (SKILL.md, tile.json, bundled workflow).
  • Registered in .claude-plugin/marketplace.json, the README skills table, and the tile-publish GitHub Action.

hefgi added 4 commits July 3, 2026 16:02
Mechanical phases (scope, file-listing) run on Haiku; the review ladder
converges on Sonnet then re-converges on Opus for the final sign-off, with
each round's verify matching its review tier. The no-progress guard resets
per tier so a smarter model surfacing new findings counts as progress.
Accept a preset (sonnet-opus, sonnet, opus) or an explicit models= list; when
none is given, ask the user with Sonnet -> Opus as the recommended default,
plus Sonnet-only and Opus-only. The chosen list becomes the review tiers.
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