summaryrefslogtreecommitdiff
path: root/scripts
diff options
context:
space:
mode:
authorYuriy Andamasov <yuriy@vyos.io>2026-05-29 12:15:36 +0300
committerMergify <37929162+mergify[bot]@users.noreply.github.com>2026-05-29 09:21:01 +0000
commita45fc8e983b72ff5d0d99427fb564ad0ef2f7ce1 (patch)
tree1d6c9ec2c1a90dc3a5ff7d95cd1ee8fd8705a676 /scripts
parentb438d113486c1be4094f95ffe15bf67fbc6521be (diff)
downloadvyos-documentation-a45fc8e983b72ff5d0d99427fb564ad0ef2f7ce1.tar.gz
vyos-documentation-a45fc8e983b72ff5d0d99427fb564ad0ef2f7ce1.zip
ci(ai-validation): allow Pass 2 review on external-contributor PRs
The upstream `anthropics/claude-code-action` performs a write-permission check on `github.actor` before our skip logic runs. On `pull_request_target` the actor is the PR author; external contributors resolve to `read` and the action exits 1 with `Actor does not have write permissions to the repository`. Net effect: AI validation has been failing on every external-contributor PR (LiudmylaNad, teslazonda, scottlaird in the last 4 weeks) while succeeding on maintainer PRs. Failure reproduced on run 26541079685 (PR #2061). Fix: set `allowed_non_write_users: '*'` on the Pass 2 step. The action bypasses the actor check when this input is set and `github_token` is provided (already the case). The action also auto-scrubs Anthropic / cloud / GHA secrets from subprocess envs when this input is set. Safe in THIS workflow because the existing defense-in-depth bounds what Pass 2 can do with untrusted PR content: - `allowedTools` restricted to inline-comment + read-only surfaces - `github_token` is the PR-scoped default (not the broader VYOS_APP_ID) - prompt marks PR content as untrusted via `<UNTRUSTED-PR-CONTENT>` - workspace-wipe removes `CLAUDE.md` / `.claude/` before Pass 2 - prepare bundles MD via `git show HEAD:<path>` (blob, not `cp`) Full rationale inlined as a comment block above the new input. 🤖 Generated by [robots](https://vyos.io) (cherry picked from commit 1fa39bd7ac64898f7a21922bc0f195d115d3cdeb)
Diffstat (limited to 'scripts')
0 files changed, 0 insertions, 0 deletions