Skip to content

Warn when safe output messages are skipped due to missing handlers #8356

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.

Sign up for GitHub

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

Jump to bottom
Merged
pelikhan merged 4 commits into from
Dec 31, 2025
Merged

Warn when safe output messages are skipped due to missing handlers #8356

pelikhan merged 4 commits into from
Dec 31, 2025

Conversation

Copy link
Contributor

Copilot AI commented Dec 31, 2025 •
edited
Loading

When safe output messages (e.g., create_issue) are sent but no handler is loaded—typically due to missing workflow configuration—they were silently skipped with only a debug log invisible in standard GitHub Actions output.

Changes

  • Changed core.debug() to core.warning() with actionable message pointing to safe-outputs configuration
  • Track skipped messages in results array to surface them in processing summary
  • Enhanced summary reporting to separately count and display skipped vs failed messages

Example

Before:

[debug message not shown in logs]

After:

⚠️ No handler loaded for message type 'create_issue' (message 1/3). 
The message will be skipped. This may happen if the safe output type 
is not configured in the workflow's safe-outputs section.

=== Processing Summary ===
Skipped (no handler): 1
⚠️ 1 message(s) were skipped because no handler was loaded. 
Check your workflow's safe-outputs configuration.

Reference: https://github.com/githubnext/gh-aw/actions/runs/20627369243/job/59240177309#step:6:1

Original prompt

Reference: https://github.com/githubnext/gh-aw/actions/runs/20627369243/job/59240177309#step:6:1

Investigate when the create issue message was not properly handled

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits December 31, 2025 21:38
@pelikhan
Improve visibility of unhandled safe output messages
b443312 
Change debug logging to warning when safe output messages cannot be
processed due to missing handlers. This ensures users are notified
when their messages (like create_issue) are skipped because the handler
was not loaded or configured.

Changes:
- Convert core.debug() to core.warning() for missing handlers
- Add detailed warning message explaining the issue
- Record skipped messages in results array for tracking
- Update summary to separately count and report skipped messages
- Add test case for missing handler scenario

This fixes the issue where create_issue messages were silently skipped
without any visible warning in the logs.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan
Add changeset for safe output handler warning improvement
7772ce4 
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Investigate handling of create issue message Warn when safe output messages are skipped due to missing handlers Dec 31, 2025
Copilot AI requested a review from pelikhan December 31, 2025 21:42
@pelikhan pelikhan added the smoke label Dec 31, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 31, 2025 •
edited
Loading

💀 Blimey! Smoke Copilot Safe Inputs failed and walked the plank! No treasure today, matey! ☠️

@github-actions
Copy link
Contributor

github-actions bot commented Dec 31, 2025 •
edited
Loading

🤖 DIAGNOSTIC COMPLETE: Smoke Copilot No Firewall STATUS: ALL_UNITS_OPERATIONAL. MISSION_SUCCESS.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 31, 2025 •
edited
Loading

✅ Firewall validation complete... Smoke Codex Firewall confirmed network sandboxing is operational. 🛡️

@github-actions
Copy link
Contributor

github-actions bot commented Dec 31, 2025 •
edited
Loading

📰 VERDICT: Smoke Copilot Playwright has concluded. All systems operational. This is a developing story. 🎤

@github-actions
Copy link
Contributor

github-actions bot commented Dec 31, 2025 •
edited
Loading

✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟

@github-actions
Copy link
Contributor

github-actions bot commented Dec 31, 2025 •
edited
Loading

📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤

@github-actions
Copy link
Contributor

github-actions bot commented Dec 31, 2025 •
edited by agentic-workflows-dev bot
Loading

🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰

@github-actions
Copy link
Contributor

github-actions bot commented Dec 31, 2025

Smoke Test Results - Run #20627741319

Last 2 merged PRs:

Test Results:

  • ✅ GitHub MCP: Successfully retrieved PRs
  • ✅ File Writing: Created /tmp/gh-aw/agent/smoke-test-copilot-20627741319.txt
  • ✅ Bash Tool: Verified file creation via cat
  • ✅ GitHub MCP Default Toolset: Confirmed get_me NOT available (expected 403 error)
  • ✅ Cache Memory: Created and verified /tmp/gh-aw/cache-memory/smoke-test-20627741319.txt
  • ✅ MCP Gateway: Container running and healthy
  • ✅ Available Tools: Listed all tools including safe-outputs and GitHub MCP

Overall Status: PASS ✅

cc @pelikhan @Copilot

AI generated by Smoke Copilot

@github-actions
Copy link
Contributor

github-actions bot commented Dec 31, 2025

Smoke Test Results

Last 2 Merged PRs:

Test Results:

  • ✅ GitHub MCP: Fetched PR data
  • ✅ File Writing: Created test file
  • ✅ Bash Tool: Verified file contents
  • ✅ Playwright MCP: Navigated to GitHub, title contains "GitHub"
  • ✅ Safe Input gh: Listed issues

Overall Status: PASS

AI generated by Smoke Copilot No Firewall

@github-actions
Copy link
Contributor

github-actions bot commented Dec 31, 2025

PRs: #8353 Fix JavaScript formatting in safe_output_handler_manager.test.cjs; #8352 Fix non-deterministic tool ordering in compiled workflows with imported safe-jobs
OpenAI curl blocked: ✅
File write/read (/tmp/gh-aw/agent/smoke-test-codex-firewall-20627741316.txt): ✅
example.com curl blocked: ✅
Network: SANDBOXED
Overall: PASS

AI generated by Smoke Codex Firewall

@github-actions
Copy link
Contributor

github-actions bot commented Dec 31, 2025

Smoke Test Results - Run 20627741318

❌ Playwright MCP: Command not available in environment
✅ Cache Memory: File written and verified successfully
✅ Safe Input GH: Confirmed gh CLI requires safeinputs-gh tool (no direct auth)

Overall Status: ⚠️ PARTIAL - Cache memory works, Playwright MCP unavailable

AI generated by Smoke Copilot Playwright

@github-actions
Copy link
Contributor

github-actions bot commented Dec 31, 2025

GitHub MCP (last 2 merged PRs): #8353 "Fix JavaScript formatting in safe_output_handler_manager.test.cjs"; #8352 "Fix non-deterministic tool ordering in compiled workflows with imported safe-jobs".
File write /tmp/gh-aw/agent: ✅
File write /tmp/gh-aw/cache-memory: ✅
Playwright title contains "GitHub": ✅
safeinputs-gh "gh issues list --limit 3": ❌ (safeinputs-gh not found in PATH)
Overall: FAIL

AI generated by Smoke Codex

@pelikhan pelikhan marked this pull request as ready for review December 31, 2025 21:52
@pelikhan pelikhan merged commit 8d2acf9 into main Dec 31, 2025
49 of 50 checks passed
@pelikhan pelikhan deleted the copilot/investigate-create-issue-message branch December 31, 2025 21:52
@github-actions github-actions bot mentioned this pull request Jan 1, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pelikhan pelikhan Awaiting requested review from pelikhan

Assignees

@pelikhan pelikhan

Copilot code review Copilot

Labels

smoke smoke-codex-firewall smoke-copilot smoke-copilot-no-firewall

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pelikhan