feat(#38): field declaration move to ctor, GitHub typo

Author: h1alexbel

src/chat-gpt.ts

@@ -32,22 +32,15 @@ import OpenAI from "openai";
  */
 export class ChatGpt implements Model {
 
-  /**
-   * Open AI.
-   */
-  private readonly open: OpenAI;
-
-  /**
-   * Model name.
-   */
-  private readonly model: string;
-
   /**
    * Ctor.
    * @param open Open AI
    * @param model Model name
    */
-  constructor(open: OpenAI, model: string) {
+  constructor(
+    private readonly open: OpenAI,
+    private readonly model: string
+  ) {
     this.open = open;
     this.model = model;
   }

src/comment.ts

@@ -28,31 +28,16 @@ import {Octokit} from "@octokit/rest";
  */
 export class Comment {
 
-  /**
-   * Github.
-   */
-  private readonly github: Octokit;
-
-  /**
-   * Issue.
-   */
-  private readonly issue: Issue;
-
-  /**
-   * Text to post.
-   */
-  private readonly text: string;
-
   /**
    * Ctor.
-   * @param github Github
+   * @param github GitHub
    * @param issue Issue
    * @param text Text
    */
   constructor(
-    github: Octokit,
-    issue: Issue,
-    text: string
+    private readonly github: Octokit,
+    private readonly issue: Issue,
+    private readonly text: string
   ) {
     this.github = github;
     this.issue = issue;

src/covered.ts

@@ -23,24 +23,14 @@
  */
 export class Covered {
 
-  /**
-   * Username.
-   */
-  private readonly username;
-
-  /**
-   * Message.
-   */
-  private readonly message: string;
-
   /**
    * Ctor.
    * @param username Username
    * @param message Message
    */
   constructor(
-    username: string | undefined,
-    message: string
+    private readonly username: string | undefined,
+    private readonly message: string
   ) {
     this.username = username;
     this.message = message;

src/deep-infra.ts

@@ -31,22 +31,15 @@ import {Rules} from "./rules";
  */
 export class DeepInfra implements Model {
 
-  /**
-   * API Key.
-   */
-  private readonly token: string;
-
-  /**
-   * Model.
-   */
-  private readonly model: string;
-
   /**
    * Ctor.
    * @param token Token
    * @param model Model
    */
-  constructor(token: string, model: string) {
+  constructor(
+    private readonly token: string,
+    private readonly model: string
+  ) {
     this.token = token;
     this.model = model;
   }

src/feedback.ts

@@ -33,38 +33,18 @@ import {NonRelevant} from "./non-relevant";
  */
 export class Feedback {
 
-  /**
-   * Summary.
-   */
-  private readonly summary: string | undefined;
-
-  /**
-   * GitHub.
-   */
-  private readonly github: Octokit;
-
-  /**
-   * Issue.
-   */
-  private readonly issue: Issue;
-
-  /**
-   * Username.
-   */
-  private readonly username: string | undefined;
-
   /**
    * Ctor.
    * @param summary Summary
-   * @param github Github
+   * @param github GitHub
    * @param issue Issue
    * @param username Username
    */
   constructor(
-    summary: string | undefined,
-    github: Octokit,
-    issue: Issue,
-    username: string | undefined
+    private readonly summary: string | undefined,
+    private readonly github: Octokit,
+    private readonly issue: Issue,
+    private readonly username: string | undefined
   ) {
     this.summary = summary;
     this.github = github;

src/label.ts

@@ -0,0 +1,56 @@
+/*
+ * The MIT License (MIT)
+ *
+ * Copyright (c) 2023-2024 Tracehub.git
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included
+ * in all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+import {Octokit} from "@octokit/rest";
+
+/**
+ * GitHub label.
+ */
+export class Label {
+
+  /**
+   * Ctor.
+   * @param github GitHub
+   * @param issue Issue
+   * @param label Label
+   */
+  constructor(
+    private readonly github: Octokit,
+    private readonly issue: Issue,
+    private readonly label: string) {
+  }
+
+  /**
+   * Attach label.
+   */
+  async attach() {
+    await this.github.issues.addLabels(
+      {
+        owner: this.issue.owner,
+        repo: this.issue.repo,
+        issue_number: this.issue.number,
+        labels: [this.label]
+      }
+    );
+  }
+}

src/non-relevant.ts

@@ -23,37 +23,23 @@
  */
 import {Octokit} from "@octokit/rest";
 import {Comment} from "./comment";
+import {Label} from "./label";
 
 /**
  * Non Relevant bug report.
  */
 export class NonRelevant {
 
-  /**
-   * Github.
-   */
-  private readonly github: Octokit;
-
-  /**
-   * Issue.
-   */
-  private readonly issue: Issue;
-
-  /**
-   * Creator.
-   */
-  private readonly creator: string | undefined;
-
   /**
    * Ctor.
    * @param github Github
    * @param issue Issue
    * @param creator Creator
    */
   constructor(
-    github: Octokit,
-    issue: Issue,
-    creator: string | undefined
+    private readonly github: Octokit,
+    private readonly issue: Issue,
+    private readonly creator: string | undefined
   ) {
     this.github = github;
     this.issue = issue;
@@ -71,14 +57,7 @@ export class NonRelevant {
       + " this issue is not relevant to " + "\`" + this.issue.owner + "/" + this.issue.repo + "\`" + "."
       + " We should close it."
     ).post();
-    await this.github.issues.addLabels(
-      {
-        owner: this.issue.owner,
-        repo: this.issue.repo,
-        issue_number: this.issue.number,
-        labels: ["invalid"]
-      }
-    );
+    await new Label(this.github, this.issue, "invalid").attach();
     await this.github.issues.update(
       {
         owner: this.issue.owner,

src/smart-issue.ts

@@ -30,28 +30,14 @@ import {Octokit} from "@octokit/rest";
  */
 export class SmartIssue {
 
-  /**
-   * Github.
-   */
-  private readonly github: Octokit;
-
-  /**
-   * Issue.
-   */
-  private readonly issue: {
-    owner: string,
-    repo: string,
-    number: number
-  };
-
   /**
    * Ctor.
-   * @param github Github
+   * @param github GitHub
    * @param issue Issue
    */
   constructor(
-    github: Octokit,
-    issue: {
+    private readonly github: Octokit,
+    private readonly issue: {
       owner: string,
       repo: string,
       number: number

src/user-prompt.ts

@@ -29,29 +29,18 @@ import {Rules} from "./rules";
  */
 export class UserPrompt {
 
-  /**
-   * Example.
-   */
-  private readonly example: Example;
-
-  /**
-   * Rules.
-   */
-  private readonly rules: Rules;
-
-  /**
-   * Bug report.
-   */
-  private readonly report: string | null | undefined;
-
   /**
    * Ctor.
    *
    * @param example Example
    * @param rules Rules
    * @param report Bug report
    */
-  constructor(example: Example, rules: Rules, report: string | null | undefined) {
+  constructor(
+    private readonly example: Example,
+    private readonly rules: Rules,
+    private readonly report: string | null | undefined
+  ) {
     this.example = example;
     this.rules = rules;
     this.report = report;

src/with-summary.ts

@@ -28,22 +28,15 @@ import {Covered} from "./covered";
  */
 export class WithSummary {
 
-  /**
-   * Origin.
-   */
-  private readonly origin: Covered;
-
-  /**
-   * Summary.
-   */
-  private readonly summary: string | undefined;
-
   /**
    * Ctor.
    * @param origin Origin
    * @param summary Summary
    */
-  constructor(origin: Covered, summary: string | undefined) {
+  constructor(
+    private readonly origin: Covered,
+    private readonly summary: string | undefined
+  ) {
     this.origin = origin;
     this.summary = summary;
   }