Skip to content

fix(argus): support fork PR reviews safely#256

Draft
bokelley wants to merge 1 commit into
mainfrom
fix/argus-fork-pr-reviews
Draft

fix(argus): support fork PR reviews safely#256
bokelley wants to merge 1 commit into
mainfrom
fix/argus-fork-pr-reviews

Conversation

@bokelley
Copy link
Copy Markdown
Contributor

Summary

  • run Argus on pull_request_target so fork PRs can access the review App token and Anthropic secret
  • keep checkout pinned to the trusted base SHA and inspect PR changes through GitHub APIs
  • avoid local prior-SHA diffs in the skip check, falling back to full review when compare data is unavailable
  • read the review prompt from the trusted base SHA with a randomized heredoc sentinel

Validation

  • git diff --check

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.

1 participant