wa:develop-review

Validate feature-branch task completion, AC-ID coverage, tests, lint, and TODO/FIXME absence.

Updated Mar 26, 2026
One-click install
npx skills add https://github.com/jeremy-newhouse/eck-e2e-weather-app --skill wa-develop-review
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: wa:develop-review
Source: https://github.com/jeremy-newhouse/eck-e2e-weather-app/tree/main/.claude/skills/develop-review
Command: npx skills add https://github.com/jeremy-newhouse/eck-e2e-weather-app --skill wa-develop-review

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

It prevents incomplete or low-quality feature work from advancing by verifying that tasks and acceptance criteria are fully covered, tests and lint pass, simplification is done when required, and no new TODO/FIXME items remain.

Core Features & Use Cases

  • Task and AC-ID coverage verification: Ensures every task is committed and every active acceptance criterion is addressed by at least one task.
  • Quality gates for confidence: Runs the resolved test suite and linter, and flags any failures as part of the gate.
  • Change hygiene enforcement: Detects newly introduced TODO/FIXME comments in changed files and validates PR phase expectations via lifecycle recording.

Use Case: Before moving from the Develop phase to the Validate phase, you can run this gate to confirm that a feature branch has complete commit coverage, 100% AC-ID mapping, clean tests/lint, and no lingering TODO/FIXME issues.

Quick Start

Run wa:develop-review with a short description of your feature to generate docs/{feature}/DEVELOP-REVIEW.md and determine whether you can proceed to /eck:validate.

Frequently Asked Questions about wa:develop-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I enforce acceptance criteria coverage before merging a feature branch?

Acceptance criteria coverage is enforced by verifying every active AC-ID is addressed by at least one committed task. This quality gate checks task completion against your FRD.md and TASKS.md files to ensure full coverage before advancing.

How do I prevent new TODO comments from being introduced in changed files?

Preventing new TODO comments involves scanning git diffs relative to a base branch to detect any newly introduced TODO or FIXME items. This change hygiene enforcement flags lingering issues as part of the develop review gate.

What's the best way to gate feature changes with lint and test validation?

Gating feature changes requires running the resolved project test suite and linter, then classifying any failures as blocking or advisory based on dev rigor level. The gate produces a DEVELOPMENT-REVIEW artifact recording the final verdict.

Do I need docs/{feature}/TASKS.md and FRD.md files to run a develop review?

Yes, you need TASKS.md and FRD.md files under docs/{feature} to run a develop review. The gate inspects these files to verify task completion, AC-ID coverage, and validate PR phase expectations for feature-branch workflows.

How does a develop review gate check git history and diffs?

A develop review gate inspects git history and diffs relative to a base branch to confirm every task is committed and detect new TODO/FIXME items. It validates commit coverage and change hygiene before recording the lifecycle verdict.

Can I classify quality gate checks as advisory instead of blocking?

Yes, quality gate checks can be classified as advisory or blocking. The gate reads the dev rigor level to determine severity, ensuring non-critical lint or test issues can be flagged without fully blocking the transition to the validate phase.