mcpField
P
piv-fix-review-findings

piv-fix-review-findings

github.com/coleam00Development722228 installsUpdated 3w ago
Repository

About piv-fix-review-findings

A skill for handling code review findings by triaging them into fixes, deferrals, manual checks, or noise before making changes. It is for developers who want to keep PR scope deliberate rather than treating every finding as an automatic work order.

Key features

  1. 01Triage review findings first
  2. 02Decide fixes or deferrals by scope
  3. 03Fix issues individually and add tests
  4. 04Run piv-validate
  5. 05Commit and push PR changes

Use cases

  1. 01Choose which findings belong in the current PR
  2. 02Log deferred findings as tracker issues
  3. 03Fix review findings and verify them with tests

Install

Copy the command and run it in a terminal.

$npx skills add https://github.com/coleam00/skills --skill piv-fix-review-findings

After installing, type/piv-fix-review-findingsin Claude Code to confirm it loaded.

Details

From the project README

Fix Review Findings

A review produced findings — but a review is input, not a work order. You decide what happens to each one.

Code-review (file or description of issues): $review

Direction / scope (what to fix now vs defer): $scope

If the Code-review is a file, read the entire file first so you understand every finding before triaging.

1. Triage first (the human's call)

Sort the findings before touching code. Honor any direction in the scope argument; if it's unclear, surface the findings grouped and ask rather than fixing everything by default:

  • Fix now (this PR) — real, in-scope, belongs with this change.
  • Defer / log as an issue — real but later; don't bloat this PR. Create a tracker issue (or note it) instead of fixing it here.
  • Needs a human look / manual test — anything you should inspect or test by hand before trusting it. Flag it, don't silently auto-fix.
  • Noise / won't-fix — say why, then drop it.

Don't let the reviewer dictate scope — "real, but later" is a valid and common call; a clean small PR beats a sprawling one.

2. Fix the "fix now" set — one at a time

For each:

  1. Explain what was wrong.
  2. Make the fix.
  3. Create and run a test that proves it.

3. Validate

Run the piv-validate skill to finalize the fixes.

4. If operating on a PR — commit and push

If these fixes are on a PR branch, commit them (use piv-commit) and push so the PR reflects the fixes and the review can re-run on the updated PR. If nothing was fixed (everything deferred), there's nothing to push — just make sure the deferred items are logged as issues.

Output

A short report: what was fixed (with its test), what was deferred/logged (with issue refs), what needs a manual look/test — and, if on a PR, the pushed commit + confirmation the PR is updated.

Comments

Markdown supported