adr check: find the author of a status change as the forge PR author #655

New issue

Open

opened 2026-09-29 16:37:15 +00:00 by hexajon · 0 comments

hexajon commented

2026-09-29 16:37:15 +00:00

Owner

What to build

For each decided record, find the commit that set its status, then the PR that merged that commit, through the forge's commit-to-PR endpoint (GetCommitPR, internal/gitea/client_branchedit.go:35). The PR author is the author of the status change. When no PR exists, the author is unknown, and the check accepts a sign-off token from any contributor.

The lookup must pass the full commit hash. This repo uses SHA-256 object IDs, and the endpoint returns 404 for a short hash.

Acceptance

Tests against the live forge API and a test repo: a status set in a PR gives that PR's author; a short hash is expanded; a commit with no PR gives no author.

## What to build For each decided record, find the commit that set its status, then the PR that merged that commit, through the forge's commit-to-PR endpoint (`GetCommitPR`, `internal/gitea/client_branchedit.go:35`). The PR author is the author of the status change. When no PR exists, the author is unknown, and the check accepts a sign-off token from any contributor. The lookup must pass the full commit hash. This repo uses SHA-256 object IDs, and the endpoint returns 404 for a short hash. ## Acceptance Tests against the live forge API and a test repo: a status set in a PR gives that PR's author; a short hash is expanded; a commit with no PR gives no author.

hexajon referenced this issue

2026-09-29 16:40:53 +00:00

adr check: enforce sign-off on decided records #661

hexajon added a new dependency

2026-09-29 16:41:10 +00:00

#661 adr check: enforce sign-off on decided records

hexajon referenced this issue

2026-09-29 19:08:36 +00:00

epic: a decided record needs a second contributor's sign-off #673

hexajon added a new dependency

2026-09-29 19:08:42 +00:00

#673 epic: a decided record needs a second contributor's sign-off