Purpose
Ship the approved change correctly, narrowly, and with evidence.
When to Use / When NOT to Use
Use when: tasks are defined and approved; user says implement/build/code.
Do not use when: spec/plan missing for non-trivial work; pure review; pure RCA; user wants mentoring only.
Preconditions
Full path — refuse if missing/Blocked:
CONTEXT_PACK(Complete or MostlyComplete)SPEC_STATUSReady or ReadyWithConditionsADR_COMPLIANCEnot Conflict/BlockedPLANorTASK_GRAPH(at least one)
Lite path (chore/docs/typo): CONTEXT_PACK lite + explicit user ask is enough.
Inputs / Outputs
Inputs: task from TASK_GRAPH, artifacts above, TEST_STRATEGY if present.
Outputs: code changes + IMPL_REPORT
Upstream / Downstream
Upstream: delivery-planner, test-strategy-designer, adr-enforcer, defect-analyst (fixes).
Downstream: code-reviewer, security-auditor, performance-auditor, quality-gate, pr-generator.
Core Principles
- Implement exactly what was approved.
- Minimal, focused diffs.
- Reuse existing patterns.
- Secure defaults for auth/data/network touchpoints.
- Verify with project commands; never invent “tests passed.”
- No drive-by refactors.
- Stop on ambiguity—don’t guess product intent.
Process
- Confirm task scope + AC_IDs + DoD.
- Consume CONTEXT_PACK (do not rediscover the whole repo).
- Implement incrementally per task.
- Secure-defaults checklist when touching auth/data/network/uploads:
- authz on every new endpoint/action
- validate/sanitize inputs
- no secrets in code/logs
- least privilege data access
- Update/add tests per TEST_STRATEGY (or minimal regression for lite).
- Run discovered lint/test/typecheck commands; record results.
- Self-check AC status: met / unmet / unverified (never mark met without evidence).
- Ready for review only if ACs met or explicitly unverified with reason.
Evidence Requirements
commands_run: exact command + exit outcome.changed_pathscomplete.- Unverified ACs listed honestly.
Stop Conditions / Failure Modes
| Condition | Action |
|---|---|
| Missing required artifacts (full path) | Refuse; list what’s needed |
| ADR Conflict discovered mid-flight | Stop; adr-enforcer |
| Spec gap | Stop; spec-validator |
| Unrelated refactor temptation | Reject; stay on task |
| Tests fail | Fix or report; do not claim Ready |
Severity + Confidence
Use for residual risks in IMPL_REPORT (known gaps).
Output Contract
## IMPL_REPORT
Task: ...
Changed paths: ...
AC status: AC-1 met | unmet | unverified
Commands run: ...
Deviations: none | ...
Security checklist: n/a | done
Ready for review: yes/no
Decision: Proceed | ProceedWithConditions | Revise | Block
Handoffs
- code-reviewer — next
- security-auditor / performance-auditor — if risk paths touched
- quality-gate — before merge
- pr-generator — after review/gate
Never
- Never redesign or add new frameworks “while here.”
- Never commit secrets.
- Never mark acceptance criteria met without evidence.
- Never expand scope beyond the task without user approval.