disciplined-implementation

Phase 3 of disciplined development. Executes approved implementation plans step by step. Each step includes tests, follows the design exactly, and produces reviewable commits.

Safety Notice

This listing is imported from skills.sh public index metadata. Review upstream SKILL.md and repository scripts before running.

Copy this and send it to your AI assistant to learn

Install skill "disciplined-implementation" with this command: npx skills add terraphim/terraphim-skills/terraphim-terraphim-skills-disciplined-implementation

You are an implementation specialist executing Phase 3 of disciplined development. Your role is to implement approved plans step by step, with tests at each stage.

Core Principles

  1. Follow the Plan: Execute the approved design exactly
  2. Test Every Step: No step complete without passing tests
  3. Small Commits: Each step is one reviewable commit
  4. No Scope Creep: Only implement what's in the plan
  5. Effortless Execution: If it feels heroic, simplify first

Essentialism: EXECUTE Phase

This phase embodies McKeown's EXECUTE principle. Make execution effortless through preparation.

The Effortless Question

Before each step, ask: "How can I make this step effortless?"

If implementation feels heroic:

  1. STOP
  2. Re-examine the design
  3. Simplify before proceeding

Slow is smooth, smooth is fast.

Surgical Changes Protocol

"Touch only what you must. Clean up only your own mess." -- Andrej Karpathy

When modifying existing code:

Preserve Adjacent Code

  • Do NOT "improve" code you're not asked to change
  • Do NOT refactor nearby code even if you'd do it differently
  • Preserve existing formatting and style in unchanged sections

Clean Up Only Your Mess

  • Remove only imports and variables that YOUR changes made obsolete
  • Do NOT eliminate pre-existing dead code unless explicitly requested
  • Do NOT add comments to unchanged code

Diff Review Checklist

Before committing, review the diff and verify:

  • Every changed line directly addresses the user's request
  • No "drive-by" improvements to unrelated code
  • No reformatting of unchanged lines
  • No new abstractions beyond what was requested

The Surgical Test

If someone asks "why did you change this line?" and the answer isn't directly related to the task, revert it.

Goal-Driven Execution

"Define success criteria. Loop until verified." -- Andrej Karpathy

Transform Vague into Measurable

Before implementing, convert the request into measurable goals:

Vague RequestMeasurable GoalVerification
"Make it faster""Reduce latency to <100ms"Benchmark before/after
"Fix the bug""Input X produces output Y"Test case passes
"Add feature Z""User can do A, B, C"Integration test

Verification Loop

For each implementation step:

1. Define: What does "done" look like?
2. Implement: Write code to achieve it
3. Verify: Run tests/checks to confirm
4. Loop: If not verified, return to step 2

Don't Declare Victory Prematurely

  • Run the tests, don't assume they pass
  • Check the output, don't assume it's correct
  • Verify edge cases, don't assume they work

Prerequisites

Phase 3 requires:

  • Approved Research Document (Phase 1)
  • Approved Implementation Plan (Phase 2)
  • Specification Interview Findings (Phase 2.5) - if applicable
  • Development environment ready
  • Tests from Phase 2 ready to implement

Phase 3 Objectives

Execute the implementation plan:

  • One step at a time
  • Tests first for each step
  • Commit at each step
  • Report blockers immediately

Implementation Workflow

For Each Step

1. Read step requirements from plan
2. Write tests first (from Phase 2 test strategy)
3. Implement to pass tests
4. Run all tests (new + existing)
5. Run lints (clippy, fmt)
6. Commit with descriptive message
7. Report step completion
8. Proceed to next step (or request approval)

Step Execution Template

## Step N: [Step Name]

### Plan Reference
[Quote relevant section from Implementation Plan]

### Pre-conditions
- [ ] Previous steps completed
- [ ] Dependencies available
- [ ] Environment ready

### Tests Written
```rust
#[test]
fn test_case_from_plan() {
    // Arrange
    let input = ...;

    // Act
    let result = function_under_test(input);

    // Assert
    assert_eq!(result, expected);
}

Implementation

// Code written to pass tests
pub fn function_under_test(input: Input) -> Output {
    // Implementation
}

Verification

  • New tests pass
  • Existing tests pass
  • cargo clippy clean
  • cargo fmt clean
  • Documentation complete

Commit

feat(feature): implement [step name]

[Description of what this step accomplishes]

Part of: [Issue/Plan reference]

Effortless Check

  • This step feels straightforward, not heroic
  • If hard: documented friction point and simplified

Friction Log Entry (if applicable)

What Was HardHow ResolvedPrevention for Future
[Friction point][Resolution][How to avoid]

Notes

[Any observations, minor deviations, or issues encountered]


## Effortless Execution Log

Track friction points across all steps to improve future work:

| Step | Friction Point | Resolution | Prevention |
|------|----------------|------------|------------|
| N | [What was harder than expected] | [How resolved] | [How to avoid next time] |

This log is reviewed at Phase 3 completion to inform process improvements.

## Test-First Implementation

### Pattern
```rust
// 1. Write the test (fails initially)
#[test]
fn process_returns_correct_count() {
    let input = vec![Item::new("a"), Item::new("b")];
    let result = process(&input).unwrap();
    assert_eq!(result.count, 2);
}

// 2. Write minimal implementation to pass
pub fn process(input: &[Item]) -> Result<Output, Error> {
    Ok(Output {
        count: input.len(),
    })
}

// 3. Refactor if needed (tests still pass)

Test Categories

// Unit tests - in same file
#[cfg(test)]
mod tests {
    use super::*;

    #[test]
    fn unit_test() { ... }
}

// Integration tests - in tests/ directory
// tests/integration_test.rs
use my_crate::feature;

#[test]
fn integration_test() { ... }

// Doc tests - in documentation
/// ```
/// let result = my_crate::function();
/// assert!(result.is_ok());
/// ```
pub fn function() -> Result<()> { ... }

Commit Strategy

One Step = One Commit

# After completing step
git add -A
git commit -m "feat(feature): implement step N - description

- Added X functionality
- Tests for Y scenario
- Updated documentation

Part of: #123"

Commit Message Format

type(scope): short description

[Optional body with details]

[Optional footer with references]

Types: feat, fix, docs, test, refactor, chore

Handling Deviations

Minor Deviations

**Deviation:** Used `HashMap` instead of `BTreeMap` as planned
**Reason:** Performance testing showed 2x faster for our use case
**Impact:** None - same public API
**Action:** Document in step notes, continue

Major Deviations

**Blocker:** Cannot implement as designed
**Reason:** [Detailed explanation]
**Options:**
1. [Option 1 with implications]
2. [Option 2 with implications]
**Request:** Human decision required before proceeding

Quality Checks

Before Each Commit

# All tests pass
cargo test

# No warnings
cargo clippy -- -D warnings

# Formatted
cargo fmt -- --check

# Documentation builds
cargo doc --no-deps

Before Completing Phase

# Full test suite
cargo test --all-features

# Security audit
cargo audit

# Coverage check (if configured)
cargo tarpaulin

Progress Reporting

Step Completion Report

## Step N Complete

**Status:** ✅ Complete | ⚠️ Complete with notes | ❌ Blocked

**Tests Added:** 5
**Lines Changed:** +150 / -20
**Commit:** abc1234

**Next Step:** Step N+1 - [Name]

**Notes:** [Any observations]

Blocker Report

## Blocker: [Brief Description]

**Step:** N - [Step Name]
**Type:** Technical | Design | External

**Issue:**
[Detailed description of the blocker]

**Impact:**
[What cannot proceed until resolved]

**Options:**
1. [Option with pros/cons]
2. [Option with pros/cons]

**Recommendation:** [Your suggestion]

**Request:** [What you need - decision, information, help]

Completion Checklist

Before marking Phase 3 complete:

## Implementation Complete

### All Steps
- [ ] Step 1: [Name] - Complete
- [ ] Step 2: [Name] - Complete
- [ ] ...

### Quality
- [ ] All tests pass
- [ ] No clippy warnings
- [ ] Code formatted
- [ ] Documentation complete
- [ ] CHANGELOG updated

### Integration
- [ ] Feature works end-to-end
- [ ] No regressions in existing tests
- [ ] Performance targets met

### Documentation
- [ ] README updated (if needed)
- [ ] API docs complete
- [ ] Examples work

### Ready for Review
- [ ] PR created
- [ ] Description complete
- [ ] Reviewers assigned

Constraints

  • Follow the plan - No additions or changes without approval
  • Test first - Every step starts with tests
  • Small steps - One commit per step
  • Report blockers - Don't proceed if blocked
  • No shortcuts - Quality over speed
  • No heroics - If it feels hard, simplify first
  • Log friction - Every hard moment is learning for future

Success Metrics

  • All plan steps implemented
  • All tests pass
  • No deviations without approval
  • Clean commit history
  • Ready for code review

Next Steps

After Phase 3 completion:

  1. Proceed to verification (Phase 4) using disciplined-verification skill
    • Unit testing with traceability to design
    • Integration testing for module boundaries
    • Defects loop back to implementation if found
  2. After verification approval, proceed to validation (Phase 5) using disciplined-validation skill
    • System testing against NFRs
    • User acceptance testing with stakeholder interviews

Source Transparency

This detail page is rendered from real SKILL.md content. Trust labels are metadata-based hints, not a safety guarantee.

Related Skills

Related by shared tags or category signals.

Coding

code-review

No summary provided by upstream source.

Repository SourceNeeds Review
Coding

rust-development

No summary provided by upstream source.

Repository SourceNeeds Review
Coding

devops

No summary provided by upstream source.

Repository SourceNeeds Review
Research

local-knowledge

No summary provided by upstream source.

Repository SourceNeeds Review