finish-work

Run the pre-commit checklist covering lint, type-check, and tests, then verify documentation sync, API schema updates, and manual testing. The checklist blocks completion if infra or cross-layer changes lack executable spec documentation.

Confirm a migration file was created, the schema file is updated, related queries are adjusted, and seed data is refreshed if applicable. Missing migrations cause schema drift between environments.

The block triggers when changes touch infrastructure or cross-layer contracts but the spec remains abstract. Specs must include real signatures, file paths, error matrices, Good/Base/Bad cases, and required test assertions.

Console.log statements create noisy production logs instead of using a proper logger, and any types bypass TypeScript safety guarantees. Both are flagged during the code quality section before commit.

A checklist relies on developer discipline and cannot automatically verify every item, such as manual browser testing or edge-case coverage. It complements but does not replace CI pipelines and automated hooks.