fix(ci-gate): tag PR owner with instructions#5116
Conversation
Instead of @-mentioning @copilot (which didn't trigger CCA reliably), tag the PR author directly with clear instructions: 1. Ask @copilot to address review feedback 2. Add ready-for-aw label once fixes are pushed ci-gate re-triggers on synchronize (CCA push), so after feedback is addressed the 'passed' comment will update automatically. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
✅ Coverage Check PassedOverall Coverage
📁 Per-file Coverage Changes (1 files)
Coverage comparison generated by |
There was a problem hiding this comment.
Pull request overview
This PR updates the CI Gate workflow’s gate-comment messaging so that, instead of @-mentioning @copilot, it @-mentions the PR author and provides explicit instructions for how to proceed and how to trigger agentic CI via the ready-for-aw label.
Changes:
- Add PR-author @-mentioning and updated instructions when Copilot review has inline comments.
- Update the “review passed” message to direct the PR owner to add
ready-for-aw(instead of instructing@copilot). - Adjust logging text to match the new “owner-directed” behavior.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/ci-gate.yml | Updates the gate comment content to tag the PR author with next-step instructions. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 1
| if (hasInlineComments) { | ||
| const prAuthor = context.payload.pull_request?.user?.login | ||
| || context.payload.review?.pull_request?.user?.login; | ||
| await postGateComment( | ||
| '⏳ Copilot review has inline comments.\n\n@copilot Please address the review feedback from the Copilot pull request reviewer above. Fix the issues raised in the inline comments and push your changes.' | ||
| `⏳ Copilot review left inline comments.\n\n@${prAuthor} To proceed:\n1. Ask \`@copilot\` to address the review feedback (reply to this comment or the review thread)\n2. Once the fix is pushed, add the \`ready-for-aw\` label to trigger agentic CI smoke tests` |
|
@copilot address review feedback |
|
✅ Copilot review passed with no inline comments. @lpcox Add the |
Fixed in the latest commit. The |
Old Copilot review comments persist even after fixes are pushed, so checking them on synchronize events permanently blocks the gate. The push itself signals the author has addressed feedback. The inline comment check is still performed on pull_request_review events for fresh reviews. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
📡 Smoke OTel Tracing completed. All tracing scenarios validated. ✅ |
|
🔑 Smoke Copilot PAT PAT auth validated. All systems operational. ✅ |
|
❌ Contribution Check failed. Please review the logs for details. |
|
✅ Smoke Copilot BYOK AOAI (api-key) completed. Copilot AOAI BYOK (api-key) mode operational. 🔓 |
|
Chroot tests passed! Smoke Chroot - All security and functionality tests succeeded. |
|
🔌 Smoke Services — All services reachable! ✅ |
|
🚀 Security Guard has started processing this pull request |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
✅ Build Test Suite completed successfully! |
|
✅ Smoke Gemini completed. All facets verified. 💎 |
|
✅ Smoke Copilot BYOK AOAI (Entra) completed. Copilot AOAI BYOK (Entra) mode operational. 🔓 |
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
❌ Smoke Claude failed |
|
✅ Smoke Copilot BYOK completed. Copilot BYOK mode operational. 🔓 |
🔥 Smoke Test: PAT Auth — PASS
Overall: PASS · Auth mode: PAT (COPILOT_GITHUB_TOKEN) PR: fix(ci-gate): tag PR owner with instructions · Author: @lpcox
|
🔭 Smoke Test: API Proxy OpenTelemetry Tracing
All scenarios passed. ✅
|
Running in direct BYOK mode (COPILOT_PROVIDER_API_KEY + COPILOT_PROVIDER_BASE_URL) via api-proxy → Azure OpenAI (Foundry, o4-mini-aw) Overall: PASS
|
🤖 Smoke Test Results — PASS
PR: fix(ci-gate): tag PR owner with instructions Overall: ✅ PASS
|
|
PR titles:
Results:
Overall: PASS Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "registry.npmjs.org"See Network Configuration for more information.
|
Smoke Test: Copilot BYOK (Direct Mode) ✅All tests passed. Running in direct BYOK mode via Results:
CC: @lpcox Status: PASS
|
🧪 Chroot Version Comparison Results
Overall: ❌ Not all tests passed — Python and Node.js versions differ between host and chroot environments.
|
|
Smoke Test Results:
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 @lpcox Overall: PASS
|
🏗️ Build Test Suite Results
Overall: 8/8 ecosystems passed — ✅ PASS
|
Smoke Test: GitHub Actions Services Connectivity
Overall: FAIL
|
Smoke Test Results: Gemini
Overall status: PASS Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "localhost"See Network Configuration for more information.
|
Problem
The previous ci-gate approach of @-mentioning
@copilotin comments didn't reliably trigger Copilot Coding Agent (CCA).Fix
Tag the PR author directly with clear instructions:
When Copilot review has inline comments:
When Copilot review passes (no inline comments):
Re-triggering
ci-gate automatically re-runs on
pull_request: [synchronize]— when CCA pushes a fix, ci-gate checks the existing review again. If the review has no inline comments, the comment updates to the 'passed' state directing the owner to add the label.