Merge pull request #37 from MrAlders0n/copilot/add-debug-console-logs-again

Add debug console logs and visual feedback for ping transmission status

Author: MrAlders0n

content/wardrive.js

@@ -20,6 +20,13 @@ const COOLDOWN_MS = 7000;                      // Cooldown period for manual pin
 const REPEATER_LISTEN_MS = 7000;               // Listen for repeater echoes for 7 seconds
 const STATUS_UPDATE_DELAY_MS = 100;            // Brief delay to ensure "Ping sent" status is visible
 const MAP_REFRESH_DELAY_MS = 1000;             // Delay after API post to ensure backend updated
+const DEBUG_STARTUP_BANNER = `
+╔═══════════════════════════════════════════════════════════╗
+║  MeshCore GOME WarDriver - Debug Console Logs Active     ║
+║  Open DevTools Console to monitor ping activity          ║
+║  Look for [Debug] and [Repeater Tracker] messages        ║
+╚═══════════════════════════════════════════════════════════╝
+`;
 const WARDROVE_KEY     = new Uint8Array([
   0x40, 0x76, 0xC3, 0x15, 0xC1, 0xEF, 0x38, 0x5F,
   0xA9, 0x3F, 0x06, 0x60, 0x27, 0x32, 0x0F, 0xE5
@@ -460,11 +467,19 @@ function buildPayload(lat, lon) {
   return `${PING_PREFIX} ${coordsStr} ${suffix}`;
 }
 
+// ---- Helper Functions ----
+function formatSessionLogLine(nowStr, lat, lon, status = '') {
+  const coords = `${lat.toFixed(5)} ${lon.toFixed(5)}`;
+  const statusStr = status ? ` ${status}` : '';
+  return `${nowStr}  ${coords}${statusStr}`;
+}
+
 // ---- Repeater Tracking ----
 // Tracks channel echoes (repeater responses) after sending a ping
 // Debug output is logged to browser console with [Repeater Tracker] prefix
 // Open DevTools console to see real-time echo detection and diagnose connectivity issues
 function startRepeaterTracking(sessionLi) {
+  console.log(`[Debug] startRepeaterTracking called, sessionLi:`, sessionLi);
   // Clean up any existing tracking
   stopRepeaterTracking();
 
@@ -475,6 +490,8 @@ function startRepeaterTracking(sessionLi) {
   };
 
   console.log(`[Repeater Tracker] Started listening for channel echoes (${REPEATER_LISTEN_MS}ms window)`);
+  console.log(`[Debug] Connection object exists:`, !!state.connection);
+  console.log(`[Debug] LogRxData push code constant:`, Constants.PushCodes.LogRxData);
 
   // Create listener for LogRxData events
   state.repeaterLogListener = (logData) => {
@@ -546,16 +563,22 @@ function startRepeaterTracking(sessionLi) {
 
   // Start listening for LogRxData events
   if (state.connection) {
+    console.log(`[Debug] Registering LogRxData event listener on connection`);
     state.connection.on(Constants.PushCodes.LogRxData, state.repeaterLogListener);
+    console.log(`[Debug] LogRxData event listener registered successfully`);
+  } else {
+    console.warn(`[Debug] Cannot register LogRxData listener: connection is null`);
   }
 
   // Schedule stop after REPEATER_LISTEN_MS
   state.repeaterListenTimer = setTimeout(() => {
+    console.log(`[Debug] Repeater tracking timeout reached after ${REPEATER_LISTEN_MS}ms`);
     stopRepeaterTracking();
   }, REPEATER_LISTEN_MS);
 }
 
 function stopRepeaterTracking() {
+  console.log(`[Debug] stopRepeaterTracking called`);
   // Stop the timer
   if (state.repeaterListenTimer) {
     clearTimeout(state.repeaterListenTimer);
@@ -564,11 +587,18 @@ function stopRepeaterTracking() {
 
   // Remove the event listener
   if (state.repeaterLogListener && state.connection) {
+    console.log(`[Debug] Removing LogRxData event listener`);
     state.connection.off(Constants.PushCodes.LogRxData, state.repeaterLogListener);
     state.repeaterLogListener = null;
   }
 
   // Update the session log entry with repeater data
+  console.log(`[Debug] Updating session log with repeater data:`, {
+    hasRepeaterData: !!state.repeaterData,
+    hasSessionLi: !!state.repeaterData?.sessionLi,
+    repeaterCount: state.repeaterData?.repeaters?.size || 0
+  });
+  
   if (state.repeaterData && state.repeaterData.sessionLi) {
     const repeaters = state.repeaterData.repeaters;
     if (repeaters.size > 0) {
@@ -581,11 +611,16 @@ function stopRepeaterTracking() {
 
       // Append to the existing text in the session log
       const currentText = state.repeaterData.sessionLi.textContent;
-      state.repeaterData.sessionLi.textContent = `${currentText}  [${repeaterList}]`;
+      const updatedText = `${currentText}  [${repeaterList}]`;
+      state.repeaterData.sessionLi.textContent = updatedText;
       console.log(`[Repeater Tracker] Stopped listening. Total ${repeaters.size} unique repeater(s) detected: [${repeaterList}]`);
+      console.log(`[Debug] Session log updated with repeater data:`, updatedText);
     } else {
       console.log(`[Repeater Tracker] Stopped listening. No channel echoes detected in ${REPEATER_LISTEN_MS}ms window`);
+      console.log(`[Debug] No repeaters to add to session log`);
     }
+  } else {
+    console.warn(`[Debug] Cannot update session log: repeaterData or sessionLi is null`);
   }
 
   // Clear repeater data
@@ -638,12 +673,14 @@ async function postToMeshMapperAPI(lat, lon) {
 
 // ---- Ping ----
 async function sendPing(manual = false) {
+  console.log(`[Debug] sendPing called: manual=${manual}, timestamp=${new Date().toISOString()}`);
   try {
     // Check cooldown only for manual pings
     if (manual && isInCooldown()) {
       const remainingMs = state.cooldownEndTime - Date.now();
       const remainingSec = Math.ceil(remainingMs / 1000);
       setStatus(`Please wait ${remainingSec}s before sending another ping`, "text-amber-300");
+      console.log(`[Debug] Ping blocked by cooldown: ${remainingSec}s remaining`);
       return;
     }
 
@@ -694,6 +731,7 @@ async function sendPing(manual = false) {
     }
 
     const payload = buildPayload(lat, lon);
+    console.log(`[Debug] Payload built: "${payload}"`);
 
     // Format timestamp as ISO 8601 without milliseconds: YYYY-MM-DDTHH:MM:SSZ
     const nowStr = new Date().toISOString().split('.')[0] + 'Z';
@@ -702,7 +740,8 @@ async function sendPing(manual = false) {
     // This ensures we capture echoes that arrive immediately after transmission
     let sessionLogLi = null;
     if (sessionPingsEl) {
-      const line = `${nowStr}  ${lat.toFixed(5)} ${lon.toFixed(5)}`;
+      const line = formatSessionLogLine(nowStr, lat, lon, '[sending...]');
+      console.log(`[Debug] Creating session log entry: "${line}"`);
       sessionLogLi = document.createElement('li');
       sessionLogLi.textContent = line;
       sessionPingsEl.appendChild(sessionLogLi);
@@ -711,11 +750,21 @@ async function sendPing(manual = false) {
 
       // Start tracking repeater echoes BEFORE sending the ping
       // This is critical: echoes can arrive within milliseconds of transmission
+      console.log(`[Debug] Starting repeater tracking BEFORE sending ping`);
       startRepeaterTracking(sessionLogLi);
     }
 
     const ch = await ensureChannel();
+    console.log(`[Debug] Sending ping to channel ${ch.channelIdx}: "${payload}"`);
     await state.connection.sendChannelTextMessage(ch.channelIdx, payload);
+    console.log(`[Debug] Ping sent successfully`);
+    
+    // Update session log to show ping was sent
+    if (sessionLogLi) {
+      const line = formatSessionLogLine(nowStr, lat, lon);
+      sessionLogLi.textContent = line;
+      console.log(`[Debug] Session log updated to show ping sent`);
+    }
 
     // Start cooldown period after successful ping
     startCooldown();
@@ -823,8 +872,10 @@ function scheduleNextAutoPing() {
 }
 
 function startAutoPing() {
+  console.log(`[Debug] startAutoPing called`);
   if (!state.connection) {
     alert("Connect to a MeshCore device first.");
+    console.log(`[Debug] startAutoPing aborted: no connection`);
     return;
   }
 
@@ -833,6 +884,7 @@ function startAutoPing() {
     const remainingMs = state.cooldownEndTime - Date.now();
     const remainingSec = Math.ceil(remainingMs / 1000);
     setStatus(`Please wait ${remainingSec}s before toggling auto mode`, "text-amber-300");
+    console.log(`[Debug] startAutoPing aborted: in cooldown (${remainingSec}s remaining)`);
     return;
   }
 
@@ -853,6 +905,7 @@ function startAutoPing() {
   acquireWakeLock().catch(console.error);
 
   // Send first ping immediately
+  console.log(`[Debug] startAutoPing: sending first ping immediately`);
   sendPing(false).catch(console.error);
 }
 
@@ -870,10 +923,12 @@ async function connect() {
     state.connection = conn;
 
     conn.on("connected", async () => {
+      console.log(`[Debug] Connection established successfully`);
       setStatus("Connected", "text-emerald-300");
       setConnectButton(true);
       connectBtn.disabled = false;
       const selfInfo = await conn.getSelfInfo();
+      console.log(`[Debug] Device info retrieved:`, selfInfo);
       deviceInfoEl.textContent = selfInfo?.name || "[No device]";
       updateAutoButton();
       try { await conn.syncDeviceTime?.(); } catch { /* optional */ }
@@ -965,6 +1020,10 @@ document.addEventListener("visibilitychange", async () => {
 
 // ---- Bind UI & init ----
 export async function onLoad() {
+  console.log(DEBUG_STARTUP_BANNER);
+  console.log(`[Debug] Application loaded at ${new Date().toISOString()}`);
+  console.log(`[Debug] Session pings element:`, sessionPingsEl);
+  
   setStatus("Disconnected", "text-red-300");
   enableControls(false);
   updateAutoButton();
@@ -981,8 +1040,12 @@ export async function onLoad() {
       setStatus(e.message || "Connection error", "text-red-300");
     }
   });
-  sendPingBtn.addEventListener("click", () => sendPing(true).catch(console.error));
+  sendPingBtn.addEventListener("click", () => {
+    console.log(`[Debug] Manual ping button clicked at ${new Date().toISOString()}`);
+    sendPing(true).catch(console.error);
+  });
   autoToggleBtn.addEventListener("click", () => {
+    console.log(`[Debug] Auto toggle button clicked, current state.running=${state.running}`);
     if (state.running) {
       stopAutoPing();
       setStatus("Auto mode stopped", "text-slate-300");