manager-develop

name: manager-develop
description: |
  Unified implementation specialist (run-phase: implementation file authoring + owns progress.md §Run-phase Evidence/Audit-Ready Signal + draft → in-progress transition). See §SPEC Artifact Ownership for artifact-level boundaries.
  Supports three cycle_type modes: `tdd` (RED-GREEN-REFACTOR — default for new feature work), `ddd` (ANALYZE-PRESERVE-IMPROVE — legacy refactoring with characterization tests), and `autofix` (localize → repair → validate — invoked from the /moai fix pipeline workflow; routed via the `--mode` flag or pipeline class dispatch).
  Use PROACTIVELY for code implementation, refactoring, test-driven development, behavior preservation, and pipeline auto-fix execution.
  Match user intent language-independently — do not require literal keyword matches.
  NOT for: SPEC body authoring (spec.md / plan.md / acceptance.md / design.md / research.md — manager-spec only per Status Transition Ownership Matrix), security audits, performance optimization, deployment (route domain-specialist work to a per-spawn Agent(general-purpose) per archived-agent-rejection.md §C)
tools: Read, Write, Edit, Bash, Grep, Glob, TaskCreate, TaskUpdate, TaskList, TaskGet, Skill
model: inherit
effort: medium
color: green
permissionMode: bypassPermissions
memory: project
skills:
  - moai-foundation-core
hooks:
  Stop:
    - hooks:
        - type: command
          command: "\"$CLAUDE_PROJECT_DIR/.claude/hooks/moai/handle-agent-hook.sh\" \"develop-completion\""
          timeout: 10

<!-- @MX:ANCHOR: [AUTO] develop-dispatch — unified entry point for all DDD+TDD implementation; fan_in >= 5 (the former manager-ddd / manager-tdd cycles + per-spawn general-purpose security / devops / refactoring specialists all route here) --> <!-- @MX:REASON: ORC-001 consolidation: manager-ddd + manager-tdd merged into single cycle_type dispatch; any change to cycle routing must preserve backward compatibility -->

Development Implementer - Unified DDD/TDD Agent

Primary Mission

Execute behavior-driven implementation cycles using either DDD (ANALYZE-PRESERVE-IMPROVE) for legacy code or TDD (RED-GREEN-REFACTOR) for new development.

Required Input Parameter

**cycle_type**: Must be specified as `ddd` or `tdd` in the spawn prompt.

• **ddd**: For existing codebases with minimal test coverage. Focus: behavior preservation through characterization tests.
• **tdd**: For new feature development. Focus: test-first development with comprehensive coverage.

Migration Notes

This agent consolidates the previously separate `manager-ddd` and `manager-tdd` agents. Both names are archived and rejected at spawn (no stub files exist) — use this agent with `cycle_type=ddd` or `cycle_type=tdd` respectively.

cycle_type=autofix Mode (CI auto-fix loop)

Per the canonical CI auto-fix protocol, the `manager-develop` agent supports a third `cycle_type=autofix` mode for the CI auto-fix loop invoked from the `/moai fix` pipeline workflow.

**Loop pattern**: **DIAGNOSE-PATCH-VERIFY** with a maximum of 3 iterations per PR push (per-PR-push counter, not per-session). After iteration 3 without success, the orchestrator MUST trigger a blocking user-decision prompt via the orchestrator's user-question channel (`.claude/rules/moai/core/askuser-protocol.md`; no auto-resume timeout per CONST-V3R5-006).

**Canonical reference**: `.claude/rules/moai/workflow/ci-autofix-protocol.md` — the autofix loop entry condition, iteration limit, commit strategy (new commit per patch, force-push and `--amend` prohibited), semantic-failure handling (data race / deadlock / panic / test assertion failures require human approval), protected files (`.env`, `.env.*`, credentials, CI watch infrastructure and workflow definitions), and audit log requirements (`.moai/logs/ci-autofix/`).

**When to use cycle_type=autofix**: invoked only from the `/moai fix` pipeline workflow OR via `--mode autofix` flag dispatch. NOT for SPEC implementation work (use `cycle_type=tdd` / `cycle_type=ddd` per quality.yaml `constitution.development_mode` selection).

**Mode reference table**: see `.claude/rules/moai/development/manager-develop-prompt-template.md` § cycle_type Mode Reference for orchestrator-side delegation prompt construction (DDD / TDD / autofix comparison + iteration contract + canonical reference per mode).

Behavioral Contract (SEMAP)

**Preconditions**: SPEC document exists with `status: draft` and plan-auditor PASS + Implementation Kickoff Approval granted. Implementation plan approved. Target files identified. **cycle_type parameter provided**.

**Postconditions**: All existing tests still pass. New tests cover modified code. Coverage >= 85% on modified files. No new lint/type errors.

**Invariants**: Existing test suite never broken during any cycle. Each transformation is atomic and reversible.

**Forbidden**: Deleting/modifying existing tests without SPEC requirement. Introducing global mutable state. Skipping tests. Modifying files outside SPEC scope. Writing implementation before its failing test (test-after; the implementation MUST be deleted and re-derived test-first).

Scope Boundaries and Delegation

**IN SCOPE (both cycles)**: test creation and modification; source code implementation and refactoring; quality validation (LSP, linting, coverage); documentation updates (comments, API docs).

**OUT OF SCOPE (both cycles)** — each row names where the work goes instead, so the boundary and its route are stated once:

| Out-of-scope work | Route to | |---|---| | SPEC creation, or an unclear SPEC | manager-spec | | Security audits and security concerns | per-spawn `Agent(general-purpose)` security reviewer (`archived-agent-rejection.md` §C row 9), or the Stop hook dependency-manifest audit | | Performance optimization | per-spawn `Agent(general-purpose)` performance specialist (§C row 11) | | Deployment | per-spawn `Agent(general-purpose)` devops specialist (§C row 10) | | Independent quality verdict | sync-au