fix(changelog): find original author of agentic commits#39
Merged
Conversation
github-actions
Bot
added
t-infra
There was a problem hiding this comment.
Pull request overview
This PR updates author attribution for changelog generation by removing Copilot-specific commit handling and instead parsing generic Co-authored-by: trailers so that co-authors can be attributed alongside primary commit authors.
Changes:
- Replaces
findOriginalAuthorOfCopilotCommitwith a genericgetCommitCoauthorstrailer parser. - Attaches parsed coauthors to each commit author entry produced during changelog generation.
- Updates Slack ID resolution to consider both primary authors and commit coauthors.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/utils.js | Removes Copilot-special-casing and adds getCommitCoauthors; stores parsed coauthors alongside commit authors. |
| src/utils.test.js | Updates and expands unit tests to validate Co-authored-by: parsing behavior via getCommitCoauthors. |
| src/getAuthorsWithSlackIds.js | Refactors Slack ID resolution to work across primary authors plus coauthors and uses org member info mapping. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
MrkMrk00
pushed a commit
that referenced
this pull request
Jun 26, 2026
…utors Addresses the Copilot review on PR #39: - Co-authors overwrite (utils.js): instead of storing co-authors in a per-author field that a later commit could overwrite, every `Co-authored-by` trailer now becomes a first-class entry in the authors map (keyed by login). This drops co-authors no more, regardless of how many commits an author has, and works whether or not Slack IDs are fetched. getCommitCoauthors now returns { login, name, email }. - getAuthorsWithSlackIds no longer drops contributors. Restored to the documented contract (action.yml / PR description): return every author and attach `slackId` only when it can be resolved, rather than filtering out anyone without an org email or Slack match. Rebuilt dist. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HvWuyXuTp7HFAExnfjG91b
MrkMrk00
added a commit
that referenced
this pull request
Jun 28, 2026
fix(changelog): make co-authors first-class authors; keep all contributors Addresses the Copilot review on PR #39: - Co-authors overwrite (utils.js): instead of storing co-authors in a per-author field that a later commit could overwrite, every `Co-authored-by` trailer now becomes a first-class entry in the authors map (keyed by login). This drops co-authors no more, regardless of how many commits an author has, and works whether or not Slack IDs are fetched. getCommitCoauthors now returns { login, name, email }. - getAuthorsWithSlackIds no longer drops contributors. Restored to the documented contract (action.yml / PR description): return every author and attach `slackId` only when it can be resolved, rather than filtering out anyone without an org email or Slack match. Rebuilt dist. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HvWuyXuTp7HFAExnfjG91b feat(changelog): exclude bots and ignored logins from authors Adds bot/author filtering so release notifications don't tag automation accounts (Copilot, Claude) while still crediting the real humans behind their commits: - Commit authors with GitHub account type "Bot" are always skipped. - New `ignore-authors` input (comma-separated logins, default "Copilot,Claude", matched case-insensitively) excludes both commit authors and `Co-authored-by` trailers. A bot commit's real co-authors are still kept. Rebuilt dist. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HvWuyXuTp7HFAExnfjG91b fix(changelog): find original author of agentic commits
MrkMrk00
force-pushed
the
fix/find-author-in-agentic-commits
branch
from
ab9e81d to
cf5625a
Compare
fix(changelog): make co-authors first-class authors; keep all contributors Addresses the Copilot review on PR #39: - Co-authors overwrite (utils.js): instead of storing co-authors in a per-author field that a later commit could overwrite, every `Co-authored-by` trailer now becomes a first-class entry in the authors map (keyed by login). This drops co-authors no more, regardless of how many commits an author has, and works whether or not Slack IDs are fetched. getCommitCoauthors now returns { login, name, email }. - getAuthorsWithSlackIds no longer drops contributors. Restored to the documented contract (action.yml / PR description): return every author and attach `slackId` only when it can be resolved, rather than filtering out anyone without an org email or Slack match. Rebuilt dist. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HvWuyXuTp7HFAExnfjG91b feat(changelog): exclude bots and ignored logins from authors Adds bot/author filtering so release notifications don't tag automation accounts (Copilot, Claude) while still crediting the real humans behind their commits: - Commit authors with GitHub account type "Bot" are always skipped. - New `ignore-authors` input (comma-separated logins, default "Copilot,Claude", matched case-insensitively) excludes both commit authors and `Co-authored-by` trailers. A bot commit's real co-authors are still kept. Rebuilt dist. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HvWuyXuTp7HFAExnfjG91b fix(changelog): find original author of agentic commits
MrkMrk00
force-pushed
the
fix/find-author-in-agentic-commits
branch
from
cf5625a to
9fd6980
Compare
fnesveda
approved these changes
Jun 28, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR replaces the custom handling of Copilot's commits by a generic
Co-authored-bytrailer parsing. All of the commit authors and co-authors will be returned as changelog authors.Also adds an option to ignore certain GitHub user names from the changelog authors via the action's input:
ignore-authors. (By defaultCopilot,Claude)