← ClaudeAtlas

donelisted

End-of-session shipping gate — "prove it works, then ship it." Runs an evidence checklist (full test suite, lint/typecheck, build, and actually running the change — output quoted, never asserted), then a fresh-context two-stage review by a sub-agent that sees only the diff and the spec: spec-compliance first, correctness second, style never — joined by a cross-model second opinion when a codex/gemini CLI is installed. Flags fake-green tripwires (edited or deleted tests, new TODOs or skips, hardcoded expectations) and ends with a verdict — SHIP, FIX FIRST, or NEEDS HUMAN — before drafting the commit/PR for the user's approval. Use at the end of any coding session or when the user asks "is this done", "ready to ship?", "/done", "wrap this up", "review and commit this". It is the closing step of both feature and bugfix, but works standalone on any uncommitted or branch diff.
duthaho/skillhub · ★ 6 · Code & Development · score 75
Install: claude install-skill duthaho/skillhub
# done — the evidence gate `/done [what was being built, or a path to its spec/plan]` Answer one question: **is there evidence this change works and does what was asked — or does it merely look finished?** "The work looks done" is the weakest signal in agentic coding; this gate replaces it with quoted output and an independent review. It never makes the change better — it decides whether the change is *ready*. ## Step 0 — Establish the yardstick Identify **the diff** (uncommitted changes, or the branch vs its merge base) and **the intent**: the change folder's `spec.md`/`plan.md` if this came from the feature skill, the bug repro if from bugfix, otherwise ask the user for one sentence of "what was this supposed to do". No yardstick → the review below can only check correctness, not compliance; say so. ## Step 1 — Evidence checklist Copy this checklist into the response and fill it in as you go — every line gets **quoted command output or an explicit ✗ with the reason**. An unticked line with a reason is honest; a ticked line without evidence is the exact failure this skill exists to prevent. ``` - [ ] Full test suite: <command> → <pass/fail counts, verbatim tail> - [ ] Lint / typecheck: <command> → <result> (✗ if repo has none) - [ ] Build: <command> → <result> (✗ if n/a) - [ ] Ran the actual change: <command / flow> → <observed behavior> - [ ] Diff hygiene: no debug prints (grep the diff for [DEBUG- — the tag bugfix leaves), no comme