From fe92906c46b7e20e0faa7bb7288590563ff578c9 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 23 Feb 2026 15:41:20 +0000 Subject: [PATCH 1/2] Initial plan From b4628b65e5cdcf23b517b051c8696b66ce5a4877 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 23 Feb 2026 15:48:33 +0000 Subject: [PATCH 2/2] fix: update tests to match capitalized "GitHub toolsets" in permissions validation message Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --- pkg/workflow/permissions_no_github_tool_test.go | 2 +- pkg/workflow/permissions_validator_test.go | 6 +++--- pkg/workflow/permissions_warning_test.go | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/pkg/workflow/permissions_no_github_tool_test.go b/pkg/workflow/permissions_no_github_tool_test.go index 5c3035b3b3..89c8c05d7f 100644 --- a/pkg/workflow/permissions_no_github_tool_test.go +++ b/pkg/workflow/permissions_no_github_tool_test.go @@ -55,7 +55,7 @@ tools: `, expectError: false, expectWarning: true, - warningMessage: "Missing required permissions for github toolsets:", + warningMessage: "Missing required permissions for GitHub toolsets:", }, { name: "no permissions, no github tool - no warning", diff --git a/pkg/workflow/permissions_validator_test.go b/pkg/workflow/permissions_validator_test.go index 8a15a758b5..b060908612 100644 --- a/pkg/workflow/permissions_validator_test.go +++ b/pkg/workflow/permissions_validator_test.go @@ -274,7 +274,7 @@ func TestFormatValidationMessage(t *testing.T) { }, strict: false, expectContains: []string{ - "Missing required permissions for github toolsets:", + "Missing required permissions for GitHub toolsets:", "contents: write (required by repos)", "issues: write (required by issues)", "Option 1: Add missing permissions to your workflow frontmatter:", @@ -351,7 +351,7 @@ func TestValidatePermissions_ComplexScenarios(t *testing.T) { ReadOnly: false, }, expectMsg: []string{ - "Missing required permissions for github toolsets:", + "Missing required permissions for GitHub toolsets:", "contents: write", "issues: write", "pull-requests: write", @@ -365,7 +365,7 @@ func TestValidatePermissions_ComplexScenarios(t *testing.T) { ReadOnly: false, }, expectMsg: []string{ - "Missing required permissions for github toolsets:", + "Missing required permissions for GitHub toolsets:", "discussions: write", }, }, diff --git a/pkg/workflow/permissions_warning_test.go b/pkg/workflow/permissions_warning_test.go index 1b3fa54a07..48843ce8ac 100644 --- a/pkg/workflow/permissions_warning_test.go +++ b/pkg/workflow/permissions_warning_test.go @@ -41,7 +41,7 @@ tools: strict: false, expectError: false, expectWarning: true, - warningMessage: "Missing required permissions for github toolsets:", + warningMessage: "Missing required permissions for GitHub toolsets:", }, { name: "missing permissions in strict mode produces error", @@ -214,7 +214,7 @@ tools: // Check that the warning includes both options expectedPhrases := []string{ - "Missing required permissions for github toolsets:", + "Missing required permissions for GitHub toolsets:", "Option 1: Add missing permissions to your workflow frontmatter:", "Option 2: Reduce the required toolsets in your workflow:", "issues",