[safe-output-health] Safe Output Health Report - 2026-04-05 #24739
Closed
Replies: 1 comment
-
|
This discussion has been marked as outdated by Safe Output Health Monitor. A newer discussion is available at Discussion #24892. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Executive Summary
Safe Output Job Statistics
create_issuecreate_discussionadd_commentcreate_pull_requestcreate_pull_request_review_commentupdate_issueadd_labelsupdate_pull_requestsubmit_pull_request_reviewpush_to_pull_request_branchsend_slack_messagedispatch_workflowcreate_code_scanning_alertadd_reviewerset_issue_typepost_slack_messageremove_labelsupload_assetnoopmissing_toolError Clusters
Cluster 1:
add_commentLimit Exceeded — Smoke Copilotadd_commentadd_comment.max=2. The agent submitted 3add_commentitems — one to a discussion, one to the PR under test, and a 3rd at the very end of the output. The 3rd was rejected.View detailed context
The Smoke Copilot run performed a comprehensive test of all Copilot capabilities including GitHub MCP, MCP Scripts GH CLI, Serena MCP, Playwright, Web Fetch, file writing, PR review, discussion creation, and workflow dispatch. The safe output list contained 13 items (excluding a
missing_toolitem for Serena which is also informational):create_pull_request_review_comment(×2) — PR inline review comments ✅dispatch_workflow— triggered haiku-printer workflow ✅submit_pull_request_review— submitted PR review ✅create_discussion— "copilot was here" discussion ✅create_issue— Smoke Test issue ✅set_issue_type— set issue type to Bug ✅send_slack_message— Slack notification ✅add_comment(×2) — comments on discussion and PR ✅add_labels— addedsmoke-copilotlabel ✅remove_labels— removedsmokelabel ✅add_comment(3rd) — extra comment at the end ❌ rejectedThe 3rd
add_commentwas a duplicate celebration comment on the same discussion, generated at the end of the Copilot's output.Root Cause Analysis
Validation Issues
The sole error today is an agent over-generation issue where Smoke Copilot generated one more
add_commentitem than the workflow's configured maximum. This is a low-severity boundary condition — the validation system correctly caught and rejected the excess item while allowing all prior items to execute.This pattern is distinct from previous issues tracked in the error-patterns database and represents the first occurrence of
add_comment_limit_exceeded.Upload Asset Handling (Non-Issue)
The Daily Firewall Logs Collector run (§24000767919) submitted 2
upload_assetitems that appear absent fromsafe-output-items.jsonl. This is expected behavior:upload_assetitems are handled by a dedicatedupload_assetsGitHub Actions job (separate from thesafe_outputsjob). Logs confirm both PNG files were successfully committed and pushed toassets/Daily-Firewall-Logs-Collector-and-Reporter.Staged Push to PR Branch (Non-Issue)
Smoke Claude runs (§23998581834, §24001448881) show
push_to_pull_request_branchas submitted but not insafe-output-items.jsonl. This is expected behavior: the Smoke Claude workflow runs this operation instaged: truemode, which produces a preview in the step summary but does not actually push. The preview executed successfully.Recommendations
Low Priority
add_commentOver-Generationadd_commentcalls it has already queued.add_commentlimit is 2. Alternatively, increaseadd_comment.maxto 3 in the Smoke Copilot workflow config if a 3rd comment is genuinely useful.Historical Context
View 7-day trend
submit_pr_review_no_contextlow severityadd_commentlimit exceeded (low severity)Trend: Strong positive trend. After a spike on 2026-04-02 (7 of 10 failures from an API rate limit burst during concurrent runs), the system has been at zero hard failures for 3 consecutive days. Soft validation catches are working correctly.
Resolved patterns:
submit_pr_review_no_context(first seen 2026-04-04) — not seen again todaycreate_discussion_missing_labels(2026-04-03) — resolved since 2026-04-04push_to_pull_request_branch_files_outside_allowed_list(2026-04-01/02) — not seen since 2026-04-02api_rate_limit_concurrent_burst(2026-04-02) — not seen sinceMetrics and KPIs
Next Steps
add_commentover-generation in future runs — if it recurs, update the prompt or raiseadd_comment.maxto 3submit_pr_review_no_contextpattern first seen 2026-04-04 — not seen today (potentially a transient issue)References:
Beta Was this translation helpful? Give feedback.
All reactions