[safe-output-health] 🏥 Safe Output Health Report - 2026-08-10 #51688
Closed
Replies: 1 comment
|
This discussion has been marked as outdated by Safe Output Health Monitor. A newer discussion is available at Discussion #51935. |
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_outputsjob invocations: 202 (184 success, 15 skipped — no items to process, 3 failures)push_repo_memory,push_evals_state,push_experiments_state,update_cache_memory,upload_assets,unlock): 0 failuresScope note: this audit covers only
safe_outputsjob health. The 10 agent-job (driver_exit/agent_logic) failures seen in the same window are out of scope — those are handled by other monitors.Safe Output Job Statistics
* 15 of the 202 runs had no safe-output items to process (job skipped); of the 187 runs that actually attempted to process items, 184 succeeded (98.4%).
Per-type breakdown isn't separately trackable from job-level status (all safe-output types for a run are processed in one
safe_outputsjob), but the 3 failures were all specifically caused by one output type:resolve_pull_request_review_thread.Error Clusters
Cluster 1: Stale review-thread node ID in
resolve_pull_request_review_threadsafe_outputs→ step "Process Safe Outputs"pi)resolve_pull_request_review_threadcalled the GraphQLresolveReviewThreadmutation using a thread node ID that no longer resolved by the time thesafe_outputsjob ran (thread already resolved/deleted, or ID gone stale between agent run and safe-outputs processing). The processor treated this as fatal and failed the whole message/job instead of skipping just that one stale item.safe_outputsjob (conclusion: failure) even though other safe-output items in the same run (comments, issues) succeeded independently.This is already fixed. Commit
59c283e("Skip stale review-thread node IDs in resolve_pull_request_review_thread instead of failing safe_outputs", PR #51630) merged 2026-08-10T00:26:15Z — after all 3 observed failures and before the rest of this audit window. Checked every "PR Sous Chef"safe_outputsrun after that timestamp in this window: all succeeded, with zero recurrences of this error signature.Root Cause Analysis
API-Related Issues
The one real failure cluster (above) was a GitHub GraphQL "could not resolve to a node" response for an already-resolved review thread — not a rate limit, auth, or permission problem. No other API-related
safe_outputsfailures were found (no rate-limit or auth errors in anysafe_outputsjob log this period).Data Validation / Parsing Issues
None found in
safe_outputsjob logs this period.Permission Issues
None found. All
safe_outputsjobs that ran had the tokens/scopes they needed; the only related warnings were non-fatalupdate pull request branch from basewarnings ("There are no new commits on the base branch" / workflow-scope timeout) which are expected, already logged as warnings (not errors), and don't fail the job.Other Issues
None found in
safe_outputsjob logs. Zero failures in adjacent write jobs (push_evals_state,push_experiments_state,upload_assets, etc.).Recommendations
Critical Issues (Immediate Action Required)
None. The one cluster found this period is already fixed on
main.Bug Fixes Required
None outstanding — the only bug found (stale review-thread node ID crashing the whole
safe_outputsjob) was already fixed in commit59c283e/ PR #51630 before this audit ran.Configuration Changes
None needed.
Process Improvements
update pull request branch from basefailures ("no new commits on base branch", workflow-scope timeouts) are correctly logged as warnings and don't fail the job.workflowsscope gaps worth a follow-up look. Flagging for future-audit awareness only.Work Item Plans
No new work items — the single defect found this period is already resolved. Recommend closing out tracking of the
resolve-review-thread-stale-node-idpattern in the next audit (2026-08-11) once confirmed there are zero further occurrences.Historical Context
Compared with the 2026-08-09 audit (cache memory: clean, 0 failures out of 201 executions, 100% success):
Trends
mainas of59c283e.Metrics and KPIs
push_evals_state,push_experiments_state,upload_assets, etc.) — 100% across all runs checkedsafe_outputsfor the "PR Sous Chef" workflow specifically, driven entirely by the now-fixedresolve_pull_request_review_threadbugsafe_outputsjob run, no retries observed for this mutation)Next Steps
resolve_pull_request_review_threadstale-node-ID failures recur (verifying commit59c283efully resolves the issue in production)References:
All reactions