Move GetMapboxDocSourceTool to RESOURCE_FALLBACK_TOOLS

The get_latest_mapbox_docs_tool duplicates functionality of the
resource://mapbox-documentation resource. It should only be available
to clients that don't properly support resources.

Changes:
- Move GetMapboxDocSourceTool from CORE_TOOLS to RESOURCE_FALLBACK_TOOLS
- CORE_TOOLS now has 24 tools (down from 25)
- RESOURCE_FALLBACK_TOOLS now has 2 tools (up from 1):
  * get_reference_tool (reference resources)
  * get_latest_mapbox_docs_tool (documentation resource)
- Update tests to reflect new categorization
- Add test for get_latest_mapbox_docs_tool in resource fallback tools

Clients that properly support resources will no longer see this tool,
reducing clutter in their tool list.

All 536 tests pass.

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

Author: mattpodwysocki

src/tools/toolRegistry.ts

@@ -52,7 +52,6 @@ export const CORE_TOOLS = [
   new CoordinateConversionTool(),
   new GetFeedbackTool({ httpRequest }),
   new ListFeedbackTool({ httpRequest }),
-  new GetMapboxDocSourceTool({ httpRequest }),
   new TilequeryTool({ httpRequest }),
   new ValidateExpressionTool(),
   new ValidateGeojsonTool(),
@@ -73,11 +72,17 @@ export const ELICITATION_TOOLS = [] as const;
  * Tools that serve as bridges/workarounds for missing resource support
  * These tools are only registered if client does NOT support resources properly
  *
- * Context: GetReferenceTool exists as a workaround for Claude Desktop's limitation
- * where it can list resources but doesn't automatically fetch them. Clients that
- * properly support resources don't need this bridge tool.
+ * Context: These tools exist as workarounds for clients (like Claude Desktop) that
+ * can list resources but don't automatically fetch them. Clients that properly
+ * support resources don't need these bridge tools.
+ *
+ * - GetReferenceTool: Provides access to reference resources (style layers, Streets v8 fields, token scopes, layer type mapping)
+ * - GetMapboxDocSourceTool: Provides access to Mapbox documentation (resource://mapbox-documentation)
  */
-export const RESOURCE_FALLBACK_TOOLS = [new GetReferenceTool()] as const;
+export const RESOURCE_FALLBACK_TOOLS = [
+  new GetReferenceTool(),
+  new GetMapboxDocSourceTool({ httpRequest })
+] as const;
 
 /**
  * All tools combined (for backward compatibility and testing)

test/tools/toolRegistry.test.ts

@@ -44,6 +44,7 @@ describe('Tool Registry', () => {
 
       // Resource fallback tools should not be in core
       expect(toolNames).not.toContain('get_reference_tool');
+      expect(toolNames).not.toContain('get_latest_mapbox_docs_tool');
     });
   });
 
@@ -70,14 +71,20 @@ describe('Tool Registry', () => {
     it('should return an array of resource fallback tools', () => {
       const resourceFallbackTools = getResourceFallbackTools();
       expect(Array.isArray(resourceFallbackTools)).toBe(true);
-      expect(resourceFallbackTools.length).toBe(1);
+      expect(resourceFallbackTools.length).toBe(2);
     });
 
     it('should include get_reference_tool', () => {
       const resourceFallbackTools = getResourceFallbackTools();
       const toolNames = resourceFallbackTools.map((tool) => tool.name);
       expect(toolNames).toContain('get_reference_tool');
     });
+
+    it('should include get_latest_mapbox_docs_tool', () => {
+      const resourceFallbackTools = getResourceFallbackTools();
+      const toolNames = resourceFallbackTools.map((tool) => tool.name);
+      expect(toolNames).toContain('get_latest_mapbox_docs_tool');
+    });
   });
 
   describe('getAllTools', () => {
@@ -109,8 +116,9 @@ describe('Tool Registry', () => {
       // Core tools
       expect(toolNames).toContain('list_styles_tool');
       expect(toolNames).toContain('preview_style_tool');
-      // Resource fallback tool
+      // Resource fallback tools
       expect(toolNames).toContain('get_reference_tool');
+      expect(toolNames).toContain('get_latest_mapbox_docs_tool');
       // Note: No elicitation tools yet (empty array)
     });
   });