-
Notifications
You must be signed in to change notification settings - Fork 36
Add step summaries for safe-output processing results #11453
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
- Created safe_output_summary.cjs helper with generateSafeOutputSummary and writeSafeOutputSummaries functions - Updated safe_output_handler_manager.cjs to generate step summaries after processing messages - Updated safe_output_project_handler_manager.cjs to generate step summaries after processing messages - Each safe-output result is wrapped in a <details> section for collapsibility - Added comprehensive tests for the new functionality - All JavaScript tests pass (3021 passed) Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
Smoke Test ResultsPRs Reviewed:
Test Results:
Overall Status: PASS
|
Smoke Test Results ✅PRs: #11425 (docs: remove-labels), #11450 (MCP tool definition)
Overall: PASS @pelikhan
|
|
docs: Document remove-labels safe output type
|
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
Safe-output handlers now generate collapsible step summaries for each processed message, providing visibility into what was created/updated during workflow execution.
Changes
New module:
safe_output_summary.cjsgenerateSafeOutputSummary()- Renders individual results as<details>blocks with metadata (URL, location, temporary IDs, labels)writeSafeOutputSummaries()- Aggregates and writes summaries to GitHub Actions step summaryHandler integration
safe_output_handler_manager.cjsandsafe_output_project_handler_manager.cjscallwriteSafeOutputSummaries()after message processingExample
Processing 2 safe-outputs now generates:
Failures use ❌ and include error details. Skips standalone-handled types (e.g.,
noop).Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.
Changeset