-
Notifications
You must be signed in to change notification settings - Fork 34
Add MCP gateway to smoke-copilot workflow #9048
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: Mossaka <5447827+Mossaka@users.noreply.github.com>
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
🎉 Yo ho ho! Smoke Copilot Safe Inputs found the treasure and completed successfully! ⚓💰 |
|
✅ Firewall validation complete... Smoke Codex Firewall confirmed network sandboxing is operational. 🛡️ |
|
✨ 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... |
|
🤖 DIAGNOSTIC COMPLETE: Smoke Copilot No Firewall STATUS: ALL_UNITS_OPERATIONAL. MISSION_SUCCESS. |
|
📰 VERDICT: Smoke Copilot Playwright has concluded. All systems operational. This is a developing story. 🎤 |
Smoke Test Results✅ File Writing: Created test file successfully Overall Status: PASS
|
|
✅ Smoke test completed successfully for Copilot engine (no firewall). Test Results:
|
|
Smoke test (Codex firewall):
|
Smoke Test Results❌ Playwright MCP: Command not found in PATH Overall Status: TESTING
|
|
PR titles: Pin
|
The verify_mcp_gateway_health.sh script was unconditionally requiring the safeinputs server to be present in the MCP configuration. However, safeinputs is only configured when a workflow uses safe-inputs feature. The smoke-copilot workflow only uses safe-outputs and the github MCP server, so safeinputs was not present, causing the health check to fail. This fix makes safeinputs optional while keeping safeoutputs required (since all gateway-enabled workflows need safe-outputs for output handling). 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
🤖 DIAGNOSTIC COMPLETE: Smoke Copilot No Firewall STATUS: ALL_UNITS_OPERATIONAL. MISSION_SUCCESS. |
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
✅ Firewall validation complete... Smoke Codex Firewall confirmed network sandboxing is operational. 🛡️ |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
📰 VERDICT: Smoke Copilot Playwright has concluded. All systems operational. This is a developing story. 🎤 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
🎉 Yo ho ho! Smoke Copilot Safe Inputs found the treasure and completed successfully! ⚓💰 |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
Smoke Test Results - Safe Inputs Validation✅ File Writing: Created Overall Status: ✅ PASS
|
|
Latest merged PRs: Consolidate consecutive shell redirects to fix SC2129 warnings; Fix jsweep workflow: preinstall npm dependencies and enable all bash commands
|
Smoke Test Results ✅All tests passed successfully:
|
Smoke Test ResultsLast 2 Merged PRs:
Test Results:
Status: PASS (5/6 core tests passed, safeinputs-gh unavailable)
|
|
Smoke test results:
|
Smoke Test Results❌ Playwright MCP: Failed (inspector mode requires browser interaction) Overall Status: FAIL
|
Enables MCP gateway in the smoke-copilot workflow for proxying MCP server requests through a single HTTP endpoint.
Changes
sandbox.mcp.port: 8080andfeatures.mcp-gateway: trueConfiguration mirrors the existing
smoke-copilot-no-firewall.mdsetup.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.
Changeset
smoke-copilotworkflow by settingsandbox.mcp.port: 8080and enablingfeatures.mcp-gateway. Regenerated the lockfile to include gateway start and health check steps.Changeset
smoke-copilotworkflow by settingsandbox.mcp.port: 8080and enablingfeatures.mcp-gateway. Regenerated the lockfile to include gateway start and health check steps.