Verification before completion
Skill pgoell/pgoell-claude-tools/plugins/workbench/skills/verification-before-completion
Use when about to claim work is complete, fixed, passing, ready, or safe to merge, especially before commits, pushes, PRs, or handoff summaries.From its SKILL.md
npx -y skills add pgoell/pgoell-claude-tools --skill verification-before-completionAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 0 stars0 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
SKILL.md
3.1 KB, 583 tokens by cl100k_base, as published. Nobody here has run it
Verification Before Completion
Require fresh evidence before any completion claim.
Core Rule
Do not claim that work is complete, fixed, passing, ready, or safe to merge unless the proving command or check ran after the final relevant edit and its output was read.
Evidence comes before claims.
Gate
Before claiming status:
- Identify the command or check that proves the claim.
- Run the full command or complete the full check.
- Read the output, exit code, and failure count.
- Compare the evidence to the claim.
- Report the actual status with the evidence.
If the evidence does not prove the claim, report the gap instead of softening the wording.
Required Evidence
| Claim | Required evidence |
|---|---|
| Tests pass | Fresh test output with a passing exit code and no failures |
| Lint is clean | Fresh lint output with a passing exit code |
| Build succeeds | Fresh build output with a passing exit code |
| Bug is fixed | The original symptom or regression test now passes |
| Regression test is valid | Red, green, and restored-green evidence |
| Agent completed work | VCS diff plus relevant verification, not the agent report alone |
| Requirements are met | Re-read checklist or spec and verify each item |
| PR is ready | Local verification plus any required pre-PR checks |
Red Flags
Stop and verify before using wording like:
- "done"
- "complete"
- "fixed"
- "passes"
- "ready"
- "green"
- "safe to merge"
- "should work"
- "looks good"
Also stop before committing, pushing, opening a PR, handing work back to the user, or moving to the next task.
Failure Handling
If verification fails:
- State the failing command or check.
- Summarize the relevant error.
- Do not claim completion.
- Either fix and rerun, or report the blocker.
If verification cannot be run:
- State that it was not run.
- Explain the concrete reason.
- Report any partial checks as partial only.
Reporting Pattern
Use concise evidence:
Verified: bash tests/unit/test-workbench-verification-before-completion-skill.sh, passed.
For partial status:
Not fully verified: lint passed, but integration tests were not run because live auth is unavailable.
Behavioral Rules
- Never use prior-session output as proof.
- Never use pre-final-edit output as proof, even if it came from the same turn.
- Never treat confidence as evidence.
- Never treat an agent report as proof without checking the diff and relevant commands.
- Never omit verification results from a completion summary.
- Keep the report short, but include what ran and whether it passed.
What ships with it
Read from the repository
Just SKILL.md. No reference files, no scripts.