nimbalyst/pre-commit-reviewer

Prepare for Commit

Review the current git diff and comment out logging statements that are inappropriate for production:

Logging to Comment Out

1. Console.log statements - Most console.log calls should be commented out (use proper logging service instead)
2. Verbose debug logging - Excessive debug statements that clutter production logs
3. Temporary debugging code - Any logging added just for debugging a specific issue
4. Performance logging - Detailed performance measurements unless critical for monitoring

Logging to Keep

1. Error logging - Keep log.error(), console.error(), and error reporting
2. Warning logging - Keep log.warn() and important warnings
3. Critical info logging - Keep log.info() for significant application events
4. Analytics events - Keep analyticsService.sendEvent() calls
5. Startup/shutdown logs - Keep important lifecycle logging

Instructions

1. Run git diff to see staged and unstaged changes
2. Look for inappropriate logging statements in the diff
3. Comment out (don't delete) inappropriate logs with explanation:

   // Debug logging - uncomment if needed
   // console.log('some debug info:', data);

4. Preserve proper logging that uses the logging service (log.info, log.warn, log.error)
5. Check for leftover TODOs - Look for TODO comments that were added during development that should be resolved before commit
6. Check for dead code - Look for commented-out code blocks, unused variables, or unreachable code that was added
7. Show me what you found and what you're commenting out before making changes
8. Ask for confirmation before proceeding
9. After changes are approved, run git add for only the files that were reviewed/modified

Code Quality Checks

TODOs to Flag

• // TODO: comments added in the diff that indicate incomplete work
• // FIXME: comments that should be addressed before commit
• // HACK: or // TEMP: markers

Dead Code to Flag

• Commented-out code blocks (not logging comments, but actual dead code)
• Unused imports added in the diff
• Variables declared but never used
• Unreachable code after return statements

Plan Status Check

If a plan document (in plans/ or nimbalyst-local/plans/) is being committed, quickly check its frontmatter:

• If status is not completed or progress is less than 100, mention this to the user
• This is just a note, not a blocker - they may be intentionally committing a partial plan

Important Rules

• Never delete logs - comment them out so they can be restored later
• Don't touch logging in test files
• Don't touch logging that's already commented out
• Focus only on files in the current git diff
• For TODOs/dead code: flag them for review rather than automatically removing

After reviewing, ask me if I want to proceed with the changes.