Refactor code structure for improved readability and maintainability

Author: PythonSmall-Q

CHANGELOG.md

@@ -19,6 +19,7 @@ Format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 - **Default profile storage path regression** — the default profile now uses the legacy low-version database path (`app_data/timelens.db`) instead of a separate `app_data/profiles/default/timelens.db` folder. On startup, any data already written to the v2.0.0 `profiles/default` folder is automatically merged into the legacy database; raw usage rows are inserted without their synthetic `id` to avoid accidental data loss, and `daily_app_usage` is rebuilt from the merged raw rows so today’s totals stay correct. Other conflicting rows are skipped (`INSERT OR IGNORE`). Encrypted default-profile databases are relocated to the legacy path before decryption so they continue to work.
 - **Dashboard period total label** — the overview card now shows “Week Total” / “Month Total” instead of “Today’s Total” when the period selector is set to week or month; the numeric value already matched the selected range.
 - **VS Code extension API network errors** — dashboard no longer logs `Failed to fetch` errors when the optional local VS Code extension API is unreachable; it falls back to empty stats and emits the unavailable event once.
+- **Dependency security updates** — updated `sharp` to `^0.35.3` and added `overrides` for `brace-expansion`, `minimatch`, and `js-yaml` in the root `package.json` to resolve high-severity advisories. Updated `postcss` to `^8.5.23`. In `vscode-extension`, updated `@vscode/vsce` to `^3.9.2` and added overrides for `brace-expansion`/`minimatch`, clearing all high-severity Dependabot alerts in that package. `react-router-dom` remains at `^6.30.4` because the only patched v7 versions currently available introduce a separate high-severity advisory; this will be revisited once a clean patched release exists.
 
 ---
 

browser-extension/api.js

@@ -55,11 +55,14 @@ async function getConnectionSettings() {
 export async function discoverApiBaseUrl() {
   const now = Date.now();
   if (discoveredApiBaseCache && discoveredApiBaseCache.expiresAt > now) {
+    log("Using cached API base URL:", discoveredApiBaseCache.value, "expires in", discoveredApiBaseCache.expiresAt - now, "ms");
     return discoveredApiBaseCache.value;
   }
 
+  log("Discovering API base URL...");
   const { manualPort, cacheMode, cacheSeconds } = await getConnectionSettings();
   const cacheMs = cacheMode === "startup" ? Number.MAX_SAFE_INTEGER : Math.max(0, cacheSeconds) * 1000;
+  log("Cache mode:", cacheMode, "cacheMs:", cacheMs);
 
   const portsToTry = [];
   const manualPortAllowed =
@@ -68,6 +71,8 @@ export async function discoverApiBaseUrl() {
     now > manualPortDisabledUntil &&
     manualPortFailureCount < MANUAL_PORT_FAILURE_THRESHOLD;
 
+  log("Manual port:", manualPort, "allowed:", manualPortAllowed, "failureCount:", manualPortFailureCount, "disabledUntil:", manualPortDisabledUntil);
+
   if (manualPortAllowed) {
     portsToTry.push(manualPort);
   }
@@ -77,6 +82,7 @@ export async function discoverApiBaseUrl() {
       portsToTry.push(port);
     }
   }
+  log("Will scan", portsToTry.length, "ports, first few:", portsToTry.slice(0, 5));
 
   let manualPortTried = false;
   for (const port of portsToTry) {
@@ -94,40 +100,50 @@ export async function discoverApiBaseUrl() {
       if (response.ok) {
         const data = await response.json();
         if (data && typeof data.version === "string") {
+          log("API found on port", port, "version:", data.version);
           if (port === manualPort) {
             manualPortFailureCount = 0;
             manualPortDisabledUntil = 0;
           }
           discoveredApiBaseCache = { value: baseUrl, expiresAt: now + cacheMs };
+          log("Cached API base URL:", baseUrl, "until:", now + cacheMs);
           return baseUrl;
         }
+        logWarn("Port", port, "responded but missing version field");
       }
-    } catch {
+    } catch (error) {
       // port not reachable — try next
+      log("Port", port, "unreachable:", error?.name || error?.message || error);
     }
   }
 
   if (manualPortTried) {
     manualPortFailureCount += 1;
+    logWarn("Manual port", manualPort, "failed; count:", manualPortFailureCount, "/", MANUAL_PORT_FAILURE_THRESHOLD);
     if (manualPortFailureCount >= MANUAL_PORT_FAILURE_THRESHOLD) {
-      // Temporarily ignore the manual port for 5 minutes so the fallback scan can work.
       manualPortDisabledUntil = now + 5 * 60 * 1000;
+      logWarn("Manual port temporarily disabled until", new Date(manualPortDisabledUntil).toISOString());
     }
   }
 
+  logWarn("No TimeLens API found after scanning", portsToTry.length, "ports");
   discoveredApiBaseCache = { value: "", expiresAt: now + FALLBACK_CACHE_MS };
   return null;
 }
 
 export function getApiBaseUrl() {
-  return discoveredApiBaseCache?.value || `http://127.0.0.1:${DEFAULT_API_PORT}`;
+  const url = discoveredApiBaseCache?.value || `http://127.0.0.1:${DEFAULT_API_PORT}`;
+  log("getApiBaseUrl returning:", url);
+  return url;
 }
 
 export function clearApiBaseUrlCache() {
+  log("Clearing API base URL cache");
   discoveredApiBaseCache = null;
 }
 
 export function resetManualPortFailureTracking() {
+  log("Resetting manual port failure tracking");
   manualPortFailureCount = 0;
   manualPortDisabledUntil = 0;
 }