Skip to content
Development
Skill

/verify-before-done

Use before claiming "done", "fixed", "tested", or "it runs", and before committing or opening a PR or MR—no task is exempt. Accept only evidence gathered after the last relevant change and just sufficient to prove the claim; do not over-verify small changes.

From plugin
coding-discipline
258 skills

Claude Code

$ npx -y skills add chipfighter/coding-discipline --skill verify-before-done --agent claude-code

SKILL.md

verify-before-done.SKILL.md
name: verify-before-done
description: Use before claiming "done", "fixed", "tested", or "it runs", and before committing or opening a PR or MR—no task is exempt. Accept only evidence gathered after the last relevant change and just sufficient to prove the claim; do not over-verify small changes.
license: MIT

Hard rule: **Only evidence gathered after the last relevant change to the code, files, or environment, and directly sufficient to prove the claim, counts.**

  • Claiming completion without sufficient evidence = lying; running the full test suite for a typo = waste.
  • You may reuse evidence just gathered, but state when it was gathered and confirm that no code was changed, nothing was pulled, and the environment was not modified afterward. If you cannot say for sure, check again.

Required checks (before any claim of success or completion)

1. What claim must be proven? What check is just sufficient (command output / diff / artifact inspection…)? 2. Was the evidence gathered after the last relevant change? Yes—state when. No / uncertain—**check again**. 3. If you ran a command, read all output, inspect the exit code, and count failures. If you inspected a diff or artifact, state what you checked and the result. 4. Does the evidence actually prove the claim? Yes—state the conclusion together with the evidence. No—report the true status.

Skipping any step = lying, not verification.

Could not run ≠ skipped

If a command cannot run (missing environment, broken dependencies, timeout), **verification failed**. Report that verification could not be completed and what blocked it. Do not quietly mark the step green as “skipped,” and do not misreport an environment problem as a product failure.

None of these counts as "verified"

| Claimed evidence | Why it does not count | |---|---| | lint / type checks passed | This proves only that the checks pass, not that the project builds, runs, or behaves correctly. | | A subagent reported "success" | That is its claim. Inspect the diff / actual artifact yourself. | | The regression test passed once | This does not prove it catches the problem. In a safe, isolated way, temporarily disable this round's fix → the test must fail → restore the fix → the test must pass. Do not do this by reverting or overwriting user files. | | "Tests are green = requirements are met" | Check each requirement explicitly; green tests alone are not enough. | | Only part of the suite ran | This does not mean the whole suite passed. |

Hard rules

  • Do not substitute "should", "probably", or "looks right" for evidence.
  • Do not celebrate ("Done!" / "Fixed!" / "Perfect!") before obtaining evidence.
  • Before committing, pushing, or opening a PR or MR after an important change, pass the checks above.
  • If this work includes a confirmed change that makes the current source of truth inaccurate, or the work spans sessions but still lacks a source of truth, use spec-sync before claiming it is ready for handoff. If the user explicitly refuses persistence, explain the consequence and follow their latest instruction.
Read more
@chipfighter
Ships withcoding-discipline

Chinese → README.zh-CN.md Guardrails, not a workflow. Quiet on routine work; firm when a named failure mode is at risk. AI coding agents tend to fail in two expensive ways: They drift off-goal.

Stars on GitHub

Momentum

stars in total
25
Star history of chipfighter/coding-discipline on GitHub, a chart by star-history.com

Other skills on coding-discipline.

git-flow

git-flow

Use when creating a branch, using worktrees for parallel work, committing, or wrapping up. Covers branch naming, conventional…

spec-sync

spec-sync

Use in cross-session work when user-confirmed outcomes, non-goals, hard constraints, or acceptance criteria have not been written…

More skills from other plugins.

brainstorming
Auto-invoked

brainstorming

You MUST use this before any creative work - creating features, building components, adding functionality, modifying behavior, or…

@obra@obraInstall Skill
ask-matt

ask-matt

Ask which skill or flow fits your situation. A router over the skills in this repo.

skills10d309k
@mattpocock@mattpocockInstall Skill
accessibility
Auto-invoked

accessibility

Design, implement, and audit accessible UI to WCAG 2.2 Level AA across Web, iOS, and Android — semantic ARIA roles and labels,…

ecc11d293k
@affaan-m@affaan-mInstall Skill
compiler-commit

compiler-commit

Use when you want to verify compiler changes and commit with the correct convention. Runs tests, lint, and format, then commits…

react1d252k
@facebook@facebookInstall Skill

Questions about verify-before-done.

Which coding agents can use verify-before-done?
verify-before-done installs in Claude Code, Codex, Cursor, Gemini CLI, GitHub Copilot, OpenCode and Windsurf with the skills CLI, using each agent's flag shown on this page. Flowy's auto-invocation, which fires a skill when your prompt matches, runs in Claude Code only.