-
Notifications
You must be signed in to change notification settings - Fork 36
Simplify firewall activity summary format #6946
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
Simplify firewall activity summary format #6946
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
✅ Firewall validation complete... Smoke Codex Firewall confirmed network sandboxing is operational. 🛡️ |
|
🤖 DIAGNOSTIC COMPLETE: Smoke Copilot No Firewall STATUS: ALL_UNITS_OPERATIONAL. MISSION_SUCCESS. |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
🎉 Yo ho ho! Smoke Copilot Safe Inputs found the treasure and completed successfully! ⚓💰 |
Smoke Test ResultsGitHub MCP: ✅ (Last 2 PRs reviewed) Status: PASS
|
Smoke Test ResultsLast 2 Merged PRs:
Test Results: Status: PASS cc: @pelikhan
|
|
Last merged PRs: Add AWF support for Codex engine; Replace Node.js file embedding with bash script in setup-safe-outputs action
|
Smoke Test Results for Copilot Engine (No Firewall)Last 2 Merged PRs:
Test Results:
Overall Status: PASS (4/4 core tests successful)
|
|
Last 2 merged PRs: Add AWF support for Codex engine; Replace Node.js file embedding with bash script in setup-safe-outputs action
|
Removed emoji decoration and moved header into collapsible summary line for cleaner step output.
Changes
### 🔥 Firewall Activitymarkdown header📊emoji tosandbox agent:text prefix<summary>tag instead of separate elementResult
Before:
After:
Files Modified
pkg/workflow/js/parse_firewall_logs.cjs- Summary generation logicpkg/workflow/js/parse_firewall_logs.test.cjs- Test assertionspkg/cli/logs_firewall_parse_test.go- Integration test expectationsWarning
Firewall rules blocked me from connecting to one or more addresses (expand for details)
I tried to connect to the following addresses, but was blocked by firewall rules:
https://api.github.com/user/usr/bin/gh gh api user --jq .login ion-precompile.git l /opt/hostedtoolcache/go/1.25.0/x64/pkg/tool/linux_amd64/cgo REDACTED refs/tags/v1 /usr/bin/gh /opt/hostedtoolcache/go/1.25.0/x64/pkg/tool/linu--jq -V=f�� ithub-script/git/ref/tags/v8 gh /usr/bin/git /repos/actions/ugit --jq /usr/bin/infocmp168.63.129.16 git(http block)/usr/bin/gh gh api user --jq .login orkflows git ache/go/1.25.0/x64/pkg/tool/linux_amd64/compile runs/20251219-13git rev-parse /usr/bin/git ache/go/1.25.0/x64/pkg/tool/linu-address orig�� tags/v6 git e/git wall_logs.test.c/tmp/gh-aw-compile-integration-2246389799/gh-aw config 0/x64/bin/node e/git(http block)/usr/bin/gh gh api user --jq .login ithub/workflows -trimpath 017d8ed99eb3fe1423c4f3ad715ed1b4121/log.json l main -lang=go1.25 e/git-remote-https om/a�� it/ref/tags/v5 -dwarf=false ache/node/24.12.0/x64/bin/node go1.25.0 -c=4 -nolocalimports git(http block)If you need me to access, download, or install something from one of these locations, you can either:
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.