Skip to content

fix(cla): store CLA signatures locally so the check can actually run - #5

Open
AbirAbbas wants to merge 1 commit into
mainfrom
fix/cla-workflow-remote-storage
Open

AbirAbbas wants to merge 1 commit into
mainfrom
fix/cla-workflow-remote-storage

Conversation

@AbirAbbas

Copy link
Copy Markdown

The CLA check has never passed in this repo. .github/workflows/cla.yml sets remote-organization-name/remote-repository-name to this same repo, which switches contributor-assistant onto its remote-storage path — and that path calls getPATOctokit(), which fails the job outright when PERSONAL_ACCESS_TOKEN is absent (only GITHUB_TOKEN is provided, and the repo has no secrets). It happens inside getFileContent(), which runs unconditionally after the allowlist filter, so allowlisted maintainers fail too — the errors are Please add a personal access token… followed by Could not retrieve repository contents. Status: unknown.

All 12 pull_request_target runs since 2026-03-09 failed identically, across four branches and every author. Dropping the two inputs points the action at the current repo — the same destination — using GITHUB_TOKEN, and contents moves to write so it can create and update cla-signatures.json on main. workflow-templates/cla.yml is byte-identical and ships the same bug to any repo that adopts the org template, so it gets the same fix.

Because pull_request_target loads the workflow from the base branch, the cla check on this PR still runs the broken copy from main and will be red. After this merges, a recheck comment on #3 and #4 re-runs them against the fixed workflow and turns them green.

🤖 Generated with Claude Code

The workflow set remote-organization-name/remote-repository-name to this
same repo, which sends contributor-assistant down its remote-storage path.
That path calls getPATOctokit(), which fails the job when PERSONAL_ACCESS_TOKEN
is absent — and it runs unconditionally in getFileContent(), after the
allowlist filter, so even allowlisted maintainers hit it.

Dropping the two inputs makes the action use GITHUB_TOKEN against the current
repo, which is the same destination; contents needs write so it can create and
update cla-signatures.json on main.

workflow-templates/cla.yml carries the identical bug to every repo that adopts
the org template, so it gets the same fix.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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