-
Notifications
You must be signed in to change notification settings - Fork 51
Closed
Labels
Description
Objective
Add 4 missing MCP properties to main_workflow_schema.json that are already implemented in the codebase but not validated by the schema.
Context
From discussion #11706: The main schema is missing registry, allowed, entrypoint, and mounts properties that are already implemented in pkg/parser/mcp.go, pkg/workflow/mcp-config-custom.go, and tested in mcp_config_compilation_test.go. This causes users to not discover these fields via schema validation.
Approach
Update pkg/parser/schemas/main_workflow_schema.json (lines 3398-3477) in the tools.additionalProperties section to include:
registry(string) - URI to installation location from MCP registryallowed(array of strings) - List of allowed tool namesentrypoint(string) - Container entrypoint overridemounts(array) - Volume mounts with pattern^[^:]+:[^:]+:(ro|rw)$
Files to Modify
pkg/parser/schemas/main_workflow_schema.json- Add 4 missing properties to tools.additionalProperties
Acceptance Criteria
-
registryproperty added with type string and description -
allowedproperty added as array of strings -
entrypointproperty added with type string -
mountsproperty added as array with pattern validation - All properties include examples matching existing documentation
- Schema remains valid JSON
- Run
make buildto embed updated schema
AI generated by Plan Command for discussion #11706
Reactions are currently unavailable