Skip to content

Commit 5a74ace

Browse files
authored
Remove vestigial applyTo from parser validFields allowlist (#36309)
1 parent 9f2186c commit 5a74ace

2 files changed

Lines changed: 0 additions & 27 deletions

File tree

pkg/parser/include_processor.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -264,7 +264,6 @@ func collectUnexpectedIncludedFrontmatterFields(frontmatter map[string]any) []st
264264
"runtimes": true,
265265
"permissions": true,
266266
"secret-masking": true,
267-
"applyTo": true,
268267
"inputs": true,
269268
"import-schema": true,
270269
"disable-model-invocation": true,

pkg/parser/schema_utilities_test.go

Lines changed: 0 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -35,32 +35,6 @@ func TestFilterIgnoredFields(t *testing.T) {
3535
"on": "push",
3636
},
3737
},
38-
{
39-
name: "frontmatter with applyTo - no longer filtered",
40-
frontmatter: map[string]any{
41-
"applyTo": "some-value",
42-
"on": "push",
43-
},
44-
expected: map[string]any{
45-
"applyTo": "some-value",
46-
"on": "push",
47-
},
48-
},
49-
{
50-
name: "frontmatter with both description and applyTo - no longer filtered",
51-
frontmatter: map[string]any{
52-
"description": "This is a test workflow",
53-
"applyTo": "some-value",
54-
"on": "push",
55-
"engine": "claude",
56-
},
57-
expected: map[string]any{
58-
"description": "This is a test workflow",
59-
"applyTo": "some-value",
60-
"on": "push",
61-
"engine": "claude",
62-
},
63-
},
6438
{
6539
name: "frontmatter with only valid fields",
6640
frontmatter: map[string]any{

0 commit comments

Comments
 (0)