[safe-output-health] π₯ Safe Output Health Report β 2026-07-24 #47712
Closed
Replies: 1 comment
|
This discussion was automatically closed because it expired on 2026-07-25T05:29:58.976Z.
|
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
run_summary.jsonjob conclusions read directly fromjob_details[])pr_sous_chef_scheduled_target_resolution(both failures)Two
Process Safe Outputshard failures, both the same workflow βPR Sous Chefβ on scheduled/mainruns ~4h apart, each emitting a mixedupdate_pull_request+add_comment+create_issuepayload. Combined with the identical 07-23 failure, this is 3 hard-fails of PR Sous Chef scheduled runs in 2 days. The review-comment family (07-20 / 07-23) did not recur this window.Safe-Output Job Statistics
Handler mix in the 2 failed runs (the only write-actuating safe_outputs jobs observed this window):
Error Cluster:
pr_sous_chef_scheduled_target_resolutionupdate_pull_request,add_commentmain)update_pull_request/add_commenthandlers hard-fail when they cannot resolve a target PR, unlikecreate_pull_request_review_commentwhich soft-skips under the same missing-context condition. This soft-skip-vs-hard-fail handler inconsistency is the long-trackedtarget_star_*family root cause.create_issueis collateral.Failure details (2 runs)
Decisive isolation: PR Sous Chef ran 6Γ this window (all schedule/
main). The 4 read-only/noop runs (23:58Z, 01:25Z, 02:29Z, 03:48Z) succeeded; only the 2 write-actuating runs failed. The failure is specific to actuating target-requiring writes on a scheduled run β not infrastructure and not a blanket PR Sous Chef regression.Observability Gap (recurring, 6th+ occurrence)
The exact
Process Safe Outputsstep stdout/stderr is not pre-bundled in run artifacts. This window theagenticworkflows logsCLI ran quickly (8.9s, no timeout) but returned 0 runs for the{workflow_name:"PR Sous Chef", start_date:"-1d"}filter, so the step log was still unrecoverable. Both failures therefore have UNCONFIRMED exact error text β root cause is inferred from item type + trigger context + family history. This remains the #1 cross-cutting remediation ask.Standing-Open Signatures
review_path_unresolved_422β NOT recurred this window. Full reviewer suite (Matt Pocock, PR Code Quality, Test Quality Sentinel, Impeccable Skills, Design Decision Gate) present at scale but all read-only β no line-anchored review comments, no 422,pr_review_buffer.cjs:554Path-variant fix UNVALIDATED (56th audit). The two recent in-prod hard-fails (07-20, 07-23) remain the standing concern for this family.changeset_generator_bundle_hardfailβ Changeset Generator ABSENT (~28 days since last actuation 2026-06-26).patch-format:BUNDLEbundle-transport signature not re-exercised. Highest-priority UNVALIDATED open signature.assign_to_agentfamily β Issue Monster present but read-only; not exercised in a failing path.Recommendations
Critical / High
update_pull_requestandadd_commentsoft-skip (likecreate_pull_request_review_comment) when no triggering context and no explicit resolvable target exists, instead of hard-failing the job. Root cause of all 3 recent PR Sous Chef failures.Process Safe Outputsstep log on failure (Priority: High) β Every in-scope failure for weeks has had unrecoverable exact error text. Without this, root cause stays inferred.Medium
pull_request_numberperupdate_pull_request/add_commentitem, or gate those writes on a resolvable target, so scheduled runs stop redding.Work Item Plans
WI-1 β Soft-skip target-requiring writes on missing context (Bug Fix, High)
add_comment/update_pull_requestwith no resolvable target soft-skips those items (β) and the safe_outputs job concludes success; regression test covers schedule-trigger + mixed payload.update_pull_request/add_commenthandlers with thecreate_pull_request_review_commentsoft-skip branch. Effort: SmallβMedium.WI-2 β Pre-bundle Process Safe Outputs step log on failure (Enhancement, High)
Historical Context / Trends
Metrics & KPIs
update_pull_request/add_commenton scheduled runs (PR Sous Chef)Next Steps
patch-format:BUNDLE(unvalidated ~28d) andreview_path_unresolved_422Path-variant (unvalidated 56 audits)References:
All reactions