fix: Normalize datetimes for timezone-aware vs naive comparison

[evansenter]

Summary

• Fixes Fix timezone-aware vs naive datetime comparison in correlate_git_with_sessions #34 - TypeError when comparing timezone-aware git commit timestamps with naive session timestamps
• Adds normalize_datetime() helper that strips timezone info while preserving local time values
• Removes pytest.skip workarounds and adds comprehensive regression tests

Test plan

• All 259 existing tests pass
• New unit tests for normalize_datetime() in test_queries.py
• Regression test with timezone-aware commit in test_ingest.py
• CI passes

🤖 Generated with Claude Code

[claude]

Prompt: evansenter/dotfiles/.../claude-review.md

Code Review

Summary

This PR fixes issue #34 by adding a normalize_datetime() helper function that strips timezone info from datetime objects, enabling safe comparison between timezone-aware git commit timestamps and naive session timestamps stored in SQLite.

Issues Found

Critical

None

Important

None

Suggestions

None

Verdict

APPROVE - Code looks good, no issues found.

The implementation is well-designed:

• The normalize_datetime() function correctly strips timezone info while preserving local time values, which is the right approach given that SQLite stores naive local time
• The docstring clearly explains the rationale for not converting to UTC
• The function is applied at both comparison points in correlate_git_with_sessions() (session ranges and commit times)
• Comprehensive test coverage including unit tests for the helper function and a regression test that exercises the full correlation flow
• The pytest.skip() workarounds are correctly removed from both test_cli.py and test_server.py
• The function is properly exported in __init__.py for public API access

---

Automated review by Claude Code