-
Notifications
You must be signed in to change notification settings - Fork 35
Mount MCP config directory in AWF container for Copilot workflows #8249
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.
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
Conversation
Co-authored-by: mnkiefer <[email protected]>
Co-authored-by: mnkiefer <[email protected]>
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
💀 Blimey! Smoke Copilot Safe Inputs failed and walked the plank! No treasure today, matey! ☠️ |
|
📰 DEVELOPING STORY: Smoke Copilot Playwright reports failed. Our correspondents are investigating the incident... |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
📰 DEVELOPING STORY: Smoke Copilot reports failed. Our correspondents are investigating the incident... |
|
✅ Firewall validation complete... Smoke Codex Firewall confirmed network sandboxing is operational. 🛡️ |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
🤖 ALERT: Smoke Copilot No Firewall failed. ANOMALY_DETECTED. REPAIR_REQUIRED. |
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.
|
@copilot must be rw mount |
Co-authored-by: pelikhan <[email protected]>
|
📰 VERDICT: Smoke Copilot Playwright has concluded. All systems operational. This is a developing story. 🎤 |
|
🎉 Yo ho ho! Smoke Copilot Safe Inputs found the treasure and completed successfully! ⚓💰 |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
🤖 DIAGNOSTIC COMPLETE: Smoke Copilot No Firewall STATUS: ALL_UNITS_OPERATIONAL. MISSION_SUCCESS. |
|
✅ Firewall validation complete... Smoke Codex Firewall confirmed network sandboxing is operational. 🛡️ |
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
Smoke Test Results - Run 20612062324Last 2 Merged PRs:
Test Results: Available Tools: add_comment, add_labels, create_issue, missing_tool, noop Overall Status: PASS (5/7 tests passed, 2 expected failures)
|
Smoke Test Results✅ File Writing: Created test file successfully Overall Status: PARTIAL PASS (2/3 tests passed) Note: safeinputs-gh tool requires GITHUB_TOKEN to be available in the workflow environment.
|
Smoke Test Results: Copilot Engine (No Firewall)Last 2 Merged PRs:
Test Results:
Overall Status: PASS (4/5 tests passed, 1 expected unavailability)
|
|
Merged PRs: Add debug logging and extract glob pattern helper module for repo-memory validation
|
Smoke Test Results (Run 20612062303)✅ Cache Memory Test: Successfully wrote and verified file at Overall Status: ❌ FAIL (1/3 tests passed) Note: Playwright and safeinputs-gh tools were not found in available tool set.
|
|
PRs reviewed: Add debug logging and extract glob pattern helper module for repo-memory validation; Refactor compile_orchestrator.go: split 1200-line file into focused modules
|
|
Claude Smoke Test Results Last 2 merged PRs:
✅ GitHub MCP - list_pull_requests Status: PASS (5/6 tests, gh tool unavailable)
|
Smoke test failures showed Playwright MCP and Safe Input gh tools unavailable to Copilot agent. The MCP configuration file at
/home/runner/.copilot/mcp-config.jsonwas generated but inaccessible to the Copilot CLI running inside the AWF container.Changes
pkg/workflow/copilot_engine.go: Added mount for/home/runner/.copilotdirectory to AWF container arguments with read-write permissions.lock.ymlfiles: Recompiled with updated mount configurationTechnical Details
The Copilot CLI uses
XDG_CONFIG_HOME=/home/runnerand looks for MCP configuration at$XDG_CONFIG_HOME/.copilot/mcp-config.json. The AWF command now includes:The mount is configured with read-write (
:rw) permissions to allow the Copilot CLI to write state files, logs, and other runtime data to the.copilotdirectory. This makes MCP servers (playwright, safeinputs, github) accessible to all Copilot-based workflows running under AWF.Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.