fix: post leaderboard validation comments via workflow_run for fork PRs#454
Conversation
Fork PRs get a scoped-down GITHUB_TOKEN that lacks write permissions, so the inline github-script comment step always 403s. Split the comment posting into a separate workflow_run-triggered workflow that runs in the upstream repo context with elevated permissions, following the same pattern as coverage-comment.yml. The validate job now saves results as an artifact; the new leaderboard-comment.yml downloads it and posts/updates the PR comment. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
📈 Test Coverage Report
Coverage calculated from unit tests only |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Enterprise Run ID: 📒 Files selected for processing (2)
Cache: Disabled due to data retention organization setting Knowledge base: Disabled due to data retention organization setting 📝 WalkthroughWalkthroughThe PR decouples leaderboard validation from PR commenting by introducing a two-workflow system. The upstream ChangesLeaderboard Validation Artifact and Comment Workflow
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
🎉 This PR is included in version 2.39.3 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary
GITHUB_TOKENthat lacks write permissions, so the inlinegithub-scriptcomment step in the leaderboard validation always returns 403.workflow_run-triggered workflow (leaderboard-comment.yml) that runs in the upstream repo context with elevated permissions.coverage-comment.yml.Fixes the comment-posting failure seen on #424 and #448.
Test plan
Generated with assistance from Claude Code by Bill Murdock.
Summary by CodeRabbit