Improve style preview#11
Merged
Merged
Conversation
jussi-sa
approved these changes
Sep 3, 2025
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.
This pull request updates the
ListStylesToolandPreviewStyleToolto improve clarity and security around token usage, especially for previewing Mapbox styles. The main changes clarify input requirements, enforce public token usage, and update related tests and documentation.Tool input and schema improvements
ListStylesTooland its schema to recommend using thelimitparameter (5-10) and clarify the use of thestartparameter for pagination, improving guidance for API consumers. [1] [2] [3] [4]PreviewStyleToolschema to require a public access token (pk.*) and provide clear instructions about token requirements and permissions.PreviewStyleTool logic and security
PreviewStyleToolto accept a user-provided public token directly, removing the dependency on listing tokens internally. The tool now rejects secret (sk.*) and temporary (tk.*) tokens, ensuring only public tokens are used for preview URLs. [1] [2]Test updates
Package metadata
0.3.0to0.3.1in bothmanifest.jsonandpackage.jsonto reflect these changes. [1] [2]