clear-pr-merge-commit-message - Clear redundant co-authors

[agriyakhetarpal]

This PR is a follow-up to #9336 as discussed in #9336 (comment) with @fregante. The idea is that if a PR author creates a PR, the author's own "Co-authored-by" commit messages added to the commit description at squash-merge time are redundant and can be removed. This also applies to bots, such as Dependabot, Renovate, pre-commit-ci, github-actions, and so on. In this case, we can remove them, as they also use private/noreply email addresses.

I've slightly adjusted the logic for getConversationAuthor, because the previous version wasn't working for me. Suggestions are welcome, thanks!

Test URLs

I tested this in a PR on my profile repository at agriyakhetarpal/agriyakhetarpal#1.

Screenshot

Here is a video:

Screen.Recording.2026-05-04.at.17.54.26.mov

As it can be seen, Co-authored-by: 74401230+agriyakhetarpal@users.noreply.github.com was cleared and is absent. example (I added them as a co-author on commit five)'s co-author message is preserved. For users with accounts before July 18, 2017, their privacy emails are of a different kind and are preserved (commit seven). For the bot case, i.e., github-actions[bot], its co-author message is preserved as well. Lastly, commit sign-offs are treated as separate from co-authoring and are thus preserved, too.

Thank you :)

[fregante]

Looks good! I just have some nits

[agriyakhetarpal]

I hope this aligns better with what you had in mind – thanks for the suggestions!

[fregante]

🤘

[SunsetTechuila]

I don't understand why we need the getFirstComment function when we can reliably select the PR/issue body and pass it to getCommentAuthor:

export function getConversationAuthor(): string | undefined {
	const body = $optional(['.react-issue-body', '.js-command-palette-pull-body']);
	return body ? getCommentAuthor(body) : undefined;
}

[fregante]

I don't understand why we need the getFirstComment function

There are probably other features that can (and should) use this function. Since getCommentAuthor specifically expects one comment, the question would be why would you not pass one comment?

[SunsetTechuila]

Let's at least rename getFirstComment to getConversationBody

[agriyakhetarpal]

Agreed!

[SunsetTechuila]

This doesn't work with bots due to a bug in getCommentAuthor. It doesn't add [bot] to bot names because it looks for an anchor with a URL starting with https://github.com/apps/ when it should look for a URL starting with /apps/

[SunsetTechuila]

webpack/webpack#20899

refined-github/source/github-helpers/get-comment-author.ts

Lines 40 to 43 in 7b9a915

	if (!name.endsWith('[bot]') && $closestOptional('[href^="https://github.com/apps/"]', avatar)) {
	// Example: https://github.com/webpack/webpack/pull/15926#issuecomment-1170670173
	return name + '[bot]';
	}

[agriyakhetarpal]

Thanks, good catch! I couldn't have caught it with my own PR. For my curiosity: what is changing in October 2026?

[SunsetTechuila]

what is changing in October 2026?

Six months will pass. I don't know when those links became relative, so it's safest to assume it happened just now. For users on older GitHub Enterprise versions, they are still absolute. We support GHE versions that are no older than six months

[SunsetTechuila]

After writing this, I realized that links to public GitHub apps should probably always be absolute on GHE

[agriyakhetarpal]

Thanks to you both for making my recent contributions a pleasurable experience! I appreciate it a lot!