-
Notifications
You must be signed in to change notification settings - Fork 45
Closed
Labels
Description
Objective
Determine the purpose of the mode property in main schema and either document it clearly or deprecate it.
Context
From discussion #11706: The main schema defines both mode and type with identical enum values, but MCP config schema only has type. It's unclear if mode is:
- An alias for
type - A deprecated field
- A workflow-specific property
Approach
- Search codebase for usage of
modefield in MCP context - Check if parser/compiler treats
modeandtypedifferently - Review git history to understand when/why
modewas added - Either:
- Document
modeas an alias (if intentional) - Mark as deprecated with migration guide
- Remove if unused
- Document
Files to Investigate
pkg/parser/mcp.go- Check parsing logicpkg/workflow/mcp-config-*.go- Check compilation logic.github/workflows/*.md- Check for usage in workflows- Git history of
main_workflow_schema.json
Acceptance Criteria
- Usage of
modevstypeis clarified - Decision documented in schema description or comments
- If deprecated: Add deprecation notice with migration path
- If alias: Add clear documentation explaining relationship
- If unused: Remove from schema
- Update docs if needed
AI generated by Plan Command for discussion #11706
Reactions are currently unavailable