| name | solution-review |
| description | Review already-applied code changes against the original task statement or task folder artifacts under `tmp/tasks/<task-name>/`. Use when implementation work is done and Codex must verify scope coverage, detect mismatches or regressions, assess verification gaps, and write a review summary before handoff or merge. |
Solution Review
Use this skill when code has already been changed and the next step is to assess whether the implementation actually matches the task.
Workflow
Follow the canonical workflow in ../../../.memory-bank/workflows/solution-review.md.
Required Guardrails
- Review against the original task source of truth first:
task.md, solution_X.md, implementation-plan-X.md, coding-task-X.md, or the user-provided task statement.
- Inspect the actual applied change, not just the final code shape. Read the relevant diff, changed files, and nearby tests.
- Treat this as a post-implementation review step, not a chance to silently finish missing work.
- Separate confirmed findings, uncertainties, and verification gaps.
- Prefer concrete evidence with file paths, acceptance-criteria mapping, and commands that were or were not run.
- Write the review artifact into the existing task folder as
review.md when the task lives under tmp/tasks/<task-name>/.
Minimal Starting Reads
Start with:
../../../.memory-bank/project-context.md../../../.memory-bank/architecture.md../../../.memory-bank/gotchas.md../../../tmp/tasks/<task-name>/task.md if present../../../tmp/tasks/<task-name>/solution_*.md if present../../../tmp/tasks/<task-name>/implementation-plan*.md if present../../../tmp/tasks/<task-name>/coding-task*.md if present
Then read the current diff, changed files, tests, configs, and any user-provided task statement needed to judge whether the implementation is complete and correct.
