fix(ci): handle 403 when updating coverage comment on fork PRs

Fork PR tokens cannot PATCH comments owned by a different integration.
Fall back to creating a new comment when the update fails instead of
failing the entire step.

Closes #913

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
fireddd 2026-04-10 02:25:01 +05:30
parent 10f906c4d8
commit 4844e2eafa
1 changed files with 9 additions and 3 deletions

View File

@ -101,10 +101,16 @@ jobs:
| tail -1)
if [ -n "$COMMENT_ID" ]; then
gh api "repos/$REPO/issues/comments/$COMMENT_ID" \
# Update may fail on fork PRs (403) — fall back to creating a new comment
if gh api "repos/$REPO/issues/comments/$COMMENT_ID" \
-X PATCH \
-F body=@coverage-comment.md
echo "Updated existing comment $COMMENT_ID"
-F body=@coverage-comment.md 2>/dev/null; then
echo "Updated existing comment $COMMENT_ID"
else
echo "Could not update comment $COMMENT_ID, creating new one"
gh pr comment "$PR_NUMBER" --body-file coverage-comment.md
echo "Created new comment"
fi
else
gh pr comment "$PR_NUMBER" --body-file coverage-comment.md
echo "Created new comment"