-
Notifications
You must be signed in to change notification settings - Fork 45
Closed as not planned
Closed as not planned
Copy link
Labels
Description
Objective
Resolve additionalProperties conflict and enum value mismatch between main and MCP config schemas.
Context
From discussion #11706:
- additionalProperties conflict: Main schema uses
true(permissive), MCP config usesfalse(strict) - Enum mismatch: Main schema includes
"remote"in type/mode enums, MCP config doesn't
Approach
- Investigate which
additionalPropertiessetting is correct for workflow validation - Align both schemas to use the same setting
- Determine if
"remote"is valid for type enum or should be removed - Document the decision in schema comments
Files to Modify
pkg/parser/schemas/main_workflow_schema.json- Update additionalProperties and/or enum valuespkg/parser/schemas/mcp_config_schema.json- Update to match (if needed)
Acceptance Criteria
- additionalProperties setting is consistent between schemas
- Enum values for type/mode are identical
- Decision documented in schema comments
- Existing workflows continue to validate correctly
- Run
make buildto embed updated schemas - All tests pass with
make test
AI generated by Plan Command for discussion #11706
Reactions are currently unavailable