done

Validate coding changes against tests, linting, builds, and execution evidence.

8|1|Updated Jul 2, 2026
One-click install
npx skills add https://github.com/duthaho/skillhub --skill done-duthaho
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: done
Source: https://github.com/duthaho/skillhub/tree/main/.claude/skills/done
Command: npx skills add https://github.com/duthaho/skillhub --skill done-duthaho

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill prevents agentic coding from shipping changes that merely look finished by requiring quoted evidence that the work was run, tested, and reviewed before a commit or PR is drafted.

Core Features & Use Cases

  • Evidence-driven release gate: Checks the full test suite, lint or typecheck, build, and an actual run of the change rather than relying on assertions.
  • Independent review workflow: Uses fresh-context review stages to assess spec compliance and behavioral correctness from the diff and intent alone.
  • Safety tripwires: Flags fake-green tests, source-security risks, and other blockers such as edited tests, skipped checks, hardcoded expectations, or destructive patterns.
  • Use Case: After finishing a feature branch, run this Skill to decide whether the change is truly ready to ship, needs fixes, or requires human judgment.

Quick Start

Use the done skill to verify the current diff against its spec, run the required checks, and tell me whether it is safe to ship.

Frequently Asked Questions about done

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

FAQPage Schema
How do I verify a code change is ready to ship with real evidence?

To verify a code change is ready to ship, you need an evidence-driven release gate that runs tests, linting, builds, and a real execution of the change rather than relying on assertions. This validates behavioral proof and diff hygiene.

What is a fake-green test and how do I catch it before releasing?

A fake-green test is a check that passes falsely due to edited tests, skipped checks, or hardcoded expectations. You catch it before releasing by using safety tripwires that detect these destructive patterns and flag source-security risks during diff inspection.

How do I perform a fresh-context review on a feature branch diff?

To perform a fresh-context review on a feature branch diff, you assess spec compliance and behavioral correctness using only the diff and intent. This independent review workflow ensures the change meets requirements without relying on prior context.

Can I use automated linting and typecheck results as proof for a release gate?

You can use linting and typecheck results as part of a release gate, but they are not sufficient alone. A robust release gate requires a full test suite, successful builds, and an actual run of the change to provide genuine behavioral proof.

When should I require diff inspection before drafting a pull request?

You should require diff inspection before drafting a pull request whenever finishing a feature branch or bug fix. It ensures the change is reviewed for spec compliance, behavioral correctness, and source-security risks before committing.

Best way to prevent agentic coding from shipping unfinished changes?

The best way to prevent agentic coding from shipping unfinished changes is to enforce an evidence-driven release gate. This requires quoted evidence that the work was run, tested, and reviewed, applying tripwires for fake-green tests before drafting a commit.