Skip to content

check: no commit by a test identity - #10

Merged
aledbf merged 1 commit into
mainfrom
authors-gate
Nov 13, 2026
Merged

aledbf merged 1 commit into
mainfrom
authors-gate

Conversation

@aledbf

@aledbf aledbf commented Nov 13, 2026

Copy link
Copy Markdown
Member

Follow-up to spin-stack/spin#1196 and spin-stack/go-tools#104. A test in spin wrote ana@corp.example into a checkout's .git/config, and every commit made after that carried it to GitHub.

  • task check now runs go tool testidentity -base origin/main. It fails on a commit since main whose raw author or committer address is at a reserved domain, has no dot, or appears in this tree's tests or fixtures, and the failure says how to fix it. GitHub's noreply and bot addresses pass.
  • go.mod adds the tool and moves go-tools to 1c6d5ec, the commit that has it. go mod tidy moved otel and grpc (indirect only).
  • The Gate's checkout uses fetch-depth: 0 so origin/main is in the clone on a pull request.

Checked: go tool task check passes. A planted commit by ana@corp.example fails it with exit 1 and the fix steps.

🤖 Generated with Claude Code

A test in spin wrote git's author into the checkout's config, and every commit after it was the
test's, on GitHub. task check now runs go-tools' testidentity over the branch's commits since
origin/main - a reserved-domain address, one with no dot, or one this tree's tests spell - so
such a commit stops here. The Gate fetches the history it reads; go-tools moves to the commit
that has the command.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@aledbf
aledbf merged commit cf985f4 into main Nov 13, 2026
1 check passed
@aledbf
aledbf deleted the authors-gate branch November 13, 2026 06:29
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