Finalize
How it works
This skill runs a universal checklist, then checks the project's agent
instructions file (CLAUDE.md, or AGENTS.md in harnesses that use it) for any
project-specific finalization requirements (look for a "Finalize Checklist"
section or similar). The project knows what it needs — this skill just makes sure
nothing gets skipped.
Harness-neutral. The checklist is tool-agnostic: run the project's real
lint/test/build commands with whatever shell and tooling the harness has. Steps
that name a specific tool (e.g. the Impeccable UI pass, or the Memory step) apply
only where that tool exists — skip them, with a note, where it doesn't. Nothing
here requires subagents.
Universal Checklist
Work through each section. Report what passes and what needs fixing.
Fix issues directly rather than just listing them — the user wants the
work done, not a report.
1. Lint & Format
Run whatever lint/format tools the project uses. Check CLAUDE.md/AGENTS.md or
package.json/Gemfile for the commands. Common ones:
- Ruby:
standardrb, rubocop
- JS/TS:
eslint, oxlint, prettier, oxfmt
- CSS:
stylelint
- Python:
ruff, black
2. Tests
Run the project's test suite. Check for:
- All tests pass
- No skipped tests that shouldn't be skipped
- New code has test coverage (or note if it doesn't)
3. Code Quality Sweep
Scan changed files for common issues:
- Raw/hardcoded values that should use project utilities or constants
- Inconsistent patterns with existing code (naming, structure, style)
- Dead code, redundant logic, TODO/FIXME comments left behind
- Debug statements (console.log, binding.pry, debugger, puts)
- Security: hardcoded secrets, unsafe patterns
4. Documentation
Check that new/changed code has corresponding documentation:
- README or docs pages updated if user-facing behavior changed
- Inline docs (JSDoc, YARD, docstrings) on public APIs
- Any project-specific docs (check CLAUDE.md/AGENTS.md for the list)
5. UI & Design Quality
Skip this section if the changeset has no view/template/component files.
Run the Impeccable skill on changed view files:
$impeccable audit <target> — accessibility, performance, responsive behavior, theming
$impeccable polish <target> — final UI quality pass: spacing, interaction states, typography, transitions, component-library fit, design-system compliance, tokens
Use the current Impeccable command routing rather than legacy standalone
skills. normalize was folded into polish; do not try to invoke it
separately.
Report which Impeccable commands were run and what they found. Fix issues directly.
Consistency with existing pages (not covered by Impeccable):
- New pages match the structure of similar existing pages
- Headers, content wrappers, and action placement follow established patterns
- If the page deviates from the dominant pattern, the deviation is intentional
and justified
Visual verification:
- Changes render correctly in dev server, Storybook, or Lookbook
- Light mode and dark mode if applicable
- Mobile/responsive if applicable
6. Git & Issue Hygiene
- PR description reflects the actual scope of changes
Closes #N or equivalent issue linking in PR body
- Parent issues/epics updated if this is part of a larger effort
- Follow-on issues created for deferred or discovered work
- Issues added to project board with correct status
7. Project-Specific Checks
Read the project's CLAUDE.md/AGENTS.md for a "Finalize Checklist" section (or
similar). If it exists, work through those items too. These are project-specific
requirements that the universal checklist can't know about.
8. Memory
If this is a Claude Code session with persistent memory:
- Update memory files with key learnings, decisions, or patterns discovered
- Update status tracking (component counts, phase progress, etc.)
1---2name: finalize3description: Post-implementation checklist — docs, code quality, conventions, visual verification, issue hygiene. Triggers on 'finalize', 'ready to merge', 'preflight check', 'wrap this up', 'final review'.4---56# Finalize78## How it works910This skill runs a universal checklist, then checks the project's agent11instructions file (`CLAUDE.md`, or `AGENTS.md` in harnesses that use it) for any12project-specific finalization requirements (look for a "Finalize Checklist"13section or similar). The project knows what it needs — this skill just makes sure14nothing gets skipped.1516**Harness-neutral.** The checklist is tool-agnostic: run the project's real17lint/test/build commands with whatever shell and tooling the harness has. Steps18that name a specific tool (e.g. the Impeccable UI pass, or the Memory step) apply19only where that tool exists — skip them, with a note, where it doesn't. Nothing20here requires subagents.2122## Universal Checklist2324Work through each section. Report what passes and what needs fixing.25Fix issues directly rather than just listing them — the user wants the26work done, not a report.2728### 1. Lint & Format2930Run whatever lint/format tools the project uses. Check CLAUDE.md/AGENTS.md or31package.json/Gemfile for the commands. Common ones:3233- Ruby: `standardrb`, `rubocop`34- JS/TS: `eslint`, `oxlint`, `prettier`, `oxfmt`35- CSS: `stylelint`36- Python: `ruff`, `black`3738### 2. Tests3940Run the project's test suite. Check for:41- All tests pass42- No skipped tests that shouldn't be skipped43- New code has test coverage (or note if it doesn't)4445### 3. Code Quality Sweep4647Scan changed files for common issues:48- Raw/hardcoded values that should use project utilities or constants49- Inconsistent patterns with existing code (naming, structure, style)50- Dead code, redundant logic, TODO/FIXME comments left behind51- Debug statements (console.log, binding.pry, debugger, puts)52- Security: hardcoded secrets, unsafe patterns5354### 4. Documentation5556Check that new/changed code has corresponding documentation:57- README or docs pages updated if user-facing behavior changed58- Inline docs (JSDoc, YARD, docstrings) on public APIs59- Any project-specific docs (check CLAUDE.md/AGENTS.md for the list)6061### 5. UI & Design Quality6263Skip this section if the changeset has no view/template/component files.6465**Run the Impeccable skill on changed view files:**66- `$impeccable audit <target>` — accessibility, performance, responsive behavior, theming67- `$impeccable polish <target>` — final UI quality pass: spacing, interaction states, typography, transitions, component-library fit, design-system compliance, tokens6869Use the current Impeccable command routing rather than legacy standalone70skills. `normalize` was folded into `polish`; do not try to invoke it71separately.7273Report which Impeccable commands were run and what they found. Fix issues directly.7475**Consistency with existing pages** (not covered by Impeccable):76- New pages match the structure of similar existing pages77- Headers, content wrappers, and action placement follow established patterns78- If the page deviates from the dominant pattern, the deviation is intentional79 and justified8081**Visual verification:**82- Changes render correctly in dev server, Storybook, or Lookbook83- Light mode and dark mode if applicable84- Mobile/responsive if applicable8586### 6. Git & Issue Hygiene8788- PR description reflects the actual scope of changes89- `Closes #N` or equivalent issue linking in PR body90- Parent issues/epics updated if this is part of a larger effort91- Follow-on issues created for deferred or discovered work92- Issues added to project board with correct status9394### 7. Project-Specific Checks9596Read the project's CLAUDE.md/AGENTS.md for a "Finalize Checklist" section (or97similar). If it exists, work through those items too. These are project-specific98requirements that the universal checklist can't know about.99100### 8. Memory101102If this is a Claude Code session with persistent memory:103- Update memory files with key learnings, decisions, or patterns discovered104- Update status tracking (component counts, phase progress, etc.)