Skip to content

Document kosli_github_token secret and its required permission#19

Merged
gsavage merged 1 commit into
mainfrom
document-kosli-github-token
May 26, 2026
Merged

Document kosli_github_token secret and its required permission#19
gsavage merged 1 commit into
mainfrom
document-kosli-github-token

Conversation

@gsavage

@gsavage gsavage commented May 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add kosli_github_token to the Secrets table in the README — previously it was only documented in the workflow file itself.
  • Update the example caller workflow to show passing the token and granting the pull-requests: read permission, so anyone copy-pasting it doesn't later hit Resource not accessible by integration when wiring up the PR attestation step.
  • Add a short note explaining that both the caller job and the called job must grant pull-requests: read, since GitHub computes the token's permissions in a reusable workflow as the intersection of the two.

Companion to #18 (which restores the permission on the called side).

Test plan

  • Render the README on GitHub and confirm the table and example look right.
  • Copy the example caller workflow into a fresh repo and confirm kosli attest pr github succeeds.

🤖 Generated with Claude Code

The Secrets table did not list `kosli_github_token`, and the example
caller workflow did not show passing the token or granting the
`pull-requests: read` permission. As a result, anyone copy-pasting the
example and later wiring up the PR attestation step hits a confusing
`Resource not accessible by integration` failure.

Add `kosli_github_token` to the Secrets table, add the secret and the
permission to the example caller workflow, and explain why both sides
of a reusable workflow must grant the permission.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@gsavage gsavage merged commit 4d3bb9b into main May 26, 2026
1 check passed
@gsavage gsavage deleted the document-kosli-github-token branch May 26, 2026 09:57
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.

2 participants