fix

Fix Phase

You are running the fix phase. Your job is to fix all issues the human marked as "fix" in the review output — efficiently and correctly.

Step 1 — Load context

Read the triaged review output: tasks/sprints/sprint-*/review-output.md. Read tasks/lessons.md — avoid repeating past fix mistakes.

Identify all items marked as "fix" (or "critical" / "major" that weren't explicitly deferred).

Step 2 — Fix issues

For each issue to fix:

1. Understand the issue — read the finding, the file, and the context.
2. Plan if complex — if the fix touches 3+ files, switch to plan mode first to design the approach.
3. Implement the fix — make the minimal change that resolves the issue. Don't refactor unrelated code.
4. Run tests — verify the fix works and doesn't break anything else.
5. Document — note what you did in the fix output.

If a fix introduces a new issue or breaks something: stop, assess, and fix the regression before moving on.

Investigation-shaped findings cannot be deferred

If a review finding requires a quick investigation to diagnose (a grep to confirm a code path exists, a read to verify the production shape of an input, a 5-second check on whether a function already handles the case), do that investigation now — not later.

Specifically: do not mark such a finding as "deferred to claude -p investigation" during the fix phase. Treat the deferral pattern as an anti-pattern: the investigation it points to is almost always shorter than the deferral message itself.

Decision rule:

• Investigation < 15 minutes (a grep, a read, a short script) → do it inside the current fix phase, before declaring the finding resolved or deferred.
• Investigation ≥ 15 minutes (cross-cutting refactor, semantic ambiguity, requires architectural review) → escalate to the human with explicit scope, do not silently defer.

The deferral pattern hides Critical findings under the guise of "need to investigate", lets the sprint close, and the bug ships. Closing a finding requires either a fix or a deliberate, scope-explicit escalation — never a vague "investigation later".