-
Notifications
You must be signed in to change notification settings - Fork 32
Add firewall health endpoint test and available tools display to smoke-copilot workflow #5816
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: pelikhan <4175913+pelikhan@users.noreply.github.com>
…e-copilot workflow Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…InputsFiles Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot merge main |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
🔮 The ancient spirits stir... Smoke Codex awakens to divine this pull request... |
|
💥 WHOOSH! Smoke Claude springs into action on this pull request! [Panel 1 begins...] |
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
🤖 DIAGNOSTIC COMPLETE: Smoke Copilot No Firewall STATUS: ALL_UNITS_OPERATIONAL. MISSION_SUCCESS. |
|
📰 BREAKING: Smoke Copilot Playwright is now investigating this pull request. Sources say the story is developing... |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
Smoke Test Results (Claude, Run 20020016317)Last 2 Merged PRs:
Test Results:
Overall Status: FAIL - gh safe-input tool requires GH_TOKEN environment variable
|
|
PR titles reviewed: [docs] Update documentation for features from 2025-12-08; Document branches/paths mutual exclusivity in workflow schema
|
Smoke Test Results - Run #20020016327❌ Playwright MCP: Tools not exposed to agent Status: FAIL Issue: MCP tools configured in workflow but not accessible as agent function calls.
|
Merge main into PR branch
Successfully merged main branch into
copilot/modify-agent-workflow-health-checkand recompiled all workflow files.Changes
pkg/cli/mcp_inspect_safe_inputs_test.goby removingmcp-server.cjsfrom expectedJSFiles array (it's verified separately)Pre-existing Issues
Note: Tests
TestSafeInputsStdioModeandTestSafeInputsHTTPModeare failing on both main and this branch. These were introduced in commit c9dba82 on main and are unrelated to this PR's changes.PR Changes Remain Intact
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. Non-breaking test and workflow updates for debugging and verification.