refactor(idlewatcher): simplify loading.js console line handling and message fallback

Replace chained level checks with a validLevels list and includes().
Coalesce addConsoleLine call sites. Default missing payload.message to an
empty string so the console does not show undefined.
This commit is contained in:
yusing
2026-04-23 17:20:15 +08:00
parent e1eb8e9c03
commit 5699f004c8

View File

@@ -51,15 +51,12 @@ window.onload = async () => {
* @param {EventLevel} [level]
*/
function addConsoleLine(type, message, timestamp, level) {
const lvl =
level === "debug" ||
level === "info" ||
level === "warn" ||
level === "error"
? level
: type === "error"
? "error"
: "info";
const validLevels = ["debug", "info", "warn", "error"];
const lvl = validLevels.includes(level)
? level
: type === "error"
? "error"
: "info";
const line = document.createElement("div");
line.className = `console-line ${type} level-${lvl}`;
@@ -140,7 +137,7 @@ window.onload = async () => {
eventSource.close();
} else {
// Show other message types
const message = payload.message;
const message = payload.message || "";
addConsoleLine(type, message, timestamp, evt.level);
}
};