Fix DISABLE_TOOLS array parsing error#11
Open
DanielSinclair wants to merge 1 commit intodirectus:mainfrom
Open
Fix DISABLE_TOOLS array parsing error#11DanielSinclair wants to merge 1 commit intodirectus:mainfrom
DanielSinclair wants to merge 1 commit intodirectus:mainfrom
Conversation
* Initial plan * Initial commit - no changes yet, planning the fix Co-authored-by: DanielSinclair <4412473+DanielSinclair@users.noreply.github.com> * Fix DISABLE_TOOLS JSON parsing error - Add JSON parsing to config schema - Update docs to use valid JSON format - Fix mcpServers examples with escaped quotes Co-authored-by: DanielSinclair <4412473+DanielSinclair@users.noreply.github.com> * Remove package-lock.json and add to .gitignore Co-authored-by: DanielSinclair <4412473+DanielSinclair@users.noreply.github.com> * Update .gitignore --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: DanielSinclair <4412473+DanielSinclair@users.noreply.github.com>
archawat
added a commit
to archawat/mcp-directus
that referenced
this pull request
Dec 1, 2025
- Add unique constraint warnings for resolve/reject fields
- Document correct workflow for inserting operations into chains
- Add data reference patterns ($last.field, {{key.field}})
- Update system prompt with operation chaining section
- Add common mistakes directus#11-13 for operation errors
- Improve create-operation and update-operation descriptions
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.
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\"]" } } } }