chore: improve proxy test todo content and revert skip removal

Author: jescalada

test/proxy.test.ts

@@ -2,12 +2,19 @@ import https from 'https';
 import { describe, it, beforeEach, afterEach, expect, vi } from 'vitest';
 import fs from 'fs';
 
-// jescalada: these tests are currently causing the following error
-// when running tests in the CI or for the first time locally:
-// Error: listen EADDRINUSE: address already in use :::8000
-// This is likely due to improper test isolation or cleanup in another test file
-// TODO: Find root cause of this error and fix it
-describe('Proxy Module TLS Certificate Loading', () => {
+/* 
+  jescalada: these tests are currently causing the following error
+  when running tests in the CI or for the first time locally:
+  Error: listen EADDRINUSE: address already in use :::8000
+
+  This is likely due to improper test isolation or cleanup in another test file
+  especially related to proxy.start() and proxy.stop() calls
+
+  Related: skipped tests in testProxyRoute.test.ts - these have a race condition
+  where either these or those tests fail depending on execution order
+  TODO: Find root cause of this error and fix it
+*/
+describe.skip('Proxy Module TLS Certificate Loading', () => {
   let proxyModule: any;
   let mockConfig: any;
   let mockHttpServer: any;