[safe-output-health] π₯ Safe Output Health Report β 2026-06-14 #39187
Closed
Replies: 1 comment
-
|
This discussion was automatically closed because it expired on 2026-06-15T06:06:51.457Z.
|
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
Smoke Claude,Smoke Copilot) β no production safe_outputs job failedSafe Output Job Statistics
Job conclusions confirmed per-run via the
auditMCP tool. The 6 out-of-scope runs all hadsafe_outputsskipped (agent failed first) or emitted no items.Failed messages:
add_commentβdiscussion (Smoke Claude),add_labels+remove_labels(Smoke Copilot).Error Clusters
Cluster 1 β RECURRENCE:
add_labels/remove_labelshard-fail with no issue/PR contexttarget_star_add_labels_no_item_number_hardfail(first seen 06-11). 2 messages, 1 run.workflow_dispatch)β Message 14 (add_labels) failed: No issue/PR number available(+remove_labelsmsg 15)workflow_dispatchcontext.add_labels/remove_labelshard-fail the job β yet in the same runcomment_memory,create_pull_request_review_comment, andreply_to_pull_request_review_commentsoft-skip (β) the identical condition. Reproduced on 2 separate days β strongest evidence yet of the soft-skip-vs-hard-fail inconsistency.Cluster 2 β NEW:
add_commentβ discussion "Resource not accessible by integration"add_comment_discussion_resource_not_accessible_by_integration. 1 message, 1 run.pull_requestFix AIC usage cache always empty in activation jobΒ #39130)#335 not found as issue/PR, retrying as discussion... ##[error]Failed to add comment to discussion: ... Resource not accessible by integrationsafe_outputsjob token grants Checks/Issues/PullRequests/SecurityEvents=write, Contents/Metadata=read β noDiscussionsscope.add_comment404'd as issue/PR, fell through to the discussion path, andaddDiscussionCommentneedsdiscussions: write. The 06-04 audit saw this same 404βdiscussion fallback succeed, so this is a fresh exposure of the missing scope. (resolve_pull_request_review_threadin the same run soft-skipped on the same permission family; the PR review #4492134465 submitted cleanly, no 422.)Root Cause Analysis
resolve_pull_request_review_threadsoft-skip (job stays green);add_labels,remove_labels,update_issue(06-11), and the discussionadd_commentfallback hard-fail (job red).discussions: writeon the smokesafe_outputstoken (Cluster 2).Recommendations & Work Items
No critical issues β both in-scope failures are smoke-only.
Soft-skip
add_labels/remove_labelson missing context (Bug, Medium, Small effort).When no issue/PR number is resolvable, emit
βwith a message suggestingtarget: "*"+ explicit number, instead of##[error]. Add a unit test for the no-context branch. Ideally landed with the open LintMonsterupdate_issue_target_triggering_default_hardfails_scheduled_runfix (same family).add_labels/remove_labelsno longer fail the job; Smoke Copilotworkflow_dispatchgoes green.Graceful discussion
add_commentpermission handling (Bug/Config, Low, Small effort).Either grant the
safe_outputsjobdiscussions: write(if discussion comments are intended), or treat a permission-scope failure on the best-effort discussion fallback as a soft-skip (likeresolve_pull_request_review_thread); surface the missing scope in the error. Optionally point the smokeadd_commenttarget (Welcome to Agentic Workflows!Β #335) at a resolvable issue/PR.Historical Trend
target:"*"fix still UNVALIDATED).review_path_unresolved_422Path-variant fallback (pr_review_buffer.cjs:554) UNVALIDATED for the 17th consecutive audit.Next Steps
add_labels/remove_labels(+ LintMonsterupdate_issue).add_comment.References:
Beta Was this translation helpful? Give feedback.
All reactions