Skip to content

fix: make GitHub MCP tool optional in AOAI Copilot smoke workflows #5124

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
lpcox merged 4 commits into from
Jun 16, 2026
+10 −2
Merged

fix: make GitHub MCP tool optional in AOAI Copilot smoke workflows #5124

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
f9089d1
Initial plan
Copilot Jun 16, 2026
26755a3
fix: handle missing github MCP tool in AOAI smoke workflows
Copilot Jun 16, 2026
7d0ad08
Potential fix for pull request finding
lpcox Jun 16, 2026
1f7b105
Potential fix for pull request finding
lpcox Jun 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion .github/workflows/smoke-copilot-byok-aoai-apikey.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,11 @@ This smoke test validates that Copilot CLI runs in **direct BYOK mode against Az
The following tests were already executed in a deterministic pre-agent step. Your job is to verify the results and produce the summary comment.

### 1. GitHub MCP Testing
Verify MCP connectivity by calling `github-list_pull_requests` for ${{ github.repository }} (limit 1, state merged). Confirm the result matches the pre-fetched data below.
Verify MCP connectivity via the GitHub MCP tool `github-list_pull_requests` for ${{ github.repository }} (limit 2, state merged).
- If the tool responds successfully, confirm the result matches the Pre-Fetched PR Data below. ✅
- If the tool is unavailable (missing from context), skip the live call and validate the Pre-Fetched PR Data instead. Mark ✅ (pre-fetched data validated). Do **not** call `missing_tool` for this optional skip.
- If the tool is available but the call fails for another reason, mark ❌ and include the error.
Either way, continue to the **Output** section below and follow the required output rules.

### 2. GitHub.com Connectivity
Pre-step result: HTTP ${{ steps.smoke-data.outputs.SMOKE_HTTP_CODE }} from github.com.
Expand Down
6 changes: 5 additions & 1 deletion .github/workflows/smoke-copilot-byok-aoai-entra.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -182,7 +182,11 @@ This smoke test validates that Copilot CLI runs in **direct BYOK mode against Az
The following tests were already executed in a deterministic pre-agent step. Your job is to verify the results and produce the summary comment.

### 1. GitHub MCP Testing
Verify MCP connectivity by calling `github-list_pull_requests` for ${{ github.repository }} (limit 1, state merged). Confirm the result matches the pre-fetched data below.
Verify MCP connectivity via the GitHub MCP tool `github-list_pull_requests` for ${{ github.repository }} (limit 2, state merged).
- If the tool responds successfully, confirm the result matches the Pre-Fetched PR Data below. ✅
- If the tool is unavailable (missing from context), skip the live call and validate the Pre-Fetched PR Data instead. Mark ✅ (pre-fetched data validated). Do **not** call `missing_tool` for this optional skip.
- If the tool is available but the call fails for another reason, mark ❌ and include the error.
Either way, continue to the **Output** section below and follow the required output rules.

### 2. GitHub.com Connectivity
Pre-step result: HTTP ${{ steps.smoke-data.outputs.SMOKE_HTTP_CODE }} from github.com.
Expand Down
Loading