Fix/prevent double console log commenting

utils/actions/detectConsoleLogs.ts

@@ -17,6 +17,21 @@ const app = new App({
   appId: process.env.GITHUB_APP_ID!,
   privateKey: process.env.GITHUB_PRIVATE_KEY!,
 });
+const commentBody = `This PR contains console logs. Please review or remove them.`;
+const consoleLogDetectionPrompt = `This is a list of code additions. Identify 
+if there's a console log or its equivalent in another programming language 
+such as Java, Golang, Python, C, Rust, C++, Ruby, etc.
+(console.log(), println(), println!(), System.out.println(), print(), fmt.Println(), puts, and  cout << "Print a String" << endl; are some examples). 
+If the console log or its equivalent in another language is in a code comment, don't
+count it as a detected console log. For example JavaScript comments start with // or /*, 
+Python comments start with #.
+Other console functions such as console.info() shouldn't be counted as console logs.
+Ignore code comments from this analysis. 
+Something like 'input[type="email"]' is fine and should not be counted as a console log.
+If there is a console log, return "true", else return "false".
+If you return true, return a string that that has 2 values: result (true) and the line of code.
+The line value, is the actual line in the file that contains the console log.
+For example: true,console.log("hello world");`;
 
 function getAdditions(filePatch: string) {
   const additions: string[] = [];
@@ -113,24 +128,25 @@ export default async function detectConsoleLogs({
     }
   );
 
+  function getLatestCommitHash() {
+    return octokit
+      .request("GET /repos/{owner}/{repo}/pulls/{pull_number}", {
+        owner,
+        repo,
+        pull_number: issue_number,
+      })
+      .then((result) => {
+        return result.data.head.sha;
+      })
+      .catch((err) => {
+        console.log(err);
+      });
+  }
+  const latestCommitHash = await getLatestCommitHash();
+
   const commentPromises = diffFiles.map(async (file) => {
     const additions = getAdditions(file.patch ?? "");
 
-    const consoleLogDetectionPrompt = `This is a list of code additions. Identify 
-    if there's a console log or its equivalent in another programming language 
-    such as Java, Golang, Python, C, Rust, C++, Ruby, etc.
-    (console.log(), println(), println!(), System.out.println(), print(), fmt.Println(), puts, and  cout << "Print a String" << endl; are some examples). 
-    If the console log or its equivalent in another language is in a code comment, don't
-    count it as a detected console log. For example JavaScript comments start with // or /*, 
-    Python comments start with #.
-    Other console functions such as console.info() shouldn't be counted as console logs.
-    Ignore code comments from this analysis. 
-    Something like 'input[type="email"]' is fine and should not be counted as a console log.
-    If there is a console log, return "true", else return "false".
-    If you return true, return a string that that has 2 values: result (true) and the line of code.
-    The line value, is the actual line in the file that contains the console log.
-    For example: true,console.log("hello world");`;
-
     // detect if the additions contain console logs or not
     try {
       return await openai
@@ -152,43 +168,33 @@ export default async function detectConsoleLogs({
 
           if (addtionsHaveConsoleLog === "true") {
             const commentFileDiff = () => {
+              const consoleLogPosition = getConsoleLogPosition({
+                filePatch: file.patch ?? "",
+                individualLine,
+              });
+
               return octokit
-                .request("GET /repos/{owner}/{repo}/pulls/{pull_number}", {
-                  owner,
-                  repo,
-                  pull_number: issue_number,
-                })
-                .then((result) => {
-                  const latestCommitHash = result.data.head.sha;
-
-                  const consoleLogPosition = getConsoleLogPosition({
-                    filePatch: file.patch ?? "",
-                    individualLine,
-                  });
-
-                  return octokit
-                    .request(
-                      "POST /repos/{owner}/{repo}/pulls/{pull_number}/reviews",
+                .request(
+                  "POST /repos/{owner}/{repo}/pulls/{pull_number}/reviews",
+                  {
+                    owner,
+                    repo,
+                    pull_number: issue_number,
+                    commit_id:
+                      typeof latestCommitHash === "string"
+                        ? latestCommitHash
+                        : undefined,
+                    event: "COMMENT",
+                    path: file.filename,
+                    comments: [
                       {
-                        owner,
-                        repo,
-                        pull_number: issue_number,
-                        commit_id: latestCommitHash,
-                        event: "COMMENT",
                         path: file.filename,
-                        comments: [
-                          {
-                            path: file.filename,
-                            position: consoleLogPosition || 1, // comment at the beggining of the file by default
-                            body: "This file contains at least one console log. Please remove any present.",
-                          },
-                        ],
-                      }
-                    )
-                    .catch((err) => {
-                      console.log(err);
-                    });
-                })
+                        position: consoleLogPosition || 1, // comment at the beggining of the file by default
+                        body: commentBody,
+                      },
+                    ],
+                  }
+                )
                 .catch((err) => {
                   console.log(err);
                 });