-
Notifications
You must be signed in to change notification settings - Fork 33
Closed
5 / 55 of 5 issues completedLabels
Description
Overview
This tracking issue covers improvements to Go type consistency and type safety based on the Typist analysis from Discussion #6020.
Source: Discussion #6020
Key Findings
The Typist analysis identified:
- Critical: Duplicated
MCPServerConfigtype definition with incompatible structures - High Priority: Incomplete migration to
ToolsConfigstructured types - Medium Priority: Extensive use of
map[string]anyin frontmatter parsing
Planned Tasks
This work is broken down into focused, actionable tasks:
- Consolidate MCPServerConfig type - Fix the critical duplicate type definition
- Complete ToolsConfig migration - Finish the structured types work already started
- Add frontmatter structure types - Reduce type assertions in configuration parsing
- Improve runtime detection typing - Eliminate nested type assertion pyramids
Strategy
Following a gradual migration approach:
- Keep
map[string]anyat YAML parse boundaries - Convert to structured types immediately after parsing
- Maintain backward compatibility where needed
- Test thoroughly after each change
References
- Analysis: 🔤 Typist - Go Type Consistency Analysis #6020
- Key files:
pkg/workflow/tools_types.go,pkg/parser/mcp.go,pkg/gateway/gateway.go
AI generated by Plan Command for discussion #6020