Skip to content

Record the upstream commit as the Mayhem run's build id - #260

Merged
ethan42 merged 2 commits into
parent-revisionfrom
build-id
Oct 20, 2026
Merged

ethan42 merged 2 commits into
parent-revisionfrom
build-id

Conversation

@ethan42

@ethan42 ethan42 commented Oct 19, 2026

Copy link
Copy Markdown
Member

Pass --build-id set to the fork's merge base with upstream (overridable via a build-id input). Stacked on
#259.

For a fork, mcode-action now passes --build-id: the newest commit the recorded
revision shares with the upstream default branch, from the GitHub compare API.
A build-id input overrides it; nothing is sent for a repository that is not a fork.
@ethan42
ethan42 merged commit 030f7e5 into parent-revision Oct 20, 2026
1 check passed
@ethan42
ethan42 deleted the build-id branch October 20, 2026 10:03
ethan42 added a commit that referenced this pull request Oct 20, 2026
* Record the parent commit on the Mayhem run

The run's parent_revision was never set. mcode-action now passes
--parent-revision: an explicit parent-revision input, else the first parent
of the recorded revision, read from its commit object so it works in a
depth-1 checkout where `git rev-parse <rev>^` fails. Nothing is sent when
that commit is not checked out (a pull request head) or is a root commit.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Record the upstream commit as the Mayhem run's build id (#260)

* Record the upstream commit as the Mayhem run's build id

For a fork, mcode-action now passes --build-id: the newest commit the recorded
revision shares with the upstream default branch, from the GitHub compare API.
A build-id input overrides it; nothing is sent for a repository that is not a fork.

* Ensure note for behavior change is visible.

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: Thanassis Avgerinos <thanassis.avgerinos@bugcrowd.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.

2 participants