fix: prevent empty commits and remove unintentional leading blank lines

- Add logic to skip commits when solution content is unchanged
- Remove unintentional leading blank lines that occur when submitting solution via CMD+Enter
- Bump version of CodeHub in manifest.json

Author: febinbellamy

manifest.backup.json

@@ -2,7 +2,7 @@
   "name": "CodeHub",
   "description": "Automatically pushes your Codewars submissions to GitHub",
   "homepage_url": "https://github.com/FebinBellamy/CodeHub",
-  "version": "1.0.0",
+  "version": "2.0.2",
   "author": "Febin Bellamy",
   "action": {
     "default_icon": {
@@ -23,11 +23,17 @@
     "type": "module",
     "persistent": false
   },
-  "permissions": ["storage", "tabs", "scripting", "webNavigation", "activeTab"],
+  "permissions": [
+    "storage",
+    "tabs",
+    "scripting",
+    "webNavigation",
+    "activeTab"
+  ],
   "host_permissions": [
     "https://www.codewars.com/*",
     "https://github.com/*",
     "https://api.github.com/*"
   ],
   "manifest_version": 3
-}
+}

manifest.json

@@ -2,7 +2,7 @@
   "name": "CodeHub",
   "description": "Automatically pushes your Codewars submissions to GitHub",
   "homepage_url": "https://github.com/FebinBellamy/CodeHub",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "author": "Febin Bellamy",
   "action": {
     "default_icon": {
@@ -23,11 +23,17 @@
     "type": "module",
     "persistent": false
   },
-  "permissions": ["storage", "tabs", "scripting", "webNavigation", "activeTab"],
+  "permissions": [
+    "storage",
+    "tabs",
+    "scripting",
+    "webNavigation",
+    "activeTab"
+  ],
   "host_permissions": [
     "https://www.codewars.com/*",
     "https://github.com/*",
     "https://api.github.com/*"
   ],
   "manifest_version": 3
-}
+}

scripts/codewars.js

@@ -36,19 +36,16 @@ chrome.storage.local.get(["isRepoConnected"], (result) => {
       const getUserSolution = () => {
         const userCodeBeforeParsing =
           document.querySelectorAll(".CodeMirror-lines")[0].innerText;
-        const userCodeArr = userCodeBeforeParsing.split("\n");
-        let parsedUserSolution = "";
-
-        for (let i = 0; i < userCodeArr.length; i++) {
-          let elem = userCodeArr[i];
-          if (/[^0-9]/.test(elem)) {
-            parsedUserSolution += elem;
-            if (i !== userCodeArr.length - 1) {
-              parsedUserSolution += "\n";
-            }
-          }
+        let userCodeArr = userCodeBeforeParsing.split("\n");
+        
+        const firstNonEmptyIndex = userCodeArr.findIndex(line => /[^0-9]/.test(line) && line.trim() !== "");
+        if (firstNonEmptyIndex > 0) {
+          userCodeArr = userCodeArr.slice(firstNonEmptyIndex);
         }
-        data["userSolution"] = parsedUserSolution;
+        
+        data["userSolution"] = userCodeArr
+          .filter(elem => /[^0-9]/.test(elem) && elem !== "")
+          .join("\n");
       };
 
       const interceptRedirection = () => {
@@ -78,10 +75,14 @@ chrome.storage.local.get(["isRepoConnected"], (result) => {
       document.addEventListener("keydown", (event) => {
         const isCmdOrCtrlKeyPressed = event.ctrlKey || event.metaKey;
         const isEnterKeyPressed = event.key === "Enter";
+        const attemptButton = document.querySelector("#attempt_btn");
+        const isSubmitButtonVisible = 
+          !submitButton.classList.contains("is-hidden") &&
+          attemptButton.style.display === "none";
         if (
           isCmdOrCtrlKeyPressed &&
           isEnterKeyPressed &&
-          !submitButton.classList.contains("is-hidden")
+          isSubmitButtonVisible
         ) {
           submitButton.click();
         }

scripts/codewarsToGithub.js

@@ -89,6 +89,27 @@ const checkFileExists = async (baseUrl, accessToken) => {
   };
 };
 
+const isContentIdentical = (existingContentBase64, newContentBase64) => {
+  try {
+    const existingContentBase64Clean = existingContentBase64.replace(/\s/g, "");
+    const existingContent = decodeURIComponent(
+      escape(atob(existingContentBase64Clean))
+    ).replace(/\r\n/g, "\n").replace(/\r/g, "\n");
+
+    const newContent = decodeURIComponent(escape(atob(newContentBase64)))
+      .replace(/\r\n/g, "\n")
+      .replace(/\r/g, "\n");
+
+    const existingNormalized = existingContent.replace(/^\n+/, "").replace(/\n+$/, "");
+    const newNormalized = newContent.replace(/^\n+/, "").replace(/\n+$/, "");
+
+    return existingNormalized === newNormalized;
+  } catch (error) {
+    console.log("Error comparing file content:", error);
+    return false;
+  }
+};
+
 const getUrl = (
   githubUsername,
   repo,
@@ -164,6 +185,12 @@ const addOrUpdateSolution = async (
   };
 
   if (fileExists) {
+    if (isContentIdentical(fileData.content, encodedSolution)) {
+      console.log(
+        `There were no changes detected in your solution file! No update needed.`
+      );
+      return;
+    }
     data.sha = fileData.sha;
   }