Skip to content

Commit 793959e

Browse files
authored
Merge pull request #220 from QuantStrategyLab/codex/migrate-review-gate-20260710
chore(ci): harden Codex PR review gate
2 parents 212ecac + 4b45787 commit 793959e

2 files changed

Lines changed: 5 additions & 4 deletions

File tree

.github/workflows/codex_pr_review.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
name: Codex PR Review
22

33
on:
4-
pull_request:
4+
pull_request_target:
55
types: [opened, synchronize, reopened]
66

77
permissions:
@@ -19,10 +19,11 @@ jobs:
1919
uses: QuantStrategyLab/AIAuditBridge/.github/workflows/codex_pr_review.yml@main
2020
with:
2121
caller_concurrency_key: pr-${{ github.event.pull_request.number || github.run_id }}
22-
allow_unconfigured_backend: true
22+
allow_unconfigured_backend: false
2323
api_fallback_enabled: "false"
2424
direct_api_primary_enabled: "false"
25-
secrets: inherit
25+
secrets:
26+
CODEX_AUDIT_SERVICE_URL: ${{ secrets.CODEX_AUDIT_SERVICE_URL }}
2627
permissions:
2728
contents: read
2829
id-token: write

.github/workflows/codex_review_gate.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ permissions:
2323
pull-requests: read
2424

2525
concurrency:
26-
group: codex-review-gate-${{ github.event.pull_request.number }}
26+
group: codex-review-gate-${{ github.event.pull_request.number }}-${{ github.event_name }}
2727
cancel-in-progress: true
2828

2929
jobs:

0 commit comments

Comments
 (0)