Skip to content

refactor: modularize SendoWorkerService into focused modules and full Wallet Integration#8

Open
standujar wants to merge 2 commits intomainfrom
refactor/modularize-sendo-worker-service
Open

refactor: modularize SendoWorkerService into focused modules and full Wallet Integration#8
standujar wants to merge 2 commits intomainfrom
refactor/modularize-sendo-worker-service

Conversation

@standujar
Copy link
Contributor

Summary

Refactored the monolithic SendoWorkerService (1223 lines) into a modular architecture with 9 specialized modules, reducing the main service to 257 lines (-79%).

Key Changes

  • 9 new modules with single responsibilities (WorldManager, ProviderCollector, DataSelector, DataExecutor, ActionCategorizer, AnalysisGenerator, RecommendationGenerator, DecisionProcessor, AnalysisRepository)
  • SendoWorkerService now acts as orchestrator, delegating to modules
  • Improved error handling with proper error type classification (initialization/execution/technical)
  • Better testability - unit tests now test modules directly
  • Integration tests updated to use proper service loading mechanism

Test Plan

  • All unit tests passing (9 test files)
  • Integration tests updated and passing
  • Build successful
  • Error handling properly categorizes error types

@standujar standujar changed the title refactor: modularize SendoWorkerService into focused modules refactor: modularize SendoWorkerService into focused modules and full Wallet Integration Oct 30, 2025
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