CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review

Perform Fallow's comprehensive pre-merge review. Use after implementation or when asked to review a branch, pull request, or diff.

Quality

87%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Low

Low-risk findings worth noting

SKILL.md
Quality
Evals
Security

Review

  1. Re-read AGENTS.md, the active plan, the current branch, and the complete diff against its intended base.
  2. Run the applicable gates in docs/development/quality-gates.md.
  3. Select reviewers using docs/development/review-routing.md.
  4. Review public contracts, all affected output formats, filters, integrations, generated surfaces, security boundaries, and companion parity.
  5. Check each open item of the PR body against the three groups in CONTRIBUTING.md. A defect or an unfinished part of the task under "Possible follow-ups" or "Known limits" is a BLOCK.
  6. When runtime behavior changed, run the behavior comparison on public projects from docs/development/quality-gates.md and explain every difference. When the change rejects input that was accepted before, also run the full analysis on a rejected input and read every finding, not only the diagnostic.
  7. Filter reviewer findings before you act on them:
    • trace the call site before you accept a bug claim;
    • drop style preferences and "a different design would also work";
    • check the diff or output that a reviewer cites, not its summary;
    • mark each accepted claim with its proof level. Record dismissed findings with a one-line reason in the verification notes. More than about five blocks usually means the filter is too weak.
  8. Classify each result as APPROVE, CONCERN, or BLOCK.
  9. Fix every block, rerun the blocking reviewer, and update verification.

Review the actual source and live output. Do not infer correctness from a green compile, a narrow unit test, or a previously cached report.

Repository

fallow-rs/fallow

Path

.agents/skills/review/SKILL.md

Last updated
First committed

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.