pal-e-docs

Skill: Fix Review

skill-fix-review Skill

active skill
Project

Skill: Fix Review

Dev Agent workflow for addressing QA review findings. Reads PR comments, applies fixes, pushes, and comments explaining changes.

Steps

  • Read the review — mcp__forgejo__review_pr(owner="forgejo_admin", repo="<repo>", pr_number=N). Read the QA Agent's structured review — focus on BLOCKERS first, then NITS.
  • Read the plan — mcp__pal-e-docs__get_note(slug="<plan-slug>"). Confirm fixes stay within the phase scope — don't introduce scope creep while fixing.
  • Apply fixes — Address each BLOCKER. Fix NITS where reasonable. Keep changes minimal and focused on the findings.
  • Run tests — All tests must pass after fixes.
  • Push — Push fixes to the existing PR branch. Do not create a new PR.
  • Comment on PR — mcp__forgejo__comment_on_pr(owner="forgejo_admin", repo="<repo>", pr_number=N, body="<response>"). Explain what was fixed, what was deferred, and why. Reference specific BLOCKER/NIT items.
  • Report and stop — Tell the user fixes are pushed. Do not merge. Do not request re-review (user decides).

MCP Tools

Step Tool Purpose
1 <code>mcp__forgejo__review_pr</code> Read PR diff and review comments
2 <code>mcp__pal-e-docs__get_note</code> Read plan for scope
2 <code>mcp__pal-e-docs__list_notes</code> Discover relevant conventions
6 <code>mcp__forgejo__comment_on_pr</code> Post fix explanation

Agent

agent-dev
  • skill-review-pr — QA Agent produces the findings this skill addresses
  • skill-implement-phase — the original implementation skill
  • pr-review-loop — the cycle: implement → review → fix → re-review
  • agent-dev — the agent that runs this skill