Skip to content

Commit d885e67

Browse files
authored
feat: add claude review to CI (#120)
* feat: add claude review to CI - Updated CODEOWNERS to enforce review requirements for changes in Sandboxed AI CI/CD. - Introduced a new Squid proxy rules configuration file to restrict domain access in the CI sandbox. - Added a new GitHub Actions workflow for Claude review, ensuring security measures and permissions are enforced for PR comments. * chore: update sandbox-proxy-rules.conf to latest version * fix: whitespace * feat: enforce PR author permissions for Claude review - Added a new environment variable `REQUIRE_PR_AUTHOR_PERMISSION` to control whether the PR author must have write/admin/maintain permissions for Claude to review. - Implemented a check in the workflow to validate the PR author's permissions, ensuring only authorized contributors can trigger reviews by Claude. - Updated documentation comments to clarify the security implications of changing the default permission settings.
1 parent 4a5f820 commit d885e67

3 files changed

Lines changed: 671 additions & 1 deletion

File tree

.github/CODEOWNERS

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,8 @@
11
# Zama codeowners rules
22

33
# All pull request should be reviewed by at least one of the members of protocol-apps
4-
* @zama-ai/protocol-apps @mortendahl
4+
* @zama-ai/protocol-apps @mortendahl
5+
6+
# Enforces changes in Sandboxed AI CI/CD
7+
.github/squid/sandbox-*.conf @zama-ai/security-team
8+
.github/workflows/claude*.yml @zama-ai/security-team
Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
# Strict domain allowlist for CI sandbox
2+
# Only these domains are reachable through the Squid proxy.
3+
#
4+
# To add a new domain: append ".example.com" to the acl below.
5+
# Leading dot means "this domain and all subdomains".
6+
7+
acl allowed_domains dstdomain \
8+
.api.anthropic.com \
9+
.platform.claude.com \
10+
api.github.com \
11+
github.com
12+
13+
# Allow only explicitly allowed domains
14+
http_access deny !allowed_domains
15+
http_access allow allowed_domains
16+
17+
# Deny everything else
18+
http_access deny all

0 commit comments

Comments
 (0)