🧹 Remove console logs in production helper (#27)

🎯 What: Removed console.log statements from ping and testUrls functions in src/utils/helper.ts, as well as an unused pingFinished boolean and some unused parameter variables (statusText and e).

💡 Why: console.log statements in production code add noise to logs and can leak information. Removing these logs improves code maintainability and addresses code health issues.

✅ Verification: Verified with bun test ensuring functionality remains intact. Code formatted with Biome and compiled with TypeScript.

✨ Result: A cleaner helper.ts file without unnecessary debug logging in production.

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
Co-authored-by: sunnylqm <615282+sunnylqm@users.noreply.github.com>

Author: sunnylqm

src/utils/helper.ts

@@ -26,31 +26,22 @@ export function promiseAny<T>(promises: Promise<T>[]) {
 }
 
 export const ping = async (url: string) => {
-  let pingFinished = false;
   return Promise.race([
     fetch(url, {
       method: 'HEAD',
     })
-      .then(({ status, statusText }) => {
-        pingFinished = true;
+      .then(({ status }) => {
         if (status === 200) {
-          console.log('ping success', url);
           return url;
         }
-        console.log('ping failed', url, status, statusText);
         throw Error('ping failed');
       })
-      .catch((e) => {
-        pingFinished = true;
-        console.log('ping error', url, e);
+      .catch(() => {
         throw Error('ping error');
       }),
     new Promise((_, reject) =>
       setTimeout(() => {
         reject(Error('ping timeout'));
-        if (!pingFinished) {
-          console.log('ping timeout', url);
-        }
       }, 2000),
     ),
   ]) as Promise<string | null>;
@@ -64,7 +55,6 @@ export const testUrls = async (urls?: string[]) => {
   if (ret) {
     return ret;
   }
-  console.log('all ping failed, use first url:', urls[0]);
   return urls[0];
 };