Refactor common vcs engine to use proper composition (#141)

* Create common VCS engine

* Add suppressPattern config, pass config object to gitlab and github directly

* Create VCSEngine config interface to avoid accessing unrelated configs from VCSEngine

* Move bot logic into AnalyzerBot

* Move files around and clean up complex methods

* Remove unused imports

* Change inheritance into composition

* Remove dedicate GitHub and GitLab engine and use VCSEngine directly with different adapter

* Remove unrelated changes from refactoring to make it less confusing

Author: icmpecho

src/AnalyzerBot/@interfaces/AnalyzerBotConfig.ts

@@ -0,0 +1,3 @@
+export interface AnalyzerBotConfig {
+  failOnWarnings: boolean;
+}

src/Provider/@types/CommentTypes.ts src/AnalyzerBot/@types/CommentTypes.tssrc/Provider/@types/CommentTypes.ts renamed to src/AnalyzerBot/@types/CommentTypes.ts

@@ -0,0 +1,45 @@
+import { LogSeverity, LogType } from '../Parser';
+import { Diff } from '../Git/@types/PatchTypes';
+import { onlyIn, onlySeverity } from './utils/filter.util';
+import { groupComments } from './utils/commentUtil';
+import { MessageUtil } from './utils/message.util';
+import { AnalyzerBotConfig } from './@interfaces/AnalyzerBotConfig';
+import { Comment } from './@types/CommentTypes';
+
+export class AnalyzerBot {
+  readonly touchedFileLog: LogType[];
+  readonly comments: Comment[];
+  readonly nError: number;
+  readonly nWarning: number;
+
+  constructor(
+    private readonly config: AnalyzerBotConfig,
+    private readonly logs: LogType[],
+    private readonly touchedDiff: Diff[],
+  ) {
+    this.touchedFileLog = logs
+      .filter(onlySeverity(LogSeverity.error, LogSeverity.warning))
+      .filter(onlyIn(touchedDiff));
+    this.comments = groupComments(this.touchedFileLog);
+    this.nError = this.comments.reduce((sum, comment) => sum + comment.errors, 0);
+    this.nWarning = this.comments.reduce((sum, comment) => sum + comment.warnings, 0);
+  }
+
+  shouldGenerateOverview(): boolean {
+    return this.nError + this.nWarning > 0;
+  }
+
+  getOverviewMessage(): string {
+    return MessageUtil.generateOverviewMessage(this.nError, this.nWarning);
+  }
+
+  getCommitDescription(): string {
+    return MessageUtil.generateCommitDescription(this.nError);
+  }
+
+  isSuccess(): boolean {
+    return this.config.failOnWarnings
+      ? this.nError + this.nWarning === 0
+      : this.nError === 0;
+  }
+}

src/AnalyzerBot/AnalyzerBot.ts

@@ -1,5 +1,5 @@
 import { LogSeverity, LogType } from '../../Parser';
-import { Diff } from '../@types/PatchTypes';
+import { Diff } from '../../Git/@types/PatchTypes';
 
 export const onlyIn = (diffs: Diff[]) => (log: LogType): boolean =>
   diffs.some(