Repository navigation
Add Claude review workflow with automated approval - #47
Merged
Merged
Conversation
Claude reviews every same-repository, non-draft PR against AGENTS.md (including the security invariants) and posts inline comments for blocking findings. It only returns a structured verdict; a separate deterministic step approves as github-actions[bot] when the verdict is "no blocking findings", otherwise the job fails. Adapted from the OpenSourceInitiative.LicenseApi setup to this repository's CI rules: actions pinned to commit SHAs, workflow-level contents: read with job-level grants, expressions passed via env, checkout without persisted credentials. AGENTS.md documents the new workflow, its permissions and the CLAUDE_CODE_OAUTH_TOKEN secret. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
frequency403
added a commit
that referenced
this pull request
Nov 10, 2026
claude-code-action only runs when the workflow file on the PR is identical to the one on the default branch. This adds the same file as on development (#47), without a release. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
Automated review by Claude: no blocking findings.
I reviewed the new claude-review.yml workflow and the AGENTS.md update. The workflow uses pull_request rather than pull_request_target, skips fork and Dependabot PRs, and has Claude's tools limited to read-only plus inline comments. Both third-party actions are pinned to commit SHAs. Approval comes from a separate step using GITHUB_TOKEN. That step fails closed when the structured result is empty or reports any findings. I did not verify that the pinned SHAs match their version comments. I found no blocking issues. The PR doesn't touch app code, so the security invariants, resx strings and tests are unaffected.
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 an automated Claude review for pull requests, adapted from the setup in
frequency403/OpenSourceInitiative.LicenseApi.Changes Made
New feature (non-breaking change)
Documentation update
New
.github/workflows/claude-review.yml:pull_request(opened/synchronize/reopened/ready_for_review) for same-repository, non-draft, non-Dependabot PRs. Fork PRs get no secrets;pull_request_targetis not used.anthropics/claude-code-actionreviews the PR diff with read-only tools (Read,Glob,Grep) plus inline comments. The prompt checks correctness, the security invariants fromAGENTS.md(TOFU, key file modes, passphrases, backup/restore, config-provided keys, SFTP), cross-platform tests, both resx files, CPM/DI rules, and CI hardening (SHA pins, secrets, release asset names).{approve, findings, summary}; a separate bash step approves asgithub-actions[bot]only whenapprove == trueandfindings == 0, otherwise the job fails.Differences from the LicenseApi version, to follow this repo's CI rules:
actions/checkoutandanthropics/claude-code-actionare pinned to commit SHAs (claude-code-actionv1.0.262, released 2026-10-24) instead of@v7/@v1.permissions: contents: read, write grants only on the job.envinstead of inline${{ }}inrun:.persist-credentials: false.AGENTS.mdsecurity invariants, resx, release asset contract).AGENTS.md: documents the workflow, its permissions and theCLAUDE_CODE_OAUTH_TOKENsecret.Prerequisites (repository settings, before this works)
CLAUDE_CODE_OAUTH_TOKEN(generate locally withclaude setup-token)Until these are in place, the Claude Review job on this PR is expected to fail.
Testing Performed
uses:are SHA-pinnedNotes for Reviewers:
release/*PRs into master.🤖 Generated with Claude Code