ci: point the Claude reviewer pilot at the current accounts - #37293
Open
danielsolis-dotcms wants to merge 2 commits into
Open
ci: point the Claude reviewer pilot at the current accounts#37293danielsolis-dotcms wants to merge 2 commits into
danielsolis-dotcms wants to merge 2 commits into
Conversation
The reviewer's Gate 1 allowlist skips any PR whose author is not listed, so review was silently not running for this author. Adding the login; no other change to the workflow. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
danielsolis-dotcms
changed the title
Two of the listed logins are no longer members of the dotCMS org, so they cannot open a PR here and their entries had no effect. Meanwhile two accounts that do open PRs here were not listed, so the reviewer was skipping them. Replaces the list rather than appending, so it reflects who can currently receive a review. gortiz-dotcms is unchanged. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
danielsolis-dotcms
force-pushed
the
add-danielsolis-to-claude-reviewer-pilot
branch
from
August 31, 2026 02:55
1b6e555 to
217ace8
Compare
Open
5 tasks
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
1 participant
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.
Proposed Changes
Updates
PILOT_AUTHORSin.github/workflows/ai_claude-backend-reviewer.yml:Two of the listed logins are no longer members of the dotCMS organization, so they cannot open a PR here and their entries had no effect. Meanwhile two accounts that do open PRs here were not listed, so the reviewer was skipping them.
Replaced rather than appended, so the list reflects who can currently receive a review.
gortiz-dotcmsis unchanged.Checklist
Additional Info
Found because the reviewer silently skipped #37288. Gate 1 logs
ℹ️ … is not in the pilot listand exits, but nothing surfaces on the PR — so a gated skip is indistinguishable from a broken bot.The workflow's own comment suggests the durable fix: "To expand: add logins to PILOT_AUTHORS or replace with a team membership check." A team check would keep this in step with org membership automatically. Not attempted here — it needs a token with
read:org, whichsecrets.GITHUB_TOKENdoes not provide.Related to #37294, which tracks that change. This PR does not close it — it is the interim correction to the hardcoded list, and #37294 remains open until the allowlist is replaced by a team membership check.
Requires
@dotCMS/dotDevelopersreview per CODEOWNERS.