diff options
| author | Yuriy Andamasov <yuriy@vyos.io> | 2026-05-12 09:21:42 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2026-05-12 09:21:42 +0300 |
| commit | 566cf955b4348cb81fea27e8d21a54e0b335f10e (patch) | |
| tree | f69a3c085fef5c0916cea0429092ee8299cefce5 /.github/workflows | |
| parent | 127041799fb1d329322d4953373ba03343dcd44d (diff) | |
| parent | 68f87deca228ad2b6b07c4eb64f201c0838e494f (diff) | |
| download | vyos-documentation-566cf955b4348cb81fea27e8d21a54e0b335f10e.tar.gz vyos-documentation-566cf955b4348cb81fea27e8d21a54e0b335f10e.zip | |
Merge pull request #2002 from vyos/yuriy/ai-validation-revert-origin-retarget
ci(ai-validation): stop retargeting origin to fork URL
Diffstat (limited to '.github/workflows')
| -rw-r--r-- | .github/workflows/ai-validation.yml | 28 |
1 files changed, 7 insertions, 21 deletions
diff --git a/.github/workflows/ai-validation.yml b/.github/workflows/ai-validation.yml index 993d1876..2060e042 100644 --- a/.github/workflows/ai-validation.yml +++ b/.github/workflows/ai-validation.yml @@ -293,27 +293,13 @@ jobs: persist-credentials: false fetch-depth: 2 - # claude-code-action subsequently runs `git fetch origin <head-ref>` - # internally during its setup. `<head-ref>` is the branch name in - # the PR's HEAD repo — for fork PRs that branch does NOT exist on - # the base repo (which is what `origin` points at after the merge- - # ref checkout above), so the fetch fails with - # fatal: couldn't find remote ref refs/heads/<head-ref> - # - # Re-point origin to the HEAD repo's URL so the action's fetch - # resolves. The workspace tree is unchanged — it remains the merge - # tree from above. For public forks (the only kind that target - # vyos/vyos-documentation, since the repo is public) the - # unauthenticated fetch from the fork succeeds without any credential - # (persist-credentials:false stripped the extraheader at the end of - # the previous step). - - name: Re-point origin to fork URL for claude-code-action's git fetch - if: steps.secrets-check.outputs.skip != 'true' - env: - FORK: ${{ github.event.pull_request.head.repo.full_name }} - run: | - set -euo pipefail - git remote set-url origin "https://github.com/$FORK" + # Note: claude-code-action runs `git fetch origin pull/<n>/head` for + # fork PRs during its setup. Those refs only exist on the base repo, + # so `origin` must remain pointed at vyos/vyos-documentation (which + # is where actions/checkout above left it). Do NOT retarget origin + # to the fork URL — that breaks the fork-PR fetch path with + # fatal: couldn't find remote ref pull/<n>/head + # (verified on run 25689321499, PR #1891). # Defense-in-depth: actions/checkout above brings the PR's merge # tree into the workspace root, which means a malicious fork could |
