Skip to content

ci: allow Argus on contributor PRs#2124

Merged
bokelley merged 5 commits into
mainfrom
merge-prs-2100-2102
May 30, 2026
Merged

ci: allow Argus on contributor PRs#2124
bokelley merged 5 commits into
mainfrom
merge-prs-2100-2102

Conversation

@bokelley
Copy link
Copy Markdown
Contributor

Summary

  • Run Argus on pull_request_target so contributor and fork PRs can access the GitHub App and Anthropic secrets needed for review.
  • Keep the workflow on the trusted base SHA and inspect PRs via GitHub APIs instead of checking out PR code.
  • Add a human-review gate for PRs that modify the Argus workflow or prompt, and pin review verification to aao-ipr-bot[bot].

Validation

  • `ruby -e 'require "yaml"; YAML.load_file(".github/workflows/ai-review.yml"); puts "yaml ok"'
  • npx prettier --check .github/workflows/ai-review.yml
  • git diff --check
  • pre-push hook: npm run typecheck and library build

@bokelley bokelley merged commit 387b47d into main May 30, 2026
29 checks passed
@bokelley bokelley deleted the merge-prs-2100-2102 branch May 30, 2026 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants