Safe Output Health Report - 2026-03-30 #23553
Closed
Replies: 1 comment
-
|
This discussion has been marked as outdated by Safe Output Health Monitor. A newer discussion is available at Discussion #23723. |
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
All failures were low-to-medium severity. No critical failures. Two recurring pattern types identified.
Safe Output Job Statistics
Error Clusters
Cluster 1:
add_labels— No issue/PR number availableadd_labelsaftercreate_issue, but:create_issueusedgroup_by_daymode and found an existing issue for the day — it posted the content as a comment to issue [Contribution Check Report] Contribution Check — 2026-03-30 #23502 rather than creating a new issue. No new issue number was registered, so the subsequentadd_labelshad no target.create_issuesuccessfully created issue Smoke Test: Claude - 23744944648 #23546, butadd_labelsstill failed with "no issue/PR number available." The temporary IDaw_6GYZbdqqwas registered to Smoke Test: Claude - 23744944648 #23546 butadd_labelsdid not reference it.Cluster 2:
update_issue— Not running in issue contextupdate_issuewithtarget="triggering"three times, but the workflow ran in a non-issue context (scheduled or workflow_dispatch event). The workflow hasupdate_issueconfigured withtarget: triggeringinGH_AW_SAFE_OUTPUTS_HANDLER_CONFIG, and the agent used it accordingly — but the triggering event was not an issue.Cluster 2 — Full Details
Handler config:
{"update_issue":{"allow_body":true,"max":5}}All 3
update_issuemessages hadtarget="triggering"but the run had no triggering issue. The 4add_commentcalls in the same run used explicititem_numberfields (#23447, #23193, #23447 again, #23447 third time) and all succeeded. The agent properly used explicit issue numbers for comments but relied on implicit triggering context forupdate_issue.Root Cause Analysis
Dependent Context Missing (
add_labels)When
create_issueusesgroup_by_day: trueorclose_older_issuesdedup logic, it may not create a new issue — instead posting a comment to an existing one. In this case, no new issue number is exported to the context. A subsequentadd_labelstool call with no explicitissue_numberortemporary_idreference has nothing to target and fails.Additionally, even when
create_issuedoes create a new issue (Smoke Claude case), ifadd_labelsdoesn't reference the temporary ID that was registered, it may not pick up the new issue number automatically.Invalid Event Context (
update_issue/add_comment)This is a recurring pattern (also seen 2026-03-29 with
add_commentin the same Workflow Health Manager). The agent usesupdate_issuewithtarget="triggering"in a workflow that can be triggered by schedule or dispatch events. The agent should use explicitissue_numberfields or guard against non-issue contexts.Recommendations
Immediate Actions
add_labelscross-reference fix — Medium Priorityadd_labelsaftercreate_issue, it should pass thetemporary_idused increate_issueas theissue_numbertarget (e.g.,"issue_number": "aw_6GYZbdqq"). The safe outputs framework supports temporary ID resolution.add_labelshandler should auto-inherit the last created issue number when no explicit target is provided and acreate_issuemessage was processed earlier in the same run.Workflow Health Manager
update_issuefix — Medium Priorityissue_numberfields when callingupdate_issueinstead of relying ontarget="triggering".add_commentcontext failures on 2026-03-29 — the pattern is recurring.Bug Fix Candidates
add_labelsshould inherit created issue numberadd_labelswith noissue_numberfails when called aftercreate_issueingroup_by_dayor dedup modes that don't create a new issue.created_issue_numberfrom earlier in the same run when no explicit target provided.temporary_idreference inadd_labelsaftercreate_issue.add_labelsupdate_issuecontext guardupdate_issuewithtarget="triggering"on workflows triggered without an issue context.update_issuewithtarget="triggering"is used, fail gracefully (already does) OR add agent guidance to always use explicit issue numbers in workflows that can run on non-issue events.update_issueProcess Improvements
Agent prompting for
add_labelsaftercreate_issue: Docs/examples should show thatadd_labelsrequires an explicitissue_numberortemporary_id— it does not auto-inherit from a precedingcreate_issue.Recurring context issue in Workflow Health Manager: This workflow has now produced context-target failures on consecutive days (2026-03-29 with
add_comment, 2026-03-30 withupdate_issue). The agent prompt for this workflow should be updated to use explicit issue numbers for all output operations.Work Item Plans
Work Item 1: Fix
add_labelsto supporttemporary_idresolutionadd_labelsfails with "No issue/PR number available" when called aftercreate_issuein dedup/group_by_day mode because no new issue was created. The safe outputs framework supports temporary ID cross-referencing in other handlers — extendadd_labelsto resolve temporary IDs as issue numbers.add_labelswithissue_number: "aw_XXXX"resolves the temporary ID to the real issue numberadd_labelsaftercreate_issuein same run picks up the created issue number when no explicit target providedgroup_by_daydedup case wherecreate_issueposts as commentadd_commenthandler intoadd_labels. Additionally, consider auto-inheritingcreated_issue_numberexport when no explicit target is set.Work Item 2: Recurring Workflow Health Manager output context failures
issue_numberforupdate_issueandadd_commentcalls. Optionally fail-safe: whenupdate_issuewithtarget="triggering"is called without issue context, silently skip rather than error.Historical Context
Trends:
invalid_event_context(target="triggering" without issue context) appeared 2 days in a row, in the same workflow (Workflow Health Manager). Pattern is escalating: 1 failure yesterday → 3 failures today.add_labels"no issue/PR number" appeared for the first time (2 occurrences, 2 workflows).protected-files-pr-blockandupload-assets-missing-action-setupnot seen today.Metrics and KPIs
Next Steps
add_labelshandler to support temporary ID resolution and/or inheritcreated_issue_numberupdate_issue/add_commentadd_labelscall uses atemporary_idreference after fixReferences:
Beta Was this translation helpful? Give feedback.
All reactions