Skip to content

chore: remove docs tools/resources, keep static reference data#102

Merged
mattpodwysocki merged 6 commits intomainfrom
chore/remove-docs-resources
Apr 22, 2026
Merged

chore: remove docs tools/resources, keep static reference data#102
mattpodwysocki merged 6 commits intomainfrom
chore/remove-docs-resources

Conversation

@mattpodwysocki
Copy link
Copy Markdown
Contributor

Summary

Documentation retrieval is now owned by the dedicated mcp-docs-server, which provides richer functionality: full-text search, per-page fetching with .md support, caching, and per-product resources. This PR removes the lightweight/duplicate docs functionality from DevKit to avoid drift and confusion.

Removed:

  • MapboxDocumentationResource — fetched llms.txt, superseded by docs server
  • GetMapboxDocSourceTool (get_latest_mapbox_docs_tool) — docs server equivalent is better
  • GetReferenceTool (get_reference_tool) — was a resource fallback tool for the above
  • RESOURCE_FALLBACK_TOOLS category and CLIENT_NEEDS_RESOURCE_FALLBACK env var handling

Kept (static reference data relevant to DevKit's core purpose):

  • MapboxStyleLayersResource — layer types, paint/layout properties for style building
  • MapboxStreetsV8FieldsResource — used directly by StyleBuilderTool
  • MapboxTokenScopesResource — token scope reference for auth/validation flows
  • MapboxLayerTypeMappingResource — source layer to GL layer type mapping

Related

Test plan

  • All 523 tests pass
  • Snapshot updated to reflect removed tools

🤖 Generated with Claude Code

mattpodwysocki and others added 3 commits April 1, 2026 14:16
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Documentation retrieval (llms.txt fetching, get_latest_mapbox_docs_tool,
get_reference_tool) is now owned by the dedicated mcp-docs-server which
provides richer functionality: search, per-page fetching with .md support,
caching, and per-product resources.

Removed:
- MapboxDocumentationResource (fetched llms.txt — superseded by docs server)
- GetMapboxDocSourceTool / GetReferenceTool (resource fallback tools)
- RESOURCE_FALLBACK_TOOLS category and CLIENT_NEEDS_RESOURCE_FALLBACK handling

Kept:
- MapboxStyleLayersResource — layer types, paint/layout properties
- MapboxStreetsV8FieldsResource — used directly by StyleBuilderTool
- MapboxTokenScopesResource — token scope reference for auth flows
- MapboxLayerTypeMappingResource — source layer to GL layer type mapping

These four static resources are directly relevant to DevKit's core purpose
of style creation, validation, and optimization.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@mattpodwysocki mattpodwysocki requested a review from a team as a code owner April 21, 2026 15:07
Copy link
Copy Markdown

@Valiunia Valiunia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In readme there is ### Documentation Tools section

also there is mapbox/mcp-devkit-server/docs/mapbox-docs-tool-demo.md

and there could be other stale references - please have another look

mattpodwysocki and others added 3 commits April 21, 2026 13:06
…and prompts

- Remove Documentation Tools section from README (moved to mcp-docs-server)
- Delete docs/mapbox-docs-tool-demo.md
- Remove CLIENT_NEEDS_RESOURCE_FALLBACK section from README
- Update Resources section to remove get_reference_tool references
- Update prompt files to reference resource URIs directly
- Update tracing docs to remove stale tool names
- Fix prettier formatting in test files

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ocs tools

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
package-lock.json was pinned to 3.6.2 while local install had 3.8.x,
causing conflicting formatting between CI and the pre-commit hook.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@mattpodwysocki mattpodwysocki requested a review from Valiunia April 21, 2026 17:39
@mattpodwysocki mattpodwysocki merged commit b669f64 into main Apr 22, 2026
1 check passed
@mattpodwysocki mattpodwysocki deleted the chore/remove-docs-resources branch April 22, 2026 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants