cvg-plan-review-feedback

Plan Review Feedback

Use this skill in the planner session after a plan reviewer returns blocker findings.

The input is the reviewer blocker findings appended to the prompt, plus the original planning goal, source prompt, current plan, behavior contract if present, and relevant codebase evidence.

Do not treat reviewer findings as an edit list. First decide whether each finding is valid under the accepted goal and current code reality, then revise only plan-owned artifacts.

If the task context provides no coordination channel (standalone use), the coordination channel is the user: stop and ask directly.

Process

1. Reload planning authority

Read:

• original user goal, issue, acceptance criteria, and non-goals,
• current plan document,
• behavior contract or linked design notes, if present,
• relevant codebase evidence for each reviewer finding,
• exact reviewer blocker findings.

2. Evaluate each blocker

For each finding, decide whether it is:

• Valid plan gap: the plan misses a required surface, lifecycle, invariant, matrix cell, ownership boundary, or verification gate.
• Source-backed contract gap: the source authority defines the behavior, but the plan or contract failed to capture it.
• Decision-required contract gap: neither the plan nor source authority defines the behavior clearly enough to proceed.
• Reviewer clarification needed: the finding cannot be mapped to concrete plan or code evidence after re-reading the relevant context.
• Invalid or out of scope: the finding is stale, already covered, contradicts the accepted goal, or asks for work outside the agreed scope.

3. Revise only planner-owned artifacts

For valid plan gaps and source-backed contract gaps:

• update the plan, contract, surfaces, invariant matrix, slices, or verification gates as needed,
• keep the revision scoped to the accepted goal,
• preserve reviewer finding ids or exact wording so re-review can check them.

For decision-required contract gaps: