From ea839ec09c381123d5da697619e1acdcef3b97de Mon Sep 17 00:00:00 2001
From: Jacob Magar <jmagar@gmail.com>
Date: Sat, 14 Mar 2026 03:39:49 -0400
Subject: [PATCH] feat(elicitation): add CredentialsNotConfiguredError sentinel

---
 tests/test_setup.py           | 11 +++++++++++
 unraid_mcp/core/exceptions.py | 14 +++++++++++++-
 2 files changed, 24 insertions(+), 1 deletion(-)
 create mode 100644 tests/test_setup.py

diff --git a/tests/test_setup.py b/tests/test_setup.py
new file mode 100644
index 0000000..404fa4c
--- /dev/null
+++ b/tests/test_setup.py
@@ -0,0 +1,11 @@
+def test_credentials_not_configured_error_exists():
+    from unraid_mcp.core.exceptions import CredentialsNotConfiguredError
+
+    err = CredentialsNotConfiguredError()
+    assert str(err) == "Unraid credentials are not configured."
+
+
+def test_credentials_not_configured_error_is_exception():
+    from unraid_mcp.core.exceptions import CredentialsNotConfiguredError
+
+    assert issubclass(CredentialsNotConfiguredError, Exception)
diff --git a/unraid_mcp/core/exceptions.py b/unraid_mcp/core/exceptions.py
index 7737155..b63460e 100644
--- a/unraid_mcp/core/exceptions.py
+++ b/unraid_mcp/core/exceptions.py
@@ -23,6 +23,16 @@ class ToolError(FastMCPToolError):
     pass
 
 
+class CredentialsNotConfiguredError(Exception):
+    """Raised when UNRAID_API_URL or UNRAID_API_KEY are not set.
+
+    Used as a sentinel to trigger elicitation rather than a hard crash.
+    """
+
+    def __str__(self) -> str:
+        return "Unraid credentials are not configured."
+
+
 @contextlib.contextmanager
 def tool_error_handler(
     tool_name: str,
@@ -45,7 +55,9 @@ def tool_error_handler(
     except ToolError:
         raise
     except TimeoutError as e:
-        logger.exception(f"Timeout in unraid_{tool_name} action={action}: request exceeded time limit")
+        logger.exception(
+            f"Timeout in unraid_{tool_name} action={action}: request exceeded time limit"
+        )
         raise ToolError(
             f"Request timed out executing {tool_name}/{action}. The Unraid API did not respond in time."
         ) from e