feat: convert remaining cli-proxy workflows to byok-copilot#2058
feat: convert remaining cli-proxy workflows to byok-copilot#2058
Conversation
Convert firewall-issue-dispatcher, smoke-copilot, and smoke-services from features.cli-proxy to features.byok-copilot. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
✅ Coverage Check PassedOverall Coverage
📁 Per-file Coverage Changes (1 files)
Coverage comparison generated by |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Pull request overview
Converts the remaining Copilot cli-proxy agentic workflows to run in byok-copilot mode, aligning these workflows with the repository’s offline/BYOK Copilot execution path.
Changes:
- Switch workflow frontmatter feature flag from
cli-proxy: truetobyok-copilot: truein the remaining 3 workflows. - Recompile/postprocess corresponding lock workflows, updating Copilot CLI install to
latestand ensuring BYOK-related env is present (e.g.,COPILOT_API_KEY, non-emptyCOPILOT_MODELfallback).
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/smoke-services.md | Flip feature flag to byok-copilot. |
| .github/workflows/smoke-services.lock.yml | Recompiled lock reflecting BYOK Copilot execution (incl. env + Copilot CLI install change). |
| .github/workflows/smoke-copilot.md | Flip feature flag to byok-copilot. |
| .github/workflows/smoke-copilot.lock.yml | Recompiled lock reflecting BYOK Copilot execution (incl. env + Copilot CLI install change). |
| .github/workflows/firewall-issue-dispatcher.md | Flip feature flag to byok-copilot. |
| .github/workflows/firewall-issue-dispatcher.lock.yml | Recompiled lock reflecting BYOK Copilot execution (incl. env + Copilot CLI install change). |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 6/6 changed files
- Comments generated: 3
- Copilot engine workflows: byok-copilot: true - Non-copilot engine workflows (claude, codex, opencode): cli-proxy: true All 29 workflows now have explicit feature flags for proxy support. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Smoke Test Results
Warning
|
This comment has been minimized.
This comment has been minimized.
The agent was timing out at 5 minutes with Redis + PostgreSQL services. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Update ghcr.io/github/gh-aw-mcpg from v0.2.22 to v0.2.23 across all 29 workflow lock files. This picks up the DIFC proxy fix for /api/graphql 404 errors on github.com repos. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Smoke Test Results ✅ PASS
|
🔬 Smoke Test Results
Overall: PARTIAL — two tests passed; file test skipped due to unexpanded template variables. Author: @lpcox | No assignees.
|
Smoke Test: Copilot BYOK (Offline) Mode
Running in BYOK offline mode ( Overall: PASS — @lpcox (no assignees)
|
🏗️ Build Test Suite Results
All
Overall: 0/8 ecosystems passed — ❌ FAIL Root Cause: The proxy at Fix: Trust the proxy CA certificate in the git/system trust store before running clone operations.
|
Smoke Test: GitHub Actions Services Connectivity ✅All checks passed:
Note:
|
Chroot Runtime Version Comparison
Result: Not all tests passed. Python and Node.js versions differ between host and chroot environments. The
|
Converts the remaining 3 Copilot
cli-proxyworkflows tobyok-copilot:firewall-issue-dispatcher.mdsmoke-copilot.mdsmoke-services.mdAll lock files recompiled and post-processed.