Fix recursive acceptance helper search (#23323) #691
Workflow file for this run
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
| name: Claude PR Review | |
| on: | |
| pull_request: | |
| types: [opened, ready_for_review, reopened] | |
| # NOTE: `pull_request_target` would let this workflow review fork PRs | |
| # (with access to secrets), but the action currently fails during OIDC | |
| # token exchange with "401 Unauthorized - Invalid OIDC token" on that | |
| # event. PR #579 added `pull_request_target` routing to the action, but | |
| # Anthropic's `/github-app-token-exchange` endpoint appears not to | |
| # accept the token claims produced by that event. Re-enable once the | |
| # upstream issue is resolved. | |
| # See: https://github.com/anthropics/claude-code-action/issues/347 | |
| # https://github.com/anthropics/claude-code-action/issues/621 | |
| # pull_request_target: | |
| # types: [opened, ready_for_review] | |
| permissions: | |
| contents: read | |
| pull-requests: write | |
| id-token: write | |
| actions: read | |
| jobs: | |
| review: | |
| # Skip fork PRs: secrets are not exposed on `pull_request` events from | |
| # forks, so the action would fail with a red check. Remove this clause | |
| # once upstream fork support lands (tracked in | |
| # https://github.com/anthropics/claude-code-action/issues/939) and we | |
| # can re-enable the `pull_request_target` trigger above. | |
| if: >- | |
| github.event.pull_request.draft == false | |
| && github.event.pull_request.head.repo.full_name == github.repository | |
| runs-on: ubuntu-latest | |
| steps: | |
| - uses: actions/checkout@v6 | |
| with: | |
| fetch-depth: 1 | |
| - uses: anthropics/claude-code-action@v1 | |
| with: | |
| anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY_03 }} | |
| # Enable progress tracking | |
| track_progress: true | |
| # Debug (set to true to show full output in logs, false to hide it and only post comments on the PR) | |
| show_full_output: false | |
| additional_permissions: "actions: read" | |
| claude_args: "--model claude-sonnet-4-6 --allowedTools 'Bash(gh:*),Bash(git:*)'" | |
| prompt: | | |
| You are reviewing pull request #${{ github.event.pull_request.number }} | |
| in the Umbraco CMS repository. | |
| Read and execute the review procedure defined in `.claude/skills/umb-review/SKILL.md`. | |
| For each finding that references a specific file and line: | |
| - Post an individual inline PR comment on that line. | |
| - Format: **[Severity]** explanation, then suggestion. | |
| For the overall summary (header, impact, verdict): | |
| - Post ONE top-level PR comment. | |
| Do NOT use sticky/updating comments — post new individual comments. | |
| After reviewing, apply labels to the PR based on changed files: | |
| - `area/frontend` — if files under `src/Umbraco.Web.UI.Client/` are changed | |
| - `area/backend` — if .cs files outside the frontend client are changed | |
| - `area/test` — if only test files are changed | |
| - `category/api` — if Management API or Delivery API files are changed | |
| - `category/breaking` — if breaking changes were detected in the review | |
| - `category/localization` — if localization/language files are changed | |
| - `category/test-automation` — if only test files are changed | |
| - `category/refactor` — if the PR is pure refactoring with no new features | |
| - `category/performance` — if performance-related changes are detected | |
| - `category/ux` — if user-facing changes are detected | |
| - `category/ui` — if changes to the UI layer are detected | |
| Only apply labels you are confident about. Never remove existing labels. | |
| Be friendly and constructive. This project values community contributions. | |
| Frame feedback as suggestions where possible. | |
| Reserve firm language for genuine Critical issues only. | |
| Run fully autonomously. Do NOT ask questions. | |
| Only review changed files. Do not flag pre-existing issues. | |
| Do not suggest changes that would themselves introduce breaking changes. |