Objective
Split pkg/workflow/frontmatter_types.go (957 lines) by separating type definitions from parsing logic.
Context
Discussion: #26221
This file contains both type definitions and parsing logic. Per AGENTS.md conventions, parsing functions should live in a separate file when the combination exceeds the threshold.
Approach
- Keep all
type and struct declarations in frontmatter_types.go (target ≤ 300 lines)
- Move
ParseFrontmatterConfig, parseRuntimesConfig, parsePermissionsConfig, and other parse functions to a new frontmatter_parsing.go file
Files to Modify
- Split:
pkg/workflow/frontmatter_types.go
- Create:
pkg/workflow/frontmatter_parsing.go
Acceptance Criteria
Generated by Plan Command for issue #discussion #26221 · ● 150.1K · ◷
Objective
Split
pkg/workflow/frontmatter_types.go(957 lines) by separating type definitions from parsing logic.Context
Discussion: #26221
This file contains both type definitions and parsing logic. Per AGENTS.md conventions, parsing functions should live in a separate file when the combination exceeds the threshold.
Approach
typeandstructdeclarations infrontmatter_types.go(target ≤ 300 lines)ParseFrontmatterConfig,parseRuntimesConfig,parsePermissionsConfig, and other parse functions to a newfrontmatter_parsing.gofileFiles to Modify
pkg/workflow/frontmatter_types.gopkg/workflow/frontmatter_parsing.goAcceptance Criteria
frontmatter_types.gocontains only type/struct declarations and is ≤ 300 linesfrontmatter_parsing.gois ≤ 300 linesmake build && make test-unitpass without errorsmake fmt && make lintpass without errors