A1

[shaykeren]

This PR fixes the IllegalStateException in MonitorService and improves its overall implementation:

Changes Made

1. Fixed isRunning() to return the actual running state instead of always false
2. Removed deliberate exception throwing in monitor() method
3. Added proper state validation in start() method
4. Improved thread management and interruption handling
5. Added proper documentation
6. Made the running flag volatile for thread safety
7. Added better error handling and logging

Testing

• The service now properly reports its running state
• No more IllegalStateException is thrown
• Proper thread lifecycle management is implemented

Related Issues

Fixes the IllegalStateException that was occurring in the monitor service.

Additional Notes

• The monitor() method now has a placeholder for actual monitoring logic
• Thread management has been improved with proper interrupt handling
• Added daemon thread setting for better JVM lifecycle management