From 1f18441170fbea26cd70be91d7e5115871b8f7ef Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 13 Jun 2026 14:32:38 +0000 Subject: [PATCH 1/5] Initial plan From c81502068e9e6b4d5a151d43e9862b5b59ef8fb3 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 13 Jun 2026 14:38:33 +0000 Subject: [PATCH 2/5] fix(smoke): add non-PR noop fallback for AOAI Entra copilot workflow --- .../workflows/smoke-copilot-byok-aoai-entra.md | 6 ++++-- ...moke-copilot-byok-aoai-entra-workflow.test.ts | 16 ++++++++++++++++ 2 files changed, 20 insertions(+), 2 deletions(-) create mode 100644 scripts/ci/smoke-copilot-byok-aoai-entra-workflow.test.ts diff --git a/.github/workflows/smoke-copilot-byok-aoai-entra.md b/.github/workflows/smoke-copilot-byok-aoai-entra.md index be5c55d59..4b62d8096 100644 --- a/.github/workflows/smoke-copilot-byok-aoai-entra.md +++ b/.github/workflows/smoke-copilot-byok-aoai-entra.md @@ -202,11 +202,13 @@ ${{ steps.smoke-data.outputs.SMOKE_PR_DATA }} ## Output -Add a **very brief** comment (max 5-10 lines) to the current pull request with: +**If triggered by a pull request**, add a **very brief** comment (max 5-10 lines) to the current pull request with: - PR titles only (no descriptions) - ✅ or ❌ for each test result - Note: "Running in direct BYOK mode (AWF_AUTH_TYPE=github-oidc + AWF_AUTH_AZURE_* + COPILOT_PROVIDER_BASE_URL) via api-proxy → Azure OpenAI (Foundry, o4-mini-aw) authenticated via Microsoft Entra" - Overall status: PASS or FAIL - Mention the pull request author and any assignees -If all tests pass, add the label `smoke-copilot-byok-aoai-entra` to the pull request. +If all tests pass on a pull request trigger, add the label `smoke-copilot-byok-aoai-entra` to the pull request. + +**If triggered by workflow_dispatch or schedule** (no PR context), call `noop` with a concise PASS/FAIL summary instead. Do NOT attempt to add pull request comments or labels when there is no pull request. diff --git a/scripts/ci/smoke-copilot-byok-aoai-entra-workflow.test.ts b/scripts/ci/smoke-copilot-byok-aoai-entra-workflow.test.ts new file mode 100644 index 000000000..31c4ef819 --- /dev/null +++ b/scripts/ci/smoke-copilot-byok-aoai-entra-workflow.test.ts @@ -0,0 +1,16 @@ +import * as fs from 'fs'; +import * as path from 'path'; + +const workflowsDir = path.resolve(__dirname, '../../.github/workflows'); +const workflowSourcePath = path.join(workflowsDir, 'smoke-copilot-byok-aoai-entra.md'); + +describe('smoke copilot byok aoai entra workflow output requirements', () => { + it('requires noop fallback when no pull request context exists', () => { + const source = fs.readFileSync(workflowSourcePath, 'utf-8'); + + expect(source).toContain('**If triggered by a pull request**, add a **very brief** comment'); + expect(source).toContain('If all tests pass on a pull request trigger, add the label `smoke-copilot-byok-aoai-entra`'); + expect(source).toContain('**If triggered by workflow_dispatch or schedule** (no PR context), call `noop`'); + expect(source).toContain('Do NOT attempt to add pull request comments or labels when there is no pull request.'); + }); +}); From ebeaf96588a58ca475b833ec21503654fc6658ef Mon Sep 17 00:00:00 2001 From: Landon Cox Date: Sat, 13 Jun 2026 08:55:42 -0700 Subject: [PATCH 3/5] Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- .github/workflows/smoke-copilot-byok-aoai-entra.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/smoke-copilot-byok-aoai-entra.md b/.github/workflows/smoke-copilot-byok-aoai-entra.md index 4b62d8096..dd9bb6e10 100644 --- a/.github/workflows/smoke-copilot-byok-aoai-entra.md +++ b/.github/workflows/smoke-copilot-byok-aoai-entra.md @@ -202,7 +202,7 @@ ${{ steps.smoke-data.outputs.SMOKE_PR_DATA }} ## Output -**If triggered by a pull request**, add a **very brief** comment (max 5-10 lines) to the current pull request with: +**If triggered by a pull request**, call `add_comment` to post a **very brief** comment (max 5-10 lines) to the current pull request with: - PR titles only (no descriptions) - ✅ or ❌ for each test result - Note: "Running in direct BYOK mode (AWF_AUTH_TYPE=github-oidc + AWF_AUTH_AZURE_* + COPILOT_PROVIDER_BASE_URL) via api-proxy → Azure OpenAI (Foundry, o4-mini-aw) authenticated via Microsoft Entra" From 3328deb4b046084ca6d067fc942cbff4ff0f3d3f Mon Sep 17 00:00:00 2001 From: Landon Cox Date: Sat, 13 Jun 2026 08:55:52 -0700 Subject: [PATCH 4/5] Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- .github/workflows/smoke-copilot-byok-aoai-entra.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/smoke-copilot-byok-aoai-entra.md b/.github/workflows/smoke-copilot-byok-aoai-entra.md index dd9bb6e10..ff947a391 100644 --- a/.github/workflows/smoke-copilot-byok-aoai-entra.md +++ b/.github/workflows/smoke-copilot-byok-aoai-entra.md @@ -209,6 +209,6 @@ ${{ steps.smoke-data.outputs.SMOKE_PR_DATA }} - Overall status: PASS or FAIL - Mention the pull request author and any assignees -If all tests pass on a pull request trigger, add the label `smoke-copilot-byok-aoai-entra` to the pull request. +If all tests pass on a pull request trigger, call `add_labels` to add the label `smoke-copilot-byok-aoai-entra` to the pull request. **If triggered by workflow_dispatch or schedule** (no PR context), call `noop` with a concise PASS/FAIL summary instead. Do NOT attempt to add pull request comments or labels when there is no pull request. From eabe24d233126a3847121781c6097e758f8126f9 Mon Sep 17 00:00:00 2001 From: Landon Cox Date: Sat, 13 Jun 2026 08:56:03 -0700 Subject: [PATCH 5/5] Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- scripts/ci/smoke-copilot-byok-aoai-entra-workflow.test.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/ci/smoke-copilot-byok-aoai-entra-workflow.test.ts b/scripts/ci/smoke-copilot-byok-aoai-entra-workflow.test.ts index 31c4ef819..5fb19fc31 100644 --- a/scripts/ci/smoke-copilot-byok-aoai-entra-workflow.test.ts +++ b/scripts/ci/smoke-copilot-byok-aoai-entra-workflow.test.ts @@ -8,8 +8,8 @@ describe('smoke copilot byok aoai entra workflow output requirements', () => { it('requires noop fallback when no pull request context exists', () => { const source = fs.readFileSync(workflowSourcePath, 'utf-8'); - expect(source).toContain('**If triggered by a pull request**, add a **very brief** comment'); - expect(source).toContain('If all tests pass on a pull request trigger, add the label `smoke-copilot-byok-aoai-entra`'); + expect(source).toContain('**If triggered by a pull request**, call `add_comment` to post a **very brief** comment'); + expect(source).toContain('If all tests pass on a pull request trigger, call `add_labels` to add the label `smoke-copilot-byok-aoai-entra`'); expect(source).toContain('**If triggered by workflow_dispatch or schedule** (no PR context), call `noop`'); expect(source).toContain('Do NOT attempt to add pull request comments or labels when there is no pull request.'); });