From c9c2ed772f6f297af3a5bdde07d83a27d6ac33e3 Mon Sep 17 00:00:00 2001
From: Andreas Wrede <andreas@wrede.ca>
Date: Sat, 25 Apr 2026 16:10:35 +0200
Subject: [PATCH] fix: document skip_reason in Plugin docstring; remove unused
 import in test

---
 hbd/client/plugin.py | 3 ++-
 tests/test_plugin.py | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/hbd/client/plugin.py b/hbd/client/plugin.py
index e3d36e0..9c5b22f 100644
--- a/hbd/client/plugin.py
+++ b/hbd/client/plugin.py
@@ -22,13 +22,14 @@ from typing import Any, Dict, List, Optional, Type
 
 class Plugin(ABC):
     """Base class for all plugins.
-    
+
     Attributes:
         name: Unique plugin identifier (e.g., "os_info", "cpu_monitor")
         version: Plugin version string
         description: Human-readable description
         interval: Collection interval in seconds (0 for InfoPlugin = collect once)
         enabled: Whether plugin is active (can be disabled via config)
+        skip_reason: Optional[str] = None  # Set before returning False from initialize() so loader logs INFO not WARNING
     """
     
     name: str = ""
diff --git a/tests/test_plugin.py b/tests/test_plugin.py
index 338e89d..9599c42 100644
--- a/tests/test_plugin.py
+++ b/tests/test_plugin.py
@@ -2,7 +2,7 @@ import asyncio
 import logging
 import textwrap
 
-from hbd.client.plugin import Plugin, PluginLoader, PluginRegistry
+from hbd.client.plugin import PluginLoader, PluginRegistry
 
 
 def test_plugin_skip_reason_defaults_none(tmp_path):