Repository navigation
Add mapbox-map-display and mapbox-reachability-analysis skills - #94
Open
mattpodwysocki wants to merge 2 commits into
Open
mattpodwysocki wants to merge 2 commits into
mattpodwysocki wants to merge 2 commits into
Conversation
Moves the two skills the Mapbox MCP Server serves over the MCP Skills extension here, so this repo is their only copy; mcp-server bundles them from a pinned commit at build time. Also fixes tool references in mapbox-location-grounding and mapbox-search-patterns: show maps with render_map_tool (static images only when asked for), points_within_polygon_tool instead of the nonexistent point_in_polygon_tool, and bbox_tool instead of the devkit's bounding_box_tool. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
It returns a mapboxRender.ref for render_map_tool, not a static map image. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
Author
|
@ctufts could you take a look at this one when you get a chance? mcp-server#295 bundles these two skills at build time, so it's waiting on this to merge before we can cut the release. |
ctufts
reviewed
Oct 29, 2026
|
|
||
| # Places within reach | ||
|
|
||
| Answers questions like "what's within a 10 minute walk of my hotel?" with the |
Contributor
There was a problem hiding this comment.
Doesn't this skill have direct overlap with the location-grounding skill? Is it solving a specific problem? From grounding skill:
### Proximity search with routing context
For "find [category] within X minutes of [location]":
1. Geocode the origin if needed: `search_and_geocode_tool(address)`
2. Get the reachable area: `isochrone_tool(coordinates, profile, contours_minutes)`
3. Search for POIs: `ground_location_tool(longitude, latitude, query: "category")` — the isochrone
contours tell the user what's within each time band
Or more directly: `ground_location_tool` returns both nearby POIs and isochrone reachability in one
call, which covers most proximity + routing use cases without additional composition.
ctufts
reviewed
Oct 29, 2026
|
|
||
| # Showing results on a Mapbox map | ||
|
|
||
| Use `render_map_tool` whenever the user wants to see something on a map. It |
Contributor
There was a problem hiding this comment.
Should this be explicit to be used during a non-"build" use case?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds the two skills the Mapbox MCP Server serves over the MCP Skills extension (SEP-2640), so this repo is the only copy of them. mapbox/mcp-server#295 bundles them from a pinned commit of this repo at build time instead of keeping its own.
mapbox-map-display: showing tool results withrender_map_toolby passing refs, several results on one map, hand-placed markers withmapbox_id, base map restyling, and when a static image is right instead.mapbox-reachability-analysis: places within a travel time or distance. Isochrone, thenbbox_tool, then category search in that box, thenpoints_within_polygon_toolto drop places in the box's corners, then one map with the area and only the places inside it.Each has
evals/evals.json(4 and 3 evals) and a README entry.Fixes to existing skills
mcp-server#295 will also serve
mapbox-location-groundingandmapbox-search-patterns, and it checks that every tool a skill names exists. These had a few that don't match the server:mapbox-location-grounding: the visual step usedstatic_map_image_tool. It now usesrender_map_toolwith the results' refs, and keeps static images for when the user asks for an image file. That matches the MCP server's own instructions.mapbox-location-grounding: saidground_location_toolreturns a static map image. It returns amapboxRender.refforrender_map_tool.mapbox-search-patterns/references/optimization-combining.md:point_in_polygon_tool(doesn't exist) is nowpoints_within_polygon_tool, and the static-map example is nowrender_map_tool.mapbox-search-patterns/references/workflows.md:bounding_box_toolis the DevKit server's; the runtime server's isbbox_tool.Testing
npm run checkpasses on a clean checkout: format, spellcheck, markdownlint, Codex plugin build and validation, and skill validation.SkillsClient(sizes, digests, frontmatter), and MCP Inspector--verify --require-digestsreports no conformance errors.🤖 Generated with Claude Code