fix: correct legacy Discord config detection in validate_config

The has_legacy_discord check was incorrectly checking if any notification
URL contained 'discord://', which would always be true for Discord users
regardless of config format. Now properly detects legacy Discord config
by checking for 'webhook_url' in the raw config data.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: m4r1k

src/eneru/monitor.py

@@ -7,7 +7,7 @@
 
 # Version is set at build time via git describe --tags
 # Format: "4.3.0" for tagged releases, "4.3.0-5-gabcdef1" for dev builds
-__version__ = "4.7.0-rc0"
+__version__ = "4.7.0-rc1"
 
 import subprocess
 import sys
@@ -513,7 +513,7 @@ def _parse_config(cls, data: Dict[str, Any]) -> Config:
         return config
 
     @classmethod
-    def validate_config(cls, config: Config) -> List[str]:
+    def validate_config(cls, config: Config, raw_data: Optional[Dict[str, Any]] = None) -> List[str]:
         """Validate configuration and return list of warnings/info messages."""
         messages = []
 
@@ -524,14 +524,23 @@ def validate_config(cls, config: Config) -> List[str]:
                 "Notifications will be disabled. Install with: pip install apprise"
             )
 
-        # Check for legacy Discord configuration
-        has_legacy_discord = any(
-            'discord://' in url.lower() for url in config.notifications.urls
-        )
+        # Check for legacy Discord configuration (webhook_url in discord section)
+        has_legacy_discord = False
+        if raw_data:
+            # Check for legacy discord.webhook_url in notifications section
+            if 'notifications' in raw_data:
+                notif_data = raw_data['notifications']
+                if 'discord' in notif_data and notif_data['discord'].get('webhook_url'):
+                    has_legacy_discord = True
+            # Check for top-level legacy discord section
+            if 'discord' in raw_data and 'notifications' not in raw_data:
+                if raw_data['discord'].get('webhook_url'):
+                    has_legacy_discord = True
+
         if has_legacy_discord:
             messages.append(
-                "INFO: Discord webhook detected. Using Apprise for notifications. "
-                "Native Discord embeds are now handled via Apprise."
+                "INFO: Legacy Discord webhook_url detected. Using Apprise for notifications. "
+                "Consider migrating to the 'notifications.urls' format."
             )
 
         return messages

tests/test_config.py

@@ -727,11 +727,39 @@ class TestConfigValidation:
     """Test configuration validation."""
 
     @pytest.mark.unit
-    def test_validate_config_with_apprise(self, full_config):
-        """Test validation returns info about Apprise."""
+    def test_validate_config_with_modern_discord(self, full_config):
+        """Test validation with modern discord:// URL format."""
         messages = ConfigLoader.validate_config(full_config)
-        # Should have message about Discord via Apprise
-        assert any("Discord" in msg or "Apprise" in msg for msg in messages)
+        # Modern discord:// URLs should not trigger legacy warning
+        assert not any("Legacy" in msg for msg in messages)
+
+    @pytest.mark.unit
+    def test_validate_config_with_legacy_discord(self, full_config):
+        """Test validation returns info about legacy Discord webhook_url."""
+        # Simulate raw config data with legacy discord.webhook_url
+        raw_data = {
+            'notifications': {
+                'discord': {
+                    'webhook_url': 'https://discord.com/api/webhooks/123/abc'
+                }
+            }
+        }
+        messages = ConfigLoader.validate_config(full_config, raw_data)
+        # Should have message about legacy Discord webhook_url
+        assert any("Legacy Discord webhook_url" in msg for msg in messages)
+
+    @pytest.mark.unit
+    def test_validate_config_with_toplevel_legacy_discord(self, full_config):
+        """Test validation detects top-level legacy discord config."""
+        # Simulate raw config data with top-level legacy discord section
+        raw_data = {
+            'discord': {
+                'webhook_url': 'https://discord.com/api/webhooks/456/def'
+            }
+        }
+        messages = ConfigLoader.validate_config(full_config, raw_data)
+        # Should have message about legacy Discord webhook_url
+        assert any("Legacy Discord webhook_url" in msg for msg in messages)
 
     @pytest.mark.unit
     def test_validate_config_empty_notifications(self, minimal_config):