git-final-pass

name	git-final-pass
disable-model-invocation	true
description	Pre-PR quality checklist encoding recurring review feedback. Run before creating a PR to catch common mistakes. Covers: error handling, naming, resource cleanup, test coverage, documentation, commit hygiene, and language-specific gotchas. Use when code is "done" and about to be PR'd.
allowed-tools	["Read","Grep","Glob","Bash"]

Final Pass

Run this checklist before creating a PR. Every item is a mistake that has actually happened.

Process

1. Identify changed files: git diff --name-only HEAD~1 or git diff --name-only main
2. Run each check category against changed files only
3. Produce a pass/fail table
4. Fix failures before proceeding to PR

Checks

Error Handling

Check	What to look for
Unchecked errors	Go: err assigned but not checked. TS: .catch() missing on promises
Swallowed errors	catch {} or if err != nil { return nil } with no logging
Error message quality	Does the error say what went wrong AND what was being attempted?