fix(ci): restrict self-hosted PR-Agent comment triggers

This commit is contained in:
Tam Nhu Tran
2026-04-14 12:43:25 -04:00
parent 25216eaf33
commit e1271e2dbf
4 changed files with 7 additions and 7 deletions
+3 -6
View File
@@ -60,13 +60,10 @@ jobs:
(
github.event_name == 'issue_comment' &&
github.event.issue.pull_request &&
startsWith(github.event.comment.body, '/') &&
github.event.comment.body == '/review' &&
(
github.event.sender.type != 'Bot' ||
(
github.event.sender.login == 'github-actions[bot]' &&
github.event.comment.body == '/review'
)
contains(fromJSON('["COLLABORATOR","MEMBER","OWNER"]'), github.event.comment.author_association) ||
github.event.sender.login == 'github-actions[bot]'
)
)
)
+1
View File
@@ -83,6 +83,7 @@ CCS PR review no longer depends on `anthropics/claude-code-action`. The reposito
- The retained `.github/workflows/ai-review.yml` runs PR-Agent in GitHub Actions.
- PR-Agent reviews run on the existing self-hosted `cliproxy` runner.
- Use `/review` on the PR when you need a fresh pass after follow-up commits.
- Only the trusted `/review` comment path is enabled on the privileged self-hosted runner.
- Keep repository-level model choice and reviewer instructions in the root `.pr_agent.toml`.
- Keep automation wiring in `ai-review.yml`, using the workflow env for `OPENAI.*` and `github_action_config.*` settings rather than `.pr_agent.toml`.
- If you change review defaults, update the workflow or `.pr_agent.toml` alongside the contributor or architecture docs in the same PR.
+1 -1
View File
@@ -438,7 +438,7 @@ Configured model from `.pr_agent.toml`
- `ai-review.yml` owns automation wiring such as runner selection, PR-Agent action usage, and environment flags like `OPENAI.*` plus `github_action_config.*`.
- `.pr_agent.toml` in the repo root owns model selection and review instructions for this repository.
- Contributors should treat PR-Agent comments and `/review` reruns as the primary AI review path for PRs targeting CCS.
- Contributors should treat PR-Agent comments and trusted `/review` reruns as the primary AI review path for PRs targeting CCS.
### Runtime Dependencies
@@ -24,6 +24,8 @@ describe('PR-Agent review lane migration', () => {
expect(workflow).toContain('OPENAI_KEY');
expect(workflow).toContain('config.model');
expect(workflow).toContain('github_action_config.auto_review');
expect(workflow).toContain("github.event.comment.body == '/review'");
expect(workflow).toContain('github.event.comment.author_association');
expect(workflow).not.toContain('uses: anthropics/claude-code-action@v1');
expect(workflow).not.toContain('AI_REVIEW_API_KEY');