Fix DISABLE_TOOLS array parsing error #11
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.
Problem
The current implementation expects
DISABLE_TOOLSto be an array directly in the Zod schema:However, environment variables are always strings. When users set:
The configuration parsing would fail because the string couldn't be converted to an array.
Solution
Updated the configuration schema to accept a JSON string and parse it into an array:
Changes Made
DISABLE_TOOLSschema to parse JSON stringsReal-world Usage
Users can now properly set this in their environment:
{ "mcpServers": { "directus": { "command": "npx", "args": ["@directus/content-mcp@latest"], "env": { "DIRECTUS_URL": "https://your-directus-instance.com", "DIRECTUS_TOKEN": "your_directus_token", "DISABLE_TOOLS": "[\"delete-item\", \"update-field\"]" } } } }