# Post Work Review

> Use after completing implementation work to catch AGENTS.md violations, dead code, and issues missed during development. Run before showing results to the user. Use when this capability is needed.

- Skill: `tomevault-io/post-work-review` (Agent Skill, multi-file: 2 files)
- Install (CLI): `npx skillmds@latest add tomevault-io/post-work-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/tomevault-io/post-work-review/raw
- Safety review: pending (external: skill-scanner PASS, skillspector PASS)
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: tomevault-io (https://skillmd.com/u/tomevault-io)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/tomevault-io/post-work-review

---


# Post-Work Review

Review your diff against AGENTS.md rules and fix violations before presenting work.

## Process

1. Run `git diff HEAD` (or appropriate base) to see all changes
2. Read `AGENTS.md` fully -- the rules section is authoritative
3. For each changed file, check every rule that applies
4. Fix violations directly -- do not just report them
5. Re-run type check (`npx tsc --noEmit` or `go build ./...`)

## High-Signal Checks

These are the violations most commonly introduced under time pressure:

| Check | Rule | How to spot |
|-------|------|-------------|
| Type casts | AVOID `as` statements | grep for ` as ` in diff |
| Any types | AVOID `any` type | grep for `: any` or `as any` |
| Let bindings | AVOID `let` statements | grep for `let ` in diff |
| Else branches | DO NOT use `else` unless necessary | grep for `} else` |
| Dead code | Delete unused code when changing things | check if old code is still referenced |
| String interpolation | Use `cn()` not template literals for className | grep for `` className={` `` |
| Obvious comments | DO NOT use obvious comments | read comments -- are they saying what the code already says? |
| Disabled linters | DO NOT disable linter warnings unless necessary | grep for `eslint-disable` in new lines |

## Quick Grep

```bash
# Run against staged/unstaged changes
git diff HEAD -U0 | grep -E '^\+.*\bas\b' | grep -v '^+++' | grep -v 'import.*as'
git diff HEAD -U0 | grep -E '^\+.*: any' | grep -v '^+++'
git diff HEAD -U0 | grep -E '^\+.*\blet\b ' | grep -v '^+++'
git diff HEAD -U0 | grep -E '^\+.*\} else' | grep -v '^+++'
git diff HEAD -U0 | grep -E '^\+.*eslint-disable' | grep -v '^+++'
```

## Common Misses

- **Casts introduced to satisfy a generic** -- rethink the generic or type parameter instead
- **`undefined as void`** -- `undefined` is assignable to `void` directly
- **`let` for a variable reassigned once** -- restructure to use `const`
- **Dead imports** -- after deleting code, check if its imports are still needed
- **Comments describing the obvious** -- `// Release all resources` above `releaseAllResources()` adds nothing

---
> Source: [s4wave/spacewave](https://github.com/s4wave/spacewave) — distributed by [TomeVault](https://tomevault.io).
<!-- tomevault:4.0:skill_md:2026-07-04 -->