refactor: remove modules that don't fit core debugging use case

[rishitank]

Summary

Remove modules that are designed for CI/monitoring workflows rather than interactive debugging.

Rationale

AnimaWatch is an MCP server for on-demand animation debugging where:

• Developer asks AI agent to check animations
• AI responds immediately with findings
• Developer is already present in the conversation

Notifications, baselines, and PDF reports don't fit this use case:

Module	Why Removed
notifications.py	Slack/Discord alerts are redundant - Claude already tells you the result
baseline.py	Regression tracking is for CI pipelines, not interactive debugging
reports.py	PDF export is unnecessary - Claude reports findings conversationally

Modules Kept

These improve the core debugging experience:

• cache.py - Faster responses during debugging
• devices.py - Test on mobile viewports
• diff.py - Compare before/after screenshots
• fps.py - Quantitative frame analysis
• frames.py - Efficient video processing
• grounding.py - Reduce hallucinations
• consensus.py - Better accuracy with multi-model
• metrics.py - Core Web Vitals for performance debugging
• models.py - Structured output

Changes

• Deleted src/animawatch/notifications.py (225 lines)
• Deleted src/animawatch/baseline.py (279 lines)
• Deleted src/animawatch/reports.py (262 lines)

Total: -766 lines

Testing

• All 50 tests pass
• Ruff lint checks pass
• Mypy type checks pass

---

Pull Request opened by Augment Code with guidance from the PR author

[coderabbitai]

Summary by CodeRabbit

Release Notes

• Breaking Changes
  • Removed baseline comparison functionality
  • Removed notification system (Slack, Discord, and webhook support)
  • Removed report generation capabilities (HTML and PDF formats)

Walkthrough

Three feature modules have been entirely removed from the animawatch project: baseline management (279 lines), notification distribution to webhooks (225 lines), and report generation capabilities (262 lines). No new functionality has been introduced; these represent pure deletions totalling 766 lines of code.

Changes

Cohort / File(s)	Summary
Deleted Feature Modules src/animawatch/baseline.py, src/animawatch/notifications.py, src/animawatch/reports.py	Complete removal of baseline management system with regression detection, webhook-based notification service supporting Slack/Discord/generic payloads, and HTML/PDF report generation functionality.

Estimated Code Review Effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

The deletions are extensive and homogeneous in nature, yet require verification that no surviving code depends upon these removed entities and confirmation that this removal aligns with the project's strategic direction.

Poem

Three modules fade into shadow's embrace,
Baseline, notification, report—erased without trace.
The codebase contracts, lean and severe,
Your commitment to simplicity is clear—
The power of deletion, a Sith's finest spell. ✨🔱

---

Apprentice, I sense you have made a decisive choice. Removing these substantial systems demonstrates either remarkable clarity of purpose or... considerable risk. The code you have purged once served your vision. Ensure its absence does not haunt you.

Should any rebel scum—dependencies masquerading as innocent imports—still cling to these deleted modules, they shall expose themselves quickly. Guard against this weakness. The absence of comprehensive testing here would be... troubling.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title precisely describes the main changeset: removing three modules (notifications, baseline, reports) that don't align with the interactive debugging use case.
Description check	✅ Passed	The description is comprehensive and directly related to the changeset, providing clear rationale, listing removed modules, detailing why each was removed, and confirming all tests pass.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch refactor/remove-unused-modules

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}