fix: Improve plugin loading error handling and fix analytics plugin

- Update plugin manager to gracefully handle abstract method errors
- Fix analytics plugin metadata property implementation
- Ensure application continues running even if plugins fail to load
- Add better error logging for plugin loading failures

Author: VoxHash

CHANGELOG.md

@@ -5,7 +5,7 @@ All notable changes to the Telegram Multi-Account Message Sender project will be
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [1.2.8] - 2025-01-XX
+## [1.2.8] - 2025-12-07
 
 ### Added
 - **Proxy Test Functionality**: Added "Test Proxy" button in account proxy settings to verify proxy connection before saving

app/plugins/example_analytics_plugin.py

@@ -17,11 +17,7 @@ def __init__(self, api):
         """Initialize plugin."""
         super().__init__(api)
         self.events = []
-    
-    @property
-    def metadata(self) -> PluginMetadata:
-        """Get plugin metadata."""
-        return PluginMetadata(
+        self._metadata = PluginMetadata(
             name="Example Analytics Plugin",
             version="1.0.0",
             description="Example plugin that demonstrates analytics tracking",
@@ -31,6 +27,11 @@ def metadata(self) -> PluginMetadata:
             tags=["example", "analytics", "tracking"]
         )
 
+    @property
+    def metadata(self) -> PluginMetadata:
+        """Get plugin metadata."""
+        return self._metadata
+    
     def get_default_config(self) -> Dict[str, Any]:
         """Get default configuration."""
         return {

app/services/plugin_manager.py

@@ -140,15 +140,30 @@ def load_plugin(self, plugin_path: Path) -> Optional[PluginInfo]:
             for name, obj in inspect.getmembers(module):
                 if (inspect.isclass(obj) and 
                     issubclass(obj, Plugin) and 
-                    obj is not Plugin):
+                    obj is not Plugin and
+                    obj is not MessageFilterPlugin and
+                    obj is not AnalyticsPlugin and
+                    obj is not NotificationPlugin and
+                    obj is not IntegrationPlugin):
                     plugin_class = obj
                     break
 
             if plugin_class is None:
                 raise ValueError(f"No Plugin subclass found in {plugin_path}")
 
-            # Instantiate plugin
-            plugin_instance = plugin_class(self.api)
+            # Try to instantiate plugin - Python's ABC will raise TypeError if abstract methods aren't implemented
+            # We catch it and provide better error context
+            try:
+                plugin_instance = plugin_class(self.api)
+            except TypeError as e:
+                error_msg = str(e)
+                if "abstract" in error_msg.lower() or "abstractmethod" in error_msg.lower():
+                    # This is an abstract method error - log it and skip this plugin
+                    self.logger.error(f"Error loading plugin {plugin_path.name}: {error_msg}")
+                    return None
+                else:
+                    # Some other TypeError during instantiation
+                    raise
 
             # Get metadata
             metadata = plugin_instance.metadata