Merge pull request #2592 from microsoft/dev/v-peq/removeNode10TodoComments

chore: remove misleading Node 10 TODO comments

Author: ConnorQi01

src/debugger/appWorker.ts

@@ -63,7 +63,8 @@ Object.defineProperty(process, "versions", {
     value: undefined
 });
 
-// TODO: Replace by url.fileURLToPath method when Node 10 LTS become deprecated
+// Polyfill for url.fileURLToPath - needed because this code runs in user's RN environment
+// where we cannot use import/require. Keep this implementation.
 function fileUrlToPath(url) {
   if (process.platform === 'win32') {
       return url.toString().replace('file:///', '');

src/debugger/forkedAppWorker.ts

@@ -203,7 +203,8 @@ export class ForkedAppWorker implements IDebuggeeWorker {
         return promise;
     }
 
-    // TODO: Replace by url.pathToFileURL method when Node 10 LTS become deprecated
+    // Simple and reliable implementation for converting file path to file:// URL
+    // Keep this instead of using url.pathToFileURL() to avoid URL encoding issues
     public pathToFileUrl(url: string): string {
         const filePrefix = process.platform === "win32" ? "file:///" : "file://";
         return filePrefix + url;