| name | review-task |
| description | Review a completed implementation task before push/PR. Validates issue-spec acceptance criteria, scope, tests, docs, and lint readiness on a local branch. Use when asked to verify issue completion without an open PR. |
| metadata | {"author":"torrust","version":"1.0"} |
Reviewing A Task (Pre-PR)
Use this skill when there is no pull request yet and the goal is to verify that implementation for
an issue/task is complete and ready to be pushed.
Preconditions
- An issue spec exists (typically under
docs/issues/open/).
- Local changes are available on the branch.
- No PR review workflow is required yet.
Workflow
- Read the issue spec and extract acceptance criteria.
- Map each criterion to concrete evidence in changed files/tests.
- Run relevant validation checks (
linter all minimum, plus focused tests when applicable).
- Classify each criterion as
PASS, FAIL, or PENDING.
- Update only verified checklist items in the issue spec.
- Report pass/fail with remediation for any gaps.
Task Review Checklist
Scope And Criteria
Verification
Spec Hygiene
Output
Return:
- Scope reviewed
- Acceptance criteria matrix (
PASS/FAIL/PENDING + evidence)
- Blocking findings
- Issue spec updates made
- Overall result (
REVIEW PASSED or REVIEW FAILED)
Not In Scope
- Reviewing an open pull request (use
review-pr for that).
- Publishing review comments to a PR.
- Merging or closing PRs.