Skip to content

fix(ci): support external contributors in Claude workflows - #92

Merged
thomhurst merged 1 commit into
mainfrom
fix/claude-external-users
Sep 4, 2026
Merged

fix(ci): support external contributors in Claude workflows#92
thomhurst merged 1 commit into
mainfrom
fix/claude-external-users

Conversation

@thomhurst

Copy link
Copy Markdown
Owner

Summary

  • allow Claude issue triage and pull request reviews for contributors without repository write access
  • pass the scoped GITHUB_TOKEN explicitly and opt in via allowed_non_write_users
  • isolate untrusted issue/PR content behind pinned-target helper scripts, minimal permissions, and narrow tool allowlists
  • port the fixes from TUnit #6720 and TUnit #6722

Validation

  • actionlint 1.7.12 (both workflows)
  • PyYAML parse (both workflows)
  • bash -n (both helper scripts)
  • helper behavior checks for pinned targets, valid forwarding, invalid-label rejection, and empty-comment rejection

Setup required

This repository does not currently expose a CLAUDE_CODE_OAUTH_TOKEN Actions secret. Add it before expecting either workflow to run successfully.

Pass the scoped GITHUB_TOKEN explicitly because the Claude OIDC token exchange rejects users without write access.

Isolate untrusted issue and pull request input behind pinned write helpers before using pull_request_target.
@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 1611a4b1-a221-4f0d-9100-1c7b02aa3a51


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@thomhurst
thomhurst merged commit 625a845 into main Sep 4, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant