Skip to content

Attest pr#13

Merged
gsavage merged 3 commits into
mainfrom
attest-pr
May 20, 2026
Merged

Attest pr#13
gsavage merged 3 commits into
mainfrom
attest-pr

Conversation

@gsavage
Copy link
Copy Markdown
Contributor

@gsavage gsavage commented May 19, 2026

Adds a kosli-attest step to the apply workflow, to attest the PR to Kosli.

gsavage and others added 3 commits May 19, 2026 17:06
Adds a `kosli attest pr github` step at the end of the
reset-drift-detection job so the PR that triggered the apply is
recorded against the Kosli trail. The Kosli CLI auto-detects the
org, repo, and commit from the GitHub Actions environment, and the
flow/trail/host/api-token come from existing env vars on the job.

A new `github_token` secret is added to the reusable workflow
inputs to authenticate the lookup; callers should pass
`secrets.GITHUB_TOKEN`.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
To support a gradual rollout, callers should be able to adopt the new
reusable-workflow version without immediately supplying a GitHub token.
The github_token secret is already declared `required: false`, but the
attest-PR step previously ran unconditionally and would fail with an
empty token. Gate the step on `secrets.github_token != ''` so it is
skipped when no token is passed, and update the secret description to
match.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The `kosli attest pr github` step queries the GitHub API to find the
pull request associated with the merge commit, which requires the
GITHUB_TOKEN to carry the `pull-requests: read` scope. Declare it on
the job so the token used inside the reusable workflow has the
permission it needs.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@gsavage gsavage merged commit 9769404 into main May 20, 2026
1 check passed
@gsavage gsavage deleted the attest-pr branch May 20, 2026 18:05
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