From 66b4614f74ac8058f6d6bc6f6302a28c6ed5e4ec Mon Sep 17 00:00:00 2001 From: Saurabh Kumar Bajpai Date: Sun, 2 Aug 2026 17:27:25 +0530 Subject: [PATCH 1/3] fix: code quality and safety improvements --- src/pages/CodingVerse.jsx | 6 +++--- src/services/rankHistoryService.js | 2 +- src/utils/githubRateLimit.js | 6 +++--- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/pages/CodingVerse.jsx b/src/pages/CodingVerse.jsx index 0ef630b..41f903e 100644 --- a/src/pages/CodingVerse.jsx +++ b/src/pages/CodingVerse.jsx @@ -808,9 +808,9 @@ export const CodingVerse = () => { if (todayUTCStr !== newLastCodingVerseSolveDate) { const lastDateParts = newLastCodingVerseSolveDate.split("-"); const lastUTC = Date.UTC( - parseInt(lastDateParts[0]), - parseInt(lastDateParts[1]) - 1, - parseInt(lastDateParts[2]), + parseInt(lastDateParts[0], 10), + parseInt(lastDateParts[1], 10) - 1, + parseInt(lastDateParts[2], 10), ); const diffDays = Math.floor( (todayUTC - lastUTC) / (1000 * 60 * 60 * 24), diff --git a/src/services/rankHistoryService.js b/src/services/rankHistoryService.js index 597cd60..2f1aeb0 100644 --- a/src/services/rankHistoryService.js +++ b/src/services/rankHistoryService.js @@ -26,7 +26,7 @@ export const saveRankSnapshot = async (uid, rank, totalPoints, timezone) => { // Extract number from "#42" or similar const numericRank = typeof rank === "string" ? parseInt(rank.replace(/[^0-9]/g, ""), 10) : rank; - if (isNaN(numericRank)) return; + if (Number.isNaN(numericRank)) return; const timeZone = resolveTimezone(timezone); const todayStr = toLocalDateString(new Date(), timeZone); diff --git a/src/utils/githubRateLimit.js b/src/utils/githubRateLimit.js index 1696333..425e2eb 100644 --- a/src/utils/githubRateLimit.js +++ b/src/utils/githubRateLimit.js @@ -5,9 +5,9 @@ * and returns structured rate limit info. */ export const parseRateLimitHeaders = (headers) => { - const remaining = parseInt(headers?.["x-ratelimit-remaining"] ?? -1); - const limit = parseInt(headers?.["x-ratelimit-limit"] ?? -1); - const resetTimestamp = parseInt(headers?.["x-ratelimit-reset"] ?? 0); + const remaining = parseInt(headers?.["x-ratelimit-remaining"] ?? -1, 10); + const limit = parseInt(headers?.["x-ratelimit-limit"] ?? -1, 10); + const resetTimestamp = parseInt(headers?.["x-ratelimit-reset"] ?? 0, 10); const resetDate = resetTimestamp ? new Date(resetTimestamp * 1000) : null; const resetInMs = resetDate ? resetDate.getTime() - Date.now() : 0; From 0fc1a100cdda87afce1d098f02cc77e3f95a78e7 Mon Sep 17 00:00:00 2001 From: Saurabh Kumar Bajpai Date: Mon, 3 Aug 2026 00:39:59 +0530 Subject: [PATCH 2/3] fix: additional real bug fixes --- src/pages/CodingVerse.jsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/pages/CodingVerse.jsx b/src/pages/CodingVerse.jsx index 41f903e..48cee51 100644 --- a/src/pages/CodingVerse.jsx +++ b/src/pages/CodingVerse.jsx @@ -84,7 +84,7 @@ function runJavaScript(code, timeoutMs = 2000) { const logs = []; const errors = []; const fakeconsole = { - log: (...args) => logs.push(args.map(formatValue).join(" ")), + log: (...args) => logs.push((args ?? []).map(formatValue).join(" ")), error: (...args) => errors.push(args.map(formatValue).join(" ")), warn: (...args) => logs.push("[WARN] " + args.map(formatValue).join(" ")), info: (...args) => logs.push("[INFO] " + args.map(formatValue).join(" ")), From b294439daeec42b1304489f3d8873748e06ab1b5 Mon Sep 17 00:00:00 2001 From: Saurabh Kumar Bajpai Date: Mon, 3 Aug 2026 00:50:03 +0530 Subject: [PATCH 3/3] fix: additional real bug fixes --- src/pages/CodingVerse.jsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/pages/CodingVerse.jsx b/src/pages/CodingVerse.jsx index 48cee51..d2fad04 100644 --- a/src/pages/CodingVerse.jsx +++ b/src/pages/CodingVerse.jsx @@ -85,7 +85,7 @@ function runJavaScript(code, timeoutMs = 2000) { const errors = []; const fakeconsole = { log: (...args) => logs.push((args ?? []).map(formatValue).join(" ")), - error: (...args) => errors.push(args.map(formatValue).join(" ")), + error: (...args) => errors.push((args ?? []).map(formatValue).join(" ")), warn: (...args) => logs.push("[WARN] " + args.map(formatValue).join(" ")), info: (...args) => logs.push("[INFO] " + args.map(formatValue).join(" ")), };