Repository navigation
Add shadow-mode PR risk check - #531
Merged
Merged
Conversation
Contributor
Author
|
Picked sjmiller609 to review these related PRs (kernel/security-workflows#25, kernel/infra#1825, kernel/kernel#5021, kernel/kernel-images-private#517, #531): recent commits in |
sjmiller609
approved these changes
Oct 27, 2026
|
|
||
| jobs: | ||
| classify: | ||
| uses: kernel/security-workflows/.github/workflows/pr-risk.yml@main |
Collaborator
There was a problem hiding this comment.
can you double check we only auto approve for members of the kernel org?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds the caller for the shared
pr-risk.ymlworkflow from kernel/security-workflows. When a PR is opened or marked ready, and on every push, it comments whether a PR would have been auto-approved and why. It does not approve anything, so the required human review is unchanged.PRs that are drafts, come from forks, change more than 50 lines, or touch a deny-listed path are skipped with no comment. The
deny-pathshere add this repo's sensitive areas on top of the shared defaults (.github/**, dependency manifests and lockfiles, key and secret material).Secrets are passed by name rather than inherited. This repo already has all three, because Semgrep and vuln-remediation use them.
Merge order
Merge after kernel/security-workflows#25. Until that lands, this workflow references a file that doesn't exist yet on
main. This PR changes.github/**, so the classifier will skip it once it is live.Testing
actionlintis clean.deny-pathsglob matches at least one file currently in the repo.Note
Low Risk
Adds a read-only CI workflow that comments on PRs; it does not change merge gates or application code.
Overview
Adds a shadow-mode PR risk GitHub Actions workflow that runs on PR open, ready-for-review, and sync. It reuses
kernel/security-workflows/.github/workflows/pr-risk.yml@mainto comment whether a small PR would have been auto-approved and why—without granting approval, so human review stays required.Repo-specific
deny-pathsextend the shared defaults to always excludedeploy/**,lib/network/**, andlib/egressproxy/**. Three secrets (ADMIN_APP_ID,ADMIN_APP_PRIVATE_KEY,ANTHROPIC_API_KEY) are passed explicitly instead ofsecrets: inheritwhen calling the external workflow.Reviewed by Cursor Bugbot for commit 4c5c3ca. Bugbot is set up for automated code reviews on this repo. Configure here.