Skip to content

fix(commit): preserve author metadata in parseConventionalCommits - #2874

Open
anishesg wants to merge 1 commit into
googleapis:mainfrom
proudhare:fix/ph-issue-2761
Open

fix(commit): preserve author metadata in parseConventionalCommits#2874
anishesg wants to merge 1 commit into
googleapis:mainfrom
proudhare:fix/ph-issue-2761

Conversation

@anishesg

@anishesg anishesg commented Aug 9, 2026

Copy link
Copy Markdown

The parseConventionalCommits() function in src/commit.ts was dropping the author field when transforming Commit objects into ConventionalCommit objects. This caused author metadata to be lost before changelog rendering, even though the include-commit-authors config option was enabled and the GitHub API provided author data.

The fix adds author: commit.author to the object literal at line 434 where other fields like sha, message, files, and pullRequest are already being preserved. This ensures that author information flows through the entire pipeline from commit fetch to changelog generation.

A regression test was added to verify that author metadata (name, email, username) is correctly preserved when parsing conventional commits.

Fixes #2761

The `parseConventionalCommits()` function in `src/commit.ts` was dropping the `author` field when transforming `Commit` objects into `ConventionalCommit` objects. This caused author metadata to be lost before changelog rendering, even though the `include-commit-authors` config option was enabled and the GitHub API provided author data.

Signed-off-by: anish <anishesg@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

include-commit-authors has no effect because author metadata is dropped in parseConventionalCommits()

1 participant