Skip to content

Bring detective noop-on-no-failure changes onto updated branch#609

Merged
strawgate merged 5 commits intomainfrom
fix/pr-labeler-on-the-fence
Mar 8, 2026
Merged

Bring detective noop-on-no-failure changes onto updated branch#609
strawgate merged 5 commits intomainfrom
fix/pr-labeler-on-the-fence

Conversation

@strawgate
Copy link
Collaborator

Summary

  • Merge the detective workflow updates from PR Noop on no failure in detective workflows #606 into this branch.
  • Keep the branch updated with latest origin/main and preserve regenerated detective lockfiles.
  • Include the prompt/README behavior where detective workflows explicitly noop when no jobs are failing.

Test plan

  • make compile

Made with Cursor

Copilot AI and others added 5 commits March 8, 2026 19:39
When the PR Actions Detective or Branch Actions Detective runs against a
workflow run that has no failed jobs, the agent now calls noop instead of
posting a comment or creating an issue. This prevents noisy output on
successful runs.

Fixes: Noop on no failure

Co-authored-by: strawgate <6384545+strawgate@users.noreply.github.com>
Teach PR Labeler to keep existing classification labels when confidence is borderline, and expose GH_TOKEN/GITHUB_TOKEN during repo-specific setup commands so workflow setup hooks can use gh/REST auth consistently.

Made-with: Cursor
@strawgate strawgate merged commit 7c3df58 into main Mar 8, 2026
18 of 19 checks passed
@strawgate strawgate deleted the fix/pr-labeler-on-the-fence branch March 8, 2026 23:02
@github-actions github-actions bot added the medium_boom Medium PR blast radius; likely benefits from human review label Mar 8, 2026
@coderabbitai
Copy link

coderabbitai bot commented Mar 8, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Multiple GitHub Actions Detective workflow files were updated to include an early no-op guard that triggers when a workflow run has no jobs with failure conclusions. When triggered, the system emits noop with the message "No failed jobs in workflow run; nothing to report" and halts further processing. Documentation for the detective workflows was also updated to reflect noop as a supported Safe Output.

Possibly related PRs

Suggested labels

medium_boom

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/pr-labeler-on-the-fence
  • 🛠️ Update Documentation: Commit on current branch
  • 🛠️ Update Documentation: Create PR

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

medium_boom Medium PR blast radius; likely benefits from human review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants