proto: Replace all "This comment ..." lines

We want to add some linting rules to our proto files, but it seems like
there are a bunch of comments which use the filler line "// This comment
is left unintentionally blank.".

Let's use a python script to replace them with the field, message, rpc,
service or enum name.

Script:

import re
import sys

import os
from glob import glob
result = [y for x in os.walk(".") for y in glob(os.path.join(x[0], '*.proto'))]

for filename in result:
    print("doing", filename)

    file = open(filename, "r")

    lines = file.readlines()

    for i, line in enumerate(lines):
        if "// This comment is left unintentionally blank." in line:
            next_line = lines[i+1]
            sub = ""

            if re.search("^\s*rpc", next_line):
                x = re.findall(r"rpc (\w*)\s?\(", next_line)
                if len(x) != 1:
                    print("error finding rpc name:", next_line, "match:", x)
                    sys.exit()
                sub = x[0]
            elif re.search("^\s*message", next_line):
                x = re.findall(r"message (\w*)\s?\{", next_line)
                if len(x) != 1:
                    print("error finding message name:", next_line, "match:", x)
                    sys.exit()
                sub = x[0]
            elif re.search("^\s*service", next_line):
                x = re.findall(r"service (\w*)\s?\{", next_line)
                if len(x) != 1:
                    print("error finding service name:", next_line, "match:", x)
                    sys.exit()
                sub = x[0]
            elif re.search("^\s*enum", next_line):
                x = re.findall(r"enum (\w*)\s?\{", next_line)
                if len(x) != 1:
                    print("error enum service name:", next_line, "match:", x)
                    sys.exit()
                sub = x[0]
            else:
                x = re.findall(r"([\w_]*) = \d+", next_line)
                if len(x) != 1:
                    print("error finding field name:", next_line, "match:", x)
                    sys.exit()
                sub = x[0]

            lines[i] = line.replace("This comment is left unintentionally blank.", sub + " ...")

    file.close()

    file = open(filename, "w")
    file.writelines(lines)
    file.close()

17 files changed, 697 insertions, 697 deletions

diff --git a/proto/cleanup.proto b/proto/cleanup.proto
index dd172b6bb..71da84561 100644
--- a/proto/cleanup.proto
+++ b/proto/cleanup.proto
@@ -10,7 +10,7 @@ option go_package = "gitlab.com/gitlab-org/gitaly/v16/proto/go/gitalypb";
 // CleanupService provides RPCs to clean up a repository's contents.
 service CleanupService {
 
-  // This comment is left unintentionally blank.
+  // ApplyBfgObjectMapStream ...
   rpc ApplyBfgObjectMapStream(stream ApplyBfgObjectMapStreamRequest) returns (stream ApplyBfgObjectMapStreamResponse) {
     option (op_type) = {
       op: MUTATOR
@@ -19,7 +19,7 @@ service CleanupService {
 
 }
 
-// This comment is left unintentionally blank.
+// ApplyBfgObjectMapStreamRequest ...
 message ApplyBfgObjectMapStreamRequest {
   // Only available on the first message
   Repository repository = 1 [(target_repository)=true];
@@ -30,19 +30,19 @@ message ApplyBfgObjectMapStreamRequest {
   bytes object_map = 2;
 }
 
-// This comment is left unintentionally blank.
+// ApplyBfgObjectMapStreamResponse ...
 message ApplyBfgObjectMapStreamResponse {
   // We send back each parsed entry in the request's object map so the client
   // can take action
   message Entry {
-    // This comment is left unintentionally blank.
+    // type ...
     ObjectType type = 1;
-    // This comment is left unintentionally blank.
+    // old_oid ...
     string old_oid = 2;
-    // This comment is left unintentionally blank.
+    // new_oid ...
     string new_oid = 3;
   }
 
-  // This comment is left unintentionally blank.
+  // entries ...
   repeated Entry entries = 1;
 }
diff --git a/proto/commit.proto b/proto/commit.proto
index 52341c539..b203c39f8 100644
--- a/proto/commit.proto
+++ b/proto/commit.proto
@@ -68,21 +68,21 @@ service CommitService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // ListFiles ...
   rpc ListFiles(ListFilesRequest) returns (stream ListFilesResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // FindCommit ...
   rpc FindCommit(FindCommitRequest) returns (FindCommitResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // CommitStats ...
   rpc CommitStats(CommitStatsRequest) returns (CommitStatsResponse) {
     option (op_type) = {
       op: ACCESSOR
@@ -96,7 +96,7 @@ service CommitService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // FindCommits ...
   rpc FindCommits(FindCommitsRequest) returns (stream FindCommitsResponse) {
     option (op_type) = {
       op: ACCESSOR
@@ -131,49 +131,49 @@ service CommitService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // ListLastCommitsForTree ...
   rpc ListLastCommitsForTree(ListLastCommitsForTreeRequest) returns (stream ListLastCommitsForTreeResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // CommitsByMessage ...
   rpc CommitsByMessage(CommitsByMessageRequest) returns (stream CommitsByMessageResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // ListCommitsByOid ...
   rpc ListCommitsByOid(ListCommitsByOidRequest) returns (stream ListCommitsByOidResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // ListCommitsByRefName ...
   rpc ListCommitsByRefName(ListCommitsByRefNameRequest) returns (stream ListCommitsByRefNameResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // FilterShasWithSignatures ...
   rpc FilterShasWithSignatures(stream FilterShasWithSignaturesRequest) returns (stream FilterShasWithSignaturesResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // GetCommitSignatures ...
   rpc GetCommitSignatures(GetCommitSignaturesRequest) returns (stream GetCommitSignaturesResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // GetCommitMessages ...
   rpc GetCommitMessages(GetCommitMessagesRequest) returns (stream GetCommitMessagesResponse) {
     option (op_type) = {
       op: ACCESSOR
@@ -284,21 +284,21 @@ message ListAllCommitsResponse {
   repeated GitCommit commits = 1;
 }
 
-// This comment is left unintentionally blank.
+// CommitStatsRequest ...
 message CommitStatsRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // revision ...
   bytes revision = 2;
 }
 
-// This comment is left unintentionally blank.
+// CommitStatsResponse ...
 message CommitStatsResponse {
   // OID is the commit. Empty means not found
   string oid = 1;
-  // This comment is left unintentionally blank.
+  // additions ...
   int32 additions = 2;
-  // This comment is left unintentionally blank.
+  // deletions ...
   int32 deletions = 3;
 }
 
@@ -476,11 +476,11 @@ message GetTreeEntriesRequest {
   bool skip_flat_paths = 7;
 }
 
-// This comment is left unintentionally blank.
+// GetTreeEntriesResponse ...
 message GetTreeEntriesResponse {
-  // This comment is left unintentionally blank.
+  // entries ...
   repeated TreeEntry entries = 1;
-  // This comment is left unintentionally blank.
+  // pagination_cursor ...
   PaginationCursor pagination_cursor = 2;
 }
 
@@ -495,11 +495,11 @@ message GetTreeEntriesError {
   }
 }
 
-// This comment is left unintentionally blank.
+// ListFilesRequest ...
 message ListFilesRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // revision ...
   bytes revision = 2;
 }
 
@@ -509,131 +509,131 @@ message ListFilesResponse {
   repeated bytes paths = 1;
 }
 
-// This comment is left unintentionally blank.
+// FindCommitRequest ...
 message FindCommitRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // revision ...
   bytes revision = 2;
-  // This comment is left unintentionally blank.
+  // trailers ...
   bool trailers = 3;
 }
 
-// This comment is left unintentionally blank.
+// FindCommitResponse ...
 message FindCommitResponse {
   // commit is nil when the commit was not found
   GitCommit commit = 1;
 }
 
-// This comment is left unintentionally blank.
+// ListCommitsByOidRequest ...
 message ListCommitsByOidRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // oid ...
   repeated string oid = 2; // protolint:disable:this REPEATED_FIELD_NAMES_PLURALIZED
 }
 
-// This comment is left unintentionally blank.
+// ListCommitsByOidResponse ...
 message ListCommitsByOidResponse {
-  // This comment is left unintentionally blank.
+  // commits ...
   repeated GitCommit commits = 1;
 }
 
-// This comment is left unintentionally blank.
+// ListCommitsByRefNameRequest ...
 message ListCommitsByRefNameRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // ref_names ...
   repeated bytes ref_names = 2;
 }
 
-// This comment is left unintentionally blank.
+// ListCommitsByRefNameResponse ...
 message ListCommitsByRefNameResponse {
   reserved 1;
 
-  // This comment is left unintentionally blank.
+  // CommitForRef ...
   message CommitForRef {
-    // This comment is left unintentionally blank.
+    // commit ...
     GitCommit commit = 1;
-    // This comment is left unintentionally blank.
+    // ref_name ...
     bytes ref_name = 2;
   }
 
-  // This comment is left unintentionally blank.
+  // commit_refs ...
   repeated CommitForRef commit_refs = 2;
 }
 
-// This comment is left unintentionally blank.
+// FindAllCommitsRequest ...
 message FindAllCommitsRequest {
-  // This comment is left unintentionally blank.
+  // Order ...
   enum Order {
-    // This comment is left unintentionally blank.
+    // NONE ...
     NONE = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
-    // This comment is left unintentionally blank.
+    // TOPO ...
     TOPO = 1; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-    // This comment is left unintentionally blank.
+    // DATE ...
     DATE = 2; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
   }
 
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
   // When nil, return all commits reachable by any branch in the repo
   bytes revision = 2;
-  // This comment is left unintentionally blank.
+  // max_count ...
   int32 max_count = 3;
-  // This comment is left unintentionally blank.
+  // skip ...
   int32 skip = 4;
-  // This comment is left unintentionally blank.
+  // order ...
   Order order = 5;
 }
 
 // A single 'page' of the result set
 message FindAllCommitsResponse {
-  // This comment is left unintentionally blank.
+  // commits ...
   repeated GitCommit commits = 1;
 }
 
-// This comment is left unintentionally blank.
+// FindCommitsRequest ...
 message FindCommitsRequest {
-  // This comment is left unintentionally blank.
+  // Order ...
   enum Order {
-    // This comment is left unintentionally blank.
+    // NONE ...
     NONE = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
-    // This comment is left unintentionally blank.
+    // TOPO ...
     TOPO = 1; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
   }
 
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // revision ...
   bytes revision = 2;
-  // This comment is left unintentionally blank.
+  // limit ...
   int32 limit = 3;
-  // This comment is left unintentionally blank.
+  // offset ...
   int32 offset = 4;
-  // This comment is left unintentionally blank.
+  // paths ...
   repeated bytes paths = 5;
-  // This comment is left unintentionally blank.
+  // follow ...
   bool follow = 6;
-  // This comment is left unintentionally blank.
+  // skip_merges ...
   bool skip_merges = 7;
-  // This comment is left unintentionally blank.
+  // disable_walk ...
   bool disable_walk = 8;
-  // This comment is left unintentionally blank.
+  // after ...
   google.protobuf.Timestamp after = 9;
-  // This comment is left unintentionally blank.
+  // before ...
   google.protobuf.Timestamp before = 10;
   // all and revision are mutually exclusive
   bool all = 11;
-  // This comment is left unintentionally blank.
+  // first_parent ...
   bool first_parent = 12;
-  // This comment is left unintentionally blank.
+  // author ...
   bytes author = 13;
-  // This comment is left unintentionally blank.
+  // order ...
   Order order = 14;
-  // This comment is left unintentionally blank.
+  // global_options ...
   GlobalOptions global_options = 15;
-  // This comment is left unintentionally blank.
+  // trailers ...
   bool trailers = 16;
   // include_shortstat determines whether to include the number of lines and files
   // changed in the commits. Populates the `short_stats` field.
@@ -649,7 +649,7 @@ message FindCommitsRequest {
 
 // A single 'page' of the result set
 message FindCommitsResponse {
-  // This comment is left unintentionally blank.
+  // commits ...
   repeated GitCommit commits = 1;
 }
 
@@ -748,105 +748,105 @@ message LastCommitForPathResponse {
   GitCommit commit = 1;
 }
 
-// This comment is left unintentionally blank.
+// ListLastCommitsForTreeRequest ...
 message ListLastCommitsForTreeRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // revision ...
   string revision = 2;
-  // This comment is left unintentionally blank.
+  // path ...
   bytes path = 3;
-  // This comment is left unintentionally blank.
+  // limit ...
   int32 limit = 4;
-  // This comment is left unintentionally blank.
+  // offset ...
   int32 offset = 5;
-  // This comment is left unintentionally blank.
+  // literal_pathspec ...
   bool literal_pathspec = 6 [deprecated = true];
-  // This comment is left unintentionally blank.
+  // global_options ...
   GlobalOptions global_options = 7;
 }
 
-// This comment is left unintentionally blank.
+// ListLastCommitsForTreeResponse ...
 message ListLastCommitsForTreeResponse {
-  // This comment is left unintentionally blank.
+  // CommitForTree ...
   message CommitForTree {
     reserved 1;
 
-    // This comment is left unintentionally blank.
+    // commit ...
     GitCommit commit = 2;
     reserved 3;
-    // This comment is left unintentionally blank.
+    // path_bytes ...
     bytes path_bytes = 4;
   }
 
-  // This comment is left unintentionally blank.
+  // commits ...
   repeated CommitForTree commits = 1;
 }
 
-// This comment is left unintentionally blank.
+// CommitsByMessageRequest ...
 message CommitsByMessageRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // revision ...
   bytes revision = 2;
-  // This comment is left unintentionally blank.
+  // offset ...
   int32 offset = 3;
-  // This comment is left unintentionally blank.
+  // limit ...
   int32 limit = 4;
-  // This comment is left unintentionally blank.
+  // path ...
   bytes path = 5;
-  // This comment is left unintentionally blank.
+  // query ...
   string query = 6;
-  // This comment is left unintentionally blank.
+  // global_options ...
   GlobalOptions global_options = 7;
 }
 
 // One 'page' of the paginated response of CommitsByMessage
 message CommitsByMessageResponse {
-  // This comment is left unintentionally blank.
+  // commits ...
   repeated GitCommit commits = 1;
 }
 
-// This comment is left unintentionally blank.
+// FilterShasWithSignaturesRequest ...
 message FilterShasWithSignaturesRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // shas ...
   repeated bytes shas = 2;
 }
 
-// This comment is left unintentionally blank.
+// FilterShasWithSignaturesResponse ...
 message FilterShasWithSignaturesResponse {
-  // This comment is left unintentionally blank.
+  // shas ...
   repeated bytes shas = 1;
 }
 
-// This comment is left unintentionally blank.
+// ExtractCommitSignatureRequest ...
 message ExtractCommitSignatureRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // commit_id ...
   string commit_id = 2;
 }
 
 // Either of the 'signature' and 'signed_text' fields may be present. It
 // is up to the caller to stitch them together.
 message ExtractCommitSignatureResponse {
-  // This comment is left unintentionally blank.
+  // signature ...
   bytes signature = 1;
-  // This comment is left unintentionally blank.
+  // signed_text ...
   bytes signed_text = 2;
 }
 
-// This comment is left unintentionally blank.
+// GetCommitSignaturesRequest ...
 message GetCommitSignaturesRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // commit_ids ...
   repeated string commit_ids = 2;
 }
 
-// This comment is left unintentionally blank.
+// GetCommitSignaturesResponse ...
 message GetCommitSignaturesResponse {
   // Signer of the commit. A commit can be signed either by a user or by Gitaly itself.
   enum Signer {
@@ -868,19 +868,19 @@ message GetCommitSignaturesResponse {
   Signer signer = 4;
 }
 
-// This comment is left unintentionally blank.
+// GetCommitMessagesRequest ...
 message GetCommitMessagesRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // commit_ids ...
   repeated string commit_ids = 2;
 }
 
-// This comment is left unintentionally blank.
+// GetCommitMessagesResponse ...
 message GetCommitMessagesResponse {
   // Only present for a new commit message
   string commit_id = 1;
-  // This comment is left unintentionally blank.
+  // message ...
   bytes message = 2;
 }
 
@@ -896,16 +896,16 @@ message CheckObjectsExistRequest {
   repeated bytes revisions = 2;
 }
 
-// This comment is left unintentionally blank.
+// CheckObjectsExistResponse ...
 message CheckObjectsExistResponse {
-  // This comment is left unintentionally blank.
+  // RevisionExistence ...
   message RevisionExistence {
-    // This comment is left unintentionally blank.
+    // name ...
     bytes name = 1;
-    // This comment is left unintentionally blank.
+    // exists ...
     bool exists = 2;
   };
 
-  // This comment is left unintentionally blank.
+  // revisions ...
   repeated RevisionExistence revisions = 1;
 }
diff --git a/proto/conflicts.proto b/proto/conflicts.proto
index ba3515d48..f7a4942cc 100644
--- a/proto/conflicts.proto
+++ b/proto/conflicts.proto
@@ -49,34 +49,34 @@ message ListConflictFilesRequest {
   bool skip_content = 5;
 }
 
-// This comment is left unintentionally blank.
+// ConflictFileHeader ...
 message ConflictFileHeader {
   reserved 1;
-  // This comment is left unintentionally blank.
+  // commit_oid ...
   string commit_oid = 2;
-  // This comment is left unintentionally blank.
+  // their_path ...
   bytes their_path = 3;
-  // This comment is left unintentionally blank.
+  // our_path ...
   bytes our_path = 4;
-  // This comment is left unintentionally blank.
+  // our_mode ...
   int32 our_mode = 5;
-  // This comment is left unintentionally blank.
+  // ancestor_path ...
   bytes ancestor_path = 6;
 }
 
-// This comment is left unintentionally blank.
+// ConflictFile ...
 message ConflictFile {
   oneof conflict_file_payload {
-    // This comment is left unintentionally blank.
+    // header ...
     ConflictFileHeader header = 1;
-    // This comment is left unintentionally blank.
+    // content ...
     bytes content = 2;
   }
 }
 
-// This comment is left unintentionally blank.
+// ListConflictFilesResponse ...
 message ListConflictFilesResponse {
-  // This comment is left unintentionally blank.
+  // files ...
   repeated ConflictFile files = 1;
 }
 
diff --git a/proto/diff.proto b/proto/diff.proto
index 585d6f4ba..8ef5b133d 100644
--- a/proto/diff.proto
+++ b/proto/diff.proto
@@ -25,21 +25,21 @@ service DiffService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // RawDiff ...
   rpc RawDiff(RawDiffRequest) returns (stream RawDiffResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // RawPatch ...
   rpc RawPatch(RawPatchRequest) returns (stream RawPatchResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // DiffStats ...
   rpc DiffStats(DiffStatsRequest) returns (stream DiffStatsResponse) {
     option (op_type) = {
       op: ACCESSOR
@@ -64,9 +64,9 @@ service DiffService {
   }
 }
 
-// This comment is left unintentionally blank.
+// CommitDiffRequest ...
 message CommitDiffRequest {
-  // This comment is left unintentionally blank.
+  // DiffMode ...
   enum DiffMode {
     // DEFAULT is the standard diff mode and results in a linewise diff for textfiles.
     DEFAULT = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
@@ -87,27 +87,27 @@ message CommitDiffRequest {
     WHITESPACE_CHANGES_IGNORE_ALL = 2;
   }
 
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // left_commit_id ...
   string left_commit_id = 2;
-  // This comment is left unintentionally blank.
+  // right_commit_id ...
   string right_commit_id = 3;
   // This field is deprecated, use the `whitespace_changes` field instead.
   reserved "ignore_whitespace_change";
   reserved 4;
-  // This comment is left unintentionally blank.
+  // paths ...
   repeated bytes paths = 5;
-  // This comment is left unintentionally blank.
+  // collapse_diffs ...
   bool collapse_diffs = 6;
-  // This comment is left unintentionally blank.
+  // enforce_limits ...
   bool enforce_limits = 7;
 
   // These limits are only enforced when enforce_limits == true.
   int32 max_files = 8;
-  // This comment is left unintentionally blank.
+  // max_lines ...
   int32 max_lines = 9;
-  // This comment is left unintentionally blank.
+  // max_bytes ...
   int32 max_bytes = 10;
   // Limitation of a single diff patch,
   // patches surpassing this limit are pruned by default.
@@ -116,9 +116,9 @@ message CommitDiffRequest {
 
   // These limits are only enforced if collapse_diffs == true.
   int32 safe_max_files = 11;
-  // This comment is left unintentionally blank.
+  // safe_max_lines ...
   int32 safe_max_lines = 12;
-  // This comment is left unintentionally blank.
+  // safe_max_bytes ...
   int32 safe_max_bytes = 13;
 
   // DiffMode is the mode used for generating the diff. Please refer to the enum declaration for supported modes.
@@ -144,23 +144,23 @@ message CommitDiffRequest {
 message CommitDiffResponse {
   reserved 8;
 
-  // This comment is left unintentionally blank.
+  // from_path ...
   bytes from_path = 1;
-  // This comment is left unintentionally blank.
+  // to_path ...
   bytes to_path = 2;
   // Blob ID as returned via `git diff --full-index`
   string from_id = 3;
-  // This comment is left unintentionally blank.
+  // to_id ...
   string to_id = 4;
-  // This comment is left unintentionally blank.
+  // old_mode ...
   int32 old_mode = 5;
-  // This comment is left unintentionally blank.
+  // new_mode ...
   int32 new_mode = 6;
-  // This comment is left unintentionally blank.
+  // binary ...
   bool binary = 7;
-  // This comment is left unintentionally blank.
+  // raw_patch_data ...
   bytes raw_patch_data = 9;
-  // This comment is left unintentionally blank.
+  // end_of_patch ...
   bool end_of_patch = 10;
   // OverflowMarker Indicates if the diff file exceeded limitations, in which case
   // there will be no patch data sent, only information about the patch.
@@ -173,97 +173,97 @@ message CommitDiffResponse {
   bool too_large = 13;
 }
 
-// This comment is left unintentionally blank.
+// CommitDeltaRequest ...
 message CommitDeltaRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // left_commit_id ...
   string left_commit_id = 2;
-  // This comment is left unintentionally blank.
+  // right_commit_id ...
   string right_commit_id = 3;
-  // This comment is left unintentionally blank.
+  // paths ...
   repeated bytes paths = 4;
 }
 
-// This comment is left unintentionally blank.
+// CommitDelta ...
 message CommitDelta {
-  // This comment is left unintentionally blank.
+  // from_path ...
   bytes from_path = 1;
-  // This comment is left unintentionally blank.
+  // to_path ...
   bytes to_path = 2;
   // Blob ID as returned via `git diff --full-index`
   string from_id = 3;
-  // This comment is left unintentionally blank.
+  // to_id ...
   string to_id = 4;
-  // This comment is left unintentionally blank.
+  // old_mode ...
   int32 old_mode = 5;
-  // This comment is left unintentionally blank.
+  // new_mode ...
   int32 new_mode = 6;
 }
 
-// This comment is left unintentionally blank.
+// CommitDeltaResponse ...
 message CommitDeltaResponse {
-  // This comment is left unintentionally blank.
+  // deltas ...
   repeated CommitDelta deltas = 1;
 }
 
-// This comment is left unintentionally blank.
+// RawDiffRequest ...
 message RawDiffRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // left_commit_id ...
   string left_commit_id = 2;
-  // This comment is left unintentionally blank.
+  // right_commit_id ...
   string right_commit_id = 3;
 }
 
-// This comment is left unintentionally blank.
+// RawDiffResponse ...
 message RawDiffResponse {
-  // This comment is left unintentionally blank.
+  // data ...
   bytes data = 1;
 }
 
-// This comment is left unintentionally blank.
+// RawPatchRequest ...
 message RawPatchRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // left_commit_id ...
   string left_commit_id = 2;
-  // This comment is left unintentionally blank.
+  // right_commit_id ...
   string right_commit_id = 3;
 }
 
-// This comment is left unintentionally blank.
+// RawPatchResponse ...
 message RawPatchResponse {
-  // This comment is left unintentionally blank.
+  // data ...
   bytes data = 1;
 }
 
-// This comment is left unintentionally blank.
+// DiffStatsRequest ...
 message DiffStatsRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // left_commit_id ...
   string left_commit_id = 2;
-  // This comment is left unintentionally blank.
+  // right_commit_id ...
   string right_commit_id = 3;
 }
 
-// This comment is left unintentionally blank.
+// DiffStats ...
 message DiffStats {
-  // This comment is left unintentionally blank.
+  // path ...
   bytes path = 1;
-  // This comment is left unintentionally blank.
+  // additions ...
   int32 additions = 2;
-  // This comment is left unintentionally blank.
+  // deletions ...
   int32 deletions = 3;
-  // This comment is left unintentionally blank.
+  // old_path ...
   bytes old_path = 4;
 }
 
-// This comment is left unintentionally blank.
+// DiffStatsResponse ...
 message DiffStatsResponse {
-  // This comment is left unintentionally blank.
+  // stats ...
   repeated DiffStats stats = 1;
 }
 
@@ -317,7 +317,7 @@ message FindChangedPathsRequest {
     }
   }
 
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
   // commits is the list of commits to compare to their parents. This field is deprecated. To adapt to the new calling
   // convention you can create one `CommitRequest` per commit, where each `CommitRequest` has only the `commit_revision`
@@ -340,23 +340,23 @@ message FindChangedPathsResponse {
 
 // Includes the path of the file, and the status of the change
 message ChangedPaths {
-  // This comment is left unintentionally blank.
+  // Status ...
   enum Status {
-    // This comment is left unintentionally blank.
+    // ADDED ...
     ADDED = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
-    // This comment is left unintentionally blank.
+    // MODIFIED ...
     MODIFIED = 1; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-    // This comment is left unintentionally blank.
+    // DELETED ...
     DELETED = 2; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-    // This comment is left unintentionally blank.
+    // TYPE_CHANGE ...
     TYPE_CHANGE = 3; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-    // This comment is left unintentionally blank.
+    // COPIED ...
     COPIED = 4; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
   }
 
-  // This comment is left unintentionally blank.
+  // path ...
   bytes path = 1;
-  // This comment is left unintentionally blank.
+  // status ...
   Status status = 2;
   // old_mode is the mode of the changed path previous to the change. May be one of the following values:
   //
diff --git a/proto/hook.proto b/proto/hook.proto
index 2b02f4a1c..ae8009b70 100644
--- a/proto/hook.proto
+++ b/proto/hook.proto
@@ -12,28 +12,28 @@ option go_package = "gitlab.com/gitlab-org/gitaly/v16/proto/go/gitalypb";
 // ensure that the actual hook logic is executed in the context of the server.
 service HookService {
 
-  // This comment is left unintentionally blank.
+  // PreReceiveHook ...
   rpc PreReceiveHook(stream PreReceiveHookRequest) returns (stream PreReceiveHookResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // PostReceiveHook ...
   rpc PostReceiveHook(stream PostReceiveHookRequest) returns  (stream PostReceiveHookResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // UpdateHook ...
   rpc UpdateHook(UpdateHookRequest) returns (stream UpdateHookResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // ReferenceTransactionHook ...
   rpc ReferenceTransactionHook(stream ReferenceTransactionHookRequest) returns (stream ReferenceTransactionHookResponse) {
     option (op_type) = {
       op: ACCESSOR
@@ -50,109 +50,109 @@ service HookService {
 
 }
 
-// This comment is left unintentionally blank.
+// PreReceiveHookRequest ...
 message PreReceiveHookRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // environment_variables ...
   repeated string environment_variables = 2;
-  // This comment is left unintentionally blank.
+  // stdin ...
   bytes stdin = 4;
-  // This comment is left unintentionally blank.
+  // git_push_options ...
   repeated string git_push_options = 5;
 }
 
-// This comment is left unintentionally blank.
+// PreReceiveHookResponse ...
 message PreReceiveHookResponse{
-  // This comment is left unintentionally blank.
+  // stdout ...
   bytes stdout = 1;
-  // This comment is left unintentionally blank.
+  // stderr ...
   bytes stderr = 2;
-  // This comment is left unintentionally blank.
+  // exit_status ...
   ExitStatus exit_status = 3;
 }
 
-// This comment is left unintentionally blank.
+// PostReceiveHookRequest ...
 message PostReceiveHookRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // environment_variables ...
   repeated string environment_variables = 2;
-  // This comment is left unintentionally blank.
+  // stdin ...
   bytes stdin = 3;
-  // This comment is left unintentionally blank.
+  // git_push_options ...
   repeated string git_push_options = 4;
 }
 
-// This comment is left unintentionally blank.
+// PostReceiveHookResponse ...
 message PostReceiveHookResponse{
-  // This comment is left unintentionally blank.
+  // stdout ...
   bytes stdout = 1;
-  // This comment is left unintentionally blank.
+  // stderr ...
   bytes stderr = 2;
-  // This comment is left unintentionally blank.
+  // exit_status ...
   ExitStatus exit_status = 3;
 }
 
-// This comment is left unintentionally blank.
+// UpdateHookRequest ...
 message UpdateHookRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // environment_variables ...
   repeated string environment_variables = 2;
-  // This comment is left unintentionally blank.
+  // ref ...
   bytes ref = 3;
-  // This comment is left unintentionally blank.
+  // old_value ...
   string old_value = 4;
-  // This comment is left unintentionally blank.
+  // new_value ...
   string new_value = 5;
 }
 
-// This comment is left unintentionally blank.
+// UpdateHookResponse ...
 message UpdateHookResponse{
-  // This comment is left unintentionally blank.
+  // stdout ...
   bytes stdout = 1;
-  // This comment is left unintentionally blank.
+  // stderr ...
   bytes stderr = 2;
-  // This comment is left unintentionally blank.
+  // exit_status ...
   ExitStatus exit_status = 3;
 }
 
-// This comment is left unintentionally blank.
+// ReferenceTransactionHookRequest ...
 message ReferenceTransactionHookRequest {
-  // This comment is left unintentionally blank.
+  // State ...
   enum State {
-    // This comment is left unintentionally blank.
+    // PREPARED ...
     PREPARED = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
-    // This comment is left unintentionally blank.
+    // COMMITTED ...
     COMMITTED = 1; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-    // This comment is left unintentionally blank.
+    // ABORTED ...
     ABORTED = 2; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
   }
 
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // environment_variables ...
   repeated string environment_variables = 2;
-  // This comment is left unintentionally blank.
+  // stdin ...
   bytes stdin = 3;
-  // This comment is left unintentionally blank.
+  // state ...
   State state = 4;
 }
 
-// This comment is left unintentionally blank.
+// ReferenceTransactionHookResponse ...
 message ReferenceTransactionHookResponse {
-  // This comment is left unintentionally blank.
+  // stdout ...
   bytes stdout = 1;
-  // This comment is left unintentionally blank.
+  // stderr ...
   bytes stderr = 2;
-  // This comment is left unintentionally blank.
+  // exit_status ...
   ExitStatus exit_status = 3;
 }
 
-// This comment is left unintentionally blank.
+// PackObjectsHookWithSidechannelRequest ...
 message PackObjectsHookWithSidechannelRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
   // args contains the arguments passed to the pack-objects hook, without the leading "git"
   repeated string args = 2;
@@ -167,6 +167,6 @@ message PackObjectsHookWithSidechannelRequest {
   string remote_ip = 7;
 }
 
-// This comment is left unintentionally blank.
+// PackObjectsHookWithSidechannelResponse ...
 message PackObjectsHookWithSidechannelResponse {
 }
diff --git a/proto/internal.proto b/proto/internal.proto
index 44f18d609..47d826777 100644
--- a/proto/internal.proto
+++ b/proto/internal.proto
@@ -20,15 +20,15 @@ service InternalGitaly {
   }
 }
 
-// This comment is left unintentionally blank.
+// WalkReposRequest ...
 message WalkReposRequest {
-  // This comment is left unintentionally blank.
+  // storage_name ...
   string storage_name = 1 [(storage)=true];
 }
 
-// This comment is left unintentionally blank.
+// WalkReposResponse ...
 message WalkReposResponse {
-  // This comment is left unintentionally blank.
+  // relative_path ...
   string relative_path = 1;
   // modification_time is the modification time of the repository directory.
   // This can be used as a proxy for when the repository was last
diff --git a/proto/lint.proto b/proto/lint.proto
index 62e4c5f88..264748209 100644
--- a/proto/lint.proto
+++ b/proto/lint.proto
@@ -6,32 +6,32 @@ import "google/protobuf/descriptor.proto";
 
 option go_package = "gitlab.com/gitlab-org/gitaly/v16/proto/go/gitalypb";
 
-// This comment is left unintentionally blank.
+// OperationMsg ...
 message OperationMsg {
-  // This comment is left unintentionally blank.
+  // Operation ...
   enum Operation {
-    // This comment is left unintentionally blank.
+    // UNKNOWN ...
     UNKNOWN = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
-    // This comment is left unintentionally blank.
+    // MUTATOR ...
     MUTATOR = 1; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-    // This comment is left unintentionally blank.
+    // ACCESSOR ...
     ACCESSOR = 2; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-    // This comment is left unintentionally blank.
+    // MAINTENANCE ...
     MAINTENANCE = 3; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
   }
 
-  // This comment is left unintentionally blank.
+  // Scope ...
   enum Scope {
-    // This comment is left unintentionally blank.
+    // REPOSITORY ...
     REPOSITORY = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
-    // This comment is left unintentionally blank.
+    // STORAGE ...
     STORAGE = 2; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
 
     reserved 1;
     reserved "SERVER";
   }
 
-  // This comment is left unintentionally blank.
+  // op ...
   Operation op = 1;
 
   // Scope level indicates what level an RPC interacts with a server:
diff --git a/proto/namespace.proto b/proto/namespace.proto
index 5a225f0ec..3c3eb7569 100644
--- a/proto/namespace.proto
+++ b/proto/namespace.proto
@@ -11,7 +11,7 @@ option go_package = "gitlab.com/gitlab-org/gitaly/v16/proto/go/gitalypb";
 // and shouldn't be used nowadays anymore.
 service NamespaceService {
 
-  // This comment is left unintentionally blank.
+  // AddNamespace ...
   rpc AddNamespace(AddNamespaceRequest) returns (AddNamespaceResponse) {
     option (op_type) = {
       op: MUTATOR
@@ -19,7 +19,7 @@ service NamespaceService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // RemoveNamespace ...
   rpc RemoveNamespace(RemoveNamespaceRequest) returns (RemoveNamespaceResponse) {
     option (op_type) = {
       op: MUTATOR
@@ -27,7 +27,7 @@ service NamespaceService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // RenameNamespace ...
   rpc RenameNamespace(RenameNamespaceRequest) returns (RenameNamespaceResponse) {
     option (op_type) = {
       op: MUTATOR
@@ -35,7 +35,7 @@ service NamespaceService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // NamespaceExists ...
   rpc NamespaceExists(NamespaceExistsRequest) returns (NamespaceExistsResponse) {
     option (op_type) = {
       op: ACCESSOR
@@ -45,54 +45,54 @@ service NamespaceService {
 
 }
 
-// This comment is left unintentionally blank.
+// AddNamespaceRequest ...
 message AddNamespaceRequest {
-  // This comment is left unintentionally blank.
+  // storage_name ...
   string storage_name = 1 [(storage)=true];
-  // This comment is left unintentionally blank.
+  // name ...
   string name = 2;
 }
 
-// This comment is left unintentionally blank.
+// RemoveNamespaceRequest ...
 message RemoveNamespaceRequest {
-  // This comment is left unintentionally blank.
+  // storage_name ...
   string storage_name = 1 [(storage)=true];
-  // This comment is left unintentionally blank.
+  // name ...
   string name = 2;
 }
 
-// This comment is left unintentionally blank.
+// RenameNamespaceRequest ...
 message RenameNamespaceRequest {
-  // This comment is left unintentionally blank.
+  // storage_name ...
   string storage_name = 1 [(storage)=true];
-  // This comment is left unintentionally blank.
+  // from ...
   string from = 2;
-  // This comment is left unintentionally blank.
+  // to ...
   string to = 3;
 }
 
-// This comment is left unintentionally blank.
+// NamespaceExistsRequest ...
 message NamespaceExistsRequest {
-  // This comment is left unintentionally blank.
+  // storage_name ...
   string storage_name = 1 [(storage)=true];
-  // This comment is left unintentionally blank.
+  // name ...
   string name = 2;
 }
 
-// This comment is left unintentionally blank.
+// NamespaceExistsResponse ...
 message NamespaceExistsResponse {
-  // This comment is left unintentionally blank.
+  // exists ...
   bool exists = 1;
 }
 
-// This comment is left unintentionally blank.
+// AddNamespaceResponse ...
 message AddNamespaceResponse {
 }
 
-// This comment is left unintentionally blank.
+// RemoveNamespaceResponse ...
 message RemoveNamespaceResponse {
 }
 
-// This comment is left unintentionally blank.
+// RenameNamespaceResponse ...
 message RenameNamespaceResponse {
 }
diff --git a/proto/operations.proto b/proto/operations.proto
index d61710044..5540ff5cf 100644
--- a/proto/operations.proto
+++ b/proto/operations.proto
@@ -15,14 +15,14 @@ option go_package = "gitlab.com/gitlab-org/gitaly/v16/proto/go/gitalypb";
 // '/allowed' endpoint.
 service OperationService {
 
-  // This comment is left unintentionally blank.
+  // UserCreateBranch ...
   rpc UserCreateBranch(UserCreateBranchRequest) returns (UserCreateBranchResponse) {
     option (op_type) = {
       op: MUTATOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // UserUpdateBranch ...
   rpc UserUpdateBranch(UserUpdateBranchRequest) returns (UserUpdateBranchResponse) {
     option (op_type) = {
       op: MUTATOR
@@ -57,7 +57,7 @@ service OperationService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // UserDeleteTag ...
   rpc UserDeleteTag(UserDeleteTagRequest) returns (UserDeleteTagResponse) {
     option (op_type) = {
       op: MUTATOR
@@ -175,21 +175,21 @@ service OperationService {
 
 }
 
-// This comment is left unintentionally blank.
+// UserCreateBranchRequest ...
 message UserCreateBranchRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // branch_name ...
   bytes branch_name = 2;
-  // This comment is left unintentionally blank.
+  // user ...
   User user = 3;
-  // This comment is left unintentionally blank.
+  // start_point ...
   bytes start_point = 4;
 }
 
-// This comment is left unintentionally blank.
+// UserCreateBranchResponse ...
 message UserCreateBranchResponse {
-  // This comment is left unintentionally blank.
+  // branch ...
   Branch branch = 1;
 
   // PreReceiveError had previously been set when creation of the branch was refused by hooks.
@@ -208,23 +208,23 @@ message UserCreateBranchError {
   }
 }
 
-// This comment is left unintentionally blank.
+// UserUpdateBranchRequest ...
 message UserUpdateBranchRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // branch_name ...
   bytes branch_name = 2;
-  // This comment is left unintentionally blank.
+  // user ...
   User user = 3;
-  // This comment is left unintentionally blank.
+  // newrev ...
   bytes newrev = 4;
-  // This comment is left unintentionally blank.
+  // oldrev ...
   bytes oldrev = 5;
 }
 
-// This comment is left unintentionally blank.
+// UserUpdateBranchResponse ...
 message UserUpdateBranchResponse {
-  // This comment is left unintentionally blank.
+  // pre_receive_error ...
   string pre_receive_error = 1;
 }
 
@@ -274,13 +274,13 @@ message UserDeleteBranchError {
   }
 }
 
-// This comment is left unintentionally blank.
+// UserDeleteTagRequest ...
 message UserDeleteTagRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // tag_name ...
   bytes tag_name = 2;
-  // This comment is left unintentionally blank.
+  // user ...
   User user = 3;
   // expected_old_oid is the object ID which tag is expected to point to.
   // This is used as a safety guard to avoid races when tag has been
@@ -292,9 +292,9 @@ message UserDeleteTagRequest {
   string expected_old_oid = 4;
 }
 
-// This comment is left unintentionally blank.
+// UserDeleteTagResponse ...
 message UserDeleteTagResponse {
-  // This comment is left unintentionally blank.
+  // pre_receive_error ...
   string pre_receive_error = 1;
 }
 
@@ -354,7 +354,7 @@ message UserCreateTagError {
   }
 }
 
-// This comment is left unintentionally blank.
+// UserMergeBranchRequest ...
 message UserMergeBranchRequest {
   // The following parameters must only be set in the first message to declare
   // parameters for the merge.
@@ -391,7 +391,7 @@ message UserMergeBranchRequest {
   bool apply = 6;
 }
 
-// This comment is left unintentionally blank.
+// UserMergeBranchResponse ...
 message UserMergeBranchResponse {
   // First message
   // The merge commit the branch will be updated to. The caller can still abort the merge.
@@ -426,7 +426,7 @@ message UserMergeBranchError {
   }
 }
 
-// This comment is left unintentionally blank.
+// UserMergeToRefRequest ...
 message UserMergeToRefRequest {
   // repository is the repository in which the merge shall be computed.
   Repository repository = 1 [(target_repository)=true];
@@ -471,7 +471,7 @@ message UserMergeToRefRequest {
   string expected_old_oid = 10;
 }
 
-// This comment is left unintentionally blank.
+// UserMergeToRefResponse ...
 message UserMergeToRefResponse {
   // commit_id is the object ID of the computed merge commit.
   string commit_id = 1;
@@ -553,15 +553,15 @@ message UserFFBranchRequest {
   string expected_old_oid = 5;
 }
 
-// This comment is left unintentionally blank.
+// UserFFBranchResponse ...
 message UserFFBranchResponse {
-  // This comment is left unintentionally blank.
+  // branch_update ...
   OperationBranchUpdate branch_update = 1;
-  // This comment is left unintentionally blank.
+  // pre_receive_error ...
   string pre_receive_error = 2;
 }
 
-// This comment is left unintentionally blank.
+// UserCherryPickRequest ...
 message UserCherryPickRequest {
   // repository is the repository into which the cherry-pick shall be
   // performed.
@@ -600,7 +600,7 @@ message UserCherryPickRequest {
   string expected_old_oid = 10;
 }
 
-// This comment is left unintentionally blank.
+// UserCherryPickResponse ...
 message UserCherryPickResponse {
   // branch_update represents details about the updated branch.
   OperationBranchUpdate branch_update = 1;
@@ -644,7 +644,7 @@ message UserCherryPickError {
   }
 }
 
-// This comment is left unintentionally blank.
+// UserRevertRequest ...
 message UserRevertRequest {
   // repository is the repository in which the revert shall be applied.
   Repository repository = 1 [(target_repository)=true];
@@ -682,7 +682,7 @@ message UserRevertRequest {
   string expected_old_oid = 10;
 }
 
-// This comment is left unintentionally blank.
+// UserRevertResponse ...
 message UserRevertResponse {
   // CreateTreeError represents an error which happened when computing the
   // revert.
@@ -713,7 +713,7 @@ message UserRevertResponse {
 
 // UserCommitFilesActionHeader contains the details of the action to be performed.
 message UserCommitFilesActionHeader {
-  // This comment is left unintentionally blank.
+  // ActionType ...
   enum ActionType {
     // CREATE creates a new file.
     CREATE = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
@@ -850,7 +850,7 @@ message UserCommitFilesError {
   }
 }
 
-// This comment is left unintentionally blank.
+// UserRebaseConfirmableRequest ...
 message UserRebaseConfirmableRequest {
   // Header contains information to compute the rebase and must be sent as
   // first message.
@@ -893,7 +893,7 @@ message UserRebaseConfirmableRequest {
   }
 }
 
-// This comment is left unintentionally blank.
+// UserRebaseConfirmableResponse ...
 message UserRebaseConfirmableResponse {
   oneof user_rebase_confirmable_response_payload {
     // The first response will contain the rebase commit the branch will
@@ -910,7 +910,7 @@ message UserRebaseConfirmableResponse {
   string git_error = 4;
 }
 
-// This comment is left unintentionally blank.
+// UserSquashRequest ...
 message UserSquashRequest {
   // repository is the repository into which the squashed commit shall be
   // written.
@@ -935,7 +935,7 @@ message UserSquashRequest {
   reserved "squash_id";
 }
 
-// This comment is left unintentionally blank.
+// UserSquashResponse ...
 message UserSquashResponse {
   // squash_sha is the object ID of the squashed commit.
   string squash_sha = 1;
@@ -951,7 +951,7 @@ message UserSquashResponse {
   reserved "git_error";
 }
 
-// This comment is left unintentionally blank.
+// UserRebaseConfirmableError ...
 message UserRebaseConfirmableError {
   oneof error {
     // RebaseConflict is returned in case rebasing commits on top of the start
@@ -977,7 +977,7 @@ message UserSquashError {
   }
 }
 
-// This comment is left unintentionally blank.
+// UserApplyPatchRequest ...
 message UserApplyPatchRequest {
   // Header contains information about how to apply the patches.
   message Header {
@@ -1010,13 +1010,13 @@ message UserApplyPatchRequest {
   }
 }
 
-// This comment is left unintentionally blank.
+// UserApplyPatchResponse ...
 message UserApplyPatchResponse {
   // branch_update contains information about the updated branch.
   OperationBranchUpdate branch_update = 1;
 }
 
-// This comment is left unintentionally blank.
+// UserUpdateSubmoduleRequest ...
 message UserUpdateSubmoduleRequest {
   // repository is the repository in which the submodule shall be updated.
   Repository repository = 1 [(target_repository)=true];
@@ -1047,7 +1047,7 @@ message UserUpdateSubmoduleRequest {
   string expected_old_oid = 8;
 }
 
-// This comment is left unintentionally blank.
+// UserUpdateSubmoduleResponse ...
 message UserUpdateSubmoduleResponse {
   // branch_update contains information about the updated branch.
   OperationBranchUpdate branch_update = 1;
diff --git a/proto/praefect.proto b/proto/praefect.proto
index b0cd76056..8fa22c074 100644
--- a/proto/praefect.proto
+++ b/proto/praefect.proto
@@ -13,7 +13,7 @@ option go_package = "gitlab.com/gitlab-org/gitaly/v16/proto/go/gitalypb";
 service PraefectInfoService {
   option (intercepted) = true;
 
-  // This comment is left unintentionally blank.
+  // RepositoryReplicas ...
   rpc RepositoryReplicas(RepositoryReplicasRequest) returns (RepositoryReplicasResponse);
 
   // DatalossCheck provides information on repositories in Praefect that are in a degraded state and
@@ -82,7 +82,7 @@ message MarkUnverifiedResponse {
 
 // GetRepositoryMetadataRequest specifies the repository to retrieve metadata for.
 message GetRepositoryMetadataRequest {
-  // This comment is left unintentionally blank.
+  // Path ...
   message Path {
     // virtual_storage is the virtual storage where the repository is.
     string virtual_storage = 1;
@@ -100,7 +100,7 @@ message GetRepositoryMetadataRequest {
 
 // GeRepositoryMetadataResponse contains the repository's cluster metadata.
 message GetRepositoryMetadataResponse {
-  // This comment is left unintentionally blank.
+  // Replica ...
   message Replica {
     // storage is the name of the replica's storage.
     string storage = 1;
@@ -149,17 +149,17 @@ message SetReplicationFactorResponse {
   repeated string storages = 1;
 }
 
-// This comment is left unintentionally blank.
+// SetAuthoritativeStorageRequest ...
 message SetAuthoritativeStorageRequest {
-  // This comment is left unintentionally blank.
+  // virtual_storage ...
   string virtual_storage = 1;
-  // This comment is left unintentionally blank.
+  // relative_path ...
   string relative_path = 2;
-  // This comment is left unintentionally blank.
+  // authoritative_storage ...
   string authoritative_storage = 3;
 }
 
-// This comment is left unintentionally blank.
+// SetAuthoritativeStorageResponse ...
 message SetAuthoritativeStorageResponse {
 }
 
@@ -177,7 +177,7 @@ message DatalossRequest {
 message DatalossResponse {
   // status of a repository
   message Repository {
-    // This comment is left unintentionally blank.
+    // Storage ...
     message Storage {
       // name of the storage
       string name = 1;
@@ -206,22 +206,22 @@ message DatalossResponse {
   repeated Repository repositories = 2;
 }
 
-// This comment is left unintentionally blank.
+// DatalossCheckRequest ...
 message DatalossCheckRequest {
   option deprecated = true;
-  // This comment is left unintentionally blank.
+  // virtual_storage ...
   string virtual_storage = 1;
   // include_partially_unavailable indicates whether to include repositories which are available but
   // are unavailable on some assigned storages.
   bool include_partially_replicated = 2;
 }
 
-// This comment is left unintentionally blank.
+// DatalossCheckResponse ...
 message DatalossCheckResponse {
   option deprecated = true;
-  // This comment is left unintentionally blank.
+  // Repository ...
   message Repository {
-    // This comment is left unintentionally blank.
+    // Storage ...
     message Storage {
       // name of the storage
       string name = 1;
@@ -250,24 +250,24 @@ message DatalossCheckResponse {
   repeated Repository repositories = 2;
 }
 
-// This comment is left unintentionally blank.
+// RepositoryReplicasRequest ...
 message RepositoryReplicasRequest{
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1;
 }
 
-// This comment is left unintentionally blank.
+// RepositoryReplicasResponse ...
 message RepositoryReplicasResponse{
-  // This comment is left unintentionally blank.
+  // RepositoryDetails ...
   message RepositoryDetails {
-    // This comment is left unintentionally blank.
+    // repository ...
     Repository repository = 1;
-    // This comment is left unintentionally blank.
+    // checksum ...
     string checksum = 2;
   };
 
-  // This comment is left unintentionally blank.
+  // primary ...
   RepositoryDetails primary = 1;
-  // This comment is left unintentionally blank.
+  // replicas ...
   repeated RepositoryDetails replicas = 2;
 }
diff --git a/proto/ref.proto b/proto/ref.proto
index 9d09bac91..37bf4f16a 100644
--- a/proto/ref.proto
+++ b/proto/ref.proto
@@ -33,7 +33,7 @@ service RefService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // FindAllBranches ...
   rpc FindAllBranches(FindAllBranchesRequest) returns (stream FindAllBranchesResponse) {
     option (op_type) = {
       op: ACCESSOR
@@ -56,14 +56,14 @@ service RefService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // FindAllRemoteBranches ...
   rpc FindAllRemoteBranches(FindAllRemoteBranchesRequest) returns (stream FindAllRemoteBranchesResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // RefExists ...
   rpc RefExists(RefExistsRequest) returns (RefExistsResponse) {
     option (op_type) = {
       op: ACCESSOR
@@ -88,21 +88,21 @@ service RefService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // DeleteRefs ...
   rpc DeleteRefs(DeleteRefsRequest) returns (DeleteRefsResponse) {
     option (op_type) = {
       op: MUTATOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // ListBranchNamesContainingCommit ...
   rpc ListBranchNamesContainingCommit(ListBranchNamesContainingCommitRequest) returns (stream ListBranchNamesContainingCommitResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // ListTagNamesContainingCommit ...
   rpc ListTagNamesContainingCommit(ListTagNamesContainingCommitRequest) returns (stream ListTagNamesContainingCommitResponse) {
     option (op_type) = {
       op: ACCESSOR
@@ -119,7 +119,7 @@ service RefService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // GetTagMessages ...
   rpc GetTagMessages(GetTagMessagesRequest) returns (stream GetTagMessagesResponse) {
     option (op_type) = {
       op: ACCESSOR
@@ -161,21 +161,21 @@ message FindDefaultBranchNameResponse {
   bytes name = 1;
 }
 
-// This comment is left unintentionally blank.
+// FindLocalBranchesRequest ...
 message FindLocalBranchesRequest {
-  // This comment is left unintentionally blank.
+  // SortBy ...
   enum SortBy {
-    // This comment is left unintentionally blank.
+    // NAME ...
     NAME = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
-    // This comment is left unintentionally blank.
+    // UPDATED_ASC ...
     UPDATED_ASC = 1; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-    // This comment is left unintentionally blank.
+    // UPDATED_DESC ...
     UPDATED_DESC = 2; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
   }
 
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // sort_by ...
   SortBy sort_by = 2;
   // The page token is the branch name, with the `refs/heads/` prefix, for
   // example "refs/heads/master". After the first branch name is encountered
@@ -184,19 +184,19 @@ message FindLocalBranchesRequest {
   PaginationParameter pagination_params = 3;
 }
 
-// This comment is left unintentionally blank.
+// FindLocalBranchesResponse ...
 message FindLocalBranchesResponse {
   // The field Branches has been removed in favor of LocalBranches.
   // Issue: https://gitlab.com/gitlab-org/gitaly/-/issues/1294
   reserved "branches";
   reserved 1;
-  // This comment is left unintentionally blank.
+  // local_branches ...
   repeated Branch local_branches = 2;
 }
 
-// This comment is left unintentionally blank.
+// FindAllBranchesRequest ...
 message FindAllBranchesRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
   // Only return branches that are merged into root ref
   bool merged_only = 2;
@@ -205,17 +205,17 @@ message FindAllBranchesRequest {
   repeated bytes merged_branches = 3;
 }
 
-// This comment is left unintentionally blank.
+// FindAllBranchesResponse ...
 message FindAllBranchesResponse {
-  // This comment is left unintentionally blank.
+  // Branch ...
   message Branch {
-    // This comment is left unintentionally blank.
+    // name ...
     bytes name = 1;
-    // This comment is left unintentionally blank.
+    // target ...
     GitCommit target = 2;
   }
 
-  // This comment is left unintentionally blank.
+  // branches ...
   repeated Branch branches = 1;
 }
 
@@ -244,15 +244,15 @@ message FindTagError {
   }
 }
 
-// This comment is left unintentionally blank.
+// FindAllTagsRequest ...
 message FindAllTagsRequest {
   // SortBy allows to specify desired order of the elements.
   message SortBy {
     // Key is a key used for sorting.
     enum Key {
-      // This comment is left unintentionally blank.
+      //  ...
       REFNAME     = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
-      // This comment is left unintentionally blank.
+      // CREATORDATE ...
       CREATORDATE = 1; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
       // VERSION_REFNAME sorts tags by their semantic versions (https://semver.org/).
       // Tag names that are not semantic versions are sorted lexicographically. They come before
@@ -261,13 +261,13 @@ message FindAllTagsRequest {
       VERSION_REFNAME = 2; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
     }
 
-    // This comment is left unintentionally blank.
+    //  ...
     Key           key       = 1;
-    // This comment is left unintentionally blank.
+    // direction ...
     SortDirection direction = 2;
   }
 
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
   // SortBy allows to request tags in particular order.
   SortBy sort_by = 2;
@@ -277,61 +277,61 @@ message FindAllTagsRequest {
   PaginationParameter pagination_params = 3;
 }
 
-// This comment is left unintentionally blank.
+// FindAllTagsResponse ...
 message FindAllTagsResponse {
-  // This comment is left unintentionally blank.
+  // tags ...
   repeated Tag tags = 1;
 }
 
-// This comment is left unintentionally blank.
+// RefExistsRequest ...
 message RefExistsRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
   // Any ref, e.g. 'refs/heads/master' or 'refs/tags/v1.0.1'. Must start with 'refs/'.
   bytes ref = 2;
 }
 
-// This comment is left unintentionally blank.
+// RefExistsResponse ...
 message RefExistsResponse {
-  // This comment is left unintentionally blank.
+  // value ...
   bool value = 1;
 }
 
-// This comment is left unintentionally blank.
+// CreateBranchRequest ...
 message CreateBranchRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // name ...
   bytes name = 2;
-  // This comment is left unintentionally blank.
+  // start_point ...
   bytes start_point = 3;
 }
 
-// This comment is left unintentionally blank.
+// CreateBranchResponse ...
 message CreateBranchResponse {
-  // This comment is left unintentionally blank.
+  // Status ...
   enum Status {
-    // This comment is left unintentionally blank.
+    // OK ...
     OK = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
-    // This comment is left unintentionally blank.
+    // ERR_EXISTS ...
     ERR_EXISTS = 1; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-    // This comment is left unintentionally blank.
+    // ERR_INVALID ...
     ERR_INVALID = 2; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-    // This comment is left unintentionally blank.
+    // ERR_INVALID_START_POINT ...
     ERR_INVALID_START_POINT = 3; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
   }
 
-  // This comment is left unintentionally blank.
+  // status ...
   Status status = 1;
-  // This comment is left unintentionally blank.
+  // branch ...
   Branch branch = 2;
 }
 
-// This comment is left unintentionally blank.
+// DeleteBranchRequest ...
 message DeleteBranchRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // name ...
   bytes name = 2;
 }
 
@@ -339,7 +339,7 @@ message DeleteBranchRequest {
 message DeleteBranchResponse {
 }
 
-// This comment is left unintentionally blank.
+// FindBranchRequest ...
 message FindBranchRequest {
   // repository is the repository in which the branch should be looked up.
   Repository repository = 1 [(target_repository)=true];
@@ -348,9 +348,9 @@ message FindBranchRequest {
   bytes name = 2;
 }
 
-// This comment is left unintentionally blank.
+// FindBranchResponse ...
 message FindBranchResponse {
-  // This comment is left unintentionally blank.
+  // branch ...
   Branch branch = 1;
 }
 
@@ -401,19 +401,19 @@ message UpdateReferencesError {
   }
 }
 
-// This comment is left unintentionally blank.
+// DeleteRefsRequest ...
 message DeleteRefsRequest{
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
   // The following two fields are mutually exclusive
   repeated bytes except_with_prefix = 2; // protolint:disable:this REPEATED_FIELD_NAMES_PLURALIZED
-  // This comment is left unintentionally blank.
+  // refs ...
   repeated bytes refs = 3;
 }
 
-// This comment is left unintentionally blank.
+// DeleteRefsResponse ...
 message DeleteRefsResponse {
-  // This comment is left unintentionally blank.
+  // git_error ...
   string git_error = 1;
 }
 
@@ -429,39 +429,39 @@ message DeleteRefsError {
   }
 }
 
-// This comment is left unintentionally blank.
+// ListBranchNamesContainingCommitRequest ...
 message ListBranchNamesContainingCommitRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // commit_id ...
   string commit_id = 2;
   // Limit the number of tag names to be returned
   // If the limit is set to zero, all items will be returned
   uint32 limit = 3;
 }
 
-// This comment is left unintentionally blank.
+// ListBranchNamesContainingCommitResponse ...
 message ListBranchNamesContainingCommitResponse {
   reserved 1;
-  // This comment is left unintentionally blank.
+  // branch_names ...
   repeated bytes branch_names = 2;
 }
 
-// This comment is left unintentionally blank.
+// ListTagNamesContainingCommitRequest ...
 message ListTagNamesContainingCommitRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // commit_id ...
   string commit_id = 2;
   // Limit the number of tag names to be returned
   // If the limit is set to zero, all items will be returned
   uint32 limit = 3;
 }
 
-// This comment is left unintentionally blank.
+// ListTagNamesContainingCommitResponse ...
 message ListTagNamesContainingCommitResponse {
   reserved 1;
-  // This comment is left unintentionally blank.
+  // tag_names ...
   repeated bytes tag_names = 2;
 }
 
@@ -496,61 +496,61 @@ message GetTagSignaturesResponse {
   repeated TagSignature signatures = 1;
 }
 
-// This comment is left unintentionally blank.
+// GetTagMessagesRequest ...
 message GetTagMessagesRequest {
   reserved 2;
   reserved "tag_names";
 
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // tag_ids ...
   repeated string tag_ids = 3;
 }
 
-// This comment is left unintentionally blank.
+// GetTagMessagesResponse ...
 message GetTagMessagesResponse {
   reserved 1;
   reserved "tag_name";
 
-  // This comment is left unintentionally blank.
+  // message ...
   bytes message = 2;
   // Only present for a new tag message
   string tag_id = 3;
 }
 
-// This comment is left unintentionally blank.
+// FindAllRemoteBranchesRequest ...
 message FindAllRemoteBranchesRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // remote_name ...
   string remote_name = 2;
 }
 
-// This comment is left unintentionally blank.
+// FindAllRemoteBranchesResponse ...
 message FindAllRemoteBranchesResponse {
-  // This comment is left unintentionally blank.
+  // branches ...
   repeated Branch branches = 1;
 }
 
 // ListRefsRequest is a request for the ListRefs RPC.
 message ListRefsRequest {
-  // This comment is left unintentionally blank.
+  // SortBy ...
   message SortBy {
-    // This comment is left unintentionally blank.
+    // Key ...
     enum Key {
-      // This comment is left unintentionally blank.
+      //  ...
       REFNAME       = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
-      // This comment is left unintentionally blank.
+      //  ...
       CREATORDATE   = 1; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-      // This comment is left unintentionally blank.
+      //  ...
       AUTHORDATE    = 2; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-      // This comment is left unintentionally blank.
+      // COMMITTERDATE ...
       COMMITTERDATE = 3; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
     }
 
     // Key is a key used for sorting.
     Key           key       = 1;
-    // This comment is left unintentionally blank.
+    // direction ...
     SortDirection direction = 2;
   }
 
@@ -592,7 +592,7 @@ message ListRefsResponse{
   repeated Reference references = 1;
 }
 
-// This comment is left unintentionally blank.
+// FindRefsByOIDRequest ...
 message FindRefsByOIDRequest {
   // repository is the repository in which references will be looked for.
   Repository repository = 1 [(target_repository)=true];
@@ -609,7 +609,7 @@ message FindRefsByOIDRequest {
   uint32 limit = 5;
 }
 
-// This comment is left unintentionally blank.
+// FindRefsByOIDResponse ...
 message FindRefsByOIDResponse {
   // refs is the set of fully-qualified references which have been found.
   repeated string refs = 1;
diff --git a/proto/remote.proto b/proto/remote.proto
index c18695426..839dbf6f5 100644
--- a/proto/remote.proto
+++ b/proto/remote.proto
@@ -22,7 +22,7 @@ service RemoteService {
       };
   }
 
-  // This comment is left unintentionally blank.
+  // FindRemoteRepository ...
   rpc FindRemoteRepository(FindRemoteRepositoryRequest) returns (FindRemoteRepositoryResponse) {
     option (op_type) = {
         op: ACCESSOR
@@ -42,9 +42,9 @@ service RemoteService {
   }
 }
 
-// This comment is left unintentionally blank.
+// UpdateRemoteMirrorRequest ...
 message UpdateRemoteMirrorRequest {
-  // This comment is left unintentionally blank.
+  // Remote ...
   message Remote {
     // URL is the URL of the remote repository.
     string url = 1;
@@ -91,16 +91,16 @@ message UpdateRemoteMirrorRequest {
   reserved "ref_name";
 }
 
-// This comment is left unintentionally blank.
+// UpdateRemoteMirrorResponse ...
 message UpdateRemoteMirrorResponse {
   // DivergentRefs contains a list of references that had diverged in the
   // mirror from the source repository.
   repeated bytes divergent_refs = 1;
 }
 
-// This comment is left unintentionally blank.
+// FindRemoteRepositoryRequest ...
 message FindRemoteRepositoryRequest {
-  // This comment is left unintentionally blank.
+  // remote ...
   string remote = 1;
   // This field is used to redirect request to proper storage where it can be handled.
   // As of now it doesn't matter what storage will be used, but it still must be a valid.
@@ -111,7 +111,7 @@ message FindRemoteRepositoryRequest {
 // This migth throw a GRPC Unavailable code, to signal the request failure
 // is transient.
 message FindRemoteRepositoryResponse {
-  // This comment is left unintentionally blank.
+  // exists ...
   bool exists = 1;
 }
 
diff --git a/proto/repository.proto b/proto/repository.proto
index 72a62b04b..d343ee8d0 100644
--- a/proto/repository.proto
+++ b/proto/repository.proto
@@ -10,7 +10,7 @@ option go_package = "gitlab.com/gitlab-org/gitaly/v16/proto/go/gitalypb";
 // RepositoryService is a service providing RPCs accessing repositories as a whole.
 service RepositoryService {
 
-  // This comment is left unintentionally blank.
+  // RepositoryExists ...
   rpc RepositoryExists(RepositoryExistsRequest) returns (RepositoryExistsResponse) {
     option (op_type) = {
       op: ACCESSOR
@@ -68,7 +68,7 @@ service RepositoryService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // ApplyGitattributes ...
   rpc ApplyGitattributes(ApplyGitattributesRequest) returns (ApplyGitattributesResponse) {
     option (op_type) = {
       op: MUTATOR
@@ -83,21 +83,21 @@ service RepositoryService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // CreateRepository ...
   rpc CreateRepository(CreateRepositoryRequest) returns (CreateRepositoryResponse) {
     option (op_type) = {
       op: MUTATOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // GetArchive ...
   rpc GetArchive(GetArchiveRequest) returns (stream GetArchiveResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // HasLocalBranches ...
   rpc HasLocalBranches(HasLocalBranchesRequest) returns (HasLocalBranchesResponse) {
     option (op_type) = {
       op: ACCESSOR
@@ -120,14 +120,14 @@ service RepositoryService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // WriteRef ...
   rpc WriteRef(WriteRefRequest) returns (WriteRefResponse) {
     option (op_type) = {
       op: MUTATOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // FindMergeBase ...
   rpc FindMergeBase(FindMergeBaseRequest) returns (FindMergeBaseResponse) {
     option (op_type) = {
       op: ACCESSOR
@@ -147,7 +147,7 @@ service RepositoryService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // CreateRepositoryFromURL ...
   rpc CreateRepositoryFromURL(CreateRepositoryFromURLRequest) returns (CreateRepositoryFromURLResponse) {
     option (op_type) = {
       op: MUTATOR
@@ -202,49 +202,49 @@ service RepositoryService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // GetInfoAttributes ...
   rpc GetInfoAttributes(GetInfoAttributesRequest) returns (stream GetInfoAttributesResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // CalculateChecksum ...
   rpc CalculateChecksum(CalculateChecksumRequest) returns (CalculateChecksumResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // GetSnapshot ...
   rpc GetSnapshot(GetSnapshotRequest) returns (stream GetSnapshotResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // CreateRepositoryFromSnapshot ...
   rpc CreateRepositoryFromSnapshot(CreateRepositoryFromSnapshotRequest) returns (CreateRepositoryFromSnapshotResponse) {
     option (op_type) = {
       op: MUTATOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // GetRawChanges ...
   rpc GetRawChanges(GetRawChangesRequest) returns (stream GetRawChangesResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // SearchFilesByContent ...
   rpc SearchFilesByContent(SearchFilesByContentRequest) returns (stream SearchFilesByContentResponse) {
     option (op_type) = {
       op: ACCESSOR
     };
   }
 
-  // This comment is left unintentionally blank.
+  // SearchFilesByName ...
   rpc SearchFilesByName(SearchFilesByNameRequest) returns (stream SearchFilesByNameResponse) {
     option (op_type) = {
       op: ACCESSOR
@@ -289,7 +289,7 @@ service RepositoryService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // GetObjectDirectorySize ...
   rpc GetObjectDirectorySize(GetObjectDirectorySizeRequest) returns (GetObjectDirectorySizeResponse) {
     option (op_type) = {
       op: ACCESSOR
@@ -305,7 +305,7 @@ service RepositoryService {
     };
   }
 
-  // This comment is left unintentionally blank.
+  // RenameRepository ...
   rpc RenameRepository(RenameRepositoryRequest) returns (RenameRepositoryResponse) {
     option (op_type) = {
       op: MUTATOR
@@ -402,15 +402,15 @@ service RepositoryService {
   }
 }
 
-// This comment is left unintentionally blank.
+// RepositoryExistsRequest ...
 message RepositoryExistsRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
 }
 
-// This comment is left unintentionally blank.
+// RepositoryExistsResponse ...
 message RepositoryExistsResponse {
-  // This comment is left unintentionally blank.
+  // exists ...
   bool exists = 1;
 }
 
@@ -525,19 +525,19 @@ message ObjectFormatResponse {
   ObjectFormat format = 1;
 }
 
-// This comment is left unintentionally blank.
+// ApplyGitattributesRequest ...
 message ApplyGitattributesRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // revision ...
   bytes revision = 2;
 }
 
-// This comment is left unintentionally blank.
+// ApplyGitattributesResponse ...
 message ApplyGitattributesResponse {
 }
 
-// This comment is left unintentionally blank.
+// FetchBundleRequest ...
 message FetchBundleRequest {
   // Repository into which the reference shall be fetched.
   Repository repository = 1 [(target_repository)=true];
@@ -549,13 +549,13 @@ message FetchBundleRequest {
   bool update_head = 3;
 }
 
-// This comment is left unintentionally blank.
+// FetchBundleResponse ...
 message FetchBundleResponse {
 }
 
-// This comment is left unintentionally blank.
+// FetchRemoteRequest ...
 message FetchRemoteRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
   // force determines if references should be force-updated in case they have
   // diverged.
@@ -564,9 +564,9 @@ message FetchRemoteRequest {
   bool no_tags = 4;
   // timeout specifies a timeout for the fetch.
   int32 timeout = 5;
-  // This comment is left unintentionally blank.
+  // ssh_key ...
   string ssh_key = 6;
-  // This comment is left unintentionally blank.
+  // known_hosts ...
   string known_hosts = 7;
   reserved 8;
   // no_prune will the fetch to not prune remote references which do not exist
@@ -584,7 +584,7 @@ message FetchRemoteRequest {
   reserved "remote";
 }
 
-// This comment is left unintentionally blank.
+// FetchRemoteResponse ...
 message FetchRemoteResponse {
   // If check_tags_changed was set in the FetchRemoteRequest, the FetchRemote
   // RPC will return false when no tags were changed, and true if tags were
@@ -592,9 +592,9 @@ message FetchRemoteResponse {
   bool tags_changed = 1;
 }
 
-// This comment is left unintentionally blank.
+// CreateRepositoryRequest ...
 message CreateRepositoryRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
   // Provide a branch name to set as the default branch of a newly created
   // repository. Note, this will be treated as the branch name and not a
@@ -606,35 +606,35 @@ message CreateRepositoryRequest {
   ObjectFormat object_format = 3;
 }
 
-// This comment is left unintentionally blank.
+// CreateRepositoryResponse ...
 message CreateRepositoryResponse {
 }
 
-// This comment is left unintentionally blank.
+// GetArchiveRequest ...
 message GetArchiveRequest {
-  // This comment is left unintentionally blank.
+  // Format ...
   enum Format {
-    // This comment is left unintentionally blank.
+    // ZIP ...
     ZIP = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
-    // This comment is left unintentionally blank.
+    // TAR ...
     TAR = 1; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-    // This comment is left unintentionally blank.
+    // TAR_GZ ...
     TAR_GZ = 2; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-    // This comment is left unintentionally blank.
+    // TAR_BZ2 ...
     TAR_BZ2 = 3; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
   }
 
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // commit_id ...
   string commit_id = 2;
-  // This comment is left unintentionally blank.
+  // prefix ...
   string prefix = 3;
-  // This comment is left unintentionally blank.
+  // format ...
   Format format = 4;
-  // This comment is left unintentionally blank.
+  // path ...
   bytes path = 5;
-  // This comment is left unintentionally blank.
+  // exclude ...
   repeated bytes exclude = 6; // protolint:disable:this REPEATED_FIELD_NAMES_PLURALIZED
   // If `elide_path` is true and `path` refers to a subdirectory, that
   // subdirectory will be elided from archive entries. For example, if `dir`
@@ -643,29 +643,29 @@ message GetArchiveRequest {
   // `README.md`. `elide_path` has no effect if `path` refers to the repository
   // root. `elide_path = true` is not supported if `path` refers to a file.
   bool elide_path = 7;
-  // This comment is left unintentionally blank.
+  // include_lfs_blobs ...
   bool include_lfs_blobs = 8;
 }
 
-// This comment is left unintentionally blank.
+// GetArchiveResponse ...
 message GetArchiveResponse {
-  // This comment is left unintentionally blank.
+  // data ...
   bytes data = 1;
 }
 
-// This comment is left unintentionally blank.
+// HasLocalBranchesRequest ...
 message HasLocalBranchesRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
 }
 
-// This comment is left unintentionally blank.
+// HasLocalBranchesResponse ...
 message HasLocalBranchesResponse {
-  // This comment is left unintentionally blank.
+  // value ...
   bool value = 1;
 }
 
-// This comment is left unintentionally blank.
+// FetchSourceBranchRequest ...
 message FetchSourceBranchRequest {
   // Repository into which the reference shall be fetched. After a successful
   // call, it should contain the target reference which points to the same
@@ -680,7 +680,7 @@ message FetchSourceBranchRequest {
   bytes target_ref = 4;
 }
 
-// This comment is left unintentionally blank.
+// FetchSourceBranchResponse ...
 message FetchSourceBranchResponse {
   // True if the source branch was successfully fetched into the target
   // repository, false if resolving the remote reference or fetching it failed.
@@ -699,33 +699,33 @@ message FsckResponse {
   bytes error = 1;
 }
 
-// This comment is left unintentionally blank.
+// WriteRefRequest ...
 message WriteRefRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // ref ...
   bytes ref = 2;
-  // This comment is left unintentionally blank.
+  // revision ...
   bytes revision = 3;
-  // This comment is left unintentionally blank.
+  // old_revision ...
   bytes old_revision = 4;
-  // This comment is left unintentionally blank.
+  // force ...
   bool force = 5;
   // This used to be a boolean indicating whether or not to shell out or use
   // the rugged implementation
   reserved 6;
 }
 
-// This comment is left unintentionally blank.
+// WriteRefResponse ...
 message WriteRefResponse {
   // This used to contain an error message. Since we're shelling out
   // all exceptions are wrapped in GRPC errors.
   reserved 1;
 }
 
-// This comment is left unintentionally blank.
+// FindMergeBaseRequest ...
 message FindMergeBaseRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
   // We use a repeated field because rugged supports finding a base
   // for more than 2 revisions, so if we needed that in the future we don't
@@ -733,9 +733,9 @@ message FindMergeBaseRequest {
   repeated bytes revisions = 2;
 }
 
-// This comment is left unintentionally blank.
+// FindMergeBaseResponse ...
 message FindMergeBaseResponse {
-  // This comment is left unintentionally blank.
+  // base ...
   string base = 1;
 }
 
@@ -759,11 +759,11 @@ message CreateForkRequest {
 message CreateForkResponse {
 }
 
-// This comment is left unintentionally blank.
+// CreateRepositoryFromURLRequest ...
 message CreateRepositoryFromURLRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // url ...
   string url = 2;
   // http_authorization_header is the HTTP header which can be added to
   // the request in order to authenticate against the repository.
@@ -787,23 +787,23 @@ message CreateRepositoryFromURLRequest {
   reserved "http_host";
 }
 
-// This comment is left unintentionally blank.
+// CreateRepositoryFromURLResponse ...
 message CreateRepositoryFromURLResponse {
 }
 
-// This comment is left unintentionally blank.
+// CreateBundleRequest ...
 message CreateBundleRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
 }
 
-// This comment is left unintentionally blank.
+// CreateBundleResponse ...
 message CreateBundleResponse {
-  // This comment is left unintentionally blank.
+  // data ...
   bytes data = 1;
 }
 
-// This comment is left unintentionally blank.
+// CreateBundleFromRefListRequest ...
 message CreateBundleFromRefListRequest {
   // Repository is the repository that the bundle is created from.
   Repository repository = 1 [(target_repository)=true];
@@ -814,9 +814,9 @@ message CreateBundleFromRefListRequest {
   repeated bytes patterns = 2;
 }
 
-// This comment is left unintentionally blank.
+// CreateBundleFromRefListResponse ...
 message CreateBundleFromRefListResponse {
-  // This comment is left unintentionally blank.
+  // data ...
   bytes data = 1;
 }
 
@@ -833,51 +833,51 @@ message GetConfigResponse {
   bytes data = 1;
 }
 
-// This comment is left unintentionally blank.
-message  RestoreCustomHooksRequest {
-  // This comment is left unintentionally blank.
+// RestoreCustomHooksRequest ...
+message RestoreCustomHooksRequest {
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // data ...
   bytes data = 2;
 }
 
-// This comment is left unintentionally blank.
-message  SetCustomHooksRequest {
-  // This comment is left unintentionally blank.
+// SetCustomHooksRequest ...
+message SetCustomHooksRequest {
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // data ...
   bytes data = 2;
 }
 
-// This comment is left unintentionally blank.
+// RestoreCustomHooksResponse ...
 message RestoreCustomHooksResponse {
 }
 
-// This comment is left unintentionally blank.
+// SetCustomHooksResponse ...
 message SetCustomHooksResponse {
 }
 
-// This comment is left unintentionally blank.
+// BackupCustomHooksRequest ...
 message BackupCustomHooksRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
 }
 
-// This comment is left unintentionally blank.
+// GetCustomHooksRequest ...
 message GetCustomHooksRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
 }
 
-// This comment is left unintentionally blank.
+// BackupCustomHooksResponse ...
 message BackupCustomHooksResponse {
-  // This comment is left unintentionally blank.
+  // data ...
   bytes data = 1;
 }
 
-// This comment is left unintentionally blank.
+// GetCustomHooksResponse ...
 message GetCustomHooksResponse {
-  // This comment is left unintentionally blank.
+  // data ...
   bytes data = 1;
 }
 
@@ -921,49 +921,49 @@ message FindLicenseResponse {
   string license_nickname = 5;
 }
 
-// This comment is left unintentionally blank.
+// GetInfoAttributesRequest ...
 message GetInfoAttributesRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
 }
 
-// This comment is left unintentionally blank.
+// GetInfoAttributesResponse ...
 message GetInfoAttributesResponse {
-  // This comment is left unintentionally blank.
+  // attributes ...
   bytes attributes = 1;
 }
 
-// This comment is left unintentionally blank.
+// CalculateChecksumRequest ...
 message CalculateChecksumRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
 }
 
-// This comment is left unintentionally blank.
+// CalculateChecksumResponse ...
 message CalculateChecksumResponse {
-  // This comment is left unintentionally blank.
+  // checksum ...
   string checksum = 1;
 }
 
-// This comment is left unintentionally blank.
+// GetSnapshotRequest ...
 message GetSnapshotRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
 }
 
-// This comment is left unintentionally blank.
+// GetSnapshotResponse ...
 message GetSnapshotResponse {
-  // This comment is left unintentionally blank.
+  // data ...
   bytes data = 1;
 }
 
-// This comment is left unintentionally blank.
+// CreateRepositoryFromSnapshotRequest ...
 message CreateRepositoryFromSnapshotRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // http_url ...
   string http_url = 2;
-  // This comment is left unintentionally blank.
+  // http_auth ...
   string http_auth = 3;
   // ResolvedAddress holds the resolved IP address of the remote_url. This is
   // used to avoid DNS rebinding by mapping the url to the resolved address.
@@ -978,46 +978,46 @@ message CreateRepositoryFromSnapshotRequest {
   reserved "http_host";
 }
 
-// This comment is left unintentionally blank.
+// CreateRepositoryFromSnapshotResponse ...
 message CreateRepositoryFromSnapshotResponse {
 }
 
-// This comment is left unintentionally blank.
+// GetRawChangesRequest ...
 message GetRawChangesRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // from_revision ...
   string from_revision = 2;
-  // This comment is left unintentionally blank.
+  // to_revision ...
   string to_revision = 3;
 }
 
-// This comment is left unintentionally blank.
+// GetRawChangesResponse ...
 message GetRawChangesResponse {
-  // This comment is left unintentionally blank.
+  // RawChange ...
   message RawChange {
-    // This comment is left unintentionally blank.
+    // Operation ...
     enum Operation {
-      // This comment is left unintentionally blank.
+      // UNKNOWN ...
       UNKNOWN = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
-      // This comment is left unintentionally blank.
+      // ADDED ...
       ADDED = 1; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-      // This comment is left unintentionally blank.
+      // COPIED ...
       COPIED = 2; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-      // This comment is left unintentionally blank.
+      // DELETED ...
       DELETED = 3; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-      // This comment is left unintentionally blank.
+      // MODIFIED ...
       MODIFIED = 4; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-      // This comment is left unintentionally blank.
+      // RENAMED ...
       RENAMED = 5; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-      // This comment is left unintentionally blank.
+      // TYPE_CHANGED ...
       TYPE_CHANGED = 6; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
     }
 
-    // This comment is left unintentionally blank.
+    // blob_id ...
     string blob_id = 1;
-    // This comment is left unintentionally blank.
-    int64 size= 2;
+    // size ...
+    int64 size = 2;
 
     // This used to be a string that is now represented by the field 9 as byte array.
     reserved 3;
@@ -1026,32 +1026,32 @@ message GetRawChangesResponse {
     reserved 4;
     reserved "old_path";
 
-    // This comment is left unintentionally blank.
-    Operation operation= 5;
-    // This comment is left unintentionally blank.
+    // operation ...
+    Operation operation = 5;
+    // raw_operation ...
     string raw_operation = 6;
-    // This comment is left unintentionally blank.
+    // old_mode ...
     int32 old_mode = 7;
-    // This comment is left unintentionally blank.
+    // new_mode ...
     int32 new_mode = 8;
 
     // the following fields, 9 and 10, will eventually replace 3 and 4
     bytes new_path_bytes = 9;
-    // This comment is left unintentionally blank.
+    // old_path_bytes ...
     bytes old_path_bytes = 10;
   }
 
-  // This comment is left unintentionally blank.
+  // raw_changes ...
   repeated RawChange raw_changes = 1;
 }
 
-// This comment is left unintentionally blank.
+// SearchFilesByNameRequest ...
 message SearchFilesByNameRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // query ...
   string query = 2;
-  // This comment is left unintentionally blank.
+  // ref ...
   bytes ref = 3;
   // If `filter` is specified and non-empty, it will be parsed as a regular
   // expression and used to filter the result set before it is transmitted. It is
@@ -1067,31 +1067,31 @@ message SearchFilesByNameRequest {
   uint32 offset = 6;
 }
 
-// This comment is left unintentionally blank.
+// SearchFilesByNameResponse ...
 message SearchFilesByNameResponse {
   // Files contains the paths of files that have been found to match the query.
   repeated bytes files = 1;
 }
 
-// This comment is left unintentionally blank.
+// SearchFilesByContentRequest ...
 message SearchFilesByContentRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // query ...
   string query = 2;
-  // This comment is left unintentionally blank.
+  // ref ...
   bytes ref = 3;
-  // This comment is left unintentionally blank.
+  // chunked_response ...
   bool chunked_response = 4;
 }
 
-// This comment is left unintentionally blank.
+// SearchFilesByContentResponse ...
 message SearchFilesByContentResponse {
-  // This comment is left unintentionally blank.
+  // matches ...
   repeated bytes matches = 1;
-  // This comment is left unintentionally blank.
+  // match_data ...
   bytes match_data = 2;
-  // This comment is left unintentionally blank.
+  // end_of_match ...
   bool end_of_match = 3;
 }
 
@@ -1135,37 +1135,37 @@ message Remote {
   reserved "http_host";
 }
 
-// This comment is left unintentionally blank.
+// GetObjectDirectorySizeRequest ...
 message GetObjectDirectorySizeRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
 }
 
-// This comment is left unintentionally blank.
+// GetObjectDirectorySizeResponse ...
 message GetObjectDirectorySizeResponse {
   // Object directory size in kilobytes
   int64 size = 1;
 }
 
-// This comment is left unintentionally blank.
+// RemoveRepositoryRequest ...
 message RemoveRepositoryRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
 }
 
-// This comment is left unintentionally blank.
+// RemoveRepositoryResponse ...
 message RemoveRepositoryResponse {
 }
 
-// This comment is left unintentionally blank.
+// RenameRepositoryRequest ...
 message RenameRepositoryRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
-  // This comment is left unintentionally blank.
+  // relative_path ...
   string relative_path = 2;
 }
 
-// This comment is left unintentionally blank.
+// RenameRepositoryResponse ...
 message RenameRepositoryResponse{
 }
 
@@ -1219,7 +1219,7 @@ message OptimizeRepositoryResponse {
 // PruneUnreachableObjectsRequest is a request for the PruneUnreachableObjects
 // RPC call.
 message PruneUnreachableObjectsRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(target_repository)=true];
 }
 
diff --git a/proto/server.proto b/proto/server.proto
index 9e4953b06..83fab5616 100644
--- a/proto/server.proto
+++ b/proto/server.proto
@@ -11,10 +11,10 @@ option go_package = "gitlab.com/gitlab-org/gitaly/v16/proto/go/gitalypb";
 service ServerService {
   option (intercepted) = true;
 
-  // This comment is left unintentionally blank.
+  // ServerInfo ...
   rpc ServerInfo(ServerInfoRequest) returns (ServerInfoResponse);
 
-  // This comment is left unintentionally blank.
+  // DiskStatistics ...
   rpc DiskStatistics(DiskStatisticsRequest) returns (DiskStatisticsResponse);
 
   // ClockSynced checks if machine clock is synced
@@ -25,54 +25,54 @@ service ServerService {
   rpc ReadinessCheck(ReadinessCheckRequest) returns (ReadinessCheckResponse);
 }
 
-// This comment is left unintentionally blank.
+// ServerInfoRequest ...
 message ServerInfoRequest {
 }
 
-// This comment is left unintentionally blank.
+// ServerInfoResponse ...
 message ServerInfoResponse {
-  // This comment is left unintentionally blank.
+  // StorageStatus ...
   message StorageStatus {
-    // This comment is left unintentionally blank.
+    // storage_name ...
     string storage_name = 1;
-    // This comment is left unintentionally blank.
+    // readable ...
     bool readable = 2;
-    // This comment is left unintentionally blank.
+    // writeable ...
     bool writeable = 3;
-    // This comment is left unintentionally blank.
+    // fs_type ...
     string fs_type = 4;
-    // This comment is left unintentionally blank.
+    // filesystem_id ...
     string filesystem_id = 5;
-    // This comment is left unintentionally blank.
+    // replication_factor ...
     uint32 replication_factor = 6;
   }
 
-  // This comment is left unintentionally blank.
+  // server_version ...
   string server_version = 1;
-  // This comment is left unintentionally blank.
+  // git_version ...
   string git_version = 2;
-  // This comment is left unintentionally blank.
+  // storage_statuses ...
   repeated StorageStatus storage_statuses = 3;
 }
 
-// This comment is left unintentionally blank.
+// DiskStatisticsRequest ...
 message DiskStatisticsRequest {
 }
 
-// This comment is left unintentionally blank.
+// DiskStatisticsResponse ...
 message DiskStatisticsResponse {
-  // This comment is left unintentionally blank.
+  // StorageStatus ...
   message StorageStatus {
     // When both available and used fields are equal 0 that means that
     // Gitaly was unable to determine storage stats.
     string storage_name = 1;
-    // This comment is left unintentionally blank.
+    // available ...
     int64 available = 2;
-    // This comment is left unintentionally blank.
+    // used ...
     int64 used = 3;
   }
 
-  // This comment is left unintentionally blank.
+  // storage_statuses ...
   repeated StorageStatus storage_statuses = 1;
 }
 
diff --git a/proto/shared.proto b/proto/shared.proto
index ddbc5ead2..de0a617bd 100644
--- a/proto/shared.proto
+++ b/proto/shared.proto
@@ -7,17 +7,17 @@ import "lint.proto";
 
 option go_package = "gitlab.com/gitlab-org/gitaly/v16/proto/go/gitalypb";
 
-// This comment is left unintentionally blank.
+// ObjectType ...
 enum ObjectType {
-  // This comment is left unintentionally blank.
+  // UNKNOWN ...
   UNKNOWN = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
-  // This comment is left unintentionally blank.
+  // COMMIT ...
   COMMIT = 1; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-  // This comment is left unintentionally blank.
+  // BLOB ...
   BLOB = 2; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-  // This comment is left unintentionally blank.
+  // TREE ...
   TREE = 3; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-  // This comment is left unintentionally blank.
+  // TAG ...
   TAG = 4; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
 }
 
@@ -32,28 +32,28 @@ enum ObjectFormat {
   OBJECT_FORMAT_SHA256 = 2;
 }
 
-// This comment is left unintentionally blank.
+// SignatureType ...
 enum SignatureType {
-  // This comment is left unintentionally blank.
+  // NONE ...
   NONE = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
-  // This comment is left unintentionally blank.
+  // PGP ...
   PGP = 1; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-  // This comment is left unintentionally blank.
+  // X509 ...
   X509 = 2; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-  // This comment is left unintentionally blank.
+  // SSH ...
   SSH = 3; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
   // maybe add X509+TSA or other combinations at a later step
 }
 
-// This comment is left unintentionally blank.
+// Repository ...
 message Repository {
   // DEPRECATED: https://gitlab.com/gitlab-org/gitaly/issues/151
   reserved 1;
   reserved "path";
 
-  // This comment is left unintentionally blank.
+  // storage_name ...
   string storage_name = 2;
-  // This comment is left unintentionally blank.
+  // relative_path ...
   string relative_path = 3;
   // Sets the GIT_OBJECT_DIRECTORY envvar on git commands to the value of this field.
   // It influences the object storage directory the SHA1 directories are created underneath.
@@ -95,23 +95,23 @@ message CommitStatInfo {
 
 // Corresponds to Gitlab::Git::Commit
 message GitCommit {
-  // This comment is left unintentionally blank.
+  // id ...
   string id = 1;
-  // This comment is left unintentionally blank.
+  // subject ...
   bytes subject = 2;
-  // This comment is left unintentionally blank.
+  // body ...
   bytes body = 3;
-  // This comment is left unintentionally blank.
+  // author ...
   CommitAuthor author = 4;
-  // This comment is left unintentionally blank.
+  // committer ...
   CommitAuthor committer = 5;
-  // This comment is left unintentionally blank.
+  // parent_ids ...
   repeated string parent_ids = 6;
   // If body exceeds a certain threshold, it will be nullified,
   // but its size will be set in body_size so we can know if
   // a commit had a body in the first place.
   int64 body_size = 7;
-  // This comment is left unintentionally blank.
+  // signature_type ...
   SignatureType signature_type = 8;
   // The tree ID will always be filled, even if the tree is empty. In that case
   // the value will be `4b825dc642cb6eb9a060e54bf8d69288fbee4904`.
@@ -135,74 +135,74 @@ message GitCommit {
   string encoding = 13;
 }
 
-// This comment is left unintentionally blank.
+// CommitAuthor ...
 message CommitAuthor {
-  // This comment is left unintentionally blank.
+  // name ...
   bytes name = 1;
-  // This comment is left unintentionally blank.
+  // email ...
   bytes email = 2;
-  // This comment is left unintentionally blank.
+  // date ...
   google.protobuf.Timestamp date = 3;
-  // This comment is left unintentionally blank.
+  // timezone ...
   bytes timezone = 4;
 }
 
-// This comment is left unintentionally blank.
+// ExitStatus ...
 message ExitStatus {
-  // This comment is left unintentionally blank.
+  // value ...
   int32 value = 1;
 }
 
 // Corresponds to Gitlab::Git::Branch
 message Branch {
-  // This comment is left unintentionally blank.
+  // name ...
   bytes name = 1;
-  // This comment is left unintentionally blank.
+  // target_commit ...
   GitCommit target_commit = 2;
 }
 
-// This comment is left unintentionally blank.
+// Tag ...
 message Tag {
-  // This comment is left unintentionally blank.
+  // name ...
   bytes name = 1;
-  // This comment is left unintentionally blank.
+  // id ...
   string id = 2;
-  // This comment is left unintentionally blank.
+  // target_commit ...
   GitCommit target_commit = 3;
   // If message exceeds a certain threshold, it will be nullified,
   // but its size will be set in message_size so we can know if
   // a tag had a message in the first place.
   bytes message = 4;
-  // This comment is left unintentionally blank.
+  // message_size ...
   int64 message_size = 5;
-  // This comment is left unintentionally blank.
+  // tagger ...
   CommitAuthor tagger = 6;
-  // This comment is left unintentionally blank.
+  // signature_type ...
   SignatureType signature_type = 7;
 }
 
-// This comment is left unintentionally blank.
+// User ...
 message User {
-  // This comment is left unintentionally blank.
+  // gl_id ...
   string gl_id = 1;
-  // This comment is left unintentionally blank.
+  // name ...
   bytes name = 2;
-  // This comment is left unintentionally blank.
+  // email ...
   bytes email = 3;
-  // This comment is left unintentionally blank.
+  // gl_username ...
   string gl_username = 4;
   // Timezone is the timezone as configured by the user in the web interface. This
   // timezone may be used when new commits are created via RPC calls.
   string timezone = 5;
 }
 
-// This comment is left unintentionally blank.
+// ObjectPool ...
 message ObjectPool {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1 [(gitaly.repository)=true];
 }
 
-// This comment is left unintentionally blank.
+// PaginationParameter ...
 message PaginationParameter {
   // Instructs pagination to start sending results after the provided page
   // token appears. A page token allows for a generic pattern to uniquely
@@ -219,7 +219,7 @@ message PaginationParameter {
   int32 limit = 2;
 }
 
-// This comment is left unintentionally blank.
+// PaginationCursor ...
 message PaginationCursor {
   // To the caller, this is an opaque token to indicate what the caller
   // should present as a page_token to get subsequent results.
diff --git a/proto/ssh.proto b/proto/ssh.proto
index afe3e70d8..22d0400f2 100644
--- a/proto/ssh.proto
+++ b/proto/ssh.proto
@@ -39,7 +39,7 @@ service SSHService {
   }
 }
 
-// This comment is left unintentionally blank.
+// SSHUploadPackRequest ...
 message SSHUploadPackRequest {
   // 'repository' must be present in the first message.
   Repository repository = 1 [(target_repository)=true];
@@ -55,7 +55,7 @@ message SSHUploadPackRequest {
   string git_protocol = 5;
 }
 
-// This comment is left unintentionally blank.
+// SSHUploadPackResponse ...
 message SSHUploadPackResponse {
   // A chunk of raw data from 'git upload-pack' standard output
   bytes stdout = 1;
@@ -66,7 +66,7 @@ message SSHUploadPackResponse {
   ExitStatus exit_status = 3;
 }
 
-// This comment is left unintentionally blank.
+// SSHUploadPackWithSidechannelRequest ...
 message SSHUploadPackWithSidechannelRequest {
   // 'repository' must be present in the first message.
   Repository repository = 1 [(target_repository)=true];
@@ -77,13 +77,13 @@ message SSHUploadPackWithSidechannelRequest {
   string git_protocol = 3;
 }
 
-// This comment is left unintentionally blank.
+// SSHUploadPackWithSidechannelResponse ...
 message SSHUploadPackWithSidechannelResponse {
   // Packfile negotiation statistics.
   PackfileNegotiationStatistics packfile_negotiation_statistics = 1;
 }
 
-// This comment is left unintentionally blank.
+// SSHReceivePackRequest ...
 message SSHReceivePackRequest {
   // 'repository' must be present in the first message.
   Repository repository = 1 [(target_repository)=true];
@@ -92,9 +92,9 @@ message SSHReceivePackRequest {
   // Contents of GL_ID, GL_REPOSITORY, and GL_USERNAME environment variables
   // for 'git receive-pack'
   string gl_id = 3;
-  // This comment is left unintentionally blank.
+  // gl_repository ...
   string gl_repository = 4;
-  // This comment is left unintentionally blank.
+  // gl_username ...
   string gl_username = 5;
   // Git protocol version
   string git_protocol = 6;
@@ -102,7 +102,7 @@ message SSHReceivePackRequest {
   repeated string git_config_options = 7;
 }
 
-// This comment is left unintentionally blank.
+// SSHReceivePackResponse ...
 message SSHReceivePackResponse {
   // A chunk of raw data from 'git receive-pack' standard output
   bytes stdout = 1;
@@ -113,7 +113,7 @@ message SSHReceivePackResponse {
   ExitStatus exit_status = 3;
 }
 
-// This comment is left unintentionally blank.
+// SSHUploadArchiveRequest ...
 message SSHUploadArchiveRequest {
   // 'repository' must be present in the first message.
   Repository repository = 1 [(target_repository)=true];
@@ -121,7 +121,7 @@ message SSHUploadArchiveRequest {
   bytes stdin = 2;
 }
 
-// This comment is left unintentionally blank.
+// SSHUploadArchiveResponse ...
 message SSHUploadArchiveResponse {
   // A chunk of raw data from 'git upload-archive' standard output
   bytes stdout = 1;
diff --git a/proto/transaction.proto b/proto/transaction.proto
index 9e8d0f5ee..7656aa4e0 100644
--- a/proto/transaction.proto
+++ b/proto/transaction.proto
@@ -42,9 +42,9 @@ service RefTransaction {
 
 }
 
-// This comment is left unintentionally blank.
+// VoteTransactionRequest ...
 message VoteTransactionRequest {
-  // This comment is left unintentionally blank.
+  // Phase ...
   enum Phase {
     // UNKNOWN_PHASE is the unknown voting phase. This value has been the
     // default because phases have been introduced. Eventually, using this
@@ -62,7 +62,7 @@ message VoteTransactionRequest {
     SYNCHRONIZED_PHASE = 3;  // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
   };
 
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1[(target_repository)=true];
   // ID of the transaction we're processing
   uint64 transaction_id = 2;
@@ -74,31 +74,31 @@ message VoteTransactionRequest {
   Phase phase = 5;
 }
 
-// This comment is left unintentionally blank.
+// VoteTransactionResponse ...
 message VoteTransactionResponse {
   // The outcome of the given transaction telling the client whether the
   // transaction should be committed or rolled back.
   enum TransactionState {
-    // This comment is left unintentionally blank.
+    // COMMIT ...
     COMMIT = 0; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX ENUM_FIELD_NAMES_ZERO_VALUE_END_WITH
-    // This comment is left unintentionally blank.
+    // ABORT ...
     ABORT = 1; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
-    // This comment is left unintentionally blank.
+    // STOP ...
     STOP = 2; // protolint:disable:this ENUM_FIELD_NAMES_PREFIX
   }
 
-  // This comment is left unintentionally blank.
+  // state ...
   TransactionState state = 1;
 }
 
-// This comment is left unintentionally blank.
+// StopTransactionRequest ...
 message StopTransactionRequest {
-  // This comment is left unintentionally blank.
+  // repository ...
   Repository repository = 1[(target_repository)=true];
   // ID of the transaction we're processing
   uint64 transaction_id = 2;
 }
 
-// This comment is left unintentionally blank.
+// StopTransactionResponse ...
 message StopTransactionResponse {
 }