autoreview

Skill: Autoreview Session Changes for Production Quality

Purpose

Before considering the task complete, review all changes made in this session and bring them up to production-quality, industry-standard implementation.

This skill is a hard acceptance gate. The work is not done until this review is complete and any discovered issues are fixed properly.

Instructions

Review every file, decision, and implementation change made during this session.

Do not leave behind tech debt, shortcuts, brittle logic, temporary fixes, TODOs, dead code, duplicated logic, unnecessary complexity, or bandaids.

If any shortcut was taken, replace it with the correct long-term implementation before finishing.

Review Checklist

1. Correctness

   • Verify the implementation fully satisfies the user request.
   • Check edge cases, error handling, empty states, invalid inputs, and failure paths.
   • Confirm there are no hidden regressions.

2. Code Quality

   • Remove duplicated, dead, unused, or overly clever code.
   • Simplify unnecessarily complex logic.
   • Use clear names, consistent patterns, and idiomatic structure.
   • Keep concerns separated and avoid mixing unrelated responsibilities.

3. Architecture

   • Ensure the solution fits the existing project architecture.
   • Avoid one-off hacks that bypass established patterns.
   • Refactor properly instead of patching around poor structure.
   • Do not introduce unnecessary dependencies or abstractions.

4. Maintainability

   • Remove temporary comments, TODOs, console logs, debug code, and stale branches.
   • Make the implementation easy for another engineer to understand and extend.
   • Ensure future changes will not require working around this solution.

5. Security and Reliability

   • Validate inputs where appropriate.
   • Avoid unsafe assumptions.
   • Ensure secrets, credentials, private data, and environment-specific values are not exposed.
   • Handle failures gracefully.

6. Testing and Verification