Fix filtering out contributions created by bots

Author: ZibanPirate

api/src/contribution/controller.ts

@@ -20,6 +20,7 @@ export class ContributionController {
   ): Promise<GetContributionsResponseDto> {
     const { contributions, filters } = await this.contributionRepository.find(
       (contribution) =>
+        !contribution.createdBy.username.includes("[bot]") &&
         (labels.length === 0 || labels.some((label) => contribution.labels.includes(label))) &&
         (languages.length === 0 ||
           languages.some((language) => contribution.languages.includes(language))) &&

api/src/contribution/repository.ts

@@ -22,7 +22,7 @@ export class ContributionRepository {
 
     let contributions = (
       await Promise.all(
-        projects.reduce<Promise<Model<ContributionEntity, "project">[]>[]>(
+        projects.reduce<Promise<Model<ContributionEntity, "project" | "createdBy">[]>[]>(
           (pV, { repositories, name, slug }) => [
             ...pV,
             ...repositories
@@ -38,8 +38,7 @@ export class ContributionRepository {
                     owner,
                     repository,
                   });
-                  // @TODO-ZM: filter out the ones created by bots
-                  return issuesIncludingPRs.map<Model<ContributionEntity, "project">>(
+                  return issuesIncludingPRs.map<Model<ContributionEntity, "project" | "createdBy">>(
                     ({
                       number,
                       labels: gLabels,
@@ -49,6 +48,7 @@ export class ContributionRepository {
                       created_at, // eslint-disable-line camelcase
                       updated_at, // eslint-disable-line camelcase
                       comments,
+                      user,
                     }) => ({
                       id: `${number}`,
                       labels: gLabels.map(({ name }) => name),
@@ -64,6 +64,7 @@ export class ContributionRepository {
                       updatedAt: updated_at, // eslint-disable-line camelcase
                       commentsCount: comments,
                       /* eslint-enable camelcase */
+                      createdBy: this.githubService.githubUserToAccountEntity(user),
                     }),
                   );
                 } catch (error) {

api/src/contribution/types.ts

@@ -31,7 +31,7 @@ export class FilterDto {
 export class GetContributionsResponseDto extends GeneralResponseDto {
   @ValidateNested({ each: true })
   @Type(() => ContributionEntity)
-  contributions!: Model<ContributionEntity, "project">[];
+  contributions!: Model<ContributionEntity, "project" | "createdBy">[];
 
   @ValidateNested({ each: true })
   @Type(() => FilterDto)

packages/models/src/contribution/index.ts

@@ -1,6 +1,7 @@
 import { Type } from "class-transformer";
 import { IsDateString, IsNumber, IsString, IsUrl, ValidateNested } from "class-validator";
 import { BaseEntity, Model } from "src/_base";
+import { AccountEntity } from "src/account";
 import { ProjectReferenceEntity } from "src/project-reference";
 
 export class ContributionEntity extends BaseEntity {
@@ -14,6 +15,10 @@ export class ContributionEntity extends BaseEntity {
   @Type(() => ProjectReferenceEntity)
   project!: Model<ProjectReferenceEntity>;
 
+  @ValidateNested()
+  @Type(() => AccountEntity)
+  createdBy!: Model<AccountEntity>;
+
   @IsString()
   type!: "issue" | "pullRequest";