Tolerate a Per-Field workflowRun Error in the Check Query

[ptr727]

Summary

• gh_graphql reads the JSON gh api graphql leaves on stdout when it exits non-zero, and tolerates an error set made only of errors whose path ends in workflowRun. Any other error, an empty or missing path, a non-zero exit with no JSON errors, or null data still aborts the digest.
• A tolerated response carries a degraded marker to check_nodes, which then keys each Actions run by its own suite and drops none, and the digest says so (CHECKS DEDUPED WITHOUT WORKFLOW KEYS). A warning goes to stderr.
• Tests cover a tolerated workflowRun-only error, a mixed or other-path error aborting, null data aborting, the same-named jobs of two workflows both tallying, and a third-party app rerun still deduping by app slug.

Deliberate deviation from #2338 "Done When"

The issue asked for the dedupe to fall back to the app slug. While degraded, that collapses same-named jobs of two workflows into one group and keeps only the highest suite id, which reads a failure as green. So Actions runs are keyed by suite instead (a superseded failure may show, which reads red rather than hiding a failing check), and other apps keep the app-slug key.

Not reproduced

The error path was never reproduced against a live token. The tolerance matches on path only, with no error type filter. A denial reported on workflowRun.workflow rather than workflowRun would still abort, as before.

Addresses #2338

🤖 Generated with Claude Code

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

• Configuration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: 7909be36-93a1-451f-b73c-55acf47cba2b

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 57.86%. Comparing base (f63c3fb) to head (855a76b).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #2355      +/-   ##
===========================================
+ Coverage    57.70%   57.86%   +0.15%     
===========================================
  Files           16       16              
  Lines         7919     7944      +25     
===========================================
+ Hits          4570     4597      +27     
+ Misses        3349     3347       -2     

Flag	Coverage Δ
python-3.13	57.86% <100.00%> (+0.15%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Copilot review overview

🟢 Approval recommended

Only a minor documentation nit remains, with no approval-blocking issues identified.

Review effort: Lite
Findings: 1

Open (1)

• Correct suite-key fallback docstring and grammar · New

What changed in this PR

This pull request makes GraphQL check retrieval resilient to per-field workflowRun errors while preserving conservative deduplication.

Changes:

• Tolerates only workflow-specific GraphQL errors and marks degraded responses.
• Uses suite-based deduplication for affected Actions checks.
• Adds coverage for tolerated, rejected, and deduplication scenarios.

File	Summary
tests/​test_pr_review.py	Adds transport, failure-path, and degraded deduplication tests.
scripts/​pr_review.py	Implements tolerant GraphQL handling, degraded propagation, diagnostics, and suite-based fallback.

---

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Copilot review overview

🟡 Changes recommended

A critical degraded Actions deduplication issue remains unresolved.

Review effort: Lite
Findings: 1

Open (1)

• Key degraded Actions runs by suite before workflow · New

Resolved since last review (1)

• Correct suite-key fallback docstring and grammar