Skip to content

refactor: restructure core services and monitors architecture#93

Merged
shinonomeow merged 1 commit into
3.2.0-betafrom
shino_aio
Aug 9, 2025
Merged

refactor: restructure core services and monitors architecture#93
shinonomeow merged 1 commit into
3.2.0-betafrom
shino_aio

Conversation

@shinonomeow
Copy link
Copy Markdown
Owner

  • Move monitors from separate modules to module/core/monitors/
    • download_monitor.py -> core/monitors/download_monitor.py
    • notification_monitor.py -> core/monitors/notification_monitor.py
    • rename_monitor.py -> core/monitors/rename_monitor.py
  • Split large services.py into modular service files in core/services/
    • BaseService -> base_services.py
    • DownloadService -> download_service.py
    • RSSService -> rss_service.py
    • RenamerService -> renamer_service.py (moved from core/)
  • Update imports and references across affected modules
  • Remove obsolete notification config and manager modules
  • Clean up unused status reporting methods

🤖 Generated with Claude Code

- Move monitors from separate modules to module/core/monitors/
  - download_monitor.py -> core/monitors/download_monitor.py
  - notification_monitor.py -> core/monitors/notification_monitor.py
  - rename_monitor.py -> core/monitors/rename_monitor.py
- Split large services.py into modular service files in core/services/
  - BaseService -> base_services.py
  - DownloadService -> download_service.py
  - RSSService -> rss_service.py
  - RenamerService -> renamer_service.py (moved from core/)
- Update imports and references across affected modules
- Remove obsolete notification config and manager modules
- Clean up unused status reporting methods

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@shinonomeow shinonomeow merged commit 6fd699d into 3.2.0-beta Aug 9, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant