Merge pull request #1020 from bajrangCoder/logging-system

Implement Logger system

Author: bajrangCoder

src/ace/colorView.js

@@ -141,7 +141,8 @@ function afterRender() {
 				el.classList.add("ace_color");
 				el.style.cssText = `background-color: ${content}; color: ${fontColorString}; pointer-events: all;`;
 			} catch (error) {
-				console.log("Invalid color", content);
+				window.log("error", `Invalid color: ${content}`);
+				window.log("error", error);
 			}
 		});
 }

src/ace/commands.js

@@ -259,6 +259,13 @@ const commands = [
 			editor._emit("select-word");
 		},
 	},
+	{
+		name: "openLogFile",
+		description: "Open Log File",
+		exec() {
+			acode.exec("open-log-file");
+		},
+	},
 ];
 
 export function setCommands(editor) {
@@ -314,6 +321,7 @@ export async function resetKeyBindings() {
 		}
 		await fs.writeFile(content);
 	} catch (error) {
-		console.error(error);
+		window.log("error", "Reset Keybinding failed!");
+		window.log("error", error);
 	}
 }

src/components/palette/index.js

@@ -138,7 +138,7 @@ export default function palette(getList, onsSelectCb, placeholder, onremove) {
 		}
 
 		remove = () => {
-			console.error("Palette already removed");
+			window.log("warn", "Palette already removed.");
 		};
 	}
 }

src/components/searchbar/index.js

@@ -79,7 +79,8 @@ function searchBar($list, setHide, onhideCb, searchFunction) {
 				try {
 					result = await result;
 				} catch (error) {
-					console.error("Search function failed:", error);
+					window.log("error", "Search function failed:");
+					window.log("error", error);
 					result = [];
 				}
 			}

src/components/settingsPage.js

@@ -304,7 +304,7 @@ function listItems($list, items, callback) {
 				return;
 			}
 		} catch (error) {
-			console.log(error);
+			window.log("error", error);
 		}
 
 		item.value = res;

src/lib/actionStack.js

@@ -36,8 +36,9 @@ export default {
 		const copyStack = { ...this };
 		delete copyStack.windowCopy;
 		copyStack.pop = (repeat) => {
-			console.error(
-				"Deprecated: window.actionStack is deprecated, import actionStack instead",
+			window.log(
+				"error",
+				"Deprecated: `window.actionStack` is deprecated, import `actionStack` instead",
 			);
 			this.pop(repeat);
 		};

src/lib/commands.js

@@ -14,6 +14,7 @@ import commandPalette from "palettes/commandPalette";
 import findFile from "palettes/findFile";
 import help from "settings/helpSettings";
 import mainSettings from "settings/mainSettings";
+import Url from "utils/Url";
 import { getColorRange } from "utils/color/regex";
 import helpers from "utils/helpers";
 import checkFiles from "./checkFiles";
@@ -362,4 +363,7 @@ export default {
 		});
 		editorManager.activeFile.eol = eol;
 	},
+	"open-log-file"() {
+		openFile(Url.join(DATA_STORAGE, constants.LOG_FILE_NAME));
+	},
 };

src/lib/constants.js

@@ -22,4 +22,5 @@ export default {
 	API_BASE: "https://acode.app/api",
 	// API_BASE: 'https://192.168.0.102:3001/api', // test api
 	SKU_LIST: ["basic", "bronze", "silver", "gold", "platinum"],
+	LOG_FILE_NAME: "Acode.log",
 };

src/lib/editorFile.js

@@ -477,7 +477,8 @@ export default class EditorFile {
 
 			await fs.writeFile(text);
 		} catch (error) {
-			console.error(error);
+			window.log("error", "Writing to cache failed:");
+			window.log("error", error);
 		}
 	}
 
@@ -519,7 +520,7 @@ export default class EditorFile {
 			if (oldText.length !== text.length) return true;
 			return oldText !== text;
 		} catch (error) {
-			console.log(error);
+			window.log("error", error);
 			return false;
 		}
 	}
@@ -789,7 +790,8 @@ export default class EditorFile {
 			if (!(await fs.exists())) return;
 			fs.renameTo(newId);
 		} catch (error) {
-			console.error("renameCacheFile", error);
+			window.log("error", "renameCacheFile");
+			window.log("error", error);
 		}
 	}
 
@@ -802,7 +804,7 @@ export default class EditorFile {
 			if (!(await fs.exists())) return;
 			await fs.delete();
 		} catch (error) {
-			console.error(error);
+			window.log("error", error);
 		}
 	}
 
@@ -838,6 +840,7 @@ export default class EditorFile {
 				} else if (!cacheExists && fileExists) {
 					value = await file.readFile(this.encoding);
 				} else if (!cacheExists && !fileExists) {
+					window.log("error", "unable to load file");
 					throw new Error("Unable to load file");
 				}
 			}
@@ -870,7 +873,8 @@ export default class EditorFile {
 			this.#emit("loaderror", createFileEvent(this));
 			this.remove();
 			toast(`Unable to load: ${this.filename}`);
-			console.log(error);
+			window.log("error", "Unable to load: " + this.filename);
+			window.log("error", error);
 		} finally {
 			this.#emit("loadend", createFileEvent(this));
 		}

src/lib/fileList.js

@@ -222,7 +222,7 @@ export async function addRoot({ url, name }) {
 		emit("add-folder", tree);
 	} catch (error) {
 		// ignore
-		console.error(error);
+		window.log("error", error);
 	}
 }