From ca43c24c283ca3d58041bac7121b003e8b3ff289 Mon Sep 17 00:00:00 2001
From: Siddharth Asthana <siddharthasthana31@gmail.com>
Date: Thu, 2 Oct 2025 19:43:07 +0530
Subject: [PATCH 1/3] Add committer avatar and profile link to last commit info

This enhancement displays the committer's avatar and links to
their GitLab profile in the last commit info box when the
committer differs from the author. Previously, only the
committer's name was shown as plain text.

The implementation adds three new fields to the CommitType
GraphQL API: committerAvatarUrl, committerWebUrl, and a
committer User object. The backend resolvers check if the
committer is a GitLab user and return their profile info,
falling back to Gravatar URLs for non-GitLab users.

The frontend now displays a clickable avatar using the
UserAvatarLink component and makes the committer name a link
to their profile when available. For non-GitLab users, the
avatar displays as a Gravatar image without a link.

This provides better UX and consistency with how author
information is displayed, making it easier to identify and
navigate to contributor profiles directly from the repository
file browser.

Changelog: added

Signed-off-by: Siddharth Asthana <siddharthasthana31@gmail.com>
---
 .../repository/components/commit_info.vue     |  30 +++-
 .../repository/path_last_commit.query.graphql |   9 ++
 app/graphql/types/repositories/commit_type.rb |  27 ++++
 doc/api/graphql/reference/_index.md           |   3 +
 .../repository/components/commit_info_spec.js | 129 ++++++++++++++++++
 .../types/repositories/commit_type_spec.rb    |   3 +-
 6 files changed, 197 insertions(+), 4 deletions(-)

diff --git a/app/assets/javascripts/repository/components/commit_info.vue b/app/assets/javascripts/repository/components/commit_info.vue
index f9fb971b2968fc..64578570224ab9 100644
--- a/app/assets/javascripts/repository/components/commit_info.vue
+++ b/app/assets/javascripts/repository/components/commit_info.vue
@@ -146,13 +146,37 @@ export default {
             </template>
 
             <template #committer>
+              <user-avatar-link
+                v-if="commit.committer"
+                :link-href="commit.committer.webPath"
+                :img-src="commit.committer.avatarUrl"
+                :img-alt="commit.committer.name"
+                :img-size="16"
+                class="gl-mx-1"
+              />
               <user-avatar-image
-                v-if="commit.committerAvatarUrl"
-                class="gl-mx-1 gl-my-2"
+                v-else-if="commit.committerAvatarUrl"
+                class="gl-mx-1"
                 :img-src="commit.committerAvatarUrl"
                 :size="16"
               />
-              {{ commit.committerName }}
+              <gl-link
+                v-if="commit.committer"
+                :href="commit.committer.webPath"
+                class="commit-committer-link"
+              >
+                {{ commit.committer.name }}
+              </gl-link>
+              <gl-link
+                v-else-if="commit.committerWebUrl"
+                :href="commit.committerWebUrl"
+                class="commit-committer-link"
+              >
+                {{ commit.committerName }}
+              </gl-link>
+              <template v-else>
+                {{ commit.committerName }}
+              </template>
             </template>
 
             <template #committedDate>
diff --git a/app/graphql/queries/repository/path_last_commit.query.graphql b/app/graphql/queries/repository/path_last_commit.query.graphql
index d90ce5c16693a4..d69b15f212508f 100644
--- a/app/graphql/queries/repository/path_last_commit.query.graphql
+++ b/app/graphql/queries/repository/path_last_commit.query.graphql
@@ -15,6 +15,8 @@ query pathLastCommit($projectPath: ID!, $path: String, $ref: String!, $refType:
         webPath
         committerName
         committerEmail
+        committerAvatarUrl
+        committerWebUrl
         committedDate
         authoredDate
         authorName
@@ -27,6 +29,13 @@ query pathLastCommit($projectPath: ID!, $path: String, $ref: String!, $refType:
           avatarUrl
           webPath
         }
+        committer {
+          __typename
+          id
+          name
+          avatarUrl
+          webPath
+        }
         signature {
           __typename
           ... on GpgSignature {
diff --git a/app/graphql/types/repositories/commit_type.rb b/app/graphql/types/repositories/commit_type.rb
index 1c6a6d22162cb2..1a0596fd99ac9f 100644
--- a/app/graphql/types/repositories/commit_type.rb
+++ b/app/graphql/types/repositories/commit_type.rb
@@ -65,10 +65,19 @@ class CommitType < BaseObject
       field :committer_name, type: GraphQL::Types::String, null: true,
         description: "Name of the committer."
 
+      field :committer_avatar_url, type: GraphQL::Types::String, null: true,
+        description: 'Avatar URL of the committer.'
+
+      field :committer_web_url, type: GraphQL::Types::String, null: true,
+        description: 'Web URL of the committer.'
+
       # models/commit lazy loads the author by email
       field :author, type: Types::UserType, null: true,
         description: 'Author of the commit.'
 
+      field :committer, type: Types::UserType, null: true,
+        description: 'Committer of the commit.'
+
       field :diffs, [Types::DiffType], null: true, calls_gitaly: true,
         description: 'Diffs contained within the commit. ' \
           'This field can only be resolved for 10 diffs in any single request.' do
@@ -93,6 +102,24 @@ def diffs
       def author_gravatar
         GravatarService.new.execute(object.author_email, 40)
       end
+
+      def committer_avatar_url
+        return unless object.committer_email
+
+        if object.committer
+          object.committer.avatar_url
+        else
+          GravatarService.new.execute(object.committer_email, 40)
+        end
+      end
+
+      def committer_web_url
+        return unless object.committer_email
+
+        return unless object.committer
+
+        Gitlab::Routing.url_helpers.user_url(object.committer)
+      end
     end
   end
 end
diff --git a/doc/api/graphql/reference/_index.md b/doc/api/graphql/reference/_index.md
index 4ede04ffbf3b24..cc851156eea454 100644
--- a/doc/api/graphql/reference/_index.md
+++ b/doc/api/graphql/reference/_index.md
@@ -26653,8 +26653,11 @@ Represents a summary of the compared codequality report.
 | <a id="commitauthorname"></a>`authorName` | [`String`](#string) | Commit authors name. |
 | <a id="commitauthoreddate"></a>`authoredDate` | [`Time`](#time) | Timestamp of when the commit was authored. |
 | <a id="commitcommitteddate"></a>`committedDate` | [`Time`](#time) | Timestamp of when the commit was committed. |
+| <a id="commitcommitter"></a>`committer` | [`UserCore`](#usercore) | Committer of the commit. |
+| <a id="commitcommitteravatarurl"></a>`committerAvatarUrl` | [`String`](#string) | Avatar URL of the committer. |
 | <a id="commitcommitteremail"></a>`committerEmail` | [`String`](#string) | Email of the committer. |
 | <a id="commitcommittername"></a>`committerName` | [`String`](#string) | Name of the committer. |
+| <a id="commitcommitterweburl"></a>`committerWebUrl` | [`String`](#string) | Web URL of the committer. |
 | <a id="commitdescription"></a>`description` | [`String`](#string) | Description of the commit message. |
 | <a id="commitdescriptionhtml"></a>`descriptionHtml` | [`String`](#string) | GitLab Flavored Markdown rendering of `description`. |
 | <a id="commitdiffs"></a>`diffs` | [`[Diff!]`](#diff) | Diffs contained within the commit. This field can only be resolved for 10 diffs in any single request. |
diff --git a/spec/frontend/repository/components/commit_info_spec.js b/spec/frontend/repository/components/commit_info_spec.js
index 1a451030f65ca5..0f5fb368353a7c 100644
--- a/spec/frontend/repository/components/commit_info_spec.js
+++ b/spec/frontend/repository/components/commit_info_spec.js
@@ -160,5 +160,134 @@ describe('Repository last commit component', () => {
         commitMockWithDifferentCommitter.committedDate,
       );
     });
+
+    describe('when committer is a GitLab user', () => {
+      const commitMockWithCommitterUser = {
+        author: { name: 'John Doe', email: 'john@example.com' },
+        committerName: 'Jane Smith',
+        committerEmail: 'jane@example.com',
+        committedDate: '2019-02-01',
+        committer: {
+          name: 'Jane Smith',
+          avatarUrl: 'https://gitlab.com/jane-avatar.jpg',
+          webPath: '/jane-smith',
+        },
+      };
+
+      beforeEach(() => createComponent({ commitMock: commitMockWithCommitterUser }));
+
+      it('displays clickable committer avatar using UserAvatarLink', () => {
+        const avatarLinks = wrapper.findAllComponents(UserAvatarLink);
+        const committerAvatar = avatarLinks.at(1); // First is author, second is committer
+
+        expect(committerAvatar.exists()).toBe(true);
+        expect(committerAvatar.props()).toMatchObject({
+          linkHref: '/jane-smith',
+          imgSrc: 'https://gitlab.com/jane-avatar.jpg',
+          imgAlt: 'Jane Smith',
+          imgSize: 16,
+        });
+      });
+
+      it('displays committer name as a clickable link', () => {
+        const committerLink = wrapper.find('.commit-committer-link');
+
+        expect(committerLink.exists()).toBe(true);
+        expect(committerLink.attributes('href')).toBe(
+          commitMockWithCommitterUser.committer.webPath,
+        );
+        expect(committerLink.text()).toBe('Jane Smith');
+      });
+    });
+
+    describe('when committer is not a GitLab user but has avatar URL', () => {
+      const commitMockWithNonUserCommitter = {
+        author: { name: 'John Doe', email: 'john@example.com' },
+        committerName: 'Jane Smith',
+        committerEmail: 'jane@example.com',
+        committerAvatarUrl: 'https://gravatar.com/avatar/123',
+        committerWebUrl: null,
+        committedDate: '2019-02-01',
+        committer: null,
+      };
+
+      beforeEach(() => createComponent({ commitMock: commitMockWithNonUserCommitter }));
+
+      it('displays committer avatar without link using UserAvatarImage', () => {
+        const avatarImages = findUserAvatarImages();
+        const committerAvatar = avatarImages.at(0);
+
+        expect(committerAvatar.exists()).toBe(true);
+        expect(committerAvatar.props()).toMatchObject({
+          size: 16,
+          imgSrc: 'https://gravatar.com/avatar/123',
+        });
+      });
+
+      it('displays committer name as plain text without link', () => {
+        const committerLink = wrapper.find('.commit-committer-link');
+        const text = findCommitterWrapper().text();
+
+        expect(committerLink.exists()).toBe(false);
+        expect(text).toContain('Jane Smith');
+      });
+    });
+
+    describe('when committer has web URL but no user object', () => {
+      const commitMockWithWebUrl = {
+        author: { name: 'John Doe', email: 'john@example.com' },
+        committerName: 'Jane Smith',
+        committerEmail: 'jane@example.com',
+        committerAvatarUrl: 'https://gravatar.com/avatar/123',
+        committerWebUrl: '/jane-smith',
+        committedDate: '2019-02-01',
+        committer: null,
+      };
+
+      beforeEach(() => createComponent({ commitMock: commitMockWithWebUrl }));
+
+      it('displays committer avatar without UserAvatarLink', () => {
+        const avatarImages = findUserAvatarImages();
+
+        expect(avatarImages.at(0).exists()).toBe(true);
+        expect(avatarImages.at(0).props('imgSrc')).toBe('https://gravatar.com/avatar/123');
+      });
+
+      it('displays committer name as a clickable link', () => {
+        const committerLink = wrapper.find('.commit-committer-link');
+
+        expect(committerLink.exists()).toBe(true);
+        expect(committerLink.attributes('href')).toBe('/jane-smith');
+        expect(committerLink.text()).toBe('Jane Smith');
+      });
+    });
+
+    describe('when committer has no avatar or profile URL', () => {
+      const commitMockMinimalCommitter = {
+        author: { name: 'John Doe', email: 'john@example.com' },
+        committerName: 'Jane Smith',
+        committerEmail: 'jane@example.com',
+        committerAvatarUrl: null,
+        committerWebUrl: null,
+        committedDate: '2019-02-01',
+        committer: null,
+      };
+
+      beforeEach(() => createComponent({ commitMock: commitMockMinimalCommitter }));
+
+      it('does not display committer avatar', () => {
+        const avatarImages = findUserAvatarImages();
+
+        expect(avatarImages).toHaveLength(0);
+      });
+
+      it('displays committer name as plain text', () => {
+        const committerLink = wrapper.find('.commit-committer-link');
+        const text = findCommitterWrapper().text();
+
+        expect(committerLink.exists()).toBe(false);
+        expect(text).toContain('Jane Smith');
+      });
+    });
   });
 });
diff --git a/spec/graphql/types/repositories/commit_type_spec.rb b/spec/graphql/types/repositories/commit_type_spec.rb
index bc6da8931d3c67..d96cf719b6b733 100644
--- a/spec/graphql/types/repositories/commit_type_spec.rb
+++ b/spec/graphql/types/repositories/commit_type_spec.rb
@@ -14,7 +14,8 @@
       :id, :sha, :short_id, :title, :full_title, :full_title_html, :description, :description_html, :message,
       :title_html, :authored_date,
       :author_name, :author_email, :author_gravatar, :author, :diffs, :web_url, :web_path,
-      :pipelines, :signature_html, :signature, :committer_name, :committer_email, :committed_date,
+      :pipelines, :signature_html, :signature, :committer_name, :committer_email, :committer_avatar_url,
+      :committer_web_url, :committer, :committed_date,
       :name
     )
   end
-- 
GitLab


From fbb27851e20469885a2faf117530e676ac9bdef7 Mon Sep 17 00:00:00 2001
From: Siddharth Asthana <siddharthasthana31@gmail.com>
Date: Thu, 2 Oct 2025 21:49:32 +0530
Subject: [PATCH 2/3] Fix CI failure: Add missing GraphQL fields to test mock

---
 spec/frontend/repository/components/last_commit_spec.js | 2 +-
 spec/frontend/repository/mock_data.js                   | 9 +++++++++
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/spec/frontend/repository/components/last_commit_spec.js b/spec/frontend/repository/components/last_commit_spec.js
index dc181d86b831da..a05d72bfcbd5d9 100644
--- a/spec/frontend/repository/components/last_commit_spec.js
+++ b/spec/frontend/repository/components/last_commit_spec.js
@@ -70,7 +70,7 @@ describe('Repository last commit component', () => {
 
     await waitForPromises();
 
-    const commit = { ...commitData.project?.repository.paginatedTree.nodes[0].lastCommit };
+    const commit = { ...commitData.data.project?.repository.lastCommit };
 
     expect(findCommitInfo().props().commit).toMatchObject(commit);
   });
diff --git a/spec/frontend/repository/mock_data.js b/spec/frontend/repository/mock_data.js
index 23c730d8c75789..de534f95543337 100644
--- a/spec/frontend/repository/mock_data.js
+++ b/spec/frontend/repository/mock_data.js
@@ -340,6 +340,8 @@ export const createCommitData = ({ pipelineEdges = defaultPipelineEdges, signatu
             webPath: '/commit/123',
             committerName: 'Test Committer',
             committerEmail: 'testcommitter@example.com',
+            committerAvatarUrl: 'https://test.com/committer-avatar',
+            committerWebUrl: 'https://test.com/committer',
             committedDate: '2019-02-02',
             authoredDate: '2019-01-01',
             authorName: 'Test',
@@ -352,6 +354,13 @@ export const createCommitData = ({ pipelineEdges = defaultPipelineEdges, signatu
               avatarUrl: 'https://test.com',
               webPath: '/test',
             },
+            committer: {
+              __typename: 'UserCore',
+              id: 'gid://gitlab/User/2',
+              name: 'Test Committer',
+              avatarUrl: 'https://test.com/committer-avatar',
+              webPath: '/committer',
+            },
             signature,
             pipelines: {
               __typename: 'PipelineConnection',
-- 
GitLab


From 0eb693da9356242b376ea17d2b126cc379f2573a Mon Sep 17 00:00:00 2001
From: Siddharth Asthana <siddharthasthana31@gmail.com>
Date: Fri, 3 Oct 2025 23:21:01 +0530
Subject: [PATCH 3/3] Simplify committer_avatar_url and committer_web_url logic

---
 app/graphql/types/repositories/commit_type.rb | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/app/graphql/types/repositories/commit_type.rb b/app/graphql/types/repositories/commit_type.rb
index 1a0596fd99ac9f..e822dcaac88fa7 100644
--- a/app/graphql/types/repositories/commit_type.rb
+++ b/app/graphql/types/repositories/commit_type.rb
@@ -104,18 +104,14 @@ def author_gravatar
       end
 
       def committer_avatar_url
-        return unless object.committer_email
-
         if object.committer
           object.committer.avatar_url
-        else
+        elsif object.committer_email
           GravatarService.new.execute(object.committer_email, 40)
         end
       end
 
       def committer_web_url
-        return unless object.committer_email
-
         return unless object.committer
 
         Gitlab::Routing.url_helpers.user_url(object.committer)
-- 
GitLab