Paths: File paths (references/, ../ln-*) are relative to this skill directory.
Type: L3 Worker
Category: 4XX Execution
Task Reviewer
MANDATORY after every task execution. Reviews a single task in To Review and decides Done vs To Rework with immediate fixes or clear rework notes.
This skill is NOT optional. Every executed task MUST be reviewed immediately. No exceptions, no batching, no skipping.
Purpose & Scope
Resolve task ID (per Input Resolution Chain); load full task and parent Story independently via the configured tracker provider (getTask, getStory).
Check architecture, correctness, configuration hygiene, docs, and tests.
For test tasks, verify risk-based limits and priority (≤15) per planner template.
Update only this task: accept (Done) or send back (To Rework) with explicit reasons and fix suggestions tied to best practices.
Inputs
related skills
semantically similar in the cross-vendor index
don't have the plugin yet? install it then click "run inline in claude" again.
+clear scope boundary - single task review, no batching, immediate decision
+explicit decision criteria present (done vs rework with reasons)
+acknowledges tracker provider abstraction and independent parent story load
weaknesses
~procedure stops mid-sentence at inputs section - actual step sequence is absent, leaving unclear how to parse tests, architecture checks, or hygiene validation
~trigger phrases generic (review a task, check architecture) - no specifics on which tracker types, how to validate test coverage, or code standard framework
~edge case coverage thin - no mention of task load failures, provider timeout, or what happens when parent story is missing or contradicts task
