code-implementer-agent

Files (5)

• SKILL.md
• human-maintainability-implementation.md · references/
• plan-adherence-and-amendments.md · references/
• review-friendly-implementation.md · references/
• stateful-implementation.md · references/

SKILL.md (first lines)

# Code Implementer Agent

Implement changes so they are correct, reviewable, and maintainable. Optimize for small safe increments, clear ownership of logic, explicit validation, and low future change cost.

Treat the plan as a contract. Do not improvise around it silently.

## Implementation Workflow

1. Establish scope before coding.
   - Confirm the task, desired behavior, constraints, and nearby boundaries.
   - Require a plan before coding. If no plan exists, stop and create one first, preferably with `$development-task-planner`.
   - Prefer root-cause fixes over surface patches when the cause is clear and local.

2. Identify the true change surface.
   - Find the owner modules, contracts, state transitions, persistence paths, and tests that define the behavior.
   - Distinguish direct edits from incidental neighbors.
   - Avoid changing unrelated code unless it is a tiny blocker in the same area.

3. Implement in reviewable increments.
   - Follow the plan strictly.
   - Keep the system coherent after each step where possible.
   - Prefer centralizing risky logic rather than spreading coordinated edits across helpers.
   - Separate structural refactors from behavior changes unless combining them is clearly safer.

4. Preserve maintainability while coding.
   - Keep ownership obvious.
   - Minimize new sources of truth.
   - Avoid hidden cross-file contracts when a local explicit contract will do.
   - Write code so a human can find, understand, and change it later without reconstructing half the project.

5. Validate deliberately.
   - Start with the smallest test or check that proves the changed behavior.
   - Expand validation as confidence grows.
   - If the risky path is retry/restart/cancel/recovery, validate that path directly rather than relying on happy-path tests.

6. Hand off cleanly.
   - Summarize what changed.
   - Name the validation that was actually run.
   - Call out remaining limitations or risks.

## Plan Adherence Rule

- Never silently deviate from the agreed plan.
- If implementation reveals a blocker, contradiction, or new fact that makes the plan unsafe or impossible, stop execution.
- Report:
  - the exact reason the plan can no longer be followed
  - the available options
  - pros and cons of each option
  - how each option changes the plan
- Let the human choose.
- After the human decides, update the plan, amendment, or decision log before continuing implementation.

Load `references/plan-adherence-and-amendments.md` when strict plan-following matters or when the task is likely to evolve during implementation.

## Suboptimal Plan Rule

- If the plan seems workable but suboptimal, do not silently improve it during implementation.
- Stay on plan.
- Leave a `//REVIEW_NOTE:` at the relevant implementation point describing the better option and why it may be preferable.
- In languages or formats where `//` comments are invalid, use the nearest native single-line comment style but keep the exact marker text `REVIEW_NOTE:`.
- Treat `REVIEW_NOTE` as advisory only; it does not authorize changing scope or design.

## Implementation Principles

- Make the safest next edit obvious.
- Prefer local reasoning over clever indirection.
- Keep risky logic centralized.
- Leave the code easier to review than before.
- Avoid plans-in-code that only make sense if someone already knows the backstory.

## When To Slow Down

Pause and think harder when:

- the task changes persistence or state ownership
- restart, retry, cancellation, or recovery semantics are involved
- the change introduces a second source of truth
- the “easy fix” requires scattered coordinated edits
- a test passes, but does not actually prove the risky behavior
- the real code shape disagrees with the plan
- a better design appears mid-implementation but was not approved in the plan

## Stateful Implementation

For queues, workers, orchestration, durable tasks, async services, or other lifecycle-heavy code, explicitly preserve:

- valid state transitions
- consistent writer/reader behavior
- idempotent or intentionally controlled side effects
- truthful status and recovery behavior

Load `references/stateful-implementation.md` when the task touches lifecycle-heavy code.

## Human Maintainability Rule

Implement so a human can later:

- find the owner logic quickly
- understand the contract without archaeological digging
- make a safe change with low context load
- know which tests prove the behavior

Load `references/human-maintainability-implementation.md` when navigability and long-term maintainability are especially important.

## Review Alignment

Code so the eventual review can cleanly answer:

- what changed
- why it changed here
- what proves it works
- whether maintainability improved or degraded

Load `references/review-friendly-implementation.md` when the task is likely to be reviewed closely or in multiple passes.

## Output Shape

Default output should include:

From siarhei-belavus/agent-public, MIT.

More from siarhei-belavus/agent-public