fix: codex pipeline — auto-trigger review and prevent duplicate PRs#10
Conversation
|
To use Codex here, create an environment for this repo. |
|
@codex Please review this PR. |
1 similar comment
|
@codex Please review this PR. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 13df90cef4
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| jobs: | ||
| trigger: | ||
| if: contains(github.event.pull_request.labels.*.name, 'auto-fix') |
There was a problem hiding this comment.
The labeled trigger is currently gated by contains(github.event.pull_request.labels.*.name, 'auto-fix'), so once a PR has auto-fix, adding any other label will post @codex Please review this PR. again and can retrigger duplicate Codex review runs/comments. This should be narrowed to the specific label event (for example, checking that the newly added label is auto-fix) to avoid repeated triggers on unrelated label changes.
Useful? React with 👍 / 👎.
|
@codex Please review this PR. If you find no critical issues, reply with "LGTM". If you find critical issues, describe them clearly. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4a2c22dd48
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| github-token: ${{ secrets.GITHUB_TOKEN }} | ||
| script: | | ||
| const CODEX_USER = 'FuugaMo'; | ||
| const CODEX_USER = 'chatgpt-codex-connector'; |
There was a problem hiding this comment.
Updating CODEX_USER here to chatgpt-codex-connector makes codex-gate.yml accept approvals from that account, but .github/workflows/auto-merge.yml still excludes only FuugaMo when checking for a human approver (CODEX_USER = 'FuugaMo' and user !== CODEX_USER). In this state, a Codex approval from chatgpt-codex-connector can satisfy both the Codex status and the “human approval” condition, allowing auto-merge without any real human reviewer.
Useful? React with 👍 / 👎.
1 participant
Changes
didn't find any,no major issues,hoorayto positive keyword list so Codex's typical comment style is recognizedauto-fixPR is opened, automatically posts@codex Please review this PR.as FuugaMo usingFUUGAMO_PATsecretauto-fixPR is opened, addshas-prlabel to the referenced issue — prevents OpenClaw from opening duplicate PRsTest plan
has-prlabel in GitHub Labels