Expand tool descriptions to follow Anthropic best practices (#10)

Update all 23 MCP tool descriptions from compressed format (~15 words)
to detailed descriptions (48-67 words each) following Anthropic's
official guidance for tool use.

Each description now includes:
- What the tool does
- When to use it
- What it returns
- Caveats/limitations

This aligns with Anthropic's recommendation of "at least 3-4 sentences
per tool description" for optimal tool selection by Claude.

Also includes:
- New get_storage_layout tool for storage slot analysis
- Minor fixes to find_dead_code and list_functions
- Version bump to 2.2.0

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: dguido

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "slither-mcp"
-version = "2.2.0"
+version = "2.3.0"
 description = "MCP server for Slither static analysis of Solidity contracts"
 readme = "README.md"
 requires-python = ">=3.11"

slither_mcp/__init__.py

@@ -1,3 +1,3 @@
 """Slither MCP Server - MCP server for Slither static analysis."""
 
-__version__ = "0.1.0"
+__version__ = "2.3.0"

slither_mcp/tool_registry.py

@@ -76,6 +76,12 @@
     InheritanceNode,
     get_inherited_contracts,
 )
+from slither_mcp.tools.get_storage_layout import (
+    GetStorageLayoutRequest,
+    GetStorageLayoutResponse,
+    StorageSlotInfo,
+    get_storage_layout,
+)
 from slither_mcp.tools.get_project_overview import (
     GetProjectOverviewRequest,
     GetProjectOverviewResponse,
@@ -165,6 +171,11 @@
     "GetDerivedContractsRequest",
     "GetDerivedContractsResponse",
     "get_derived_contracts",
+    # Storage layout
+    "StorageSlotInfo",
+    "GetStorageLayoutRequest",
+    "GetStorageLayoutResponse",
+    "get_storage_layout",
     "ImplementationInfo",
     "ListFunctionImplementationsRequest",
     "ListFunctionImplementationsResponse",

slither_mcp/tools/__init__.py

@@ -12,6 +12,13 @@
 from slither_mcp.pagination import PaginatedRequest, apply_pagination
 from slither_mcp.types import ContractKey, FunctionKey, ProjectFacts, path_matches_exclusion
 
+# Default paths to exclude when exclude_test_frameworks is True
+DEFAULT_TEST_FRAMEWORK_PATHS = [
+    "lib/forge-std/",
+    "lib/ds-test/",
+    "node_modules/",
+]
+
 
 class DeadCodeFunction(BaseModel):
     """A function identified as potentially dead code."""
@@ -46,6 +53,13 @@ class FindDeadCodeRequest(PaginatedRequest):
         list[str] | None,
         Field(description="Path prefixes to exclude (e.g., ['lib/', 'node_modules/'])"),
     ] = None
+    exclude_test_frameworks: Annotated[
+        bool,
+        Field(
+            description="Exclude common test framework paths like lib/forge-std/, lib/ds-test/, "
+            "node_modules/ (default: True)"
+        ),
+    ] = True
 
 
 class FindDeadCodeResponse(BaseModel):
@@ -178,14 +192,19 @@ def find_dead_code(
         else:
             contracts_to_check = list(project_facts.contracts.items())
 
+        # Build effective exclusions list
+        effective_exclusions = list(request.exclude_paths or [])
+        if request.exclude_test_frameworks:
+            effective_exclusions.extend(DEFAULT_TEST_FRAMEWORK_PATHS)
+
         for contract_key, contract_model in contracts_to_check:
             # Skip interfaces and libraries for dead code analysis
             if contract_model.is_interface or contract_model.is_library:
                 continue
 
             # Skip contracts matching exclude_paths
-            if request.exclude_paths:
-                if path_matches_exclusion(contract_key.path, request.exclude_paths):
+            if effective_exclusions:
+                if path_matches_exclusion(contract_key.path, effective_exclusions):
                     continue
 
             # Check declared functions